test: P3-1 第41波单元测试（login_use_case） #823

Merged

xiaoxia merged 6 commits from test/unit-test-wave41 into develop 2026-07-24 16:45:41 +08:00

Conversation 2 Commits 6 Files Changed 19

+4560 -2747

xiaoxia commented 2026-07-24 12:30:41 +08:00

Owner

Copy Link

Copy Source

变更内容

• 新增 login_use_case 模块单元测试 32 个
  • TestLegacyHelpers: 2个（legacy sha256 哈希验证）
  • TestLoginRequest: 4个（邮箱规范化、默认设备/IP）
  • TestLoginUseCase: 12个（登录成功/空邮箱/空密码/用户不存在/密码错误/更新最后登录/会话保存/旧哈希迁移/旧哈希错密码/异常处理）
  • TestRefreshTokenUseCase: 8个（刷新成功/空token/无效token/不匹配/用户不存在/无效会话/有效access_token）
  • TestLogoutUseCase: 6个（单设备登出/全部设备/无session_id/会话不存在/异常处理）

测试结果

• 本地全量：4591 passed, 8 skipped

## 变更内容 - 新增 login_use_case 模块单元测试 32 个 - TestLegacyHelpers: 2个（legacy sha256 哈希验证） - TestLoginRequest: 4个（邮箱规范化、默认设备/IP） - TestLoginUseCase: 12个（登录成功/空邮箱/空密码/用户不存在/密码错误/更新最后登录/会话保存/旧哈希迁移/旧哈希错密码/异常处理） - TestRefreshTokenUseCase: 8个（刷新成功/空token/无效token/不匹配/用户不存在/无效会话/有效access_token） - TestLogoutUseCase: 6个（单设备登出/全部设备/无session_id/会话不存在/异常处理） ## 测试结果 - 本地全量：4591 passed, 8 skipped

xiaoxia added 6 commits 2026-07-24 12:30:41 +08:00

test: P3-1 第36波单元测试（assets/jwt_handler/password_handler/video_share） ... 040237169d

- test_assets_use_cases: 15个（ListAssetsUseCase + CreateAssetUseCase）
- test_jwt_handler: 18个（JWTHandler + 全局配置函数）
- test_password_handler: 22个（PasswordHandler + 全局配置函数）
- test_video_share_use_cases: 36个（8个UseCase全覆盖）
- 合计+77个测试，全量4554 passed

test: P3-1 第37波单元测试（text_splitter/pagination/password_hasher/bind_contact） ... d3ebfec015

- test_text_splitter: 18个（长文本分段工具）
- test_pagination: 25个（通用分页器）
- test_password_hasher: 24个（PasswordHasher + PasswordValidator）
- test_bind_contact_use_case: 29个（绑定手机/邮箱 + 发验证码）
- 合计+96个测试，全量通过

test: P3-1 第38波单元测试（ingest/classification/generation_tasks/password_reset） ... 304dd364e3

- test_ingest_jobs: 5个（SubmitIngestJobUseCase）
- test_classification_jobs: 6个（SubmitClassificationJobUseCase）
- test_generation_tasks: 20个（4个UseCase）
- test_password_reset_use_case: 20个（请求重置+重置密码）
- 合计+41个测试，全量4556 passed

test: P3-1 第39波单元测试（audio_merger/jwt_service/verification_code/register_user） ... 8fce9bf708

- test_audio_merger: 13个（FFmpeg音频合并器）
- test_jwt_service: 30个（JWT配置+服务+类型校验）
- test_verification_code_service: 31个（验证码生成/验证/频控+手机号邮箱校验）
- test_register_user_use_case: 20个（用户注册+邮箱验证）
- 合计+94个测试，全量4577 passed

test: P3-1 第40波单元测试（wechat_oauth + wechat_sync） ... e1be9191f7

- test_wechat_oauth_service: 31个（MemoryStateStore + OAuth服务 + 回调处理）
- test_wechat_sync_use_case: 16个（同步登录/注册 + 新用户创建）
- 合计+47个测试，全量4582 passed

test: P3-1 第41波单元测试（login_use_case） 0e1b2096f4

auto-approve-bot commented 2026-07-24 12:45:52 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #823

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

无

💡 建议（3个可选）

1. test_audio_merger.py 第119行：import subprocess 在该测试函数中未被使用，建议移除以保持代码整洁。
2. test_audio_merger.py 第28行：建议将 import shutil 移至文件顶部。在 fixture 的清理代码块中（yield 之后）进行导入是不规范的做法，虽然标准库通常不会出错，但这降低了代码的可读性和维护性。
3. test_audio_merger.py 第103行：在 test_merge_multiple_files 的 fake_run_ffmpeg 闭包中，计算了 output_idx = cmd.index("-c") + 2 但并未使用，实际使用的是 cmd[-1]。这行无效代码不仅多余，且如果未来命令参数变化导致 -c 不存在，还会导致测试本身抛出 ValueError，建议删除。

---

✅ 格式检查通过 | ✅ 逻辑审查需修改 | ✅ 性能

---

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

## 代码审查结果 - PR #823 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（3个可选） 1. **test_audio_merger.py 第119行**：`import subprocess` 在该测试函数中未被使用，建议移除以保持代码整洁。 2. **test_audio_merger.py 第28行**：建议将 `import shutil` 移至文件顶部。在 fixture 的清理代码块中（yield 之后）进行导入是不规范的做法，虽然标准库通常不会出错，但这降低了代码的可读性和维护性。 3. **test_audio_merger.py 第103行**：在 `test_merge_multiple_files` 的 `fake_run_ffmpeg` 闭包中，计算了 `output_idx = cmd.index("-c") + 2` 但并未使用，实际使用的是 `cmd[-1]`。这行无效代码不仅多余，且如果未来命令参数变化导致 `-c` 不存在，还会导致测试本身抛出 `ValueError`，建议删除。 --- ✅ 格式检查通过 | ✅ 逻辑审查需修改 | ✅ 性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-24 04:45:52 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 08cb1cfc23 into develop 2026-07-24 16:45:41 +08:00

xiaoxia deleted branch test/unit-test-wave41 2026-07-24 16:45:41 +08:00

gitea-actions bot commented 2026-07-24 16:48:40 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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