forked from ethereum/execution-spec-tests
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Refactor package and pytest config files (mario + dan) #11
Merged
danceratopz
merged 15 commits into
feature/use-pytest-to-collect-and-hydrate-test-fillers-v2
from
refactor-pytest-config-files-dan
May 12, 2023
Merged
Refactor package and pytest config files (mario + dan) #11
danceratopz
merged 15 commits into
feature/use-pytest-to-collect-and-hydrate-test-fillers-v2
from
refactor-pytest-config-files-dan
May 12, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The top-level package 'test-filler' included the sub-packages defined in setup.cfg (evm_block_builder, evm_transition_tool, ethereum_test_forks, ethereum_test_tools and pytest_plugins), but didn't include any of the sub-packages contained within these (ethereum_test_tools.forks for example). As tox ran on an installed (incorrect) build of this package, any commands ran that triggered imports of these sub-packages failed, as they were simply not present. Now setuptools 'find_packages' automatically includes all sub-packages contained in these directories.
This fixes mypy errors for fillers that haven't yet been converted to pytest format. Note that the import of the decorators (test_from, etc) and the decorators.py module are flagged to be ignored by mypy using a '# type: ignore' line. The errors are caused because the types in the fill_test function signature have changed. This code should be removed again before merging the pytest feature branch to main.
marioevz
approved these changes
May 12, 2023
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.
This is just absolutely perfect, it's amazing how everything just works 😀
Just a small thing I'd like to get your opinion on regarding the settings.json
, but otherwise I think it's very very good and we should merge this.
5 tasks
73bc923
into
feature/use-pytest-to-collect-and-hydrate-test-fillers-v2
2 checks passed
danceratopz
added a commit
that referenced
this pull request
Apr 23, 2024
* feat(all): add EOF fixture format and test spec * feat(tests): add EOF example * feat(fw): EOF library tools/vm: EOF opcode changes tools: CREATE3 helper tox: EOF Keywords post rebase changes: align EOF tests with main. Co-authored-by: spencer-tb <spencer@spencertaylorbrown.uk> post rebase: tox fixes. Co-authored-by: spencer-tb <spencer@spencertaylorbrown.uk> Fix existing EOF tests (#11) Co-authored-by: Hugo <jhugodc@gmail.com> * new(tests): prague: EOF V1 tests tests/eof: move to prague tests/eof: fixes post rebase changes: align EOF tests with main. (tests) Co-authored-by: spencer-tb <spencer@spencertaylorbrown.uk> post rebase: tox fixes (tests). Co-authored-by: spencer-tb <spencer@spencertaylorbrown.uk> Fix existing EOF tests (tests) (#11) Co-authored-by: Hugo <jhugodc@gmail.com> * refactor(fw): EOF container code * fix(fw): vm: re-order JUMPDEST, NOOP * fix(fw): cli tests * fix(tests): refactor fixes * fix(fw): eof: fix raw bytes processing * fix(tests): eof: change `test_code_validation.py` to use `EOFTestFiller` * eof unit tests * fix eof exceptions * refactor(fw): compute_create3_address * fix(tests): tox * refactor(fw): eof_v1: use pydantic * fix(fw): tpx * fix(fw): RJUMPV custom dataportion parser * fix(tests): eofv1: RJUMPI calls * new(fw): RJUMPV unit test * feat(fw): subscriptable opcodes * fix(fw): Opcode add, mul * fix(tests): test hardcoding bytecode of initcode * fix(fw): fix * fix(tests): eof: fix RJUMP*, CALLF data portions * hack(fw): Add str as allowed EOF exception * fix(tests): eof: fix test_execution_function.py * changelog * chore: add unknown words to dictionary * fetch exceptions with evmone lots of TODOs * eof .py test example * fix(fw): add unchecked_stack opcode property (CALLF) * refactor(fw): eof: refactor and add section creation helpers * fix: tox * fix(tests): eof: refactor * refactor(tests): replace Section(kind=, with Section.Code, Section.Data * fix(fw): eof: consistent magic, version, and header terminator types. * fix(fw): eof: proper magic * rm(tests): eof: duplicate prague folder with redundant examples * fix(fw): eof: data-container body order * docs: add EOF * fix(docs): word * fix(tests): eof: file rename --------- Co-authored-by: Mario Vega <marioevz@gmail.com> Co-authored-by: Hugo <jhugodc@gmail.com> Co-authored-by: danceratopz <danceratopz@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This builds on the initial branch and work from @marioevz in marioevz:refactor-pytest-config-files.