Description

418ecfc6e5db60cb4a9b1d59db72322f0bd797c3 was moved from #25230 into this separate PR.

@micaelae wrote:

the main change in the commit is to update the test ganache network. we were configuring the tests without EIP-1559 support before. and as a result, gas fees in the Send page were calculated using the response of https://gas.api.cx.metamask.io/networks/1/gasPrices. afaik the response was getting mocked to a constant value before so the tests succeeded reliably.
after your change, the response of that endpoint became variable. which made me look into why the gas fees were excessively high within the test in the first place
since the Swap+Send experience only runs in networks that support EIP-1559, I updated the test network. as a result the tests now use the suggestedGasFees endpoint mock, which also corrected the gas calculation test bug

Related issues

Fixes:

Manual testing steps

1. Go to this page...

Screenshots/Recordings

Before

After

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.