feat(test): (LAR-86) adding test for bannished user

Author: StevyMarlino

app/Filament/Resources/UserResource.php

@@ -123,6 +123,16 @@ public static function getPages(): array
 
     public static function BanUserAction(User $record, $reason): void
     {
+        if ($record->banned_at !== null) {
+            Notification::make()
+                ->warning()
+                ->title(__('Impossible de bannir'))
+                ->body(__('Cet utilisateur est déjà banni.'))
+                ->send();
+    
+            return;
+        }
+        
         $record->banned_at = Carbon::now();
         $record->banned_reason = $reason;
         $record->save();

routes/cpanel.php

@@ -5,11 +5,9 @@
 use App\Http\Controllers\Cpanel;
 use Illuminate\Support\Facades\Route;
 
-Route::group(['middleware' => ['role:admin']], function () { 
-    Route::redirect('/', 'cpanel/home');
-    Route::get('/home', Cpanel\DashboardController::class)->name('home');
-    Route::get('/analytics', Cpanel\AnalyticsController::class)->name('analytics');
-    Route::prefix('users')->as('users.')->group(function (): void {
-        Route::get('/', Cpanel\UserController::class)->name('browse');
-    });
+Route::redirect('/', 'cpanel/home');
+Route::get('/home', Cpanel\DashboardController::class)->name('home');
+Route::get('/analytics', Cpanel\AnalyticsController::class)->name('analytics');
+Route::prefix('users')->as('users.')->group(function (): void {
+    Route::get('/', Cpanel\UserController::class)->name('browse');
 });

tests/Feature/UserResourceTest.php renamed to tests/Feature/Filament/UserResourceTest.php

@@ -4,33 +4,31 @@
 
 use Carbon\Carbon;
 use App\Models\User;
+use function Pest\Laravel\get;
 use App\Events\UserBannedEvent;
 use App\Events\UserUnbannedEvent;
 use Spatie\Permission\Models\Role;
 use Illuminate\Support\Facades\Event;
-use Illuminate\Support\Facades\Queue;
 use App\Filament\Resources\UserResource;
-use Illuminate\Support\Facades\Notification;
-use App\Filament\Resources\UserResource\Pages\ListUsers;
-
 
 beforeEach(function (): void {
     Event::fake();
-    Notification::fake();
-    Queue::fake();
-    $this->user = $this->login();
+    $this->user = User::factory(['email' => 'user@laravel.cm'])->create();
+    Role::create(['name' => 'admin']);
+    $this->user->assignRole(['admin']);
+    $this->actingAs($this->user, 'web');
 });
 
 describe(UserResource::class, function() {
-    it('only admin can ban a user and send a ban notification', function () {
+    it('can render admin page', function (): void {
+        get(UserResource::getUrl())->assertSuccessful();
+    });
 
-        Role::create(['name' => 'user']);
-        $admin = $this->user->assignRole('user');
+    it('only admin can ban a user and send a ban notification', function () {
+        $this->get('/cp')->assertSuccessful();
 
         $user = User::factory()->create();
 
-        // $this->actingAs($admin);
-
         UserResource::BanUserAction($user, 'Violation des règles de la communauté');
 
         $user->refresh();
@@ -42,16 +40,13 @@
     });
 
     it('can unban a user and send a unban notification', function () {
-        Role::create(['name' => 'admin']);
-        $admin = $this->user->assignRole('admin');
-
+        $this->get('/cp')->assertSuccessful();
+        
         $user = User::factory()->create([
             'banned_at' => now(),
             'banned_reason' => 'Violation des règles de la communauté'
         ]);
 
-        $this->actingAs($admin);
-
         UserResource::UnbanUserAction($user);
 
         $user->refresh();
@@ -62,6 +57,17 @@
         Event::assertDispatched(UserUnbannedEvent::class);
     });
 
+    it('does not ban an already banned user', function () {
+        $this->get('/cp')->assertSuccessful();
+        
+        $user = User::factory()->create(['banned_at' => now()]);
+    
+        UserResource::BanUserAction($user, 'Violation des règles');
+    
+        expect($user->banned_reason)->not->toBe('Violation des règles')
+            ->and($user->banned_at)->not->toBeNull();
+    });
+
     it('prevents a banned user from logging in', function () {
         $user = User::factory()->create([
             'banned_at' => now(),
@@ -72,4 +78,4 @@
             ->assertRedirect(route('login'))  
             ->assertSessionHasErrors(['email']);
     });
-});
+})->group('users');

tests/Feature/Livewire/Pages/Forum/IndexTest.php

@@ -24,4 +24,4 @@
         ->assertViewHas('threads', fn ($threads) => count($threads) === 30)
         ->assertSee(__('pagination.next'))
         ->assertStatus(200);
-});
+});