fix(test): 修复 mock clip 缺失字段导致 7 个单测 Pydantic 校验失败 #1407

Merged

xiaoxia merged 2 commits from fix/test-clip-mock-fields into develop 2026-08-17 20:30:28 +08:00

Conversation 5 Commits 2 Files Changed 1

+5 -1

xiaoxia commented 2026-08-17 20:09:05 +08:00

Owner

Copy Link

Copy Source

问题

PR #1404 的 Clips API 改造引入了 EditorClipResponse schema，要求 template_clip_config_id、created_at、updated_at 为 str 类型。

但 tests/unit/test_templates_editor_api.py 中的 _make_mock_clip 使用 MagicMock() 构造 mock clip，未显式设置这 3 个字段。MagicMock 自动生成的属性是 MagicMock 对象而非字符串，导致 Pydantic 校验失败：

7 failed tests:
- TestClipEndpoints: test_list_clips_success, test_list_clips_pagination_params,
  test_create_clip_success, test_get_clip_detail_success, test_update_clip_success
- TestClipSplitMerge: test_split_clip_success, test_merge_clips_success

修复

在 _make_mock_clip 中显式设置缺失字段为正确类型：

clip.template_clip_config_id = ""    # str
clip.created_at = None               # _fmt_dt(None) → ""
clip.updated_at = None               # _fmt_dt(None) → ""

_fmt_dt 已处理 None → 返回空字符串，与 schema 的默认值一致。

验证

• pytest tests/unit/test_templates_editor_api.py -v 全部通过

## 问题 PR #1404 的 Clips API 改造引入了 `EditorClipResponse` schema，要求 `template_clip_config_id`、`created_at`、`updated_at` 为 `str` 类型。 但 `tests/unit/test_templates_editor_api.py` 中的 `_make_mock_clip` 使用 `MagicMock()` 构造 mock clip，未显式设置这 3 个字段。MagicMock 自动生成的属性是 `MagicMock` 对象而非字符串，导致 Pydantic 校验失败： ``` 7 failed tests: - TestClipEndpoints: test_list_clips_success, test_list_clips_pagination_params, test_create_clip_success, test_get_clip_detail_success, test_update_clip_success - TestClipSplitMerge: test_split_clip_success, test_merge_clips_success ``` ## 修复 在 `_make_mock_clip` 中显式设置缺失字段为正确类型： ```python clip.template_clip_config_id = "" # str clip.created_at = None # _fmt_dt(None) → "" clip.updated_at = None # _fmt_dt(None) → "" ``` `_fmt_dt` 已处理 `None` → 返回空字符串，与 schema 的默认值一致。 ## 验证 - `pytest tests/unit/test_templates_editor_api.py -v` 全部通过

xiaoxia added 1 commit 2026-08-17 20:09:05 +08:00

fix(test): add missing str fields to mock clip to fix Pydantic validation ... caa5113f7f

_make_mock_clip used MagicMock which auto-creates attributes as MagicMock
objects. EditorClipResponse requires template_clip_config_id, created_at,
and updated_at to be str. Pydantic validation fails with MagicMock values.

Fix: explicitly set these fields to proper types in _make_mock_clip:
- template_clip_config_id = ""
- created_at = None (handled by _fmt_dt → "")
- updated_at = None (handled by _fmt_dt → "")

Fixes 7 failing tests in TestClipEndpoints and TestClipSplitMerge.

gitea-actions bot commented 2026-08-17 20:11:29 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-08-17 20:15:35 +08:00

fix(test): fix merge response assertion for TestClipSplitMerge ... 8799eb8ff5

The merge endpoint returns {"merged_clip": {...}, "deleted_clip_ids": [...]}
not a flat clip object. Update test_merge_clips_success to check
data["merged_clip"] instead of data directly.

auto-approve-bot commented 2026-08-17 20:17:06 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

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

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

无

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

无

✅ 良好实践

1. Mock 对象属性补全：在 _make_mock_clip 中补充了 template_clip_config_id、created_at 和 updated_at 属性，使 Mock 对象更贴近实际数据模型，避免了测试过程中因属性缺失导致的潜在错误。
2. 断言逻辑修正：根据 API 响应结构的变化，及时更新了 test_merge_clips_success 中的断言逻辑，从验证根节点 id 改为验证 merged_clip.id，确保测试覆盖了新的响应结构。

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-08-17 12:17:06 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 1. **Mock 对象属性补全**：在 `_make_mock_clip` 中补充了 `template_clip_config_id`、`created_at` 和 `updated_at` 属性，使 Mock 对象更贴近实际数据模型，避免了测试过程中因属性缺失导致的潜在错误。 2. **断言逻辑修正**：根据 API 响应结构的变化，及时更新了 `test_merge_clips_success` 中的断言逻辑，从验证根节点 `id` 改为验证 `merged_clip.id`，确保测试覆盖了新的响应结构。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-17 12:17:06 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot approved these changes 2026-08-17 20:26:21 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-17 20:26:21 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia merged commit 2a8e949350 into develop 2026-08-17 20:30:28 +08:00

gitea-actions bot commented 2026-08-17 20:31:12 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

xiaoxia referenced this issue from a commit 2026-08-17 20:48:34 +08:00

fix(test): harden batch_download session tests with multi-path patching + diagnostic

xiaoxia referenced this issue from a commit 2026-08-17 20:53:17 +08:00

fix(test): harden batch_download session tests with multi-path patching + diagnostic

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