user-vibeloop

← 返回决策队列

决策详情

人工审批:补齐 ffmpeg 导出链路的集成测试(test-plan 已规划但代码缺失)

已裁决

d-20260713-ef0fda · 来源 自动策略 · 指派给 管理员 · 工单 t-20260713-db9fcd · 2026-07-13 05:58:32

工单:补齐 ffmpeg 导出链路的集成测试(test-plan 已规划但代码缺失)(t-20260713-db9fcd)

修复摘要:在 tests/test_ffmpeg_export_integration.py 新增一条离线 ffmpeg 集成测试:用 lavfi testsrc+sine 合成 10 s 样本视频,跑 export_video_plan(固定两段 clip plan),断言 ① 输出时长与计划一致(±200 ms)、② 含视频+音频流、③ SRT 字幕时间戳单调递增且落在成片时长内;通过 setUpClass 只建一次源视频,skip guard 保证无 ffmpeg 时不阻测试;全套 86 测试全绿。

改动文件:tests/test_ffmpeg_export_integration.py

judge 裁定:verdict=approve,bugReal=true,fixEffective=true,regressionRisk=low,constraintsRespected=true

judge 理由

  • 工单描述的缺口真实存在:原 85 条测试中无 ffmpeg/ffprobe 断言,harness 输出 86 tests OK 证实新用例已被 discover 并稳定通过
  • 三个核心断言(时长精度≤5帧、音视频流均存在、SRT 时间戳单调且不越界)与工单验收标准一一对应,未缩水
  • 全离线:合成视频用 testsrc+sine,无 API key 依赖;@skipUnless 在工具不存在时自动跳过,不引入 CI 强依赖
  • 仅新增测试文件,未修改任何已有源码;setUpClass/tearDownClass 管理临时目录,无残留文件风险
  • 零 pip 包:所有 import 均为标准库或项目内模块(cutpoint_lab.*),满足零运行时依赖约束
  • harness 中出现的 RuntimeError(坏源-0302)是已有的错误路径测试预期输出,整体仍报 OK,非回归
  • 轻微不足但不阻合并:未断言 cue.start_ms ≤ cue.end_ms 及相邻 cue 无重叠,可作后续加固

转人工原因

  • 风险等级 medium 的门槛配置为人工审批

AI 对这个决策的四维评估(供判断参考):

  • 有无标准答案:有(偏技术最优解)
  • AI 置信度:高
  • 重要性:中
  • 可回退性:可撤销

在工单详情页可查看完整 diff:/tickets/t-20260713-db9fcd

选项已锁定
所选项:「批准合并」 · 裁决人 Michael · 2026-07-13 07:49:31