From 200374407c1ec46255fcb2b06891f820162fdb48 Mon Sep 17 00:00:00 2001 From: Karan Hotchandani <33024356+karanh37@users.noreply.github.com> Date: Thu, 19 Mar 2026 12:46:10 +0530 Subject: [PATCH 1/3] run subdomain pagination only in oss and not on aut --- .../ui/playwright/e2e/Pages/SubDomainPagination.spec.ts | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/SubDomainPagination.spec.ts b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/SubDomainPagination.spec.ts index 2fa020bc5ed..5cf745e83a0 100644 --- a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/SubDomainPagination.spec.ts +++ b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/SubDomainPagination.spec.ts @@ -29,7 +29,8 @@ const domain = new Domain(); const subDomains: SubDomain[] = []; const SUBDOMAIN_COUNT = 60; -test.describe('SubDomain Pagination', () => { +if (process.env.PLAYWRIGHT_IS_OSS) { + test.describe('SubDomain Pagination', () => { test.slow(true); test.beforeAll('Setup domain and subdomains', async ({ browser }) => { @@ -85,7 +86,7 @@ test.describe('SubDomain Pagination', () => { await test.step('Navigate to subdomains tab and verify initial data load', async () => { const subDomainRes = page.waitForResponse( - '/api/v1/search/query?q=&index=domain&from=0&size=9*' + '/api/v1/search/query?q=&index=domain_search_index&from=0&size=9*' ); await page.getByTestId('subdomains').click(); await subDomainRes; @@ -139,3 +140,4 @@ test.describe('SubDomain Pagination', () => { }); }); }); +} From 70b71802f1b60a4e93aa4108b2b1ab276b0232b1 Mon Sep 17 00:00:00 2001 From: Karan Hotchandani <33024356+karanh37@users.noreply.github.com> Date: Mon, 23 Mar 2026 17:51:17 +0530 Subject: [PATCH 2/3] fix test --- .../e2e/Pages/SubDomainPagination.spec.ts | 205 +++++++++--------- 1 file changed, 103 insertions(+), 102 deletions(-) diff --git a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/SubDomainPagination.spec.ts b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/SubDomainPagination.spec.ts index 5cf745e83a0..4d31e7c58ab 100644 --- a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/SubDomainPagination.spec.ts +++ b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/SubDomainPagination.spec.ts @@ -29,115 +29,116 @@ const domain = new Domain(); const subDomains: SubDomain[] = []; const SUBDOMAIN_COUNT = 60; -if (process.env.PLAYWRIGHT_IS_OSS) { - test.describe('SubDomain Pagination', () => { +test.describe('SubDomain Pagination', () => { test.slow(true); - test.beforeAll('Setup domain and subdomains', async ({ browser }) => { - test.slow(true); + if(process.env.PLAYWRIGHT_IS_OSS) { + test.beforeAll('Setup domain and subdomains', async ({ browser }) => { + test.slow(true); - const { apiContext, afterAction } = await createNewPage(browser); + const { apiContext, afterAction } = await createNewPage(browser); - await domain.create(apiContext); + await domain.create(apiContext); - const createPromises = []; - for (let i = 1; i <= SUBDOMAIN_COUNT; i++) { - const subDomain = new SubDomain( - domain, - `TestSubDomain${i.toString().padStart(2, '0')}` - ); - subDomains.push(subDomain); - createPromises.push(subDomain.create(apiContext)); - } + const createPromises = []; + for (let i = 1; i <= SUBDOMAIN_COUNT; i++) { + const subDomain = new SubDomain( + domain, + `TestSubDomain${i.toString().padStart(2, '0')}` + ); + subDomains.push(subDomain); + createPromises.push(subDomain.create(apiContext)); + } - await Promise.all(createPromises); + await Promise.all(createPromises); - await afterAction(); - }); - - test.afterAll('Cleanup', async ({ browser }) => { - test.slow(); - - const { apiContext, afterAction } = await createNewPage(browser); - await domain.delete(apiContext); - await afterAction(); - }); - - test.beforeEach('Navigate to domain page', async ({ page }) => { - await redirectToHomePage(page); - await sidebarClick(page, SidebarItem.DOMAIN); - await waitForAllLoadersToDisappear(page); - }); - - test('Verify subdomain count and pagination functionality', async ({ - page, - }) => { - await selectDomain(page, domain.data); - - await waitForAllLoadersToDisappear(page); - - await test.step('Verify subdomain count in tab label', async () => { - const subDomainsTab = page.getByTestId('subdomains'); - - await expect(subDomainsTab).toBeVisible(); - - await expect(subDomainsTab).toContainText('60'); - }); - - await test.step('Navigate to subdomains tab and verify initial data load', async () => { - const subDomainRes = page.waitForResponse( - '/api/v1/search/query?q=&index=domain_search_index&from=0&size=9*' - ); - await page.getByTestId('subdomains').click(); - await subDomainRes; - await waitForAllLoadersToDisappear(page); - - await expect(page.locator('table')).toBeVisible(); - - await expect(page.locator('[data-testid="pagination"]')).toBeVisible(); - - // Verify current page shows page 1 - const tableRows = page.locator('table tbody tr'); - - await expect(tableRows).toHaveCount(9); - }); - - await test.step('Test pagination navigation', async () => { - const nextPageResponse = page.waitForResponse('/api/v1/search/query?*'); - await page.locator('[data-testid="next"]').click(); - await nextPageResponse; - - const prevPageResponse = page.waitForResponse('/api/v1/search/query?*'); - await page.locator('[data-testid="previous"]').click(); - await prevPageResponse; - }); - - await test.step('Create new subdomain and verify count updates', async () => { - const subDomain = new SubDomain(domain); - await createSubDomain(page, subDomain.data); - - await redirectToHomePage(page); - - await sidebarClick(page, SidebarItem.DOMAIN); - - await selectDomain(page, domain.data); - - const subDomainsTab = page.getByTestId('subdomains'); - - await expect(subDomainsTab).toContainText('61'); - - const { apiContext, afterAction } = await getApiContext(page); - - const response = await apiContext.get( - '/api/v1/domains/name/' + - encodeURIComponent(`"${domain.data.name}"."NewTestSubDomain"`) - ); - const subDomainData = await response.json(); - await apiContext.delete( - `/api/v1/domains/${subDomainData.id}?hardDelete=true` - ); await afterAction(); }); - }); + + test.afterAll('Cleanup', async ({ browser }) => { + test.slow(); + + const { apiContext, afterAction } = await createNewPage(browser); + await domain.delete(apiContext); + await afterAction(); + }); + + test.beforeEach('Navigate to domain page', async ({ page }) => { + await redirectToHomePage(page); + await sidebarClick(page, SidebarItem.DOMAIN); + await waitForAllLoadersToDisappear(page); + }); + + test('Verify subdomain count and pagination functionality', async ({ + page, + }) => { + await selectDomain(page, domain.data); + + await waitForAllLoadersToDisappear(page); + + await test.step('Verify subdomain count in tab label', async () => { + const subDomainsTab = page.getByTestId('subdomains'); + + await expect(subDomainsTab).toBeVisible(); + + await expect(subDomainsTab).toContainText('60'); + }); + + await test.step('Navigate to subdomains tab and verify initial data load', async () => { + const subDomainRes = page.waitForResponse( + '/api/v1/search/query?q=&index=domain_search_index&from=0&size=9*' + ); + await page.getByTestId('subdomains').click(); + await subDomainRes; + await waitForAllLoadersToDisappear(page); + + await expect(page.locator('table')).toBeVisible(); + + await expect(page.locator('[data-testid="pagination"]')).toBeVisible(); + + // Verify current page shows page 1 + const tableRows = page.locator('table tbody tr'); + + await expect(tableRows).toHaveCount(9); + }); + + await test.step('Test pagination navigation', async () => { + const nextPageResponse = page.waitForResponse('/api/v1/search/query?*'); + await page.locator('[data-testid="next"]').click(); + await nextPageResponse; + + const prevPageResponse = page.waitForResponse('/api/v1/search/query?*'); + await page.locator('[data-testid="previous"]').click(); + await prevPageResponse; + }); + + await test.step('Create new subdomain and verify count updates', async () => { + const subDomain = new SubDomain(domain); + await createSubDomain(page, subDomain.data); + + await redirectToHomePage(page); + + await sidebarClick(page, SidebarItem.DOMAIN); + + await selectDomain(page, domain.data); + + const subDomainsTab = page.getByTestId('subdomains'); + + await expect(subDomainsTab).toContainText('61'); + + const { apiContext, afterAction } = await getApiContext(page); + + const response = await apiContext.get( + '/api/v1/domains/name/' + + encodeURIComponent(`"${domain.data.name}"."NewTestSubDomain"`) + ); + const subDomainData = await response.json(); + await apiContext.delete( + `/api/v1/domains/${subDomainData.id}?hardDelete=true` + ); + await afterAction(); + }); + }); + } }); -} + From 988d3acfa0a050a60f2b0ecb3c8de9c5d7451b60 Mon Sep 17 00:00:00 2001 From: Karan Hotchandani <33024356+karanh37@users.noreply.github.com> Date: Mon, 23 Mar 2026 19:18:00 +0530 Subject: [PATCH 3/3] fix index --- .../ui/playwright/e2e/Pages/SubDomainPagination.spec.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/SubDomainPagination.spec.ts b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/SubDomainPagination.spec.ts index 4d31e7c58ab..0ba2589e019 100644 --- a/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/SubDomainPagination.spec.ts +++ b/openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/SubDomainPagination.spec.ts @@ -86,7 +86,7 @@ test.describe('SubDomain Pagination', () => { await test.step('Navigate to subdomains tab and verify initial data load', async () => { const subDomainRes = page.waitForResponse( - '/api/v1/search/query?q=&index=domain_search_index&from=0&size=9*' + '/api/v1/search/query?q=&index=domain&from=0&size=9*' ); await page.getByTestId('subdomains').click(); await subDomainRes;