#89 - fix wrong admin invitation receiver

blumilksoftware · May 11, 2022 · 56b5dfd · 56b5dfd
1 parent d6cd834
commit 56b5dfd
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/src/Services/InvitationsService.php b/src/Services/InvitationsService.php
@@ -6,11 +6,12 @@
 
 use Blumilk\Meetup\Core\Models\User;
 use Blumilk\Meetup\Core\Notifications\InvitationEmailNotification;
+use Illuminate\Support\Facades\Notification;
 
 class InvitationsService
 {
     public function sendInvitation(User $senderUser, string $email): void
     {
-        $senderUser->notify(new InvitationEmailNotification($senderUser, $email));
+        Notification::route("mail", $email)->notify(new InvitationEmailNotification($senderUser, $email));
     }
 }
diff --git a/tests/Feature/InviteAdminTest.php b/tests/Feature/InviteAdminTest.php
@@ -36,17 +36,18 @@ public function testAdminCanSendInvitation(): void
     {
         Notification::fake();
 
-        $invitedUser = User::factory([
-            "email" => "[email protected]",
-        ])->make();
-
         $this->actingAs($this->admin)
             ->post("/invitation", [
-                "email" => $invitedUser->email,
+                "email" => "[email protected]"
             ])
             ->assertSessionHasNoErrors();
 
-        Notification::assertSentTo($invitedUser, InvitationEmailNotification::class);
+        Notification::assertSentOnDemand(
+            InvitationEmailNotification::class,
+            function ($notification, $channels, $notifiable) {
+                return $notifiable->routes['mail'] === '[email protected]';
+            }
+        );
     }
 
     public function testUserCannotSendInvitation(): void