swadhinbiswas Complete merge-gate policy introspection and PR UX · 2/19/2026
df63ee1
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
import { beforeEach, describe, expect, it, vi } from "vitest";
const {
canReadRepoMock,
evaluateGatesMock,
fakeSchema,
} = vi.hoisted(() => ({
canReadRepoMock: vi.fn(async () => true),
evaluateGatesMock: vi.fn(async () => ({
canMerge: false,
results: [
{ passed: false, gateName: "Review", message: "At least one approval required" },
{ passed: true, gateName: "Merge Conflicts", message: "No conflicts" },
],
})),
fakeSchema: {
users: { username: {} },
repositories: { ownerId: {}, name: {}, id: {} },
pullRequests: { repositoryId: {}, number: {}, id: {} },
} as any,
}));
let mockDb: any;
vi.mock("@/db", () => ({
getDatabase: () => mockDb,
schema: fakeSchema,
}));
vi.mock("@/lib/permissions", () => ({
canReadRepo: canReadRepoMock,
}));
vi.mock("@/lib/ci-gates", () => ({
evaluateGates: evaluateGatesMock,
}));
import { GET as mergeReadinessGet } from "@/pages/api/repos/[owner]/[repo]/pulls/[number]/merge-readiness";
function makeDb(prState: "open" | "closed" = "open") {
return {
query: {
users: {
findFirst: vi.fn(async () => ({ id: "owner-1" })),
},
repositories: {
findFirst: vi.fn(async () => ({ id: "repo-1", ownerId: "owner-1", name: "demo" })),
},
pullRequests: {
findFirst: vi.fn(async () => ({
id: "pr-1",
number: 12,
state: prState,
isDraft: false,
mergeable: true,
mergeableState: "clean",
})),
},
},
};
}
async function readJson(response: Response): Promise<any> {
return response.json();
}
describe("pull request merge-readiness route", () => {
beforeEach(() => {
mockDb = makeDb("open");
canReadRepoMock.mockResolvedValue(true);
evaluateGatesMock.mockResolvedValue({
canMerge: false,
results: [
{ passed: false, gateName: "Review", message: "At least one approval required" },
{ passed: true, gateName: "Merge Conflicts", message: "No conflicts" },
],
});
});
it("returns gate blockers for open PRs", async () => {
const response = await mergeReadinessGet({
params: { owner: "owner-1", repo: "demo", number: "12" },
locals: { user: { id: "user-1", isAdmin: false } },
} as any);
const body = await readJson(response);
expect(response.status).toBe(200);
expect(body?.data?.canMerge).toBe(false);
expect(body?.data?.blockers).toContain("At least one approval required");
expect(body?.data?.policyReport?.failedGates).toBe(1);
expect(body?.data?.policyReport?.failedByType?.review).toBe(1);
expect(body?.data?.policyReport?.recommendations).toContain(
"Request required approvals and resolve review feedback."
);
expect(evaluateGatesMock).toHaveBeenCalledWith("pr-1");
});
it("returns closed-state blocker without evaluating gates", async () => {
mockDb = makeDb("closed");
const response = await mergeReadinessGet({
params: { owner: "owner-1", repo: "demo", number: "12" },
locals: { user: { id: "user-1", isAdmin: false } },
} as any);
const body = await readJson(response);
expect(response.status).toBe(200);
expect(body?.data?.canMerge).toBe(false);
expect(body?.data?.blockers).toEqual(["Pull request is not open"]);
expect(body?.data?.policyReport?.failedGates).toBe(0);
expect(body?.data?.policyReport?.recommendations).toContain(
"Re-open the pull request before attempting to merge."
);
});
});