test(unit): P3-1 第三波 新增3个领域模块单元测试(94个用例) #680

Merged

auto-approve-bot merged 4 commits from test/p3-1-wave3-domain-tests into develop 2026-07-21 19:00:12 +08:00

Conversation 5 Commits 4 Files Changed 4

+556 -547

xiaoxia commented 2026-07-21 13:53:18 +08:00

Owner

Copy Link

Copy Source

新增测试模块 (3个模块，94个测试用例)

模块	测试数	说明
tts_config	29	TTS配音配置 parse + clamp + 默认值 + 类型校验
subtitle	30	字幕时间轴 + 合并短片段 + 拆分长片段 + 标点智能断句
edit_plan	35	剪辑计划状态机 + create工厂方法 + 完整流程

覆盖重点

• TTS 配置的边界钳制（speed/pitch/volume）
• 字幕合并与拆分算法（短片段合并、长片段按标点拆分）
• EditPlan 状态机全量流转 + 非法流转校验
• 工厂方法参数校验（空名称、空template_id）

验证

本地 94 个测试全部通过，0 失败。

## 新增测试模块 (3个模块，94个测试用例) | 模块 | 测试数 | 说明 | |------|--------|------| | tts_config | 29 | TTS配音配置 parse + clamp + 默认值 + 类型校验 | | subtitle | 30 | 字幕时间轴 + 合并短片段 + 拆分长片段 + 标点智能断句 | | edit_plan | 35 | 剪辑计划状态机 + create工厂方法 + 完整流程 | ## 覆盖重点 - TTS 配置的边界钳制（speed/pitch/volume） - 字幕合并与拆分算法（短片段合并、长片段按标点拆分） - EditPlan 状态机全量流转 + 非法流转校验 - 工厂方法参数校验（空名称、空template_id） ## 验证 本地 94 个测试全部通过，0 失败。

xiaoxia added 1 commit 2026-07-21 13:53:18 +08:00

test(unit): P3-1 第三波 新增3个领域模块单元测试(94个用例) ... b2bdc3d742

新增测试模块 (3个模块，94个测试用例):
- tts_config: 29个 - TTS配音配置 parse + clamp + 默认值
- subtitle: 30个 - 字幕时间轴 + 合并短片段 + 拆分长片段
- edit_plan: 35个 - 剪辑计划状态机 + create工厂方法 + 完整流程

所有测试本地全绿。

gitea-actions bot commented 2026-07-21 14:01:47 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

gitea-actions bot added 1 commit 2026-07-21 18:41:40 +08:00

style: auto-format with black + isort + prettier [ci skip] 19299cdb05

xiaoxia referenced this issue from a commit 2026-07-21 18:56:11 +08:00

ci: trigger CI run for PR #680

xiaoxia added 1 commit 2026-07-21 18:56:11 +08:00

ci: trigger CI run for PR #680 91f6ada673

xiaoxia referenced this issue from a commit 2026-07-21 18:56:40 +08:00

ci: trigger CI run for PR #680

xiaoxia added 1 commit 2026-07-21 18:56:40 +08:00

ci: trigger CI run for PR #680 707304ae6d

auto-approve-bot commented 2026-07-21 18:58:56 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #680

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

无

💡 建议（2个可选）

1. tests/unit/test_edit_plan_domain.py：建议统一异常断言的严格程度。在 TestEditPlanStateMachine 类中，部分测试用例（如 test_invalid_start_editing_from_editing）使用了 pytest.raises(ValueError, match=...) 来校验错误信息，而另一部分（如 test_invalid_start_editing_from_rendering）仅使用了 pytest.raises(ValueError)。为了确保错误提示的准确性和测试的一致性，建议补充 match 参数校验具体的错误信息。
2. .ci-trigger：文件末尾缺少换行符（No newline at end of file）。虽然不影响功能，但符合 POSIX 标准的文本文件通常建议在末尾保留一个换行符。

---

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

---

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

## 代码审查结果 - PR #680 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（2个可选） 1. **tests/unit/test_edit_plan_domain.py**：建议统一异常断言的严格程度。在 `TestEditPlanStateMachine` 类中，部分测试用例（如 `test_invalid_start_editing_from_editing`）使用了 `pytest.raises(ValueError, match=...)` 来校验错误信息，而另一部分（如 `test_invalid_start_editing_from_rendering`）仅使用了 `pytest.raises(ValueError)`。为了确保错误提示的准确性和测试的一致性，建议补充 `match` 参数校验具体的错误信息。 2. **.ci-trigger**：文件末尾缺少换行符（No newline at end of file）。虽然不影响功能，但符合 POSIX 标准的文本文件通常建议在末尾保留一个换行符。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-21 18:58:56 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot approved these changes 2026-07-21 18:59:06 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-21 18:59:06 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit d66046aa4b into develop 2026-07-21 19:00:12 +08:00

auto-approve-bot deleted branch test/p3-1-wave3-domain-tests 2026-07-21 19:00:12 +08:00

gitea-actions bot commented 2026-07-21 19:00:46 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

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#680