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

ci(fuzz): Bug fix: debug output was missing set_flush_on_finish_file

Browse source
This commit is contained in:
Chris Hennick 2024-06-08 14:19:14 -07:00
parent 62b9c53651
commit 5bb31e3558
No known key found for this signature in database
GPG key ID: DA47AABA4961C509
1 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
fuzz/fuzz_targets/fuzz_write.rs
View file

@ -70,11 +70,12 @@ impl <'k> Debug for FileOperation<'k> {
}, },
BasicFileOperation::MergeWithOtherFile {operations} => { BasicFileOperation::MergeWithOtherFile {operations} => {
f.write_str("let sub_writer = {\n\ f.write_str("let sub_writer = {\n\
let mut writer = ZipWriter::new(Cursor::new(Vec::new()));\n")?; let mut writer = ZipWriter::new(Cursor::new(Vec::new()));\n\
writer.set_flush_on_finish_file(false);\n")?;
operations.iter().map(|op| { operations.iter().map(|op| {
f.write_fmt(format_args!("{:?}", op.0))?; f.write_fmt(format_args!("{:?}", op.0))?;
if op.1 { if op.1 {
f.write_str("writer.abort_file()?;") f.write_str("writer.abort_file()?;\n")
} else { } else {
Ok(()) Ok(())
} }
@ -195,10 +196,10 @@ where
match operation.reopen { match operation.reopen {
ReopenOption::DoNotReopen => {}, ReopenOption::DoNotReopen => {},
ReopenOption::ViaFinish => replace_with_or_abort(writer, |old_writer: zip::ZipWriter<T>| { ReopenOption::ViaFinish => replace_with_or_abort(writer, |old_writer: zip::ZipWriter<T>| {
zip::ZipWriter::new_append(old_writer.finish().unwrap()).unwrap() zip::ZipWriter::new_append(old_writer.finish()?)?
}), }),
ReopenOption::ViaFinishIntoReadable => replace_with_or_abort(writer, |old_writer: zip::ZipWriter<T>| { 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() zip::ZipWriter::new_append(old_writer.finish_into_readable()?.into_inner())?
}), }),
} }
assert_eq!(&old_comment, writer.get_raw_comment()); assert_eq!(&old_comment, writer.get_raw_comment());