Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

state: test when contract creation fails, old storage values should re-appear #9532

Merged
merged 1 commit into from
Sep 12, 2018

Conversation

sorpaas
Copy link
Collaborator

@sorpaas sorpaas commented Sep 11, 2018

Because more tests won't hurt. :)

Add a test case for ethereum/py-evm#1224 (comment) where if contract creation fails, old storage values (if ever existed) should re-appear.

@sorpaas sorpaas added A0-pleasereview 🤓 Pull request needs code review. A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). M4-core ⛓ Core client code / Rust. labels Sep 11, 2018
@sorpaas sorpaas added this to the 2.2 milestone Sep 11, 2018
@ordian ordian added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Sep 11, 2018
@andresilva andresilva merged commit 018e240 into master Sep 12, 2018
@ordian ordian deleted the sp-test-storage-reappear branch September 12, 2018 10:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants