more handle errors with multiple files

src/uu/more/src/more.rs

@@ -24,8 +24,8 @@
 
 use unicode_segmentation::UnicodeSegmentation;
 use unicode_width::UnicodeWidthStr;
-use uucore::display::Quotable;
 use uucore::error::{UResult, USimpleError, UUsageError};
+use uucore::{display::Quotable, show};
 use uucore::{format_usage, help_about, help_usage};
 
 const ABOUT: &str = help_about!("more.md");
@@ -105,25 +105,31 @@
             let file = Path::new(file);
             if file.is_dir() {
                 terminal::disable_raw_mode().unwrap();
-                return Err(UUsageError::new(
-                    1,
+                show!(UUsageError::new(
+                    0,
                     format!("{} is a directory.", file.quote()),
                 ));
+                terminal::enable_raw_mode().unwrap();
+                continue;
             }
             if !file.exists() {
                 terminal::disable_raw_mode().unwrap();
-                return Err(USimpleError::new(
-                    1,
+                show!(USimpleError::new(
+                    0,
                     format!("cannot open {}: No such file or directory", file.quote()),
                 ));
+                terminal::enable_raw_mode().unwrap();
+                continue;
             }
             let opened_file = match File::open(file) {
                 Err(why) => {
                     terminal::disable_raw_mode().unwrap();
-                    return Err(USimpleError::new(
-                        1,
+                    show!(USimpleError::new(
+                        0,
                         format!("cannot open {}: {}", file.quote(), why.kind()),
                     ));
+                    terminal::enable_raw_mode().unwrap();
+                    continue;
                 }
                 Ok(opened_file) => opened_file,
             };

tests/by-util/test_more.rs

@@ -91,8 +91,8 @@
     if std::io::stdout().is_terminal() {
         new_ucmd!()
             .arg(".")
-            .fails()
-            .usage_error("'.' is a directory.");
+            .succeeds()
+            .stderr_contains("'.' is a directory.");
     }
 }
 
@@ -108,8 +108,46 @@
         at.make_file("invalid-perms.txt");
         set_permissions(at.plus("invalid-perms.txt"), permissions).unwrap();
         ucmd.arg("invalid-perms.txt")
-            .fails()
-            .code_is(1)
+            .succeeds()
             .stderr_contains("permission denied");
     }
 }
+
+#[test]
+fn test_more_error_on_single_arg() {
+    if std::io::stdout().is_terminal() {
+        let ts = TestScenario::new("more");
+        ts.fixtures.mkdir_all("folder");
+        ts.ucmd()
+            .arg("folder")
+            .succeeds()
+            .stderr_contains("is a directory");
+        ts.ucmd()
+            .arg("file1")
+            .succeeds()
+            .stderr_contains("No such file or directory");
+    }
+}
+
+#[test]
+fn test_more_error_on_multiple_files() {
+    if std::io::stdout().is_terminal() {
+        let ts = TestScenario::new("more");
+        ts.fixtures.mkdir_all("folder");
+        ts.fixtures.make_file("file1");
+        ts.ucmd()
+            .arg("folder")
+            .arg("file2")
+            .arg("file1")
+            .succeeds()
+            .stderr_contains("folder")
+            .stderr_contains("file2")
+            .stdout_contains("file1");
+        ts.ucmd()
+            .arg("file2")
+            .arg("file3")
+            .succeeds()
+            .stderr_contains("file2")
+            .stderr_contains("file3");
+    }
+}