test(wave161): domain entities领域模型 +64测 #1094

Merged

xiaoxia merged 1 commits from test/wave161-domain-entities into develop 2026-07-28 16:18:46 +08:00

Conversation 2 Commits 1 Files Changed 1

+520

xiaoxia commented 2026-07-28 14:06:05 +08:00

Owner

Copy Link

Copy Source

变更内容

• 4个枚举：AssetLibraryKind/IngestJobStatus/AssetStatus/ClassificationStatus
• User模型：最小创建/默认值/完整字段/时间戳
• Project模型：create校验/访问权限控制(is_owner/is_shared_with/can_access)
• AssetLibrary模型：create校验/时间戳/唯一ID
• Asset模型：create校验(名称/storage_key/mime_type)/完整参数/tag增删改
• IngestJob模型：create校验(3个字段)/strip处理/时间戳
• 边界场景：空字符串/空白字符/strip行为/去重/幂等删除

测试数据

• 新增测试：64个
• 测试文件：tests/unit/domain/test_entities.py
• 本地验证：pytest 全绿 + ruff 全绿 + black 格式化

### 变更内容 - 4个枚举：AssetLibraryKind/IngestJobStatus/AssetStatus/ClassificationStatus - User模型：最小创建/默认值/完整字段/时间戳 - Project模型：create校验/访问权限控制(is_owner/is_shared_with/can_access) - AssetLibrary模型：create校验/时间戳/唯一ID - Asset模型：create校验(名称/storage_key/mime_type)/完整参数/tag增删改 - IngestJob模型：create校验(3个字段)/strip处理/时间戳 - 边界场景：空字符串/空白字符/strip行为/去重/幂等删除 ### 测试数据 - 新增测试：64个 - 测试文件：tests/unit/domain/test_entities.py - 本地验证：pytest 全绿 + ruff 全绿 + black 格式化

gitea-actions bot commented 2026-07-28 14:46:39 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot commented 2026-07-28 14:48:51 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

• 整体评价：有建议
• 建议级问题数量：3 个

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

无

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

1. [tests/unit/domain/test_entities.py: 414, 467] 避免在测试中使用 time.sleep

   • 具体内容：在 test_add_tag_updates_updated_at 和 test_remove_tag_updates_updated_at 中使用了 time.sleep(0.001) 来等待时间变化。这会拖慢测试套件速度，且在不同机器上的可靠性不一致。建议使用 unittest.mock.patch 或 freezegun 等 Mock 工具来控制时间，确保测试既快速又准确。

2. [tests/unit/domain/test_entities.py: 414, 467] 模块导入位置不规范

   • 具体内容：import time 语句出现在测试方法内部。根据 PEP 8 规范，标准库导入应统一放在文件顶部。虽然功能上没问题，但调整位置有助于提高代码可读性和维护性。

3. [tests/unit/domain/test_entities.py: 89, 193, 282, 391, 512] 时间断言存在潜在的抖动风险

   • 具体内容：多处使用 datetime.now(timezone.utc) 构造 before 和 after 时间窗口来断言 created_at。虽然逻辑正确，但在 CI/CD 或高负载环境下，系统时钟可能发生微小回拨或跳变，导致这类夹逼断言偶发性失败。建议 Mock 时间生成固定值，或仅验证时间戳与当前时间的差值在合理范围内，而非严格验证 before <= x <= after。

✅ 良好实践

• 测试覆盖度很高，涵盖了枚举值、默认值、必填校验（空值/纯空格）、字段自动去等核心逻辑。
• 测试用例命名清晰，结构分层明确，易于理解。
• 对字符串输入的 strip 处理逻辑进行了细致的验证，体现了对边界条件的关注。

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-28 06:48:51 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：有建议 - 建议级问题数量：3 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[tests/unit/domain/test_entities.py: 414, 467] 避免在测试中使用 time.sleep** - 具体内容：在 `test_add_tag_updates_updated_at` 和 `test_remove_tag_updates_updated_at` 中使用了 `time.sleep(0.001)` 来等待时间变化。这会拖慢测试套件速度，且在不同机器上的可靠性不一致。建议使用 `unittest.mock.patch` 或 `freezegun` 等 Mock 工具来控制时间，确保测试既快速又准确。 2. **[tests/unit/domain/test_entities.py: 414, 467] 模块导入位置不规范** - 具体内容：`import time` 语句出现在测试方法内部。根据 PEP 8 规范，标准库导入应统一放在文件顶部。虽然功能上没问题，但调整位置有助于提高代码可读性和维护性。 3. **[tests/unit/domain/test_entities.py: 89, 193, 282, 391, 512] 时间断言存在潜在的抖动风险** - 具体内容：多处使用 `datetime.now(timezone.utc)` 构造 `before` 和 `after` 时间窗口来断言 `created_at`。虽然逻辑正确，但在 CI/CD 或高负载环境下，系统时钟可能发生微小回拨或跳变，导致这类夹逼断言偶发性失败。建议 Mock 时间生成固定值，或仅验证时间戳与当前时间的差值在合理范围内，而非严格验证 `before <= x <= after`。 ### ✅ 良好实践 - 测试覆盖度很高，涵盖了枚举值、默认值、必填校验（空值/纯空格）、字段自动去等核心逻辑。 - 测试用例命名清晰，结构分层明确，易于理解。 - 对字符串输入的 `strip` 处理逻辑进行了细致的验证，体现了对边界条件的关注。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-28 06:48:51 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia added 1 commit 2026-07-28 16:06:09 +08:00

test(wave161): domain entities领域模型 +64测 ... 7dc97dd8c8

- 4个枚举：AssetLibraryKind/IngestJobStatus/AssetStatus/ClassificationStatus
- User模型：最小创建/默认值/完整字段/时间戳
- Project模型：create校验/访问权限控制(is_owner/is_shared_with/can_access)
- AssetLibrary模型：create校验/时间戳/唯一ID
- Asset模型：create校验(名称/storage_key/mime_type)/完整参数/tag增删改
- IngestJob模型：create校验(3个字段)/strip处理/时间戳
- 边界场景：空字符串/空白字符/strip行为/去重/幂等删除

xiaoxia force-pushed test/wave161-domain-entities from f3a1d01ea6 to 7dc97dd8c8 2026-07-28 16:06:09 +08:00 Compare

xiaoxia merged commit 33fd762eea into develop 2026-07-28 16:18:46 +08:00

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