add -q, --quiet flag to surpress non-error output

CHANGELOG.md

@@ -11,6 +11,9 @@ Possible log types:
 - `[fixed]` for any bug fixes.
 - `[security]` to invite users to upgrade in case of vulnerabilities.
 
+### Unreleased
+
+- [added] New --quiet / -q option to suppress most non-error messages
 
 ### v1.0.0 (2018-02-11)
 

Cargo.toml

@@ -32,6 +32,7 @@ xdg = "2.1.0"
 [dev-dependencies]
 tempdir = "^0.3"
 assert_cli = "^0.5"
+utime = "0.2.0"
 
 [features]
 dev = ["clippy"]

README.md

@@ -70,6 +70,7 @@ These are the clients I tried but failed to compile or run:
         -o --os <type>       Override the operating system [linux, osx, sunos]
         -u --update          Update the local cache
         -c --clear-cache     Clear the local cache
+        -q --quiet           Suppress informational messages
         --config-path        Show config file path
         --seed-config        Create a basic config
 

bash_tealdeer

@@ -6,7 +6,7 @@ _tealdeer()
 	_init_completion || return
 
 	case $prev in
-		-h|--help|-v|--version|-l|--list|-u|--update|-c|--clear-cache|--config-path|--seed-config)
+		-h|--help|-v|--version|-l|--list|-u|--update|-c|--clear-cache|--config-path|--seed-config|-q|--quiet)
 			return
 			;;
 		-f|--render)

src/main.rs

@@ -75,6 +75,7 @@ Options:
     -o --os <type>      Override the operating system [linux, osx, sunos]
     -u --update         Update the local cache
     -c --clear-cache    Clear the local cache
+    -q --quiet          Suppress informational messages
     --config-path       Show config file path
     --seed-config       Create a basic config
 
@@ -105,6 +106,7 @@ struct Args {
     flag_os: Option<OsType>,
     flag_update: bool,
     flag_clear_cache: bool,
+    flag_quiet: bool,
     flag_config_path: bool,
     flag_seed_config: bool,
 }
@@ -142,6 +144,7 @@ fn check_cache(args: &Args, cache: &Cache) {
     if !args.flag_update {
         match cache.last_update() {
             Some(ago) if ago > MAX_CACHE_AGE => {
+                if args.flag_quiet { return; }
                 println!("{}", Color::Red.paint(format!(
                     "Cache wasn't updated in {} days.\n\
                     You should probably run `tldr --update` soon.",
@@ -207,7 +210,9 @@ fn main() {
             };
             process::exit(1);
         });
-        println!("Successfully deleted cache.");
+        if !args.flag_quiet {
+            println!("Successfully deleted cache.");
+        }
     }
 
     // Update cache, pass through
@@ -219,7 +224,9 @@ fn main() {
             };
             process::exit(1);
         });
-        println!("Successfully updated cache.");
+        if !args.flag_quiet {
+            println!("Successfully updated cache.");
+        }
     }
 
     // Render local file and exit
@@ -266,9 +273,11 @@ fn main() {
                 process::exit(0);
             }
         } else {
-            println!("Page {} not found in cache", &command);
-            println!("Try updating with `tldr --update`, or submit a pull request to:");
-            eprintln!("https://github.com/tldr-pages/tldr");
+            if !args.flag_quiet {
+                println!("Page {} not found in cache", &command);
+                println!("Try updating with `tldr --update`, or submit a pull request to:");
+                println!("https://github.com/tldr-pages/tldr");
+            }
             process::exit(1);
         }
     }

tests/lib.rs

@@ -2,6 +2,7 @@
 
 extern crate assert_cli;
 extern crate tempdir;
+extern crate utime;
 
 use std::fs::File;
 use std::io::Write;
@@ -67,6 +68,71 @@ fn test_update_cache() {
         .unwrap();
 }
 
+#[test]
+fn test_quiet_cache() {
+    let testenv = TestEnv::new();
+    testenv
+        .assert()
+        .with_args(&["--update", "--quiet"])
+        .succeeds()
+        .stdout().is("")
+        .unwrap();
+
+    testenv
+        .assert()
+        .with_args(&["--clear-cache", "--quiet"])
+        .succeeds()
+        .stdout().is("")
+        .unwrap();
+}
+
+#[test]
+fn test_quiet_failures() {
+    let testenv = TestEnv::new();
+
+    testenv
+        .assert()
+        .with_args(&["--update", "-q"])
+        .succeeds()
+        .stdout().is("")
+        .unwrap();
+
+    testenv
+        .assert()
+        .with_args(&["fakeprogram", "-q"])
+        .fails()
+        .stdout().is("")
+        .unwrap();
+}
+
+#[test]
+fn test_quiet_old_cache() {
+    let testenv = TestEnv::new();
+
+    testenv
+        .assert()
+        .with_args(&["--update", "-q"])
+        .succeeds()
+        .stdout().is("")
+        .unwrap();
+
+    let _ = utime::set_file_times(testenv.cache_dir.path().join("tldr-master"), 1, 1).unwrap();
+
+    testenv
+        .assert()
+        .with_args(&["tldr"])
+        .succeeds()
+        .stdout().contains("Cache wasn't updated in ")
+        .unwrap();
+
+    testenv
+        .assert()
+        .with_args(&["tldr", "--quiet"])
+        .succeeds()
+        .stdout().doesnt_contain("Cache wasn't updated in ")
+        .unwrap();
+}
+
 #[test]
 fn test_setup_seed_config() {
     let testenv = TestEnv::new();