Add some verbose logging to bitcoin-util-test.py #8881

pull jnewbery wants to merge 1 commits into bitcoin:master from jnewbery:verbose-bitcoin-util-output changing 3 files +65 −30
  1. jnewbery commented at 6:24 PM on October 4, 2016: member

    Adds a new command line parameter to output verbose logging when running bitcoin-util-test.py manually.

  2. jnewbery commented at 6:28 PM on October 4, 2016: member

    Sample output:

    ~/bitcoin/src$ test/bitcoin-util-test.py --srcdir='.' --verbose
    Testing: Creates a blank transaction
    PASS
    Testing: Creates a blank transaction (output in json)
    PASS
    Testing: Creates a blank transaction when nothing is piped into bitcoin-tx
    PASS
    Testing: Creates a blank transaction when nothing is piped into bitcoin-tx (output in json)
    PASS
    Testing: Deletes a single input from a transaction
    PASS
    Testing: Deletes a single input from a transaction (output in json)
    PASS
    Testing: Attempts to delete an input with a bad index from a transaction. Expected to fail.
    PASS
    Testing: Deletes a single output from a transaction
    PASS
    Testing: Deletes a single output from a transaction (output in json)
    PASS
    Testing: Attempts to delete an output with a bad index from a transaction. Expected to fail.
    PASS
    Testing: Adds an nlocktime to a transaction
    PASS
    Testing: Adds an nlocktime to a transaction (output in json)
    PASS
    Testing: Creates a new transaction with three inputs and two outputs
    PASS
    Testing: Creates a new transaction with three inputs and two outputs (output in json)
    PASS
    Testing: Creates a new transaction with a single empty output script
    PASS
    Testing: Creates a new transaction with a single empty output script (output in json)
    PASS
    Testing: Creates a new transaction with a single input and a single output, and then signs the transaction
    PASS
    Testing: Creates a new transaction with a single input and a single output, and then signs the transaction (output in json)
    PASS
    Testing: Attempts to create a new transaction with one input and an output with malformed hex data. Expected to fail
    PASS
    Testing: Attempts to create a new transaction with one input and an output with no value and malformed hex data. Expected to fail
    PASS
    Testing: Creates a new transaction with one input, one address output and one data output
    PASS
    Testing: Creates a new transaction with one input, one address output and one data output. Output in json
    PASS
    Testing: Creates a new transaction with one input, one address output and one data (zero value) output
    PASS
    Testing: Creates a new transaction with one input, one address output and one data (zero value) output. Output in json
    PASS
    Testing: Creates a new transaction with one input with sequence number and one address output
    PASS
    Testing: Creates a new transaction with one input with sequence number and one address output. Output in json
    PASS
    Testing: Adds a new input with sequence number to a transaction
    PASS
    Testing: Adds a new input with sequence number to a transaction. Output in json
    PASS
    
  3. fanquake added the label Tests on Oct 5, 2016
  4. in src/test/data/bitcoin-util-test.json:None in 81a40d6df2 outdated
     156 | @@ -136,15 +157,17 @@
     157 |       "in=5897de6bd6027a475eadd57019d4e6872c396d0716c4875a5f1a6fcfdf385c1f:0",
     158 |       "outaddr=0.18:13tuJJDR2RgArmgfv6JScSdreahzgc4T6o",
     159 |       "outdata=4:54686973204f505f52455455524e207472616e73616374696f6e206f7574707574207761732063726561746564206279206d6f646966696564206372656174657261777472616e73616374696f6e2e"],
     160 | -    "output_cmp": "txcreatedata1.json"
     161 | +    "output_cmp": "txcreatedata1.json",
     162 | +    "description": "Creates a new transaction with one input, one address output and one data output. Output in json"
    


    fanquake commented at 7:04 AM on October 5, 2016:

    nit: lines from here down can be changed to use "(output in json)" to match the above.

  5. fanquake commented at 7:06 AM on October 5, 2016: member

    Getting the same output. etc

    bash-3.2$ test/bitcoin-util-test.py --srcdir='.' --verbose
    Testing: Creates a blank transaction
    PASS
    Testing: Creates a blank transaction (output in json)
    PASS
    .......
    Testing: Adds a new input with sequence number to a transaction
    PASS
    Testing: Adds a new input with sequence number to a transaction. Output in json
    PASS
    
  6. add verbose mode to bitcoin-util-test.py 621441a7a7
  7. jnewbery force-pushed on Oct 5, 2016
  8. jnewbery commented at 1:16 PM on October 5, 2016: member

    Thanks @fanquake . Descriptions fixed and commits squashed.

  9. MarcoFalke commented at 1:18 PM on October 5, 2016: member

    utACK 621441a7a72ec0b17594a24a2c5fedcac0c605f6

  10. jonasschnelli commented at 4:38 PM on October 8, 2016: contributor

    utACK 621441a7a72ec0b17594a24a2c5fedcac0c605f6

  11. MarcoFalke merged this on Oct 8, 2016
  12. MarcoFalke closed this on Oct 8, 2016

  13. MarcoFalke referenced this in commit e66cc1d58e on Oct 8, 2016
  14. jnewbery deleted the branch on Oct 11, 2016
  15. codablock referenced this in commit 86e8b3f608 on Sep 19, 2017
  16. codablock referenced this in commit 76641e68a2 on Jan 12, 2018
  17. andvgal referenced this in commit 2e2123ccdd on Jan 6, 2019
  18. zkbot referenced this in commit 44f5ef7c9d on Nov 9, 2020
  19. zkbot referenced this in commit bd5ff7c889 on Nov 9, 2020
  20. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-30 12:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me