fix: repair test_batch_cli and clean up lint warnings

- Add missing total_detections param to format_summary test calls
- Update assertions to match current output format
- Remove unused imports in test_patch.py and test_patch_cli.py
- Apply ruff format to 3 files

Author: swaynehales

tests/test_batch_cli.py

@@ -97,21 +97,31 @@ def test_format_summary():
     from uitag.batch_cli import format_summary
 
     summary = format_summary(
-        succeeded=4, failed=1, total_seconds=9.5, output_dir="output/"
+        succeeded=4,
+        failed=1,
+        total_detections=42,
+        total_seconds=9.5,
+        output_dir="output/",
     )
-    assert "4 succeeded" in summary
+    assert "42 detections" in summary
     assert "1 failed" in summary
     assert "9.5s" in summary
+    assert "4 images" in summary
 
 
 def test_format_summary_no_failures():
     """Summary omits failure count when all succeed."""
     from uitag.batch_cli import format_summary
 
     summary = format_summary(
-        succeeded=3, failed=0, total_seconds=5.2, output_dir="out/"
+        succeeded=3,
+        failed=0,
+        total_detections=30,
+        total_seconds=5.2,
+        output_dir="out/",
     )
-    assert "3 succeeded" in summary
+    assert "30 detections" in summary
+    assert "3 images" in summary
     assert "failed" not in summary
 
 

tests/test_patch.py

@@ -1,8 +1,6 @@
 """Tests for patch JSON application."""
 
-import json
 import pytest
-from PIL import Image
 
 from uitag.types import Detection
 

tests/test_patch_cli.py

@@ -1,7 +1,6 @@
 """Tests for patch and render CLI subcommands."""
 
 import json
-import pytest
 from PIL import Image
 
 from uitag.types import Detection
@@ -88,15 +87,17 @@ def test_patch_cli_produces_output(tmp_path):
     patch_path = _write_patch(tmp_path, [{"som_id": 1, "label": "new"}])
 
     out_dir = tmp_path / "out"
-    patch_main([
-        str(img_path),
-        "--manifest",
-        str(manifest_path),
-        "--patch",
-        str(patch_path),
-        "-o",
-        str(out_dir),
-    ])
+    patch_main(
+        [
+            str(img_path),
+            "--manifest",
+            str(manifest_path),
+            "--patch",
+            str(patch_path),
+            "-o",
+            str(out_dir),
+        ]
+    )
 
     assert (out_dir / "test-uitag.png").exists()
     assert (out_dir / "test-uitag-manifest.json").exists()
@@ -114,12 +115,14 @@ def test_render_cli_produces_output(tmp_path):
     manifest_path = _write_manifest(tmp_path, dets, 200, 200)
 
     out_dir = tmp_path / "out"
-    render_main([
-        str(img_path),
-        "--manifest",
-        str(manifest_path),
-        "-o",
-        str(out_dir),
-    ])
+    render_main(
+        [
+            str(img_path),
+            "--manifest",
+            str(manifest_path),
+            "-o",
+            str(out_dir),
+        ]
+    )
 
     assert (out_dir / "test-uitag.png").exists()

tests/test_rescan.py

@@ -109,6 +109,7 @@ def test_run_pipeline_accepts_rescan_params():
     from uitag.run import run_pipeline
 
     import inspect
+
     sig = inspect.signature(run_pipeline)
     assert "rescan" in sig.parameters
     assert "rescan_threshold" in sig.parameters

uitag/batch_cli.py

@@ -135,7 +135,9 @@ def batch_main(argv: list[str] | None = None) -> None:
 
     # Warm the backend import before starting timer
     info = backend.info()
-    source_label = args.path[0].rstrip("/") if len(args.path) == 1 else f"{len(args.path)} paths"
+    source_label = (
+        args.path[0].rstrip("/") if len(args.path) == 1 else f"{len(args.path)} paths"
+    )
     print(f"Running pipeline on: {len(image_paths)} images in {source_label}/")
     print(f"Backend: {info.name} ({info.device}) | OCR mode: {ocr_mode}\n")