The E2E tests are failing because of the enhance experience bottom sheet that appears in some of the tests. Furthermore, there is also a policy alert which also affects the e2e tests. The purpose of this PR is to stabilize the E2E tests so they handle the bottom sheet and the policy alert.
Keep in mind this
bug also affects the e2e tests and a few try catches were added to deal with the bug.
[ ] I've completed the PR template to the best of my ability
[ ] I’ve included tests if applicable
[ ] I’ve documented my code using JSDoc format if applicable
[ ] I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.
[ ] 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.
[ ] I've completed the PR template to the best of my ability
[ ] I’ve included tests if applicable
[ ] I’ve documented my code using JSDoc format if applicable
[ ] I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.
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.
The E2E tests are failing because of the enhance experience bottom sheet that appears in some of the tests. Furthermore, there is also a policy alert which also affects the e2e tests. The purpose of this PR is to stabilize the E2E tests so they handle the bottom sheet and the policy alert.
Keep in mind this bug also affects the e2e tests and a few try catches were added to deal with the bug.
Passing smoke run: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/7457565d-c8e4-4fd6-9df6-1cd6a1d76861
Fixes:
[ ] I’ve followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
[ ] I've completed the PR template to the best of my ability
[ ] I’ve included tests if applicable
[ ] I’ve documented my code using JSDoc format if applicable
[ ] I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.
[ ] 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.
Description
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist