test locator exists theory

Synthetixio · Jun 19, 2024 · d57e462 · d57e462
1 parent 4d856a4
commit d57e462
Showing 1 changed file with 3 additions and 14 deletions.
diff --git a/wallets/metamask/src/pages/NotificationPage/page.ts b/wallets/metamask/src/pages/NotificationPage/page.ts
@@ -32,20 +32,9 @@ export class NotificationPage {
 
   private async beforeMessageSignature(extensionId: string) {
     const notificationPage = await getNotificationPageAndWaitForLoad(this.page.context(), extensionId)
-    const scrollDownButton = notificationPage.locator(Selectors.SignaturePage.structuredMessage.scrollDownButton)
-
-    let isScrollButtonVisible = false
-
-    const scrollButtonPromise = scrollDownButton
-      .waitFor({ state: 'visible' })
-      .then(async () => {
-        isScrollButtonVisible = true
-        await scrollDownButton.click()
-        return true
-      })
-      .catch(() => false)
-
-    await Promise.race([scrollButtonPromise, notificationPage.waitForLoadState('load').then(() => false)])
+    notificationPage.waitForLoadState('load')
+    const isScrollButtonVisible =
+      (await notificationPage.locator(Selectors.SignaturePage.structuredMessage.scrollDownButton).count()) > 0
 
     return {
       notificationPage,