From fca4c41d959b39f01a9bed6949e9419c448500db Mon Sep 17 00:00:00 2001 From: Don Garrett Date: Sat, 17 Oct 2026 07:06:10 +0000 Subject: [PATCH] Later review test: its earlier summary is in the brief too From nisse-review's draft on #156: it thought points made in a review's summary never reached a later review. They do, as "your review: ...". Co-Authored-By: Claude Opus 5.5 (1M context) Claude-Session: https://claude.ai/code/session_01J5Utyfzw7gF7UhjRDEtjTk --- nisse-review/tests/test_review_rounds.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/nisse-review/tests/test_review_rounds.py b/nisse-review/tests/test_review_rounds.py index 2c09c2f..4e6f870 100644 --- a/nisse-review/tests/test_review_rounds.py +++ b/nisse-review/tests/test_review_rounds.py @@ -120,7 +120,8 @@ def test_a_push_gets_a_later_review_of_what_changed(tmp_path: Path) -> None: assert w["scheduler"].round() is True brief = last_brief(w) assert "The commits since" in brief and f"diff of {PUSHED}" in brief - assert "split() handles tabs too." in brief # its earlier point + assert "split() handles tabs too." in brief # its earlier point on a line + assert "- your review: Looks right; two small points." in brief # its summary review = w["gitea"].posted[-1] assert review["commit"] == PUSHED and marker(PUSHED, 2) in review["body"] known = pr_of(item(w))