fix bug

Author: white-poto

tests/CrontabTest.php

@@ -11,7 +11,8 @@ class CrontabTest extends PHPUnit_Framework_TestCase
     public function testStart()
     {
         $log_file = "/tmp/test_start.log";
-        $logger = new \Monolog\Logger(new \Monolog\Handler\StreamHandler($log_file));
+        $logger = new \Monolog\Logger(\Jenner\Crontab\Crontab::NAME);
+        $logger->pushHandler(new \Monolog\Handler\StreamHandler($log_file));
         $mission = new \Jenner\Crontab\Mission("mission_test", "ls /", "* * * * *", $logger);
         $crontab = new \Jenner\Crontab\Crontab(null, array($mission));
 
@@ -25,8 +26,10 @@ public function testError()
     {
         $log_file = "/tmp/test_error.log";
         $err_file = "/tmp/test_error_err.log";
-        $out = new \Monolog\Logger(new \Monolog\Handler\StreamHandler($log_file));
-        $err = new \Monolog\Logger(new \Monolog\Handler\StreamHandler($err_file));
+        $out = new \Monolog\Logger(\Jenner\Crontab\Crontab::NAME);
+        $out->pushHandler(new \Monolog\Handler\StreamHandler($log_file));
+        $err = new \Monolog\Logger(\Jenner\Crontab\Crontab::NAME);
+        $err->pushHandler(new \Monolog\Handler\StreamHandler($err_file));
         $mission = new \Jenner\Crontab\Mission("mission_test", "ls / && command_not_exists", "* * * * *", $out, $err);
         $crontab = new \Jenner\Crontab\Crontab(null, array($mission));
 
@@ -42,7 +45,8 @@ public function testError()
     public function testNotStart()
     {
         $log_file = "/tmp/test_not_start.log";
-        $logger = new \Monolog\Logger(new \Monolog\Handler\StreamHandler($log_file));
+        $logger = new \Monolog\Logger(\Jenner\Crontab\Crontab::NAME);
+        $logger->pushHandler(new \Monolog\Handler\StreamHandler($log_file));
         $mission = new \Jenner\Crontab\Mission("mission_test", "ls /", "3 * * * *", $logger);
         $crontab = new \Jenner\Crontab\Crontab(null, array($mission));
 

tests/MissionTest.php

@@ -20,19 +20,19 @@ class MissionTest extends PHPUnit_Framework_TestCase
 
     public function setUp()
     {
-
-    }
-
-    public function testNeed()
-    {
-        $logger = new \Monolog\Logger(new \Monolog\Handler\StreamHandler($this->log_file));
+        $logger = new \Monolog\Logger(\Jenner\Crontab\Crontab::NAME);
+        $logger->pushHandler(new \Monolog\Handler\StreamHandler($this->log_file));
 
         $this->mission = new \Jenner\Crontab\Mission(
             "mission_test",
             "ls /",
             "* * * * *",
             $logger
         );
+    }
+
+    public function testNeed()
+    {
         $this->assertTrue($this->mission->needRun(time()));
         $this->assertTrue($this->mission->needRun(time() + 60));
         $this->assertTrue($this->mission->needRun(time() + 120));
@@ -41,19 +41,8 @@ public function testNeed()
 
     public function testRun()
     {
-        $stream = new \Monolog\Handler\StreamHandler($this->log_file);
-        $logger = new \Monolog\Logger($stream);
-
-        $this->mission = new \Jenner\Crontab\Mission(
-            "mission_test",
-            "ls /",
-            "* * * * *",
-            $logger
-        );
-
         $this->mission->start();
         $this->mission->wait();
-
         $this->assertEquals($this->mission->errno(), 0);
         $out = file_get_contents($this->log_file);
         $except = shell_exec("ls /");