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.

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

Ensure that same_contract is called for state:modified #7283

Merged
merged 1 commit into from
Apr 6, 2023

Conversation

gshank
Copy link
Contributor

@gshank gshank commented Apr 5, 2023

resolves #7282

Description

Breaking changes in a contract throws an error for state:modified. Since the comparison methods did not throw errors before, the node "same_contents" method did an "and" of the comparison methods, which will shortcircuit calling any other methods after a "False" is encountered. Explicitly call the "same_contract" method to ensure it's always called.

Checklist

@gshank gshank requested review from a team as code owners April 5, 2023 22:35
@cla-bot cla-bot bot added the cla:yes label Apr 5, 2023
@gshank gshank requested review from peterallenwebb and QMalcolm and removed request for ChenyuLInx April 5, 2023 22:35
@gshank gshank merged commit 893daed into main Apr 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CT-2378] [Bug] state:modified not catching breaking changes if model body also changes
3 participants