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.

2 participants