Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Enable state tests for Constantinople #1181

Closed
wants to merge 6 commits into from
Closed

Enable state tests for Constantinople #1181

wants to merge 6 commits into from

Commits on Dec 10, 2018

  1. Enable Constantinople State tests

    @cburgdorf @pipermerriam
    cburgdorf authored and pipermerriam committed Dec 10, 2018
    Configuration menu
    Copy the full SHA
    39e52e9 View commit details
    Browse the repository at this point in the history

  2. Upgrade tests to v6.0.0-beta.2

    @cburgdorf @pipermerriam
    cburgdorf authored and pipermerriam committed Dec 10, 2018
    Configuration menu
    Copy the full SHA
    04ac2a5 View commit details
    Browse the repository at this point in the history

  3. Fix some failing tests

    @pipermerriam
    pipermerriam committed Dec 10, 2018
    3 Configuration menu
    Copy the full SHA
    010b840 View commit details
    Browse the repository at this point in the history

Commits on Dec 11, 2018

  1. misc: fix up test harness. 

    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.
    @veox
    veox committed Dec 11, 2018
    Configuration menu
    Copy the full SHA
    0117604 View commit details
    Browse the repository at this point in the history

Commits on Dec 12, 2018

  1. tests: add more tests to INCORRECT_UPSTREAM_TESTS. 

    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.
    @veox
    veox committed Dec 12, 2018
    Configuration menu
    Copy the full SHA
    d2d119d View commit details
    Browse the repository at this point in the history

  2. Merge pull request #2 from veox/investigate-create2-failures 

    Enable tests that are _still_ disabled
    @cburgdorf
    cburgdorf authored Dec 12, 2018
    Configuration menu
    Copy the full SHA
    84b2ca7 View commit details
    Browse the repository at this point in the history