Skip to content

Commit 8a0a380

Browse files
committed
fixup
1 parent fddba26 commit 8a0a380

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

tests/v3/test_store/test_remote.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -145,9 +145,7 @@ def test_store_supports_partial_writes(self, store: RemoteStore) -> None:
145145
def test_store_supports_listing(self, store: RemoteStore) -> None:
146146
assert store.supports_listing
147147

148-
async def test_remote_store_from_uri(
149-
self, store: RemoteStore, store_kwargs: dict[str, str | bool]
150-
):
148+
async def test_remote_store_from_uri(self, store: RemoteStore):
151149
storage_options = {
152150
"endpoint_url": endpoint_url,
153151
"anon": False,
@@ -197,6 +195,7 @@ def test_from_upath(self) -> None:
197195
assert result.path == f"{test_bucket_name}/foo/bar"
198196

199197
def test_init_raises_if_path_has_scheme(self, store_kwargs) -> None:
198+
# regression test for https://github.com/zarr-developers/zarr-python/issues/2342
200199
store_kwargs["path"] = "s3://" + store_kwargs["path"]
201200
with pytest.raises(
202201
ValueError, match="path argument to RemoteStore must not include scheme .*"

0 commit comments

Comments
 (0)