diff --git a/tests/Feature/Checkins/AssetCheckinTest.php b/tests/Feature/Checkins/AssetCheckinTest.php index f3518d11ba..76e1c7bffb 100644 --- a/tests/Feature/Checkins/AssetCheckinTest.php +++ b/tests/Feature/Checkins/AssetCheckinTest.php @@ -25,14 +25,12 @@ class AssetCheckinTest extends TestCase ->assertForbidden(); } - public function testCannotCheckInNonExistentAsset() - { - $this->markTestIncomplete(); - } - public function testCannotCheckInAssetThatIsNotCheckedOut() { - $this->markTestIncomplete(); + $this->actingAs(User::factory()->checkinAssets()->create()) + ->post(route('hardware.checkin.store', ['assetId' => Asset::factory()->create()->id])) + ->assertSessionHas('error') + ->assertRedirect(route('hardware.index')); } public function testAssetCanBeCheckedIn() @@ -72,24 +70,6 @@ class AssetCheckinTest extends TestCase $this->assertEquals($status->id, $asset->status_id); } - public function testCheckinTimeAndActionLogNoteCanBeSet() - { - Event::fake(); - - $this->actingAs(User::factory()->checkinAssets()->create()) - ->post(route( - 'hardware.checkin.store', - ['assetId' => Asset::factory()->assignedToUser()->create()->id] - ), [ - 'checkin_at' => '2023-01-02 12:45:56', - 'note' => 'hello' - ]); - - Event::assertDispatched(function (CheckoutableCheckedIn $event) { - return $event->action_date === '2023-01-02 12:45:56' && $event->note === 'hello'; - }, 1); - } - public function testLastCheckInFieldIsSetOnCheckin() { $admin = User::factory()->superuser()->create(); @@ -113,6 +93,24 @@ class AssetCheckinTest extends TestCase $this->markTestIncomplete(); } + public function testCheckinTimeAndActionLogNoteCanBeSet() + { + Event::fake(); + + $this->actingAs(User::factory()->checkinAssets()->create()) + ->post(route( + 'hardware.checkin.store', + ['assetId' => Asset::factory()->assignedToUser()->create()->id] + ), [ + 'checkin_at' => '2023-01-02 12:45:56', + 'note' => 'hello' + ]); + + Event::assertDispatched(function (CheckoutableCheckedIn $event) { + return $event->action_date === '2023-01-02 12:45:56' && $event->note === 'hello'; + }, 1); + } + public function testCheckInEmailSentToUserIfSettingEnabled() { Notification::fake();