bug fixes in persistentcache location init

Author: tclose

pydra/utils/hash.py

@@ -60,7 +60,8 @@ def location_converter(path: ty.Union[Path, str, None]) -> Path:
     if path is None:
         path = PersistentCache.location_default()
     path = Path(path)
-    path.mkdir(parents=True, exist_ok=True)
+    if not path.exists():
+        path.mkdir(parents=True)
     return path
 
 

pydra/utils/tests/test_hash.py

@@ -3,7 +3,7 @@
 from hashlib import blake2b
 from pathlib import Path
 import time
-
+from unittest import mock
 import attrs
 import pytest
 import typing as ty
@@ -336,7 +336,7 @@ def test_persistent_hash_cache(cache_path, text_file):
 
     # Test that changes to the text file result in new hash
     time.sleep(2)  # Need to ensure that the mtimes will be different
-    text_file_path.write_text("bar")
+    text_file.fspath.write_text("bar")
     assert hash_object(text_file, persistent_cache=cache_path) != modified_hash
     assert len(list(cache_path.iterdir())) == 2