diff --git a/tests/Feature/Admin/System/ActivityLogTest.php b/tests/Feature/Admin/System/ActivityLogTest.php new file mode 100644 index 0000000..c33d283 --- /dev/null +++ b/tests/Feature/Admin/System/ActivityLogTest.php @@ -0,0 +1,56 @@ +assertRedirect(route('login')); + }); + + it('returns 403 when user has no permission to view activity logs', function () { + actingAs(createUnauthorizedUser()) + ->get(route('system.activity-logs.index')) + ->assertStatus(403); + }); +}); + +describe('Activity Log Module - Authorized Actions', function () { + beforeEach(function () { + $user = createAuthorizedUser(['View:Activity']); + actingAs($user); + }); + + it('can access activity logs index page', function () { + // Create a dummy activity to ensure data is present + activity() + ->useLog('Testing') + ->log('Test Activity Log Entry'); + + get(route('system.activity-logs.index')) + ->assertOk() + ->assertInertia(fn ($page) => $page + ->component('admin/system/activity-log/index') + ->has('activities') + ); + }); +}); + +describe('Activity Log Module - Unauthorized Actions', function () { + it('cannot view activity logs without permission', function () { + actingAs(createUnauthorizedUser()) + ->get(route('system.activity-logs.index')) + ->assertStatus(403); + }); +}); diff --git a/tests/Feature/Admin/System/LogTest.php b/tests/Feature/Admin/System/LogTest.php index 1130208..b0d06c3 100644 --- a/tests/Feature/Admin/System/LogTest.php +++ b/tests/Feature/Admin/System/LogTest.php @@ -1,47 +1,44 @@ assertRedirect(route('login')); - get(route('system.activity-logs.index'))->assertRedirect(route('login')); + get(route('system.logs.index')) + ->assertRedirect(route('login')); }); it('returns 403 when user has no permission to view logs', function () { - actingAs(createUnauthorizedUser()); - - get(route('system.logs.index'))->assertStatus(403); - get(route('system.activity-logs.index'))->assertStatus(403); + actingAs(createUnauthorizedUser()) + ->get(route('system.logs.index')) + ->assertStatus(403); }); }); -describe('Log Module - Authorized Actions', function () { +describe('System Log Module - Authorized Actions', function () { beforeEach(function () { - $user = createAuthorizedUser([ - 'View:Log', - 'View:Activity', - ]); + $user = createAuthorizedUser(['View:Log']); actingAs($user); }); it('can access system logs index page', function () { - // Create a dummy log file to ensure it doesn't crash if no logs exist + // Create a dummy log file to ensure it doesn't crash $path = storage_path('logs/laravel.log'); if (! File::exists(storage_path('logs'))) { File::makeDirectory(storage_path('logs'), 0755, true); } - File::put($path, "[2026-04-30 08:00:00] local.INFO: Test log message\n"); + File::put($path, "[2026-04-30 08:00:00] local.INFO: Test system log entry\n"); get(route('system.logs.index')) ->assertOk() @@ -51,32 +48,12 @@ ->has('logs') ); }); - - it('can access activity logs index page', function () { - // Create a dummy activity - activity() - ->useLog('TestModule') - ->log('Test Activity'); - - get(route('system.activity-logs.index')) - ->assertOk() - ->assertInertia(fn ($page) => $page - ->component('admin/system/activity-log/index') - ->has('activities') - ); - }); }); -describe('Log Module - Unauthorized Actions', function () { - beforeEach(function () { - actingAs(createUnauthorizedUser()); - }); - +describe('System Log Module - Unauthorized Actions', function () { it('cannot view system logs without permission', function () { - get(route('system.logs.index'))->assertStatus(403); - }); - - it('cannot view activity logs without permission', function () { - get(route('system.activity-logs.index'))->assertStatus(403); + actingAs(createUnauthorizedUser()) + ->get(route('system.logs.index')) + ->assertStatus(403); }); });