fix(test): unflake GlossaryPermissions team-based test (#27422)

* fix(test): assign role to team in P-11 team-based permissions test

The "Team-based permissions work correctly" test created a team and added
testUser to it, but never attached the new role to the team — so the user
inherited no permissions from team membership. The test relied on default
Organization conditional rules, which the frontend treats as no-permission
(only Access.Allow becomes truthy in PermissionsUtils). The glossary page
then rendered the no-permission placeholder and never fired
/api/v1/glossaries?fields=*, hanging visitGlossaryPage's waitForResponse
for the full 30s toPass budget.

Patch the team's defaultRoles after initializePermissions so the user
actually inherits Allow via team membership, matching the test's intent.

Local repro (full file × 10 repeats × 4 workers, retries=0):
- Before: 10/10 P-11 failures
- After:  10/10 P-11 pass, 93/93 overall pass

* fix(test): isolate P-11 to a dedicated user/team

Avoid mutating the file-shared testUser. Previously the fix added testUser
to a temporary team and attached an Allow role; if cleanup failed, the
elevated permissions would leak into subsequent tests sharing the same
worker.

Now P-11 creates its own UserClass + page, runs the team-permission
verification in isolation, and tears down the user, team, role, and policy
at the end. Other tests in the file see no state change from this test
beyond what initializePermissions/cleanupPermissions already do.

Local verification (full file × 10 repeats × 4 workers, retries=0):
93/93 pass, P-11 10/10.

---------

Co-authored-by: Siddhant <siddhant@MacBook-Pro-621.local>

Author: siddhant1

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

@@ -354,15 +354,14 @@ test.describe('Glossary Permissions', () => {
   });
 
   // P-11: Team-based permissions work correctly
-  test('Team-based permissions work correctly', async ({
-    testUserPage,
-    page,
-  }) => {
+  test('Team-based permissions work correctly', async ({ browser, page }) => {
     test.slow(true);
 
     const { apiContext } = await getApiContext(page);
 
-    // Create a team and add testUser to it
+    const teamUser = new UserClass();
+    await teamUser.create(apiContext);
+
     const teamName = `TestTeam${Date.now()}`;
     const teamResponse = await apiContext.post('/api/v1/teams', {
       data: {
@@ -375,14 +374,13 @@ test.describe('Glossary Permissions', () => {
     const teamData = await teamResponse.json();
     const teamId = teamData.id;
 
-    // Add user to team
-    await apiContext.patch(`/api/v1/teams/${teamData.id}`, {
+    await apiContext.patch(`/api/v1/teams/${teamId}`, {
       data: [
         {
           op: 'add',
           path: '/users/0',
           value: {
-            id: testUser.responseData.id,
+            id: teamUser.responseData.id,
             type: 'user',
           },
         },
@@ -392,30 +390,49 @@ test.describe('Glossary Permissions', () => {
       },
     });
 
-    // Set up permissions with team as the principal
-    await initializePermissions(page, 'allow', [
+    const { role: teamRole } = await initializePermissions(page, 'allow', [
       'EditDescription',
       'EditOwners',
     ]);
 
-    // Login as test user and verify permissions inherited from team
-    await expect(async () => {
-      await glossary.visitEntityPage(testUserPage);
-      await waitForAllLoadersToDisappear(testUserPage);
+    await apiContext.patch(`/api/v1/teams/${teamId}`, {
+      data: [
+        {
+          op: 'add',
+          path: '/defaultRoles/0',
+          value: {
+            id: teamRole.responseData.id,
+            type: 'role',
+            name: teamRole.responseData.name,
+          },
+        },
+      ],
+      headers: {
+        'Content-Type': 'application/json-patch+json',
+      },
+    });
 
-      const glossaryHeader = testUserPage.getByTestId(
-        'entity-header-display-name'
-      );
+    const teamUserPage = await browser.newPage();
+    try {
+      await teamUser.login(teamUserPage);
 
-      await expect(glossaryHeader).toBeVisible({ timeout: 5_000 });
-      await expect(glossaryHeader).toContainText(glossary.data.displayName);
-    }).toPass({ timeout: 30_000, intervals: [2_000, 5_000] });
+      await expect(async () => {
+        await glossary.visitEntityPage(teamUserPage);
+        await waitForAllLoadersToDisappear(teamUserPage);
 
-    // Clean up
-    await cleanupPermissions(apiContext);
+        const glossaryHeader = teamUserPage.getByTestId(
+          'entity-header-display-name'
+        );
 
-    if (teamId) {
-      await apiContext.delete(`/api/v1/teams/${teamId}?hardDelete=true`);
+        await expect(glossaryHeader).toBeVisible({ timeout: 5_000 });
+        await expect(glossaryHeader).toContainText(glossary.data.displayName);
+      }).toPass({ timeout: 30_000, intervals: [2_000, 5_000] });
+    } finally {
+      await teamUserPage.close();
     }
+
+    await cleanupPermissions(apiContext);
+    await apiContext.delete(`/api/v1/teams/${teamId}?hardDelete=true`);
+    await teamUser.delete(apiContext);
   });
 });