test_freeze_review_cycle.py
python
sha256:0e9549ec7b463911bc08b7d586dc320b1ac9b1f5c943ee7e3865dcc6cb0f6f83
chore(governance): sync handover+roadmap to 84db8c8 (drift:…
Human
9 hours ago
| 1 | """End-to-end freeze review lifecycle (§K5.12).""" |
| 2 | |
| 3 | from __future__ import annotations |
| 4 | |
| 5 | from pathlib import Path |
| 6 | |
| 7 | from cli.kit_root import kit_root |
| 8 | from tests.support import ( |
| 9 | findings_provider_factory, |
| 10 | git_status_runner, |
| 11 | pass_provider_factory, |
| 12 | run_cli, |
| 13 | seed_freeze_repo, |
| 14 | ) |
| 15 | from tools.freeze_reviewer.types import Finding |
| 16 | |
| 17 | |
| 18 | def test_full_freeze_review_cycle(tmp_path: Path) -> None: |
| 19 | artifact = seed_freeze_repo(tmp_path) |
| 20 | rel = artifact.relative_to(tmp_path).as_posix() |
| 21 | runner = git_status_runner() |
| 22 | |
| 23 | assert ( |
| 24 | run_cli( |
| 25 | ["review", "--freeze", rel, "--dry-run"], |
| 26 | cwd=tmp_path, |
| 27 | runner=runner, |
| 28 | kit=kit_root(), |
| 29 | review_provider_factory=pass_provider_factory(), |
| 30 | ) |
| 31 | == 0 |
| 32 | ) |
| 33 | assert "review_stamp:" not in artifact.read_text(encoding="utf-8") |
| 34 | |
| 35 | assert ( |
| 36 | run_cli( |
| 37 | ["review", "--freeze", rel], |
| 38 | cwd=tmp_path, |
| 39 | runner=runner, |
| 40 | kit=kit_root(), |
| 41 | review_provider_factory=pass_provider_factory(), |
| 42 | ) |
| 43 | == 0 |
| 44 | ) |
| 45 | stamped = artifact.read_text(encoding="utf-8") |
| 46 | assert "review_stamp:" in stamped |
| 47 | |
| 48 | assert ( |
| 49 | run_cli( |
| 50 | ["review", "--freeze", rel], |
| 51 | cwd=tmp_path, |
| 52 | runner=runner, |
| 53 | kit=kit_root(), |
| 54 | review_provider_factory=pass_provider_factory(), |
| 55 | ) |
| 56 | == 0 |
| 57 | ) |
| 58 | assert artifact.read_text(encoding="utf-8") == stamped |
| 59 | |
| 60 | artifact.write_text(stamped + "\n# edited\n", encoding="utf-8") |
| 61 | findings = [ |
| 62 | Finding( |
| 63 | check="C3", |
| 64 | severity="MAJOR", |
| 65 | category="consistency", |
| 66 | path=rel, |
| 67 | line=1, |
| 68 | message="edited", |
| 69 | ).with_citation() |
| 70 | ] |
| 71 | code = run_cli( |
| 72 | ["review", "--freeze", rel], |
| 73 | cwd=tmp_path, |
| 74 | runner=runner, |
| 75 | kit=kit_root(), |
| 76 | review_provider_factory=findings_provider_factory(findings), |
| 77 | ) |
| 78 | assert code == 7 |
| 79 | assert all("commit" not in call[0].lower() for call in runner.calls) |
File History
1 commit
sha256:6abcf1fa82a7a621ccbc945f19acdba5bc0db54569599404a1452fb4a096a199
fix(ISR): default require_independent_second_reviewer to require
Human
minor
⚠
8 hours ago