feat(spec-tests): add basic eip7778 test by LouisTsai-Csie · Pull Request #2045 · ethereum/execution-specs

LouisTsai-Csie · 2026-01-20T08:45:57Z

🗒️ Description

Due to an incorrect git operation, PR #2026 was closed. I’ve recreated the PR with the same content and updated the related links to point to this new PR.

Implement eip-7778 test cases.

test_simple_gas_accounting: User should receive the refund, block accounting should exclude refund.
test_multi_block_gas_accounting: Block accounting should exclude refund, and the transaction receipt should reflect the cumulative gas usage.

🔗 Related Issues or PRs

Require PR #1401
Tracker:

EIP-7778 Implementation Tracker: Block Gas Accounting without Refunds #1874

✅ 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.
Tests: For PRs implementing a missed test case, update the post-mortem document to add an entry the list.
Ported Tests: All converted JSON/YML tests from ethereum/tests or tests/static have been assigned @ported_from marker.

Cute Animal Picture

LouisTsai-Csie · 2026-01-20T08:55:42Z

Link @marioevz 's comment from old PR.

#2026 (comment)

danceratopz · 2026-01-20T10:28:59Z

Due to an incorrect git operation, PR #2026 was closed. I’ve recreated the PR with the same content and updated the related links to point to this new PR.

No worries @LouisTsai-Csie! Just to better understand, is this the status quo:

feat(spec-tests): add eip7778 simple test #2026 got accidentally merged into https://github.com/ethereum/execution-specs/tree/eips/amsterdam/eip-7778
You cleaned up the history of eips/amsterdam/eip-7778 (as I can't see the merged changes there).
This PR replaces feat(spec-tests): add eip7778 simple test #2026.

If that's the case, we should update this PR's target branch to eips/amsterdam/eip-7778.

danceratopz · 2026-01-20T10:54:47Z

Nerolation just merged #1401 into eips/amsterdam/eip-7778. @LouisTsai-Csie I think you need to update the target branch to eips/amsterdam/eip-7778 and rebase to get those changes.

gurukamath

This looks good to me! One suggestion would be to keep track of the implemented tests and further test ideas using a BAL style test tracking table. See here.

codecov · 2026-01-21T11:05:25Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.07%. Comparing base (e958b62) to head (13f2920).
⚠️ Report is 85 commits behind head on eips/amsterdam/eip-7778.

Additional details and impacted files

@@                     Coverage Diff                     @@
##           eips/amsterdam/eip-7778    #2045      +/-   ##
===========================================================
- Coverage                    86.33%   86.07%   -0.27%     
===========================================================
  Files                          538      599      +61     
  Lines                        34557    39530    +4973     
  Branches                      3222     3780     +558     
===========================================================
+ Hits                         29835    34024    +4189     
- Misses                        4148     4872     +724     
- Partials                       574      634      +60

Flag	Coverage Δ
unittests	`86.07% <ø> (-0.27%)`	⬇️

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.

spencer-tb

Some small cosmetic changes. LGTM otherwise.

marioevz

Added many changes on top of your cases @LouisTsai-Csie

…est_gas_accounting.py

fselmo

I think really just the pytest-xdist issue jumped out at me.

Some other cases to think about for a subsequent PR:

multiple txs with refunds in same block
- verifying each tx receipt's gas_spent and gas_used
- verify cumulative_gas_used across transactions (I wonder if this is a good time to add validation for this in expected_receipt)?
combine refund types for the fork in a single tx

Ah did not mean to approve. Please fix pytest-xdist issue. Other than that, this is a great addition!

codecov · 2026-01-22T14:09:18Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.07%. Comparing base (e958b62) to head (13f2920).
⚠️ Report is 86 commits behind head on eips/amsterdam/eip-7778.

Additional details and impacted files

@@                     Coverage Diff                     @@
##           eips/amsterdam/eip-7778    #2045      +/-   ##
===========================================================
- Coverage                    86.33%   86.07%   -0.27%     
===========================================================
  Files                          538      599      +61     
  Lines                        34557    39530    +4973     
  Branches                      3222     3780     +558     
===========================================================
+ Hits                         29835    34024    +4189     
- Misses                        4148     4872     +724     
- Partials                       574      634      +60

Flag	Coverage Δ
unittests	`86.07% <ø> (-0.27%)`	⬇️

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.

@gurukamath

…ut refunds (#2840) * feat(spec-specs): Implement EIP-7778 Block Gas Accounting without Refunds (#1401) * feat(specs): add eip-7778 implementation * fix(specs) spliting receipt gas from block gas * revert receipt_gas_used * make sure that the calldata floor cost overwrites the tx gas before refund * receipt gas used after refunds * clarify variable naming @gurukamath * revert to using gas used after refunds for cumulative gas in receipt and add gasSpent to receipt * spec(amsterdam): fix code formatting --------- Co-authored-by: Guruprasad Kamath <guru241987@gmail.com> * fix(tests): fix legacy tests to run with eip-7778 (#2048) * fix(tests): fix legacy tests to run with eip-7778 * fix(tests): check gas_spent and gas_used after Amsterdam * feat(spec-tests): add basic eip7778 test (#2045) * feat(spec-tests): add eip-7778 tests for calldata checks (#2060) * feat(spec-tests): add eip-7778 for calldata checks * feat(spec-tests): post review - use code gas_cost function Co-authored-by: Louis Tsai <72684086+LouisTsai-Csie@users.noreply.github.com> * Apply suggestions from code review --------- Co-authored-by: Louis Tsai <72684086+LouisTsai-Csie@users.noreply.github.com> Co-authored-by: Mario Vega <marioevz@gmail.com> * fix(tests): Fix gas used/spent in EIP-7702 tests (#2068) * feat(spec-specs): use post refund gas in receipts in EIP-7778 (#2073) * fix(test): Update refspec for EIP-7778 to match latest revision (#2093) * feat(test-forks): Add EIP-7778 * refactor(tests): Condition EIP-7778 tests to EIP inclusion * fix(test): use deterministic iteration order for refund types The set iteration in `build_refund_tx` is non-deterministic due to Python's hash randomization, causing fixture output to vary between runs. Sort by enum value to ensure reproducible fixtures. * doc: remove changelog md * fix: broken auth tx benchmark * doc: add eip checklist * fix: typo * Update tests/benchmark/compute/scenario/test_transaction_types.py * Apply suggestions from code review Co-authored-by: Mario Vega <marioevz@gmail.com> --------- Co-authored-by: Toni Wahrstätter <51536394+nerolation@users.noreply.github.com> Co-authored-by: Guruprasad Kamath <guru241987@gmail.com> Co-authored-by: Guruprasad Kamath <48196632+gurukamath@users.noreply.github.com> Co-authored-by: Mario Vega <marioevz@gmail.com> Co-authored-by: felipe <fselmo2@gmail.com> Co-authored-by: spencer-tb <spencer.tb@ethereum.org>

LouisTsai-Csie mentioned this pull request Jan 20, 2026

feat(spec-tests): add eip7778 simple test #2026

Merged

8 tasks

danceratopz added this to the bal devnet-2 milestone Jan 20, 2026

LouisTsai-Csie changed the base branch from forks/amsterdam to eips/amsterdam/eip-7778 January 21, 2026 09:51

LouisTsai-Csie added 2 commits January 21, 2026 17:52

feat: add basic eip7778 test case

30545fd

doc: add changelog

73ab16d

LouisTsai-Csie force-pushed the eip7778-test branch from 112a7dd to 73ab16d Compare January 21, 2026 09:56

LouisTsai-Csie requested a review from gurukamath January 21, 2026 09:57

LouisTsai-Csie self-assigned this Jan 21, 2026

fix: linting issue

6599522

gurukamath approved these changes Jan 21, 2026

View reviewed changes

spencer-tb approved these changes Jan 21, 2026

View reviewed changes

Comment thread tests/amsterdam/eip7778_block_gas_accounting_without_refunds/test_gas_accounting.py Outdated

Comment thread packages/testing/src/execution_testing/specs/blockchain.py Outdated

spencer-tb mentioned this pull request Jan 21, 2026

STEEL Weekly Meeting, Wed 21st Jan ethsteel/pm#39

Closed

LouisTsai-Csie and others added 2 commits January 22, 2026 00:27

fix: comment issue

78b1713

fix(testing/specs): Expected gas used as block property

e4a602d

marioevz self-requested a review January 21, 2026 21:54

marioevz added 5 commits January 21, 2026 22:38

feat(testing/forks): Refund types parametrizer

7d860fd

feat: implement many new cases

b36a9c1

fix(docs): tox

7336559

revert removal of docstring (prob. mistake)

0e8ffbd

fix: correct refunds on forks

27a1a4e

marioevz reviewed Jan 22, 2026

View reviewed changes

Comment thread packages/testing/src/execution_testing/specs/blockchain.py Outdated

Comment thread tests/amsterdam/eip7778_block_gas_accounting_without_refunds/test_gas_accounting.py Outdated

Update tests/amsterdam/eip7778_block_gas_accounting_without_refunds/t…

a17a587

…est_gas_accounting.py

marioevz requested a review from spencer-tb January 22, 2026 00:29

fselmo previously approved these changes Jan 22, 2026

View reviewed changes

Comment thread tests/amsterdam/eip7778_block_gas_accounting_without_refunds/test_gas_accounting.py Outdated

Comment thread packages/testing/src/execution_testing/forks/forks/forks.py Outdated

Comment thread packages/testing/src/execution_testing/forks/forks/forks.py Outdated

fix: pytest-xdist issue

13f2920

LouisTsai-Csie merged commit abca655 into ethereum:eips/amsterdam/eip-7778 Jan 22, 2026
17 of 18 checks passed

gurukamath pushed a commit that referenced this pull request Jan 22, 2026

feat(spec-tests): add basic eip7778 test (#2045)

927849e

github-actions Bot pushed a commit that referenced this pull request Jan 22, 2026

feat(spec-tests): add basic eip7778 test (#2045)

afed200

fselmo pushed a commit that referenced this pull request Jan 23, 2026

feat(spec-tests): add basic eip7778 test (#2045)

af1956c

github-actions Bot pushed a commit that referenced this pull request Jan 23, 2026

feat(spec-tests): add basic eip7778 test (#2045)

3c97a89

github-actions Bot pushed a commit that referenced this pull request Jan 26, 2026

feat(spec-tests): add basic eip7778 test (#2045)

60cdeb3

marioevz pushed a commit that referenced this pull request Jan 29, 2026

feat(spec-tests): add basic eip7778 test (#2045)

87b02dd

fselmo pushed a commit that referenced this pull request Feb 3, 2026

feat(spec-tests): add basic eip7778 test (#2045)

21f252f

spencer-tb pushed a commit that referenced this pull request Feb 11, 2026

feat(spec-tests): add basic eip7778 test (#2045)

57515a7

fselmo pushed a commit to fselmo/execution-specs that referenced this pull request Feb 12, 2026

feat(spec-tests): add basic eip7778 test (ethereum#2045)

61ed005

fselmo pushed a commit that referenced this pull request Feb 12, 2026

feat(spec-tests): add basic eip7778 test (#2045)

6f8d0f4

fselmo pushed a commit that referenced this pull request Feb 13, 2026

feat(spec-tests): add basic eip7778 test (#2045)

58bb7ba

gurukamath pushed a commit that referenced this pull request Feb 24, 2026

feat(spec-tests): add basic eip7778 test (#2045)

3590fd8

gurukamath pushed a commit that referenced this pull request Feb 25, 2026

feat(spec-tests): add basic eip7778 test (#2045)

17807a3

github-actions Bot pushed a commit that referenced this pull request Feb 27, 2026

feat(spec-tests): add basic eip7778 test (#2045)

7e379da

spencer-tb pushed a commit to spencer-tb/execution-specs that referenced this pull request Mar 3, 2026

feat(spec-tests): add basic eip7778 test (ethereum#2045)

a912417

spencer-tb pushed a commit to spencer-tb/execution-specs that referenced this pull request Mar 6, 2026

feat(spec-tests): add basic eip7778 test (ethereum#2045)

8e4a97b

spencer-tb pushed a commit to spencer-tb/execution-specs that referenced this pull request Mar 10, 2026

feat(spec-tests): add basic eip7778 test (ethereum#2045)

518f11f

fselmo pushed a commit to fselmo/execution-specs that referenced this pull request Mar 10, 2026

feat(spec-tests): add basic eip7778 test (ethereum#2045)

734702d

fselmo pushed a commit to fselmo/execution-specs that referenced this pull request Mar 10, 2026

feat(spec-tests): add basic eip7778 test (ethereum#2045)

1f41b2f

fselmo pushed a commit to fselmo/execution-specs that referenced this pull request Mar 13, 2026

feat(spec-tests): add basic eip7778 test (ethereum#2045)

38addac

fselmo pushed a commit to fselmo/execution-specs that referenced this pull request Mar 17, 2026

feat(spec-tests): add basic eip7778 test (ethereum#2045)

12a9640

fselmo pushed a commit to fselmo/execution-specs that referenced this pull request Mar 18, 2026

feat(spec-tests): add basic eip7778 test (ethereum#2045)

08cfa8a

marioevz pushed a commit that referenced this pull request Mar 24, 2026

feat(spec-tests): add basic eip7778 test (#2045)

579772f

marioevz pushed a commit that referenced this pull request Apr 8, 2026

feat(spec-tests): add basic eip7778 test (#2045)

e208ed6

github-actions Bot pushed a commit that referenced this pull request Apr 10, 2026

feat(spec-tests): add basic eip7778 test (#2045)

f94106b

spencer-tb pushed a commit to spencer-tb/execution-specs that referenced this pull request Apr 20, 2026

feat(spec-tests): add basic eip7778 test (ethereum#2045)

519be09

LouisTsai-Csie added a commit that referenced this pull request May 13, 2026

feat(spec-tests): add basic eip7778 test (#2045)

83aee58

Conversation

LouisTsai-Csie commented Jan 20, 2026 • edited by danceratopz 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!

LouisTsai-Csie commented Jan 20, 2026

Uh oh!

danceratopz commented Jan 20, 2026

Uh oh!

danceratopz commented Jan 20, 2026

Uh oh!

gurukamath left a comment

Choose a reason for hiding this comment

Uh oh!

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

Uh oh!

Codecov Report

Uh oh!

spencer-tb left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

marioevz left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

fselmo left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

codecov Bot commented Jan 22, 2026

Codecov Report

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

LouisTsai-Csie commented Jan 20, 2026 •

edited by danceratopz

Loading

codecov Bot commented Jan 21, 2026 •

edited

Loading