DevComp/zip-rs-wasm
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

ci(fuzz): Weaken a failing assertion

Browse source
This commit is contained in:
Chris Hennick 2024-06-18 20:37:32 -07:00
parent a895aa57b1
commit 4c2e9f61ab
No known key found for this signature in database
GPG key ID: DA47AABA4961C509
1 changed files with 18 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
fuzz/fuzz_targets/fuzz_write.rs
View file

@ -235,17 +235,26 @@ where
writer.abort_file()?;
*files_added -= 1;
}
let old_comment = writer.get_raw_comment().to_owned();
// If a comment is set, we finish the archive, reopen it for append and then set a shorter
// comment, then there will be junk after the new comment that we can't get rid of. Thus, we
// can only check that the expected is a prefix of the actual
match operation.reopen {
ReopenOption::DoNotReopen => {},
ReopenOption::ViaFinish => replace_with_or_abort(writer, |old_writer: zip::ZipWriter<T>| {
zip::ZipWriter::new_append(old_writer.finish().unwrap()).unwrap()
}),
ReopenOption::ViaFinishIntoReadable => replace_with_or_abort(writer, |old_writer: zip::ZipWriter<T>| {
zip::ZipWriter::new_append(old_writer.finish_into_readable().unwrap().into_inner()).unwrap()
}),
ReopenOption::DoNotReopen => return Ok(()),
ReopenOption::ViaFinish => {
let old_comment = writer.get_raw_comment().to_owned();
replace_with_or_abort(writer, |old_writer: zip::ZipWriter<T>| {
zip::ZipWriter::new_append(old_writer.finish().unwrap()).unwrap()
});
assert!(writer.get_raw_comment().starts_with(&old_comment));
},
ReopenOption::ViaFinishIntoReadable => {
let old_comment = writer.get_raw_comment().to_owned();
replace_with_or_abort(writer, |old_writer: zip::ZipWriter<T>| {
zip::ZipWriter::new_append(old_writer.finish_into_readable().unwrap().into_inner()).unwrap()
});
assert!(writer.get_raw_comment().starts_with(&old_comment));
},
}
assert_eq!(&old_comment, writer.get_raw_comment());
Ok(())
}