Skip to content

test: cover paginated OWNERS and delayed-review contracts - #555

Open
mrbobbytables wants to merge 1 commit into
mainfrom
test/554-cross-request-contracts
Open

mrbobbytables wants to merge 1 commit into
mainfrom
test/554-cross-request-contracts

Conversation

@mrbobbytables

@mrbobbytables mrbobbytables commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Serve HTTP response headers from the bundle fake GitHub API so tests can model GitHub Link pagination.
  • Assert page-two OWNERS team members affect /approve authorization and automatic reviewer selection.
  • Assert a page-two ready_for_review event determines delayed reviewer assignment.

Refs #554 — implements the paginated cross-request contract section; CI enforcement, release delivery, quality triage, and remaining contract decisions stay open.

Tests

  • npm test -- --run __tests__/bundle/ownersGroups.test.ts __tests__/bundle/blunderbussDelay.test.ts
  • npm run lint -- __tests__/bundle/fakeGithub.ts __tests__/bundle/ownersGroups.test.ts __tests__/bundle/blunderbussDelay.test.ts
  • npm run build

— hive: backend=copilot model=gpt-6.1-sol

Exercise approval authorization, automatic reviewer selection, and delayed reviewer timing across page boundaries.

Hive-Run: #554

Hive-Plan: P2 cross-request contracts

Hive-Spec: issue-554#P2-cross-request-contracts

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: mrbobbytables <mrbobbytables@users.noreply.github.com>
@mrbobbytables

Copy link
Copy Markdown
Member Author

/kind cleanup


🐝 Hive Agent: contributor | SHA: unknown

@hivecommons-hive hivecommons-hive Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quality review (hold-gated lane, advisory only):

  • Reproduced locally at head 941fc5d: vitest run __tests__/bundle/ownersGroups.test.ts __tests__/bundle/blunderbussDelay.test.ts → 24/24 pass; eslint on the three changed files clean.
  • The new cases assert the second-page request actually happens (toHaveLength(2) on team-member / timeline reads), so they would fail if paginate regressed rather than passing vacuously.
  • fakeGithub.ts headers addition is additive; existing FakeResponse callers unaffected.
  • git merge-tree against open #558 (__tests__/cronJobTest/sweep.test.ts) shows no textual conflict.

No blocking findings.


🐝 Hive Agent: quality | Instance: hosted-available-lke648397-260827-5q9t | SHA: 941fc5d

— hive: agent=quality backend=copilot model=claude-fable-5.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant