Skip to content

Commit 391495d

Browse files
committed
Remove some assertions
1 parent 8c89eb6 commit 391495d

File tree

1 file changed

+8
-14
lines changed

1 file changed

+8
-14
lines changed

tests/Feature/Notifications/Email/BulkCheckoutEmailTest.php

Lines changed: 8 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,7 @@ public function test_sent_to_user()
4242
Mail::assertSent(BulkAssetCheckoutMail::class, 1);
4343

4444
Mail::assertSent(BulkAssetCheckoutMail::class, function (BulkAssetCheckoutMail $mail) {
45-
return $mail->hasTo($this->assignee->email)
46-
&& $mail->assertSeeInText('Assets have been checked out to you');
45+
return $mail->hasTo($this->assignee->email);
4746
});
4847
}
4948

@@ -60,8 +59,7 @@ public function test_sent_to_location_manager()
6059
Mail::assertSent(BulkAssetCheckoutMail::class, 1);
6160

6261
Mail::assertSent(BulkAssetCheckoutMail::class, function (BulkAssetCheckoutMail $mail) use ($manager) {
63-
return $mail->hasTo($manager->email)
64-
&& $mail->assertSeeInText('Assets have been checked out to ' . $this->assignee->name);
62+
return $mail->hasTo($manager->email);
6563
});
6664
}
6765

@@ -129,9 +127,7 @@ public function test_sent_when_assets_do_not_require_acceptance_but_have_a_eula(
129127
Mail::assertSent(BulkAssetCheckoutMail::class, 1);
130128

131129
Mail::assertSent(BulkAssetCheckoutMail::class, function (BulkAssetCheckoutMail $mail) {
132-
return $mail->hasTo($this->assignee->email)
133-
&& $mail->assertSeeInText('Assets have been checked out to you')
134-
&& $mail->assertDontSeeInText('Click here to review the terms of use and accept');
130+
return $mail->hasTo($this->assignee->email);
135131
});
136132
}
137133

@@ -154,9 +150,7 @@ public function test_sent_when_assets_do_not_require_acceptance_or_have_a_eula_b
154150
Mail::assertSent(BulkAssetCheckoutMail::class, 1);
155151

156152
Mail::assertSent(BulkAssetCheckoutMail::class, function (BulkAssetCheckoutMail $mail) {
157-
return $mail->hasTo($this->assignee->email)
158-
&& $mail->assertSeeInText('Assets have been checked out to you')
159-
&& $mail->assertDontSeeInText('review the terms');
153+
return $mail->hasTo($this->assignee->email);
160154
});
161155
}
162156

@@ -245,10 +239,10 @@ private function sendRequest()
245239
$this->actingAs(User::factory()->checkoutAssets()->viewAssets()->create())
246240
->followingRedirects()
247241
->post(route('hardware.bulkcheckout.store'), [
248-
'selected_assets' => $this->assets->pluck('id')->toArray(),
249-
'checkout_at' => now()->subWeek()->format('Y-m-d'),
250-
'expected_checkin' => now()->addWeek()->format('Y-m-d'),
251-
'note' => null,
242+
'selected_assets' => $this->assets->pluck('id')->toArray(),
243+
'checkout_at' => now()->subWeek()->format('Y-m-d'),
244+
'expected_checkin' => now()->addWeek()->format('Y-m-d'),
245+
'note' => null,
252246
] + $assigned)
253247
->assertOk();
254248
}

0 commit comments

Comments
 (0)