Fuzzing: Log the error we get for an input that errored after being r…

…eprinted (#172)
kaleidawave · Jul 10, 2024 · d8d5656 · d8d5656
1 parent 245d530
commit d8d5656
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/parser/fuzz/fuzz_targets/module_roundtrip_naive.rs b/parser/fuzz/fuzz_targets/module_roundtrip_naive.rs
@@ -30,8 +30,11 @@ fn do_fuzz(data: &str) -> Corpus {
 
 	let output1 = module1.to_string(&to_string_options);
 
-	let Ok(module2) = Module::from_string(output1.to_owned(), parse_options) else {
-		panic!("input: `{input}`\noutput1: `{output1}`\n\nThis parse should not error because it was just parsed above");
+	let module2 = match Module::from_string(output1.to_owned(), parse_options) {
+		Ok(module2) => module2,
+		Err(error) => {
+			panic!("input: `{input}`\noutput1: `{output1}`\n\nThis parse should not error because it was just parsed above. \nerror: `{:?}`", error);
+		}
 	};
 
 	let output2 = module2.to_string(&to_string_options);

diff --git a/parser/fuzz/fuzz_targets/module_roundtrip_structured.rs b/parser/fuzz/fuzz_targets/module_roundtrip_structured.rs
@@ -30,8 +30,11 @@ fn do_fuzz(data: common::FuzzSource) -> Corpus {
 
 	let output1 = module1.to_string(&to_string_options);
 
-	let Ok(module2) = Module::from_string(output1.to_owned(), parse_options) else {
-		panic!("input: `{input}`\noutput1: `{output1}`\n\nThis parse should not error because it was just parsed above");
+	let module2 = match Module::from_string(output1.to_owned(), parse_options) {
+		Ok(module2) => module2,
+		Err(error) => {
+			panic!("input: `{input}`\noutput1: `{output1}`\n\nThis parse should not error because it was just parsed above. \nerror: `{:?}`", error);
+		}
 	};
 
 	let output2 = module2.to_string(&to_string_options);