fix(test): test_ingest_hevc_transcode_task 模块级 mock 用后清理，修复跨文件 mock 污染 #1566

Merged

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

Conversation 5 Commits 1 Files Changed 1

+17

xiaoxia commented 2026-08-31 01:17:19 +08:00

Owner

Copy Link

Copy Source

模块导入时往 sys.modules 注入 worker_app.* / video_processing.* MagicMock 后从不清理，xdist 同 worker 后续测试文件拿到 MagicMock 模块，test_thumbnail_generator 50 个纯逻辑用例非确定性失败。加 module 作用域 autouse fixture 测试后恢复真实模块。配套 #1563。

模块导入时往 sys.modules 注入 worker_app.* / video_processing.* MagicMock 后从不清理，xdist 同 worker 后续测试文件拿到 MagicMock 模块，test_thumbnail_generator 50 个纯逻辑用例非确定性失败。加 module 作用域 autouse fixture 测试后恢复真实模块。配套 #1563。

gitea-actions bot commented 2026-08-31 01:21:05 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-08-31 01:23:10 +08:00

fix(test): test_ingest_hevc_transcode_task 导入后立即恢复 sys.modules，修复跨文件 mock 污染 ... 704cf63b6c

该文件模块级向 sys.modules 注入 worker_app.db/celery_app/core.config、
video_processing.oss_helpers/thumbnail_generator 的 MagicMock（供 ingest
任务模块导入），但导入后从不撤销。pytest-xdist 下同一 worker 进程内
其他测试文件 from video_processing.xxx import 会拿到 MagicMock——
test_thumbnail_generator 的 50 个纯逻辑用例断言到 <MagicMock> 而失败
（与文件分发/收集顺序相关，非确定性）。

按 test_dedup_pure.py 的既有模式：成功 import ingest_mod 后立即恢复
sys.modules（删除注入的 mock、必要项保留）。patch.object 仍通过本文件
持有的 mock 引用工作，ingest_mod 顶部 from import 已绑定 mock，
本文件测试行为不变。

配套 #1563（test_dedup_engine 父包 mock 污染）。

xiaoxia force-pushed fix/test-mock-cleanup-hevc from fe79cd90c3 to 704cf63b6c 2026-08-31 01:23:10 +08:00 Compare

auto-approve-bot commented 2026-08-31 01:29:46 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

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

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

无

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

无

✅ 良好实践

1. 解决测试隔离问题：针对 pytest-xdist 环境下的 sys.modules 污染问题，采取了“快照-恢复”的清理策略，有效防止了 Mock 对象泄漏到其他测试文件，避免了非预期的测试失败。
2. 精确的清理策略：在清理 sys.modules 时，使用了白名单机制保留本文件测试所需的特定模块（video_processing.oss_helpers 等），既保证了隔离性，又维持了本文件后续 patch.object 的可用性。
3. 安全的迭代方式：使用 list(sys.modules.keys()) 进行迭代，避免了在遍历过程中修改字典大小导致的 RuntimeError。
4. 详尽的注释：代码注释清晰地解释了修改的背景（xdist 污染）、原因（引用失效）以及参考实现（test_dedup_pure.py），极大提升了代码的可维护性。

---

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

---

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

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 1. **解决测试隔离问题**：针对 `pytest-xdist` 环境下的 `sys.modules` 污染问题，采取了“快照-恢复”的清理策略，有效防止了 Mock 对象泄漏到其他测试文件，避免了非预期的测试失败。 2. **精确的清理策略**：在清理 `sys.modules` 时，使用了白名单机制保留本文件测试所需的特定模块（`video_processing.oss_helpers` 等），既保证了隔离性，又维持了本文件后续 `patch.object` 的可用性。 3. **安全的迭代方式**：使用 `list(sys.modules.keys())` 进行迭代，避免了在遍历过程中修改字典大小导致的 RuntimeError。 4. **详尽的注释**：代码注释清晰地解释了修改的背景（xdist 污染）、原因（引用失效）以及参考实现（`test_dedup_pure.py`），极大提升了代码的可维护性。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-30 17:29:46 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

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

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

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

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit 4cf6f222ed into develop 2026-08-31 01:36:02 +08:00

auto-approve-bot deleted branch fix/test-mock-cleanup-hevc 2026-08-31 01:36:03 +08:00

gitea-actions bot commented 2026-08-31 01:41:44 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

xiaoxia referenced this issue from a commit 2026-08-31 02:16:42 +08:00

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

xiaoxia referenced this pull request 2026-08-31 02:16:49 +08:00

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

xiaoxia referenced this issue from a commit 2026-08-31 02:29:20 +08:00

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

xiaoxia referenced this issue from a commit 2026-08-31 02:39:59 +08:00

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

auto-approve-bot referenced this issue from a commit 2026-08-31 02:51:03 +08:00

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

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