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: contributorThis 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.
-
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
-
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 dir0./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 -
maflcko commented at 9:37 am on April 27, 2025: memberlgtm ACK 61f238e84ac6d24d8f420c2eabcbb2980d7fcb1e
-
mabu44 commented at 2:08 pm on April 27, 2025: noneTested ACK 61f238e84ac6d24d8f420c2eabcbb2980d7fcb1e
-
hebasto approved
-
hebasto commented at 3:16 pm on April 27, 2025: memberACK 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: contributorFor 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 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
More mirrored repositories can be found on mirror.b10c.me