test: add CLI-level tests for remove --category, stats nonexistent dir, and --ignore option (#30)

Coding-Dev-Tools · web-flow · commit a1a548f0ec30 · 2026-05-21T16:41:27.000-04:00
Sentinel-approved: 2 files, 36 additions. CLI guard for nonexistent dirs + 4 focused tests.
diff --git a/src/deadcode/cli.py b/src/deadcode/cli.py
@@ -171,6 +171,10 @@ def remove(ctx: click.Context, dry_run: bool, category: str | None) -> None:
     project = ctx.obj["project"]
     ignore = _merge_config_ignore(ctx)
 
+    if not Path(project).exists():
+        err_console.print(f"[red]Project directory '{project}' not found.[/red]")
+        sys.exit(1)
+
     if not dry_run:
         console.print("[red]WARNING: This will modify files. Use --dry-run first![/red]")
         console.print("[dim]Press Ctrl+C to abort. Running in 3 seconds...[/dim]")
diff --git a/tests/test_scanner.py b/tests/test_scanner.py
@@ -293,6 +293,38 @@ def test_stats_command(self, runner, sample_project):
         assert "Files scanned" in result.output
         assert "Unused exports" in result.output
 
+    def test_scan_ignore_option(self, runner, tmp_path):
+        """--ignore option should exclude matching files from scan."""
+        mod = tmp_path / "src" / "mod.ts"
+        mod.parent.mkdir(parents=True, exist_ok=True)
+        mod.write_text('export function unusedFunc() { return 1; }\n')
+
+        result = runner.invoke(cli, ["-p", str(tmp_path), "-i", "src/", "scan", "--json-output"])
+        assert result.exit_code == 0
+        import json
+        data = json.loads(result.output, strict=False)
+        assert data["files_scanned"] == 0, "src/ ignored, should have 0 files"
+
+    def test_remove_category_filter(self, runner, sample_project):
+        """remove --dry-run --category should filter by category."""
+        result = runner.invoke(cli, ["-p", str(sample_project), "remove", "--dry-run", "-c", "orphaned_css"])
+        assert result.exit_code == 0
+        # Should mention orphaned class
+        assert "orphaned-class" in result.output or "Nothing removable" in result.output
+
+    def test_remove_nonexistent_dir(self, runner):
+        """remove should give graceful error for nonexistent project dir."""
+        result = runner.invoke(cli, ["-p", "/nonexistent/test/path", "remove", "--dry-run"])
+        assert result.exit_code != 0
+        assert "not found" in result.output
+
+    def test_stats_nonexistent_dir(self, runner):
+        """stats should handle nonexistent project dir gracefully."""
+        result = runner.invoke(cli, ["-p", "/nonexistent/stats/path", "stats"])
+        # Should not crash — scan returns 0 files for nonexistent dir
+        assert result.exit_code == 0
+        assert "Files scanned: 0" in result.output
+
     def test_main_module_entry_point(self, runner):
         """Test that python -m deadcode works (__main__ entry point fix)."""
         import subprocess
