From adfa650222b79f275b3fa40cb199f31c90ea4d5f Mon Sep 17 00:00:00 2001
From: Ahmed Darrazi <ahmeddarrazi@adsmac.local>
Date: Sun, 4 Jan 2026 13:56:58 +0100
Subject: [PATCH] ui: group tenant view actions

---
 .../TenantResource/Pages/ViewTenant.php       | 61 ++++++++++---------
 tests/Feature/Filament/TenantSetupTest.php    |  1 +
 2 files changed, 34 insertions(+), 28 deletions(-)

diff --git a/app/Filament/Resources/TenantResource/Pages/ViewTenant.php b/app/Filament/Resources/TenantResource/Pages/ViewTenant.php
index cad7dac..ee10dca 100644
--- a/app/Filament/Resources/TenantResource/Pages/ViewTenant.php
+++ b/app/Filament/Resources/TenantResource/Pages/ViewTenant.php
@@ -18,34 +18,39 @@ class ViewTenant extends ViewRecord
     protected function getHeaderActions(): array
     {
         return [
-            Actions\EditAction::make(),
-            Actions\Action::make('admin_consent')
-                ->label('Admin consent')
-                ->icon('heroicon-o-clipboard-document')
-                ->url(fn (Tenant $record) => TenantResource::adminConsentUrl($record))
-                ->visible(fn (Tenant $record) => TenantResource::adminConsentUrl($record) !== null)
-                ->openUrlInNewTab(),
-            Actions\Action::make('open_in_entra')
-                ->label('Open in Entra')
-                ->icon('heroicon-o-arrow-top-right-on-square')
-                ->url(fn (Tenant $record) => TenantResource::entraUrl($record))
-                ->visible(fn (Tenant $record) => TenantResource::entraUrl($record) !== null)
-                ->openUrlInNewTab(),
-            Actions\Action::make('verify')
-                ->label('Verify configuration')
-                ->icon('heroicon-o-check-badge')
-                ->color('primary')
-                ->requiresConfirmation()
-                ->action(function (
-                    Tenant $record,
-                    TenantConfigService $configService,
-                    TenantPermissionService $permissionService,
-                    RbacHealthService $rbacHealthService,
-                    AuditLogger $auditLogger
-                ) {
-                    TenantResource::verifyTenant($record, $configService, $permissionService, $rbacHealthService, $auditLogger);
-                }),
-            TenantResource::rbacAction(),
+            Actions\ActionGroup::make([
+                Actions\EditAction::make(),
+                Actions\Action::make('admin_consent')
+                    ->label('Admin consent')
+                    ->icon('heroicon-o-clipboard-document')
+                    ->url(fn (Tenant $record) => TenantResource::adminConsentUrl($record))
+                    ->visible(fn (Tenant $record) => TenantResource::adminConsentUrl($record) !== null)
+                    ->openUrlInNewTab(),
+                Actions\Action::make('open_in_entra')
+                    ->label('Open in Entra')
+                    ->icon('heroicon-o-arrow-top-right-on-square')
+                    ->url(fn (Tenant $record) => TenantResource::entraUrl($record))
+                    ->visible(fn (Tenant $record) => TenantResource::entraUrl($record) !== null)
+                    ->openUrlInNewTab(),
+                Actions\Action::make('verify')
+                    ->label('Verify configuration')
+                    ->icon('heroicon-o-check-badge')
+                    ->color('primary')
+                    ->requiresConfirmation()
+                    ->action(function (
+                        Tenant $record,
+                        TenantConfigService $configService,
+                        TenantPermissionService $permissionService,
+                        RbacHealthService $rbacHealthService,
+                        AuditLogger $auditLogger
+                    ) {
+                        TenantResource::verifyTenant($record, $configService, $permissionService, $rbacHealthService, $auditLogger);
+                    }),
+                TenantResource::rbacAction(),
+            ])
+                ->label('Actions')
+                ->icon('heroicon-o-ellipsis-vertical')
+                ->color('gray'),
         ];
     }
 }
diff --git a/tests/Feature/Filament/TenantSetupTest.php b/tests/Feature/Filament/TenantSetupTest.php
index 2d0d171..fe463b8 100644
--- a/tests/Feature/Filament/TenantSetupTest.php
+++ b/tests/Feature/Filament/TenantSetupTest.php
@@ -172,6 +172,7 @@ public function request(string $method, string $path, array $options = []): Grap
     $response = $this->get(route('filament.admin.resources.tenants.view', $tenant));
 
     $response->assertOk();
+    $response->assertSee('Actions');
     $response->assertSee($firstKey);
     $response->assertSee('ok');
     $response->assertSee('missing');