synpress icon indicating copy to clipboard operation
synpress copied to clipboard

Update `acceptMetamaskAccess()` to accept a list of account indexes.

Open neuodev opened this issue 2 years ago • 1 comments

Motivation and context

When connecting the dApp to MetaMask, It will be better to have the ability to select a list of accounts or a single account to connect the dApp to it.

cy.acceptMetamaskAccess({ 
  accountIndexes: [1, 2], // Will connect the dApp to the 1st and 2nd account 
});

Other useful info

When using this feature, the default selected account (mostly the active one) will be deselected if needed. The allAccounts flag is designed to overwrite the accountIndexes flag when it is set to true

Quality checklist

  • [x] I have performed a self-review of my code.
  • [x] If it is a core feature, I have added thorough e2e tests.

neuodev avatar Mar 15 '23 09:03 neuodev

Passing run #391 ↗︎

0 174 0 0 Flakiness 0

Details:

Update `acceptMetamaskAccess()` to accept a list of account indexes.
Project: synpress Commit: d6b5d13c8b
Status: Passed Duration: 10:34 💡
Started: Aug 18, 2023 4:24 PM Ended: Aug 18, 2023 4:35 PM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

cypress[bot] avatar Mar 15 '23 09:03 cypress[bot]