-
Notifications
You must be signed in to change notification settings - Fork 205
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 underpriced check for multi currency #2230
Conversation
Coverage from tests in coverage: 49.7% of statements across all listed packagescoverage: 63.2% of statements in consensus/istanbul coverage: 42.7% of statements in consensus/istanbul/announce coverage: 55.7% of statements in consensus/istanbul/backend coverage: 0.0% of statements in consensus/istanbul/backend/backendtest coverage: 24.3% of statements in consensus/istanbul/backend/internal/replica coverage: 61.2% of statements in consensus/istanbul/core coverage: 50.0% of statements in consensus/istanbul/db coverage: 0.0% of statements in consensus/istanbul/proxy coverage: 64.2% of statements in consensus/istanbul/uptime coverage: 51.8% of statements in consensus/istanbul/validator coverage: 79.2% of statements in consensus/istanbul/validator/random |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #2230 +/- ##
==========================================
- Coverage 55.10% 55.08% -0.03%
==========================================
Files 683 680 -3
Lines 114538 113847 -691
==========================================
- Hits 63121 62707 -414
+ Misses 47528 47293 -235
+ Partials 3889 3847 -42 ☔ View full report in Codecov by Sentry. |
8dfb01e
to
7a8a3c4
Compare
7a8a3c4
to
c50de69
Compare
Hey @hbandura I started looking into those other failing tests, because they were failing for me and it seems that they are also failing for similar reasons to |
c50de69
to
9bfde3e
Compare
Fix the underpriced check by checking against all currency heaps