tests: fix interaction between azure tests (#123)

* tests: fix interaction between azure tests

* update changelog entry

Author: ap--

CHANGELOG.md

@@ -9,6 +9,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 - updated past changelog entries.
 
+### Fixed
+- improved azure test separation (#123).
+
 ## [0.0.24] - 2023-06-19
 ### Added
 - started a changelog to keep track of significant changes (#118).

upath/tests/conftest.py

@@ -5,6 +5,7 @@
 import sys
 import threading
 import time
+import uuid
 from pathlib import Path
 
 import fsspec
@@ -32,7 +33,6 @@ def clear_registry():
 
 @pytest.fixture(scope="function")
 def local_testdir(tmp_path, clear_registry):
-    tmp_path.mkdir(exist_ok=True)
     folder1 = tmp_path.joinpath("folder1")
     folder1.mkdir()
     folder1_files = ["file1.txt", "file2.txt"]
@@ -367,13 +367,31 @@ def docker_azurite(azurite_credentials):
 
 
 @pytest.fixture(scope="session")
-def azure_fixture(azurite_credentials, docker_azurite):
+def azure_container(azurite_credentials, docker_azurite):
     azure_storage = pytest.importorskip("azure.storage.blob")
     account_name, connection_string = azurite_credentials
     client = azure_storage.BlobServiceClient.from_connection_string(
         conn_str=connection_string
     )
-    container_name = "data"
+    container_name = str(uuid.uuid4())
     client.create_container(container_name)
 
-    yield f"az://{container_name}"
+    try:
+        yield container_name
+    finally:
+        client.delete_container(container_name)
+
+
+@pytest.fixture(scope="function")
+def azure_fixture(azurite_credentials, azure_container):
+    azure_storage = pytest.importorskip("azure.storage.blob")
+    account_name, connection_string = azurite_credentials
+    client = azure_storage.BlobServiceClient.from_connection_string(
+        conn_str=connection_string
+    ).get_container_client(azure_container)
+
+    try:
+        yield f"az://{azure_container}"
+    finally:
+        for blob in client.list_blobs():
+            client.delete_blob(blob["name"])

upath/tests/implementations/test_azure.py

@@ -32,7 +32,6 @@ def test_rmdir(self):
         path = new_dir / "test.txt"
         path.write_text("hello")
         assert path.exists()
-        new_dir.fs.invalidate_cache()
         new_dir.rmdir()
         assert not new_dir.exists()
 
@@ -43,6 +42,5 @@ def test_rmdir(self):
     def test_makedirs_exist_ok_false(self):
         pass
 
-    @pytest.mark.xfail(reason="test interaction")
     def test_rglob(self, pathlib_base):
         return super().test_rglob(pathlib_base)