Add some script tests #12425

pull richardkiss wants to merge 1 commits into bitcoin:master from richardkiss:feature/bool_tests changing 1 files +12 −0
  1. richardkiss commented at 6:45 PM on February 13, 2018: contributor

    Add some script tests related to BOOL ops and odd values like negative 0.

  2. Add some script tests related to BOOL ops and odd values like negative 0. be45a67895
  3. fanquake added the label Tests on Feb 13, 2018
  4. randolf commented at 3:42 AM on February 14, 2018: contributor

    Why is it necessary to check for negative 0?

  5. laanwj commented at 3:20 PM on February 14, 2018: member

    utACK, thanks for adding tests!

    Why is it necessary to check for negative 0?

    Because the point of these tests is to cover as many as possible inputs, including weird/unexpected encodings of values.

  6. richardkiss commented at 7:34 PM on February 14, 2018: contributor

    I thought I'd found some mismatches between bitcoind's consensus mechanims and pycoin's, so I wrote these tests to check. Turns out pycoin was working properly, but I thought I'd submit the tests anyway.

  7. randolf commented at 1:15 AM on February 15, 2018: contributor

    @laanwj Okay, that answers my question and it makes sense. Thank you.

  8. randolf approved
  9. laanwj merged this on Feb 16, 2018
  10. laanwj closed this on Feb 16, 2018

  11. laanwj referenced this in commit 26dc2daf82 on Feb 16, 2018
  12. azuchi referenced this in commit 1f83e8bc26 on Feb 26, 2018
  13. braydonf referenced this in commit a111d0663c on Feb 2, 2019
  14. braydonf referenced this in commit 37fc9a78e4 on Feb 2, 2019
  15. tuxcanfly referenced this in commit 1985175c2d on Apr 19, 2019
  16. dagurval referenced this in commit abdeb3b6d4 on Aug 28, 2019
  17. gandrewstone referenced this in commit 0504448dbd on Aug 30, 2019
  18. PastaPastaPasta referenced this in commit 61f04acedc on Jun 13, 2020
  19. PastaPastaPasta referenced this in commit 011a64ac15 on Jun 13, 2020
  20. PastaPastaPasta referenced this in commit 5bdcdff600 on Jun 14, 2020
  21. PastaPastaPasta referenced this in commit 9446630591 on Jun 17, 2020
  22. PastaPastaPasta referenced this in commit 2d8d1b2d1a on Jun 17, 2020
  23. PastaPastaPasta referenced this in commit a6b9b62dde on Jun 17, 2020
  24. PastaPastaPasta referenced this in commit 128e14640e on Jun 17, 2020
  25. MarcoFalke locked this on Sep 8, 2021
Labels

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-29 03:15 UTC

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