Skip to content
This repository has been archived by the owner on Oct 28, 2021. It is now read-only.

Tests for BlockChainSync #5097

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Tests for BlockChainSync #5097

wants to merge 2 commits into from

Conversation

gumb0
Copy link
Member

@gumb0 gumb0 commented Jun 27, 2018

No description provided.

@codecov-io
Copy link

codecov-io commented Jul 10, 2018

Codecov Report

Merging #5097 into master will increase coverage by 3%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #5097     +/-   ##
=========================================
+ Coverage   59.94%   62.95%     +3%     
=========================================
  Files         337      338      +1     
  Lines       27306    26967    -339     
  Branches     3173     3162     -11     
=========================================
+ Hits        16369    16977    +608     
+ Misses       9857     8838   -1019     
- Partials     1080     1152     +72

@gumb0
Copy link
Member Author

gumb0 commented Jul 10, 2018

The first test improved coverage of BlockChainSync.cpp by 30%, if I interpret codecov correctly

@chfast chfast changed the base branch from develop to master August 2, 2018 08:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants