Skip to content

docs: some ideas on moving ./tests to ethereum/execution-specs #1645

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

danceratopz
Copy link
Member

πŸ—’οΈ Description

Just some thoughts about what we envision "The Merge" (or depending on your POV, the "The Move" πŸ˜†) should look like and what it requires.

Not complete by any means, just thought it could help serve as a basis for discussion!

Feel free to add comments to this PR or in Discord!

Background

For those unfamiliar with the discussion: The plan is to move the EL client test cases in ./tests/ to ethereum/execution-specs.

Here's the main motivation for this change (these are overlapping):

  1. Spec and tests versioned together. Less ambiguous.
  2. EIP authors/implementers:
    i. Need only concern themselves with one repo.
    ii. Can directly & very easily add basic tests to sanity check their implementations.
  3. This removes the chicken & egg problem when testing the spec: To test an EELS branch you need an EEST release which needs an EELS branch ♻️.
  4. Simplifies tooling and debugging: Removes the need for petertdavies/ethereum-spec-evm-resolver, which currently allows us to fill multiple EELS implementations from multiple repos/branches within one test session.

πŸ”— Related Issues

Current sub-tasks:

There's no plan to merge this PR - but we can update the doc with comments to gain a shared understanding!

@danceratopz danceratopz added scope:tests Scope: Changes EL client test cases in `./tests` needs-discussion Needs discussion before proceeding type:docs Type: Changes to the documentation labels May 23, 2025
@danceratopz danceratopz marked this pull request as draft May 23, 2025 07:13
@winsvega
Copy link
Contributor

So the execution-spec-tests will be used as a package in execution-specs ?

@danceratopz
Copy link
Member Author

So the execution-spec-tests will be used as a package in execution-specs ?

Yes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-discussion Needs discussion before proceeding scope:tests Scope: Changes EL client test cases in `./tests` type:docs Type: Changes to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants