CodeIssuesDiscussionsWikiPull RequestsProjectsCommitsActionsReleasesContributorsPulse● GatesSecuritySettingsDeploymentsPipelineInsightsAgents✨ Explain✨ Ask AI✨ Workspace✨ Spec✨ Tests▓ Debt Map✨ NL Search🏛 Archaeology
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 | /**
* Tests for src/lib/issue-triage.ts.
*
* Pure helper renderIssueTriageComment covered exhaustively. The
* DB-touching paths (alreadyTriaged, loadAvailableLabels,
* loadRecentIssues) are exercised via the fail-open contracts.
*
* triggerIssueTriage itself is verified to be a no-throw fire-and-forget
* even when called with garbage inputs and no API key.
*/
import { describe, it, expect } from "bun:test";
import {
ISSUE_TRIAGE_MARKER,
triggerIssueTriage,
__test,
} from "../lib/issue-triage";
import type { IssueTriage } from "../lib/ai-generators";
const triageFixture = (overrides: Partial<IssueTriage> = {}): IssueTriage => ({
suggestedLabels: ["bug", "ui"],
duplicateOfIssueNumber: null,
priority: "medium",
summary: "Login button is unreadable in dark mode.",
...overrides,
});
describe("ISSUE_TRIAGE_MARKER", () => {
it("is a stable HTML comment so future searches keep working", () => {
expect(ISSUE_TRIAGE_MARKER).toBe("<!-- gluecron-issue-triage:summary -->");
expect(ISSUE_TRIAGE_MARKER.startsWith("<!--")).toBe(true);
expect(ISSUE_TRIAGE_MARKER.endsWith("-->")).toBe(true);
});
});
describe("renderIssueTriageComment", () => {
it("includes the marker, summary, priority, labels", () => {
const out = __test.renderIssueTriageComment(triageFixture());
expect(out).toContain(ISSUE_TRIAGE_MARKER);
expect(out).toContain("## AI Triage");
expect(out).toContain("Login button is unreadable in dark mode.");
expect(out).toContain("**Priority:** medium");
expect(out).toContain("`bug`");
expect(out).toContain("`ui`");
});
it("uses an italic placeholder when there are no label suggestions", () => {
const out = __test.renderIssueTriageComment(
triageFixture({ suggestedLabels: [] })
);
expect(out).toContain("_(no label suggestions)_");
});
it("uses an italic placeholder when summary is missing/whitespace", () => {
const a = __test.renderIssueTriageComment(triageFixture({ summary: "" }));
const b = __test.renderIssueTriageComment(triageFixture({ summary: " " }));
expect(a).toContain("_(no summary)_");
expect(b).toContain("_(no summary)_");
});
it("renders a duplicate callout when AI flagged one", () => {
const out = __test.renderIssueTriageComment(
triageFixture({ duplicateOfIssueNumber: 42 })
);
expect(out).toContain("**Possible duplicate of:** #42");
});
it("omits the duplicate callout for null / 0 / negative numbers", () => {
const a = __test.renderIssueTriageComment(triageFixture({ duplicateOfIssueNumber: null }));
const b = __test.renderIssueTriageComment(
triageFixture({ duplicateOfIssueNumber: 0 as any })
);
const c = __test.renderIssueTriageComment(
triageFixture({ duplicateOfIssueNumber: -1 as any })
);
for (const out of [a, b, c]) {
expect(out).not.toContain("Possible duplicate of");
}
});
it("ends with the suggestions-only disclaimer", () => {
const out = __test.renderIssueTriageComment(triageFixture());
expect(out.trimEnd().endsWith(
"_Suggestions only — nothing has been applied. The author stays in control._"
)).toBe(true);
});
it("formats critical priority literally (no pictographic emoji)", () => {
const out = __test.renderIssueTriageComment(
triageFixture({ priority: "critical" })
);
expect(out).toContain("**Priority:** critical");
expect(/\p{Extended_Pictographic}/u.test(out)).toBe(false);
});
});
describe("__test fail-open contracts", () => {
it("alreadyTriaged returns false when no DB / unknown issue", async () => {
const out = await __test.alreadyTriaged(
"00000000-0000-0000-0000-000000000000"
);
expect(out).toBe(false);
});
it("loadAvailableLabels returns [] for an unknown repo", async () => {
const out = await __test.loadAvailableLabels(
"00000000-0000-0000-0000-000000000000"
);
expect(Array.isArray(out)).toBe(true);
});
it("loadRecentIssues returns [] for an unknown repo", async () => {
const out = await __test.loadRecentIssues(
"00000000-0000-0000-0000-000000000000",
"00000000-0000-0000-0000-000000000000"
);
expect(Array.isArray(out)).toBe(true);
});
});
describe("triggerIssueTriage — fire-and-forget never throws", () => {
const before = process.env.ANTHROPIC_API_KEY;
it("resolves without throwing when API key is absent", async () => {
delete process.env.ANTHROPIC_API_KEY;
try {
let threw = false;
try {
await triggerIssueTriage({
ownerName: "alice",
repoName: "demo",
repositoryId: "00000000-0000-0000-0000-000000000000",
issueId: "00000000-0000-0000-0000-000000000000",
issueNumber: 1,
authorId: "00000000-0000-0000-0000-000000000000",
title: "Test",
body: "",
});
} catch {
threw = true;
}
expect(threw).toBe(false);
} finally {
if (before) process.env.ANTHROPIC_API_KEY = before;
}
});
it("never throws on garbage inputs", async () => {
let threw = false;
try {
await triggerIssueTriage({
ownerName: "",
repoName: "",
repositoryId: "not-a-uuid",
issueId: "not-a-uuid",
issueNumber: -1,
authorId: "not-a-uuid",
title: "",
body: "",
});
} catch {
threw = true;
}
expect(threw).toBe(false);
});
});
|