test cli tests and disable copy

Author: Altay Sansal

src/mdio/api/convenience.py

@@ -21,7 +21,6 @@
     from mdio import MDIOReader
 
 
-# TODO(Altay): This is not implemented in Zarr v3 yet.
 def copy_mdio(  # noqa: PLR0913
     source: MDIOReader,
     dest_path_or_buffer: str,
@@ -52,20 +51,22 @@ def copy_mdio(  # noqa: PLR0913
             Default is None (will assume anonymous).
         overwrite: Overwrite destination or not.
 
+    Raises:
+        NotImplementedError: because Zarr v3 doesn't support copy.
     """
     if storage_options is None:
         storage_options = {}
 
     dest_store = process_url(
         url=dest_path_or_buffer,
-        mode="w",
-        storage_options=storage_options,
-        memory_cache_size=0,
         disk_cache=False,
     )
 
     if_exists = "replace" if overwrite is True else "raise"
 
+    # TODO(Altay): Update this function when Zarr v3 supports copy.
+    raise NotImplementedError("Zarr version 3.0.0+ does not support copy yet.")
+
     zarr.copy_store(
         source=source.store,
         dest=dest_store,

src/mdio/commands/info.py

@@ -95,7 +95,7 @@ def parse_grid(grid: Grid) -> dict[str, dict[str, int | str]]:
 def parse_access_patterns(reader: MDIOReader) -> dict[str, Any]:
     """Extract access patterns and their info."""
     access_pattern_dict = {}
-    for name, array in reader._data_group.items():
+    for name, array in reader._data_group.arrays():
         pattern = name.replace("chunked_", "")
         chunks = str(array.chunks)
         format_ = str(array.dtype)

tests/test_main.py

@@ -55,7 +55,9 @@ def test_main_copy_succeeds(runner: CliRunner, zarr_tmp: Path, zarr_tmp2: Path)
     cli_args = ["copy", str(zarr_tmp), str(zarr_tmp2)]
 
     result = runner.invoke(__main__.main, args=cli_args)
-    assert result.exit_code == 0
+
+    # TODO(Altay): Update exit code to zero once Zarr v3 store supports copy.
+    assert result.exit_code == 1
 
 
 def test_cli_version(runner: CliRunner) -> None: