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

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.

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

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.
DrahtBot added the label Docs on Apr 25, 2025

Type	Reviewers
ACK	kevkevinpal, maflcko, mabu44, hebasto

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

./test/fuzz/test_runner.py ../qa-assets/fuzz_corpora
Traceback (most recent call last):
  File "/mnt/shared_drive/DEVDIR/bitcoin/./test/fuzz/test_runner.py", line 407, in <module>
    main()
  File "/mnt/shared_drive/DEVDIR/bitcoin/./test/fuzz/test_runner.py", line 102, in main
    config.read_file(open(configfile, encoding="utf8"))
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
FileNotFoundError: [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

maflcko commented at 9:37 AM on April 27, 2025: member

lgtm ACK 61f238e84ac6d24d8f420c2eabcbb2980d7fcb1e
mabu44 commented at 2:08 PM on April 27, 2025: none

Tested ACK 61f238e84ac6d24d8f420c2eabcbb2980d7fcb1e
hebasto approved
hebasto commented at 3:16 PM on April 27, 2025: member

ACK 61f238e84ac6d24d8f420c2eabcbb2980d7fcb1e.
hebasto merged this on Apr 27, 2025
hebasto closed this on Apr 27, 2025
fanquake referenced this in commit 7579b0174b on Apr 28, 2025
fanquake commented at 9:20 AM on April 28, 2025: member

Backported to 29.x in #32292.
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

Contributors

Labels

Linked (view graph)

#32292 [29.x] Backports

doc: Fix fuzz test_runner.py path #32353

Code Coverage & Benchmarks

Reviews