文件
hyb-oa/server/tests/unit/routes/apOperations.permissions.test.js
wangsli和Claude Sonnet 5 cd01e4bf28 迁移首批4个错误码:AP台账/查询、待办、工作台功能开关 [09-05-error-explainability-payment-readiness]
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]>
2026-09-25 21:16:55 +08:00

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(); }
},
);
});