|
| 1 | +import {describe, it, expect, vi, beforeEach, afterEach} from 'vitest' |
| 2 | + |
| 3 | +const openIssue = vi.fn() |
| 4 | +const reopenIssue = vi.fn() |
| 5 | +const closeIssue = vi.fn() |
| 6 | +vi.mock('../src/openIssue.js', () => ({openIssue: (...args: unknown[]) => openIssue(...args)})) |
| 7 | +vi.mock('../src/reopenIssue.js', () => ({reopenIssue: (...args: unknown[]) => reopenIssue(...args)})) |
| 8 | +vi.mock('../src/closeIssue.js', () => ({closeIssue: (...args: unknown[]) => closeIssue(...args)})) |
| 9 | + |
| 10 | +const inputs: Record<string, string> = {} |
| 11 | +const infoLines: string[] = [] |
| 12 | +const warnLines: string[] = [] |
| 13 | +const outputs: Record<string, string> = {} |
| 14 | +vi.mock('@actions/core', () => ({ |
| 15 | + getInput: (name: string) => inputs[name] ?? '', |
| 16 | + getBooleanInput: (name: string) => (inputs[name] ?? 'false') === 'true', |
| 17 | + setOutput: (name: string, value: string) => { |
| 18 | + outputs[name] = value |
| 19 | + }, |
| 20 | + info: (msg: string) => { |
| 21 | + infoLines.push(msg) |
| 22 | + }, |
| 23 | + debug: () => {}, |
| 24 | + warning: (msg: string) => { |
| 25 | + warnLines.push(msg) |
| 26 | + }, |
| 27 | + setFailed: () => {}, |
| 28 | +})) |
| 29 | + |
| 30 | +// Feed findings/cached filings in |
| 31 | +const files: Record<string, string> = {} |
| 32 | +vi.mock('node:fs', () => ({ |
| 33 | + default: { |
| 34 | + readFileSync: (p: string) => files[p], |
| 35 | + writeFileSync: (p: string, data: string) => { |
| 36 | + files[p] = data |
| 37 | + }, |
| 38 | + }, |
| 39 | +})) |
| 40 | + |
| 41 | +// Stub Octokit: `request` serves the list of closed `wontfix` issues that |
| 42 | +// getWontfixIssueNumbers fetches once up front. |
| 43 | +const octokitRequest = vi.fn() |
| 44 | +vi.mock('@octokit/core', () => ({ |
| 45 | + Octokit: { |
| 46 | + plugin: () => |
| 47 | + class { |
| 48 | + request = octokitRequest |
| 49 | + }, |
| 50 | + }, |
| 51 | +})) |
| 52 | +vi.mock('@octokit/plugin-throttling', () => ({throttling: {}})) |
| 53 | + |
| 54 | +import runFileAction from '../src/index.ts' |
| 55 | + |
| 56 | +const wontfixFinding = { |
| 57 | + scannerType: 'axe', |
| 58 | + ruleId: 'color-contrast', |
| 59 | + url: 'https://example.com/page', |
| 60 | + html: '<span>Low contrast</span>', |
| 61 | + problemShort: 'elements must meet minimum color contrast ratio thresholds', |
| 62 | + problemUrl: 'https://dequeuniversity.com/rules/axe/4.10/color-contrast', |
| 63 | + solutionShort: 'ensure sufficient contrast', |
| 64 | +} |
| 65 | +const normalFinding = {...wontfixFinding, ruleId: 'heading-order', html: '<h3>Skipped</h3>'} |
| 66 | + |
| 67 | +// Both cached filings' findings reappear this run, so both are repeated |
| 68 | +const wontfixCached = { |
| 69 | + issue: {id: 1, nodeId: 'N1', url: 'https://github.com/org/repo/issues/1', title: 'wontfix'}, |
| 70 | + findings: [wontfixFinding], |
| 71 | +} |
| 72 | +const normalCached = { |
| 73 | + issue: {id: 3, nodeId: 'N3', url: 'https://github.com/org/repo/issues/3', title: 'normal'}, |
| 74 | + findings: [normalFinding], |
| 75 | +} |
| 76 | + |
| 77 | +function setup() { |
| 78 | + files['/tmp/findings.json'] = JSON.stringify([wontfixFinding, normalFinding]) |
| 79 | + files['/tmp/cached.json'] = JSON.stringify([wontfixCached, normalCached]) |
| 80 | + inputs.findings_file = '/tmp/findings.json' |
| 81 | + inputs.cached_filings_file = '/tmp/cached.json' |
| 82 | + inputs.repository = 'org/repo' |
| 83 | + inputs.token = 'fake-token' |
| 84 | + // Single up-front fetch: only issue 1 is closed and labeled wontfix |
| 85 | + octokitRequest.mockImplementation((route: string) => |
| 86 | + route.includes('GET /repos/org/repo/issues') ? Promise.resolve({data: [{number: 1}]}) : Promise.resolve({data: {}}), |
| 87 | + ) |
| 88 | +} |
| 89 | + |
| 90 | +describe('file action — wontfix label', () => { |
| 91 | + beforeEach(() => { |
| 92 | + vi.clearAllMocks() |
| 93 | + infoLines.length = 0 |
| 94 | + warnLines.length = 0 |
| 95 | + for (const k of Object.keys(inputs)) delete inputs[k] |
| 96 | + for (const k of Object.keys(outputs)) delete outputs[k] |
| 97 | + }) |
| 98 | + afterEach(() => { |
| 99 | + vi.restoreAllMocks() |
| 100 | + }) |
| 101 | + |
| 102 | + it('reopens the unlabeled issue but not the one labeled wontfix', async () => { |
| 103 | + setup() |
| 104 | + |
| 105 | + await runFileAction() |
| 106 | + |
| 107 | + expect(reopenIssue).toHaveBeenCalledTimes(1) |
| 108 | + const reopenedIssue = reopenIssue.mock.calls[0][1] as {url: string} |
| 109 | + expect(reopenedIssue.url).toBe('https://github.com/org/repo/issues/3') |
| 110 | + expect(openIssue).not.toHaveBeenCalled() |
| 111 | + expect(closeIssue).not.toHaveBeenCalled() |
| 112 | + }) |
| 113 | + |
| 114 | + it('logs that it skipped the wontfix issue', async () => { |
| 115 | + setup() |
| 116 | + |
| 117 | + await runFileAction() |
| 118 | + |
| 119 | + expect(infoLines.join('\n')).toContain( |
| 120 | + "Skipping reopen of issue labeled 'wontfix': https://github.com/org/repo/issues/1", |
| 121 | + ) |
| 122 | + }) |
| 123 | + |
| 124 | + it('reopens as usual (and warns) when the label check fails', async () => { |
| 125 | + setup() |
| 126 | + // The up-front wontfix fetch fails (e.g. transient API error) |
| 127 | + octokitRequest.mockRejectedValue(new Error('boom')) |
| 128 | + |
| 129 | + await runFileAction() |
| 130 | + |
| 131 | + // Both repeated filings should still be reopened rather than aborting the run |
| 132 | + expect(reopenIssue).toHaveBeenCalledTimes(2) |
| 133 | + expect(warnLines.join('\n')).toContain("Could not fetch 'wontfix' issues") |
| 134 | + }) |
| 135 | +}) |
0 commit comments