Skip to content

test(snaps): Add name-lookup E2E - #26765

Merged
FrederikBolding merged 6 commits into
mainfrom
fb/name-lookup-e2e
Mar 4, 2026
Merged

FrederikBolding merged 6 commits into
mainfrom
fb/name-lookup-e2e

Conversation

@FrederikBolding

@FrederikBolding FrederikBolding commented Mar 2, 2026 •

Copy link
Copy Markdown
Member

Description

Port extension name-lookup test to mobile for increased coverage. The PR adjusts a few shared utilities to make the tests more stable and enable the flow tested in this test.

Changelog

CHANGELOG entry: null

Related issues

Closes: MetaMask/snaps#3572


Note

Low Risk
Low risk: changes are limited to Detox E2E test code and test selectors, with a small tap-stability tweak that could affect test flakiness but not app behavior.

Overview
Adds a new smoke E2E test (test-snap-name-lookup.spec.ts) that installs the Name Lookup test snap and verifies a domain recipient resolves to the expected address during the send/confirmation flow.

Extends test infrastructure to support the scenario by adding the connectNameLookupButton selector, adding a tapAdvancedDetails() helper on TransactionConfirmView, and making RedesignedSendView.pressReviewButton() use checkStability when tapping.

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

@metamaskbot metamaskbot added team-core-platform Core Platform team INVALID-PR-TEMPLATE PR's body doesn't match template labels Mar 2, 2026
@github-actions github-actions Bot added the size-S label Mar 2, 2026
@github-actions

github-actions Bot commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

🔍 Smart E2E Test Selection

  • Selected E2E tags: FlaskBuildTests, SmokeConfirmations
  • Selected Performance tags: None (no tests recommended)
  • Risk Level: medium
  • AI Confidence: 90%
click to see 🤖 AI reasoning details

E2E Test Selection:
The PR introduces a new E2E test for the Name Lookup Snap functionality (test-snap-name-lookup.spec.ts) which is explicitly tagged with FlaskBuildTests. This test validates snap-based domain name resolution in the send flow.

Additionally, the PR modifies shared test infrastructure:

  1. RedesignedSendView.ts: Added checkStability: true to pressReviewButton() - this is a stability improvement that affects all tests using this method, including confirmation tests (send-erc20-token, send-native-token, gas-fee-tokens)
  2. TransactionConfirmView.ts: Added new tapAdvancedDetails() method used by the new snap test
  3. TestSnaps.selectors.ts: Added new selector for the name lookup snap button

FlaskBuildTests is required to run the new snap test and validate the snap functionality works correctly.

SmokeConfirmations is selected because the checkStability: true change to pressReviewButton() affects the send flow used by confirmation tests. While this is a stability improvement, running these tests ensures the change doesn't introduce any regressions in the confirmation flow.

Performance Test Selection:
The changes are purely test infrastructure additions (new test file, page object methods, selectors) with no modifications to application code that would affect performance. The checkStability flag is a test stability improvement, not an app performance change. No performance tests are needed.

View GitHub Actions results

@github-actions

github-actions Bot commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

⚠️ E2E Fixture Validation — Structural changes detected

Category Count
New keys 60
Missing keys 0
Type mismatches 0
Value mismatches 7 (informational)

The committed fixture schema is out of date. To update, comment:

@metamaskbot update-mobile-fixture

View full details | Download diff report

@sonarqubecloud

sonarqubecloud Bot commented Mar 3, 2026

Copy link
Copy Markdown

@FrederikBolding
FrederikBolding marked this pull request as ready for review March 3, 2026 13:16
@FrederikBolding
FrederikBolding requested review from a team as code owners March 3, 2026 13:16

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

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/smoke/snaps/test-snap-name-lookup.spec.ts

@christopherferreira9 christopherferreira9 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!

@FrederikBolding
FrederikBolding added this pull request to the merge queue Mar 4, 2026
Merged via the queue into main with commit b2d383c Mar 4, 2026
108 checks passed
@FrederikBolding
FrederikBolding deleted the fb/name-lookup-e2e branch March 4, 2026 09:11
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 4, 2026
@metamaskbot metamaskbot added the release-7.69.0 Issue or pull request that will be included in release 7.69.0 label Mar 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

INVALID-PR-TEMPLATE PR's body doesn't match template release-7.69.0 Issue or pull request that will be included in release 7.69.0 size-S team-core-platform Core Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Port name lookup E2E to mobile

4 participants