Update reprs for LoggingStore and WrapperStore

Author: maxrjones

src/zarr/storage/_logging.py

@@ -161,7 +161,7 @@ def __str__(self) -> str:
         return f"logging-{self._store}"
 
     def __repr__(self) -> str:
-        return f"LoggingStore({repr(self._store)!r})"
+        return f"LoggingStore({self._store.__class__.__name__}, '{self._store}')"
 
     def __eq__(self, other: object) -> bool:
         with self.log(other):

src/zarr/storage/_wrapper.py

@@ -81,7 +81,7 @@ def __str__(self) -> str:
         return f"wrapping-{self._store}"
 
     def __repr__(self) -> str:
-        return f"WrapperStore({repr(self._store)!r})"
+        return f"WrapperStore({self._store.__class__.__name__}, '{self._store}')"
 
     async def get(
         self, key: str, prototype: BufferPrototype, byte_range: ByteRequest | None = None

tests/test_store/test_logging.py

@@ -51,6 +51,9 @@ def test_store_supports_listing(self, store: LoggingStore) -> None:
         assert store.supports_listing
 
     def test_store_repr(self, store: LoggingStore) -> None:
+        assert f"{store!r}" == f"LoggingStore(LocalStore, 'file://{store._store.root.as_posix()}')"
+
+    def test_store_str(self, store: LoggingStore) -> None:
         assert str(store) == f"logging-file://{store._store.root.as_posix()}"
 
     async def test_default_handler(self, local_store, capsys) -> None:

tests/test_store/test_wrapper.py

@@ -50,6 +50,9 @@ def test_store_supports_listing(self, store: WrapperStore) -> None:
         assert store.supports_listing
 
     def test_store_repr(self, store: WrapperStore) -> None:
+        assert f"{store!r}" == f"WrapperStore(LocalStore, 'file://{store._store.root.as_posix()}')"
+
+    def test_store_str(self, store: WrapperStore) -> None:
         assert str(store) == f"wrapping-file://{store._store.root.as_posix()}"
 
     def test_is_open_setter_raises(self, store: WrapperStore) -> None: