Skip to content

Commit c59385f

Browse files
committed
Use Monolog\Level instead of deprecated Monolog\Logger
1 parent 73c47d2 commit c59385f

File tree

5 files changed

+28
-28
lines changed

5 files changed

+28
-28
lines changed

tests/HasPermissionsTest.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
use Maklad\Permission\Exceptions\PermissionDoesNotExist;
77
use Maklad\Permission\Models\Permission;
88
use Maklad\Permission\Models\Role;
9-
use Monolog\Logger;
9+
use Monolog\Level;
1010

1111
class HasPermissionsTest extends TestCase
1212
{
@@ -34,7 +34,7 @@ public function it_throws_an_exception_when_assigning_a_permission_that_does_not
3434
$this->testUser->givePermissionTo('permission-does-not-exist');
3535
} finally {
3636
$message = $this->helpers->getPermissionDoesNotExistMessage('permission-does-not-exist', 'web');
37-
$this->assertLogMessage($message, Logger::ALERT);
37+
$this->assertLogMessage($message, Level::Alert);
3838
}
3939
}
4040
}
@@ -53,7 +53,7 @@ public function it_throws_an_exception_when_assigning_a_permission_to_a_user_fro
5353
$this->testUser->givePermissionTo($this->testAdminPermission);
5454
} finally {
5555
$message = $this->helpers->getGuardDoesNotMatchMessage(collect(['web']), 'admin');
56-
$this->assertLogMessage($message, Logger::ALERT);
56+
$this->assertLogMessage($message, Level::Alert);
5757
}
5858
}
5959
}
@@ -327,7 +327,7 @@ public function it_throws_an_exception_when_the_permission_does_not_exist()
327327
$this->testUser->hasPermissionTo('does-not-exist');
328328
} finally {
329329
$message = $this->helpers->getPermissionDoesNotExistMessage('does-not-exist', 'web');
330-
$this->assertLogMessage($message, Logger::ALERT);
330+
$this->assertLogMessage($message, Level::Alert);
331331
}
332332
}
333333
}
@@ -346,7 +346,7 @@ public function it_throws_an_exception_when_the_permission_does_not_exist_for_th
346346
$this->testUser->hasPermissionTo('admin-permission');
347347
} finally {
348348
$message = $this->helpers->getPermissionDoesNotExistMessage('admin-permission', 'web');
349-
$this->assertLogMessage($message, Logger::ALERT);
349+
$this->assertLogMessage($message, Level::Alert);
350350
}
351351
}
352352
}

tests/HasRolesTest.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
use Maklad\Permission\Exceptions\GuardDoesNotMatch;
66
use Maklad\Permission\Exceptions\RoleDoesNotExist;
77
use Maklad\Permission\Models\Role;
8-
use Monolog\Logger;
8+
use Monolog\Level;
99

1010
class HasRolesTest extends TestCase
1111
{
@@ -73,7 +73,7 @@ public function it_throws_an_exception_when_assigning_a_role_that_does_not_exist
7373
$this->testUser->assignRole('evil-emperor');
7474
} finally {
7575
$message = $this->helpers->getRoleDoesNotExistMessage('evil-emperor', 'web');
76-
$this->assertLogMessage($message, Logger::ALERT);
76+
$this->assertLogMessage($message, Level::Alert);
7777
}
7878
}
7979
}
@@ -92,7 +92,7 @@ public function it_can_only_assign_roles_from_the_correct_guard()
9292
$this->testUser->assignRole('testAdminRole');
9393
} finally {
9494
$message = $this->helpers->getRoleDoesNotExistMessage('testAdminRole', 'web');
95-
$this->assertLogMessage($message, Logger::ALERT);
95+
$this->assertLogMessage($message, Level::Alert);
9696
}
9797
}
9898
}
@@ -111,7 +111,7 @@ public function it_throws_an_exception_when_assigning_a_role_from_a_different_gu
111111
$this->testUser->assignRole($this->testAdminRole);
112112
} finally {
113113
$message = $this->helpers->getGuardDoesNotMatchMessage(collect(['web']), 'admin');
114-
$this->assertLogMessage($message, Logger::ALERT);
114+
$this->assertLogMessage($message, Level::Alert);
115115
}
116116
}
117117
}
@@ -176,7 +176,7 @@ public function it_throws_an_exception_when_syncing_a_role_from_another_guard()
176176
$this->testUser->syncRoles('testRole', $this->testAdminRole);
177177
} finally {
178178
$message = $this->helpers->getGuardDoesNotMatchMessage(collect(['web']), 'admin');
179-
$this->assertLogMessage($message, Logger::ALERT);
179+
$this->assertLogMessage($message, Level::Alert);
180180
}
181181
}
182182
}

tests/MiddlewareTest.php

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
use Maklad\Permission\Exceptions\UnauthorizedException;
99
use Maklad\Permission\Middlewares\PermissionMiddleware;
1010
use Maklad\Permission\Middlewares\RoleMiddleware;
11-
use Monolog\Logger;
11+
use Monolog\Level;
1212

1313
class MiddlewareTest extends TestCase
1414
{
@@ -41,7 +41,7 @@ public function a_guest_cannot_access_a_route_protected_by_the_role_middleware()
4141
);
4242

4343
$message = $this->helpers->getUserNotLoggedINMessage();
44-
$this->assertLogMessage($message, Logger::ALERT);
44+
$this->assertLogMessage($message, Level::Alert);
4545
}
4646
}
4747

@@ -109,7 +109,7 @@ public function a_user_cannot_access_a_route_protected_by_the_role_middleware_if
109109
config('permission.display_permission_in_exception', $show_permission);
110110
$message = $this->helpers->getUnauthorizedRoleMessage('testRole2');
111111
$this->assertShowPermission($message, 'testRole2');
112-
$this->assertLogMessage($message, Logger::ALERT);
112+
$this->assertLogMessage($message, Level::Alert);
113113
}
114114
}
115115
}
@@ -138,7 +138,7 @@ public function a_user_cannot_access_a_route_protected_by_role_middleware_if_hav
138138
$message = $this->helpers->getUnauthorizedRoleMessage('testRole, testRole2');
139139
$this->assertShowPermission($message, 'testRole');
140140
$this->assertShowPermission($message, 'testRole2');
141-
$this->assertLogMessage($message, Logger::ALERT);
141+
$this->assertLogMessage($message, Level::Alert);
142142
}
143143
}
144144
}
@@ -166,7 +166,7 @@ public function a_user_cannot_access_a_route_protected_by_role_middleware_if_rol
166166
config('permission.display_permission_in_exception', $show_permission);
167167
$message = $this->helpers->getUnauthorizedRoleMessage('test');
168168
$this->assertShowPermission($message, 'test');
169-
$this->assertLogMessage($message, Logger::ALERT);
169+
$this->assertLogMessage($message, Level::Alert);
170170
}
171171
}
172172
}
@@ -188,7 +188,7 @@ public function a_guest_cannot_access_a_route_protected_by_the_permission_middle
188188
);
189189

190190
$message = $this->helpers->getUserNotLoggedINMessage();
191-
$this->assertLogMessage($message, Logger::ALERT);
191+
$this->assertLogMessage($message, Level::Alert);
192192
}
193193
}
194194

@@ -257,7 +257,7 @@ public function a_user_cannot_access_a_route_protected_by_the_permission_middlew
257257
config('permission.display_permission_in_exception', $show_permission);
258258
$message = $this->helpers->getUnauthorizedPermissionMessage('edit-news');
259259
$this->assertShowPermission($message, 'edit-news');
260-
$this->assertLogMessage($message, Logger::ALERT);
260+
$this->assertLogMessage($message, Level::Alert);
261261
}
262262
}
263263
}
@@ -286,7 +286,7 @@ public function a_user_cannot_access_a_route_protected_by_permission_middleware_
286286
$message = $this->helpers->getUnauthorizedPermissionMessage('edit-articles, edit-news');
287287
$this->assertShowPermission($message, 'edit-articles');
288288
$this->assertShowPermission($message, 'edit-news');
289-
$this->assertLogMessage($message, Logger::ALERT);
289+
$this->assertLogMessage($message, Level::Alert);
290290
}
291291
}
292292
}

tests/PermissionTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
use Maklad\Permission\Exceptions\PermissionAlreadyExists;
66
use Maklad\Permission\Models\Permission;
7-
use Monolog\Logger;
7+
use Monolog\Level;
88

99
class PermissionTest extends TestCase
1010
{
@@ -23,7 +23,7 @@ public function it_throws_an_exception_when_the_permission_already_exists()
2323
\app(\config('permission.models.permission'))->create(['name' => 'test-permission']);
2424
} finally {
2525
$message = $this->helpers->getPermissionAlreadyExistsMessage('test-permission', 'web');
26-
$this->assertLogMessage($message, Logger::ALERT);
26+
$this->assertLogMessage($message, Level::Alert);
2727
}
2828
}
2929
}

tests/RoleTest.php

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
use Maklad\Permission\Exceptions\RoleAlreadyExists;
88
use Maklad\Permission\Models\Permission;
99
use Maklad\Permission\Models\Role;
10-
use Monolog\Logger;
10+
use Monolog\Level;
1111

1212
class RoleTest extends TestCase
1313
{
@@ -50,7 +50,7 @@ public function it_throws_an_exception_when_the_role_already_exists()
5050
\app(\config('permission.models.role'))->create(['name' => 'test-role']);
5151
} finally {
5252
$message = $this->helpers->getRoleAlreadyExistsMessage('test-role', 'web');
53-
$this->assertLogMessage($message, Logger::ALERT);
53+
$this->assertLogMessage($message, Level::Alert);
5454
}
5555
}
5656
}
@@ -77,7 +77,7 @@ public function it_throws_an_exception_when_given_a_permission_that_does_not_exi
7777
$this->testUserRole->givePermissionTo('create-evil-empire');
7878
} finally {
7979
$message = $this->helpers->getPermissionDoesNotExistMessage('create-evil-empire', 'web');
80-
$this->assertLogMessage($message, Logger::ALERT);
80+
$this->assertLogMessage($message, Level::Alert);
8181
}
8282
}
8383
}
@@ -96,7 +96,7 @@ public function it_throws_an_exception_when_given_a_permission_that_belongs_to_a
9696
$this->testUserRole->givePermissionTo($this->testAdminPermission);
9797
} finally {
9898
$message = $this->helpers->getGuardDoesNotMatchMessage(collect(['web']), 'admin');
99-
$this->assertLogMessage($message, Logger::ALERT);
99+
$this->assertLogMessage($message, Level::Alert);
100100
}
101101
}
102102
}
@@ -147,7 +147,7 @@ public function it_throws_an_exception_when_syncing_permissions_that_do_not_exis
147147
$this->testUserRole->syncPermissions('permission-does-not-exist');
148148
} finally {
149149
$message = $this->helpers->getPermissionDoesNotExistMessage('permission-does-not-exist', 'web');
150-
$this->assertLogMessage($message, Logger::ALERT);
150+
$this->assertLogMessage($message, Level::Alert);
151151
}
152152
}
153153
}
@@ -166,7 +166,7 @@ public function it_throws_an_exception_when_syncing_permissions_that_belong_to_a
166166
$this->testUserRole->syncPermissions($this->testAdminPermission);
167167
} finally {
168168
$message = $this->helpers->getGuardDoesNotMatchMessage(collect(['web']), 'admin');
169-
$this->assertLogMessage($message, Logger::ALERT);
169+
$this->assertLogMessage($message, Level::Alert);
170170
}
171171
}
172172
}
@@ -227,7 +227,7 @@ public function it_throws_an_exception_if_the_permission_does_not_exist()
227227
$this->testUserRole->hasPermissionTo('doesnt-exist');
228228
} finally {
229229
$message = $this->helpers->getPermissionDoesNotExistMessage('doesnt-exist', 'web');
230-
$this->assertLogMessage($message, Logger::ALERT);
230+
$this->assertLogMessage($message, Level::Alert);
231231
}
232232
}
233233
}
@@ -256,7 +256,7 @@ public function it_throws_an_exception_when_a_permission_of_the_wrong_guard_is_p
256256
$this->testUserRole->hasPermissionTo($permission);
257257
} finally {
258258
$message = $this->helpers->getGuardDoesNotMatchMessage(collect(['web']), 'admin');
259-
$this->assertLogMessage($message, Logger::ALERT);
259+
$this->assertLogMessage($message, Level::Alert);
260260
}
261261
}
262262
}

0 commit comments

Comments
 (0)