xiaozhi-client: Skill for Claude Code

测试清理技能

我是一个测试清理专家，专门分析并清理超出测试职责范围或低价值的测试用例。

我的能力

当你需要清理测试用例时，我会：

1. 分析测试文件 - 扫描并识别问题测试用例
2. 生成分析报告 - 详细列出问题和建议
3. 等待用户确认 - 在执行清理前获得明确确认
4. 执行清理 - 删除问题用例并验证结果

使用方式

使用格式：/test-cleanup [路径]

示例：

• /test-cleanup - 分析整个项目的测试文件
• /test-cleanup src/server/handlers - 分析特定模块
• /test-cleanup src/server/handlers/__tests__/mcp-manage.handler.test.ts - 分析特定文件

分析流程

扫描阶段

1. 识别测试文件：查找目标路径下所有的 *.test.ts 文件
2. 解析导入关系：提取每个测试文件的导入语句
3. 分类测试模块：单元测试、集成测试、端到端测试

问题识别

严重问题（应立即删除）：

• 超纲测试：在消费者测试中测试外部依赖的内部实现
  • 判断标准：测试用例验证的是外部依赖的行为而非当前模块的职责

警告问题（建议删除或重构）：

• 重复测试：已在提供者模块中覆盖的单元功能
  • 判断标准：测试用例在依赖模块中已存在类似的覆盖

建议优化（可选改进）：

• 低价值测试：不提供独特价值的测试用例
  • 判断标准：测试失败时无法提供有价值的调试信息

过度 Mock：

• 问题模式：Mock 了不应 Mock 的内部实现细节
  • 判断标准：Mock 破坏了测试的真实性和价值

分析报告格式

# 测试用例清理分析报告

## 分析范围
- 路径: `<指定路径>`
- 测试文件数: X
- 问题用例数: Y

## 严重问题（应删除）
### 文件: src/server/handlers/__tests__/mcp-manage.handler.test.ts
#### 问题: 超纲测试 - 重复测试 TypeFieldNormalizer
- **位置**: 第 45-67 行
- **问题**: 测试了外部依赖中已完整覆盖的单元功能
- **建议**: 删除这些测试用例

## 清理总结
- 可删除测试用例: X 个
- 可优化测试用例: Y 个

执行阶段

用户确认

1. 展示分析报告
2. 询问是否确认执行清理
3. 等待用户确认后才进行修改

清理操作

如果用户确认，执行以下操作：

# 创建备份分支
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 类型字段", () => {
      // 这是工具函数的单元测试，不应该在 handler 测试中
    });
  });
});

清理建议:

// ✅ 正确范围
describe("MCPManageHandler", () => {
  describe("handleListTools", () => {
    it("应该正确调用 normalizeTypeField 处理工具定义", () => {
      // 验证 handler 在业务场景中正确使用了外部依赖
    });
  });
});