Filter out null labels in plugin sales SelectFilter options (#281)

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: simonhamp

app/Filament/Resources/PluginSalesResource.php

@@ -9,6 +9,7 @@
 use Filament\Resources\Resource;
 use Filament\Tables;
 use Filament\Tables\Table;
+use Illuminate\Database\Eloquent\Builder;
 
 class PluginSalesResource extends Resource
 {
@@ -71,19 +72,19 @@ public static function table(Table $table): Table
             ->filters([
                 Tables\Filters\SelectFilter::make('user_id')
                     ->label('User')
-                    ->relationship('user', 'email')
+                    ->relationship('user', 'email', fn (Builder $query) => $query->whereNotNull('email'))
                     ->searchable()
                     ->preload(),
 
                 Tables\Filters\SelectFilter::make('plugin_id')
                     ->label('Plugin')
-                    ->relationship('plugin', 'name')
+                    ->relationship('plugin', 'name', fn (Builder $query) => $query->whereNotNull('name'))
                     ->searchable()
                     ->preload(),
 
                 Tables\Filters\SelectFilter::make('plugin_bundle_id')
                     ->label('Bundle')
-                    ->relationship('pluginBundle', 'name')
+                    ->relationship('pluginBundle', 'name', fn (Builder $query) => $query->whereNotNull('name'))
                     ->searchable()
                     ->preload(),
 

tests/Feature/Filament/PluginSalesResourceTest.php

@@ -0,0 +1,48 @@
+<?php
+
+namespace Tests\Feature\Filament;
+
+use App\Filament\Resources\PluginSalesResource\Pages\ListPluginSales;
+use App\Models\Plugin;
+use App\Models\PluginLicense;
+use App\Models\User;
+use Illuminate\Foundation\Testing\RefreshDatabase;
+use Livewire\Livewire;
+use Tests\TestCase;
+
+class PluginSalesResourceTest 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_plugin_sales_page_renders_when_plugin_has_null_name(): void
+    {
+        $pluginWithName = Plugin::factory()->approved()->create(['name' => 'acme/camera-123']);
+        $pluginWithoutName = Plugin::factory()->approved()->create(['name' => null]);
+
+        PluginLicense::factory()->create(['plugin_id' => $pluginWithName->id]);
+        PluginLicense::factory()->create(['plugin_id' => $pluginWithoutName->id]);
+
+        Livewire::actingAs($this->admin)
+            ->test(ListPluginSales::class)
+            ->assertSuccessful();
+    }
+
+    public function test_plugin_sales_page_renders_successfully(): void
+    {
+        PluginLicense::factory()->count(3)->create();
+
+        Livewire::actingAs($this->admin)
+            ->test(ListPluginSales::class)
+            ->assertSuccessful();
+    }
+}