feat(test-specs): add extra eip-7708 test coverage by spencer-tb · Pull Request #2062 · ethereum/execution-specs

spencer-tb · 2026-01-22T15:29:49Z

🗒️ Description

Extra test cases for EIP-7708 including a general refactor. Adds all the cases here asked for here: #1875 (comment), including many others.

The eip_testing_checklist_template.md has been covered and ticked off via eip_checklist_not_applicable.txt.
EELS coverage is also checked off and verified, see eip_checklist_external_coverage.txt for more information.
Mainnet marked tests are included, they need to be verified on the devnet or once a testnet includes Amsterdam changes.

🔗 Related Issues or PRs

Requires #2064 for docs to build as added EIP checklist and coverage items.

✅ Checklist

All: Ran fast tox checks to avoid unnecessary CI fails, see also Code Standards and Enabling Pre-commit Checks:
```
uvx tox -e static
```
All: PR title adheres to the repo standard - it will be used as the squash commit message and should start type(scope):.
All: Considered updating the online docs in the ./docs/ directory.
All: Set appropriate labels for the changes (only maintainers can apply labels).
Tests: Ran mkdocs serve locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.
- Requires chore(docs): bump docs fork to amsterdam #2064.

Cute Animal Picture

spencer-tb · 2026-01-22T18:06:29Z

+
+
+@pytest.mark.valid_at_transition_to("Amsterdam")
+def test_transfer_log_fork_transition(


I should add selfdestruct logs here too.

…coverage check

marioevz

LGTM. I left a comment of a single test that we need to implement eventually.

codecov · 2026-01-23T06:46:02Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.97%. Comparing base (bf16746) to head (e70030f).
⚠️ Report is 12 commits behind head on eips/amsterdam/eip-7708.

Additional details and impacted files

@@                     Coverage Diff                     @@
##           eips/amsterdam/eip-7708    #2062      +/-   ##
===========================================================
- Coverage                    86.14%   83.97%   -2.18%     
===========================================================
  Files                          599      642      +43     
  Lines                        39491    42265    +2774     
  Branches                      3782     4064     +282     
===========================================================
+ Hits                         34021    35493    +1472     
- Misses                        4848     6055    +1207     
- Partials                       622      717      +95

Flag	Coverage Δ
unittests	`83.97% <ø> (-2.18%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

* feat(test-specs): add/refactor tests, add mainnet marked, checklist, coverage check * feat(test-specs): add fork transition test for selfdestruct logs * fix: tests * chore(test-specs): fix fork transition tests * test(test-specs): add code deposit oog test case --------- Co-authored-by: carsons-eels <carson@ethereum.org> Co-authored-by: Mario Vega <marioevz@gmail.com>

@marioevz

…2844) * feat(spec-specs): Add transfer log for all `CALL*` and `SELFDESTRUCT` fix(spec-specs): correct CR issues, fix formatting fix(spec-specs): inline `execute_code()` to `process_message()` chore(spec-specs): backport changes fix(spec-specs): trim out whitespace in topic hash to match tests feat(spec-specs): add selfdestruct event topic and logging function feat(spec-specs): selfdestruct to self emits selfdestruct event feat(spec-specs): define call success constant feat(spec-specs): emit selfdestruct finalization log for remaining balance * fix(spec-specs): emit account closure logs in lexicographical order * feat(test-tests): add eip-7708 eth transfer log tests test(test-tests): add selfdestruct topic and use empty account test(test-tests): add nested calls log ordering test feat(test-tests): add selfdestruct finalization test fix(test-tests): use spaces in event signature to match spec * fix(spec-specs): Refactor topic strings to match EIP * fix(spec-tests): formatting fixes so static checks pass * fix(spec-specs): Move account closure log emission before priority fee charges (#2059) * fix(spec-specs): Move account closure log emission before priority fee charges * fix(spec-specs): formatting and spelling tweaks * fix(spec-specs): remove duplicate WriteInStaticContext check * refactor(spec-specs): align memory expansion with other opcodes * fix(testing/test): Fix unit test expectation * refactor(spec-specs): move post-mining coinbase balance calculation --------- Co-authored-by: Mario Vega <marioevz@gmail.com> * feat(test-specs): add extra eip-7708 test coverage (#2062) * feat(test-specs): add/refactor tests, add mainnet marked, checklist, coverage check * feat(test-specs): add fork transition test for selfdestruct logs * fix: tests * chore(test-specs): fix fork transition tests * test(test-specs): add code deposit oog test case --------- Co-authored-by: carsons-eels <carson@ethereum.org> Co-authored-by: Mario Vega <marioevz@gmail.com> * feat(spec,test): EIP-7708 spec updates for self as target (#2086) * fix(spec,text): Updates to EIP-7708 spec for bal-devnet-2 - fix(spec,test): EIP-7708 emit selfdestruct logs only in same tx - fix(spec,test): EIP-7708, only emit on transfers to other accounts - Add more tests that match these edge cases * feat(test): tests for finalization selfdest + bal transfer in lexicographic order * fix: changes from comments on PR #2086 * add more variants to test_selfdestruct_same_tx_via_call * fix: docstring * refactor: improvements from comments on PR #2086 * Update test to use dynamic addresses --------- Co-authored-by: Mario Vega <marioevz@gmail.com> * feat(test): extend EIP-7708 tests from cases in tracker (#1875) (#2106) - Add CREATE2 support via with_all_create_opcodes marker - Add tests for SELFDESTRUCT to coinbase - revealed a change needed since the last update was made to the EIP that should be included in the currect refspec (miner fees paid before finalization LOG2). * fix(test): Use new ``pre_alloc_mutable`` marker for eip7708 test (#2199) * fix(test): Update test to account for recent Initcode updates * 🧹 chore: Remove duplicate test (#2210) Co-authored-by: raxhvl <raxhvl@users.noreply.github.com> * ♻️ refactor(tests): Rename EIP 7708 selfdestruct log to burn (#2211) * ♻️ refactor: Rename selfdestruct log to burn * 🥢 nit: * chore(tests): fix stale selfdestruct references and rename to burn --------- Co-authored-by: raxhvl <raxhvl@users.noreply.github.com> Co-authored-by: spencer-tb <spencer.tb@ethereum.org> * fix(specs): Merge issues * fix(tests): Merge issues * refactor(test-forks): Add EIP-7708 * refactor(tests): Condition EIP-7708 tests to EIP inclusion * feat(tests): EIP-7708 - finalization burn log ordering + coinbase fee no-log (#2717) * feat(tests): EIP-7708 - multi-account finalization burn log ordering Adds a dedicated test that proves finalization burn logs are emitted in lexicographical address order when multiple accounts are marked for deletion in the same transaction. Parametrized over N in {2, 5}. N accounts are created and SELFDESTRUCT'd in the same tx, then funded via payer contracts called in REVERSE sorted address order with distinct nonzero amounts. Each destroyed account ends with a unique nonzero balance at finalization, so ordering by address vs. by call order is always distinguishable. Addresses issue #2691. * feat(tests): EIP-7708 - coinbase priority fee must not emit transfer log Adds a dedicated test proving the coinbase priority fee payment does not produce a Transfer log. A contract CALLs the coinbase address with nonzero value while the tx pays a nonzero priority fee to that same coinbase. Only the CALL-with-value must produce a Transfer log; the priority fee credit happens outside the EVM as a protocol-level balance change. An implementation that hooks every balance addition (instead of only CALL / SELFDESTRUCT / tx-level value transfers) would emit an extra Transfer log for the fee and fail the exact-log assertion. Addresses issue #2692. * feat(tests): add single account multi transfer test * fix(tests): minor nit --------- Co-authored-by: marioevz <marioevz@gmail.com> * fix(tests): rename GAS_CODE_DEPOSIT_PER_BYTE to CODE_DEPOSIT_PER_BYTE Align EIP-7708 selfdestruct finalization test with the gas constant rename on forks/amsterdam. * fix(tests): EIP-7708 + 8037 cross-EIP fixes * feat(tests): Add EIP-7708 checks to 6780 tests (#2743) * feat(tests): Add EIP-7708 checks to 6780 tests * fix: add another call and properly accumulate so that burn logs are validated * fix(tests): Review fixes --------- Co-authored-by: carsons-eels <carson@ethereum.org> * feat(tests): cover EIP-7708 CREATE log rollback on outer revert (#2785) * feat(tests): cover EIP-7708 CREATE log rollback on outer revert Mirror `test_inner_call_succeeds_outer_reverts_no_log` for the bal-devnet-5 update that brings CREATE/CREATE2 under EIP-7708. A factory CREATEs a child with value (the deployment succeeds and a `factory -> created` log is emitted in the child frame) and then REVERTs; the receipt must record no logs because the outer revert discards the factory's frame along with every log it produced. * feat(tests): Review comments --------- Co-authored-by: marioevz <marioevz@gmail.com> * fix(specs-spec, tests): CR comment fixes * Apply suggestion from @marioevz --------- Co-authored-by: spencer-tb <spencer.tb@ethereum.org> Co-authored-by: Mario Vega <marioevz@gmail.com> Co-authored-by: spencer <spencer.taylor-brown@ethereum.org> Co-authored-by: felipe <fselmo2@gmail.com> Co-authored-by: raxhvl <raxhvl@users.noreply.github.com> Co-authored-by: raxhvl <10168946+raxhvl@users.noreply.github.com> Co-authored-by: danceratopz <danceratopz@gmail.com>

Carsons-Eels added P-high C-test Category: test A-test-specs Area: execution_testing.specs labels Jan 22, 2026

spencer-tb force-pushed the extra-eip-7708-tests branch from 7884a30 to 0457741 Compare January 22, 2026 16:55

This was referenced Jan 22, 2026

chore(docs): bump docs fork to amsterdam #2064

Merged

EIP-7708 Implementation Tracker: ETH transfers emit a log #1875

Open

spencer-tb commented Jan 22, 2026

View reviewed changes

Carsons-Eels force-pushed the extra-eip-7708-tests branch from 0457741 to fca2300 Compare January 22, 2026 21:28

fselmo added this to the bal devnet-2 milestone Jan 22, 2026

fselmo force-pushed the extra-eip-7708-tests branch from fca2300 to 6ad08f9 Compare January 23, 2026 01:23

Carsons-Eels force-pushed the extra-eip-7708-tests branch from eedf23b to 26616d9 Compare January 23, 2026 04:04

marioevz force-pushed the eips/amsterdam/eip-7708 branch from 8af6e58 to bf2361b Compare January 23, 2026 04:12

spencer-tb and others added 2 commits January 22, 2026 21:38

feat(test-specs): add/refactor tests, add mainnet marked, checklist, …

04ebe7f

…coverage check

feat(test-specs): add fork transition test for selfdestruct logs

40ad1de

fselmo force-pushed the extra-eip-7708-tests branch from 26616d9 to 40ad1de Compare January 23, 2026 04:39

fix: tests

3eed7c6

marioevz approved these changes Jan 23, 2026

View reviewed changes

Comment thread tests/amsterdam/eip7708_eth_transfer_logs/test_transfer_logs.py

spencer-tb added 2 commits January 23, 2026 10:51

chore(test-specs): fix fork transition tests

df674f4

test(test-specs): add code deposit oog test case

e70030f

spencer-tb merged commit b61ff06 into ethereum:eips/amsterdam/eip-7708 Jan 23, 2026
3 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(test-specs): add extra eip-7708 test coverage#2062

feat(test-specs): add extra eip-7708 test coverage#2062
spencer-tb merged 5 commits into
ethereum:eips/amsterdam/eip-7708from
spencer-tb:extra-eip-7708-tests

spencer-tb commented Jan 22, 2026 •

edited

Loading

Uh oh!

spencer-tb Jan 22, 2026

Uh oh!

marioevz left a comment

Uh oh!

Uh oh!

codecov Bot commented Jan 23, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants



		@pytest.mark.valid_at_transition_to("Amsterdam")
		def test_transfer_log_fork_transition(

Conversation

spencer-tb commented Jan 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🗒️ Description

🔗 Related Issues or PRs

✅ Checklist

Cute Animal Picture

Uh oh!

spencer-tb Jan 22, 2026

Choose a reason for hiding this comment

Uh oh!

marioevz left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

codecov Bot commented Jan 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

spencer-tb commented Jan 22, 2026 •

edited

Loading

codecov Bot commented Jan 23, 2026 •

edited

Loading