From 7842b48e60cd54be0e5f248563821a766590b0f0 Mon Sep 17 00:00:00 2001
From: Hermes Agent <hermes@localhost>
Date: Wed, 12 Aug 2026 05:45:02 +0000
Subject: Prevent archive overwrite races

---
 tests/archive.rs | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

(limited to 'tests')

diff --git a/tests/archive.rs b/tests/archive.rs
index 6f803ef..6e59837 100644
--- a/tests/archive.rs
+++ b/tests/archive.rs
@@ -124,6 +124,24 @@ fn rejects_an_archive_output_inside_the_rootfs() {
     assert!(!rootfs.join("controller.tar.partial").exists());
 }
 
+#[test]
+fn refuses_to_replace_an_existing_archive() {
+    let fixture = tempdir().expect("fixture directory");
+    let rootfs = fixture.path().join("rootfs");
+    fs::create_dir(&rootfs).expect("create rootfs");
+    fs::write(rootfs.join("controller.conf"), "controller=true\n").expect("write rootfs file");
+    let artifact = fixture.path().join("controller.tar");
+    fs::write(&artifact, "existing artifact").expect("write existing artifact");
+
+    let error = NativeTarWriter::new()
+        .write(&rootfs, &artifact)
+        .expect_err("an existing archive must not be replaced");
+
+    assert!(error.to_string().contains("archive output already exists"));
+    assert_eq!(fs::read_to_string(&artifact).expect("read existing artifact"), "existing artifact");
+    assert!(!fixture.path().join("controller.tar.partial").exists());
+}
+
 #[test]
 fn removes_partial_output_when_an_unsupported_entry_stops_packaging() {
     let fixture = tempdir().expect("fixture directory");
-- 
cgit v1.2.3