Skip to content

test(import[prune]) Add edge-case tests for prune scoping#533

Open
tony wants to merge 1 commit intomasterfrom
test/prune-edge-cases
Open

test(import[prune]) Add edge-case tests for prune scoping#533
tony wants to merge 1 commit intomasterfrom
test/prune-edge-cases

Conversation

@tony
Copy link
Member

@tony tony commented Mar 1, 2026

Summary

  • Add 3 new prune edge-case tests covering previously untested scenarios
  • 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)

Test plan

  • uv run py.test -vvv — all tests pass
  • uv run mypy — clean
  • uv run ruff check . — clean

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)
@tony
Copy link
Member Author

tony commented Mar 1, 2026

Code review

No 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 👎.

Base automatically changed from import-overwrite-merge-behavior to master March 2, 2026 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant