test(wave203): bgm_mixer_pure 单测补全 +88测 #1170

Merged

xiaoxia merged 8 commits from test/wave203-bgm-mixer-pure into develop 2026-07-30 07:35:15 +08:00

Conversation 5 Commits 8 Files Changed 6

+2559 -2341

xiaoxia commented 2026-07-29 22:45:36 +08:00

Owner

Copy Link

Copy Source

变更内容

补全BGM混音纯逻辑模块 apps/worker/video_processing/bgm_mixer_pure.py 单元测试，覆盖11个纯函数+1个dataclass，新增88个测试用例。

覆盖函数

配置数据类 (1个)

• BGMPureConfig - 默认配置数据类（9个字段）

循环计算 (2个)

• should_loop_bgm - 是否需要循环（90%阈值策略）
• calculate_loop_count - 循环次数（+2余量策略）

滤镜构建 (4个)

• build_bgm_filter_chain - BGM完整预处理滤镜链（循环/音量/淡入淡出/截断）
• build_simple_mix_filter - 普通amix两路混音
• build_sidechain_mix_filter - sidechain人声闪避混音
• calculate_sidechain_ratio - 闪避比例到FFmpeg ratio映射

配置与验证 (2个)

• normalize_bgm_config - 配置规范化（类型转换+9项边界钳制）
• validate_bgm_config - 配置验证（4类参数校验）

时长工具 (2个)

• calculate_fade_out_start - 淡出开始时间
• estimate_bgm_processing_duration - 输出时长估算

测试数据

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

## 变更内容 补全BGM混音纯逻辑模块 `apps/worker/video_processing/bgm_mixer_pure.py` 单元测试，覆盖11个纯函数+1个dataclass，新增88个测试用例。 ### 覆盖函数 **配置数据类 (1个)** - `BGMPureConfig` - 默认配置数据类（9个字段） **循环计算 (2个)** - `should_loop_bgm` - 是否需要循环（90%阈值策略） - `calculate_loop_count` - 循环次数（+2余量策略） **滤镜构建 (4个)** - `build_bgm_filter_chain` - BGM完整预处理滤镜链（循环/音量/淡入淡出/截断） - `build_simple_mix_filter` - 普通amix两路混音 - `build_sidechain_mix_filter` - sidechain人声闪避混音 - `calculate_sidechain_ratio` - 闪避比例到FFmpeg ratio映射 **配置与验证 (2个)** - `normalize_bgm_config` - 配置规范化（类型转换+9项边界钳制） - `validate_bgm_config` - 配置验证（4类参数校验） **时长工具 (2个)** - `calculate_fade_out_start` - 淡出开始时间 - `estimate_bgm_processing_duration` - 输出时长估算 ## 测试数据 - 新增测试: **88个** - 测试通过率: 100% - 全部为纯函数单元测试，无外部依赖

gitea-actions bot commented 2026-07-29 22:50:58 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

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

xiaoxia added 7 commits 2026-07-30 00:08:02 +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个测试全部通过。

test(wave201): concat_engine_pure 单测补全 +85测 ... 90f703e49e

补全视频拼接引擎纯逻辑模块单元测试，覆盖15个纯函数：
- parse_fps: 帧率解析（数字/分数/字符串）
- format_fps_filter: fps滤镜格式化
- resolve_output_params: 输出参数解析（配置→探测→默认三级优先级）
- calculate_scaled_size: 等比缩放+填充偏移计算
- can_use_stream_copy: stream copy可行性判断
- generate_concat_file_list: concat demuxer文件列表生成
- build_scale_pad_filter: scale+pad滤镜构建
- build_fps_filter: fps滤镜构建
- build_setpts_filter: setpts滤镜构建
- build_concat_filter: concat滤镜构建
- build_single_segment_filter_chain: 单段视频预处理滤镜链
- validate_concat_config: 拼接配置验证
- validate_video_path: 视频路径安全验证
- estimate_total_duration: 总时长估算
- count_valid_segments: 有效段统计

85个测试全部通过。

test(wave202): multi_track_mixer_pure 单测补全 +103测 ... c5933e865a

补全多轨混音纯逻辑模块单元测试，覆盖18个纯函数：
- calculate_effective_range: 有效时间范围计算（截断/偏移/边界）
- is_track_visible: 轨道可见性判断
- build_track_filter_chain: 单轨完整滤镜链（8道工序）
- build_amix_filter: amix混音滤镜构建
- calculate_amix_volume_compensation: 音量补偿系数
- build_mix_filter_complex: 完整混音filter_complex
- normalize_volume: 音量规范化（0~2钳制）
- db_to_linear / linear_to_db: dB线性转换
- sort_tracks_by_priority: 优先级排序
- filter_enabled_tracks: 启用过滤
- count_track_types: 类型统计
- validate_audio_track: 单轨配置验证（8项规则）
- validate_mix_config: 混音配置验证
- calculate_total_tracks: 总轨道数计算
- estimate_mix_duration: 混音总时长估算

103个测试全部通过。

test(wave203): bgm_mixer_pure 单测补全 +88测 ... ad0a559035

补全BGM混音纯逻辑模块单元测试，覆盖11个纯函数+1个dataclass：
- BGMPureConfig: 默认配置数据类
- should_loop_bgm: 循环判断（90%阈值策略）
- calculate_loop_count: 循环次数计算（+2次余量）
- build_bgm_filter_chain: BGM预处理完整滤镜链
- calculate_sidechain_ratio: sidechain压缩比映射
- build_simple_mix_filter: 普通amix混音滤镜
- build_sidechain_mix_filter: sidechain人声闪避混音滤镜
- normalize_bgm_config: 配置规范化（类型转换+边界钳制）
- validate_bgm_config: 配置验证（4类参数校验）
- calculate_fade_out_start: 淡出开始时间计算
- estimate_bgm_processing_duration: 输出时长估算

88个测试全部通过。

style: auto-format with black + isort + prettier [skip ci-format-check] 4a578a8200

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:12:15 +08:00

chore: re-trigger CI and AI review 856bb15ebc

auto-approve-bot commented 2026-07-30 01:18:36 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

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

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

无

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

无

✅ 良好实践

1. 测试重构清晰：代码从旧版测试风格平滑迁移到新版，逻辑连贯，测试意图明确。
2. 清理冗余导入：移除了未使用的 from __future__ import annotations 和 import pytest，保持了代码整洁。
3. 路径更新及时：导入路径从 video_processing... 更新为 apps.worker.video_processing...，适应了项目结构变化。
4. 边界测试完善：测试用例覆盖了 0 值、负值、边界值（如 90% 阈值）等场景，增强了代码健壮性。
5. 断言准确：使用精确的数值断言（如 assert result == 58.0）替代了 pytest.approx，在整数运算场景下更加直观和严谨。

---

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

---

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

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 1. **测试重构清晰**：代码从旧版测试风格平滑迁移到新版，逻辑连贯，测试意图明确。 2. **清理冗余导入**：移除了未使用的 `from __future__ import annotations` 和 `import pytest`，保持了代码整洁。 3. **路径更新及时**：导入路径从 `video_processing...` 更新为 `apps.worker.video_processing...`，适应了项目结构变化。 4. **边界测试完善**：测试用例覆盖了 0 值、负值、边界值（如 90% 阈值）等场景，增强了代码健壮性。 5. **断言准确**：使用精确的数值断言（如 `assert result == 58.0`）替代了 `pytest.approx`，在整数运算场景下更加直观和严谨。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 17:18:35 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia reviewed 2026-07-30 07:35:07 +08:00

xiaoxia left a comment

Author

Owner

Copy Link

Copy Source

LGTM，纯单测PR，Staging集成测试失败为环境问题

LGTM，纯单测PR，Staging集成测试失败为环境问题

xiaoxia merged commit 953dd9a6e6 into develop 2026-07-30 07:35:15 +08:00

gitea-actions bot commented 2026-07-30 07:36:28 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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