Merge pull request #3578 from jfinkels/mktemp-usage-error-status-1

mktemp: exit with status 1 on usage errors
This commit is contained in:
Terts Diepraam 2022-05-30 18:34:27 +02:00 committed by GitHub
commit a240b1cef3
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 1 deletions

View file

@ -91,7 +91,7 @@ impl Display for MkTempError {
#[uucore::main]
pub fn uumain(args: impl uucore::Args) -> UResult<()> {
let matches = uu_app().get_matches_from(args);
let matches = uu_app().try_get_matches_from(args)?;
let template = matches.value_of(ARG_TEMPLATE).unwrap();
let tmpdir = matches.value_of(OPT_TMPDIR).unwrap_or_default();

View file

@ -558,3 +558,8 @@ fn test_too_few_xs_suffix_directory() {
.fails()
.stderr_only("mktemp: too few X's in template 'aXXX'\n");
}
#[test]
fn test_too_many_arguments() {
new_ucmd!().args(&["-q", "a", "b"]).fails().code_is(1);
}