fuzz: Hide script_assets_test_minimizer #20663

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:2012-fuzzNoReg changing 3 files +25 −20
  1. MarcoFalke commented at 6:20 pm on December 15, 2020: member
    This is not an actual fuzz target. It is a hack to exploit the built-in capability of fuzz engines to measure coverage.
  2. MarcoFalke commented at 6:20 pm on December 15, 2020: member
  3. MarcoFalke commented at 6:22 pm on December 15, 2020: member
    Can be tested by running PRINT_ALL_FUZZ_TARGETS_AND_ABORT=1 ./src/test/fuzz/fuzz | grep min | wc and observing zeros
  4. sipa commented at 6:25 pm on December 15, 2020: member
    Wouldn’t it be better to leave it as-is, and just exclude it from being listed, or exclude it test-runner.py? That way you don’t need to modify the source code in order to use it.
  5. DrahtBot added the label Tests on Dec 15, 2020
  6. MarcoFalke renamed this:
    fuzz: Leave script_assets_test_minimizer unregistered
    fuzz: Hide script_assets_test_minimizer
    on Dec 16, 2020
  7. MarcoFalke force-pushed on Dec 16, 2020
  8. MarcoFalke force-pushed on Dec 16, 2020
  9. practicalswift commented at 1:19 pm on December 16, 2020: contributor
    Concept ACK: thanks for addressing this!
  10. DrahtBot commented at 7:48 am on December 21, 2020: member

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

    Conflicts

    No conflicts as of last run.

  11. MarcoFalke force-pushed on Dec 21, 2020
  12. DrahtBot added the label Needs rebase on Feb 5, 2021
  13. MarcoFalke force-pushed on Feb 8, 2021
  14. fuzz: Hide script_assets_test_minimizer
    Can be reviewed with --ignore-all-space
    fafca47adc
  15. doc: Fixup docs in fuzz/script_assets_test_minimizer.cpp fac726b1b8
  16. MarcoFalke force-pushed on Feb 8, 2021
  17. DrahtBot removed the label Needs rebase on Feb 8, 2021
  18. practicalswift commented at 9:08 pm on February 10, 2021: contributor
    cr ACK fac726b1b8331b267973138bbd2bff5304774315: patch looks correct and touches only src/test/fuzz/
  19. MarcoFalke merged this on Feb 11, 2021
  20. MarcoFalke closed this on Feb 11, 2021

  21. MarcoFalke deleted the branch on Feb 11, 2021
  22. sidhujag referenced this in commit 74d4cc59d4 on Feb 11, 2021
  23. DrahtBot locked this on Aug 16, 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: 2024-11-17 09:12 UTC

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