fix(test): test_dedup_engine 不再 mock packages.shared 父包，修复 50 个 thumbnail 单测被 mock 污染 #1563
Merged
auto-approve-bot
merged 1 commits from 2026-08-31 00:36:06 +08:00
fix/test-mock-pollution-thumbnail into develop
1 Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
 xiaoxia
|
3769d96d0d |
fix(test): test_dedup_engine 不再 mock packages.shared 父包，消除跨测试文件 mock 污染
CI/CD Pipeline / Check push changed paths (pull_request) Has been skipped
CI/CD Pipeline / Dedup Check - skip PR tests when covered by push pipeline (pull_request) Successful in 0s
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
CI/CD Pipeline / Retag skipped Staging Web Image (pull_request) Has been skipped
CI/CD Pipeline / Retag skipped Staging API Image (pull_request) Has been skipped
CI/CD Pipeline / Retag skipped Staging Worker Image (pull_request) Has been skipped
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 2m54s
CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been skipped
CI/CD Pipeline / Frontend Lint (pull_request) Has been skipped
CI/CD Pipeline / PR Build Web Image (pull_request) Has been skipped
AI Code Review / AI Code Review (pull_request) Successful in 3m17s
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 3m42s
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 4m15s
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 4m15s
CI/CD Pipeline / PR Build API Image (pull_request) Successful in 1m32s
CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 1m41s
Preview Deploy / Deploy Preview Environment (pull_request) Successful in 4m42s
PR Automation / Auto Approve on CI Green (pull_request) Successful in 5m59s
CI/CD Pipeline / Unit Tests (pull_request) Successful in 3m52s
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 7m1s
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
CI/CD Pipeline / Integration Tests (pull_request) Successful in 2m40s
CI/CD Pipeline / CI Gate (pull_request) Successful in 2m9s
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 3m23s
ACR Cleanup / ACR Image Cleanup (pull_request_target) Successful in 3m58s
_mock_if_absent("packages.shared") 会把整个父包替换为 MagicMock，
导致同 pytest 进程后续从 packages.shared.* 导入任何子模块都拿到
MagicMock（如 packages.shared.title_overlay / packages.shared.config），
连锁使依赖该包的 video_processing 模块整体被 mock，test_thumbnail_generator
的 50 个纯逻辑用例全部拿到 <MagicMock> 而失败。
dedup 模块只需要 packages.shared.storage 被 mock，父包保持真实导入即可。
|