doc: Fix fuzz test_runner.py path #32353

pull monlovesmango wants to merge 1 commits into bitcoin:master from monlovesmango:doc-fix-fuzz-test_runner-path changing 1 files +1 −1
  1. monlovesmango commented at 4:40 pm on April 25, 2025: contributor
    This commit fixes the path listed in the documentation for the fuzz testing test_runner.py. Previously the –help option worked but running fuzz tests from the documented path did not.
  2. doc: Fix fuzz test_runner.py path
    This commit fixes the path listed in the documentation for the fuzz
    testing test_runner.py. Previously the --help option worked but running
    fuzz tests from the documented path did not.
    61f238e84a
  3. DrahtBot commented at 4:40 pm on April 25, 2025: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32353.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK kevkevinpal, maflcko, mabu44, hebasto

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

  4. DrahtBot added the label Docs on Apr 25, 2025
  5. kevkevinpal commented at 5:22 pm on April 25, 2025: contributor

    ACK 61f238e


    This is the error I get if I try running test_runner.py from the non build dir

    0./test/fuzz/test_runner.py ../qa-assets/fuzz_corpora
    1Traceback (most recent call last):
    2  File "/mnt/shared_drive/DEVDIR/bitcoin/./test/fuzz/test_runner.py", line 407, in <module>
    3    main()
    4  File "/mnt/shared_drive/DEVDIR/bitcoin/./test/fuzz/test_runner.py", line 102, in main
    5    config.read_file(open(configfile, encoding="utf8"))
    6                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    7FileNotFoundError: [Errno 2] No such file or directory: '/mnt/shared_drive/DEVDIR/bitcoin/test/fuzz/../config.ini'
    

    While the command in the doc does work for --help one would assume that running the fuzz tests with that script should work too

  6. maflcko commented at 9:37 am on April 27, 2025: member
    lgtm ACK 61f238e84ac6d24d8f420c2eabcbb2980d7fcb1e
  7. mabu44 commented at 2:08 pm on April 27, 2025: none
    Tested ACK 61f238e84ac6d24d8f420c2eabcbb2980d7fcb1e
  8. hebasto approved
  9. hebasto commented at 3:16 pm on April 27, 2025: member
    ACK 61f238e84ac6d24d8f420c2eabcbb2980d7fcb1e.
  10. hebasto merged this on Apr 27, 2025
  11. hebasto closed this on Apr 27, 2025

  12. fanquake referenced this in commit 7579b0174b on Apr 28, 2025
  13. fanquake commented at 9:20 am on April 28, 2025: member
    Backported to 29.x in #32292.
  14. monlovesmango commented at 3:26 am on May 1, 2025: contributor
    For anyone interested, I just opened another PR for this same type of documentation fix in the Unit Test README.md, https://github.com/bitcoin/bitcoin/pull/32389

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: 2025-05-05 15:12 UTC

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