mirror of
https://github.com/ChrisTitusTech/winutil.git
synced 2026-08-24 08:26:32 +00:00
Ivan LepekhaandChris Titus
032ce8ead0
* feat(github): revival of triage tools * fix(github): workflow doc fix * fix(github): PR detection * fix(gh): wontfix preserve bug label Removed state_reason from issue update when adding wontfix label. * fix(gh): doc cleaning * one more thing * fix(gh): prevent Gabi moment That's the only thing I will agree with this piece of hardware * Fix syntax error in triage-tools.yaml * fix(github): restrict triage commands to issues --------- Co-authored-by: Chris Titus <contact@christitus.com>
69 lines
3.9 KiB
PowerShell
69 lines
3.9 KiB
PowerShell
#===========================================================================
|
|
# Tests - GitHub Issue Triage Workflow
|
|
#===========================================================================
|
|
|
|
BeforeAll {
|
|
$script:repoRoot = (Resolve-Path (Join-Path $PSScriptRoot "..")).Path
|
|
$script:workflowPath = Join-Path $script:repoRoot ".github\workflows\triage-tools.yaml"
|
|
$script:guidePath = Join-Path $script:repoRoot "docs\src\content\docs\code-reference\issue-triage.mdx"
|
|
$script:workflow = Get-Content -Path $script:workflowPath -Raw
|
|
$script:guide = Get-Content -Path $script:guidePath -Raw
|
|
}
|
|
|
|
Describe "GitHub issue triage workflow" {
|
|
It "cannot run against pull requests" {
|
|
$script:workflow | Should -Match '(?m)^\s+if:\s+\$\{\{\s*!github\.event\.issue\.pull_request\s*\}\}\s*$'
|
|
$script:workflow | Should -Match 'if \(context\.payload\.issue\.pull_request\)'
|
|
$script:workflow | Should -Match '(?m)^\s+issues:\s+write\s*$'
|
|
$script:workflow | Should -Match '(?m)^\s+pull-requests:\s+none\s*$'
|
|
$script:workflow | Should -Match '(?m)^\s+contents:\s+none\s*$'
|
|
$script:workflow | Should -Not -Match '(?m)^\s+pull-requests:\s+write\s*$'
|
|
}
|
|
|
|
It "authorizes trusted users by immutable numeric ID" {
|
|
$script:workflow | Should -Match 'trustedUsers\.includes\(commentAuthor\.id\)'
|
|
$script:workflow | Should -Not -Match 'trustedUsers\.includes\(commentAuthor\.login\)'
|
|
}
|
|
|
|
It "matches one complete command at a time" {
|
|
$script:workflow.Contains('const command = comment.body.trim().toLowerCase();') | Should -BeTrue
|
|
$script:workflow.Contains('const triageMatch = command.match(/^\/triage$/);') | Should -BeTrue
|
|
$script:workflow.Contains('const triageOffMatch = command.match(/^\/triageoff$/);') | Should -BeTrue
|
|
$script:workflow.Contains('const notPlannedMatch = command.match(/^\/np(?:\s+(\w+))?$/);') | Should -BeTrue
|
|
$script:workflow.Contains('const duplicateMatch = command.match(/^\/duplicate\s+#?(\d+)$/);') | Should -BeTrue
|
|
$script:workflow.Contains('} else if (triageOffMatch) {') | Should -BeTrue
|
|
$script:workflow.Contains('} else if (notPlannedMatch) {') | Should -BeTrue
|
|
$script:workflow.Contains('} else if (duplicateMatch) {') | Should -BeTrue
|
|
}
|
|
|
|
It "preserves existing labels for not-related closures" {
|
|
$notRelatedBlock = [regex]::Match(
|
|
$script:workflow,
|
|
'(?s)reason === "notrelated".*?github\.rest\.issues\.addLabels\(\{.*?labels: \["not-related"\].*?github\.rest\.issues\.update'
|
|
)
|
|
|
|
$notRelatedBlock.Success | Should -BeTrue
|
|
$notRelatedBlock.Value | Should -Not -Match 'issues\.update\(\{.*?labels:'
|
|
}
|
|
|
|
It "validates duplicate targets before closing the issue" {
|
|
$script:workflow.Contains('const duplicateIssueNumber = Number(duplicateMatch[1]);') | Should -BeTrue
|
|
$script:workflow | Should -Match 'Number\.isSafeInteger\(duplicateIssueNumber\)'
|
|
$script:workflow | Should -Match 'duplicateIssueNumber === issueNumber'
|
|
$script:workflow | Should -Match 'duplicateIssue = response\.data'
|
|
$script:workflow | Should -Match 'duplicateIssue\?\.pull_request'
|
|
$script:workflow | Should -Match 'error\.status !== 404'
|
|
$script:workflow | Should -Match 'throw error'
|
|
$script:workflow | Should -Match 'duplicate_issue_id: duplicateIssue\.id'
|
|
}
|
|
|
|
It "keeps the maintainer guide in the documentation site" {
|
|
Test-Path -Path (Join-Path $script:repoRoot ".github\TRIAGE_TOOLS.md") | Should -BeFalse
|
|
$script:guide | Should -Match 'These commands work only on issues\.'
|
|
$script:guide | Should -Match '`not-related` label without replacing existing labels'
|
|
|
|
$astroConfig = Get-Content -Path (Join-Path $script:repoRoot "docs\astro.config.mjs") -Raw
|
|
$astroConfig | Should -Match "slug: 'code-reference/issue-triage'"
|
|
}
|
|
}
|