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
Coverage for: networkGroup, nodeGroup, proofGroup, transactionGroup, validatorGroup, vmValuesGroup #5184
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## rc/v1.6.0 #5184 +/- ##
=============================================
+ Coverage 78.75% 79.01% +0.26%
=============================================
Files 680 680
Lines 88133 88018 -115
=============================================
+ Hits 69406 69550 +144
+ Misses 13502 13259 -243
+ Partials 5225 5209 -16
☔ View full report in Codecov by Sentry. |
The base branch was changed.
return expectedMap, nil | ||
}, | ||
} | ||
networkGroup, err := groups.NewNetworkGroup(&facade) |
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.
L970-L981 function extraction?
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.
System test passed.
@@ Log scanner @@
api_tests_p3
================================================================================
- Known Warnings 9
- New Warnings 2
- Known Errors 0
- New Errors 1
- Panics 0
================================================================================ - block hash does not match 12612
- wrong nonce in block 4068
- miniblocks does not match 0
- num miniblocks does not match 0
- miniblock hash does not match 0
- block bodies does not match 0
- receipts hash missmatch 0
================================================================================ - No jailed nodes on the testnet
================================================================================
…into api_tests_p3 # Conflicts: # api/mock/facadeStub.go
5ba962a
Reasoning behind the pull request
Proposed changes
Testing procedure
Pre-requisites
Based on the Contributing Guidelines the PR author and the reviewers must check the following requirements are met:
feat
branch created?feat
branch merging, do all satellite projects have a proper tag insidego.mod
?