Summary
Several PR-related skill scripts lack test coverage. This issue tracks adding Pester tests for the Get-PR* family of scripts.
Scripts Without Tests
| Script |
Purpose |
Get-PRContext.ps1 |
Get comprehensive PR metadata |
Get-PRChecks.ps1 |
Get CI check status for PR |
Get-PRReviewThreads.ps1 |
Get review threads with resolution status |
Get-PRReviewComments.ps1 |
Get review comments |
Get-PRReviewers.ps1 |
Get PR reviewers |
Get-UnresolvedReviewThreads.ps1 |
Filter to unresolved threads only |
Get-UnaddressedComments.ps1 |
Get comments without replies |
Test Pattern
Follow existing test patterns in tests/:
- Mock
gh api calls with fixture data
- Test GraphQL query construction
- Validate output object shape
- Test error handling paths
Acceptance Criteria
Related
- Existing test examples:
Test-PRMerged.Tests.ps1, Detect-CopilotFollowUpPR.Tests.ps1
- Location:
.claude/skills/github/scripts/pr/
Summary
Several PR-related skill scripts lack test coverage. This issue tracks adding Pester tests for the Get-PR* family of scripts.
Scripts Without Tests
Get-PRContext.ps1Get-PRChecks.ps1Get-PRReviewThreads.ps1Get-PRReviewComments.ps1Get-PRReviewers.ps1Get-UnresolvedReviewThreads.ps1Get-UnaddressedComments.ps1Test Pattern
Follow existing test patterns in
tests/:gh apicalls with fixture dataAcceptance Criteria
.Tests.ps1fileRelated
Test-PRMerged.Tests.ps1,Detect-CopilotFollowUpPR.Tests.ps1.claude/skills/github/scripts/pr/