search

bosnet / sebak

SEBAK, the next BOScoin Blockchain
https://devteam.blockchainos.org
GNU General Public License v3.0
44 stars 15 forks source link

Various code cleanup around test code #926

Closed Geod24 closed 5 years ago

Geod24 commented 5 years ago

The title says it all. Review commit-by-commit to better see what's happening there.

codecov-io commented 5 years ago

Codecov Report

Merging #926 into master will decrease coverage by 0.02%. The diff coverage is 75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #926      +/-   ##
==========================================
- Coverage   60.73%   60.71%   -0.03%     
==========================================
  Files         160      160              
  Lines       11267    11266       -1     
==========================================
- Hits         6843     6840       -3     
- Misses       3617     3619       +2     
  Partials      807      807
Flag Coverage Δ
#integration_tests_long_term 44.03% <25%> (-0.03%) :arrow_down:
#integration_tests_node 40.5% <25%> (-0.03%) :arrow_down:
#integration_tests_retry 37.78% <25%> (+0.04%) :arrow_up:
#unittests 47.34% <62.5%> (-0.03%) :arrow_down:
Impacted Files Coverage Δ
lib/sync/config.go 76.19% <ø> (-0.56%) :arrow_down:
lib/node/test.go 42.85% <0%> (-17.15%) :arrow_down:
cmd/sebak/cmd/run.go 56.92% <100%> (ø) :arrow_up:
lib/network/test.go 100% <100%> (ø) :arrow_up:
lib/node/runner/test.go 89.06% <100%> (ø) :arrow_up:
lib/sync/validator.go 57.06% <100%> (-0.25%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update c0f3b99...b975782. Read the comment docs.