Closed envestcc closed 3 months ago
Attention: Patch coverage is 47.94521%
with 38 lines
in your changes missing coverage. Please review.
Project coverage is 76.69%. Comparing base (
cc526cd
) to head (b63aadf
). Report is 31 commits behind head on master.:exclamation: Current head b63aadf differs from pull request most recent head 991bbab
Please upload reports for the commit 991bbab to get more accurate results.
Files | Patch % | Lines |
---|---|---|
action/signedaction.go | 50.00% | 24 Missing and 11 partials :warning: |
action/protocol/staking/protocol.go | 0.00% | 3 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Failed conditions
5.2% Duplication on New Code (required ≤ 3%)
Description
The pr introduce e2etest framework to simplify add test cases, focusing on construct actions and expectations. Futhermore, add testcases via the frameworks for original action handlings after candidate ownership transfer.
based on #4294
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Checklist: