Remove useless {...!s} (zarr-developers#2471)

Author: DimitriPapadopoulos

src/zarr/storage/logging.py

@@ -55,7 +55,7 @@ def _configure_logger(
         self, log_level: str = "DEBUG", log_handler: logging.Handler | None = None
     ) -> None:
         self.log_level = log_level
-        self.logger = logging.getLogger(f"LoggingStore({self._store!s})")
+        self.logger = logging.getLogger(f"LoggingStore({self._store})")
         self.logger.setLevel(log_level)
 
         if not self.logger.hasHandlers():
@@ -147,7 +147,7 @@ async def clear(self) -> None:
             return await self._store.clear()
 
     def __str__(self) -> str:
-        return f"logging-{self._store!s}"
+        return f"logging-{self._store}"
 
     def __repr__(self) -> str:
         return f"LoggingStore({repr(self._store)!r})"

tests/test_store/test_local.py

@@ -31,7 +31,7 @@ def store_kwargs(self, tmpdir) -> dict[str, str]:
         return {"root": str(tmpdir), "mode": "r+"}
 
     def test_store_repr(self, store: LocalStore) -> None:
-        assert str(store) == f"file://{store.root.as_posix()!s}"
+        assert str(store) == f"file://{store.root.as_posix()}"
 
     def test_store_supports_writes(self, store: LocalStore) -> None:
         assert store.supports_writes

tests/test_store/test_zip.py

@@ -53,7 +53,7 @@ async def test_not_writable_store_raises(self, store_kwargs: dict[str, Any]) ->
             await store.set("foo", cpu.Buffer.from_bytes(b"bar"))
 
     def test_store_repr(self, store: ZipStore) -> None:
-        assert str(store) == f"zip://{store.path!s}"
+        assert str(store) == f"zip://{store.path}"
 
     def test_store_supports_writes(self, store: ZipStore) -> None:
         assert store.supports_writes