test(unit): 第62波 - module_registry + asr_service_factory + sms_service (+58) #856

Merged

xiaoxia merged 1 commits from test/wave62-module-registry-asr-factory-sms into develop 2026-07-25 08:12:53 +08:00

Conversation 2 Commits 1 Files Changed 3

+442 -561

xiaoxia commented 2026-07-25 00:04:49 +08:00

Owner

Copy Link

Copy Source

第62波单元测试补充

覆盖3个模块，+58个用例全部通过：

模块	文件	用例数	说明
模块注册中心	test_module_registry.py	38	Module/QuotaRule/ModuleCapability数据类 + 注册注销查询依赖检查 + 能力发现配额规则 + 全局单例
ASR服务工厂	test_asr_service_factory.py	8	provider切换 + 单例缓存 + 未知provider降级
短信服务	test_sms_service.py	12	NoopSmsService + AliyunSmsService初始化 + 工厂函数

本地验证

pytest tests/unit/test_module_registry.py tests/unit/test_asr_service_factory.py tests/unit/test_sms_service.py -v
58 passed in 2.65s

### 第62波单元测试补充 覆盖3个模块，+58个用例全部通过： | 模块 | 文件 | 用例数 | 说明 | |------|------|--------|------| | 模块注册中心 | `test_module_registry.py` | 38 | Module/QuotaRule/ModuleCapability数据类 + 注册注销查询依赖检查 + 能力发现配额规则 + 全局单例 | | ASR服务工厂 | `test_asr_service_factory.py` | 8 | provider切换 + 单例缓存 + 未知provider降级 | | 短信服务 | `test_sms_service.py` | 12 | NoopSmsService + AliyunSmsService初始化 + 工厂函数 | ### 本地验证 ``` pytest tests/unit/test_module_registry.py tests/unit/test_asr_service_factory.py tests/unit/test_sms_service.py -v 58 passed in 2.65s ```

xiaoxia added 1 commit 2026-07-25 00:04:50 +08:00

test(unit): 第62波 - module_registry + asr_service_factory + sms_service ... 47fee9412e

- test_module_registry: 模块注册中心，38个用例
  - Module/ModuleCapability/QuotaRule 数据类
  - 注册/注销/查询/依赖检查
  - 能力发现与配额规则
  - 全局单例清理

- test_asr_service_factory: ASR服务工厂，8个用例
  - 默认/空值/空白 provider 返回 None
  - mock provider 返回 MockASRService
  - 未知 provider 降级返回 None
  - 单例缓存 + reset 缓存

- test_sms_service: 短信服务，12个用例
  - NoopSmsService 空实现
  - AliyunSmsService 初始化（env + 显式参数）
  - 验证码委托到模板短信
  - get_sms_service 工厂（noop/aliyun/未知/大小写）

+58

auto-approve-bot commented 2026-07-25 00:10:17 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #856

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

无

💡 建议（2个可选）

1. tests/unit/test_module_registry.py 第236行：避免直接访问私有成员 _modules

   • 说明：在 test_has_capability_inactive_module 测试中，代码直接通过 registry._modules["ai_mod"] = m 绕过 register 方法注入模块。虽然这能强制设置特定状态，但破坏了封装性，如果 ModuleRegistry 内部实现改变（如改用属性封装、slots或加锁），测试将失败。
   • 建议：使用公开 API 先注册再禁用。例如：

     m = Module(name="ai_mod", capabilities=[ModuleCapability(name="generate_voice")])
     registry.register(m)
     m.disable()
     assert registry.has_capability("generate_voice") is False
     

2. tests/unit/test_asr_service_factory.py 第35行：测试中存在冗余操作

   • 说明：test_default_no_provider_returns_none 测试函数内部手动调用了 reset_asr_service_cache()。由于 clean_env fixture（autouse=True）已经在每个测试用例执行前调用了该方法，测试内的调用是冗余的。
   • 建议：删除测试函数内的 reset_asr_service_cache() 调用，依赖 fixture 即可，保持代码简洁。

---

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

---

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

## 代码审查结果 - PR #856 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（2个可选） 1. **tests/unit/test_module_registry.py 第236行**：避免直接访问私有成员 `_modules` - **说明**：在 `test_has_capability_inactive_module` 测试中，代码直接通过 `registry._modules["ai_mod"] = m` 绕过 `register` 方法注入模块。虽然这能强制设置特定状态，但破坏了封装性，如果 `ModuleRegistry` 内部实现改变（如改用属性封装、slots或加锁），测试将失败。 - **建议**：使用公开 API 先注册再禁用。例如： ```python m = Module(name="ai_mod", capabilities=[ModuleCapability(name="generate_voice")]) registry.register(m) m.disable() assert registry.has_capability("generate_voice") is False ``` 2. **tests/unit/test_asr_service_factory.py 第35行**：测试中存在冗余操作 - **说明**：`test_default_no_provider_returns_none` 测试函数内部手动调用了 `reset_asr_service_cache()`。由于 `clean_env` fixture（autouse=True）已经在每个测试用例执行前调用了该方法，测试内的调用是冗余的。 - **建议**：删除测试函数内的 `reset_asr_service_cache()` 调用，依赖 fixture 即可，保持代码简洁。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-24 16:10:17 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit c90b4819c0 into develop 2026-07-25 08:12:53 +08:00

xiaoxia deleted branch test/wave62-module-registry-asr-factory-sms 2026-07-25 08:12:53 +08:00

gitea-actions bot commented 2026-07-25 08:13:37 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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