Merge pull request #4782 from cakebaker/mv_no_clobber_behavior_change

mv: adapt -n behavior to GNU mv 9.3

src/uu/mv/src/mv.rs

@@ -419,7 +419,12 @@ fn rename(
         }
 
         match b.overwrite {
-            OverwriteMode::NoClobber => return Ok(()),
+            OverwriteMode::NoClobber => {
+                return Err(io::Error::new(
+                    io::ErrorKind::Other,
+                    format!("not replacing {}", to.quote()),
+                ));
+            }
             OverwriteMode::Interactive => {
                 if !prompt_yes!("overwrite {}?", to.quote()) {
                     return Err(io::Error::new(io::ErrorKind::Other, ""));

tests/by-util/test_mv.rs

@@ -232,8 +232,9 @@ fn test_mv_no_clobber() {
     ucmd.arg("-n")
         .arg(file_a)
         .arg(file_b)
-        .succeeds()
-        .no_stderr();
+        .fails()
+        .code_is(1)
+        .stderr_only(format!("mv: not replacing '{file_b}'\n"));
 
     assert!(at.file_exists(file_a));
     assert!(at.file_exists(file_b));