-
Notifications
You must be signed in to change notification settings - Fork 4.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: swap + send eth e2e tests #24449
Conversation
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. |
a96660e
to
c0ed904
Compare
f6d9ee1
to
f380839
Compare
729b7fd
to
a9bf603
Compare
59954b4
to
64eecd2
Compare
// TODO does this need to be removed? Needs to be tested with non 1559 network | ||
networkAndAccountSupports1559 | ||
? gasFee1559 | ||
: `0x${String(gasPriceNon1559 ?? 0)}`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we use addHexPrefix
here instead
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Side note but is this not a numeric value? We would need to change the base in the next parameter as well if it's a hex
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you search for GAS_PRICE_API_MOCK_RESULT
or gasPricesCallbackMock
it looks like the prices are in decimals
The infura https://gas.api.infura.io/networks/1/gasPrices api might be deprecated tho bc I can't find docs on it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why this is blocked by @MetaMask/design-system-engineers, did it previously touch the component-library? Approving to unblock
64eecd2
to
ca537b4
Compare
ec510e2
to
0f5804a
Compare
70d6f85
to
ad4b598
Compare
fix: send-hex-address tests fix: custom-token-send-transfer tests fix: ens test fix: erc721-interaction tests fix: custom-token-add-approve tests fix: snap-account-transfers tests fix: test-snap-txinsights tests fix: failing-contract tests fix: settings-security-reveal-srp tests fix: token-details test fix: add-hide-token tests fix: show-hex-data test fix: test-snap-txinsights-v2 tests fix: increase-token-allowance test fix: increase snap-account-transfers delay (flaky) fix: rm delay for swap-send-eth fix: contract-interactions test fix: snap-account-transfers fix: failing-contract tests fix: ens test fix: rm useEthFeeData
fix: show-hex-data.spec tests fix: swap-send-eth test
ad4b598
to
3e6bb55
Compare
b10e78c
to
7e438aa
Compare
Description
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist