test(domain): wave193 video_share 单测 +43 #1157

Merged

xiaoxia merged 2 commits from test/wave193-video-share into develop 2026-07-30 00:24:44 +08:00

Conversation 3 Commits 2 Files Changed 1

+290

xiaoxia commented 2026-07-29 21:46:06 +08:00

Owner

Copy Link

Copy Source

变更说明

• 为 video_share domain 实体添加 43 个单元测试
• 覆盖：基础构造/slots/相等性、create()工厂校验、属性判断(has_password/is_expired/is_accessible)、方法(verify_password/increment_count/revoke)、token生成、密码哈希

测试统计

• 新增测试：43
• 全部通过 ✅

## 变更说明 - 为 video_share domain 实体添加 43 个单元测试 - 覆盖：基础构造/slots/相等性、create()工厂校验、属性判断(has_password/is_expired/is_accessible)、方法(verify_password/increment_count/revoke)、token生成、密码哈希 ## 测试统计 - 新增测试：43 - 全部通过 ✅

xiaoxia added 1 commit 2026-07-29 21:46:06 +08:00

test(domain): add wave193 video_share unit tests (+43) ... 3a6f765159

Add comprehensive unit tests for VideoShare domain entity:
- Basic construction, slots, equality
- create() factory with validation (video_id/user_id, expiry)
- Property checks: has_password, is_expired, is_accessible
- Methods: verify_password, increment_view/download_count, revoke
- generate_share_token helper and password hashing

gitea-actions bot commented 2026-07-29 21:47:23 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot commented 2026-07-29 21:47:38 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1157

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

无

💡 建议（2个可选）

1. tests/unit/domain/test_video_share.py 第233行：建议使用固定的过去时间点（如 datetime(2020, 1, 1, tzinfo=timezone.utc)）代替 datetime.now(...) - timedelta(...)，以避免因系统时间变动或时区问题导致的极低概率测试抖动。
2. tests/unit/domain/test_video_share.py 第197行：test_verify_password_no_password_true 测试了空字符串，考虑到 _hash_password 支持处理 None 输入，建议补充测试 verify_password(None) 的场景，确保边界行为一致性。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好

---

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

## 代码审查结果 - PR #1157 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（2个可选） 1. **tests/unit/domain/test_video_share.py 第233行**：建议使用固定的过去时间点（如 `datetime(2020, 1, 1, tzinfo=timezone.utc)`）代替 `datetime.now(...) - timedelta(...)`，以避免因系统时间变动或时区问题导致的极低概率测试抖动。 2. **tests/unit/domain/test_video_share.py 第197行**：`test_verify_password_no_password_true` 测试了空字符串，考虑到 `_hash_password` 支持处理 `None` 输入，建议补充测试 `verify_password(None)` 的场景，确保边界行为一致性。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 13:47:38 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot added 1 commit 2026-07-29 21:48:16 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] a1f42404a0

xiaoxia merged commit e902fbd65e into develop 2026-07-30 00:24:44 +08:00

gitea-actions bot commented 2026-07-30 00:26:51 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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