Do not delete cffi-provider venv

PyO3 · Nov 25, 2024 · 67e8421 · 67e8421
1 parent 75c0060
commit 67e8421
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/tests/common/integration.rs b/tests/common/integration.rs
@@ -94,7 +94,11 @@ pub fn test_integration(
         let file = File::create(venvs_dir.join("cffi-provider.lock"))?;
         file.lock_exclusive()?;
         if !python.is_file() {
-            create_named_virtualenv(cffi_provider, python_interp.clone().map(PathBuf::from))?;
+            create_named_virtualenv(
+                cffi_provider,
+                python_interp.clone().map(PathBuf::from),
+                false,
+            )?;
             let pip_install_cffi = [
                 "-m",
                 "pip",

diff --git a/tests/common/mod.rs b/tests/common/mod.rs
@@ -126,21 +126,24 @@ pub fn create_virtualenv(name: &str, python_interp: Option<PathBuf>) -> Result<(
         Err(_) => name.to_string(),
     };
 
-    let venv_dir = create_named_virtualenv(&venv_name, interp)?;
-
+    let venv_dir = create_named_virtualenv(&venv_name, interp, true)?;
     let target = Target::from_target_triple(None)?;
     let python = target.get_venv_python(&venv_dir);
     Ok((venv_dir, python))
 }
 
-pub fn create_named_virtualenv(venv_name: &str, interp: Option<PathBuf>) -> Result<PathBuf> {
+pub fn create_named_virtualenv(
+    venv_name: &str,
+    interp: Option<PathBuf>,
+    clear: bool,
+) -> Result<PathBuf> {
     let venv_dir = PathBuf::from("test-crates")
         .normalize()?
         .into_path_buf()
         .join("venvs")
         .join(venv_name);
 
-    if venv_dir.is_dir() {
+    if clear && venv_dir.is_dir() {
         fs::remove_dir_all(&venv_dir)?;
     }