test(P3-1): 第51波 domain 单测 - quota/job/entities（+183） #840

Merged

xiaoxia merged 1 commits from test/wave51-domain-quota-job-entities into develop 2026-07-24 21:30:52 +08:00

Conversation 2 Commits 1 Files Changed 3

+1034 -1339

xiaoxia commented 2026-07-24 20:23:59 +08:00

Owner

Copy Link

Copy Source

变更内容

• test_quota_domain.py: 72个测试
• test_job_domain.py: 67个测试
• test_entities_domain.py: 44个测试

覆盖模块

• domain/quota.py（QuotaService + 配额计算逻辑）
• domain/job.py（Job领域服务）
• domain/entities.py（领域实体）

测试统计

新增 183 个单测，全部通过

## 变更内容 - test_quota_domain.py: 72个测试 - test_job_domain.py: 67个测试 - test_entities_domain.py: 44个测试 ## 覆盖模块 - domain/quota.py（QuotaService + 配额计算逻辑） - domain/job.py（Job领域服务） - domain/entities.py（领域实体） ## 测试统计 新增 183 个单测，全部通过

xiaoxia added 1 commit 2026-07-24 20:23:59 +08:00

test(p3-1): wave51 - quota/job/entities 领域层单测 +183 ... b7aa4ac251

- quota: 配额维度/套餐/注册表/检查器/告警级别 全量测试 +72
- job: 任务类型/状态/创建/状态机/重试/序列化 全量测试 +67
- entities: User/Project/AssetLibrary/Asset/IngestJob/AssetStatus 全量测试 +44
- 合计 +183

auto-approve-bot commented 2026-07-24 20:29:30 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #840

⚠️ 问题（0个需要修改）

无

💡 建议（2个可选）

1. tests/unit/test_entities_domain.py (多处)：建议避免在单元测试中使用 time.sleep

   • 说明：在 test_add_tag_updates_updated_at 和 test_remove_tag_updates_updated_at 等测试方法中使用了 time.sleep(0.001) 来等待时间更新。这会导致测试运行变慢，且在系统负载高时可能因时间精度问题导致测试不稳定（Flaky Tests）。
   • 建议：建议在领域层注入 Clock 或 datetime_provider，或者在测试中使用 unittest.mock.patch 来模拟时间递增，从而实现确定性的、快速的测试。

2. tests/unit/test_entities_domain.py (多处)：ID 长度校验较为脆弱

   • 说明：多处测试断言 assert len(project.id) == 32，这强耦合了当前的 ID 生成策略（假设是 32 位 hex 字符串）。如果未来 ID 生成策略改为 UUID（带横线）或 ULID，这些测试将全部失败。
   • 建议：如果 ID 格式不是业务强约束，建议仅校验 ID 非空或符合特定正则格式（如 hex 字符），而非固定长度。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ⚠️ 建议关注性能

---

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

## 代码审查结果 - PR #840 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（2个可选） 1. **tests/unit/test_entities_domain.py (多处)**：建议避免在单元测试中使用 `time.sleep` - **说明**：在 `test_add_tag_updates_updated_at` 和 `test_remove_tag_updates_updated_at` 等测试方法中使用了 `time.sleep(0.001)` 来等待时间更新。这会导致测试运行变慢，且在系统负载高时可能因时间精度问题导致测试不稳定（Flaky Tests）。 - **建议**：建议在领域层注入 `Clock` 或 `datetime_provider`，或者在测试中使用 `unittest.mock.patch` 来模拟时间递增，从而实现确定性的、快速的测试。 2. **tests/unit/test_entities_domain.py (多处)**：ID 长度校验较为脆弱 - **说明**：多处测试断言 `assert len(project.id) == 32`，这强耦合了当前的 ID 生成策略（假设是 32 位 hex 字符串）。如果未来 ID 生成策略改为 UUID（带横线）或 ULID，这些测试将全部失败。 - **建议**：如果 ID 格式不是业务强约束，建议仅校验 ID 非空或符合特定正则格式（如 hex 字符），而非固定长度。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ⚠️ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-24 12:29:30 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 972ff89990 into develop 2026-07-24 21:30:52 +08:00

xiaoxia deleted branch test/wave51-domain-quota-job-entities 2026-07-24 21:30:52 +08:00

gitea-actions bot commented 2026-07-24 21:32:29 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #840 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 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

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#840