Minor: supporting PR for UI/UX improvement in DQ (#18663)

* Minor: supporting PR for UI/UX improvement in DQ

* updated feedback

* move incidents tab to data observabilty tab

* fixed failing unit test

* fixed failing unit test

* fixed AUT failure!

* fixed failing aut

* minor fixes

* minor fix

* updated icon

* fixed failing unit test

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

@@ -392,12 +392,16 @@ test.describe('Incident Manager', PLAYWRIGHT_INGESTION_TAG_OBJ, () => {
 
   test('Validate Incident Tab in Entity details page', async ({ page }) => {
     const testCases = table1.testCasesResponseData;
-    await table1.visitEntityPage(page);
+    await visitProfilerTab(page, table1);
+
     const incidentListResponse = page.waitForResponse(
       `/api/v1/dataQuality/testCases/testCaseIncidentStatus?*originEntityFQN=${table1.entityResponseData?.['fullyQualifiedName']}*`
     );
 
-    await page.click('[data-testid="incidents"]');
+    await page
+      .getByTestId('profiler-tab-left-panel')
+      .getByText('Incidents')
+      .click();
     await incidentListResponse;
 
     for (const testCase of testCases) {

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

@@ -33,7 +33,7 @@ test(
     const pipelineName = `test suite pipeline 2`;
 
     await test.step('Create a new pipeline', async () => {
-      await page.getByText('Profiler & Data Quality').click();
+      await page.getByText('Data Observability').click();
       await page
         .getByRole('menuitem', {
           name: 'Table Profile',
@@ -204,7 +204,7 @@ test(
       testCaseNames
     );
     await table.visitEntityPage(page);
-    await page.getByText('Profiler & Data Quality').click();
+    await page.getByText('Data Observability').click();
     await page.getByRole('menuitem', { name: 'Data Quality' }).click();
 
     await page.getByRole('tab', { name: 'Pipeline' }).click();

openmetadata-ui/src/main/resources/ui/playwright/e2e/Flow/Tour.spec.ts

@@ -105,7 +105,7 @@ test.describe('Tour should work properly', () => {
 
     await expect(page.locator(`[data-tour-elem="badge"]`)).toHaveText('12');
 
-    await expect(page.getByText('Profiler & Data Quality')).toBeVisible();
+    await expect(page.getByText('Data Observability')).toBeVisible();
 
     // step 11
     await page.locator('[data-tour-elem="right-arrow"]').click();

openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/TestCases.spec.ts

@@ -43,7 +43,7 @@ test('Table difference test case', async ({ page }) => {
       table1.entityResponseData?.['fullyQualifiedName']
     )}/tableProfile/latest`
   );
-  await page.getByText('Profiler & Data Quality').click();
+  await page.getByText('Data Observability').click();
   await profileResponse;
   await page.getByRole('menuitem', { name: 'Table Profile' }).click();
 
@@ -170,7 +170,7 @@ test('Custom SQL Query', async ({ page }) => {
       table.entityResponseData?.['fullyQualifiedName']
     )}/tableProfile/latest`
   );
-  await page.getByText('Profiler & Data Quality').click();
+  await page.getByText('Data Observability').click();
   await profileResponse;
   await page.getByRole('menuitem', { name: 'Table Profile' }).click();
 

openmetadata-ui/src/main/resources/ui/playwright/utils/permission.ts

@@ -105,7 +105,7 @@ export const validateViewPermissions = async (
   await page.waitForLoadState('domcontentloaded');
   await checkNoPermissionPlaceholder(
     page,
-    /Profiler & Data Quality/,
+    /Data Observability/,
     permission?.viewTests
   );
   await page.click('[data-testid="lineage"]');