You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
After trying to debug an e2e test that was failing on the CI, I attempted to set up my local environment to run these same e2e tests.
After a lot of trial and error, I managed to get them running, I have updated the Docs that explain how to get started and improved the install-ios-runway-app script so that it can be reused for this scenario which simplifies the tester's life and prevents them from having to download the build manually from runway
Feature: my feature nameScenario: user [verb for user action]Given [describe expected initial app state]
When user [verb for user action]
Then [describe expected outcome]
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
Low Risk
Changes are limited to developer documentation and a local iOS Runway install helper script; risk is mainly around breaking existing local workflows due to the new build/ artifact location and fixed MetaMask.app naming.
Overview
Simplifies local E2E setup docs by restructuring the flow into App Build (Expo prebuild download vs local build) and Run the E2E Tests (explicit two-terminal Metro/test execution steps), with clearer commands for running all tests, a folder, a file, or by tag.
Updates install-ios-runway-app.sh to store Runway artifacts in build/, always extract/expect MetaMask.app, and adds --skipInstall to support download-only runs without requiring a booted simulator.
Written by Cursor Bugbot for commit 3fcfa65. This will update automatically on new commits. Configure here.
Selected Performance tags: None (no tests recommended)
Risk Level: low
AI Confidence: 95%
click to see 🤖 AI reasoning details
E2E Test Selection:
The changes consist of:
docs/readme/e2e-testing.md - Pure documentation changes reorganizing and improving the E2E testing setup guide. This includes better formatting, clearer instructions for Expo prebuilds vs local builds, and improved terminal session guidance. Documentation changes have zero impact on app functionality.
Changed download directory from runway-artifacts to build
Added --skipInstall flag for download-only mode
Standardized app naming to always use "MetaMask.app"
This script is used for local development setup to download iOS builds from Runway. It does not affect:
The actual app code or functionality
E2E test specs, pages, or fixtures
CI/CD workflows or test execution
Any user-facing features
Both changes are purely developer experience improvements with no runtime impact on the MetaMask Mobile app. No E2E tests are needed to validate these changes.
Performance Test Selection:
These changes are documentation and developer tooling script updates only. They do not affect any app code, UI components, data loading, state management, or any runtime behavior. No performance impact is possible from these changes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
After trying to debug an e2e test that was failing on the CI, I attempted to set up my local environment to run these same e2e tests.
After a lot of trial and error, I managed to get them running, I have updated the Docs that explain how to get started and improved the
install-ios-runway-appscript so that it can be reused for this scenario which simplifies the tester's life and prevents them from having to download the build manually from runwayChangelog
CHANGELOG entry: simplify local e2e testing setup
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/ASSETS-2569
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Low Risk
Changes are limited to developer documentation and a local iOS Runway install helper script; risk is mainly around breaking existing local workflows due to the new
build/artifact location and fixedMetaMask.appnaming.Overview
Simplifies local E2E setup docs by restructuring the flow into App Build (Expo prebuild download vs local build) and Run the E2E Tests (explicit two-terminal Metro/test execution steps), with clearer commands for running all tests, a folder, a file, or by tag.
Updates
install-ios-runway-app.shto store Runway artifacts inbuild/, always extract/expectMetaMask.app, and adds--skipInstallto support download-only runs without requiring a booted simulator.Written by Cursor Bugbot for commit 3fcfa65. This will update automatically on new commits. Configure here.