fix(playwright): stabilize TagSuggestion select + scope tag-chip assertions (#27811)

* fix(playwright): stabilize TagSuggestion select + scope tag-chip assertions

- selectTagInTagSuggestion now resolves the listbox via the combobox's
  aria-controls attribute instead of grabbing the first [role="listbox"]
  on the page, which races with stale dropdowns left open by previous
  combobox interactions and caused click() to hang until test timeout.
- Scope post-select assertions to tags-container so they no longer match
  the still-rendered dropdown option in addition to the chip, which was
  triggering strict-mode violations.
- DataProducts spec now dismisses the domain dropdown before opening the
  tags dropdown so the two listboxes don't overlap.

* fix(playwright): simplify selectTagInTagSuggestion to anchor on tag-option testId

Drop the listbox/aria-controls indirection in favour of the unique
tag-option-${fqn} testId. click()'s actionability check covers what the
explicit visibility/scroll waits were doing, and tag option testIds are
unique across comboboxes so listbox scoping wasn't pulling its weight.

Author: siddhant1

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

@@ -479,6 +479,11 @@ test.describe('Data Products', () => {
       const domainOption = page.getByText(domain.data.displayName);
       await domainOption.waitFor({ state: 'visible' });
       await domainOption.click();
+      await page.keyboard.press('Escape');
+      await page
+        .locator('[role="listbox"]')
+        .first()
+        .waitFor({ state: 'hidden' });
     });
 
     await test.step('Search and select tag via TagSuggestion', async () => {
@@ -488,7 +493,10 @@ test.describe('Data Products', () => {
       });
 
       await expect(
-        page.getByTestId('add-domain-form').getByText(tag.data.displayName)
+        page
+          .getByTestId('add-domain-form')
+          .getByTestId('tags-container')
+          .getByText(tag.data.displayName)
       ).toBeVisible();
     });
 

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

@@ -1167,7 +1167,10 @@ test.describe('Domains', () => {
         });
 
         await expect(
-          page.getByTestId('add-domain-form').getByText(tag.data.displayName)
+          page
+            .getByTestId('add-domain-form')
+            .getByTestId('tags-container')
+            .getByText(tag.data.displayName)
         ).toBeVisible();
       });
 
@@ -1219,7 +1222,10 @@ test.describe('Domains', () => {
         });
 
         await expect(
-          page.getByTestId('add-domain-form').getByText(tag.data.displayName)
+          page
+            .getByTestId('add-domain-form')
+            .getByTestId('tags-container')
+            .getByText(tag.data.displayName)
         ).toBeVisible();
       });
 

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

@@ -616,6 +616,7 @@ export const selectTagInTagSuggestion = async (
   }
 ) => {
   const tagInput = page.getByRole('combobox', { name: 'Tags' });
+  const tagOption = page.getByTestId(`tag-option-${tagFqn}`);
 
   const tagSearchResponse = page.waitForResponse((response) => {
     const url = response.url();
@@ -626,13 +627,11 @@ export const selectTagInTagSuggestion = async (
     );
   });
 
+  await tagInput.click();
   await tagInput.fill(searchTerm);
   await tagSearchResponse;
 
-  await page.locator('[role="listbox"]').first().waitFor({ state: 'visible' });
-  const tagOption = page.getByTestId(`tag-option-${tagFqn}`);
-  await tagOption.waitFor({ state: 'visible' });
   await tagOption.click();
   await page.keyboard.press('Escape');
-  await page.locator('[role="listbox"]').first().waitFor({ state: 'hidden' });
+  await tagOption.waitFor({ state: 'hidden' });
 };