feat(ci): P1-3 增量覆盖率门禁 - diff-cover接入+全量门槛65% #474

Merged

xiaoxia merged 3 commits from ci/p1-3-coverage-gate into develop 2026-07-17 17:45:23 +08:00

Conversation 4 Commits 3 Files Changed 2

+57 -4

xiaoxia commented 2026-07-17 15:08:49 +08:00

Owner

Copy Link

Copy Source

P1-3 测试覆盖率门禁

背景

当前后端总覆盖率 68%，但只有一个形同虚设的全量门槛（60%且无增量检查）。
新代码没有测试覆盖要求，导致覆盖率提升困难。

方案

双轨覆盖率门禁：

1. 增量覆盖率门禁（核心）

   • 工具：diff-cover，只统计 PR 改动行的覆盖率
   • 全量测试模式下：门槛 60%
   • 增量测试模式下：门槛 40%（因增量测试覆盖不全，调低门槛）
   • 不达标直接阻塞合并

2. 全量覆盖率兜底

   • 全量测试模式（push 到 develop/main）：门槛从 60% → 65%
   • 当前实际覆盖率 68%，留 3% 缓冲

模块覆盖率现状

模块	平均覆盖率	<50%文件数
packages/domain	92%	1
packages/shared	90%	0
packages/ports	87%	1
packages/application	82%	6
apps/api	70%	24
packages/infrastructure	64%	1
packages/adapters	56%	20

前端

前端只有 e2e 测试（Playwright），无单元测试体系，暂不接入覆盖率门禁。
需先建立前端单测体系，后续另立任务。

改动

• requirements-dev.txt: 新增 diff-cover>=8.0
• ci-cd.yml: 新增 Diff coverage check 步骤 + 全量门槛 60→65

关联 Issue

#445

## P1-3 测试覆盖率门禁 ### 背景 当前后端总覆盖率 68%，但只有一个形同虚设的全量门槛（60%且无增量检查）。 新代码没有测试覆盖要求，导致覆盖率提升困难。 ### 方案 **双轨覆盖率门禁：** 1. **增量覆盖率门禁（核心）** - 工具：diff-cover，只统计 PR 改动行的覆盖率 - 全量测试模式下：门槛 60% - 增量测试模式下：门槛 40%（因增量测试覆盖不全，调低门槛） - 不达标直接阻塞合并 2. **全量覆盖率兜底** - 全量测试模式（push 到 develop/main）：门槛从 60% → 65% - 当前实际覆盖率 68%，留 3% 缓冲 ### 模块覆盖率现状 | 模块 | 平均覆盖率 | <50%文件数 | |------|-----------|-----------| | packages/domain | 92% | 1 | | packages/shared | 90% | 0 | | packages/ports | 87% | 1 | | packages/application | 82% | 6 | | apps/api | 70% | 24 | | packages/infrastructure | 64% | 1 | | packages/adapters | 56% | 20 | ### 前端 前端只有 e2e 测试（Playwright），无单元测试体系，暂不接入覆盖率门禁。 需先建立前端单测体系，后续另立任务。 ### 改动 - `requirements-dev.txt`: 新增 diff-cover>=8.0 - `ci-cd.yml`: 新增 Diff coverage check 步骤 + 全量门槛 60→65 ### 关联 Issue #445

xiaoxia added 1 commit 2026-07-17 15:08:49 +08:00

feat(ci): P1-3 增量覆盖率门禁 - diff-cover接入+全量门槛提至65% ... 31e29e1aed

- 新增diff-cover增量行覆盖率检查，PR改动代码需>=60%覆盖率
- 增量测试模式下门槛调低至40%（覆盖不全）
- 全量覆盖率门槛从60%提升至65%（当前68%，留3%缓冲）
- 覆盖率不达标直接阻塞合并，倒逼新代码补单元测试

xiaoxia added 1 commit 2026-07-17 17:09:29 +08:00

fix(ci): diff-cover前初始化git仓库(CI tarball checkout无.git) 84f383c6cb

xiaoxia added 1 commit 2026-07-17 17:26:02 +08:00

perf(ci): ci-l1→ci-check 充分利用全部runner eb7eb6f317

auto-approve-bot reviewed 2026-07-17 17:43:38 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot reviewed 2026-07-17 17:43:55 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot reviewed 2026-07-17 17:44:09 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia reviewed 2026-07-17 17:44:55 +08:00

xiaoxia left a comment

Author

Owner

Copy Link

Copy Source

CI全绿，审批通过。

CI全绿，审批通过。

xiaoxia merged commit 91ba55e210 into develop 2026-07-17 17:45:23 +08:00

xiaoxia referenced this pull request 2026-07-17 18:26:17 +08:00

[CI优化] P1-3 测试覆盖率门禁 #445

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#474