fix(tests): Fix gas used/spent in EIP-7702 tests by marioevz · Pull Request #2068 · ethereum/execution-specs

marioevz · 2026-01-23T15:52:52Z

🗒️ Description

Fixes gas spent/used transaction receipt expectation under EIP-7778 for EIP-7702 tests.

🔗 Related Issues or PRs

N/A.

✅ 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

LGTM

fselmo

👍🏼

codecov · 2026-01-23T16:59:44Z

Codecov Report

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

Additional details and impacted files

@@                     Coverage Diff                     @@
##           eips/amsterdam/eip-7778    #2068      +/-   ##
===========================================================
- Coverage                    86.33%   86.14%   -0.20%     
===========================================================
  Files                          538      599      +61     
  Lines                        34557    39475    +4918     
  Branches                      3222     3780     +558     
===========================================================
+ Hits                         29835    34005    +4170     
- Misses                        4148     4848     +700     
- Partials                       574      622      +48

Flag	Coverage Δ
unittests	`86.14% <ø> (-0.20%)`	⬇️

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 approved these changes Jan 23, 2026

View reviewed changes

fselmo approved these changes Jan 23, 2026

View reviewed changes

github-actions Bot force-pushed the eips/amsterdam/eip-7778 branch from b2e2e92 to fa73e11 Compare January 23, 2026 17:17

fix(tests): Fix gas used/spent in EIP-7702 tests

66c783b

marioevz force-pushed the eip-7702-7778-tests-fix branch from b2185f7 to 66c783b Compare January 23, 2026 17:20

marioevz merged commit e7f7a31 into ethereum:eips/amsterdam/eip-7778 Jan 23, 2026
3 checks passed

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

fix(tests): Fix gas used/spent in EIP-7702 tests (#2068)

374a265

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

fix(tests): Fix gas used/spent in EIP-7702 tests (#2068)

6a12370

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

fix(tests): Fix gas used/spent in EIP-7702 tests (#2068)

f76875b

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

fix(tests): Fix gas used/spent in EIP-7702 tests (#2068)

1e7e36c

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

fix(tests): Fix gas used/spent in EIP-7702 tests (ethereum#2068)

7c18ee5

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

fix(tests): Fix gas used/spent in EIP-7702 tests (#2068)

31621f7

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

fix(tests): Fix gas used/spent in EIP-7702 tests (#2068)

3107399

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

fix(tests): Fix gas used/spent in EIP-7702 tests (#2068)

9218535

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

fix(tests): Fix gas used/spent in EIP-7702 tests (#2068)

6f2603b

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

fix(tests): Fix gas used/spent in EIP-7702 tests (#2068)

41f8a24

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

fix(tests): Fix gas used/spent in EIP-7702 tests (ethereum#2068)

701bc2d

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

fix(tests): Fix gas used/spent in EIP-7702 tests (ethereum#2068)

119fbcd

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

fix(tests): Fix gas used/spent in EIP-7702 tests (ethereum#2068)

feb0c9e

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

fix(tests): Fix gas used/spent in EIP-7702 tests (ethereum#2068)

a30eb0d

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

fix(tests): Fix gas used/spent in EIP-7702 tests (ethereum#2068)

8b1c4c7

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

fix(tests): Fix gas used/spent in EIP-7702 tests (ethereum#2068)

a5d0f82

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

fix(tests): Fix gas used/spent in EIP-7702 tests (ethereum#2068)

838f3c6

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

fix(tests): Fix gas used/spent in EIP-7702 tests (ethereum#2068)

af76f5e

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

fix(tests): Fix gas used/spent in EIP-7702 tests (#2068)

5ea7306

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

fix(tests): Fix gas used/spent in EIP-7702 tests (#2068)

a7522f7

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

fix(tests): Fix gas used/spent in EIP-7702 tests (#2068)

61d417f

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

fix(tests): Fix gas used/spent in EIP-7702 tests (ethereum#2068)

45126e3

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

fix(tests): Fix gas used/spent in EIP-7702 tests (#2068)

30bb980

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(tests): Fix gas used/spent in EIP-7702 tests#2068

fix(tests): Fix gas used/spent in EIP-7702 tests#2068
marioevz merged 1 commit into
ethereum:eips/amsterdam/eip-7778from
marioevz:eip-7702-7778-tests-fix

marioevz commented Jan 23, 2026

Uh oh!

LouisTsai-Csie left a comment

Uh oh!

fselmo left a comment

Uh oh!

codecov Bot commented Jan 23, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

marioevz commented Jan 23, 2026

🗒️ Description

🔗 Related Issues or PRs

✅ Checklist

Cute Animal Picture

Uh oh!

LouisTsai-Csie left a comment

Choose a reason for hiding this comment

Uh oh!

fselmo left a comment

Choose a reason for hiding this comment

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!

3 participants

codecov Bot commented Jan 23, 2026 •

edited

Loading