test(wave156): generation_task生成任务实体单测 +24 #1085

Merged

xiaoxia merged 6 commits from test/wave156-generation-task into develop 2026-07-28 16:18:59 +08:00

Conversation 1 Commits 6 Files Changed 6

+1872

xiaoxia commented 2026-07-28 13:43:56 +08:00

Owner

Copy Link

Copy Source

概述

为 domain/generation_task.py 新增 24 个单测，纯逻辑 0 外部依赖。

覆盖范围

• GenerationTaskStatus 枚举：6个（5种状态）
• create 工厂：18个
  • 最简/全字段创建
  • project_id/template_id 二选一校验
  • asset_library_id/素材列表 二选一校验
  • asset_ids/title_ids/voice_ids 列表拷贝
  • None列表默认空
  • 唯一ID/时间戳/默认值

验证

• ruff: All checks passed
• pytest: 24 passed

## 概述 为 domain/generation_task.py 新增 24 个单测，纯逻辑 0 外部依赖。 ## 覆盖范围 - **GenerationTaskStatus 枚举**：6个（5种状态） - **create 工厂**：18个 - 最简/全字段创建 - project_id/template_id 二选一校验 - asset_library_id/素材列表 二选一校验 - asset_ids/title_ids/voice_ids 列表拷贝 - None列表默认空 - 唯一ID/时间戳/默认值 ## 验证 - ruff: All checks passed - pytest: 24 passed

auto-approve-bot commented 2026-07-28 14:51:36 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

• 整体评价：通过
• 建议级问题数量：1 个

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

无

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

1. [tests/unit/domain/test_classification.py, test_duplication.py] 异常断言风格
   • 具体内容：当前代码在多处使用了 try...except assert False 的模式来验证异常抛出。建议使用 Pytest 标准库的 pytest.raises 上下文管理器。这样代码更简洁，可读性更强，且能自动处理异常未抛出的情况。
   • 示例：

     # 建议改为
     import pytest
     
     def test_create_empty_project_id(self):
         with pytest.raises(ValueError, match="project_id"):
             ClassificationJob.create(project_id="", asset_id="a1")
     

✅ 良好实践

1. 测试覆盖全面：测试用例不仅覆盖了正常流程，还详细覆盖了边界条件（如 0 值、100 值、空字符串、纯空格、负数等），体现了良好的测试思维。
2. 防御性测试：test_segments_independent_list 测试了不同实例间的列表隔离性，有效防止了可变默认参数或引用共享导致的潜在 Bug。
3. 代码结构清晰：测试类和方法命名语义明确，能够快速对应到被测逻辑。

---

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

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-28 06:51:36 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：1 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[tests/unit/domain/test_classification.py, test_duplication.py] 异常断言风格** - 具体内容：当前代码在多处使用了 `try...except assert False` 的模式来验证异常抛出。建议使用 Pytest 标准库的 `pytest.raises` 上下文管理器。这样代码更简洁，可读性更强，且能自动处理异常未抛出的情况。 - 示例： ```python # 建议改为 import pytest def test_create_empty_project_id(self): with pytest.raises(ValueError, match="project_id"): ClassificationJob.create(project_id="", asset_id="a1") ``` ### ✅ 良好实践 1. **测试覆盖全面**：测试用例不仅覆盖了正常流程，还详细覆盖了边界条件（如 0 值、100 值、空字符串、纯空格、负数等），体现了良好的测试思维。 2. **防御性测试**：`test_segments_independent_list` 测试了不同实例间的列表隔离性，有效防止了可变默认参数或引用共享导致的潜在 Bug。 3. **代码结构清晰**：测试类和方法命名语义明确，能够快速对应到被测逻辑。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-28 06:51:36 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia changed target branch from main to develop 2026-07-28 16:09:24 +08:00

xiaoxia added 2 commits 2026-07-28 16:12:12 +08:00

test(wave151): quota配额系统单测补全 +80 ... eb65fcc609

为 domain/quota.py 新增 80 个单测，纯逻辑 0 外部依赖：

- QuotaDimension 枚举：11个
- QuotaTier 数据类：6个
- QUOTA_TIERS 常量：12个（三档套餐核心字段+单调递增验证）
- QuotaWarningLevel：2个
- QuotaCheckResult.usage_percent：7个（正常/0/100%/超量/不限量/零限制）
- get_warning_level 告警级别：13个（0%/80%/95%/100%/超量/零限制/不限量/负数）
- QuotaRegistry 注册/查询：14个
- QuotaChecker 配额检查：13个
- 全局单例：3个

test(wave152): duplication查重记录单测 +40 ... 9bbfea8b90

为 domain/duplication.py 新增 40 个单测，纯逻辑 0 外部依赖：

- DuplicateSegment.create 工厂/校验：10个
- DuplicationRecord.create 工厂/校验：11个
- 状态流转 (pending/processing/completed/failed)：13个
- can_retry + reset_for_retry：4个
- segments 列表：3个

xiaoxia force-pushed test/wave156-generation-task from 58e825f5b5 to 9bbfea8b90 2026-07-28 16:12:12 +08:00 Compare

xiaoxia added 4 commits 2026-07-28 16:13:56 +08:00

test(wave153): edit_plan_clip剪辑计划片段单测 +44 ... d47d495803

为 domain/edit_plan_clip.py 新增 44 个单测，纯逻辑 0 外部依赖：

- EditPlanClipStatus 枚举：7个
- create 工厂/校验：17个
- assign_asset 素材分配：5个
- 状态流转（pending→ready→rendered/failed）：8个
- end_time/has_asset 属性：7个

test(wave154): classification素材分类单测 +34 ... f9c2b43e61

为 domain/classification.py 新增 34 个单测：

- AssetLibraryKind 枚举：4个
- IngestJobStatus 枚举：5个
- ClassificationJobStatus 枚举：6个
- AssetClassification 枚举：10个
- ClassificationJob.create：9个

test(wave155): generated_video生成视频实体单测 +16 ... 53f85d42bf

为 domain/generated_video.py 新增 16 个单测：

- create 必填字段 + 默认值
- strip 行为
- 4个空值校验（project_id/task_id/name/file_url）
- 唯一ID + 时间戳
- 零值合法（宽高/fps）
- generation_params 独立性

test(wave156): generation_task生成任务实体单测 +24 ... af60c9cc4b

为 domain/generation_task.py 新增 24 个单测：

- GenerationTaskStatus 枚举：6个
- create 工厂：18个（最简/全字段/二选一校验/列表拷贝/默认值）

xiaoxia merged commit fe97f016b2 into develop 2026-07-28 16:18:59 +08:00

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

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1085