wormhole-foundation / wormhole-connect

Wormhole Connect brings all the functionality and utility of Wormhole right into your application and removes all of the complexity.
https://connect-in-style.wormhole.com/
Other
78 stars 59 forks source link

[QA] [Brave] "Sui MetaMask Snap" option should not be shown #1565

Open tsadovska opened 8 months ago

tsadovska commented 8 months ago

Describe the current behavior in detail

Checked on https://wormhole-connect.netlify.app/ [Testnet] with macOS and Brave browser.

  1. Click "Connect wallet" button
  2. Check if "MetaMask" option is displayed (Actual result: "MetaMask" option is not displayed)
  3. Check if "Sui MetaMask Snap" option is displayed (Actual result: "Sui MetaMask Snap" option is displayed)
  4. Click on "Sui MetaMask Snap"

Actual result: Option is not clickable. Error {code: -32601, message: 'The method wallet_getSnaps does not exist/is not available'} appears in Console.

Screenshot 2024-01-29 at 16 45 39

What is the expected/desired behavior

"Sui MetaMask Snap" option is shown only when "MetaMask" option is shown.

AigaD commented 8 months ago

Reproduced from: https://portalbridge.com/# Metamask wallet is not present at Brave browser, but Sui Metamask Snap option is shown

Attachment:

Screenshot 2024-01-31 at 12 21 38

AigaD commented 8 months ago

Linking to ticket https://github.com/wormhole-foundation/wormhole-connect/issues/1528 with similar scenarios

AigaD commented 8 months ago

Tested from: https://portalbridge.com/# Brave wallet is in use for Brave browser, Sui Metamask Snap option is offered in the wallet list Attachment:

Screenshot 2024-02-05 at 11 29 48

tsadovska commented 3 weeks ago

Issue is reproducible on wormhole-connect-mainnet (Connect v2).

Error occurs after clicking on "Sui MetaMask Snap" option. Checked with Brave Version 1.69.162 Chromium: 128.0.6613.120 (Official Build) (arm64).

Uncaught (in promise) {code: -32601, message: 'the method wallet_requestSnaps does not exist/is not available'}

image