Skip to content

test: fix perf login tests#27835

Merged
cortisiko merged 4 commits into
mainfrom
MMQA-1630
Mar 24, 2026
Merged

test: fix perf login tests#27835
cortisiko merged 4 commits into
mainfrom
MMQA-1630

Conversation

@cortisiko

@cortisiko cortisiko commented Mar 23, 2026

Copy link
Copy Markdown
Contributor

Description

Updates the LoginScreen WDIO page object to use Appwright getElementByID for the login container when running with a device, aligning element lookup with test IDs.

Refines password field lookup in getPasswordInputElement by switching Android to the more flexible getElementByCatchAll and removing the iOS-specific fallback to a generic textfield, instead targeting LoginViewSelectors.PASSWORD_INPUT directly.

Changelog

CHANGELOG entry:

Related issues

Fixes:

Manual testing steps

Feature: my feature name

  Scenario: user [verb for user action]
    Given [describe expected initial app state]

    When user [verb for user action]
    Then [describe expected outcome]

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.

Note

Low Risk
Low risk test-only change that updates element lookup strategies; main risk is flaky or broken login E2E tests if the new IDs/catch-all matching don’t resolve consistently across platforms.

Overview
Updates the WDIO LoginScreen page object to use Appwright getElementByID for the login container when a device is present, instead of using the resource-id based lookup.

Refines getPasswordInputElement to use getElementByCatchAll on Android and removes the iOS fallback to a generic textfield, consistently targeting LoginViewSelectors.PASSWORD_INPUT via Appwright.

Written by Cursor Bugbot for commit 7681d1d. This will update automatically on new commits. Configure here.

@metamaskbot metamaskbot added the team-qa QA team label Mar 23, 2026
@github-actions github-actions Bot added risk-low Low testing needed · Low bug introduction risk size-S labels Mar 23, 2026
@cortisiko cortisiko requested a review from a team as a code owner March 23, 2026 22:42
@github-actions github-actions Bot added risk-high Extensive testing required · High bug introduction risk and removed risk-low Low testing needed · Low bug introduction risk labels Mar 23, 2026
Comment thread tests/page-objects/wallet/LoginView.ts
@cortisiko cortisiko enabled auto-merge March 23, 2026 22:52
@github-actions github-actions Bot added risk-high Extensive testing required · High bug introduction risk and removed risk-high Extensive testing required · High bug introduction risk labels Mar 23, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Fix All in Cursor

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

Comment thread tests/page-objects/wallet/LoginView.ts Outdated
@github-actions github-actions Bot added risk-low Low testing needed · Low bug introduction risk and removed risk-high Extensive testing required · High bug introduction risk labels Mar 23, 2026
@cortisiko cortisiko added the no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed label Mar 23, 2026
@github-actions github-actions Bot added risk-low Low testing needed · Low bug introduction risk and removed risk-low Low testing needed · Low bug introduction risk labels Mar 23, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔍 Smart E2E Test Selection

  • Selected E2E tags: None (no tests recommended)
  • Selected Performance tags: None (no tests recommended)
  • Risk Level: low
  • AI Confidence: 97%
click to see 🤖 AI reasoning details

E2E Test Selection:
The only changed file is wdio/screen-objects/LoginScreen.js, which belongs to the WebdriverIO (wdio) test framework — a completely separate test framework from Detox. The change is a minor refactor of selector logic: simplifying the loginScreen getter to use getElementByID instead of getElementByResourceId, and updating the passwordInput getter to use getElementByCatchAll for Android while unifying the iOS path. No Detox test files import this file, and no app source code was modified. Per the explicit guidance, changes to the wdio/ directory do not require Detox E2E tags unless app code is also changed — which it is not here.

Performance Test Selection:
No app source code was changed. The modification is purely within the wdio test framework's screen object selectors, which has no impact on app rendering, data loading, or any performance-sensitive code paths.

View GitHub Actions results

@sonarqubecloud

Copy link
Copy Markdown

@chloeYue chloeYue left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for the fix !

@cortisiko cortisiko added this pull request to the merge queue Mar 24, 2026
Merged via the queue into main with commit 81ce895 Mar 24, 2026
100 of 108 checks passed
@cortisiko cortisiko deleted the MMQA-1630 branch March 24, 2026 09:25
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 24, 2026
@metamaskbot metamaskbot added the release-7.72.0 Issue or pull request that will be included in release 7.72.0 label Mar 24, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed release-7.72.0 Issue or pull request that will be included in release 7.72.0 risk-low Low testing needed · Low bug introduction risk size-S team-qa QA team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants