Fixed: AfterConstraint (minutes), added ShouldBeUnique to SendMailJob

src/Constraints/AfterConstraint.php

@@ -52,7 +52,7 @@ public function canSend(MailatorSchedule $schedule, Collection $logs): bool
             return false;
         }
 
-        $diff = (int) $schedule->timestamp_target->diffInHours(
+        $diff = (int) $schedule->timestamp_target->diffInMinutes(
             now()->floorSeconds(),
             absolute: true
         );

src/Jobs/SendMailJob.php

@@ -5,12 +5,13 @@
 use Binarcode\LaravelMailator\Models\MailatorSchedule;
 use Binarcode\LaravelMailator\Support\ClassResolver;
 use Illuminate\Bus\Queueable;
+use Illuminate\Contracts\Queue\ShouldBeUnique;
 use Illuminate\Contracts\Queue\ShouldQueue;
 use Illuminate\Foundation\Bus\Dispatchable;
 use Illuminate\Queue\InteractsWithQueue;
 use Illuminate\Queue\SerializesModels;
 
-class SendMailJob implements ShouldQueue
+class SendMailJob implements ShouldBeUnique, ShouldQueue
 {
     use ClassResolver;
     use Dispatchable;
@@ -32,6 +33,11 @@ public function __construct(MailatorSchedule $schedule)
         $this->queue = config('mailator.scheduler.send_mail_job_queue', 'default');
     }
 
+    public function uniqueId(): string
+    {
+        return 'mailator-schedule-'.$this->schedule->id;
+    }
+
     public function handle(): void
     {
         static::sendMailAction()->handle($this->schedule);

tests/Feature/AfterConstraintTest.php

@@ -94,4 +94,47 @@ public function test_past_target_with_after_now_passed_after_constraint_hourly_b
             $can
         );
     }
+
+    public function test_past_target_with_after_now_passed_after_constraint_minutes_bases()
+    {
+        Mail::fake();
+        Mail::assertNothingSent();
+
+        $scheduler = MailatorSchedule::init('reminder')
+            ->recipients('zoo@bar.com')
+            ->mailable(
+                (new InvoiceReminderMailable())->to('foo@bar.com')
+            )
+            ->minutes(10)
+            ->after(now());
+
+        $scheduler->save();
+
+        $this->travelTo(now()->addMinutes(5));
+
+        self::assertTrue(
+            $scheduler->fresh()->isFutureAction()
+        );
+
+        $this->travelTo(now()->addMinutes(5));
+
+        self::assertTrue(
+            app(AfterConstraint::class)
+                ->canSend(
+                    $scheduler,
+                    $scheduler->logs
+                )
+        );
+
+        $this->travelTo(now()->addMinutes(5));
+
+        // as long as we have passed the "after" minutes target this should return true
+        self::assertTrue(
+            app(AfterConstraint::class)
+                ->canSend(
+                    $scheduler,
+                    $scheduler->logs
+                )
+        );
+    }
 }