Skip to content

Commit

Permalink
Implement tests around consumable checkout
Browse files Browse the repository at this point in the history
  • Loading branch information
marcusmoore committed Jan 30, 2024
1 parent 3f76d65 commit 13c37e7
Show file tree
Hide file tree
Showing 3 changed files with 79 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,6 @@ public function store(Request $request, $consumableId)
return redirect()->route('consumables.index')->with('error', trans('admin/consumables/message.checkout.unavailable'));
}


$admin_user = Auth::user();
$assigned_to = e($request->input('assigned_to'));

Expand Down
18 changes: 18 additions & 0 deletions database/factories/ConsumableFactory.php
Original file line number Diff line number Diff line change
Expand Up @@ -91,4 +91,22 @@ public function ink()
];
});
}

public function withoutItemsRemaining()
{
return $this->state(function () {
return [
'qty' => 1,
];
})->afterCreating(function (Consumable $consumable) {
$user = User::factory()->create();

$consumable->users()->attach($consumable->id, [
'consumable_id' => $consumable->id,
'user_id' => $user->id,
'assigned_to' => $user->id,
'note' => '',
]);
});
}
}
67 changes: 61 additions & 6 deletions tests/Feature/Checkouts/ConsumableCheckoutTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,11 @@

namespace Tests\Feature\Checkouts;

use App\Models\Actionlog;
use App\Models\Consumable;
use App\Models\User;
use App\Notifications\CheckoutConsumableNotification;
use Illuminate\Support\Facades\Notification;
use Tests\Support\InteractsWithSettings;
use Tests\TestCase;

Expand All @@ -11,31 +16,81 @@ class ConsumableCheckoutTest extends TestCase

public function testCheckingOutConsumableRequiresCorrectPermission()
{
$this->markTestIncomplete();
$this->actingAs(User::factory()->create())
->post(route('consumables.checkout.store', Consumable::factory()->create()))
->assertForbidden();
}

public function testValidationWhenCheckingOutConsumable()
{
$this->markTestIncomplete();
$this->actingAs(User::factory()->checkoutConsumables()->create())
->post(route('consumables.checkout.store', Consumable::factory()->create()), [
// missing assigned_to
])
->assertSessionHas('error');
}

public function testConsumableMustBeAvailableWhenCheckingOut()
{
$this->markTestIncomplete();
$this->actingAs(User::factory()->checkoutConsumables()->create())
->post(route('consumables.checkout.store', Consumable::factory()->withoutItemsRemaining()->create()), [
'assigned_to' => User::factory()->create()->id,
])
->assertSessionHas('error');
}

public function testConsumableCanBeCheckedOut()
{
$this->markTestIncomplete();
$consumable = Consumable::factory()->create();
$user = User::factory()->create();

$this->actingAs(User::factory()->checkoutConsumables()->create())
->post(route('consumables.checkout.store', $consumable), [
'assigned_to' => $user->id,
]);

$this->assertTrue($user->consumables->contains($consumable));
}

public function testUserSentNotificationUponCheckout()
{
$this->markTestIncomplete();
Notification::fake();

$consumable = Consumable::factory()->create();
$user = User::factory()->create();

$this->actingAs(User::factory()->checkoutConsumables()->create())
->post(route('consumables.checkout.store', $consumable), [
'assigned_to' => $user->id,
]);

Notification::assertSentTo($user, CheckoutConsumableNotification::class);
}

public function testActionLogCreatedUponCheckout()
{
$this->markTestIncomplete();
$consumable = Consumable::factory()->create();
$actor = User::factory()->checkoutConsumables()->create();
$user = User::factory()->create();

$this->actingAs($actor)
->post(route('consumables.checkout.store', $consumable), [
'assigned_to' => $user->id,
'note' => 'oh hi there',
]);

$this->assertEquals(
1,
Actionlog::where([
'action_type' => 'checkout',
'target_id' => $user->id,
'target_type' => User::class,
'item_id' => $consumable->id,
'item_type' => Consumable::class,
'user_id' => $actor->id,
'note' => 'oh hi there',
])->count(),
'Log entry either does not exist or there are more than expected'
);
}
}

0 comments on commit 13c37e7

Please sign in to comment.