Skip to content

Conversation

@jpuri
Copy link
Contributor

@jpuri jpuri commented Oct 10, 2025

Description

fix 7702 transactions related e2e on mobile

Changelog

CHANGELOG entry:

Related issues

Fixes: #17930

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

NA

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.

@github-actions
Copy link
Contributor

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.

@metamaskbot metamaskbot added the team-confirmations Push issues to confirmations team label Oct 10, 2025
@jpuri jpuri changed the title Fix smart transaction related e2e on mobile test: fix smart transaction related e2e on mobile Oct 10, 2025
@jpuri jpuri changed the title test: fix smart transaction related e2e on mobile test: fix 7702 transactions related e2e on mobile Oct 10, 2025
@jpuri jpuri added no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed No QA Needed Apply this label when your PR does not need any QA effort. labels Oct 10, 2025
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.70%. Comparing base (3d3f00e) to head (a2bd5c0).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #21020      +/-   ##
==========================================
- Coverage   76.70%   76.70%   -0.01%     
==========================================
  Files        3466     3476      +10     
  Lines       85743    85888     +145     
  Branches    16010    16042      +32     
==========================================
+ Hits        65773    65881     +108     
- Misses      15529    15550      +21     
- Partials     4441     4457      +16     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions bot added size-M and removed size-S labels Oct 10, 2025
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No QA Needed Apply this label when your PR does not need any QA effort. no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed size-M team-confirmations Push issues to confirmations team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Re-enable 7702 Batch transaction e2e tests

3 participants