[test] Add option to run bitcoin-util-test.py manually #8830

pull jnewbery wants to merge 1 commits into bitcoin:master from jnewbery:test-bitcoin-util-manually changing 2 files +27 −3

jnewbery commented at 5:52 PM on September 28, 2016: member

I've recently added new test cases that test bitoin-tx JSON output (https://github.com/bitcoin/bitcoin/pull/8829).

Those tests are run automatically by make check, but it was previously quite difficult to run those tests manually, since the script assumed that OS environment variables had been set. This PR allows the tests to be run manually, and adds documentation explaining how to do that.
MarcoFalke added the label Tests on Sep 28, 2016
in src/test/bitcoin-util-test.py:None in 2e992b67ae outdated
```
   5 | @@ -6,8 +6,25 @@
   6 |  import os
   7 |  import bctest
   8 |  import buildenv
   9 | +import sys
```
MarcoFalke commented at 7:17 PM on September 28, 2016:

What is this used for?

jnewbery commented at 7:44 PM on September 28, 2016:

Ooops. Nothing now. I was originally using sys.argv, but then changed to use argparse. I'll remove it and squash the commits.
MarcoFalke approved
MarcoFalke commented at 7:19 PM on September 28, 2016: member

ACK 2e992b6
jnewbery force-pushed on Sep 28, 2016
btcdrak commented at 11:04 AM on September 29, 2016: contributor

ACK 7e88077
Add option to run bitcoin-util-test.py manually b82f493938
jnewbery force-pushed on Sep 29, 2016
jnewbery commented at 12:56 PM on September 29, 2016: member

rebased
MarcoFalke commented at 10:57 AM on September 30, 2016: member

utACK b82f493
laanwj merged this on Sep 30, 2016
laanwj closed this on Sep 30, 2016
laanwj referenced this in commit ef0801bd13 on Sep 30, 2016
jnewbery deleted the branch on Sep 30, 2016
codablock referenced this in commit 7d6369e8bb on Sep 19, 2017
codablock referenced this in commit 2f3abc62ee on Jan 12, 2018
andvgal referenced this in commit f9112bca7f on Jan 6, 2019
zkbot referenced this in commit 44f5ef7c9d on Nov 9, 2020
zkbot referenced this in commit bd5ff7c889 on Nov 9, 2020
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels

Linked (view graph)

#8829 Add bitcoin-tx JSON tests