document changes and format

Author: brokkoli71

changes/3128.bugfix.rst

@@ -0,0 +1 @@
+Fix `zarr.open` default for argument `mode` when `store` is `read_only`

tests/test_api.py

@@ -39,7 +39,7 @@
 )
 from zarr.core.buffer import NDArrayLike
 from zarr.errors import MetadataValidationError
-from zarr.storage import MemoryStore, ZipStore, LocalStore
+from zarr.storage import LocalStore, MemoryStore, ZipStore
 from zarr.storage._utils import normalize_path
 from zarr.testing.utils import gpu_test
 
@@ -399,6 +399,7 @@ def test_load_array(memory_store: Store) -> None:
         else:
             assert_array_equal(bar, array)
 
+
 @pytest.mark.parametrize("path", ["data", None])
 def test_load_zip(tmp_path: pathlib.Path, path: str | None) -> None:
     file = tmp_path / "test.zip"
@@ -411,10 +412,10 @@ def test_load_zip(tmp_path: pathlib.Path, path: str | None) -> None:
         assert np.array_equal(result, data)
     with ZipStore(file, mode="r") as zs:
         result = zarr.load(store=zs, path=path)
-        assert np.array_equal(result, data)
+        assert_array_equal(result, data)
     with ZipStore(file, read_only=True) as zs:
         result = zarr.load(store=zs, path=path)
-        assert np.array_equal(result, data)
+        assert_array_equal(result, data)
 
 
 @pytest.mark.parametrize("path", ["data", None])
@@ -426,10 +427,11 @@ def test_load_local(tmp_path: pathlib.Path, path: str | None) -> None:
         save(zs, data, path=path)
     with LocalStore(file, read_only=False) as zs:
         result = zarr.load(store=zs, path=path)
-        assert np.array_equal(result, data)
+        assert_array_equal(result, data)
     with LocalStore(file, read_only=True) as zs:
         result = zarr.load(store=zs, path=path)
-        assert np.array_equal(result, data)
+        assert_array_equal(result, data)
+
 
 def test_tree() -> None:
     pytest.importorskip("rich")