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
fix: LeakyContractTestsSuite
test with top-level signatures
#10486
Conversation
Signed-off-by: Stanimir Stoyanov <stanimir.stoyanov@limechain.tech>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #10486 +/- ##
==========================================
Coverage 63.01% 63.02%
- Complexity 30804 30809 +5
==========================================
Files 3360 3360
Lines 135346 135346
Branches 14088 14088
==========================================
+ Hits 85289 85301 +12
+ Misses 46695 46689 -6
+ Partials 3362 3356 -6 ☔ 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, we may want to extract these tests into a separate suite? (@netopyr?)
Yes, please move the tests to another class similar to other cases where we had tests depending on the legacy security model. Thanks! |
Description:
As per this comment #10287 (review)
The tests requiring top-level signatures are commented
Related issue(s):
Fixes #9392
Notes for reviewer:
Checklist