More tests.

Author: binaryk

src/Actions/SendMailAction.php

@@ -23,10 +23,6 @@ public function handle(MailatorSchedule $schedule)
     protected function sendMail(MailatorSchedule $schedule)
     {
         //todo - apply replacers for variables maybe
-        if (empty($schedule->getRecipients())) {
-            return;
-        }
-
         Mail::to($schedule->getRecipients())->send(
             $schedule->getMailable()
         );

tests/Feature/Models/MailatorScheduleTest.php

@@ -68,7 +68,7 @@ public function test_sending_email_only_once()
 
         MailatorSchedule::init('Invoice reminder.')
             ->recipients([
-                'foo@bar.com',
+                'zoo@bar.com',
             ])
             ->mailable(
                 (new InvoiceReminderMailable())->to('[email protected]')
@@ -82,5 +82,17 @@ public function test_sending_email_only_once()
         $_SERVER['can_send'] = true;
         MailatorSchedule::run();
         Mail::assertSent(InvoiceReminderMailable::class, 1);
+
+        $_SERVER['can_send'] = false;
+
+        MailatorSchedule::run();
+        Mail::assertSent(InvoiceReminderMailable::class, 1);
+
+        MailatorSchedule::run();
+        Mail::assertSent(InvoiceReminderMailable::class, 1);
+
+        Mail::assertSent(InvoiceReminderMailable::class, function (InvoiceReminderMailable  $mail) {
+            return $mail->hasTo('[email protected]') && $mail->hasTo('[email protected]');
+        });
     }
 }