test: migrates the page objects used by Flows.js

[christopherferreira9]

Description

This PR migrates the page objects used by Flows.js (performance related) to the new unified approach.

Changelog

CHANGELOG entry:

Related issues

Fixes: https://consensyssoftware.atlassian.net/browse/MMQA-1469

Manual testing steps

N/A

Screenshots/Recordings

Before

N/A

After

N/A

Pre-merge author checklist

• 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.

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.

---

Note

Medium Risk
Medium risk because it refactors multiple frequently-used E2E page objects and selector strategies (including new XPath/catch-all locators), which can introduce cross-platform test flakiness even though production code is unaffected.

Overview
Migrates the page objects used by performance Flows.js to the unified EncapsulatedElement/UnifiedGestures approach, adding Appium/Playwright selector paths alongside existing Detox locators.

Adds new Appium-capable page objects/modals and expands existing onboarding and wallet page objects with cross-platform isVisible/wait helpers and flow-specific actions (e.g., SRP word-by-word entry, account list syncing waits).

Updates the migration test utilities and mocks to account for additional Playwright matcher types (getElementByXPath, getElementByCatchAll) and introduces a simple PlaywrightGestures.tap wrapper used by the migrated objects.

^{Written by Cursor Bugbot for commit 211f10f. This will update automatically on new commits. Configure here.}

[github-actions]

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.

[cursor]

Inconsistent indentation in noPlaywrightMatcherWasCalled function

Low Severity

The two new expect statements at lines 81–82 use 2-space indentation while all other statements in noPlaywrightMatcherWasCalled use 4-space indentation. This breaks the visual consistency of the function body.

 

[github-actions]

🔍 Smart E2E Test Selection

⏭️ Smart E2E selection skipped - skip-smart-e2e-selection label found

All E2E tests pre-selected.

View GitHub Actions results

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

^{Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

[cursor]

tapContinueButton passes non-encapsulated element to UnifiedGestures

Medium Severity

tapContinueButton was migrated from Gestures.tap to UnifiedGestures.tap, but still passes this.continueButton which returns a plain DetoxElement instead of EncapsulatedElementType. In Appium context, AppiumGestureStrategy.tap would call .click() on a DetoxElement, which lacks that method. A properly encapsulated continueButtonEncapsulated getter already exists but isn't used here.

Additional Locations (1)

• tests/page-objects/Onboarding/ImportWalletView.ts#L91-L96

 

[christopherferreira9]

Will fix in a follow up pr

[github-actions]

✅ E2E Fixture Validation — Schema is up to date
17 value mismatches detected (expected — fixture represents an existing user).
View details

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[christopherferreira9]

Closing as not relevant for now.