-
Notifications
You must be signed in to change notification settings - Fork 20.5k
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
Access list state test format #22290
Conversation
bdb3577
to
f1e6e0e
Compare
On this PR, are you able to execute the tests in ethereum/tests#774 ? |
f1e6e0e
to
7bc462d
Compare
Rebased |
|
1f458a8
to
278351b
Compare
278351b
to
8f7dd05
Compare
It passes the newest version (ethereum/tests@0966f56) of the ethereum/tests flawlessly! |
Obviously this will need a rebase, and it's a bit hard to see the changes -- but did you add the new tests? If not, please add the updated tests (or maybe wait until they've been merged to master) |
Didn't add the tests yet, will do once they merge |
8f7dd05
to
3291e5e
Compare
Updates marshalling and execution for the new access list format in the state tests
depends on #21502