From 3825f5fb61da2cc94e15e934a37534890ac042bf Mon Sep 17 00:00:00 2001 From: snipe Date: Sat, 22 Jun 2024 19:47:59 +0100 Subject: [PATCH] Added more permissions tests to user delete UI Signed-off-by: snipe --- tests/Feature/Users/Ui/DeleteUserTest.php | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/tests/Feature/Users/Ui/DeleteUserTest.php b/tests/Feature/Users/Ui/DeleteUserTest.php index 1fde5d9558..645a715090 100644 --- a/tests/Feature/Users/Ui/DeleteUserTest.php +++ b/tests/Feature/Users/Ui/DeleteUserTest.php @@ -49,7 +49,7 @@ class DeleteUserTest extends TestCase } - public function testPermissionsToDeleteUser() + public function testFmcsPermissionsToDeleteUser() { $this->settings->enableMultipleFullCompanySupport(); @@ -60,14 +60,22 @@ class DeleteUserTest extends TestCase $userFromA = User::factory()->for($companyA)->create(); $userFromB = User::factory()->for($companyB)->create(); - $this->followingRedirects()->actingAs(User::factory()->deleteUsers()->for($companyA)->create()) + $response = $this->followingRedirects()->actingAs(User::factory()->deleteUsers()->for($companyA)->create()) ->delete(route('users.destroy', ['user' => $userFromB->id])) ->assertStatus(403); + $this->followRedirects($response)->assertSee('sad-panda.png'); - $this->actingAs(User::factory()->deleteUsers()->for($companyA)->create()) + $response = $this->actingAs(User::factory()->deleteUsers()->for($companyA)->create()) ->delete(route('users.destroy', ['user' => $userFromA->id])) ->assertStatus(302) ->assertRedirect(route('users.index')); + $this->followRedirects($response)->assertSee('sad-panda.png'); + + $response = $this->actingAs($superuser) + ->delete(route('users.destroy', ['user' => $userFromA->id])) + ->assertStatus(302) + ->assertRedirect(route('users.index')); + $this->followRedirects($response)->assertSee('Success'); }