diff --git a/tests/Feature/Notifications/Email/ExpiringAlertsNotificationTest.php b/tests/Feature/Notifications/Email/ExpiringAlertsNotificationTest.php index 57ad2cbcc3..5bc034ba5a 100644 --- a/tests/Feature/Notifications/Email/ExpiringAlertsNotificationTest.php +++ b/tests/Feature/Notifications/Email/ExpiringAlertsNotificationTest.php @@ -104,6 +104,11 @@ class ExpiringAlertsNotificationTest extends TestCase 'deleted_at' => null, ]); + $overDueForAuditableAsset = Asset::factory()->create([ + 'next_audit_date' => now()->subDays(1)->format('Y-m-d'), + 'deleted_at' => null, + ]); + $notAuditableAsset = Asset::factory()->create([ 'next_audit_date' => now()->addDays(30)->format('Y-m-d'), 'deleted_at' => null, @@ -111,8 +116,8 @@ class ExpiringAlertsNotificationTest extends TestCase $this->artisan('snipeit:upcoming-audits')->assertExitCode(0); - Mail::assertSent(SendUpcomingAuditMail::class, function($mail) use ($alert_email, $upcomingAuditableAsset) { - return $mail->hasTo($alert_email) && $mail->assets->contains($upcomingAuditableAsset); + Mail::assertSent(SendUpcomingAuditMail::class, function($mail) use ($alert_email, $upcomingAuditableAsset, $overDueForAuditableAsset) { + return $mail->hasTo($alert_email) && ($mail->assets->contains($upcomingAuditableAsset) && $mail->assets->contains($overDueForAuditableAsset)); }); Mail::assertNotSent(SendUpcomingAuditMail::class, function($mail) use ($alert_email, $notAuditableAsset) { return $mail->hasTo($alert_email) && $mail->assets->contains($notAuditableAsset);