Excluding floating-ui portals as part of hiding tooltips in Playwrigh…

…t screenshots Signed-off-by: Michael Telatynski <[email protected]>
element-hq · Oct 24, 2024 · a073cb1 · a073cb1
1 parent 6d0d237
commit a073cb1
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 9 deletions.
diff --git a/playwright/e2e/pinned-messages/index.ts b/playwright/e2e/pinned-messages/index.ts
@@ -196,14 +196,7 @@ export class Helpers {
      */
     async assertEmptyPinnedMessagesList() {
         const rightPanel = this.getRightPanel();
-        await expect(rightPanel).toMatchScreenshot(`pinned-messages-list-empty.png`, {
-            // hide the tooltip "Room information" to avoid flakiness
-            css: `
-                [data-floating-ui-portal] {
-                    display: none !important;
-                }
-            `,
-        });
+        await expect(rightPanel).toMatchScreenshot(`pinned-messages-list-empty.png`);
     }
 
     /**

diff --git a/playwright/e2e/release-announcement/index.ts b/playwright/e2e/release-announcement/index.ts
@@ -42,7 +42,7 @@ export class Helpers {
      */
     async assertReleaseAnnouncementIsVisible(name: string) {
         await expect(this.getReleaseAnnouncement(name)).toBeVisible();
-        await expect(this.page).toMatchScreenshot(`release-announcement-${name}.png`);
+        await expect(this.page).toMatchScreenshot(`release-announcement-${name}.png`, { showTooltips: true });
     }
 
     /**

diff --git a/playwright/element-web-test.ts b/playwright/element-web-test.ts
@@ -345,6 +345,7 @@ export const expect = baseExpect.extend({
 
         if (!options?.showTooltips) {
             css += `
+                [data-floating-ui-portal],
                 [role="tooltip"] {
                     visibility: hidden !important;
                 }