fix(command-vendor): reproduce the panic of cp_sources

Author: vksir

src/cargo/ops/vendor.rs

@@ -233,7 +233,9 @@ fn sync(
             &format!("{} ({}) to {}", id, src.to_string_lossy(), dst.display()),
         )?;
 
-        let _ = fs::remove_dir_all(&dst);
+        // Comment it to keep the blocking file to force rename failure.
+        // This action Will be undo later.
+        // let _ = fs::remove_dir_all(&dst);
 
         let mut file_cksums = BTreeMap::new();
 

tests/testsuite/vendor.rs

@@ -2086,3 +2086,44 @@ Caused by:
 "#]])
         .run();
 }
+
+#[cargo_test]
+fn vendor_rename_fallback() {
+    let p = project()
+        .file(
+            "Cargo.toml",
+            r#"
+                [package]
+                name = "foo"
+                version = "0.1.0"
+
+                [dependencies]
+                log = "0.3.5"
+            "#,
+        )
+        .file("src/lib.rs", "")
+        .build();
+
+    Package::new("log", "0.3.5").publish();
+
+    // Create vendor/log/blocking_file as a file to force rename failure
+    let log_pkg_dir= p.root().join("vendor").join("log");
+    let block_file_path = log_pkg_dir.join("blocking_file");
+    fs::create_dir_all(&log_pkg_dir).unwrap();
+    fs::write(block_file_path, "blocking file").unwrap();
+
+    p.cargo("vendor --respect-source-config --no-delete")
+        .env("CARGO_LOG", "cargo::ops::vendor=warn")
+        .with_status(101)
+        .with_stderr_data(str![[r#"
+...
+[..]failed to `mv "[..]vendor[..].vendor-staging[..]log-0.3.5" "[..]vendor[..]log"`: [..]
+...
+[..]StripPrefixError[..]
+...
+"#]])
+        .run();
+
+    // Assert cp_sources fallback didn't work
+    assert!(!p.root().join("vendor/log/Cargo.toml").exists());
+}