Merge pull request #12 from NauxLiu/patch-1

Add exception class name to failed message
cronitorio · Jun 26, 2017 · 3862961 · 3862961
2 parents 40b4bdc + 88ade62
commit 3862961
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/src/monitor.php b/src/monitor.php
@@ -10,7 +10,7 @@ function cronitorMonitorTask($client, $closure, $exceptionHandler = false)
         $pause = false;
 
         if (!$exceptionHandler) {
-            $msg = $e->getMessage();
+            $msg = get_class($e) . " with message '" . $e->getMessage() . "'";
         } else {
             // $exceptionHandler should return an array like the following:
             // array(

diff --git a/tests/MonitorTaskTest.php b/tests/MonitorTaskTest.php
@@ -7,7 +7,6 @@
 
 class MonitorTaskTest extends TestBase
 {
-
     public function test_it_should_return_value_on_success()
     {
         $client = $this->getOkClient();
@@ -51,7 +50,7 @@ public function test_it_should_send_exceptions_message_on_failure()
 
         $client->expects($this->once())
             ->method('fail')
-            ->with($msg);
+            ->with("Exception with message '{$msg}'");
 
         cronitorMonitorTask(
             $client,