fix: user filament access

VATSIM-UK · Apr 22, 2024 · 3c25709 · 3c25709
1 parent ffe8588
commit 3c25709
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/app/Models/User/User.php b/app/Models/User/User.php
@@ -136,7 +136,7 @@ public function dependencies(): BelongsToMany
 
     public function canAccessPanel(\Filament\Panel $panel): bool
     {
-        return true;
+        return $panel->getId() === 'admin';
     }
 
     public function getFilamentName(): string

diff --git a/tests/app/Models/User/UserTest.php b/tests/app/Models/User/UserTest.php
@@ -5,6 +5,8 @@
 use App\BaseFunctionalTestCase;
 use Carbon\Carbon;
 use App\Providers\AuthServiceProvider;
+use Filament\Facades\Filament;
+use Filament\Panel;
 use Illuminate\Database\Eloquent\Collection;
 use TestingUtils\Traits\WithSeedUsers;
 
@@ -62,9 +64,14 @@ public function testItCanBeSerializedToJson()
         $this->assertEquals($token, $jsonData['tokens']->first()->id);
     }
 
-    public function testItCanAccessFilament()
+    public function testItCanAccessDefaultFilamentPanel()
     {
-        $this->assertTrue((new User(['id' => 1234]))->canAccessFilament());
+        $this->assertTrue((new User(['id' => 1234]))->canAccessPanel(Filament::getPanel('admin')));
+    }
+
+    public function testItCantAccessOtherFilamentPanels()
+    {
+        $this->assertFalse((new User(['id' => 1234]))->canAccessPanel(Panel::make()->id('foo')));
     }
 
     public function testItHasAFilamentName()