bosnet / sebak

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

Downgrade Error log to Debug #901

Closed Geod24 closed 5 years ago

Geod24 commented 5 years ago
A failure to broadcast can be completely benign, e.g. failure to connect.
It currently pollutes the logs of the integration tests for no good reason.
codecov-io commented 5 years ago

Codecov Report

Merging #901 into master will increase coverage by 0.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #901      +/-   ##
=========================================
+ Coverage   60.38%   60.4%   +0.01%     
=========================================
  Files         153     153              
  Lines       10768   10768              
=========================================
+ Hits         6502    6504       +2     
+ Misses       3520    3519       -1     
+ Partials      746     745       -1
Flag Coverage Δ
#integration_tests_long_term 44.51% <0%> (ø) :arrow_up:
#integration_tests_node 40.37% <100%> (ø) :arrow_up:
#unittests 48.62% <0%> (+0.01%) :arrow_up:
Impacted Files Coverage Δ
lib/network/validator_connection_manager.go 83.39% <100%> (ø) :arrow_up:
lib/node/runner/node_runner.go 77.85% <0%> (+0.43%) :arrow_up:

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 a48e152...0569edc. Read the comment docs.

codecov-io commented 5 years ago

Codecov Report

Merging #901 into master will increase coverage by 0.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #901      +/-   ##
=========================================
+ Coverage   60.38%   60.4%   +0.01%     
=========================================
  Files         153     153              
  Lines       10768   10768              
=========================================
+ Hits         6502    6504       +2     
+ Misses       3520    3519       -1     
+ Partials      746     745       -1
Flag Coverage Δ
#integration_tests_long_term 44.51% <0%> (ø) :arrow_up:
#integration_tests_node 40.37% <100%> (ø) :arrow_up:
#unittests 48.62% <0%> (+0.01%) :arrow_up:
Impacted Files Coverage Δ
lib/network/validator_connection_manager.go 83.39% <100%> (ø) :arrow_up:
lib/node/runner/node_runner.go 77.85% <0%> (+0.43%) :arrow_up:

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 a48e152...0569edc. Read the comment docs.

codecov-io commented 5 years ago

Codecov Report

Merging #901 into master will increase coverage by 0.1%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #901     +/-   ##
=========================================
+ Coverage   60.28%   60.38%   +0.1%     
=========================================
  Files         153      153             
  Lines       10838    10838             
=========================================
+ Hits         6534     6545     +11     
+ Misses       3542     3532     -10     
+ Partials      762      761      -1
Flag Coverage Δ
#integration_tests_long_term 44.53% <0%> (+0.1%) :arrow_up:
#integration_tests_node 40.34% <100%> (ø) :arrow_up:
#unittests 48.46% <0%> (ø) :arrow_up:
Impacted Files Coverage Δ
lib/network/validator_connection_manager.go 88.53% <100%> (+3.55%) :arrow_up:
lib/common/message.go 42.1% <0%> (+10.52%) :arrow_up:

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 e9f09b1...cea3ac0. Read the comment docs.