Send email if asset has checkin_email set to true

Author: marcusmoore

app/Listeners/CheckoutablesCheckedOutInBulkListener.php

@@ -76,6 +76,10 @@ private function shouldSendCheckoutEmailToUser(?User $user, Collection $assets):
             return true;
         }
 
+        if ($this->hasAssetWithCategorySettingToSendEmail($assets)) {
+            return true;
+        }
+
         return $this->requiresAcceptance($assets);
     }
 
@@ -109,6 +113,17 @@ private function hasAssetWithEula(Collection $assets): bool
         return false;
     }
 
+    private function hasAssetWithCategorySettingToSendEmail(Collection $assets): bool
+    {
+        foreach ($assets as $asset) {
+            if ($asset->checkin_email()) {
+                return true;
+            }
+        }
+
+        return false;
+    }
+
     private function requiresAcceptance(Collection $assets): bool
     {
         return (bool) $assets->reduce(

database/factories/CategoryFactory.php

@@ -222,11 +222,26 @@ public function doesNotSendCheckinEmail()
         ]);
     }
 
+    public function sendsCheckinEmail()
+    {
+        return $this->state([
+            'checkin_email' => true,
+        ]);
+    }
+
     public function hasLocalEula()
     {
         return $this->state([
             'use_default_eula' => false,
             'eula_text' => 'Some EULA text here',
         ]);
     }
+
+    public function withNoLocalOrGlobalEula()
+    {
+        return $this->state([
+            'use_default_eula' => false,
+            'eula_text' => '',
+        ]);
+    }
 }

tests/Feature/Notifications/Email/BulkCheckoutEmailTest.php

@@ -132,9 +132,29 @@ public function test_email_is_sent_when_assets_do_not_require_acceptance_but_hav
         });
     }
 
-    public function test_email_is_sent_when_assets_do_not_require_acceptance_but_category_is_set_to_send_email()
+    public function test_email_is_sent_when_assets_do_not_require_acceptance_or_have_a_eula_but_category_is_set_to_send_email()
     {
-        $this->markTestIncomplete();
+        $this->assets = Asset::factory()->count(2)->create();
+
+        $category = Category::factory()
+            ->doesNotRequireAcceptance()
+            ->withNoLocalOrGlobalEula()
+            ->sendsCheckinEmail()
+            ->create();
+
+        $this->assets->each(fn($asset) => $asset->model->category()->associate($category)->save());
+
+        $this->sendRequest();
+
+        Mail::assertNotSent(CheckoutAssetMail::class);
+
+        Mail::assertSent(BulkAssetCheckoutMail::class, 1);
+
+        Mail::assertSent(BulkAssetCheckoutMail::class, function (BulkAssetCheckoutMail $mail) {
+            return $mail->hasTo($this->target->email)
+                && $mail->assertSeeInText('Assets have been checked out to you')
+                && $mail->assertDontSeeInText('review the terms');
+        });
     }
 
     public function test_email_is_sent_to_cc_address()