Skip to content

feat(tests): add execution-dominant EIP-7981 block gas coverage - #62

Closed
spencer-tb wants to merge 1 commit into
danceratopz:7981-improve-testsfrom
spencer-tb:tests/eip7981-execution-dominance
Closed

spencer-tb wants to merge 1 commit into
danceratopz:7981-improve-testsfrom
spencer-tb:tests/eip7981-execution-dominance

Conversation

@spencer-tb

Copy link
Copy Markdown

Description

Add a standalone EIP-7981 test for execution-dominant block gas accounting with nonzero state gas. A storage write is funded from an over-cap reservoir, and additional execution work makes execution gas exceed both the calldata floor and state gas.

For transaction types 1 and 2, assert the storage update, successful receipt with execution plus state gas, and a block header charging execution gas alone. This catches an implementation that uses only the floor for the execution dimension or includes state gas in the execution charge.

This is an independent test addition on top of 7981-improve-tests. It contains no changes from the review-fix branch.

Validation:

  • 553/553 EIP-7981 fill cases pass at Amsterdam, including six cases from the new test.
  • just static passes.
  • A temporary mutation forcing the execution dimension to the floor fails all four new blockchain/Engine header checks.
  • Client consumption has not been run for the new cases.

Related Issues or PRs

Checklist

  • Ran fast static checks to avoid CI fails, see Code Standards & Verifying Changes: just static
  • PR title has the form <type>(<area>): <title>, where <type> and <area> come from an appropriate C-<type>, respectively A-<area>, label. The title should match the target squash commit message.

Cute Animal Picture

Animal from the parent PR

@spencer-tb spencer-tb closed this Sep 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant