cenmiao/openspec-sp-tdd

TDD 技能 - Test-Driven Development

铁律：NO PRODUCTION CODE WITHOUT A FAILING TEST FIRST

任何生产代码编写前，必须先有失败的测试。

---

TDD 循环

1. Red - 写失败测试

编写一个测试，然后运行并确认失败。

测试标准：

• 只测试一个行为
• 测试名清晰表达被测试行为
• 使用真实代码（避免 mock，除非不可避免）
• 最小化测试代码

验证失败：

• 读取测试输出
• 确认失败原因符合预期
• 不要继续直到确认失败是正确的

2. Green - 写最小实现

编写刚好让测试通过的代码。

实现标准：

• 只写让测试通过的代码
• 不过度设计
• 不接受"应该能通过"——实际运行测试

验证通过：

• 读取测试输出
• 确认测试确实通过
• 确认没有其他测试被破坏

3. Refactor - 重构

改进代码结构，保持测试通过。

重构标准：

• 消除重复
• 改进命名
• 简化逻辑
• 保持所有测试通过

---

完成前验证清单

在标记任何实现为"完成"前，必须验证：

• [ ] 每个测试都先看过失败（Red 阶段）
• [ ] 每个实现都有对应测试
• [ ] 所有测试通过
• [ ] 代码已重构

禁止的词汇：

• "应该通过了" → 实际运行测试
• "看起来没问题" → 读取测试输出
• "我很有信心" → 信心≠证据

---

测试反模式（避免）

❌ 测试 mock 行为而非真实行为

// 错误：测试 mock 的返回值
test('mock returns value', () => {
  mock.fn.mockReturnValue(42);
  expect(mock.fn()).toBe(42);
});

// 正确：测试真实功能
test('calculate total', () => {
  const result = calculateTotal([10, 20, 12]);
  expect(result).toBe(42);
});

❌ 添加测试专用的生产方法

// 错误：为了测试添加方法
class UserService {
  // 只为测试添加
  getInternalState() { ... }
}

❌ 测试多个行为

// 错误：一个测试做太多
test('user flow', () => {
  createUser();
  login();
  updateProfile();
  logout();
  deleteAccount();
});

// 正确：每个行为独立测试
test('create user', () => { ... });
test('login with valid credentials', () => { ... });
test('update profile', () => { ... });

---

使用示例

实现新功能时：

用户：添加一个函数计算购物车总价

助手：我使用 openspec-sp-tdd 技能来实现。

**Step 1: Red - 写失败测试**

[编写测试代码，运行，显示失败]

✓ 测试失败，符合预期

**Step 2: Green - 写最小实现**

[编写实现代码，运行测试，显示通过]

✓ 测试通过

**Step 3: Refactor - 重构**

[改进代码结构]

✓ 所有测试保持通过

**验证清单：**
- [x] 测试先失败
- [x] 实现有对应测试
- [x] 所有测试通过

---

与其他技能集成

• openspec-apply-change: 每任务使用此 TDD 循环
• openspec-sp-subagent-dev: TDD 完成后进入审查流程
• openspec-sp-verify-before-complete: 验证 TDD 流程被执行

---

何时使用

• 实现新功能
• 修复 bug（先写重现测试）
• 重构代码（先有安全网测试）

何时不适用

• 纯样式修改
• 文档更新
• 配置变更（无逻辑变化）