Fix test_local

Author: dstansby

pyproject.toml

@@ -359,7 +359,8 @@ module = [
     "zarr.testing.stateful",
     "tests.test_codecs.test_transpose",
     "tests.test_config",
-    "tests.test_store.test_zip"
+    "tests.test_store.test_zip",
+    "tests.test_store.test_local",
 ]
 strict = false
 
@@ -371,7 +372,6 @@ module = [
     "tests.test_metadata.*",
     "tests.test_store.test_core",
     "tests.test_store.test_fsspec",
-    "tests.test_store.test_local",
     "tests.test_store.test_logging",
     "tests.test_store.test_memory",
     "tests.test_store.test_object",

src/zarr/testing/store.py

@@ -58,7 +58,7 @@ async def get(self, store: S, key: str) -> Buffer:
 
     @abstractmethod
     @pytest.fixture
-    def store_kwargs(self) -> dict[str, Any]:
+    def store_kwargs(self, *args: Any, **kwargs: Any) -> dict[str, Any]:
         """Kwargs for instantiating a store"""
         ...
 

tests/test_store/test_local.py

@@ -28,7 +28,7 @@ async def set(self, store: LocalStore, key: str, value: Buffer) -> None:
         (store.root / key).write_bytes(value.to_bytes())
 
     @pytest.fixture
-    def store_kwargs(self, tmpdir) -> dict[str, str]:
+    def store_kwargs(self, tmpdir: str) -> dict[str, str]:
         return {"root": str(tmpdir)}
 
     def test_store_repr(self, store: LocalStore) -> None:
@@ -48,24 +48,24 @@ async def test_empty_with_empty_subdir(self, store: LocalStore) -> None:
         (store.root / "foo/bar").mkdir(parents=True)
         assert await store.is_empty("")
 
-    def test_creates_new_directory(self, tmp_path: pathlib.Path):
+    def test_creates_new_directory(self, tmp_path: pathlib.Path) -> None:
         target = tmp_path.joinpath("a", "b", "c")
         assert not target.exists()
 
         store = self.store_cls(root=target)
         zarr.group(store=store)
 
-    def test_invalid_root_raises(self):
+    def test_invalid_root_raises(self) -> None:
         """
         Test that a TypeError is raised when a non-str/Path type is used for the `root` argument
         """
         with pytest.raises(
             TypeError,
             match=r"'root' must be a string or Path instance. Got an instance of <class 'int'> instead.",
         ):
-            LocalStore(root=0)
+            LocalStore(root=0)  # type: ignore[arg-type]
 
-    async def test_get_with_prototype_default(self, store: LocalStore):
+    async def test_get_with_prototype_default(self, store: LocalStore) -> None:
         """
         Ensure that data can be read via ``store.get`` if the prototype keyword argument is unspecified, i.e. set to ``None``.
         """