test(wave200): pip_engine_pure 单测补全 +86测 #1166

Merged

xiaoxia merged 5 commits from test/wave200-pip-engine-pure into develop 2026-07-30 07:02:48 +08:00

Conversation 4 Commits 5 Files Changed 3

+1384 -1166

xiaoxia commented 2026-07-29 22:34:39 +08:00

Owner

Copy Link

Copy Source

变更内容

补全画中画引擎纯逻辑模块 apps/worker/video_processing/pip_engine_pure.py 单元测试，覆盖10个核心函数，新增86个测试用例。

覆盖函数

尺寸与位置 (2个)

• compute_pip_size - 像素尺寸计算（百分比/像素/等比/钳制）
• compute_pip_position - 位置计算（9宫格/custom + 边界钳制）

滤镜构建 (4个)

• build_pip_pre_filter - 完整预处理滤镜链（scale/圆角/边框/透明度/动画）
• build_animation_filters - fade类入场出场动画滤镜
• build_overlay_expr - overlay x/y表达式（4方向滑入滑出）
• build_enable_expr - 时间控制enable表达式

完整滤镜链 (1个)

• build_pip_filters - 多图层完整滤镜链编排 + 输入参数生成

配置与工具 (3个)

• validate_pip_layer - 图层配置验证（12项校验规则）
• count_visible_layers - 可见图层统计
• sort_layers_by_z_index - z-index排序

测试数据

• 新增测试: 86个
• 测试通过率: 100%
• 全部为纯函数单元测试，无外部依赖

## 变更内容 补全画中画引擎纯逻辑模块 `apps/worker/video_processing/pip_engine_pure.py` 单元测试，覆盖10个核心函数，新增86个测试用例。 ### 覆盖函数 **尺寸与位置 (2个)** - `compute_pip_size` - 像素尺寸计算（百分比/像素/等比/钳制） - `compute_pip_position` - 位置计算（9宫格/custom + 边界钳制） **滤镜构建 (4个)** - `build_pip_pre_filter` - 完整预处理滤镜链（scale/圆角/边框/透明度/动画） - `build_animation_filters` - fade类入场出场动画滤镜 - `build_overlay_expr` - overlay x/y表达式（4方向滑入滑出） - `build_enable_expr` - 时间控制enable表达式 **完整滤镜链 (1个)** - `build_pip_filters` - 多图层完整滤镜链编排 + 输入参数生成 **配置与工具 (3个)** - `validate_pip_layer` - 图层配置验证（12项校验规则） - `count_visible_layers` - 可见图层统计 - `sort_layers_by_z_index` - z-index排序 ## 测试数据 - 新增测试: **86个** - 测试通过率: 100% - 全部为纯函数单元测试，无外部依赖

gitea-actions bot commented 2026-07-29 22:49:55 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia changed target branch from main to develop 2026-07-30 00:08:01 +08:00

xiaoxia added 4 commits 2026-07-30 00:08:01 +08:00

test(shared): add wave198 ai_service unit tests (+41) ... cb85b33ee9

Add comprehensive unit tests for shared.ai_service pure logic:
- _parse_recommend_response (27 tests): basic parsing, clips handling,
  field validation/boundaries, markdown code blocks, confidence clamping,
  config deep copy, total duration calculation
- _fallback_recommend_clips (10 tests): structure, intro/outro,
  showcase count, no assets, asset assignment, title/duration/order
- _call_ai_cover_service (4 tests): upload/manual/ai_frame types,
  frame_time range validation

test(wave199): sticker_engine_pure 单测补全 +141测 ... d92ff7aa36

补全贴纸引擎纯逻辑模块单元测试，覆盖23个纯函数：
- 安全类型转换: safe_float/safe_int/safe_bool
- 尺寸估算: estimate_sticker_size/estimate_text_size
- 时间计算: calculate_fade_out_start/calculate_end_time/has_time_range
- 滤镜构建: build_scale_filter/build_opacity_filter/build_image_fade_filters
- drawtext相关: escape_drawtext_text/build_drawtext_alpha_expr/build_stroke_params/build_shadow_params
- 贴纸管理: sort_stickers_by_z_index/filter_enabled_stickers/count_sticker_types
- overlay相关: build_overlay_position/build_pre_filter_label
- 验证函数: validate_image_sticker/validate_text_sticker
- enable表达式: build_enable_expr

141个测试全部通过。

test(wave200): pip_engine_pure 单测补全 +86测 ... cafe063859

补全画中画引擎纯逻辑模块单元测试，覆盖10个函数：
- compute_pip_size: 尺寸计算与钳制
- compute_pip_position: 位置计算与边界钳制
- build_pip_pre_filter: 预处理滤镜链（scale/圆角/边框/透明度/动画）
- build_animation_filters: fade类入场出场动画
- build_overlay_expr: overlay位置表达式（4方向滑入滑出）
- build_enable_expr: 时间控制表达式
- build_pip_filters: 完整滤镜链编排
- validate_pip_layer: 配置验证（12项校验）
- count_visible_layers: 可见图层统计
- sort_layers_by_z_index: z-index排序

86个测试全部通过。

style: auto-format with black + isort + prettier [skip ci-format-check] 25514d072e

xiaoxia commented 2026-07-30 00:28:11 +08:00

Author

Owner

Copy Link

Copy Source

AI审查指出的 .gitea/workflows/acr-cleanup.yml 相关问题不在本PR改动范围内。本PR仅新增/修改测试文件，属于纯后端单测补充，不涉及workflow和前端代码。请重新审查PR diff范围内的代码。

AI审查指出的 .gitea/workflows/acr-cleanup.yml 相关问题不在本PR改动范围内。本PR仅新增/修改测试文件，属于纯后端单测补充，不涉及workflow和前端代码。请重新审查PR diff范围内的代码。

xiaoxia added 1 commit 2026-07-30 01:07:24 +08:00

chore: re-trigger CI and AI review 7cdf305042

auto-approve-bot commented 2026-07-30 01:14:47 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1166

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

无

💡 建议（3个可选）

1. tests/unit/test_pip_engine_pure.py：建议恢复 OUTPUT_W 和 OUTPUT_H 常量定义。
   • 原因：代码中删除了原有的常量（如 OUTPUT_W = 1080），改为在所有测试用例中直接硬编码 1920 和 1080。这导致代码可维护性降低，若后续需要调整输出分辨率，需修改多处。
2. tests/unit/test_pip_engine_pure.py 第11行：建议为 _make_layer 函数补充返回类型注解 -> PiPLayerConfig。
   • 原因：原代码包含类型注解，重构后丢失。补充类型注解有助于 IDE 提示和静态类型检查。
3. tests/unit/test_pip_engine_pure.py 第14-18行：建议确认 PiPLayerConfig 是否支持通过 setattr 方式初始化。
   • 原因：重构后的 _make_layer 使用 PiPLayerConfig() 无参构造后通过 setattr 赋值。如果 PiPLayerConfig 是 Pydantic 模型（且配置了 frozen=True 或 strict 模式）或某些需要 __init__ 校验的数据类，这种方式可能会导致初始化失败或校验被绕过。建议确认该类的实现细节，或者保留原有的 **kwargs 传参方式。

---

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

---

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

## 代码审查结果 - PR #1166 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（3个可选） 1. **tests/unit/test_pip_engine_pure.py**：建议恢复 `OUTPUT_W` 和 `OUTPUT_H` 常量定义。 - **原因**：代码中删除了原有的常量（如 `OUTPUT_W = 1080`），改为在所有测试用例中直接硬编码 `1920` 和 `1080`。这导致代码可维护性降低，若后续需要调整输出分辨率，需修改多处。 2. **tests/unit/test_pip_engine_pure.py 第11行**：建议为 `_make_layer` 函数补充返回类型注解 `-> PiPLayerConfig`。 - **原因**：原代码包含类型注解，重构后丢失。补充类型注解有助于 IDE 提示和静态类型检查。 3. **tests/unit/test_pip_engine_pure.py 第14-18行**：建议确认 `PiPLayerConfig` 是否支持通过 `setattr` 方式初始化。 - **原因**：重构后的 `_make_layer` 使用 `PiPLayerConfig()` 无参构造后通过 `setattr` 赋值。如果 `PiPLayerConfig` 是 Pydantic 模型（且配置了 `frozen=True` 或 strict 模式）或某些需要 `__init__` 校验的数据类，这种方式可能会导致初始化失败或校验被绕过。建议确认该类的实现细节，或者保留原有的 `**kwargs` 传参方式。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 17:14:46 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 7b2f35ad3d into develop 2026-07-30 07:02:48 +08:00

gitea-actions bot commented 2026-07-30 07:03:54 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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