Fixed bad dependency assumptions

src/Illuminate/Console/Scheduling/Schedule.php

@@ -5,9 +5,12 @@
 use DateTimeInterface;
 use Illuminate\Console\Application;
 use Illuminate\Container\Container;
+use Illuminate\Contracts\Bus\Dispatcher;
+use Illuminate\Contracts\Container\BindingResolutionException;
 use Illuminate\Contracts\Queue\ShouldQueue;
 use Illuminate\Support\ProcessUtils;
 use Illuminate\Support\Traits\Macroable;
+use RuntimeException;
 
 class Schedule
 {
@@ -34,6 +37,13 @@ class Schedule
      */
     protected $schedulingMutex;
 
+    /**
+     * The job dispatcher implementation.
+     *
+     * @var \Illuminate\Contracts\Bus\Dispatcher
+     */
+    protected $dispatcher;
+
     /**
      * The timezone the date should be evaluated on.
      *
@@ -51,6 +61,12 @@ public function __construct($timezone = null)
     {
         $this->timezone = $timezone;
 
+        if (! class_exists(Container::class)) {
+            throw new RuntimeException(
+                'The container implementation is required to use the scheduler. Please install illuminate/container.'
+            );
+        }
+
         $container = Container::getInstance();
 
         $this->eventMutex = $container->bound(EventMutex::class)
@@ -110,11 +126,13 @@ public function job($job, $queue = null, $connection = null)
             $job = is_string($job) ? resolve($job) : $job;
 
             if ($job instanceof ShouldQueue) {
-                dispatch($job)
-                    ->onConnection($connection ?? $job->connection)
-                    ->onQueue($queue ?? $job->queue);
+                $pending = $this->getDispatcher()->dispatch($job);
+                if ($pending !== null) {
+                    $pending->onConnection($connection ?? $job->connection)
+                            ->onQueue($queue ?? $job->queue);
+                }
             } else {
-                dispatch_now($job);
+                $this->getDispatcher()->dispatchNow($job);
             }
         })->name(is_string($job) ? $job : get_class($job));
     }
@@ -209,4 +227,26 @@ public function useCache($store)
 
         return $this;
     }
+
+    /**
+     * Get the job dispatcher, if available.
+     *
+     * @return \Illuminate\Contracts\Bus\Dispatcher
+     */
+    protected function getDispatcher()
+    {
+        if ($this->dispatcher === null) {
+            try {
+                $this->dispatcher = Container::getInstance()->make(Dispatcher::class);
+            } catch (BindingResolutionException $e) {
+                throw new RuntimeException(
+                    'Unable to resolve the dispatcher from the service container. Please bind it or install illuminate/bus.',
+                    $e->getCode(),
+                    $e
+                );
+            }
+        }
+
+        return $this->dispatcher;
+    }
 }

src/Illuminate/Console/composer.json

@@ -33,6 +33,8 @@
     "suggest": {
         "dragonmantank/cron-expression": "Required to use scheduling component (^2.0).",
         "guzzlehttp/guzzle": "Required to use the ping methods on schedules (^6.0|^7.0).",
+        "illuminate/bus": "Required to use the scheduled job dispatcher (^6.0)",
+        "illuminate/container": "Required to use the scheduler (^6.0)",
         "illuminate/filesystem": "Required to use the generator command (^6.0)"
     },
     "config": {

tests/Integration/Console/JobSchedulingTest.php

@@ -3,6 +3,7 @@
 namespace Illuminate\Tests\Integration\Console;
 
 use Illuminate\Bus\Queueable;
+use Illuminate\Container\Container;
 use Illuminate\Console\Scheduling\Schedule;
 use Illuminate\Contracts\Queue\ShouldQueue;
 use Illuminate\Queue\InteractsWithQueue;
@@ -14,6 +15,7 @@ class JobSchedulingTest extends TestCase
     public function testJobQueuingRespectsJobQueue()
     {
         Queue::fake();
+        Container::setInstance($this->app);
 
         /** @var Schedule $scheduler */
         $scheduler = $this->app->make(Schedule::class);
@@ -42,6 +44,7 @@ public function testJobQueuingRespectsJobQueue()
     public function testJobQueuingRespectsJobConnection()
     {
         Queue::fake();
+        Container::setInstance($this->app);
 
         /** @var Schedule $scheduler */
         $scheduler = $this->app->make(Schedule::class);