| name | test-cleaner |
| description | 测试清理技能,用于分析并清理超出测试范围或低价值的测试用例 |
测试清理技能
我是一个测试清理专家,专门分析并清理超出测试职责范围或低价值的测试用例。
我的能力
当你需要清理测试用例时,我会:
- 分析测试文件 - 扫描并识别问题测试用例
- 生成分析报告 - 详细列出问题和建议
- 等待用户确认 - 在执行清理前获得明确确认
- 执行清理 - 删除问题用例并验证结果
使用方式
使用格式:/test-cleanup [路径]
示例:
/test-cleanup - 分析整个项目的测试文件
/test-cleanup src/server/handlers - 分析特定模块
/test-cleanup src/server/handlers/__tests__/mcp-manage.handler.test.ts - 分析特定文件
分析流程
扫描阶段
- 识别测试文件:查找目标路径下所有的
*.test.ts 文件
- 解析导入关系:提取每个测试文件的导入语句
- 分类测试模块:单元测试、集成测试、端到端测试
问题识别
严重问题(应立即删除):
- 超纲测试:在消费者测试中测试外部依赖的内部实现
- 判断标准:测试用例验证的是外部依赖的行为而非当前模块的职责
警告问题(建议删除或重构):
建议优化(可选改进):
过度 Mock:
- 问题模式:Mock 了不应 Mock 的内部实现细节
分析报告格式
# 测试用例清理分析报告
## 分析范围
- 路径: `<指定路径>`
- 测试文件数: X
- 问题用例数: Y
## 严重问题(应删除)
### 文件: src/server/handlers/__tests__/mcp-manage.handler.test.ts
#### 问题: 超纲测试 - 重复测试 TypeFieldNormalizer
- **位置**: 第 45-67 行
- **问题**: 测试了外部依赖中已完整覆盖的单元功能
- **建议**: 删除这些测试用例
## 清理总结
- 可删除测试用例: X 个
- 可优化测试用例: Y 个
执行阶段
用户确认
- 展示分析报告
- 询问是否确认执行清理
- 等待用户确认后才进行修改
清理操作
如果用户确认,执行以下操作:
git checkout -b test-cleanup-backup-$(date +%Y%m%d)
pnpm test <指定路径>
pnpm test:coverage
回滚选项
git checkout main
git branch -D test-cleanup-backup-$(date +%Y%m%d)
示例分析
超纲测试示例
问题代码:
import { normalizeTypeField } from "@/lib/mcp-core/utils/type-field-normalizer";
describe("MCPManageHandler", () => {
describe("normalizeTypeField", () => {
it("应该正确处理 null 类型字段", () => {
});
});
});
清理建议:
describe("MCPManageHandler", () => {
describe("handleListTools", () => {
it("应该正确调用 normalizeTypeField 处理工具定义", () => {
});
});
});
注意事项
- 谨慎删除:如果不确定测试的价值,先标记为"建议优化"而非直接删除
- 保留集成测试:删除超纲的单元测试时,确保保留必要的集成测试
- 覆盖率监控:清理后确保整体测试覆盖率仍符合 80% 的要求
- 中文描述:所有测试用例描述必须使用中文
- 路径别名:修改测试代码时使用正确的路径别名
质量检查
pnpm typecheck
pnpm lint
pnpm test
pnpm test:coverage