tests/builders/publish: Use append_data() to further simplify files() fn

Turbo87 · Turbo87 · commit e1dfcd156106 · 2023-07-27T16:07:20.000+02:00
diff --git a/src/tests/builders/publish.rs b/src/tests/builders/publish.rs
@@ -57,10 +57,8 @@ impl PublishBuilder {
             let mut ar = tar::Builder::new(GzEncoder::new(&mut tarball, Compression::default()));
             for (name, data) in files {
                 let mut header = tar::Header::new_gnu();
-                assert_ok!(header.set_path(name));
                 header.set_size(data.len() as u64);
-                header.set_cksum();
-                assert_ok!(ar.append(&header, *data));
+                assert_ok!(ar.append_data(&mut header, name, *data));
             }
             assert_ok!(ar.finish());
         }

Original file line number	Diff line number	Diff line change
`@@ -57,10 +57,8 @@ impl PublishBuilder {`
`57`	`57`	`let mut ar = tar::Builder::new(GzEncoder::new(&mut tarball, Compression::default()));`
`58`	`58`	`for (name, data) in files {`
`59`	`59`	`let mut header = tar::Header::new_gnu();`
`60`		`- assert_ok!(header.set_path(name));`
`61`	`60`	`header.set_size(data.len() as u64);`
`62`		`- header.set_cksum();`
`63`		`- assert_ok!(ar.append(&header, *data));`
	`61`	`+ assert_ok!(ar.append_data(&mut header, name, *data));`
`64`	`62`	`}`
`65`	`63`	`assert_ok!(ar.finish());`
`66`	`64`	`}`