metamask-mobile icon indicating copy to clipboard operation
metamask-mobile copied to clipboard

test: complete quiz to reveal SRP

Open plasmacorral opened this issue 6 months ago • 5 comments

Description

This PR updates the SRP quiz to move it from smoke to regression, includes revealing the SRP text and copying to clipboard. We also make sure that the SRP is not revealed with an incorrect password.

~~Also corrects an earlier spelling mistake in SecurityQuizModal.js~~ Updated selector names used in the earlier SRP quiz test.

Related issues

Fixes: 561

Manual testing steps

On local machine run on this branch... Android: yarn test:e2e:android:debug:build then yarn test:e2e:android:debug:run e2e/specs/accounts/reveal-secret-recovery-phrase.spec.js iOS: yarn test:e2e:ios:debug:build then yarn test:e2e:ios:debug:run e2e/specs/accounts/reveal-secret-recovery-phrase.spec.js

pr_regression_e2e_pipeline: PASS Note that regression test that is failing also failed on 8/26 nightly and it not related to my changes here.

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.

plasmacorral avatar Aug 23 '24 04:08 plasmacorral