Nightly fix 26-05-30 - Use evmTetragrams format for EVM address assertions in send-base#28195
Open
github-actions[bot] wants to merge 3 commits into
Open
Nightly fix 26-05-30 - Use evmTetragrams format for EVM address assertions in send-base#28195github-actions[bot] wants to merge 3 commits into
github-actions[bot] wants to merge 3 commits into
Conversation
…s in send-base T3W1 firmware now renders EVM addresses in evmTetragrams format (two leading spaces before '0x', then hex body split into 4-char chunks). Update both transformAddress calls in send-base.test.ts to pass 'evmTetragrams' as the second argument so the assertions match the firmware output.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Nightly fix — 2026-05-30
Task: fix-001
Scope: TEST_CODE
Result: ✅ pass
Root cause
T3W1 firmware now renders EVM addresses in evmTetragrams format (leading
0x+ 4-char chunks of body), butsend-base.test.tswas using the defaultfourTetragramsformat (splits the entire address including0xprefix every 4 chars).Fix
In
suite/e2e/tests/wallet/send-base.test.ts, changed both calls fromtransformAddress(sendAddress)totransformAddress(sendAddress, 'evmTetragrams'):The
'evmTetragrams'format usesformatEvmAddress()which prepends two spaces before0xand splits the hex body into 4-char chunks, matching the current T3W1 firmware display output.Validations
Commits
Prompt gaps
chrome-sandboxpermissions) and then anopenpgpmodule missing error — both environment setup issues. These were fixed by runningsudo chmod 4755on the sandbox binary and installing the missingopenpgpdependency viayarn workspace @trezor/suite-desktop add openpgp. The prompt does not mention these environment fixes; they were needed to make the test runner work at all.