No error on test cleanup in CI

micromamba/tests/conftest.py

@@ -76,11 +76,14 @@ def tmp_home(
 
     # Pytest would clean it automatically but this can be large (0.5 Gb for repodata)
     # We clean it explicitly
+    on_ci = "CI" in os.environ
     if not request.config.getoption("--no-eager-clean"):
         try:
             helpers.rmtree(new_home)
-        except PermissionError:
-            pass
+        # Silence possible cleanup exeptions on CI, weird things happening there
+        except Exception as e:
+            if not on_ci:
+                raise e
 
 
 @pytest.fixture
@@ -148,9 +151,14 @@ def tmp_root_prefix(
 
     # Pytest would clean it automatically but this can be large (0.5 Gb for repodata)
     # We clean it explicitly
+    on_ci = "CI" in os.environ
     if not request.config.getoption("--no-eager-clean"):
-        if new_root_prefix.exists():
+        try:
             helpers.rmtree(new_root_prefix)
+        # Silence possible cleanup exeptions on CI, weird things happening there
+        except Exception as e:
+            if not on_ci:
+                raise e
     # os.environ restored by tmp_clean_env and tmp_environ
 
 

micromamba/tests/helpers.py

@@ -473,17 +473,21 @@ def recursive_chmod(path: Path, permission, is_root=True):
 
 
 def rmtree(path: Path):
-    p = Path(path)
-    recursive_chmod(p, 0o700)
+    path = Path(path)
 
-    def handleError(func, path, exc_info):
-        recursive_chmod(path, 0o700)
-        func(path)
+    if not path.exists():
+        return
 
-    if p.is_dir():
-        shutil.rmtree(p, onerror=handleError)
+    recursive_chmod(path, 0o700)
+
+    def handleError(func, p, exc_info):
+        recursive_chmod(p, 0o700)
+        func(p)
+
+    if path.is_dir():
+        shutil.rmtree(path, onerror=handleError)
     else:
-        os.remove(p)
+        os.remove(path)
 
 
 def get_fake_activate(prefix):