diff --git a/scripts/reviewers/copilot.json b/scripts/reviewers/copilot.json index 34bcebac..56abc331 100644 --- a/scripts/reviewers/copilot.json +++ b/scripts/reviewers/copilot.json @@ -42,8 +42,14 @@ ] }, "closed_badges": [], - "labels": ["changes:", "review findings:", "findings:", "open findings remain:"], - "finding_lists": ["review findings:", "open findings remain:"], + "labels": [ + "changes:", + "review findings:", + "findings:", + "open findings remain:", + "unresolved findings:" + ], + "finding_lists": ["review findings:", "open findings remain:", "unresolved findings:"], "effort": "review effort(?: level)?:? *(?:default *\\( *(?Plite|balanced|max) *\\)|(?Plite|balanced|max))", "table_header": "file", "discussion_link": "#discussion_r(?P\\d+)$" diff --git a/tests/test_pr_review_copilot.py b/tests/test_pr_review_copilot.py index b927dcad..ed18e622 100755 --- a/tests/test_pr_review_copilot.py +++ b/tests/test_pr_review_copilot.py @@ -330,6 +330,18 @@ def test_a_findings_list_longer_than_the_threads_is_unthreaded(self) -> None: ) self.assertEqual(1, found.unthreaded) + def test_each_top_level_findings_list_spelling_is_placed_and_counted(self) -> None: + for label in ("Review findings:", "Open findings remain:", "Unresolved findings:"): + with self.subTest(label=label): + listed = f"**{label}**\n- P1 moderate: one.\n- P2 nit: two.\n- P2 nit: three." + found = adapter.read( + body(sections=f"{open_section()}\n\n{listed}\n\n{what_changed()}"), + threads=("101", "102"), + ) + self.assertEqual(1, found.unthreaded) + self.assertEqual((), found.notes) + self.assertEqual((), found.unplaced) + def test_a_nested_back_reference_is_not_an_entry(self) -> None: nested = open_section().replace( "\u00b7 New\n- ", "\u00b7 New\n - see [earlier](#discussion_r999)\n- ", 1