fix(test): hevc 转码测试恢复 sys.modules 时删除全部 mock 条目（补 #1566 漏修） #1567

Merged

auto-approve-bot merged 1 commits from fix/test-hevc-mock-restore-complete into develop 2026-08-31 02:51:01 +08:00

Conversation 5 Commits 1 Files Changed 1

+10 -4

xiaoxia commented 2026-08-31 02:16:49 +08:00

Owner

Copy Link

Copy Source

#1566 v1 恢复块保留了 video_processing.oss_helpers/thumbnail_generator 两个 mock 条目，同 xdist worker 后续测试文件 import 仍拿到 MagicMock，thumbnail 50 用例继续失败。patch.object 不依赖 mock 条目留在 sys.modules（download_asset 已绑定；thumb 字符串 patch 会触发真实模块重导入），全部删除。

#1566 v1 恢复块保留了 video_processing.oss_helpers/thumbnail_generator 两个 mock 条目，同 xdist worker 后续测试文件 import 仍拿到 MagicMock，thumbnail 50 用例继续失败。patch.object 不依赖 mock 条目留在 sys.modules（download_asset 已绑定；thumb 字符串 patch 会触发真实模块重导入），全部删除。

gitea-actions bot commented 2026-08-31 02:21:13 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot commented 2026-08-31 02:21:29 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

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

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

无

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

无

✅ 良好实践

1. [tests/unit/test_ingest_hevc_transcode_task.py] 详尽的注释说明

   • 具体内容：添加了清晰的注释解释了为何必须彻底清理 sys.modules，特别是针对 pytest-xdist 并行测试环境下的 mock 泄漏问题（关联 Issue #1566）。这对于后续维护和理解测试隔离机制非常有帮助。

2. [tests/unit/test_ingest_hevc_transcode_task.py] 修复测试隔离问题

   • 具体内容：移除了对特定 mock 模块（video_processing.oss_helpers 等）的保留逻辑，确保测试环境完全隔离，避免了因 mock 泄漏导致的并发测试失败。

---

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

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 1. **[tests/unit/test_ingest_hevc_transcode_task.py] 详尽的注释说明** - 具体内容：添加了清晰的注释解释了为何必须彻底清理 `sys.modules`，特别是针对 `pytest-xdist` 并行测试环境下的 mock 泄漏问题（关联 Issue #1566）。这对于后续维护和理解测试隔离机制非常有帮助。 2. **[tests/unit/test_ingest_hevc_transcode_task.py] 修复测试隔离问题** - 具体内容：移除了对特定 mock 模块（`video_processing.oss_helpers` 等）的保留逻辑，确保测试环境完全隔离，避免了因 mock 泄漏导致的并发测试失败。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-30 18:21:29 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot approved these changes 2026-08-31 02:25:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-31 02:25:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia force-pushed fix/test-hevc-mock-restore-complete from e290ff2b11 to a099e1f472 2026-08-31 02:29:20 +08:00 Compare

xiaoxia added 1 commit 2026-08-31 02:39:59 +08:00

fix(test): hevc 转码测试恢复 sys.modules 时删除全部 mock 条目 ... a8122dd634

#1566 v1 的恢复块保留了 video_processing.oss_helpers / thumbnail_generator
两个 mock 条目（误以为 patch.object 需要），导致同 xdist worker 后续测试
文件 import video_processing.* 仍拿到 MagicMock——test_thumbnail_generator
50 个用例继续失败（#1561/#1562 CI 实证）。

修复：
1. 恢复时删除全部注入的 mock 条目，杜绝跨文件 mock 污染；
2. restore 后显式 import 真实 oss_helpers 模块，upload patcher 直接打在
   真实模块上（patch stop 自动还原，无污染）；thumb patcher 为字符串目标，
   start 时自动导入真实模块打补丁。

xiaoxia force-pushed fix/test-hevc-mock-restore-complete from a099e1f472 to a8122dd634 2026-08-31 02:39:59 +08:00 Compare

auto-approve-bot merged commit d2098b03ab into develop 2026-08-31 02:51:01 +08:00

auto-approve-bot deleted branch fix/test-hevc-mock-restore-complete 2026-08-31 02:51:02 +08:00

gitea-actions bot commented 2026-08-31 02:56:01 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

xiaoxia referenced this pull request 2026-08-31 09:04:24 +08:00

fix(test): hevc 转码测试彻底移除 video_processing sys.modules mock 注入（39 用例跨文件污染修复） #1568

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