-
Notifications
You must be signed in to change notification settings - Fork 62
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
test eof container custom_size field #620
Conversation
8a5b7af
to
0579b95
Compare
tests/prague/eip7692_eof_v1/eip3540_eof_v1/test_example_valid_invalid.py
Show resolved
Hide resolved
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.
Some comments but overall it looks good π
tests/prague/eip7692_eof_v1/eip3540_eof_v1/test_example_valid_invalid.py
Show resolved
Hide resolved
tests/prague/eip7692_eof_v1/eip3540_eof_v1/test_section_size.py
Outdated
Show resolved
Hide resolved
tests/prague/eip7692_eof_v1/eip3540_eof_v1/test_section_size.py
Outdated
Show resolved
Hide resolved
tests/prague/eip7692_eof_v1/eip3540_eof_v1/test_section_size.py
Outdated
Show resolved
Hide resolved
tests/prague/eip7692_eof_v1/eip3540_eof_v1/test_section_size.py
Outdated
Show resolved
Hide resolved
hm, maybe create some test IntEnum with str method predefined so not to have this str method defined in test file. appled suggestions |
0579b95
to
6560c13
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!
ποΈ Description
test eof container custom_size field
this analyze ori test collection and extend with pyspec his test cases
π Related Issues
β Checklist
mkdocs serve
locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.