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 #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
-
61f238e84a
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.
-
DrahtBot commented at 4:40 PM on April 25, 2025: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage & Benchmarks
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32353.
<!--021abf342d371248e50ceaed478a90ca-->
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.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
- DrahtBot added the label Docs on Apr 25, 2025
-
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.pyfrom 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
--helpone 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
-
-
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
- fanquake referenced this in commit c3ac86c59c on May 1, 2025
- fanquake referenced this in commit 2f6c802b54 on May 6, 2025
- fanquake referenced this in commit 589b56192f on May 22, 2025
- stickies-v referenced this in commit 772a33e052 on May 23, 2025
- sedited referenced this in commit cb3cd9b8b9 on May 29, 2025
- yuvicc referenced this in commit 069643f094 on Jul 6, 2025
- bug-castercv502 referenced this in commit 2858098810 on Sep 28, 2025
- morozow referenced this in commit 22a6facbed on May 8, 2026
- morozow referenced this in commit edc6b10728 on May 8, 2026
- Kino1994 referenced this in commit 3f8189df27 on Jun 28, 2026
- BigcoinBGC referenced this in commit ddba9f1eb3 on Jun 30, 2026
- bitcoin locked this on Jul 30, 2026
Labels
Linked (view graph)