-
Notifications
You must be signed in to change notification settings - Fork 91
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
feat(tests): json schema verification #968
base: main
Are you sure you want to change the base?
Conversation
1538efd
to
31273b7
Compare
3e8a429
to
80a49e6
Compare
from typing import Any, Optional | ||
|
||
from pydantic import BaseModel, Field, model_validator | ||
from rlp import decode as rlp_decode |
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.
To fix the mypy issues, we should be able to extend ./stubs/
folder with the functions used here.
rlp
- https://pypi.org/project/rlp/ doesn't appear to support typing yet, ethereum/pyrlp#143
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.
I added one, check if it is how it supposed to be.
but still it does not detect in vscode where rlp is supposed to be imported from.
but strangely all works
ee69e26
to
e014743
Compare
🗒️ Description
Pydanticfy the json schemas so we verify the quality of generated test fixtures
This is an MVP to catch what I think is the cause of bug
pyspecs with evmone-t8n export extra field on all forks which should not be there. perhaps this extra field leads to wrong block init in its' t8n when we blindly feeding the env section. and to a wrong state hash in my test scenarios.
so the issue is
evmone-t8n return
on all forks. and we export it to the final test. when this field is only in blockheader in exported test since London
🔗 Related Issues
#940
✅ Checklist
mkdocs serve
locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.