-
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
Enable tests that are _still_ disabled #2
Enable tests that are _still_ disabled #2
Conversation
Tests: https://circleci.com/gh/veox/py-evm/tree/investigate-create2-failures (Does this link work for anyone but me?..) |
The link works for me! |
I see at least one CI test job fails there - There are many more failing cases, not just |
Applying this PR increases the failure count from 10 (as in ethereum#1181 ATM) to 179 (in |
4f5a8ed
to
49934bf
Compare
49934bf
to
04ac2a5
Compare
2 COMMITS SQUASHED: fixtures: include missing Constantinople in helpers. Debugging CREATE2 using the full "Blockchain" tests in `fixtures` (they are disabled in `py-evm`, because they're slow and a duplication of "State" tests). A few definitions are missing - so add them. tests/conftest: fix VM-tracing log-to-file helper. Also move `import`s out of the helper, so they're not encountered by the interpreter on every invocation of the helper.
00377ed
to
d30382f
Compare
Rebased on underlying PR - which has itself been rebased on upstream |
One is existing `RevertInCreateInInit`, but now for Constantinople, not just Byzantium. The other is `RevertInCreateInInitCreate2`, which contains the same "synthhetic" state which can't be arrived at by regular means in the EVM. It's likely a copy-paste atavism.
d30382f
to
d2d119d
Compare
What was wrong?
CREATE2
was not actingCREATE
-like.Also, some
Constantinople
tests are still disabled.How was it fixed?
See explanation in ethereum#1181 (comment) or commit 00377ed.Essential commit also submitted toethereum/py-evm
separately, as ethereum#1560.Also, the other commits relevant to ethereum#1181 are included.
Cute Animal Picture
Source: Noah's Ark companion animal hospital in Franklin, NC