test(domain): wave195 verification_code 单测 +27 #1159

Merged

xiaoxia merged 2 commits from test/wave195-verification-code into develop 2026-07-30 00:24:55 +08:00

Conversation 3 Commits 2 Files Changed 1

+280

xiaoxia commented 2026-07-29 21:52:26 +08:00

Owner

Copy Link

Copy Source

变更说明

• 为 VerificationCode domain 实体添加 27 个单元测试
• 覆盖：create()工厂、is_expired/is_used/is_valid属性、mark_used()、increment_attempts()、构造/slots/相等性

测试统计

• 新增测试：27
• 全部通过 ✅

## 变更说明 - 为 VerificationCode domain 实体添加 27 个单元测试 - 覆盖：create()工厂、is_expired/is_used/is_valid属性、mark_used()、increment_attempts()、构造/slots/相等性 ## 测试统计 - 新增测试：27 - 全部通过 ✅

xiaoxia added 1 commit 2026-07-29 21:52:27 +08:00

test(domain): add wave195 verification_code unit tests (+27) ... 098257efdc

Add comprehensive unit tests for VerificationCode domain entity:
- create() factory: basic, stripped recipient, custom code/ttl, uniqueness
- is_expired / is_used / is_valid properties
- mark_used() and increment_attempts()
- Direct construction, slots, equality

gitea-actions bot commented 2026-07-29 21:57:20 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

项目	详情
PR号	#1159
预览链接	https://pr-1159.preview.xiaoxiajianji.com
API环境	staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1159 | | 预览链接 | [https://pr-1159.preview.xiaoxiajianji.com](https://pr-1159.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

auto-approve-bot added 1 commit 2026-07-29 21:57:39 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] 50d74c454e

auto-approve-bot commented 2026-07-29 22:03:33 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

• 是否存在阻塞级问题：否
• 阻塞级问题数量：0 个

📊 审查概览

• 整体评价：有建议
• 建议级问题数量：2 个

🔴 阻塞级问题（必须修复）

无

💡 改进建议（不阻塞合并）

1. [tests/unit/domain/test_verification_code.py:238] 未使用的变量导致测试覆盖不完整

   • 具体内容：在 test_mark_used_twice_overwrites 方法中，变量 first 被赋值但从未被使用。测试的注释表明意图是验证重复调用会覆盖时间戳，但当前的断言仅检查了 used_at 不为 None。建议添加断言 assert vc.used_at > first 以确保时间戳确实被更新了。

2. [tests/unit/domain/test_verification_code.py:229] 建议使用 Mock 替代真实时间以增强确定性

   • 具体内容：在 test_mark_used_sets_timestamp 方法中，使用了 datetime.now(timezone.utc) 来进行边界检查。虽然这在大多数情况下可行，但依赖于系统时钟，在高负载下可能导致测试不稳定。鉴于文件中其他测试（如 test_create_custom_ttl）已成功使用了 patch 模拟时间，建议此处也采用相同方式，以保持测试风格的一致性和绝对的确定性。

✅ 良好实践

• 测试覆盖面广，包含了边界条件（如过期时间精确相等）、唯一性校验以及 __slots__ 限制测试。
• 使用了 patch 来模拟时间，有效避免了测试对环境时间的依赖。
• 测试用例命名清晰，结构分明，易于维护。

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 14:03:33 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：有建议 - 建议级问题数量：2 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[tests/unit/domain/test_verification_code.py:238] 未使用的变量导致测试覆盖不完整** - 具体内容：在 `test_mark_used_twice_overwrites` 方法中，变量 `first` 被赋值但从未被使用。测试的注释表明意图是验证重复调用会覆盖时间戳，但当前的断言仅检查了 `used_at` 不为 None。建议添加断言 `assert vc.used_at > first` 以确保时间戳确实被更新了。 2. **[tests/unit/domain/test_verification_code.py:229] 建议使用 Mock 替代真实时间以增强确定性** - 具体内容：在 `test_mark_used_sets_timestamp` 方法中，使用了 `datetime.now(timezone.utc)` 来进行边界检查。虽然这在大多数情况下可行，但依赖于系统时钟，在高负载下可能导致测试不稳定。鉴于文件中其他测试（如 `test_create_custom_ttl`）已成功使用了 `patch` 模拟时间，建议此处也采用相同方式，以保持测试风格的一致性和绝对的确定性。 ### ✅ 良好实践 - 测试覆盖面广，包含了边界条件（如过期时间精确相等）、唯一性校验以及 `__slots__` 限制测试。 - 使用了 `patch` 来模拟时间，有效避免了测试对环境时间的依赖。 - 测试用例命名清晰，结构分明，易于维护。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 14:03:33 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit e811516c6e into develop 2026-07-30 00:24:55 +08:00

gitea-actions bot commented 2026-07-30 00:27:02 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1159 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 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#1159