This adds coverage to the currently uncovered rpc setnetworkactive.
Steps to verify the new coverage:
./test/functional/test_runner.py --coverage net | grep setnetworkactive && echo Failed! || echo Success!
[qa] Add setnetworkactive smoke test #10077
pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:Mf1703-toggleNet changing 2 files +55 −0-
MarcoFalke commented at 2:35 PM on March 25, 2017: member
-
[qa] Add setnetworkactive smoke test fa697b7192
- MarcoFalke added the label Tests on Mar 25, 2017
-
jnewbery commented at 8:56 PM on March 30, 2017: member
Looks great. Tested ACK https://github.com/bitcoin/bitcoin/pull/10077/commits/fa697b719286c599103d04c7055b7c666e71f09d
Only comment would be that nodehandling.py could be merged into this so we have a single test script for testing the net.cpp RPCs. But also fine to have them as separate tests.
- laanwj merged this on Apr 2, 2017
- laanwj closed this on Apr 2, 2017
- laanwj referenced this in commit ebfd653798 on Apr 2, 2017
- MarcoFalke deleted the branch on Apr 3, 2017
- PastaPastaPasta referenced this in commit 4482ef6533 on May 10, 2019
- PastaPastaPasta referenced this in commit efe4f12b2f on May 15, 2019
- PastaPastaPasta referenced this in commit 22b21f50cc on May 20, 2019
- PastaPastaPasta referenced this in commit 434e03463b on May 21, 2019
- barrystyle referenced this in commit 8af39ff015 on Jan 22, 2020
- DrahtBot locked this on Sep 8, 2021
Contributors
Labels