fix: flaky test Simulation Details displays generic error message

[seaona]

Description

This test didn't have the mock correctly implemented, making the request take long, thus sometimes failing the test.
The problem is that in the mock, it sets an id of 21however this value is not always the one used in the request.
Since we cannot know which id is going to use, we can omit this param and just match the rest of the request body.

Related issues

Fixes: #27157

Manual testing steps

1. Check ci
2. Run test locally yarn test:e2e:single test/e2e/tests/simulation-details/simulation-details.spec.ts --browser=chrome --leave-running=true

Screenshots/Recordings

See how the request has an id of 0 so it won't be mocked

incorrect-mock-id.mp4

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
• 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.

[github-actions]

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.

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.96%. Comparing base (ca6fd53) to head (c4e2948).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop   #27156   +/-   ##
========================================
  Coverage    69.96%   69.96%           
========================================
  Files         1442     1442           
  Lines        50100    50100           
  Branches     14006    14006           
========================================
  Hits         35049    35049           
  Misses       15051    15051           

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

[metamaskbot]

Builds ready [c4e2948]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5
  • common: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9
  • content-script: 0
  • offscreen: 0
  • ui: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (1626 ± 81 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	1379	2074	1628	170	82
		domContentLoaded	1372	2056	1611	166	80
		load	1380	2077	1626	169	81
		domInteractive	20	89	35	17	8

Bundle size diffs

• background: 0 Bytes (0.00%)
• ui: 0 Bytes (0.00%)
• common: 0 Bytes (0.00%)

[hjetpoluru]

LGTM

[chloeYue]

LGTM !