You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Use Illuminate\Events\Dispatcher instead of contract
Laravel dispatcher should be either Illuminate\Contracts\Events\Dispatcher
or Illuminate\Events\Dispatcher, however only the latter has fire()
method, which in turn just a wrapper for dispatch()
Using fire() inplace with contract tests are failing with exceptions:
1. Tmdb\Laravel\Adapters\Tests\EventDispatcherTestLaravel5::it_dispatches_events_through_both_laravel_and_symfony
Prophecy\Exception\Doubler\MethodNotFoundException: Method `Double\Dispatcher\P25::fire()` is not defined.
2. Tmdb\Laravel\Adapters\Tests\EventDispatcherTestLaravel5::it_returns_the_event_returned_by_the_symfony_dispatcher
Prophecy\Exception\Doubler\MethodNotFoundException: Method `Double\Dispatcher\P27::fire()` not found.
0 commit comments