fix: tests

Author: Raubzeug

tests/suites/tenant/diagnostics/tabs/access.test.ts

@@ -72,7 +72,6 @@ test.describe('Diagnostics Access tab', async () => {
             database: '/local',
             tenantPage: 'diagnostics',
             diagnosticsTab: 'access',
-            summaryTab: 'acl',
         };
         const tenantPage = new TenantPage(page);
         await tenantPage.goto(pageQueryParams);
@@ -91,7 +90,6 @@ test.describe('Diagnostics Access tab', async () => {
             database: '/local',
             tenantPage: 'diagnostics',
             diagnosticsTab: 'access',
-            summaryTab: 'acl',
         };
         const tenantPage = new TenantPage(page);
         await tenantPage.goto(pageQueryParams);

tests/suites/tenant/summary/ObjectSummary.ts

@@ -8,7 +8,6 @@ import type {RowTableAction} from './types';
 
 export enum ObjectSummaryTab {
     Overview = 'Overview',
-    ACL = 'ACL',
     Schema = 'Schema',
 }
 export class ObjectSummary {
@@ -113,37 +112,6 @@ export class ObjectSummary {
         await this.createDirectoryModal.waitFor({state: 'hidden', timeout: VISIBILITY_TIMEOUT});
     }
 
-    async waitForAclVisible() {
-        // In the new UI, the ACL tab shows a redirect message instead of the actual ACL content
-        const redirectMessage = this.page.locator('text=Section was moved to Diagnostics');
-        await redirectMessage.waitFor({state: 'visible', timeout: VISIBILITY_TIMEOUT});
-        return true;
-    }
-
-    async getRedirectMessage(): Promise<string | null> {
-        const redirectMessage = this.page.locator('text=Section was moved to Diagnostics');
-        if (await redirectMessage.isVisible()) {
-            return redirectMessage.textContent();
-        }
-        return null;
-    }
-
-    async hasOpenInDiagnosticsButton(): Promise<boolean> {
-        try {
-            const diagnosticsButton = this.page.getByRole('button', {name: 'Open in Diagnostics'});
-            await diagnosticsButton.waitFor({state: 'visible', timeout: VISIBILITY_TIMEOUT});
-            return true;
-        } catch (error) {
-            console.error('Open in Diagnostics button not visible:', error);
-            return false;
-        }
-    }
-
-    async clickOpenInDiagnosticsButton(): Promise<void> {
-        const diagnosticsButton = this.page.getByRole('button', {name: 'Open in Diagnostics'});
-        await diagnosticsButton.click();
-    }
-
     async isTreeVisible() {
         await this.tree.waitFor({state: 'visible', timeout: VISIBILITY_TIMEOUT});
         return true;

tests/suites/tenant/summary/objectSummary.test.ts

@@ -269,57 +269,4 @@ test.describe('Object Summary', async () => {
         await objectSummary.expandSummary();
         await expect(objectSummary.isSummaryCollapsed()).resolves.toBe(false);
     });
-
-    test('ACL tab shows redirect message and link to Diagnostics', async ({page}) => {
-        // Define the URL parameters
-        const pageQueryParams = {
-            schema: '/local',
-            database: '/local',
-            summaryTab: 'acl',
-            tenantPage: 'query',
-        };
-
-        // Navigate to the page
-        const tenantPage = new TenantPage(page);
-        await tenantPage.goto(pageQueryParams);
-
-        // Get the ObjectSummary instance
-        const objectSummary = new ObjectSummary(page);
-
-        // Verify the ACL tab is selected
-        await objectSummary.clickTab(ObjectSummaryTab.ACL);
-
-        // Wait for the ACL content to be visible
-        await objectSummary.waitForAclVisible();
-
-        // Check for the redirect message
-        const redirectMessage = await objectSummary.getRedirectMessage();
-        expect(redirectMessage).toContain('Section was moved to Diagnostics');
-
-        // Check for the "Open in Diagnostics" button
-        const hasButton = await objectSummary.hasOpenInDiagnosticsButton();
-        expect(hasButton).toBe(true);
-
-        // Click the button and verify the URL
-        await objectSummary.clickOpenInDiagnosticsButton();
-
-        // Verify the URL contains the expected parameters
-        const expectedUrlParams = new URLSearchParams({
-            tenantPage: 'diagnostics',
-            diagnosticsTab: 'access',
-            summaryTab: 'acl',
-            schema: '/local',
-            database: '/local',
-        });
-
-        // Get the current URL and parse its parameters
-        const currentUrl = page.url();
-        const currentUrlObj = new URL(currentUrl);
-        const currentParams = currentUrlObj.searchParams;
-
-        // Verify each expected parameter is in the URL
-        for (const [key, value] of expectedUrlParams.entries()) {
-            expect(currentParams.get(key)).toBe(value);
-        }
-    });
 });