github/ouch
1
1
Fork 0
mirror of https://github.com/ouch-org/ouch.git synced 2025-06-06 11:35:45 +00:00
Code Issues 15 Packages Projects Releases Wiki Activity

Fix confusion between empty/nonempty test.

Browse Source 
The single_empty_file test was writing random content to the file,
whereas the single_file test was writing an empty file.
Only the latter tested different levels, so I figured that test
was the one that should be using actual file content.
This commit is contained in:
Ryan Roden-Corrent 2024-07-14 16:12:41 -04:00 committed by João Marcos
parent b7ebcd3703
commit 4a323aeba8
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
tests/integration.rs
View File

@ -91,10 +91,7 @@ fn single_empty_file(ext: Extension, #[any(size_range(0..8).lift())] exts: Vec<F
let before_file = &before.join("file"); let before_file = &before.join("file");
let archive = &dir.join(format!("file.{}", merge_extensions(ext, exts))); let archive = &dir.join(format!("file.{}", merge_extensions(ext, exts)));
let after = &dir.join("after"); let after = &dir.join("after");
write_random_content( fs::write(before_file, []).unwrap();
&mut fs::File::create(before_file).unwrap(),
&mut SmallRng::from_entropy(),
);
ouch!("-A", "c", before_file, archive); ouch!("-A", "c", before_file, archive);
ouch!("-A", "d", archive, "-d", after); ouch!("-A", "d", archive, "-d", after);
assert_same_directory(before, after, false); assert_same_directory(before, after, false);
@ -118,7 +115,10 @@ fn single_file(
let before_file = &before.join("file"); let before_file = &before.join("file");
let archive = &dir.join(format!("file.{}", merge_extensions(ext, exts))); let archive = &dir.join(format!("file.{}", merge_extensions(ext, exts)));
let after = &dir.join("after"); let after = &dir.join("after");
fs::write(before_file, []).unwrap(); write_random_content(
&mut fs::File::create(before_file).unwrap(),
&mut SmallRng::from_entropy(),
);
if let Some(level) = level { if let Some(level) = level {
ouch!("-A", "c", "-l", level.to_string(), before_file, archive); ouch!("-A", "c", "-l", level.to_string(), before_file, archive);
} else { } else {