Merge branch 'master' into clap-pwd

tests/by-util/test_install.rs

@@ -10,7 +10,7 @@ fn test_install_help() {
         .succeeds()
         .no_stderr()
         .stdout
-        .contains("Options:"));
+        .contains("FLAGS:"));
 }
 
 #[test]

tests/by-util/test_mkdir.rs

@@ -13,6 +13,16 @@ fn test_mkdir_mkdir() {
     new_ucmd!().arg(TEST_DIR1).succeeds();
 }
 
+#[test]
+fn test_mkdir_verbose() {
+    let expected = "mkdir: created directory 'mkdir_test1'\n";
+    new_ucmd!()
+        .arg(TEST_DIR1)
+        .arg("-v")
+        .run()
+        .stdout_is(expected);
+}
+
 #[test]
 fn test_mkdir_dup_dir() {
     let scene = TestScenario::new(util_name!());

tests/by-util/test_rm.rs

@@ -12,6 +12,14 @@ fn test_rm_one_file() {
     assert!(!at.file_exists(file));
 }
 
+#[test]
+fn test_rm_failed() {
+    let (_at, mut ucmd) = at_and_ucmd!();
+    let file = "test_rm_one_file";
+
+    ucmd.arg(file).fails(); // Doesn't exist
+}
+
 #[test]
 fn test_rm_multiple_files() {
     let (at, mut ucmd) = at_and_ucmd!();

tests/by-util/test_sync.rs

@@ -1 +1,11 @@
-// ToDO: add tests
+use crate::common::util::*;
+
+#[test]
+fn test_sync_default() {
+    new_ucmd!().run();
+}
+
+#[test]
+fn test_sync_incorrect_arg() {
+    new_ucmd!().arg("--foo").fails();
+}