contrib/linearize scripts need tests #17019

issue laanwj opened this issue on October 2, 2019
  1. laanwj commented at 8:28 AM on October 2, 2019: member

    The scripts under contrib/linearize do not have any tests.

    It would be good to have functional tests for them, to assure that they keep working, and so that changes to them are not necessarily held up on manual testing.

  2. laanwj added the label Tests on Oct 2, 2019
  3. laanwj added the label Scripts and tools on Oct 2, 2019
  4. laanwj added the label good first issue on Oct 2, 2019
  5. fjahr commented at 4:29 PM on October 9, 2019: contributor

    For reference: I am working on a test for the -loadblock option in #17044 and the scripts are implicitly tested in it. @laanwj do you have hints on configurations you think should be tested in particular?

  6. MarcoFalke commented at 4:46 PM on October 9, 2019: member

    It looks like the test can be submitted without the code changes and should still pass? If so, maybe do that as a first, separate, pull request?

  7. MarcoFalke commented at 5:20 PM on October 9, 2019: member

    I ran git checkout 6a768e516e3bafe31bd8c0792cdc93986cabcc56 && ./test/functional/feature_loadblock.py and it passed, so seems fine to do

  8. fjahr cross-referenced this on Oct 9, 2019 from issue tests: Add test for loadblock option and linearize scripts by fjahr
  9. fjahr commented at 10:58 PM on October 9, 2019: contributor

    It looks like the test can be submitted without the code changes and should still pass? If so, maybe do that as a first, separate, pull request?

    Yeah, the test got much larger than the inital improvement of #15954 so makes sense: #17091

  10. fjahr cross-referenced this on Oct 9, 2019 from issue init: Remove auto-import of `bootstrap.dat` and associated code by fjahr
  11. laanwj closed this on Oct 23, 2019

  12. sidhujag referenced this in commit 3e3f53b757 on Oct 26, 2019
  13. bitcoin locked this on Dec 16, 2021
  14. Munkybooty referenced this in commit 98325fcf26 on Apr 29, 2022
  15. Munkybooty referenced this in commit 7e77f6db47 on May 12, 2022
  16. Munkybooty referenced this in commit 84d7b7cace on May 12, 2022
  17. Munkybooty referenced this in commit bbf79db207 on May 17, 2022

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-05-20 06:54 UTC