test(fw): add/refactor test_filling tests #380
Merged
Conversation
This file contains hidden or 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
* ethereum_test_forks: implement fork operators, remove `is_fork` * changelog
* feat: add evm bytes to python converter * entry_points/tests: add tests for evm bytes parser. * evm_bytes_to_python: Add help * changelog --------- Co-authored-by: spencer-tb <[email protected]>
* all: abstract fixtures, convert StateTest into a BlockchainTest generator * fix: tox
…xture formats; Per-format sub-folder output (ethereum#358) * tools: Move fixture collector to spec * refactor: Fixtures collector * all: remove `BaseTestConfig` * fix: all filler tests * docs: update * tox: spelling * feat: Parametrize the supported fixture types for each test type * fix: many tests * fix: remaining tests * fix: fixture name case * fix: Indirect parametrize using fixture format * feat: Dynamic generation of spec test fixtures * fix: type-checking * fix: imports * feat(pytest): allow filtering by fixture format via pytest mark (#12) * feat(pytest): allow filtering by fixture format via pytest mark * feat(fw): give unset fixture format a more verbose name * feat(pytest): only paramatrize post-merge forks with hive format (#13) * feat(pytest): remove items which are pre-merge and have hive format * chore: make exception string clearer; fix whitespace * fix(pytest): test_filler: remove `enable_hive` completely * fix: Remove unused `STATE_TEST_HIVE` --------- Co-authored-by: danceratopz <[email protected]>
Co-authored-by: danceratopz <[email protected]>
Co-authored-by: spencer-tb <[email protected]>
Co-authored-by: spencer <[email protected]>
all: Support `ethereum/tests` StateTest format
spencer-tb
changed the title
…dd contract creation type-3 test (ethereum#379) * fix(tests): 4844: fix gas calculation * tests: 4844: Add more blob tx gas fee calculation scenarios * changelog * spelling * tools(feat): Allow txs to have predefined RLP * fix(tests): Fix type-3 contract creating test * typing fix * changelog
Co-authored-by: danceratopz <[email protected]>
|
I think we should improve the underlying exception classes in order to make them easier to test. It took me a bit longer than expected to understand what is being tested 😆 I always manage to find more work 😅 Will get back to this with an idea. |
|
Oh sounds interesting! More exception classes sound ideal to be honest |
No new classes needed upfront. But if you make the Exception class a dataclass, you get an |
6 tasks
4 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
🗒️ Description
Refactor of the framework unit tests within: #350