fixup

Author: jhamman

src/zarr/storage/logging.py

@@ -235,6 +235,3 @@ def with_mode(self, mode: AccessModeLiteral) -> Self:
                 log_level=self.log_level,
                 log_handler=self.log_handler,
             )
-
-
-# TODO: wrap delete methods here

src/zarr/testing/store.py

@@ -224,10 +224,12 @@ async def test_delete_dir(self, store: S) -> None:
         if not store.supports_deletes:
             pytest.skip("store does not support deletes")
         await store.set("zarr.json", self.buffer_cls.from_bytes(b"root"))
+        await store.set("foo-bar/zarr.json", self.buffer_cls.from_bytes(b"root"))
         await store.set("foo/zarr.json", self.buffer_cls.from_bytes(b"bar"))
         await store.set("foo/c/0", self.buffer_cls.from_bytes(b"chunk"))
         await store.delete_dir("foo")
         assert await store.exists("zarr.json")
+        assert await store.exists("foo-bar/zarr.json")
         assert not await store.exists("foo/zarr.json")
         assert not await store.exists("foo/c/0")
 

tests/test_array.py

@@ -52,7 +52,6 @@ def test_array_creation_existing_node(
 
     if exists_ok:
         if not store.supports_deletes:
-            # TODO: confirm you get the expected error here
             pytest.skip("store does not support deletes")
         arr_new = Array.create(
             spath / "extant",