Add tests for mktemp tmpdir flags

And set overrides_with for tmpdir flags. Tests were copied from #4275 Co-authored-by: David Matos <[email protected]>
uutils · Feb 13, 2023 · 5423deb · 5423deb
1 parent 49ddd6f
commit 5423deb
Show file tree

Hide file tree

Showing 2 changed files with 61 additions and 0 deletions.
diff --git a/src/uu/mktemp/src/mktemp.rs b/src/uu/mktemp/src/mktemp.rs
@@ -418,6 +418,7 @@ pub fn uu_app() -> Command {
                 .num_args(0..=1)
                 // Require an equals to avoid ambiguity if no tmpdir is supplied
                 .require_equals(true)
+                .overrides_with(OPT_P)
                 .value_parser(ValueParser::path_buf())
                 .value_hint(clap::ValueHint::DirPath),
         )

diff --git a/tests/by-util/test_mktemp.rs b/tests/by-util/test_mktemp.rs
@@ -826,3 +826,63 @@ fn test_default_missing_value() {
     let scene = TestScenario::new(util_name!());
     scene.ucmd().arg("-d").arg("--tmpdir").succeeds();
 }
+
+#[test]
+fn test_missing_xs_tmpdir_template() {
+    let scene = TestScenario::new(util_name!());
+    scene
+        .ucmd()
+        .arg("--tmpdir")
+        .arg(TEST_TEMPLATE3)
+        .fails()
+        .no_stdout()
+        .stderr_contains("too few X's in template");
+    scene
+        .ucmd()
+        .arg("--tmpdir=foobar")
+        .fails()
+        .no_stdout()
+        .stderr_contains("failed to create file via template");
+}
+
+#[test]
+fn test_both_tmpdir_flags_present() {
+    let scene = TestScenario::new(util_name!());
+    scene
+        .ucmd()
+        .arg("-p")
+        .arg(".")
+        .arg("--tmpdir")
+        .arg("foobarXXXX")
+        .succeeds()
+        .no_stderr()
+        .stdout_contains("/tmp/foobar");
+    scene
+        .ucmd()
+        .arg("-p")
+        .arg(".")
+        .arg("--tmpdir=foobarXXXX")
+        .fails()
+        .no_stdout()
+        .stderr_contains("failed to create file via template");
+    scene
+        .ucmd()
+        .arg("--tmpdir")
+        .arg("foobarXXXX")
+        .arg("-p")
+        .arg(".")
+        .succeeds()
+        .no_stderr()
+        .stdout_contains("./foobar");
+}
+
+#[test]
+fn test_missing_short_tmpdir_flag() {
+    let scene = TestScenario::new(util_name!());
+    scene
+        .ucmd()
+        .arg("-p")
+        .fails()
+        .no_stdout()
+        .stderr_contains("requires a value but none was supplied");
+}