Ensure missing user does not throw a type error (#366)

Author: timacdonald

src/PulseServiceProvider.php

@@ -119,6 +119,10 @@ protected function listenForEvents(): void
         $this->app->booted(function () {
             $this->callAfterResolving(Dispatcher::class, function (Dispatcher $event, Application $app) {
                 $event->listen(function (Logout $event) use ($app) {
+                    if ($event->user === null) {
+                        return;
+                    }
+
                     $pulse = $app->make(Pulse::class);
 
                     $pulse->rescue(fn () => $pulse->rememberUser($event->user));

tests/Feature/PulseTest.php

@@ -1,10 +1,12 @@
 <?php
 
+use Illuminate\Auth\Events\Logout;
 use Illuminate\Contracts\Auth\Authenticatable;
 use Illuminate\Support\Collection;
 use Illuminate\Support\Facades\App;
 use Illuminate\Support\Facades\Auth;
 use Illuminate\Support\Facades\Config;
+use Illuminate\Support\Facades\Event;
 use Illuminate\Support\Facades\Facade;
 use Laravel\Pulse\Contracts\ResolvesUsers;
 use Laravel\Pulse\Contracts\Storage;
@@ -246,6 +248,13 @@ public function find(int|string|null $key): object
     expect($persistentMiddleware)->not->toContain(MyTestMiddleware::class.':admin');
 });
 
+it('handles logout events when there is no user', function () {
+    // This will throw a type error when unhandled...
+    Event::dispatch(new Logout('session', user: null));
+
+    expect(true)->toBe(true);
+});
+
 class MyTestMiddleware
 {
     //