From 80c5cf491d8c2267c6f51886216a7332c86cdb77 Mon Sep 17 00:00:00 2001 From: Aniket Katkar Date: Fri, 5 Jun 2026 15:46:51 +0530 Subject: [PATCH 1/3] fix: resolve tooltip overlap issue in entity selection by adding hover and force click to playright tests --- .../src/main/resources/ui/playwright/utils/entity.ts | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/openmetadata-ui/src/main/resources/ui/playwright/utils/entity.ts b/openmetadata-ui/src/main/resources/ui/playwright/utils/entity.ts index 4e4ff193d7df..2023762f60d3 100644 --- a/openmetadata-ui/src/main/resources/ui/playwright/utils/entity.ts +++ b/openmetadata-ui/src/main/resources/ui/playwright/utils/entity.ts @@ -81,7 +81,16 @@ export const visitEntityPage = async (data: { response.url().includes('exclude_source_fields') ); - await page.getByTestId(dataTestId).getByTestId('data-name').click(); + // Adding a failsafe for the operation below to avoid a tooltip overlap issue + // over the option is causing playwright failure + // 1. To first hover over the option to move mouse away from tooltip trigger element + // 2. In case the tooltip is still there, force click on the option-. + await page.getByTestId(dataTestId).getByTestId('data-name').hover(); + await page + .getByTestId(dataTestId) + .getByTestId('data-name') + // eslint-disable-next-line playwright/no-force-option + .click({ force: true }); await waitForAllLoadersToDisappear(page); await page.getByTestId('searchBox').clear(); }; From 6fe880c2e0d66c4ae6a4a8ed53efada7d7ee52d2 Mon Sep 17 00:00:00 2001 From: Aniket Katkar Date: Mon, 8 Jun 2026 10:05:00 +0530 Subject: [PATCH 2/3] Fix AuditLogs flakiness --- .../main/resources/ui/playwright/e2e/Pages/AuditLogs.spec.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/AuditLogs.spec.ts b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/AuditLogs.spec.ts index 21522e61f1d3..68623d3748fb 100644 --- a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/AuditLogs.spec.ts +++ b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/AuditLogs.spec.ts @@ -284,7 +284,7 @@ test.describe('Audit Logs Page', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => { const userSearchResponse = page.waitForResponse( (response) => response.url().includes('/api/v1/search/query') && - response.url().includes('index=user') + response.url().includes('q=admin') ); await searchInput.fill('admin'); await userSearchResponse; From 906d81e7a7a370925932d92c7b51e0c0dab13dd1 Mon Sep 17 00:00:00 2001 From: Aniket Katkar Date: Mon, 8 Jun 2026 10:22:35 +0530 Subject: [PATCH 3/3] Worked on comments --- .../resources/ui/playwright/e2e/Pages/AuditLogs.spec.ts | 1 + .../src/main/resources/ui/playwright/utils/entity.ts | 8 ++++---- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/AuditLogs.spec.ts b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/AuditLogs.spec.ts index 68623d3748fb..9aa95e449459 100644 --- a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/AuditLogs.spec.ts +++ b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/AuditLogs.spec.ts @@ -284,6 +284,7 @@ test.describe('Audit Logs Page', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => { const userSearchResponse = page.waitForResponse( (response) => response.url().includes('/api/v1/search/query') && + response.url().includes('index=user') && response.url().includes('q=admin') ); await searchInput.fill('admin'); diff --git a/openmetadata-ui/src/main/resources/ui/playwright/utils/entity.ts b/openmetadata-ui/src/main/resources/ui/playwright/utils/entity.ts index 2023762f60d3..0e48f8e9a0dd 100644 --- a/openmetadata-ui/src/main/resources/ui/playwright/utils/entity.ts +++ b/openmetadata-ui/src/main/resources/ui/playwright/utils/entity.ts @@ -81,10 +81,10 @@ export const visitEntityPage = async (data: { response.url().includes('exclude_source_fields') ); - // Adding a failsafe for the operation below to avoid a tooltip overlap issue - // over the option is causing playwright failure - // 1. To first hover over the option to move mouse away from tooltip trigger element - // 2. In case the tooltip is still there, force click on the option-. + // Adding a failsafe for the operation below to avoid a tooltip overlap issue. + // A tooltip over the option can cause Playwright click failures: + // 1) Hover over the option to move the mouse away from the tooltip trigger element. + // 2) If the tooltip is still present, force-click the option. await page.getByTestId(dataTestId).getByTestId('data-name').hover(); await page .getByTestId(dataTestId)