test(import[prune]) Add edge-case tests for prune scoping#533
Conversation
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. 🤖 Generated with Claude Code - If this code review was useful, please react with 👍. Otherwise, react with 👎. |
why: Three prune edge cases had no test coverage. what: - Test --prune without import_source does nothing (Finding 13) - Test --sync --prune together is idempotent with --sync alone (Finding 14) - Test same-name same-source repos across workspaces prune correctly (Finding 1)
|
Rebased onto current Confirmed with the maintainer that |
67c5175 to
510c23e
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #533 +/- ##
=======================================
Coverage 83.30% 83.30%
=======================================
Files 32 32
Lines 4492 4492
Branches 903 903
=======================================
Hits 3742 3742
Misses 499 499
Partials 251 251 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Summary
--prunewithoutimport_sourcedoes nothing (Finding 13)--sync --prunetogether is idempotent with--syncalone (Finding 14)Test plan
uv run py.test -vvv— all tests passuv run mypy— cleanuv run ruff check .— clean