[QA] fix zapwallettxes test #5612

pull jonasschnelli wants to merge 1 commits into bitcoin:master from jonasschnelli:2015_01_zapwallettxes_test_fix changing 3 files +84 −166

jonasschnelli commented at 10:23 AM on January 7, 2015: contributor
- remove shell test script and add a python equivalent
Wallet tests (we need more!) must be available in case of a upcoming wallet file format change.

fix for #5609
jonasschnelli force-pushed on Jan 7, 2015

[QA] fix zapwallettxes test

- remove shell test script and add a python equivalent

84d919974b

in qa/pull-tester/rpc-tests.sh:None in e3a59c5fa1 outdated
```
  41 |              ${BUILDDIR}/qa/rpc-tests/${testScripts[$i]} --srcdir "${BUILDDIR}/src"
  42 |          fi
  43 |      done
  44 | +    if [ $found == 0 ]
  45 | +    then
  46 | +        echo -e "\033[1mTest not found\033[0m"
```
jonasschnelli commented at 10:24 AM on January 7, 2015:

smuggled in: give simple output when selected test script was not available

jonasschnelli commented at 4:26 PM on January 7, 2015:

Had to remove this because it had break travis.
jonasschnelli force-pushed on Jan 7, 2015
fanquake commented at 10:25 AM on January 8, 2015: member

Related #4071. Almost every test is python now, bar conflictedbalance.sh
laanwj added the label Tests on Jan 8, 2015
laanwj commented at 10:51 AM on January 8, 2015: member

Tested ACK
laanwj merged this on Jan 8, 2015
laanwj closed this on Jan 8, 2015
laanwj referenced this in commit e3f5727590 on Jan 8, 2015
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels