fuzz: Fail if message type is not fuzzed #20915

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:2101-fuzzFailMsgType changing 1 files +53 −24
  1. MarcoFalke commented at 4:40 PM on January 12, 2021: member

    process_message_* is a nice way to quickly fuzz a single message type. However, the offered message types are outdated and all BIPs implemented in the last years are missing.

    Fix that by adding them and failing when the number of message types don't add up.

  2. DrahtBot added the label Tests on Jan 12, 2021
  3. practicalswift commented at 9:04 PM on January 12, 2021: contributor

    Concept ACK: clever completeness checking!

  4. DrahtBot cross-referenced this on Jan 16, 2021 from issue fuzz: Consolidate fuzzing TestingSetup initialization by dongcarl
  5. DrahtBot commented at 3:09 AM on January 16, 2021: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--174a7506f384e20aa4161008e828411d-->

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #21003 (test: Move MakeNoLogFileContext to libtest_util, and use it in bench by MarcoFalke)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  6. DrahtBot added the label Needs rebase on Jan 21, 2021
  7. MarcoFalke force-pushed on Jan 21, 2021
  8. DrahtBot removed the label Needs rebase on Jan 21, 2021
  9. DrahtBot cross-referenced this on Jan 26, 2021 from issue test: Move MakeNoLogFileContext to libtest_util, and use it in bench by MarcoFalke
  10. fuzz: Count message type fuzzers before main() faefed8cd5
  11. fuzz: Fail if message type is not fuzzed fa4bc897fc
  12. MarcoFalke force-pushed on Feb 11, 2021
  13. MarcoFalke commented at 8:43 AM on February 11, 2021: member

    Rebased

  14. practicalswift commented at 8:52 AM on February 11, 2021: contributor

    cr ACK fa4bc897fc9332a5666ca2f3e78492cd67ee6128: patch looks correct and touches only src/test/fuzz/

  15. MarcoFalke merged this on Feb 11, 2021
  16. MarcoFalke closed this on Feb 11, 2021
  17. MarcoFalke deleted the branch on Feb 11, 2021
  18. sidhujag referenced this in commit 3afb3f9144 on Feb 11, 2021
  19. bitcoin locked this on Aug 18, 2022
Contributors
MarcoFalkepracticalswiftDrahtBot
Labels
Tests
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: 2026-05-20 06:54 UTC