From a3da7ae8e9a561e64e78cb79eda506b49530d358 Mon Sep 17 00:00:00 2001 From: Harsh Vador Date: Wed, 22 Apr 2026 19:28:51 +0530 Subject: [PATCH 1/3] refactor(playwright): reuse shared explore navigation helper in SearchExport spec --- .../ui/playwright/e2e/Features/SearchExport.spec.ts | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/SearchExport.spec.ts b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/SearchExport.spec.ts index 05f122531dc4..a6de3fdffb49 100644 --- a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/SearchExport.spec.ts +++ b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/SearchExport.spec.ts @@ -11,8 +11,8 @@ * limitations under the License. */ -import { expect, Page } from '@playwright/test'; -import { redirectToHomePage } from '../../utils/common'; +import { expect } from '@playwright/test'; +import { redirectToExplorePage } from '../../utils/common'; import { waitForAllLoadersToDisappear } from '../../utils/entity'; import { countCsvResponseRows, @@ -22,15 +22,9 @@ import { } from '../../utils/explore'; import { test } from '../fixtures/pages'; -const navigateToExplorePage = async (page: Page) => { - await redirectToHomePage(page); - await page.getByTestId('app-bar-item-explore').click(); - await expect(page.getByTestId('explore-page')).toBeVisible(); -}; - test.describe('Search Export', { tag: ['@Features', '@Discovery'] }, () => { test.beforeEach(async ({ page }) => { - await navigateToExplorePage(page); + await redirectToExplorePage(page); }); test('Export button opens scope modal with correct options', async ({ From dabbb014a0561c38cd320691b276c3385471fca1 Mon Sep 17 00:00:00 2001 From: Harsh Vador Date: Wed, 22 Apr 2026 20:43:49 +0530 Subject: [PATCH 2/3] fix(e2e): normalize sample_data service displayName before SearchExport tests --- .../e2e/Features/SearchExport.spec.ts | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/SearchExport.spec.ts b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/SearchExport.spec.ts index a6de3fdffb49..d27db59c7f8e 100644 --- a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/SearchExport.spec.ts +++ b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/SearchExport.spec.ts @@ -12,6 +12,7 @@ */ import { expect } from '@playwright/test'; +import { performAdminLogin } from '../../utils/admin'; import { redirectToExplorePage } from '../../utils/common'; import { waitForAllLoadersToDisappear } from '../../utils/entity'; import { @@ -23,6 +24,22 @@ import { import { test } from '../fixtures/pages'; test.describe('Search Export', { tag: ['@Features', '@Discovery'] }, () => { + test.beforeAll(async ({ browser }) => { + const { apiContext, afterAction } = await performAdminLogin(browser); + + const serviceRes = await apiContext.get( + '/api/v1/services/databaseServices/name/sample_data' + ); + const service = await serviceRes.json(); + + await apiContext.patch(`/api/v1/services/databaseServices/${service.id}`, { + data: [{ op: 'replace', path: '/displayName', value: 'sample_data' }], + headers: { 'Content-Type': 'application/json-patch+json' }, + }); + + await afterAction(); + }); + test.beforeEach(async ({ page }) => { await redirectToExplorePage(page); }); From b1af65c4d0ec34eb00df17dc1bd1cbe8a9fb776c Mon Sep 17 00:00:00 2001 From: Harsh Vador Date: Wed, 22 Apr 2026 20:47:47 +0530 Subject: [PATCH 3/3] add displayName check --- .../playwright/e2e/Features/SearchExport.spec.ts | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/SearchExport.spec.ts b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/SearchExport.spec.ts index d27db59c7f8e..45e506dc79b1 100644 --- a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/SearchExport.spec.ts +++ b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/SearchExport.spec.ts @@ -31,13 +31,17 @@ test.describe('Search Export', { tag: ['@Features', '@Discovery'] }, () => { '/api/v1/services/databaseServices/name/sample_data' ); const service = await serviceRes.json(); + if (service.displayName) { + await apiContext.patch( + `/api/v1/services/databaseServices/${service.id}`, + { + data: [{ op: 'replace', path: '/displayName', value: 'sample_data' }], + headers: { 'Content-Type': 'application/json-patch+json' }, + } + ); - await apiContext.patch(`/api/v1/services/databaseServices/${service.id}`, { - data: [{ op: 'replace', path: '/displayName', value: 'sample_data' }], - headers: { 'Content-Type': 'application/json-patch+json' }, - }); - - await afterAction(); + await afterAction(); + } }); test.beforeEach(async ({ page }) => {