Add exception storing on failed jobs

src/GO/FailedJob.php

@@ -0,0 +1,32 @@
+<?php namespace GO;
+
+use Exception;
+
+class FailedJob
+{
+    /**
+     * @var Job
+     */
+    private $job;
+
+    /**
+     * @var Exception
+     */
+    private $exception;
+
+    public function __construct(Job $job, Exception $exception)
+    {
+        $this->job = $job;
+        $this->exception = $exception;
+    }
+
+    public function getJob(): Job
+    {
+        return $this->job;
+    }
+
+    public function getException(): Exception
+    {
+        return $this->exception;
+    }
+}

src/GO/Scheduler.php

@@ -23,7 +23,7 @@ class Scheduler
     /**
      * Failed jobs.
      *
-     * @var array
+     * @var FailedJob[]
      */
     private $failedJobs = [];
 
@@ -256,7 +256,7 @@ public function getExecutedJobs()
      */
     private function pushFailedJob(Job $job, Exception $e)
     {
-        $this->failedJobs[] = $job;
+        $this->failedJobs[] = new FailedJob($job, $e);
 
         $compiled = $job->compile();
 
@@ -273,7 +273,7 @@ private function pushFailedJob(Job $job, Exception $e)
     /**
      * Get the failed jobs.
      *
-     * @return array
+     * @return FailedJob[]
      */
     public function getFailedJobs()
     {

tests/GO/SchedulerTest.php

@@ -2,6 +2,7 @@
 
 use GO\Job;
 use DateTime;
+use GO\FailedJob;
 use GO\Scheduler;
 use PHPUnit\Framework\TestCase;
 
@@ -82,7 +83,7 @@ public function testShouldMarkJobAsFailedIfScriptPathIsInvalid()
         $scheduler->run();
         $fail = $scheduler->getFailedJobs();
         $this->assertCount(1, $fail);
-        $this->assertContainsOnlyInstancesOf(Job::class, $fail);
+        $this->assertContainsOnlyInstancesOf(FailedJob::class, $fail);
     }
 
     public function testShouldQueueAShellCommand()
@@ -152,8 +153,9 @@ public function testShouldKeepTrackOfFailedJobs()
     {
         $scheduler = new Scheduler();
 
-        $scheduler->call(function () {
-            throw new \Exception('Something failed');
+        $exception = new \Exception('Something failed');
+        $scheduler->call(function () use ($exception) {
+            throw $exception;
         });
 
         $this->assertEquals(count($scheduler->getFailedJobs()), 0);
@@ -162,6 +164,10 @@ public function testShouldKeepTrackOfFailedJobs()
 
         $this->assertEquals(count($scheduler->getExecutedJobs()), 0);
         $this->assertEquals(count($scheduler->getFailedJobs()), 1);
+        $failedJob = $scheduler->getFailedJobs()[0];
+        $this->assertInstanceOf(FailedJob::class, $failedJob);
+        $this->assertSame($exception, $failedJob->getException());
+        $this->assertInstanceOf(Job::class, $failedJob->getJob());
     }
 
     public function testShouldKeepExecutingJobsIfOneFails()