chore: change selector for Approve button on connecting with wallet UI

agoric-labs · Mar 4, 2024 · 33c39ff · 33c39ff
1 parent fac32cd
commit 33c39ff
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 10 deletions.
diff --git a/pages/keplr/notification-page.js b/pages/keplr/notification-page.js
@@ -1,4 +1,4 @@
-const approveButton = `button[type="button"]`;
+const approveButton = `button`;
 
 module.exports.notificationPageElements = {
   approveButton,

diff --git a/tests/e2e/specs/keplr/keplr-spec.js b/tests/e2e/specs/keplr/keplr-spec.js
@@ -17,6 +17,13 @@ describe('Keplr', () => {
         });
       });
     });
+    it(`should disconnect the wallet from all the connected DAPPs`, () => {
+      cy.switchToExtensionWindow().then(() => {
+        cy.disconnectWalletFromDapp().then(taskCompleted => {
+          expect(taskCompleted).to.be.true;
+        });
+      });
+    });
     it(`should complete Keplr connect with wallet, and confirm transaction after creating a new wallet using 24 word phrase`, () => {
       cy.switchToExtensionWindow().then(() => {
         cy.setupWallet(
@@ -28,9 +35,13 @@ describe('Keplr', () => {
 
           cy.visit('/');
           cy.contains('Connect Wallet').click();
-          cy.contains('Make an Offer').click();
-          cy.confirmTransaction().then(taskCompleted => {
+          cy.acceptAccess().then(taskCompleted => {
             expect(taskCompleted).to.be.true;
+
+            cy.contains('Make an Offer').click();
+            cy.confirmTransaction().then(taskCompleted => {
+              expect(taskCompleted).to.be.true;
+            });
           });
         });
       });
@@ -44,12 +55,5 @@ describe('Keplr', () => {
         });
       });
     });
-    it(`should disconnect the wallet from all the connected DAPPs`, () => {
-      cy.switchToExtensionWindow().then(() => {
-        cy.disconnectWalletFromDapp().then(taskCompleted => {
-          expect(taskCompleted).to.be.true;
-        });
-      });
-    });
   });
 });