Refactor to remove duplicate StreamLogger test code

This commit is contained in:
Justin Hileman
2014-03-25 19:22:34 -07:00
parent 402f5013dc
commit c10c966e60
+12 -9
View File
@@ -14,23 +14,26 @@
*/ */
class Mustache_Test_Logger_StreamLoggerTest extends PHPUnit_Framework_TestCase class Mustache_Test_Logger_StreamLoggerTest extends PHPUnit_Framework_TestCase
{ {
public function testAcceptsFilename() /**
* @dataProvider acceptsStreamData
*/
public function testAcceptsStream($name, $stream)
{ {
$name = tempnam(sys_get_temp_dir(), 'mustache-test'); $logger = new Mustache_Logger_StreamLogger($stream);
$logger = new Mustache_Logger_StreamLogger($name);
$logger->log(Mustache_Logger::CRITICAL, 'message'); $logger->log(Mustache_Logger::CRITICAL, 'message');
$this->assertEquals("CRITICAL: message\n", file_get_contents($name)); $this->assertEquals("CRITICAL: message\n", file_get_contents($name));
} }
public function testAcceptsResource() public function acceptsStreamData()
{ {
$name = tempnam(sys_get_temp_dir(), 'mustache-test'); $one = tempnam(sys_get_temp_dir(), 'mustache-test');
$file = fopen($name, 'a'); $two = tempnam(sys_get_temp_dir(), 'mustache-test');
$logger = new Mustache_Logger_StreamLogger($file);
$logger->log(Mustache_Logger::CRITICAL, 'message');
$this->assertEquals("CRITICAL: message\n", file_get_contents($name)); return array(
array($one, $one),
array($two, fopen($two, 'a')),
);
} }
/** /**