Skip to content

test: cover Jules GitHub source resolution#38

Draft
cursor[bot] wants to merge 3 commits into
mainfrom
cursor/missing-test-coverage-2cd7
Draft

test: cover Jules GitHub source resolution#38
cursor[bot] wants to merge 3 commits into
mainfrom
cursor/missing-test-coverage-2cd7

Conversation

@cursor

@cursor cursor Bot commented Jul 19, 2026

Copy link
Copy Markdown

Summary

  • add deterministic coverage for realJulesPort.createSession GitHub source resolution
  • verify owner/repository matching is case-insensitive and requires both components
  • verify default-branch fallback, explicit branch precedence, and rejection of unregistered repositories

Risky behavior now covered

Jules dispatch resolves an API source and branch before creating a remote agent session. A regression can dispatch work to the wrong repository or branch, or attempt session creation for an unregistered source.

Test files added/updated

  • test/jules-port.test.ts

Why this reduces regression risk

The tests exercise the real port request flow with deterministic mocked HTTP responses, including decoy sources that catch owner-only or repository-only matching. Mutation checks confirmed the suite fails if case normalization, branch precedence, or exact owner-and-repository matching is broken.

Validation

  • npm test — 38 tests passed
  • npm run build — passed
  • npx prettier --check test/jules-port.test.ts — passed
Open in Web View Automation 

cursoragent and others added 3 commits July 19, 2026 10:04
Co-authored-by: Parvez Kose <koseparvez@gmail.com>
Co-authored-by: Parvez Kose <koseparvez@gmail.com>
Co-authored-by: Parvez Kose <koseparvez@gmail.com>
@vercel

vercel Bot commented Jul 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
polyagent Ready Ready Preview, Comment Jul 19, 2026 10:08am

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.

1 participant