Update notification test to handle hidden notification removed from DOM

1 files changed, 17 insertions, 1 deletions

diff --git a/desktop/packages/mullvad-vpn/test/e2e/mocked/notifications.spec.ts b/desktop/packages/mullvad-vpn/test/e2e/mocked/notifications.spec.ts
index b8173c1cb5..d76090814d 100644
--- a/desktop/packages/mullvad-vpn/test/e2e/mocked/notifications.spec.ts
+++ b/desktop/packages/mullvad-vpn/test/e2e/mocked/notifications.spec.ts
@@ -150,7 +150,23 @@ test.describe('Unsupported wireguard port', () => {
 
       const subTitle = page.getByTestId('notificationSubTitle');
 
-      await expect(subTitle).not.toContainText(/The selected WireGuard port is not supported/i);
+      // TODO: Remove these test cases which test for the absence of a specific
+      // notification. We prefer to have tests which test for something, rather
+      // than the absence of something.
+      await expect(async () => {
+        const visible = await subTitle.isVisible();
+        if (visible) {
+          // EITHER: A notification is displayed, but not for unsupported WireGuard port
+          expect(await subTitle.innerText()).not.toMatch(
+            /The selected WireGuard port is not supported/i,
+          );
+        } else {
+          // OR: no notification is displayed at all
+          // NO OP
+        }
+      }).toPass({
+        timeout: 5000,
+      });
     });
   });