Improve variable name

Author: marcusmoore

tests/Feature/Notifications/Email/BulkCheckoutEmailTest.php

@@ -18,7 +18,7 @@
 class BulkCheckoutEmailTest extends TestCase
 {
     private Collection $assets;
-    private Model $target;
+    private Model $assignee;
 
     protected function setUp(): void
     {
@@ -30,7 +30,7 @@ protected function setUp(): void
         $this->settings->disableAdminCCAlways();
 
         $this->assets = Asset::factory()->requiresAcceptance()->count(2)->create();
-        $this->target = User::factory()->create(['email' => '[email protected]']);
+        $this->assignee = User::factory()->create(['email' => '[email protected]']);
     }
 
     public function test_email_is_sent_to_user()
@@ -42,7 +42,7 @@ public function test_email_is_sent_to_user()
         Mail::assertSent(BulkAssetCheckoutMail::class, 1);
 
         Mail::assertSent(BulkAssetCheckoutMail::class, function (BulkAssetCheckoutMail $mail) {
-            return $mail->hasTo($this->target->email)
+            return $mail->hasTo($this->assignee->email)
                 && $mail->assertSeeInText('Assets have been checked out to you');
         });
     }
@@ -51,7 +51,7 @@ public function test_email_is_sent_to_location_manager()
     {
         $manager = User::factory()->create();
 
-        $this->target = Location::factory()->for($manager, 'manager')->create();
+        $this->assignee = Location::factory()->for($manager, 'manager')->create();
 
         $this->sendRequest();
 
@@ -61,15 +61,15 @@ public function test_email_is_sent_to_location_manager()
 
         Mail::assertSent(BulkAssetCheckoutMail::class, function (BulkAssetCheckoutMail $mail) use ($manager) {
             return $mail->hasTo($manager->email)
-                && $mail->assertSeeInText('Assets have been checked out to ' . $this->target->name);
+                && $mail->assertSeeInText('Assets have been checked out to ' . $this->assignee->name);
         });
     }
 
     public function test_email_is_sent_to_user_asset_is_checked_out_to()
     {
         $user = User::factory()->create();
 
-        $this->target = Asset::factory()->assignedToUser($user)->create();
+        $this->assignee = Asset::factory()->assignedToUser($user)->create();
 
         $this->sendRequest();
 
@@ -84,7 +84,7 @@ public function test_email_is_sent_to_user_asset_is_checked_out_to()
 
     public function test_email_is_not_sent_to_user_when_user_does_not_have_email_address()
     {
-        $this->target = User::factory()->create(['email' => null]);
+        $this->assignee = User::factory()->create(['email' => null]);
 
         $this->sendRequest();
 
@@ -129,7 +129,7 @@ public function test_email_is_sent_when_assets_do_not_require_acceptance_but_hav
         Mail::assertSent(BulkAssetCheckoutMail::class, 1);
 
         Mail::assertSent(BulkAssetCheckoutMail::class, function (BulkAssetCheckoutMail $mail) {
-            return $mail->hasTo($this->target->email)
+            return $mail->hasTo($this->assignee->email)
                 && $mail->assertSeeInText('Assets have been checked out to you')
                 && $mail->assertDontSeeInText('Click here to review the terms of use and accept');
         });
@@ -154,7 +154,7 @@ public function test_email_is_sent_when_assets_do_not_require_acceptance_or_have
         Mail::assertSent(BulkAssetCheckoutMail::class, 1);
 
         Mail::assertSent(BulkAssetCheckoutMail::class, function (BulkAssetCheckoutMail $mail) {
-            return $mail->hasTo($this->target->email)
+            return $mail->hasTo($this->assignee->email)
                 && $mail->assertSeeInText('Assets have been checked out to you')
                 && $mail->assertDontSeeInText('review the terms');
         });
@@ -171,7 +171,7 @@ public function test_email_is_sent_to_cc_address()
         Mail::assertSent(BulkAssetCheckoutMail::class, 2);
 
         Mail::assertSent(BulkAssetCheckoutMail::class, function (BulkAssetCheckoutMail $mail) {
-            return $mail->hasTo($this->target->email);
+            return $mail->hasTo($this->assignee->email);
         });
 
         Mail::assertSent(BulkAssetCheckoutMail::class, function (BulkAssetCheckoutMail $mail) {
@@ -226,18 +226,18 @@ public function test_email_is_not_sent_to_cc_address_if_assets_do_not_require_ac
 
     private function sendRequest()
     {
-        $assigned = match (get_class($this->target)) {
+        $assigned = match (get_class($this->assignee)) {
             User::class => [
                 'checkout_to_type' => 'user',
-                'assigned_user' => $this->target->id,
+                'assigned_user' => $this->assignee->id,
             ],
             Location::class => [
                 'checkout_to_type' => 'location',
-                'assigned_location' => $this->target->id,
+                'assigned_location' => $this->assignee->id,
             ],
             Asset::class => [
                 'checkout_to_type' => 'asset',
-                'assigned_asset' => $this->target->id,
+                'assigned_asset' => $this->assignee->id,
             ],
             default => [],
         };