Skip to content

Commit d4fcd2b

Browse files
authored
test: update write_empty test for zarr 3.1.2 (#10665)
1 parent 8a88781 commit d4fcd2b

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

xarray/tests/test_backends.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4301,9 +4301,10 @@ def assert_expected_files(expected: list[str], store: str) -> None:
43014301
# that was performed by the roundtrip_dir
43024302
if (write_empty is False) or (write_empty is None and has_zarr_v3):
43034303
expected.append("1.1.0")
4304-
elif not has_zarr_v3:
4305-
# TODO: remove zarr3 if once zarr issue is fixed
4306-
# https://github.com/zarr-developers/zarr-python/issues/2931
4304+
elif not has_zarr_v3 or has_zarr_v3_async_oindex:
4305+
# this was broken from zarr 3.0.0 until 3.1.2
4306+
# async oindex released in 3.1.2 along with a fix
4307+
# for write_empty_chunks in append
43074308
expected.extend(
43084309
[
43094310
"1.1.0",

0 commit comments

Comments
 (0)