fix: 修复测试用例中的问题 #13

Merged
moyin merged 1 commits from fix/test-issues into main 2025-12-18 13:36:14 +08:00
Owner

🎯 修复内容

修复了测试套件中的多个问题,确保所有测试用例正常通过。

🔧 具体修改

邮件服务测试 (email.service.spec.ts)

  • 移除4个未使用的 testService 变量,消除 TypeScript 警告
  • 优化测试代码结构

验证服务测试 (verification.service.spec.ts)

  • 添加缺失的 TTL mock 值,避免进入异常分支
  • 更新 getCodeStats 测试的期望值,包含 codecreatedAt 字段
  • 修正 TTL 期望值从 -1 改为 -2(符合 Redis 标准:-2 表示键不存在)

测试结果

  • 6 个测试套件全部通过
  • 113 个测试用例全部通过
  • 0 个失败的测试

📊 影响范围

  • 仅修改测试文件,不影响生产代码
  • 提高了测试的稳定性和准确性
  • 消除了 TypeScript 编译警告

🔍 验证方式

运行 npm test 确认所有测试通过。

## 🎯 修复内容 修复了测试套件中的多个问题,确保所有测试用例正常通过。 ## 🔧 具体修改 ### 邮件服务测试 (email.service.spec.ts) - 移除4个未使用的 `testService` 变量,消除 TypeScript 警告 - 优化测试代码结构 ### 验证服务测试 (verification.service.spec.ts) - 添加缺失的 TTL mock 值,避免进入异常分支 - 更新 `getCodeStats` 测试的期望值,包含 `code` 和 `createdAt` 字段 - 修正 TTL 期望值从 -1 改为 -2(符合 Redis 标准:-2 表示键不存在) ## ✅ 测试结果 - **6 个测试套件全部通过** - **113 个测试用例全部通过** - **0 个失败的测试** ## 📊 影响范围 - 仅修改测试文件,不影响生产代码 - 提高了测试的稳定性和准确性 - 消除了 TypeScript 编译警告 ## 🔍 验证方式 运行 `npm test` 确认所有测试通过。
moyin added 1 commit 2025-12-18 13:36:05 +08:00
- 修复邮件服务测试中未使用的变量警告
- 修复验证服务测试中的TTL和返回值期望问题
- 确保所有113个测试用例通过

详细修改:
- email.service.spec.ts: 移除4个未使用的testService变量
- verification.service.spec.ts:
  * 添加TTL mock值避免异常分支
  * 更新getCodeStats期望值包含code和createdAt字段
  * 修正TTL期望值从-1改为-2(Redis标准)

测试结果: 6个测试套件,113个测试用例全部通过
moyin merged commit 243ca05028 into main 2025-12-18 13:36:14 +08:00
moyin deleted branch fix/test-issues 2025-12-18 13:36:14 +08:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: datawhale/whale-town-end#13