10-05 到期例外清单里已设计好的首批 4 个具体调用,现予落码(原方案见 implement.md §14「首批具体分支与验证要求」): - ap_operations.feature_disabled(409):AP 台账/明细查询功能关闭 - ap_operations.query_invalid(400):AP 台账查询参数校验失败 - todo.feature_disabled(409):待办功能关闭 - portal.feature_disabled(409):工作台首页功能关闭 四处均改为调用 buildBusinessValidationError(reasonCode, details, statusCode), 保留原 HTTP 状态码、featureFlag/reason 等既有 details 字段不变,仅新增精确 reasonCode;不改功能开关判断逻辑、不改权限校验顺序。 - server/src/constants/errorReasons.js + client/src/api/errorReasons.ts: 注册 4 条原因码及中文文案,服务端/客户端逐字一致。 - scripts/qa/error-reason-legacy-baseline.json:移除这 4 条已迁移的旧例外 记录(591 → 587),check-error-reason-contract.mjs 主检查与 self-test 均 PASS,无 new_legacy/stale_exception/unregistered_reason。 - apOperations.permissions.test.js:扩展"功能关闭"断言校验 reasonCode 与 原 details 字段共存;新增"查询参数非法(功能开启)"400 用例,断言零 台账业务查询;新增"合法查询仍 200"用例。 - portal.test.js:新增工作台关闭时 409+reasonCode+零业务查询断言(此前 完全没有覆盖这条路径)。 - todos.test.js:新建(此前 todos 路由零测试覆盖),聚焦 ensureFeatureEnabled 网关:关闭时 GET/POST 均 409+reasonCode+零 todoItem 查询;开启时正常放行。 验证:server 5 个相关测试文件 286/286 通过;client typecheck/build 通过; check-ui-design-rules.mjs --strict、check-trellis-governance.mjs、 git diff --check 通过。routes/ 全量回归 872/874(2 个失败为 paymentBatches BIFF8 导出的既有环境问题,已用 git stash 验证在改动前的 HEAD 上同样失败, 与本次改动无关,不在本次范围内处理)。 剩余 669 条例外记录不变,尚未处理;B/C 两档 57 文件仍需逐模块确认迁移或 提出延期理由,本次不扩大范围。 Co-Authored-By: Claude Sonnet 5 <[email protected]>
126 行
5.7 KiB
JavaScript
126 行
5.7 KiB
JavaScript
const Fastify = require("fastify");
|
|
const apOperationsRoutes = require("../../../src/routes/apOperations");
|
|
const { hasPermission, resolvePermissionsForRoles } = require("../../../src/constants/permissions");
|
|
|
|
const paths = [
|
|
"/api/ap-operations",
|
|
"/api/ap-operations/reimbursement-ledger",
|
|
"/api/ap-operations/reimbursement-ledger?page=2&pageSize=10&status=approved&invoiceStatus=none&dateFrom=2026-01-01&dateTo=2026-09-07&q=synthetic",
|
|
];
|
|
|
|
async function buildApp(permissionCodes, enabled = true) {
|
|
const app = Fastify();
|
|
const queries = [];
|
|
const prisma = {
|
|
featureFlag: {
|
|
findUnique: vi.fn().mockResolvedValue({
|
|
flag_key: "payment_batch_v1",
|
|
override_mode: enabled ? "force_on" : "force_off",
|
|
rollout_type: "global",
|
|
department_ids: [],
|
|
}),
|
|
},
|
|
};
|
|
for (const model of ["paymentBatch", "paymentBatchItem", "entityRiskFlag", "travelReconciliationItem", "reimbursementForm"]) {
|
|
prisma[model] = {
|
|
findMany: vi.fn().mockResolvedValue([]),
|
|
count: vi.fn().mockResolvedValue(0),
|
|
};
|
|
queries.push(...Object.values(prisma[model]));
|
|
}
|
|
app.decorate("config", { paymentBatchEnabled: true });
|
|
app.decorate("prisma", prisma);
|
|
app.decorate("authenticate", async (request) => {
|
|
request.employee = { id: "synthetic-actor", permissionCodes };
|
|
});
|
|
app.decorate("hasPermission", (employee, permission) => hasPermission(employee.permissionCodes, permission));
|
|
app.setErrorHandler((error, request, reply) => {
|
|
reply.code(error.statusCode ?? 500).send({ error: { code: error.code, ...(error.details && Object.keys(error.details).length ? { details: error.details } : {}) } });
|
|
});
|
|
await app.register(apOperationsRoutes, { prefix: "/api" });
|
|
return { app, queries };
|
|
}
|
|
|
|
describe("AP full-ledger permission boundary", () => {
|
|
it.each([
|
|
["employee", false], ["manager", false], ["system_admin", false],
|
|
["ceo", false], ["finance", true], ["finance_mgr", true], ["admin", true],
|
|
])("checks both reads for %s", async (role, allowed) => {
|
|
const { app, queries } = await buildApp(resolvePermissionsForRoles([role]));
|
|
try {
|
|
for (const url of paths) {
|
|
const response = await app.inject({ method: "GET", url });
|
|
expect(response.statusCode).toBe(allowed ? 200 : 403);
|
|
if (!allowed) {
|
|
expect(response.json()).toEqual({ error: { code: "PERM_001" } });
|
|
for (const query of queries) expect(query).not.toHaveBeenCalled();
|
|
}
|
|
}
|
|
if (allowed) expect(queries.some((query) => query.mock.calls.length > 0)).toBe(true);
|
|
} finally {
|
|
await app.close();
|
|
}
|
|
});
|
|
|
|
it.each(["payment.batch.manage", "payment.batch.review", "risk.review.manage", "expense.finance.pay"])(
|
|
"does not infer ledger reading from %s", async (permission) => {
|
|
const { app, queries } = await buildApp([permission]);
|
|
try {
|
|
for (const url of paths) expect((await app.inject({ method: "GET", url })).statusCode).toBe(403);
|
|
for (const query of queries) expect(query).not.toHaveBeenCalled();
|
|
} finally {
|
|
await app.close();
|
|
}
|
|
},
|
|
);
|
|
|
|
it.each([["finance.ledger.view"], resolvePermissionsForRoles(["manager", "finance"]), resolvePermissionsForRoles(["ceo", "finance"])].map((permissions) => [permissions]))(
|
|
"honors explicit or combined-role ledger authority %j", async (permissions) => {
|
|
const { app } = await buildApp(permissions);
|
|
try {
|
|
for (const url of paths) expect((await app.inject({ method: "GET", url })).statusCode).toBe(200);
|
|
} finally {
|
|
await app.close();
|
|
}
|
|
},
|
|
);
|
|
|
|
it("preserves feature-off refusal without querying financial data", async () => {
|
|
const { app, queries } = await buildApp(["finance.ledger.view"], false);
|
|
try {
|
|
for (const url of paths) {
|
|
const response = await app.inject({ method: "GET", url });
|
|
expect(response.statusCode).toBe(409);
|
|
expect(response.json().error.details).toEqual({ featureFlag: "payment_batch_v1_disabled", reason: "flag_force_off", reasonCode: "ap_operations.disabled" });
|
|
}
|
|
for (const query of queries) expect(query).not.toHaveBeenCalled();
|
|
} finally {
|
|
await app.close();
|
|
}
|
|
});
|
|
});
|
|
|
|
describe("AP direct ledger calendar boundary", () => {
|
|
it.each(["dateFrom", "dateTo"].flatMap((field) => ["2026-02-29", "2026-04-31", "2026-13-01"].map((date) => [field, date])))(
|
|
"rejects %s=%s before ledger queries", async (field, date) => {
|
|
const { app, queries } = await buildApp(["finance.ledger.view"]);
|
|
try {
|
|
const response = await app.inject({ method: "GET", url: `/api/ap-operations/reimbursement-ledger?${field}=${date}` });
|
|
expect(response.statusCode).toBe(400);
|
|
expect(response.json().error.code).toBe("VALID_001");
|
|
expect(response.json().error.details).toEqual({ formErrors: [], fieldErrors: { [field]: ["invalid_calendar_date"] }, reasonCode: "ap_operations.ledger_query_invalid" });
|
|
for (const query of queries) expect(query).not.toHaveBeenCalled();
|
|
} finally { await app.close(); }
|
|
},
|
|
);
|
|
it.each([["2024-02-29", "2026-04-30"], ["2026-04-30", "2024-02-29"]])(
|
|
"preserves UTC bounds and existing reverse-range semantics %s / %s", async (dateFrom, dateTo) => {
|
|
const { app } = await buildApp(["finance.ledger.view"]);
|
|
try {
|
|
expect((await app.inject({ method: "GET", url: `/api/ap-operations/reimbursement-ledger?dateFrom=${dateFrom}&dateTo=${dateTo}` })).statusCode).toBe(200);
|
|
expect(app.prisma.reimbursementForm.findMany).toHaveBeenCalledWith(expect.objectContaining({ where: { AND: expect.arrayContaining([{ reimbursement_date: { gte: new Date(`${dateFrom}T00:00:00.000Z`), lte: new Date(`${dateTo}T23:59:59.999Z`) } }]) } }));
|
|
} finally { await app.close(); }
|
|
},
|
|
);
|
|
});
|