Merge bitcoin#18518: fuzz: Extend descriptor fuzz test

MarcoFalke · MarcoFalke · commit e16da90d957a · 2020-04-05T04:49:55.000+08:00
fa01899 fuzz: Extend descriptor fuzz test (MarcoFalke) Pull request description: ACKs for top commit: practicalswift: ACK fa01899 Tree-SHA512: 6d6a6417f06d90732bbf055ff54102530d6956f3082f1ff65598f790d588170768aee98e4835996876d28bca2a9c62f22fe122c3fc7eafd4b7660696f72f9835
diff --git a/src/test/fuzz/descriptor_parse.cpp b/src/test/fuzz/descriptor_parse.cpp
@@ -20,6 +20,11 @@ void test_one_input(const std::vector<uint8_t>& buffer)
     FlatSigningProvider signing_provider;
     std::string error;
     for (const bool require_checksum : {true, false}) {
-        Parse(descriptor, signing_provider, error, require_checksum);
+        const auto desc = Parse(descriptor, signing_provider, error, require_checksum);
+        if (desc) {
+            (void)desc->ToString();
+            (void)desc->IsRange();
+            (void)desc->IsSolvable();
+        }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,11 @@ void test_one_input(const std::vector<uint8_t>& buffer)`
`20`	`20`	`FlatSigningProvider signing_provider;`
`21`	`21`	`std::string error;`
`22`	`22`	`for (const bool require_checksum : {true, false}) {`
`23`		`- Parse(descriptor, signing_provider, error, require_checksum);`
	`23`	`+ const auto desc = Parse(descriptor, signing_provider, error, require_checksum);`
	`24`	`+ if (desc) {`
	`25`	`+ (void)desc->ToString();`
	`26`	`+ (void)desc->IsRange();`
	`27`	`+ (void)desc->IsSolvable();`
	`28`	`+ }`
`24`	`29`	`}`
`25`	`30`	`}`