test(unit): 第72波 - TTSJob状态机 + GeneratedVideo + VerificationCode (+76) #870

Merged

xiaoxia merged 2 commits from test/wave72-generated-video-tts-job-verification-code into develop 2026-07-25 11:32:36 +08:00

Conversation 2 Commits 2 Files Changed 2

+726

xiaoxia commented 2026-07-25 10:14:24 +08:00

Owner

Copy Link

Copy Source

内容

• TTSJob 状态机：状态枚举(5种) + create校验(3种) + is_terminal/is_retryable/is_completed属性(11种) + 13种状态转换场景 + mark_*方法(10种) + prepare_retry重试机制(4种) = 49 tests
• GeneratedVideo：create成功 + 4种空值校验 + 空白strip + 默认值 + generation_params处理 + duplicate标记 + 自定义状态/缩略图 = 18 tests
• VerificationCode：create创建(5种) + is_expired/is_used/is_valid属性(6种) + mark_used/increment_attempts/多code_type = 9 tests

测试结果

本地 pytest: 76 passed 全绿

### 内容 - **TTSJob 状态机**：状态枚举(5种) + create校验(3种) + is_terminal/is_retryable/is_completed属性(11种) + 13种状态转换场景 + mark_*方法(10种) + prepare_retry重试机制(4种) = **49 tests** - **GeneratedVideo**：create成功 + 4种空值校验 + 空白strip + 默认值 + generation_params处理 + duplicate标记 + 自定义状态/缩略图 = **18 tests** - **VerificationCode**：create创建(5种) + is_expired/is_used/is_valid属性(6种) + mark_used/increment_attempts/多code_type = **9 tests** ### 测试结果 本地 pytest: **76 passed** 全绿

xiaoxia added 1 commit 2026-07-25 10:14:24 +08:00

test(unit): 第72波 - TTSJob状态机 + GeneratedVideo + VerificationCode ... b0f1e74a85

- TTSJob: 状态枚举 + create校验 + is_terminal/is_retryable/is_completed + 13种状态转换 + mark_processing/mark_completed/mark_failed/mark_cancelled + prepare_retry重试机制(49个)
- GeneratedVideo: create校验(4种空值) + 空白strip + 默认值 + generation_params + duplicate标记 + 状态/缩略图(18个)
- VerificationCode: create创建(5种) + 属性方法(6种) + mark_used/increment_attempts/多code_type(5个)
+76 tests 全绿

xiaoxia added 1 commit 2026-07-25 11:07:34 +08:00

style: fix black formatting for wave72 generated video tests f6cdeaa62b

auto-approve-bot commented 2026-07-25 11:21:59 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #870

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

无

💡 建议（2个可选）

1. tests/unit/test_generated_video_verification_code.py 第19行, 第323行 & tests/unit/test_tts_job_domain_extended.py 第36行：ID长度校验硬编码

   • 描述：测试代码中使用了 assert len(video.id) == 32 来校验ID长度。这假设了ID生成策略固定为32位（如MD5或特定UUID）。如果未来ID生成策略变更（例如切换为标准UUID36位），测试将会失败，且这种硬编码与业务逻辑的核心功能（ID唯一性）关联度不高。
   • 建议：建议改为校验ID非空且为字符串类型，或者如果必须校验长度，请将 32 提取为常量或配置，以便统一维护。

2. tests/unit/test_generated_video_verification_code.py 第351行, 第364行 & tests/unit/test_tts_job_domain_extended.py 第398行：时间相关测试的稳定性

   • 描述：test_expiry_time_correct、test_custom_ttl 和 test_transition_updates_updated_at 等测试依赖系统当前时间（datetime.now()）。虽然使用了范围断言或 >= 来减少误差，但在高并发或系统时钟回拨（如NTP同步）的极端环境下，仍可能导致测试偶发性失败。
   • 建议：建议引入 freezegun 等 mock 时间库，或者确保被测的 Domain Model 支持传入时间参数，从而完全控制测试中的时间变量，提高测试的确定性。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好

---

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

## 代码审查结果 - PR #870 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（2个可选） 1. **tests/unit/test_generated_video_verification_code.py 第19行, 第323行 & tests/unit/test_tts_job_domain_extended.py 第36行**：ID长度校验硬编码 - **描述**：测试代码中使用了 `assert len(video.id) == 32` 来校验ID长度。这假设了ID生成策略固定为32位（如MD5或特定UUID）。如果未来ID生成策略变更（例如切换为标准UUID36位），测试将会失败，且这种硬编码与业务逻辑的核心功能（ID唯一性）关联度不高。 - **建议**：建议改为校验ID非空且为字符串类型，或者如果必须校验长度，请将 `32` 提取为常量或配置，以便统一维护。 2. **tests/unit/test_generated_video_verification_code.py 第351行, 第364行 & tests/unit/test_tts_job_domain_extended.py 第398行**：时间相关测试的稳定性 - **描述**：`test_expiry_time_correct`、`test_custom_ttl` 和 `test_transition_updates_updated_at` 等测试依赖系统当前时间（`datetime.now()`）。虽然使用了范围断言或 `>=` 来减少误差，但在高并发或系统时钟回拨（如NTP同步）的极端环境下，仍可能导致测试偶发性失败。 - **建议**：建议引入 `freezegun` 等 mock 时间库，或者确保被测的 Domain Model 支持传入时间参数，从而完全控制测试中的时间变量，提高测试的确定性。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-25 03:21:59 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit aa0b0de90f into develop 2026-07-25 11:32:36 +08:00

xiaoxia deleted branch test/wave72-generated-video-tts-job-verification-code 2026-07-25 11:32:36 +08:00

gitea-actions bot commented 2026-07-25 11:34:41 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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