TenantAtlas/apps/platform/tests/Feature/Monitoring/HeaderContextBarTest.php
ahmido b159dacd36 feat: clean up legacy tenant environment context (#372)
## Summary
- remove legacy tenant-scoped routing and middleware paths in favor of the current environment/workspace context flow
- update Filament pages and resources to use the cleaned-up admin surface and environment filter context
- add the related spec 317 artifacts and targeted tests for environment filter state and legacy context cleanup

## Testing
- not run as part of this commit/push/PR workflow

Co-authored-by: Ahmed Darrazi <ahmed.darrazi@live.de>
Reviewed-on: #372
2026-05-16 18:25:36 +00:00

295 lines
11 KiB
PHP

<?php
declare(strict_types=1);
use App\Filament\Pages\ChooseWorkspace;
use App\Models\ManagedEnvironment;
use App\Models\OperationRun;
use App\Support\ManagedEnvironmentLinks;
use App\Support\Workspaces\WorkspaceContext;
use Filament\Facades\Filament;
it('renders the tenant context picker on tenantless Monitoring → Operations', function (): void {
$tenant = ManagedEnvironment::factory()->create(['status' => 'active']);
[$user, $tenant] = createUserWithTenant($tenant, role: 'owner');
$workspaceName = $tenant->workspace?->name;
Filament::setTenant(null, true);
$response = $this->actingAs($user)
->withSession([
WorkspaceContext::SESSION_KEY => (int) $tenant->workspace_id,
WorkspaceContext::LAST_ENVIRONMENT_IDS_SESSION_KEY => [
(string) $tenant->workspace_id => (int) $tenant->getKey(),
],
])
->get(route('admin.operations.index', ['workspace' => $tenant->workspace]))
->assertOk()
->assertSee($workspaceName ?? 'Select workspace')
->assertSee(__('localization.shell.search_environments'))
->assertSee('Switch workspace')
->assertSee('admin/select-environment')
->assertSee(__('localization.shell.clear_environment_scope'))
->assertSee($tenant->getFilamentName());
$content = $response->getContent();
expect($content)->not->toBeFalse();
expect(
preg_match('/<a[^>]+href="([^"]+)"[^>]*>\s*Switch workspace\s*<\/a>/i', (string) $content, $matches)
)->toBe(1);
expect($matches[1] ?? null)->toBe(ChooseWorkspace::getUrl(panel: 'admin').'?choose=1');
expect($matches[1] ?? null)->not->toContain('/admin/workspaces');
$this->actingAs($user)
->withSession([
WorkspaceContext::SESSION_KEY => (int) $tenant->workspace_id,
])
->post(route('admin.select-environment'), ['managed_environment_id' => (int) $tenant->getKey()])
->assertRedirect();
});
it('disables the tenant picker when no workspace is active (Global Mode)', function (): void {
$user = \App\Models\User::factory()->create();
$workspace = \App\Models\Workspace::factory()->create();
\App\Models\WorkspaceMembership::factory()->create([
'workspace_id' => (int) $workspace->getKey(),
'user_id' => (int) $user->getKey(),
'role' => 'owner',
]);
Filament::setTenant(null, true);
session()->forget(WorkspaceContext::SESSION_KEY);
$this->actingAs($user)
->get('/admin/workspaces')
->assertOk()
->assertSee('Choose a workspace first.')
->assertDontSee(__('localization.shell.search_environments'));
});
it('keeps tenant-scoped pages selector read-only while exposing the clear tenant scope action', function (): void {
$tenant = ManagedEnvironment::factory()->create(['status' => 'active']);
[$user, $tenant] = createUserWithTenant($tenant, role: 'owner');
$this->actingAs($user)
->withSession([
WorkspaceContext::SESSION_KEY => (int) $tenant->workspace_id,
])
->get(ManagedEnvironmentLinks::viewUrl($tenant))
->assertOk()
->assertSee($tenant->getFilamentName())
->assertSee(__('localization.shell.clear_environment_scope'));
});
it('renders routed tenant resource view pages with a clear tenant scope action but no inline selector list', function (): void {
$currentTenant = ManagedEnvironment::factory()->create([
'status' => 'active',
'name' => 'YPTW2',
'environment' => 'dev',
]);
[$user, $currentTenant] = createUserWithTenant($currentTenant, role: 'owner');
$routedTenant = ManagedEnvironment::factory()->create([
'status' => ManagedEnvironment::STATUS_ONBOARDING,
'workspace_id' => (int) $currentTenant->workspace_id,
'name' => 'Test',
'environment' => 'dev',
]);
createUserWithTenant($routedTenant, $user, role: 'owner');
Filament::setTenant($currentTenant, true);
$this->actingAs($user)
->withSession([
WorkspaceContext::SESSION_KEY => (int) $currentTenant->workspace_id,
WorkspaceContext::LAST_ENVIRONMENT_IDS_SESSION_KEY => [
(string) $currentTenant->workspace_id => (int) $currentTenant->getKey(),
],
])
->get(ManagedEnvironmentLinks::viewUrl($routedTenant))
->assertOk()
->assertSee(__('localization.shell.clear_environment_scope'));
});
it('filters the header tenant picker to tenants the user can access', function (): void {
$tenantA = ManagedEnvironment::factory()->create(['status' => 'active']);
[$user, $tenantA] = createUserWithTenant($tenantA, role: 'owner', workspaceRole: 'readonly');
$tenantB = ManagedEnvironment::factory()->create([
'status' => 'active',
'workspace_id' => (int) $tenantA->workspace_id,
'name' => 'ZZZ-UNAUTHORIZED-TENANT-NAME-12345',
]);
Filament::setTenant(null, true);
$this->actingAs($user)
->withSession([
WorkspaceContext::SESSION_KEY => (int) $tenantA->workspace_id,
])
->get(route('admin.operations.index', ['workspace' => $tenantA->workspace]))
->assertOk()
->assertSee($tenantA->getFilamentName())
->assertDontSee($tenantB->getFilamentName());
});
it('keeps the header tenant picker limited to tenant-entitled active tenants for workspace owners', function (): void {
$tenantA = ManagedEnvironment::factory()->create(['status' => 'active']);
[$user, $tenantA] = createUserWithTenant($tenantA, role: 'owner', workspaceRole: 'owner');
$tenantB = ManagedEnvironment::factory()->create([
'status' => 'active',
'workspace_id' => (int) $tenantA->workspace_id,
'name' => 'ZZZ-UNASSIGNED-TENANT-NAME-12345',
]);
Filament::setTenant(null, true);
$this->actingAs($user)
->withSession([
WorkspaceContext::SESSION_KEY => (int) $tenantA->workspace_id,
])
->get(route('admin.operations.index', ['workspace' => $tenantA->workspace]))
->assertOk()
->assertSee($tenantA->getFilamentName())
->assertDontSee($tenantB->getFilamentName());
});
it('hides onboarding tenants from the header tenant picker even for workspace owners', function (): void {
$tenantA = ManagedEnvironment::factory()->create([
'status' => 'active',
'name' => 'YPTW2',
'environment' => 'other',
]);
[$user, $tenantA] = createUserWithTenant($tenantA, role: 'owner', workspaceRole: 'owner');
$onboardingTenant = ManagedEnvironment::factory()->create([
'status' => ManagedEnvironment::STATUS_ONBOARDING,
'workspace_id' => (int) $tenantA->workspace_id,
'name' => 'YPTW2',
'environment' => 'dev',
]);
Filament::setTenant(null, true);
$this->actingAs($user)
->withSession([
WorkspaceContext::SESSION_KEY => (int) $tenantA->workspace_id,
])
->get(route('admin.operations.index', ['workspace' => $tenantA->workspace]))
->assertOk()
->assertSee($tenantA->getFilamentName())
->assertDontSee($onboardingTenant->getFilamentName());
});
it('does not implicitly switch tenant when opening canonical operation deep links', function (): void {
$tenantA = ManagedEnvironment::factory()->create(['status' => 'active']);
[$user, $tenantA] = createUserWithTenant($tenantA, role: 'owner');
$tenantB = ManagedEnvironment::factory()->create([
'status' => 'active',
'workspace_id' => (int) $tenantA->workspace_id,
]);
$user->tenants()->syncWithoutDetaching([
$tenantB->getKey() => ['role' => 'owner'],
]);
$runA = OperationRun::factory()->create([
'managed_environment_id' => (int) $tenantA->getKey(),
'workspace_id' => (int) $tenantA->workspace_id,
'type' => 'policy.sync',
'initiator_name' => 'TenantA',
]);
OperationRun::factory()->create([
'managed_environment_id' => (int) $tenantB->getKey(),
'workspace_id' => (int) $tenantB->workspace_id,
'type' => 'inventory_sync',
'initiator_name' => 'TenantB',
]);
Filament::setTenant(null, true);
$this->actingAs($user)
->withSession([
WorkspaceContext::SESSION_KEY => (int) $tenantA->workspace_id,
])
->get(route('admin.operations.view', ['workspace' => $tenantA->workspace, 'run' => (int) $runA->getKey()]))
->assertOk();
expect(Filament::getTenant())->toBeNull();
$this->actingAs($user)
->withSession([
WorkspaceContext::SESSION_KEY => (int) $tenantA->workspace_id,
])
->get(route('admin.operations.index', ['workspace' => $tenantA->workspace]))
->assertOk()
->assertSee('Policy sync')
->assertSee('Inventory sync')
->assertSee('TenantA')
->assertSee('TenantB');
});
it('shows explicit mismatch context on canonical run pages while keeping the current header tenant label', function (): void {
$runTenant = ManagedEnvironment::factory()->create([
'name' => 'Run ManagedEnvironment',
]);
[$user, $runTenant] = createUserWithTenant($runTenant, role: 'owner');
$currentTenant = ManagedEnvironment::factory()->create([
'name' => 'Current ManagedEnvironment',
'workspace_id' => (int) $runTenant->workspace_id,
]);
createUserWithTenant($currentTenant, $user, role: 'owner');
$run = OperationRun::factory()->create([
'managed_environment_id' => (int) $runTenant->getKey(),
'workspace_id' => (int) $runTenant->workspace_id,
'type' => 'policy.sync',
]);
Filament::setTenant($currentTenant, true);
$this->actingAs($user)
->withSession([
WorkspaceContext::SESSION_KEY => (int) $runTenant->workspace_id,
])
->get(route('admin.operations.view', ['workspace' => $runTenant->workspace, 'run' => (int) $run->getKey()]))
->assertOk()
->assertSee(__('localization.shell.environment_scope').': Current ManagedEnvironment')
->assertSee('Current environment context differs from this operation')
->assertSee('Operation environment: Run ManagedEnvironment.');
});
it('shows canonical workspace framing on canonical run pages with no selected tenant context', function (): void {
$tenant = ManagedEnvironment::factory()->create([
'name' => 'Workspace ManagedEnvironment',
]);
[$user, $tenant] = createUserWithTenant($tenant, role: 'owner');
$run = OperationRun::factory()->create([
'managed_environment_id' => (int) $tenant->getKey(),
'workspace_id' => (int) $tenant->workspace_id,
'type' => 'policy.sync',
]);
Filament::setTenant(null, true);
$this->actingAs($user)
->withSession([
WorkspaceContext::SESSION_KEY => (int) $tenant->workspace_id,
])
->get(route('admin.operations.view', ['workspace' => $tenant->workspace, 'run' => (int) $run->getKey()]))
->assertOk()
->assertSee(__('localization.shell.all_environments'))
->assertSee('Canonical workspace view')
->assertSee('No environment context is currently selected.');
});