small fixes to tests

Author: K-Meech

tests/test_cli/test_migrate_to_v3.py

@@ -167,7 +167,9 @@ def test_remove_v2_metadata_option_in_place(
 
 
 async def test_remove_v2_metadata_option_separate_location(
-    tmp_path: Path, expected_paths_v2_metadata: list[Path]
+    tmp_path: Path,
+    expected_paths_v2_metadata: list[Path],
+    expected_paths_v3_metadata_no_chunks: list[Path],
 ) -> None:
     """Check that when using --remove-v2-metadata with a separate output location, no v2 metadata is removed from
     the input location."""
@@ -189,6 +191,11 @@ async def test_remove_v2_metadata_option_separate_location(
     expected_paths = [input_zarr_path / p for p in expected_paths_v2_metadata]
     assert paths == expected_paths
 
+    # output image should be only v3 metadata
+    paths = sorted(output_zarr_path.rglob("*"))
+    expected_paths = [output_zarr_path / p for p in expected_paths_v3_metadata_no_chunks]
+    assert paths == expected_paths
+
 
 def test_overwrite_option_in_place(
     local_store: Store, expected_paths_v2_v3_metadata: list[Path]
@@ -591,7 +598,9 @@ def test_dry_run(
     create_nested_zarr(local_store)
 
     if cli_command == "migrate":
-        result = runner.invoke(cli.app, ["migrate", "v3", str(local_store.root), "--dry-run"])
+        result = runner.invoke(
+            cli.app, ["migrate", "v3", str(local_store.root), "--overwrite", "--force", "--dry-run"]
+        )
     else:
         result = runner.invoke(
             cli.app, ["remove-metadata", "v2", str(local_store.root), "--force", "--dry-run"]