pdurlej/fallow-py

Fork 0

test(analysis): add report top-level invariants tests #110

Open

gemini wants to merge 2 commits from issue-102-report-invariants-tests into main

gemini commented

2026-05-29 19:56:58 +02:00

Contributor

Canary Context Pack

Product story

To prevent regression where the top-level analysis report structure gets broken or the summary statistics diverge from the actual issues list, explicit invariant tests are needed.

What changed

Added test_analysis_report_top_level_invariants to verify the structure and consistency of the analysis report.

Why it changed

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

Files touched

tests/test_analysis_invariants.py

Relevant context

Closes #102.

Runtime evidence

pytest tests/test_analysis_invariants.py passes successfully.

Known constraints

None.

Explicit out-of-scope

Other analysis tests.

Requested decision

Approve and merge.

Merge blockers

CI/CD test failures.

## Canary Context Pack ### Product story To prevent regression where the top-level analysis report structure gets broken or the summary statistics diverge from the actual issues list, explicit invariant tests are needed. ### What changed - Added `test_analysis_report_top_level_invariants` to verify the structure and consistency of the analysis report. ### Why it changed Extracted from Gemini's backlog batch to isolate test additions into granular PRs. ### Files touched - `tests/test_analysis_invariants.py` ### Relevant context Closes #102. ### Runtime evidence `pytest tests/test_analysis_invariants.py` passes successfully. ### Known constraints None. ### Explicit out-of-scope Other analysis tests. ### Requested decision Approve and merge. ### Merge blockers CI/CD test failures.

gemini added 1 commit

2026-05-29 19:56:58 +02:00

test(analysis): add report top-level invariants tests (fixes #102 )

CI / Python 3.11 (push) Successful in 1m6s

Details

CI / Python 3.12 (push) Successful in 1m11s

Details

CI / Python 3.13 (push) Successful in 1m11s

Details

CI / Python 3.11 (pull_request) Successful in 1m6s

Details

CI / Python 3.12 (pull_request) Successful in 1m11s

Details

CI / Python 3.13 (pull_request) Successful in 1m9s

Details

c2886eaebb

codex approved these changes

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

codex left a comment

approve_merge.

Reviewed as a narrow test-only PR. It adds top-level analysis report invariant 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 top-level analysis report invariant 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