Skip to content

feat(tests): implement eip-8070 sparse blob pool tests - #2948

Merged
spencer-tb merged 4 commits into
ethereum:forks/amsterdamfrom
LouisTsai-Csie:implement-eip-8070
Jul 29, 2026
Merged

spencer-tb merged 4 commits into
ethereum:forks/amsterdamfrom
LouisTsai-Csie:implement-eip-8070

Conversation

@LouisTsai-Csie

@LouisTsai-Csie LouisTsai-Csie commented Jun 2, 2026 •

Copy link
Copy Markdown
Contributor

🗒️ Description

Add test coverage for EIP-8070: eth/72 - Sparse Blobpool, specifically the new engine_getBlobsV4 engine API endpoint.

🔗 Related Issues or PRs

N/A.

✅ Checklist

  • All: Ran fast static checks to avoid unnecessary CI fails, see also Code Standards and Enabling Pre-commit Checks:
    just 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).

@codecov

codecov Bot commented Jun 2, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.42%. Comparing base (d3baec8) to head (887d193).

Additional details and impacted files
@@               Coverage Diff                @@
##           forks/amsterdam    #2948   +/-   ##
================================================
  Coverage            93.42%   93.42%           
================================================
  Files                  625      625           
  Lines                37032    37032           
  Branches              3385     3385           
================================================
  Hits                 34597    34597           
  Misses                1666     1666           
  Partials               769      769           
Flag Coverage Δ
unittests 93.42% <ø> (ø)

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

☔ View full report in Codecov by Harness.
📢 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.

@LouisTsai-Csie
LouisTsai-Csie marked this pull request as ready for review June 25, 2026 06:06
@LouisTsai-Csie LouisTsai-Csie added A-spec-specs Area: Specification—The Ethereum specification itself (eg. `src/ethereum/*`) A-spec-tests Area: tests for specifications e.g. json_infra C-feat Category: an improvement or new feature and removed A-spec-specs Area: Specification—The Ethereum specification itself (eg. `src/ethereum/*`) labels Jun 25, 2026
@spencer-tb spencer-tb added this to the Glamsterdam Devnet 7 Finalize milestone Jul 2, 2026
@spencer-tb
spencer-tb force-pushed the implement-eip-8070 branch from c9e4983 to fd794e5 Compare July 29, 2026 14:01
@spencer-tb
spencer-tb force-pushed the implement-eip-8070 branch from fd794e5 to 887d193 Compare July 29, 2026 17:38
@spencer-tb spencer-tb changed the title feat(tests,spec): implement eip-8070 sparse blob pool infra feat(tests): implement eip-8070 sparse blob pool tests Jul 29, 2026
@spencer-tb spencer-tb added the A-test-rpc Area: execution_testing.rpc label Jul 29, 2026
@spencer-tb spencer-tb added A-tests Area: Consensus tests. and removed A-spec-tests Area: tests for specifications e.g. json_infra labels Jul 29, 2026

@spencer-tb spencer-tb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🗡️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-test-rpc Area: execution_testing.rpc A-tests Area: Consensus tests. C-feat Category: an improvement or new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants