Skip to content

Commit 961bd77

Browse files
committed
refactor: remove redundant zarr format fixture
1 parent 66c4a01 commit 961bd77

File tree

2 files changed

+1
-9
lines changed

2 files changed

+1
-9
lines changed

tests/v3/conftest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ def array_fixture(request: pytest.FixtureRequest) -> npt.NDArray[Any]:
138138
)
139139

140140

141-
@pytest.fixture(params=(2, 3))
141+
@pytest.fixture(params=(2, 3), ids=["zarr2", "zarr3"])
142142
def zarr_format(request: pytest.FixtureRequest) -> ZarrFormat:
143143
if request.param == 2:
144144
return 2

tests/v3/test_group.py

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -43,14 +43,6 @@ def exists_ok(request: pytest.FixtureRequest) -> bool:
4343
return result
4444

4545

46-
@pytest.fixture(params=[2, 3], ids=["zarr2", "zarr3"])
47-
def zarr_format(request: pytest.FixtureRequest) -> ZarrFormat:
48-
result = request.param
49-
if result not in (2, 3):
50-
raise ValueError("Wrong value returned from test fixture.")
51-
return cast(ZarrFormat, result)
52-
53-
5446
def test_group_init(store: Store, zarr_format: ZarrFormat) -> None:
5547
"""
5648
Test that initializing a group from an asyncgroup works.

0 commit comments

Comments
 (0)