fix: correct e2e force tests to use file corruption instead of version swap

npm test: Swapping to minimist@1.2.5 changes the PURL version so the
crawler never matches the manifest key — apply returns success without
reaching the hash check. Instead, corrupt the file content in-place.

pypi test: The pydantic-ai patch variant may have files with empty
beforeHash (new files). Dynamically find a file with a non-empty
beforeHash to corrupt.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: mikolalysenko

crates/socket-patch-cli/tests/e2e_npm.rs

@@ -440,7 +440,7 @@ fn test_npm_save_only() {
     );
 }
 
-/// `apply --force` should apply patches even when the installed version differs.
+/// `apply --force` should apply patches even when file hashes don't match.
 #[test]
 #[ignore]
 fn test_npm_apply_force() {
@@ -452,7 +452,6 @@ fn test_npm_apply_force() {
     let dir = tempfile::tempdir().unwrap();
     let cwd = dir.path();
 
-    // Install minimist@1.2.2 first, get the patch, then swap to a different version.
     write_package_json(cwd);
     npm_run(cwd, &["install", "minimist@1.2.2"]);
 
@@ -462,13 +461,12 @@ fn test_npm_apply_force() {
     // Save the patch without applying.
     assert_run_ok(cwd, &["get", NPM_UUID, "--save-only"], "get --save-only");
 
-    // Now reinstall a different version to create a hash mismatch.
-    npm_run(cwd, &["install", "minimist@1.2.5"]);
-
-    let mismatched_hash = git_sha256_file(&index_js);
+    // Corrupt the file to create a hash mismatch (keep same version so PURL matches).
+    std::fs::write(&index_js, b"// corrupted content\n").unwrap();
     assert_ne!(
-        mismatched_hash, BEFORE_HASH,
-        "minimist@1.2.5 should have a different index.js hash"
+        git_sha256_file(&index_js),
+        BEFORE_HASH,
+        "corrupted file should have a different hash"
     );
 
     // Normal apply should fail due to hash mismatch.

crates/socket-patch-cli/tests/e2e_pypi.rs

@@ -594,22 +594,33 @@ fn test_pypi_apply_force() {
     let (_, files_value) = read_patch_files(&manifest_path);
     let files = files_value.as_object().unwrap();
 
-    // Corrupt one of the files to create a hash mismatch.
-    let messages_py = site_packages.join("pydantic_ai/messages.py");
-    let before_hash = files["pydantic_ai/messages.py"]["beforeHash"]
-        .as_str()
-        .unwrap();
+    // Find a file with a non-empty beforeHash (i.e., an existing file, not a new one).
+    let (target_rel, target_before_hash) = files
+        .iter()
+        .filter_map(|(rel_path, info)| {
+            let bh = info["beforeHash"].as_str().unwrap_or("");
+            if !bh.is_empty() {
+                Some((rel_path.clone(), bh.to_string()))
+            } else {
+                None
+            }
+        })
+        .next()
+        .expect("patch should have at least one file with a non-empty beforeHash");
+
+    let target_path = site_packages.join(&target_rel);
     assert_eq!(
-        git_sha256_file(&messages_py),
-        before_hash,
-        "file should match beforeHash before corruption"
+        git_sha256_file(&target_path),
+        target_before_hash,
+        "{target_rel} should match beforeHash before corruption"
     );
 
-    std::fs::write(&messages_py, b"# corrupted content\n").unwrap();
+    // Corrupt the file to create a hash mismatch.
+    std::fs::write(&target_path, b"# corrupted content\n").unwrap();
     assert_ne!(
-        git_sha256_file(&messages_py),
-        before_hash,
-        "file should have a different hash after corruption"
+        git_sha256_file(&target_path),
+        target_before_hash,
+        "{target_rel} should have a different hash after corruption"
     );
 
     // Normal apply should fail due to hash mismatch.