Skip to content

Commit 08d5fe0

Browse files
committed
Add Assertion in to delete test of User , Role and permission
1 parent 690656c commit 08d5fe0

File tree

3 files changed

+11
-10
lines changed

3 files changed

+11
-10
lines changed

tests/Feature/Backend/ManagePermissionsTest.php

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -110,12 +110,9 @@ public function a_user_can_delete_a_permission()
110110

111111
$permission = create(Permission::class);
112112

113-
/*$this->assertDatabaseHas(config('access.permissions_table'), [
114-
['name' => $permission->name, 'id' => $permission->id]
115-
]);*/
116-
117113
$this->actingAs($this->admin)
118114
->delete(route('admin.access.permission.destroy', $permission))
115+
->assertStatus(302)
119116
->assertSessionHas(['flash_success' => trans('alerts.backend.permissions.deleted')]);
120117

121118
/*$this->assertDatabaseMissing('permissions', [

tests/Feature/Backend/ManageRolesTest.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -160,15 +160,15 @@ public function a_user_can_delete_a_role()
160160

161161
$role = create(Role::class);
162162

163-
/*$this->assertDatabaseHas(config('access.roles_table'), [
164-
['name' => $role->name, 'id' => $role->id]
165-
]);*/
166-
167163
$this->actingAs($this->admin)
168164
->delete(route('admin.access.role.destroy', $role))
165+
->assertStatus(302)
169166
->assertSessionHas(['flash_success' => trans('alerts.backend.roles.deleted')]);
170167

171-
//$this->assertDatabaseMissing(config('access.roles_table'), ['name' => $role->name, 'id' => $role->id]);
168+
/*$this->assertDatabaseMissing(config('access.roles_table'), [
169+
'name' => $role->name,
170+
'id' => $role->id
171+
]);*/
172172

173173
Event::assertDispatched(RoleDeleted::class);
174174
}

tests/Feature/Backend/ManageUsersTest.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -285,9 +285,13 @@ public function a_user_can_delete_a_user()
285285

286286
$this->actingAs($this->admin)
287287
->delete(route('admin.access.user.destroy', $user))
288+
->assertStatus(302)
288289
->assertSessionHas(['flash_success' => trans('alerts.backend.users.deleted')]);
289290

290-
$this->assertDatabaseMissing(config('access.users_table'), ['name' => $user->first_name, 'id' => $user->id]);
291+
$this->assertDatabaseMissing(config('access.users_table'), [
292+
'name' => $user->first_name,
293+
'id' => $user->id
294+
]);
291295

292296
Event::assertDispatched(UserDeleted::class);
293297
}

0 commit comments

Comments
 (0)