Skip to content

feat(tests): Add EIP-8163 EXTENSION opcode test - #3454

Open
pdobacz wants to merge 1 commit into
ethereum:forks/amsterdamfrom
monad-developers:eips/bogota/eip-8163
Open

pdobacz wants to merge 1 commit into
ethereum:forks/amsterdamfrom
monad-developers:eips/bogota/eip-8163

Conversation

@pdobacz

@pdobacz pdobacz commented Aug 26, 2026 •

Copy link
Copy Markdown
Contributor

Description

Adds a minimal spec test for EIP-8163: Reserve EXTENSION (0xae) opcode. Test only confirms same execution as INVALID (0xfe) and jumpdest analysis neutrality.

These tests are intended to pass both on Ethereum L1 EVM which must not define any 0xae extensions, as well as EVMs which do define 0xae extensions.

There are 4 tests and 721 state_test test cases generated, if this is too much for a no-op spec change, LMK.

Related Issues or PRs

#3453

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.

@codecov

codecov Bot commented Aug 26, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.62%. Comparing base (3650d3d) to head (019619a).

Additional details and impacted files
@@               Coverage Diff                @@
##           forks/amsterdam    #3454   +/-   ##
================================================
  Coverage            94.62%   94.62%           
================================================
  Files                  624      624           
  Lines                36976    36976           
  Branches              3349     3349           
================================================
  Hits                 34988    34988           
  Misses                1397     1397           
  Partials               591      591           
Flag Coverage Δ
unittests 94.62% <ø> (ø)

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.

Bogota placeholder fork and EXTENSION (0xae) in execution_testing, tests in tests/bogota/eip8163_extension.

Co-Authored-By: Claude Fable 5
@pdobacz
pdobacz force-pushed the eips/bogota/eip-8163 branch from 60fa8cb to 019619a Compare October 1, 2026 10:27

This branch has not been deployed

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants