Skip to content
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

Implement T8N support (tracking issue) #3602

Closed
jochem-brouwer opened this issue Aug 17, 2024 · 0 comments · Fixed by #3603
Closed

Implement T8N support (tracking issue) #3602

jochem-brouwer opened this issue Aug 17, 2024 · 0 comments · Fixed by #3603

Comments

@jochem-brouwer
Copy link
Member

jochem-brouwer commented Aug 17, 2024

EthereumJS currently does not T8N support, which is the "transition-tool". This would be incredibly helpful, especially regarding filling tests on the execution-spec-tests side to help filling the tests. If the tests on their side are written, but no client implementation is ready (or: does not have t8n support), then the tests cannot be filled and thus other clients cannot consume those tests (in the form of fixtures, i.e. state, blockchain (and hive) tests).

T8Ntool docs: https://ethereum-tests.readthedocs.io/en/latest/t8ntool-ref.html

execution-spec-tests will need a EthereumJS t8n wrapper: https://github.com/ethereum/execution-spec-tests/tree/main/src/evm_transition_tool

If done rather quickly we could use this to fill the EIP-7702 tests, as in this execution-spec-tests PR: ethereum/execution-spec-tests#733. This will need #3581

We have retesteth support which looks like t8n-like: #2818 (sample PR for fixups, but ./packages/vm/test/retesteth)

Add t8ntool to client docs from ethereum-tests https://ethereum-tests.readthedocs.io/en/develop/t8ntool.html

NOTE: this issue is WIP

CC @spencer-tb

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

Successfully merging a pull request may close this issue.

1 participant