added notification fake

akaunting · Mar 22, 2023 · a20b761 · a20b761
1 parent 31c5891
commit a20b761
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 6 deletions.
diff --git a/tests/Feature/Banking/TransactionsTest.php b/tests/Feature/Banking/TransactionsTest.php
@@ -8,6 +8,7 @@
 use App\Models\Banking\Transaction;
 use Illuminate\Http\UploadedFile;
 use Illuminate\Support\Facades\File;
+use Illuminate\Support\Facades\Notification as NotificationFacade;
 use Maatwebsite\Excel\Facades\Excel;
 use Tests\Feature\FeatureTestCase;
 
@@ -73,7 +74,7 @@ public function testItShouldCreateTransactionWithRecurring()
 
     public function testItShouldSendTransactionEmail()
     {
-        config(['mail.default' => 'array']);
+        NotificationFacade::fake();
 
         $transaction = $this->dispatch(new CreateTransaction($this->getRequest()));
 
@@ -82,15 +83,19 @@ public function testItShouldSendTransactionEmail()
             ->assertStatus(200);
 
         $this->assertFlashLevel('success');
+
+        NotificationFacade::assertSentTo($transaction->contact, Notification::class);
     }
 
     public function testItShouldHitRateLimitForSendTransactionEmail()
     {
-        config(['mail.default' => 'array']);
+        NotificationFacade::fake();
+
+        $limit_per_minute = (int) config('app.throttles.email.minute');
 
         $transaction = $this->dispatch(new CreateTransaction($this->getRequest()));
 
-        for ($i = 0; $i < config('app.throttles.email.minute'); $i++) {
+        for ($i = 0; $i < $limit_per_minute; $i++) {
             $this->loginAs()
                 ->post(route('modals.transactions.emails.store', $transaction->id), $this->getEmailRequest($transaction));
         }
@@ -100,6 +105,8 @@ public function testItShouldHitRateLimitForSendTransactionEmail()
             ->assertJson([
                 'success' => false,
             ]);
+
+        NotificationFacade::assertSentTimes(Notification::class, $limit_per_minute);
     }
 
 	public function testItShouldSeeTransactionUpdatePage()

diff --git a/tests/Feature/Sales/InvoicesTest.php b/tests/Feature/Sales/InvoicesTest.php
@@ -10,6 +10,7 @@
 use Illuminate\Support\Carbon;
 use Illuminate\Support\Facades\File;
 use Illuminate\Support\Facades\Storage;
+use Illuminate\Support\Facades\Notification as NotificationFacade;
 use Maatwebsite\Excel\Facades\Excel;
 use Tests\Feature\FeatureTestCase;
 
@@ -130,7 +131,7 @@ public function testItShouldCreateInvoiceWithRecurring()
 
     public function testItShouldSendInvoiceEmail()
     {
-        config(['mail.default' => 'array']);
+        NotificationFacade::fake();
 
         $invoice = $this->dispatch(new CreateDocument($this->getRequest()));
 
@@ -139,15 +140,19 @@ public function testItShouldSendInvoiceEmail()
             ->assertStatus(200);
 
         $this->assertFlashLevel('success');
+
+        NotificationFacade::assertSentTo($invoice->contact, Notification::class);
     }
 
     public function testItShouldHitRateLimitForSendInvoiceEmail()
     {
-        config(['mail.default' => 'array']);
+        NotificationFacade::fake();
+
+        $limit_per_minute = (int) config('app.throttles.email.minute');
 
         $invoice = $this->dispatch(new CreateDocument($this->getRequest()));
 
-        for ($i = 0; $i < config('app.throttles.email.minute'); $i++) {
+        for ($i = 0; $i < $limit_per_minute; $i++) {
             $this->loginAs()
                 ->post(route('modals.invoices.emails.store', $invoice->id), $this->getEmailRequest($invoice));
         }
@@ -157,6 +162,8 @@ public function testItShouldHitRateLimitForSendInvoiceEmail()
             ->assertJson([
                 'success' => false,
             ]);
+
+        NotificationFacade::assertSentTimes(Notification::class, $limit_per_minute);
     }
 
     public function testItShouldSeeInvoiceUpdatePage()