-
Notifications
You must be signed in to change notification settings - Fork 20k
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
cmd/evm: Add --single-test
option to blocktest
#28421
Conversation
@marioevz I made a bit of a modification to follow the -- If others want to play along at home, you can verify this is working with $ go run ./cmd/evm blocktest --run 100-fork=Cancun-empty_memory-length=0-src=0-dest=0 ./tests/spec-tests/fixtures/cancun/eip5656_mcopy/mcopy/mcopy_on_empty_memory.json |
Thanks for the help @lightclient ! It looks fine to me but I'm tagging @danceratopz since he's working on the feature that uses this change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like 👍 from Dan too
Yup, works nicely on our side, thanks! |
Co-authored-by: lightclient <lightclient@protonmail.com>
This reverts commit 36d01e7.
This reverts commit 36d01e7.
Co-authored-by: lightclient <lightclient@protonmail.com>
Adds
--single-test
option toevm blocktest
to run a single test from a given json BlockchainTest file.This allows execution-spec-tests to verify each of the tests it produces individually, because at the moment we verify the entire json file and it's not simple to detect which one of the included tests was the one that failed.