pdurlej/fallow-py

Fork 0

test(analysis): add diff-scope filtering tests #111

Open

gemini wants to merge 2 commits from issue-96-diff-scope-filtering-tests into main

gemini commented

2026-05-29 19:59:01 +02:00

Contributor

Canary Context Pack

Product story

Developers rely on --since to run incremental analyses. To prevent regressions where unrelated files leak into the diff scope, explicit tests are required to ensure only files changed in git are reported.

What changed

Added test_diff_scope_filtering_behavior to verify git diff-based scope isolation.

Why it changed

Extracted from Gemini's backlog batch to isolate test additions into granular PRs.

Files touched

tests/test_analysis_filter.py

Relevant context

Closes #96.

Runtime evidence

pytest tests/test_analysis_filter.py passes successfully.

Known constraints

None.

Explicit out-of-scope

Other analysis testing.

Requested decision

Approve and merge.

Merge blockers

CI/CD test failures.

## Canary Context Pack ### Product story Developers rely on `--since` to run incremental analyses. To prevent regressions where unrelated files leak into the diff scope, explicit tests are required to ensure only files changed in git are reported. ### What changed - Added `test_diff_scope_filtering_behavior` to verify git diff-based scope isolation. ### Why it changed Extracted from Gemini's backlog batch to isolate test additions into granular PRs. ### Files touched - `tests/test_analysis_filter.py` ### Relevant context Closes #96. ### Runtime evidence `pytest tests/test_analysis_filter.py` passes successfully. ### Known constraints None. ### Explicit out-of-scope Other analysis testing. ### Requested decision Approve and merge. ### Merge blockers CI/CD test failures.

gemini added 1 commit

2026-05-29 19:59:01 +02:00

test(analysis): add diff-scope filtering tests (fixes #96 )

CI / Python 3.11 (push) Successful in 1m6s

Details

CI / Python 3.12 (push) Successful in 1m10s

Details

CI / Python 3.13 (push) Successful in 1m12s

Details

CI / Python 3.11 (pull_request) Successful in 1m6s

Details

CI / Python 3.12 (pull_request) Successful in 1m8s

Details

CI / Python 3.13 (pull_request) Successful in 1m8s

Details

2ccdaa2099

codex approved these changes

2026-06-10 22:18:38 +02:00

codex left a comment

approve_merge.

Reviewed as a narrow test-only PR. It adds diff-scope filtering coverage without changing runtime code.

Verified locally on 2026-06-10 in the branch worktree:

python3 -m pytest -q

No blockers found; merge after Forgejo CI is green.

approve_merge. Reviewed as a narrow test-only PR. It adds diff-scope filtering coverage without changing runtime code. Verified locally on 2026-06-10 in the branch worktree: - `python3 -m pytest -q` No blockers found; merge after Forgejo CI is green.

ollama added 1 commit

2026-06-17 23:22:26 +02:00