DRCIT/snipe-it
1
0
Fork 0
mirror of https://github.com/snipe/snipe-it.git synced 2025-01-12 14:27:33 -08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Reference accessory checkout and not the accessory

Browse source
This commit is contained in:
Marcus Moore 2024-10-01 13:35:39 -07:00
parent 5b90d79494
commit 3dc64cc5e0
No known key found for this signature in database
1 changed files with 9 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
tests/Feature/Checkins/Api/AccessoryCheckinTest.php
View file

@ -14,9 +14,10 @@ class AccessoryCheckinTest extends TestCase implements TestsFullMultipleCompanie
public function testRequiresPermission() public function testRequiresPermission()
{ {
$accessory = Accessory::factory()->checkedOutToUser()->create(); $accessory = Accessory::factory()->checkedOutToUser()->create();
$accessoryCheckout = $accessory->checkouts->first();
$this->actingAsForApi(User::factory()->create()) $this->actingAsForApi(User::factory()->create())
->postJson(route('api.accessories.checkin', $accessory)) ->postJson(route('api.accessories.checkin', $accessoryCheckout))
->assertForbidden(); ->assertForbidden();
} }
@ -35,11 +36,11 @@ class AccessoryCheckinTest extends TestCase implements TestsFullMultipleCompanie
$this->settings->enableMultipleFullCompanySupport(); $this->settings->enableMultipleFullCompanySupport();
$this->actingAsForApi($userInCompanyA) $this->actingAsForApi($userInCompanyA)
->postJson(route('api.accessories.checkin', $accessoryForCompanyB)) ->postJson(route('api.accessories.checkin', $accessoryForCompanyB->checkouts->first()))
->assertForbidden(); ->assertForbidden();
$this->actingAsForApi($superUser) $this->actingAsForApi($superUser)
->postJson(route('api.accessories.checkin', $anotherAccessoryForCompanyB)) ->postJson(route('api.accessories.checkin', $anotherAccessoryForCompanyB->checkouts->first()))
->assertStatusMessageIs('success'); ->assertStatusMessageIs('success');
$this->assertEquals(1, $accessoryForCompanyB->fresh()->checkouts->count(), 'Accessory should not be checked in'); $this->assertEquals(1, $accessoryForCompanyB->fresh()->checkouts->count(), 'Accessory should not be checked in');
@ -52,8 +53,10 @@ class AccessoryCheckinTest extends TestCase implements TestsFullMultipleCompanie
$this->assertEquals(1, $accessory->checkouts->count()); $this->assertEquals(1, $accessory->checkouts->count());
$accessoryCheckout = $accessory->checkouts->first();
$this->actingAsForApi(User::factory()->checkinAccessories()->create()) $this->actingAsForApi(User::factory()->checkinAccessories()->create())
->postJson(route('api.accessories.checkin', $accessory)) ->postJson(route('api.accessories.checkin', $accessoryCheckout))
->assertStatusMessageIs('success'); ->assertStatusMessageIs('success');
$this->assertEquals(0, $accessory->fresh()->checkouts->count(), 'Accessory should be checked in'); $this->assertEquals(0, $accessory->fresh()->checkouts->count(), 'Accessory should be checked in');
@ -65,9 +68,10 @@ class AccessoryCheckinTest extends TestCase implements TestsFullMultipleCompanie
$actor = User::factory()->checkinAccessories()->create(); $actor = User::factory()->checkinAccessories()->create();
$accessory = Accessory::factory()->checkedOutToUser($user)->create(); $accessory = Accessory::factory()->checkedOutToUser($user)->create();
$accessoryCheckout = $accessory->checkouts->first();
$this->actingAsForApi($actor) $this->actingAsForApi($actor)
->postJson(route('api.accessories.checkin', $accessory)) ->postJson(route('api.accessories.checkin', $accessoryCheckout))
->assertStatusMessageIs('success'); ->assertStatusMessageIs('success');
$this->assertDatabaseHas('action_logs', [ $this->assertDatabaseHas('action_logs', [