test: P3-1 第44波单元测试（email_service/session_store） #826

Merged

xiaoxia merged 1 commits from test/unit-test-wave44 into develop 2026-07-24 16:45:47 +08:00

Conversation 2 Commits 1 Files Changed 2

+650

xiaoxia commented 2026-07-24 12:57:14 +08:00

Owner

Copy Link

Copy Source

变更内容

• 新增 SMTP EmailService 模块测试 18 个（Noop、初始化、send_email 成功/失败/TLS/认证/CC/BCC、验证邮件/重置密码邮件内容、工厂函数）
• 新增 Redis SessionStore 模块测试 35 个（Noop、配置、save/get/delete/update、refresh_token 映射、用户 session 列表、批量删除、存在性检查、异常处理）
• 合计 +53 个测试

测试结果

• 本地全量：4530 passed, 8 skipped

## 变更内容 - 新增 SMTP EmailService 模块测试 18 个（Noop、初始化、send_email 成功/失败/TLS/认证/CC/BCC、验证邮件/重置密码邮件内容、工厂函数） - 新增 Redis SessionStore 模块测试 35 个（Noop、配置、save/get/delete/update、refresh_token 映射、用户 session 列表、批量删除、存在性检查、异常处理） - 合计 +53 个测试 ## 测试结果 - 本地全量：4530 passed, 8 skipped

xiaoxia added 1 commit 2026-07-24 12:57:14 +08:00

test: P3-1 第44波单元测试（email_service/session_store） e5207a4319

auto-approve-bot commented 2026-07-24 13:13:24 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #826

⚠️ 问题（2个需要修改）

1. tests/unit/test_email_service_smtp.py 第251行：单例测试逻辑错误，无法验证单例模式。

   • 问题描述：测试用例名为 test_singleton_default，但断言使用了 type(svc1) == type(svc2)。这仅验证了两次调用返回的对象类型一致，并未验证是否为同一个实例（单例）。如果工厂函数每次返回一个新的 NoopEmailService() 实例，该测试依然会通过，但这违背了单例模式的初衷。
   • 修改建议：应使用 assert svc1 is svc2 来验证对象身份，确保返回的是同一个实例。

2. tests/unit/test_redis_session_store.py 第307-311行：模拟了不可能的 Redis 状态，测试逻辑与实际行为冲突。

   • 问题描述：在 test_update_expired_session 中，mock 设置 get 返回了 JSON 数据（意味着 Key 存在），但同时设置 ttl 返回 -2（意味着 Key 不存在）。在真实 Redis 环境中，如果 Key 不存在（TTL 为 -2），get 必定返回 None。这种矛盾的 Mock 状态会导致测试逻辑混乱，无法准确反映代码在真实场景下的行为（例如，如果实现逻辑是先判断 get 结果，该测试可能掩盖错误；如果依赖 ttl 判断，则测试了不存在的分支）。
   • 修改建议：修正 Mock 数据。若要测试“Key 已过期”，应让 get 返回 None；若要测试“Key 存在但 TTL 异常”，应让 ttl 返回 -1（无过期）或正数。通常测试过期场景应模拟 get 返回 None。

💡 建议（1个可选）

1. tests/unit/test_redis_session_store.py 第358-367行：side_effect 链条过长，测试脆弱性较高。
   • 具体内容：test_delete_all_user_sessions 中通过 side_effect 硬编码了长达 6 次的 Redis 调用返回顺序。这种写法与内部实现细节（如 get_user_sessions 和 delete_session 的具体调用顺序）强耦合。一旦被测代码内部调用顺序发生微调（例如增加了日志记录或改变了查询逻辑），测试就会失败。
   • 修改建议：建议使用 side_effect 接受一个函数，根据传入的参数动态返回值，或者使用更高级的 Mock 工具（如 mock_redis.get 的 wraps 或自定义 Matcher）来解耦测试与实现细节。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-24 05:13:24 | 模型:}

## 代码审查结果 - PR #826 ### ⚠️ 问题（2个需要修改） 1. **tests/unit/test_email_service_smtp.py 第251行**：单例测试逻辑错误，无法验证单例模式。 - 问题描述：测试用例名为 `test_singleton_default`，但断言使用了 `type(svc1) == type(svc2)`。这仅验证了两次调用返回的对象类型一致，并未验证是否为同一个实例（单例）。如果工厂函数每次返回一个新的 `NoopEmailService()` 实例，该测试依然会通过，但这违背了单例模式的初衷。 - 修改建议：应使用 `assert svc1 is svc2` 来验证对象身份，确保返回的是同一个实例。 2. **tests/unit/test_redis_session_store.py 第307-311行**：模拟了不可能的 Redis 状态，测试逻辑与实际行为冲突。 - 问题描述：在 `test_update_expired_session` 中，mock 设置 `get` 返回了 JSON 数据（意味着 Key 存在），但同时设置 `ttl` 返回 `-2`（意味着 Key 不存在）。在真实 Redis 环境中，如果 Key 不存在（TTL 为 -2），`get` 必定返回 `None`。这种矛盾的 Mock 状态会导致测试逻辑混乱，无法准确反映代码在真实场景下的行为（例如，如果实现逻辑是先判断 `get` 结果，该测试可能掩盖错误；如果依赖 `ttl` 判断，则测试了不存在的分支）。 - 修改建议：修正 Mock 数据。若要测试“Key 已过期”，应让 `get` 返回 `None`；若要测试“Key 存在但 TTL 异常”，应让 `ttl` 返回 -1（无过期）或正数。通常测试过期场景应模拟 `get` 返回 `None`。 ### 💡 建议（1个可选） 1. **tests/unit/test_redis_session_store.py 第358-367行**：`side_effect` 链条过长，测试脆弱性较高。 - 具体内容：`test_delete_all_user_sessions` 中通过 `side_effect` 硬编码了长达 6 次的 Redis 调用返回顺序。这种写法与内部实现细节（如 `get_user_sessions` 和 `delete_session` 的具体调用顺序）强耦合。一旦被测代码内部调用顺序发生微调（例如增加了日志记录或改变了查询逻辑），测试就会失败。 - 修改建议：建议使用 `side_effect` 接受一个函数，根据传入的参数动态返回值，或者使用更高级的 Mock 工具（如 `mock_redis.get` 的 `wraps` 或自定义 Matcher）来解耦测试与实现细节。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-24 05:13:24 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 0bfb57813f into develop 2026-07-24 16:45:47 +08:00

xiaoxia deleted branch test/unit-test-wave44 2026-07-24 16:45:47 +08:00

gitea-actions bot commented 2026-07-24 16:48:33 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #826 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #826 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

Sign in to join this conversation.

Reviewers

No Reviewers

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

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#826