printing the summary is now the responsibility of the driver

src/Indatus/Dispatcher/Drivers/Cron/ScheduleService.php

@@ -27,4 +27,37 @@ public function isDue(ScheduledCommand $command)
         $cron = CronExpression::factory($command->schedule($scheduler)->getSchedule());
         return $cron->isDue();
     }
+
+    /**
+     * @inheritDoc
+     */
+    public function printSummary()
+    {
+        $this->table->setHeaders(array('Environment(s)', 'Name', 'Minute', 'Hour', 'Day of Month', 'Month', 'Day of Week', 'Run as'));
+        /** @var $command \Indatus\Dispatcher\ScheduledCommand */
+        $commands = 0;
+        $activeCommands = 0;
+        foreach ($this->getScheduledCommands() as $command) {
+            /** @var $command \Indatus\Dispatcher\ScheduledCommand */
+            $scheduler = $command->schedule(App::make('Indatus\Dispatcher\Schedulable'));
+
+            $this->table->addRow(array(
+                    is_array($command->environment()) ? implode(',', $command->environment()) : $command->environment(),
+                    $command->getName(),
+                    $scheduler->getScheduleMinute(),
+                    $scheduler->getScheduleHour(),
+                    $scheduler->getScheduleDayOfMonth(),
+                    $scheduler->getScheduleMonth(),
+                    $scheduler->getScheduleDayOfWeek(),
+                    $command->user()
+                ));
+            $commands++;
+            $activeCommands++;
+        }
+
+        //sort by first column
+        $this->table->sort(0);
+
+        $this->table->display();
+    }
 } 

src/Indatus/Dispatcher/Drivers/Cron/Scheduler.php

@@ -16,7 +16,7 @@ class Scheduler implements Schedulable
 {
 
     /**
-     * @var string Any of the contextual timeframe
+     * @var string Any of the contextual time frame
      */
     const ANY = '*';
 

src/Indatus/Dispatcher/Services/ScheduleService.php

@@ -68,37 +68,8 @@ public function getDueCommands()
 
     /**
      * Review scheduled commands schedule, active status, etc.
-     * @todo refactor this... it's ugly.  The output goes directly to STDOUT
      * @return void
      */
-    public function printSummary()
-    {
-        $this->table->setHeaders(array('Environment(s)', 'Name', 'Minute', 'Hour', 'Day of Month', 'Month', 'Day of Week', 'Run as'));
-        /** @var $command \Indatus\Dispatcher\ScheduledCommand */
-        $commands = 0;
-        $activeCommands = 0;
-        foreach ($this->getScheduledCommands() as $command) {
-            /** @var $command \Indatus\Dispatcher\ScheduledCommand */
-            $scheduler = $command->schedule(App::make('Indatus\Dispatcher\Schedulable'));
-
-            $this->table->addRow(array(
-                    is_array($command->environment()) ? implode(',', $command->environment()) : $command->environment(),
-                    $command->getName(),
-                    $scheduler->getScheduleMinute(),
-                    $scheduler->getScheduleHour(),
-                    $scheduler->getScheduleDayOfMonth(),
-                    $scheduler->getScheduleMonth(),
-                    $scheduler->getScheduleDayOfWeek(),
-                    $command->user()
-                ));
-            $commands++;
-            $activeCommands++;
-        }
-
-        //sort by first column
-        $this->table->sort(0);
-
-        $this->table->display();
-    }
+    abstract public function printSummary();
 
 }

tests/Drivers/Cron/TestCronScheduleService.php

@@ -0,0 +1,55 @@
+<?php
+/**
+ * @author Ben Kuhl <[email protected]>
+ */
+
+use Mockery as m;
+
+class TestCronScheduleService extends TestCase
+{
+    public function setUp()
+    {
+        parent::setUp();
+    }
+
+    public function tearDown()
+    {
+        parent::tearDown();
+        m::close();
+    }
+
+    /**
+     * Test that a summary is properly generated
+     * Dangit this is ugly... gotta find a new cli library
+     */
+    public function testPrintSummary()
+    {
+        $table = m::mock('Indatus\Dispatcher\Table', function ($m) {
+                $m->shouldReceive('setHeaders')->once();
+                $m->shouldReceive('sort')->once();
+                $m->shouldReceive('display')->once();
+            });
+        $scheduledCommand = m::mock('Indatus\Dispatcher\ScheduledCommand', function ($m) use ($table) {
+                $table->shouldReceive('addRow')->once();
+
+                $m->shouldReceive('getName')->once();
+                $m->shouldReceive('user')->once();
+                $m->shouldReceive('environment')->twice();
+                $m->shouldReceive('schedule')->once()->andReturn(m::mock('Indatus\Dispatcher\Drivers\Cron\Scheduler', function ($m) {
+                            $m->shouldReceive('getScheduleMinute');
+                            $m->shouldReceive('getScheduleHour');
+                            $m->shouldReceive('getScheduleDayOfMonth');
+                            $m->shouldReceive('getScheduleMonth');
+                            $m->shouldReceive('getScheduleDayOfWeek');
+                        }));
+            });
+        $scheduleService = m::mock('Indatus\Dispatcher\Drivers\Cron\ScheduleService[getScheduledCommands]', array(
+                $table
+            ), function ($m) use ($scheduledCommand) {
+                $m->shouldReceive('getScheduledCommands')->once()->andReturn(array(
+                        $scheduledCommand
+                    ));
+            });
+        $scheduleService->printSummary();
+    }
+} 

tests/Drivers/Cron/TestScheduler.php → tests/Drivers/Cron/TestCronScheduler.php

@@ -5,7 +5,7 @@
 
 use \Indatus\Dispatcher\Drivers\Cron\Scheduler;
 
-class TestScheduler extends TestCase
+class TestCronScheduler extends TestCase
 {
     /**
      * @var Indatus\Dispatcher\Scheduler
@@ -16,6 +16,7 @@ class TestScheduler extends TestCase
 
     public function setUp()
     {
+        parent::setUp();
         $this->scheduler = new Scheduler();
     }
 

tests/Services/TestScheduleService.php

@@ -5,8 +5,8 @@
 
 use Mockery as m;
 use Indatus\Dispatcher\Drivers\Cron\ScheduleService;
-use Indatus\Dispatcher\Table;
 use Indatus\Dispatcher\Scheduler;
+use Indatus\Dispatcher\Table;
 
 class TestScheduleService extends TestCase
 {
@@ -39,41 +39,6 @@ public function testGetScheduledCommands()
         $this->assertSameSize($scheduledCommands, $this->scheduleService->getScheduledCommands());
     }
 
-    /**
-     * Test that a summary is properly generated
-     * Dangit this is ugly... gotta find a new cli library
-     */
-    public function testPrintSummary()
-    {
-        $table = m::mock('Indatus\Dispatcher\Table', function ($m) {
-                $m->shouldReceive('setHeaders')->once();
-                $m->shouldReceive('sort')->once();
-                $m->shouldReceive('display')->once();
-            });
-        $scheduledCommand = m::mock('Indatus\Dispatcher\ScheduledCommand', function ($m) use ($table) {
-                $table->shouldReceive('addRow')->once();
-
-                $m->shouldReceive('getName')->once();
-                $m->shouldReceive('user')->once();
-                $m->shouldReceive('environment')->twice();
-                $m->shouldReceive('schedule')->once()->andReturn(m::mock('Indatus\Dispatcher\Drivers\Cron\Scheduler', function ($m) {
-                            $m->shouldReceive('getScheduleMinute');
-                            $m->shouldReceive('getScheduleHour');
-                            $m->shouldReceive('getScheduleDayOfMonth');
-                            $m->shouldReceive('getScheduleMonth');
-                            $m->shouldReceive('getScheduleDayOfWeek');
-                        }));
-            });
-        $scheduleService = m::mock('Indatus\Dispatcher\Drivers\Cron\ScheduleService[getScheduledCommands]', array(
-                $table
-            ), function ($m) use ($scheduledCommand) {
-                $m->shouldReceive('getScheduledCommands')->once()->andReturn(array(
-                        $scheduledCommand
-                    ));
-            });
-        $scheduleService->printSummary();
-    }
-
     public function testGetDueCommands()
     {
         $scheduleService = m::mock('Indatus\Dispatcher\Drivers\Cron\ScheduleService[getScheduledCommands,isDue]', array(