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
Assert EVM is set correctly in contract create #5797
Conversation
Signed-off-by: Vasil Vasilev <vasil.yanev15@gmail.com>
Signed-off-by: Vasil Vasilev <vasil.yanev15@gmail.com>
...clients/src/main/java/com/hedera/services/bdd/spec/queries/contract/HapiGetContractInfo.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good!
You can add @SuppressWarnings("java:S5960")
as an annotation to the EET method to suppress the sonar bug.
Signed-off-by: Vasil Vasilev <vasil.yanev15@gmail.com>
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## develop #5797 +/- ##
==============================================
+ Coverage 0 91.37% +91.37%
- Complexity 0 16963 +16963
==============================================
Files 0 1270 +1270
Lines 0 48486 +48486
Branches 0 4840 +4840
==============================================
+ Hits 0 44304 +44304
- Misses 0 3259 +3259
- Partials 0 923 +923 see 1270 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Description:
Extended one of the tests to assert that EVM is correctly set via Contract Create.
Removed few rows of logic from
HapiGetContractInfo
because it was doing some checks twice.Related issue(s):
Fixes #5708
Notes for reviewer:
Checklist