fuzz: Fail if message type is not fuzzed #20915

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 commented at 3:09 am on January 16, 2021: member

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

   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.

5. DrahtBot added the label Needs rebase on Jan 21, 2021
6. MarcoFalke force-pushed on Jan 21, 2021
7. DrahtBot removed the label Needs rebase on Jan 21, 2021
8. fuzz: Count message type fuzzers before main() faefed8cd5
9. fuzz: Fail if message type is not fuzzed fa4bc897fc
10. MarcoFalke force-pushed on Feb 11, 2021
11. 
    MarcoFalke commented at 8:43 am on February 11, 2021: member

    Rebased
12. 
    practicalswift commented at 8:52 am on February 11, 2021: contributor

    cr ACK fa4bc897fc9332a5666ca2f3e78492cd67ee6128: patch looks correct and touches only src/test/fuzz/
13. MarcoFalke merged this on Feb 11, 2021
14. MarcoFalke closed this on Feb 11, 2021

    ---

15. MarcoFalke deleted the branch on Feb 11, 2021
16. sidhujag referenced this in commit 3afb3f9144 on Feb 11, 2021
17. DrahtBot locked this on Aug 18, 2022

Contributors
MarcoFalke practicalswift DrahtBot

Labels
Tests