-
Notifications
You must be signed in to change notification settings - Fork 5.4k
Snaps E2E Test Upgrades for the new test-snaps UI #14980
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
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
a62507a to
a3bd87d
Compare
|
These test are now working with, and can be co-deployed with MetaMask/test-snaps#40 However it cannot be merged until MetaMask/test-snaps#40 is merged because it refactors classes to id's Also, MetaMask/test-snaps#40 breaks the current e2e's so they have to hit at the same time 😅 |
Builds ready [66f6640]Page Load Metrics (1650 ± 28 ms)
|
Builds ready [18b2393]Page Load Metrics (1857 ± 63 ms)
|
Changes need to be made in order to make existing e2e tests work with the new test-snaps UI (see MetaMask/test-snaps#40)
This fixes both navigating through the pages as well as changes necessary to the refactor of class & id tags.