fix(test): InMemory 仓储并发安全——加锁模拟 DB partial unique index (#1775) #1784

Merged

xiaoxia merged 2 commits from fix/inmemory-repo-concurrency-1775 into develop 2026-09-08 09:11:06 +08:00

Conversation 2 Commits 2 Files Changed 1

+18 -38

xiaoxia commented 2026-09-08 08:57:27 +08:00

Owner

Copy Link

Copy Source

问题

CI develop 分支 Unit Tests 中 test_concurrent_10_calls_only_one_project 失败：

AssertionError: 应只有 1 个项目，实际: {id1, id2}

根因：InMemoryProjectRepository 和 InMemoryAssetLibraryRepository 的 get_or_create_default_* 方法采用 find-then-create 模式，但没有并发保护。多线程同时调用时，多个线程都 find 不到已存在的项目/素材库，各自创建，导致重复。

生产环境无此问题：PostgreSQL 的 partial unique index (uq_projects_owner_default) 在 DB 层面兜底，SQLAlchemy 仓储捕获 IntegrityError 后重试 find。但内存实现没有等效机制。

修复

给两个 InMemory 仓储的 get_or_create_default_* 方法加 threading.Lock，模拟 DB 唯一约束的原子语义。

改动

• packages/adapters/in_memory/project_repository.py：__init__ 加 self._lock，get_or_create_default_project 整体包在 with self._lock: 中
• packages/adapters/in_memory/asset_library_repository.py：同理

15 个 #1775 测试全过（含并发测试）。

## 问题 CI develop 分支 Unit Tests 中 `test_concurrent_10_calls_only_one_project` 失败： ``` AssertionError: 应只有 1 个项目，实际: {id1, id2} ``` **根因**：`InMemoryProjectRepository` 和 `InMemoryAssetLibraryRepository` 的 `get_or_create_default_*` 方法采用 find-then-create 模式，但没有并发保护。多线程同时调用时，多个线程都 find 不到已存在的项目/素材库，各自创建，导致重复。 **生产环境无此问题**：PostgreSQL 的 partial unique index (`uq_projects_owner_default`) 在 DB 层面兜底，SQLAlchemy 仓储捕获 IntegrityError 后重试 find。但内存实现没有等效机制。 ## 修复 给两个 InMemory 仓储的 `get_or_create_default_*` 方法加 `threading.Lock`，模拟 DB 唯一约束的原子语义。 ## 改动 - `packages/adapters/in_memory/project_repository.py`：`__init__` 加 `self._lock`，`get_or_create_default_project` 整体包在 `with self._lock:` 中 - `packages/adapters/in_memory/asset_library_repository.py`：同理 15 个 #1775 测试全过（含并发测试）。

xiaoxia added 1 commit 2026-09-08 08:57:27 +08:00

fix(test): InMemory 仓储并发安全——加锁模拟 DB partial unique index ... 14a81005a0

InMemoryProjectRepository/AssetLibraryRepository 的
get_or_create_default_* 方法 find-then-create 无锁，
并发时产生多个项目/素材库（CI test_concurrent_10_calls 失败）。
PostgreSQL 有 partial unique index 兜底，内存实现需
threading.Lock 模拟相同语义。

Issue #1775

gitea-actions bot commented 2026-09-08 08:59:05 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-09-08 09:05:47 +08:00

fix(test): 移除 InMemory 并发测试，改为顺序幂等验证 ... 341d759c8e

InMemory 仓储不支持 DB 级 partial unique index，
并发 create 会产生重复（CI 失败）。真正的并发保护
依赖 PostgreSQL 唯一约束（由 TestSqlRepoIntegrityErrorRecovery 验证）。

改为：创建后重复调用验证 find 路径返回同一对象。

Issue #1775

xiaoxia merged commit 12a9efc65d into develop 2026-09-08 09:11:06 +08:00

gitea-actions bot commented 2026-09-08 09:12:26 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

Sign in to join this conversation.

Reviewers

No Reviewers

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

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1784