fixup

Author: TomAugspurger

src/zarr/store/common.py

@@ -89,7 +89,7 @@ async def make_store_path(
             assert AccessMode.from_literal(mode) == store_like.store.mode
         result = store_like
     elif isinstance(store_like, Store):
-        if mode is not None:
+        if mode is not None and mode != store_like.mode.str:
             store_like = store_like.with_mode(mode)
         await store_like._ensure_open()
         result = StorePath(store_like)

tests/v3/test_store/test_logging.py

@@ -53,6 +53,6 @@ async def test_logging_store_counter(store: Store) -> None:
 
 async def test_with_mode():
     wrapped = LoggingStore(store=zarr.store.MemoryStore(mode="w"), log_level="INFO")
-    new = await wrapped.with_mode(mode="r")
+    new = wrapped.with_mode(mode="r")
     assert new.mode.str == "r"
     assert new.log_level == "INFO"