Adds ScheduleChecker class and makes use of it in BackgroundJob::shouldRun()

Author: garethellis36

src/BackgroundJob.php

@@ -198,13 +198,8 @@ protected function shouldRun()
             }
         }
 
-        $schedule = \DateTime::createFromFormat('Y-m-d H:i:s', $this->config['schedule']);
-        if ($schedule !== false) {
-            return $schedule->format('Y-m-d H:i') == (date('Y-m-d H:i'));
-        }
-
-        $cron = CronExpression::factory($this->config['schedule']);
-        if (!$cron->isDue()) {
+        $scheduleChecker = new ScheduleChecker();
+        if (!$scheduleChecker->isDue($this->config['schedule'])) {
             return false;
         }
 

src/ScheduleChecker.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Jobby;
+
+use Cron\CronExpression;
+
+class ScheduleChecker
+{
+    /**
+     * @param string $schedule
+     * @return bool
+     */
+    public function isDue($schedule)
+    {
+        $dateTime = \DateTime::createFromFormat('Y-m-d H:i:s', $schedule);
+        if ($dateTime !== false) {
+            return $dateTime->format('Y-m-d H:i') == (date('Y-m-d H:i'));
+        }
+
+        return CronExpression::factory($schedule)->isDue();
+    }
+}

tests/ScheduleCheckerTest.php

@@ -0,0 +1,57 @@
+<?php
+
+namespace Jobby\Tests;
+
+use Jobby\ScheduleChecker;
+use PHPUnit_Framework_TestCase;
+
+class ScheduleCheckerTest extends PHPUnit_Framework_TestCase
+{
+    /**
+     * @var ScheduleChecker
+     */
+    private $scheduleChecker;
+
+    /**
+     * @return void
+     */
+    protected function setUp()
+    {
+        parent::setUp();
+
+        $this->scheduleChecker = new ScheduleChecker();
+    }
+
+    /**
+     * @return void
+     */
+    public function test_it_can_detect_a_due_job_from_a_datetime_string()
+    {
+        $this->assertTrue($this->scheduleChecker->isDue(date('Y-m-d H:i:s')));
+    }
+
+    /**
+     * @return void
+     */
+    public function test_it_can_detect_a_non_due_job_from_a_datetime_string()
+    {
+        $this->assertFalse($this->scheduleChecker->isDue(date('Y-m-d H:i:s', strtotime('tomorrow'))));
+    }
+
+    /**
+     * @return void
+     */
+    public function test_it_can_detect_a_due_job_from_a_cron_expression()
+    {
+        $this->assertTrue($this->scheduleChecker->isDue("* * * * *"));
+    }
+
+    /**
+     * @return void
+     */
+    public function test_it_can_detect_a_non_due_job_from_a_cron_expression()
+    {
+        $hour = date("H", strtotime('+1 hour'));
+        $this->assertFalse($this->scheduleChecker->isDue("* {$hour} * * *"));
+    }
+}