RGBCube/uutils-coreutils
1
Fork 0
mirror of https://github.com/RGBCube/uutils-coreutils synced 2025-07-28 11:37:44 +00:00
Code Issues Activity

ls: handle invalid block size as GNU does

Browse source
This commit is contained in:
Daniel Hofstetter 2023-12-08 15:10:48 +01:00 • committed by Sylvestre Ledru
parent 355b4872b1
commit e9dc2930ee
2 changed files with 14 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

9
src/uu/ls/src/ls.rs
View file

@ -182,7 +182,7 @@ impl UError for LsError {
Self::IOError(_) => 1,
Self::IOErrorContext(_, _, false) => 1,
Self::IOErrorContext(_, _, true) => 2,
Self::BlockSizeParseError(_) => 1,
Self::BlockSizeParseError(_) => 2,
Self::ConflictingArgumentDired => 1,
Self::DiredAndZeroAreIncompatible => 2,
Self::AlreadyListedError(_) => 2,
@ -806,10 +806,9 @@ impl Config {
match parse_size_u64(&raw_block_size.to_string_lossy()) {
Ok(size) => Some(size),
Err(_) => {
show!(LsError::BlockSizeParseError(
opt_block_size.unwrap().clone()
));
None
return Err(Box::new(LsError::BlockSizeParseError(
opt_block_size.unwrap().clone(),
)));
}
}
} else if env_var_posixly_correct.is_some() {

10
tests/by-util/test_ls.rs
View file

@ -3856,6 +3856,16 @@ fn test_posixly_correct() {
.stdout_contains_line("total 8");
}
#[test]
fn test_ls_invalid_block_size() {
new_ucmd!()
.arg("--block-size=invalid")
.fails()
.code_is(2)
.no_stdout()
.stderr_is("ls: invalid --block-size argument 'invalid'\n");
}
#[test]
fn test_ls_hyperlink() {
let scene = TestScenario::new(util_name!());