Skip to content

refactor(test-benchmark): migrate benchmark suite to devnet-8 - #3411

Open
LouisTsai-Csie wants to merge 1 commit into
ethereum:forks/amsterdamfrom
LouisTsai-Csie:devnet-benchmark-refactor
Open

refactor(test-benchmark): migrate benchmark suite to devnet-8#3411
LouisTsai-Csie wants to merge 1 commit into
ethereum:forks/amsterdamfrom
LouisTsai-Csie:devnet-benchmark-refactor

Conversation

@LouisTsai-Csie

@LouisTsai-Csie LouisTsai-Csie commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Description

Makes tests/benchmark/ devnet-8 ready.

  • Bumps the benchmark EVM pin to glamsterdam-devnet-8.
  • Fixes loop and transaction sizing that assumed devnet-7 gas values, so the affected benchmarks fill their gas target again.
  • Revives test_block_full_access_list_and_data, which was skipped on every fork with an EIP-7934 block RLP size limit.
  • Derives the remaining hardcoded gas amounts and docstring values from the fork.

Related Issues or PRs

issue #3375

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

Put a link to a cute animal picture inside the parenthesis-->

@LouisTsai-Csie LouisTsai-Csie added C-refactor Category: refactor A-test-benchmark Area: execution_testing.benchmark and tests/benchmark labels Aug 21, 2026
@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.53%. Comparing base (3d473e8) to head (ddd2184).

Additional details and impacted files
@@               Coverage Diff                @@
##           forks/amsterdam    #3411   +/-   ##
================================================
  Coverage            93.53%   93.53%           
================================================
  Files                  624      624           
  Lines                37074    37074           
  Branches              3394     3394           
================================================
  Hits                 34679    34679           
  Misses                1645     1645           
  Partials               750      750           
Flag Coverage Δ
unittests 93.53% <ø> (ø)

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.

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

Labels

A-test-benchmark Area: execution_testing.benchmark and tests/benchmark C-refactor Category: refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant