Skip to content

Conversation

@seaona
Copy link
Member

@seaona seaona commented Mar 28, 2025

Description

The problem is that we are missing the fetch trades mock, and that request is redirected to the catch-all mock returning a 200 with empty data.
Then, when the wallet tries to parse the data we get teh JSON error.

This happens ocasionally if there's "time" for that request to happen and the wallet tries to parse the JSON response, within the test run.

Screenshot from 2025-03-28 08-11-59

Circle ci error: https://app.circleci.com/pipelines/github/MetaMask/metamask-extension/134207/workflows/69597d81-ae63-41b1-b655-50d85e31b59b/jobs/4697734/tests

Open in GitHub Codespaces

Related issues

Fixes:

Manual testing steps

  1. Check ci run https://app.circleci.com/pipelines/github/MetaMask/metamask-extension/134223/workflows/9c61c340-c4b3-49c8-8590-d1e2f783f124/jobs/4697845

Screenshot from 2025-03-28 08-34-05

Screenshots/Recordings

See the error fetching quotes behind the modal, which makes the test fail due to the console error.

Screenshot from 2025-03-28 08-08-06

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-qa QA team label Mar 28, 2025
@seaona seaona self-assigned this Mar 28, 2025
@seaona seaona added the area-qa Relating to QA work (Quality Assurance) label Mar 28, 2025
@seaona seaona marked this pull request as ready for review March 28, 2025 07:34
@seaona seaona enabled auto-merge March 28, 2025 08:08
@metamaskbot
Copy link
Collaborator

Builds ready [ccc9daf]
UI Startup Metrics (1218 ± 61 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1218109314796112571320
load10599541270561136986
domContentLoaded10549491265561135989
domInteractive17134051629
firstPaint750851186405260982
backgroundConnect106283910
firstReactRender20155172136
getState12435768
initialActions001001
loadScripts837736104855869918
setupStore7519278
WebpackHomeuiStartup978799122068975990
load83863193748858909
domContentLoaded83162192547852900
domInteractive16124671436
firstPaint51557914347846871
backgroundConnect16114271539
firstReactRender14123341427
getState7416278
initialActions001001
loadScripts82961091547850890
setupStore7515279
FirefoxBrowserifyHomeuiStartup13451160185313713831669
load12091045170412712551492
domContentLoaded12081045170412712541492
domInteractive9636166258596
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2616113142555
firstReactRender21193932227
getState6340478
initialActions001001
loadScripts11841028167912312231460
setupStore6422267
WebpackHomeuiStartup9828371696147893963
load8597281441125857975
domContentLoaded8587281440125856973
domInteractive114342002815599
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2213146172232
firstReactRender18162921922
getState9464979
initialActions001001
loadScripts8427151417122863979
setupStore8572879
Bundle size diffs
  • background: 0 Bytes (0%)
  • ui: 0 Bytes (0%)
  • common: 0 Bytes (0%)

Copy link
Contributor

@chloeYue chloeYue left a comment

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 !

@seaona seaona added this pull request to the merge queue Mar 28, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 28, 2025
@seaona seaona added this pull request to the merge queue Mar 28, 2025
Merged via the queue into main with commit 0e3a38e Mar 28, 2025
152 checks passed
@seaona seaona deleted the fix-flaky-swaps-quote branch March 28, 2025 09:46
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2025
@metamaskbot metamaskbot added the release-12.17.0 Issue or pull request that will be included in release 12.17.0 label Mar 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-qa Relating to QA work (Quality Assurance) flaky tests release-12.17.0 Issue or pull request that will be included in release 12.17.0 team-qa QA team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants