xiaoxia/xiaoxia-saas
1
0
Fork 0
Code Issues 12 Pull Requests 2 Actions 2 Packages Projects Releases 7 Wiki Activity

fix(test): 修复test_dedup_pure收集阶段db连接副作用（P0） #894

Closed
xiaoxia wants to merge 1 commits from fix/dedup-pure-test-db-import into develop
pull from: fix/dedup-pure-test-db-import
merge into: xiaoxia:develop
Conversation 1 Commits 1 Files Changed 1
+57 -1

1 Commits

Author SHA1 Message Date
CI Bot c10552dae1 fix(test): 修复test_dedup_pure收集阶段db连接副作用
AI Code Review / AI Code Review (pull_request) Failing after 0s
Details
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 11s
Details
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 50s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 30s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 46s
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 30s
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 17s
Details
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build API Image (pull_request) Successful in 2m9s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 3m35s
Details
Preview Deploy / Deploy Preview Environment (pull_request) Failing after 12s
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m1s
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
CI/CD Pipeline / Unit Tests (pull_request) Successful in 2m0s
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
Details
CI/CD Pipeline / Integration Tests (pull_request) Successful in 2m0s
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 48m41s
Details
ACR Cleanup / ACR Image Cleanup (pull_request_target) Has been cancelled
Details
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 16s
Details 
根因：video_processing/dedup.py 顶层import worker_app.db，
而 db.py 模块加载时就执行 ensure_database_exists() 触发数据库连接。
纯函数测试不需要任何db/celery/cv2依赖。

修复：在测试模块顶部预先mock掉所有有副作用的模块
（cv2/celery/sqlalchemy/worker_app/packages.adapters/packages.shared），
确保 import dedup 时不会触发真实的数据库连接。

20 passed ✅
 2026-07-25 16:49:47 +08:00