Skip to content

Decouple PR review pagination tests from summary wording - #1904

Draft
v1v with Copilot wants to merge 3 commits into
mainfrom
copilot/autonomy-atomicity-fix-paginated-test
Draft

Decouple PR review pagination tests from summary wording#1904
v1v with Copilot wants to merge 3 commits into
mainfrom
copilot/autonomy-atomicity-fix-paginated-test

Conversation

Copilot AI commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

The pagination test was coupled to exact summary phrasing, allowing unrelated presentation changes to break data-path coverage.

  • Pagination coverage
    • Validate both paginated threads and exact per-thread comment counts using full-detail output.
    • Confirm paginated comments (Comment 51) and thread-specific data are retained.
  • Summary contract
    • Move wording assertions into a dedicated summary-format test.
assert example_output.count("@reviewer:") == 51
assert another_output.count("@reviewer:") == 1

Copilot AI and others added 2 commits August 25, 2026 13:47
Co-authored-by: v1v <2871786+v1v@users.noreply.github.com>
Co-authored-by: v1v <2871786+v1v@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix pagination test by decoupling from summary wording Decouple PR review pagination tests from summary wording Aug 25, 2026
Copilot finished work on behalf of v1v August 25, 2026 13:48
Copilot AI requested a review from v1v August 25, 2026 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[autonomy-atomicity] PR review pagination test couples data-path verification to summary wording

2 participants