ls: when -aA are provided, the order matters

uutils · Mar 20, 2022 · af8726a · af8726a
1 parent 04b219b
commit af8726a
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 0 deletions.
diff --git a/src/uu/ls/src/ls.rs b/src/uu/ls/src/ls.rs
@@ -1202,12 +1202,14 @@ pub fn uu_app<'a>() -> Command<'a> {
             Arg::new(options::files::ALL)
                 .short('a')
                 .long(options::files::ALL)
+                .overrides_with(options::files::ALMOST_ALL)
                 .help("Do not ignore hidden files (files with names that start with '.')."),
         )
         .arg(
             Arg::new(options::files::ALMOST_ALL)
                 .short('A')
                 .long(options::files::ALMOST_ALL)
+                .overrides_with(options::files::ALL)
                 .help(
                     "In a directory, do not ignore all file names that start with '.', \
 only ignore '.' and '..'.",

diff --git a/tests/by-util/test_ls.rs b/tests/by-util/test_ls.rs
@@ -2869,3 +2869,25 @@ fn test_ls_context_format() {
             );
     }
 }
+
+#[test]
+#[allow(non_snake_case)]
+fn test_ls_a_A() {
+    let scene = TestScenario::new(util_name!());
+
+    scene
+        .ucmd()
+        .arg("-A")
+        .arg("-a")
+        .succeeds()
+        .stdout_contains(".")
+        .stdout_contains("..");
+
+    scene
+        .ucmd()
+        .arg("-a")
+        .arg("-A")
+        .succeeds()
+        .stdout_does_not_contain(".")
+        .stdout_does_not_contain("..");
+}