More tests.

Author: binaryk

tests/Feature/SchedulerDefaultConstraintsTest.php

@@ -107,7 +107,7 @@ public function test_can_send_serialized_constrained(): void
             )
             ->many()
             ->constraint(
-                new SerializedConditionCondition(new User([
+                new SerializedConditionCondition(User::create([
                     'email' => '[email protected]',
                 ]))
             );

tests/Feature/SerializationTest.php

@@ -0,0 +1,54 @@
+<?php
+
+namespace Binarcode\LaravelMailator\Tests\Feature;
+
+use Binarcode\LaravelMailator\Models\MailatorSchedule;
+use Binarcode\LaravelMailator\Tests\Fixtures\InvoiceReminderMailable;
+use Binarcode\LaravelMailator\Tests\Fixtures\Post;
+use Binarcode\LaravelMailator\Tests\Fixtures\SerializedConditionCondition;
+use Binarcode\LaravelMailator\Tests\Fixtures\User;
+use Binarcode\LaravelMailator\Tests\TestCase;
+use Illuminate\Database\Eloquent\Model;
+use Illuminate\Support\Facades\Mail;
+
+class SerializationTest extends TestCase
+{
+    public function test_constraints_gets_latest_database_values(): void
+    {
+        Mail::fake();
+        Mail::assertNothingSent();
+
+        $user = User::factory()->has(
+            Post::factory()->state([
+                'title' => 'Test title'
+            ]), 'posts'
+        )->create([
+            'email' => '[email protected]',
+        ]);
+
+        $scheduler = MailatorSchedule::init('Invoice reminder.')
+            ->recipients([
+                '[email protected]',
+            ])
+            ->mailable(
+                (new InvoiceReminderMailable())->to('[email protected]')
+            )
+            ->many()
+            ->constraint(
+                new SerializedConditionCondition($user)
+            );
+
+        $scheduler->save();
+
+        $user->update([
+            'email' => $newEmail = '[email protected]',
+        ]);
+
+        $_SERVER['fakeSerializedEmail'] = $newEmail;
+
+        MailatorSchedule::run();
+
+        Mail::assertSent(InvoiceReminderMailable::class, 1);
+    }
+
+}

tests/Fixtures/Post.php

@@ -0,0 +1,11 @@
+<?php
+
+namespace Binarcode\LaravelMailator\Tests\Fixtures;
+
+use Illuminate\Database\Eloquent\Factories\HasFactory;
+use Illuminate\Database\Eloquent\Model;
+
+class Post extends Model
+{
+    use HasFactory;
+}

tests/Fixtures/SerializedConditionCondition.php

@@ -4,10 +4,13 @@
 
 use Binarcode\LaravelMailator\Constraints\SendScheduleConstraint;
 use Binarcode\LaravelMailator\Models\MailatorSchedule;
+use Illuminate\Queue\SerializesModels;
 use Illuminate\Support\Collection;
 
 class SerializedConditionCondition implements SendScheduleConstraint
 {
+    use SerializesModels;
+
     public User $user;
 
     public function __construct(User $user)
@@ -17,6 +20,6 @@ public function __construct(User $user)
 
     public function canSend(MailatorSchedule $schedule, Collection $logs): bool
     {
-        return $this->user->email === '[email protected]';
+        return $this->user->email === data_get($_SERVER, 'fakeSerializedEmail', '[email protected]');
     }
 }

tests/Fixtures/User.php

@@ -6,6 +6,7 @@
 use Illuminate\Auth\MustVerifyEmail;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Relations\HasMany;
 
 class User extends Model
 {
@@ -15,4 +16,9 @@ class User extends Model
 
 
     protected $guarded = [];
+
+    public function posts(): HasMany
+    {
+        return $this->hasMany(Post::class);
+    }
 }

tests/database/Factories/PostFactory.php

@@ -0,0 +1,25 @@
+<?php
+
+namespace Binarcode\LaravelMailator\Tests\database\Factories;
+
+use Binarcode\LaravelMailator\Tests\Fixtures\Post;
+use Binarcode\LaravelMailator\Tests\Fixtures\User;
+use Illuminate\Database\Eloquent\Factories\Factory;
+
+class PostFactory extends Factory
+{
+    protected $model = Post::class;
+
+    public function definition()
+    {
+        return [
+            'title' => $this->faker->sentence(),
+            'user_id' => User::factory(),
+        ];
+    }
+
+    public static function one(array $attributes = []): Post
+    {
+        return app(static::class)->create($attributes);
+    }
+}

tests/database/migrations/2017_10_10_000000_create_posts_table.php

@@ -0,0 +1,33 @@
+<?php
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+return new class extends Migration
+{
+    /**
+     * Run the migrations.
+     *
+     * @return void
+     */
+    public function up()
+    {
+        Schema::create('posts', function (Blueprint $table) {
+            $table->increments('id');
+            $table->string('title')->nullable();
+            $table->foreignId('user_id')->constrained('users');
+            $table->timestamps();
+        });
+    }
+
+    /**
+     * Reverse the migrations.
+     *
+     * @return void
+     */
+    public function down()
+    {
+        Schema::dropIfExists('posts');
+    }
+};