PayrollManagementSystem

liuwenxin/PayrollManagementSystem

Fork 0

Commit Graph

Author	SHA1	Message	Date
liuwenxin	5ece9f6dab	refactor(backend): 仓储层散参数收敛为查询 DTO 仓储层的 list_/count_ 此前以 keyword/status/offset/limit 四五个散参数传参，调用点重复铺开、易漏传且不可扩展；按项目「参数 >4 封装 DTO」的约定统一收敛为各模块的 Query 对象（如 RoleQuery、BusinessQuery），服务层与测试调用同步调整，行为不变。 - repositories 层查询方法签名改为 (session, query)，分页与筛选条件由 Query 承载，_apply_filters 同步收口 - schemas 层补齐各模块 Query 模型（approval/business_role/report 等） - services 层调用点同步改为传 Query - tests 随之调整；新增用例：待办角色流转（财务通过后领导出现待办）、参数预览顺序稳定验证：全量 pytest 900 passed（11:08）；单元 526 passed；ruff check 绿。 Co-Authored-By: Claude Code <noreply@anthropic.com>	2026-10-22 19:43:20 +08:00
liuwenxin	f75a9d534e	refactor(test): 集成工厂上提共用，submit_change 拆校验段，审计重试断言对齐全局兜底 - tests/integration/factories.py：回款/发货两文件逐字重复的 _customer/_bitem/_bpayload/_create_business 上提（t_ 前缀与默认行口径单点维护）；test_audit_retry 的持续冲突断言由 500 更新为 400「数据完整性冲突」——全局 IntegrityError handler（批 6）收敛行为属预期改进，非回归 - param_service.submit_change 拆出 _normalize_items（校验段独立，主函数降至 50 行约定内）；e2e 六个列表页 helper 收口 common.py （base_data 版为准：分组参数化、:text-is 精确匹配），两脚本变薄 - 全量回归 625 passed（基线 603 + 新增 22）；e2e 四链路（登录 8/ 系统管理 13/基础数据 13/业务 16）全过 - 测试 README 补 2026-10-15 运行记录与 10 条评审缺陷（预览千分比/ 审计假成功/字典序 MAX/无锁编辑/删产品 500/schema 口径/quantize/ 负天数外插/评级并发/大小写不对称） Co-Authored-By: Claude Code <noreply@anthropic.com>	2026-10-15 10:58:44 +08:00
liuwenxin	5a3778a424	fix(business): 唯一键冲突重试改为审计上下文内循环，单条审计无递归三处 create（业务/发货/回款）的 return _retry_create 是吞异常后正常退出：外层审计把已回滚的首次尝试记成 success（假成功），重入整个 create_* 违反 audit_service「不支持嵌套」约定，且 _NO_RETRY 死常量导致持续冲突时无界递归至 RecursionError。改为 audit 内 for 循环（上限 _NO_RETRY），rollback 后重新锁行与取号，成功/最终失败各恰好一条审计。新增集成用例锁定两条路径的日志增量。 Co-Authored-By: Claude Code <noreply@anthropic.com>	2026-10-14 15:09:05 +08:00

Author

SHA1

Message

Date

liuwenxin

5ece9f6dab

refactor(backend): 仓储层散参数收敛为查询 DTO

仓储层的 list_*/count_* 此前以 keyword/status/offset/limit 四五个散参数
传参，调用点重复铺开、易漏传且不可扩展；按项目「参数 >4 封装 DTO」的
约定统一收敛为各模块的 Query 对象（如 RoleQuery、BusinessQuery），
服务层与测试调用同步调整，行为不变。

- repositories 层查询方法签名改为 (session, query)，分页与筛选条件
  由 Query 承载，_apply_filters 同步收口
- schemas 层补齐各模块 Query 模型（approval/business_role/report 等）
- services 层调用点同步改为传 Query
- tests 随之调整；新增用例：待办角色流转（财务通过后领导出现待办）、
  参数预览顺序稳定

验证：全量 pytest 900 passed（11:08）；单元 526 passed；ruff check 绿。

Co-Authored-By: Claude Code <noreply@anthropic.com>

2026-10-22 19:43:20 +08:00

liuwenxin

f75a9d534e

refactor(test): 集成工厂上提共用，submit_change 拆校验段，审计重试断言对齐全局兜底

- tests/integration/factories.py：回款/发货两文件逐字重复的
  _customer/_bitem/_bpayload/_create_business 上提（t_ 前缀与默认行
  口径单点维护）；test_audit_retry 的持续冲突断言由 500 更新为
  400「数据完整性冲突」——全局 IntegrityError handler（批 6）收敛
  行为属预期改进，非回归
- param_service.submit_change 拆出 _normalize_items（校验段独立，
  主函数降至 50 行约定内）；e2e 六个列表页 helper 收口 common.py
  （base_data 版为准：分组参数化、:text-is 精确匹配），两脚本变薄
- 全量回归 625 passed（基线 603 + 新增 22）；e2e 四链路（登录 8/
  系统管理 13/基础数据 13/业务 16）全过
- 测试 README 补 2026-10-15 运行记录与 10 条评审缺陷（预览千分比/
  审计假成功/字典序 MAX/无锁编辑/删产品 500/schema 口径/quantize/
  负天数外插/评级并发/大小写不对称）

Co-Authored-By: Claude Code <noreply@anthropic.com>

2026-10-15 10:58:44 +08:00

liuwenxin

5a3778a424

fix(business): 唯一键冲突重试改为审计上下文内循环，单条审计无递归

三处 create（业务/发货/回款）的 return _retry_create 是吞异常后正常
退出：外层审计把已回滚的首次尝试记成 success（假成功），重入整个
create_* 违反 audit_service「不支持嵌套」约定，且 _NO_RETRY 死常量
导致持续冲突时无界递归至 RecursionError。改为 audit 内 for 循环
（上限 _NO_RETRY），rollback 后重新锁行与取号，成功/最终失败各恰好
一条审计。新增集成用例锁定两条路径的日志增量。

Co-Authored-By: Claude Code <noreply@anthropic.com>

2026-10-14 15:09:05 +08:00

3 Commits