Fix admin plugin list to show drafts when filtering by Draft status

app/Filament/Resources/PluginResource.php

@@ -23,6 +23,7 @@
 use Filament\Schemas\Schema;
 use Filament\Tables;
 use Filament\Tables\Table;
+use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Support\HtmlString;
 
 class PluginResource extends Resource
@@ -333,7 +334,11 @@ public static function table(Table $table): Table
             ])
             ->filters([
                 Tables\Filters\SelectFilter::make('status')
-                    ->options(PluginStatus::class),
+                    ->options(PluginStatus::class)
+                    ->query(fn (Builder $query, array $data): Builder => filled($data['value'])
+                        ? $query->where('status', $data['value'])
+                        : $query->where('status', '!=', PluginStatus::Draft)
+                    ),
                 Tables\Filters\SelectFilter::make('type')
                     ->options(PluginType::class),
                 Tables\Filters\TernaryFilter::make('is_official')

app/Filament/Resources/PluginResource/Pages/ListPlugins.php

@@ -2,11 +2,9 @@
 
 namespace App\Filament\Resources\PluginResource\Pages;
 
-use App\Enums\PluginStatus;
 use App\Filament\Resources\PluginResource;
 use Filament\Actions;
 use Filament\Resources\Pages\ListRecords;
-use Illuminate\Database\Eloquent\Builder;
 
 class ListPlugins extends ListRecords
 {
@@ -18,10 +16,4 @@ protected function getHeaderActions(): array
             Actions\CreateAction::make(),
         ];
     }
-
-    protected function getTableQuery(): ?Builder
-    {
-        return parent::getTableQuery()
-            ->where('status', '!=', PluginStatus::Draft);
-    }
 }

tests/Feature/Filament/PluginListDraftFilterTest.php

@@ -0,0 +1,49 @@
+<?php
+
+namespace Tests\Feature\Filament;
+
+use App\Enums\PluginStatus;
+use App\Filament\Resources\PluginResource\Pages\ListPlugins;
+use App\Models\Plugin;
+use App\Models\User;
+use Illuminate\Foundation\Testing\RefreshDatabase;
+use Livewire\Livewire;
+use Tests\TestCase;
+
+class PluginListDraftFilterTest extends TestCase
+{
+    use RefreshDatabase;
+
+    private User $admin;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $this->admin = User::factory()->create(['email' => 'admin@test.com']);
+        config(['filament.users' => ['admin@test.com']]);
+    }
+
+    public function test_draft_plugins_are_hidden_by_default(): void
+    {
+        $draft = Plugin::factory()->draft()->create();
+        $approved = Plugin::factory()->approved()->create();
+
+        Livewire::actingAs($this->admin)
+            ->test(ListPlugins::class)
+            ->assertCanNotSeeTableRecords([$draft])
+            ->assertCanSeeTableRecords([$approved]);
+    }
+
+    public function test_draft_plugins_are_visible_when_filtering_by_draft_status(): void
+    {
+        $draft = Plugin::factory()->draft()->create();
+        $approved = Plugin::factory()->approved()->create();
+
+        Livewire::actingAs($this->admin)
+            ->test(ListPlugins::class)
+            ->filterTable('status', PluginStatus::Draft->value)
+            ->assertCanSeeTableRecords([$draft])
+            ->assertCanNotSeeTableRecords([$approved]);
+    }
+}