Fixes #22399: Add persona customization for service details page

openmetadata-spec/src/main/resources/json/schema/system/ui/page.json

@@ -37,7 +37,8 @@
         "Directory",
         "File",
         "Spreadsheet",
-        "Worksheet"
+        "Worksheet",
+        "Service"
       ]
     }
   },

openmetadata-ui/src/main/resources/ui/playwright/constant/customizeDetail.ts

@@ -80,6 +80,8 @@ export enum EntityTabs {
   SUBDOMAINS = 'subdomains',
   CONTRACT = 'contract',
   ER_DIAGRAM = 'erDiagram',
+  FILES = 'files',
+  SPREADSHEETS = 'spreadsheets',
 }
 
 export const TABLE_DEFAULT_TABS = [
@@ -246,3 +248,13 @@ export const GLOSSARY_TERM_DEFAULT_TABS = [
   EntityTabs.ACTIVITY_FEED,
   EntityTabs.CUSTOM_PROPERTIES,
 ];
+
+export const SERVICE_DEFAULT_TABS = [
+  EntityTabs.INSIGHTS,
+  EntityTabs.DETAILS,
+  EntityTabs.DATA_Model,
+  EntityTabs.FILES,
+  EntityTabs.SPREADSHEETS,
+  EntityTabs.AGENTS,
+  EntityTabs.CONNECTION,
+];

openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/Pagination.spec.ts

@@ -241,12 +241,34 @@ test.describe('Pagination Tests', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => {
     test('should test pagination on Service Databases page', async ({
       page,
     }) => {
-      await page.goto(`/service/databaseServices/${databaseFqn}/databases`);
-      await testPaginationNavigation(
-        page,
-        '/api/v1/databases',
-        '[data-testid="service-children-table"]'
-      );
+      const searchApiMatcher = (response: { url: () => string }) =>
+        response.url().includes('/api/v1/search/query');
+
+      const page1ResponsePromise = page.waitForResponse(searchApiMatcher);
+      await page.goto(`/service/databaseServices/${databaseFqn}/details`);
+      await page.locator('[data-testid="service-children-table"]').waitFor({
+        state: 'visible',
+      });
+      const page1Response = await page1ResponsePromise;
+      expect(page1Response.status()).toBe(200);
+      await waitForAllLoadersToDisappear(page);
+
+      await expect(page.getByTestId('previous')).toBeDisabled();
+      const nextButton = page.getByTestId('next');
+      await expect(nextButton).toBeEnabled();
+
+      const [page2Response] = await Promise.all([
+        page.waitForResponse(searchApiMatcher),
+        nextButton.click(),
+      ]);
+      expect(page2Response.status()).toBe(200);
+      await waitForAllLoadersToDisappear(page);
+
+      await expect(page.getByTestId('previous')).toBeEnabled();
+      const paginationText = page.locator('[data-testid="page-indicator"]');
+      await expect(paginationText).toBeVisible();
+      const page2Content = await paginationText.textContent();
+      expect(page2Content).toMatch(/2\s*of\s*\d+/);
 
       const responsePromise = page.waitForResponse((response) =>
         response
@@ -262,21 +284,22 @@ test.describe('Pagination Tests', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => {
         state: 'detached',
       });
 
-      const databaseResponsePromise = page.waitForResponse((response) =>
-        response.url().includes('/api/v1/databases')
-      );
-      await page.getByTestId('databases').click();
+      const databaseResponsePromise = page.waitForResponse(searchApiMatcher);
+      await page.getByTestId('details').click();
       const response2 = await databaseResponsePromise;
       expect(response2.status()).toBe(200);
       await waitForAllLoadersToDisappear(page);
       await page.getByTestId('table-container').waitFor({
         state: 'visible',
       });
 
-      const paginationText = page.locator('[data-testid="page-indicator"]');
-      await expect(paginationText).toBeVisible();
+      const paginationTextAfterSwitch = page.locator(
+        '[data-testid="page-indicator"]'
+      );
+      await expect(paginationTextAfterSwitch).toBeVisible();
 
-      const paginationTextContent = await paginationText.textContent();
+      const paginationTextContent =
+        await paginationTextAfterSwitch.textContent();
       expect(paginationTextContent).toMatch(/1\s*of\s*\d+/);
     });
     test('should test Service Database Tables complete flow with search', async ({
@@ -286,8 +309,8 @@ test.describe('Pagination Tests', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => {
 
       await testCompletePaginationWithSearch({
         page,
-        baseUrl: `/service/databaseServices/${databaseFqn}/databases`,
-        normalApiPattern: '/api/v1/databases',
+        baseUrl: `/service/databaseServices/${databaseFqn}/details`,
+        normalApiPattern: '/api/v1/search/query',
         searchApiPattern: '/api/v1/search/query',
         searchTestTerm: 'pw',
         searchParamName: 'schema',
@@ -673,9 +696,9 @@ test.describe('Pagination Tests', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => {
         '[data-testid="data-models-table"]'
       );
       const responsePromise = page.waitForResponse((response) =>
-        response.url().includes('/api/v1/dashboard/datamodels')
+        response.url().includes('/api/v1/search/query')
       );
-      await page.getByTestId('dashboards').click();
+      await page.getByTestId('details').click();
       const response = await responsePromise;
       expect(response.status()).toBe(200);
       await waitForAllLoadersToDisappear(page);
@@ -736,14 +759,36 @@ test.describe('Pagination Tests', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => {
     });
 
     test('should test Directories normal pagination', async ({ page }) => {
+      const searchApiMatcher = (response: { url: () => string }) =>
+        response.url().includes('/api/v1/search/query');
+
+      const page1ResponsePromise = page.waitForResponse(searchApiMatcher);
       await page.goto(
-        `/service/driveServices/${serviceFqn}/directories?pageSize=15`
-      );
-      await testPaginationNavigation(
-        page,
-        '/api/v1/drives/directories',
-        '[data-testid="service-children-table"]'
+        `/service/driveServices/${serviceFqn}/details?pageSize=15`
       );
+      await page.locator('[data-testid="service-children-table"]').waitFor({
+        state: 'visible',
+      });
+      const page1Response = await page1ResponsePromise;
+      expect(page1Response.status()).toBe(200);
+      await waitForAllLoadersToDisappear(page);
+
+      await expect(page.getByTestId('previous')).toBeDisabled();
+      const nextButton = page.getByTestId('next');
+      await expect(nextButton).toBeEnabled();
+
+      const [page2Response] = await Promise.all([
+        page.waitForResponse(searchApiMatcher),
+        nextButton.click(),
+      ]);
+      expect(page2Response.status()).toBe(200);
+      await waitForAllLoadersToDisappear(page);
+
+      await expect(page.getByTestId('previous')).toBeEnabled();
+      const paginationText = page.locator('[data-testid="page-indicator"]');
+      await expect(paginationText).toBeVisible();
+      const page2Content = await paginationText.textContent();
+      expect(page2Content).toMatch(/2\s*of\s*\d+/);
     });
 
     test('should test Directories complete flow with search', async ({
@@ -753,8 +798,8 @@ test.describe('Pagination Tests', PLAYWRIGHT_BASIC_TEST_TAG_OBJ, () => {
 
       await testCompletePaginationWithSearch({
         page,
-        baseUrl: `/service/driveServices/${serviceFqn}/directories?showDeletedTables=false`,
-        normalApiPattern: '/api/v1/drives/directories',
+        baseUrl: `/service/driveServices/${serviceFqn}/details?showDeletedTables=false`,
+        normalApiPattern: '/api/v1/search/query',
         searchApiPattern: '/api/v1/search/query',
         searchTestTerm: 'pw',
         searchParamName: 'schema',