Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multichain API E2E Test: handling when MetaMask is password locked #29724

Open
wants to merge 10 commits into
base: jl/caip-multichain-migrate-core
Choose a base branch
from

Conversation

ffmcgee725
Copy link
Member

@ffmcgee725 ffmcgee725 commented Jan 15, 2025

-- test: multichain test dapp e2e tests for password locked extension;

Description

Open in GitHub Codespaces

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@metamaskbot
Copy link
Collaborator

Builds ready [4a36b78]
Page Load Metrics (1726 ± 83 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint42622031668334160
domContentLoaded14222125170216177
load14302176172617383
domInteractive2483442110
backgroundConnect96630199
firstReactRender15100392713
getState45310105
initialActions01000
loadScripts10261597126014670
setupStore65312136
uiStartup171327001976238114

@ffmcgee725 ffmcgee725 requested a review from adonesky1 January 16, 2025 10:45
@ffmcgee725 ffmcgee725 requested a review from adonesky1 January 16, 2025 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants