-
Notifications
You must be signed in to change notification settings - Fork 82
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
new(tests): EIP-7069 and EIP-7620 - failures and context vars #836
Conversation
a681c3e
to
f7fd6b6
Compare
f7fd6b6
to
9ed640d
Compare
@marioevz When you have a moment I'd appreciate your review on this one :). |
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 good overall, a few comments, thanks!
tests/prague/eip7692_eof_v1/eip7620_eof_create/test_eofcreate.py
Outdated
Show resolved
Hide resolved
b3ad2ff
to
19dd416
Compare
89417f0
to
d91b5d3
Compare
Co-authored-by: Mario Vega <[email protected]>
d91b5d3
to
df9a851
Compare
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.
LGTM, thanks for the extra test case! :)
🗒️ Description
Fills in some missing coverage regarding EXT*CALL and EOFCREATE failure modes and handling of context variables in the child context, which was reported missing recently
NOTE: should probably go after #834 so I can rebase on top and use the unified constants
🔗 Related Issues
NA
✅ Checklist
mkdocs serve
locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.