test(validation): P3-1第十六波 验证工具与schema守卫 57个 #720

Merged

xiaoxia merged 8 commits from feat/p3-1-validation-utils-tests into develop 2026-07-22 16:28:39 +08:00

Conversation 4 Commits 8 Files Changed 7

+3020 -252

xiaoxia commented 2026-07-22 14:49:04 +08:00

Owner

Copy Link

Copy Source

P3-1 第十六波单元测试

新增

• tests/unit/test_validation_utils.py - 57个测试

覆盖范围

• validate_phone (16个): 11位大陆号/+86前缀/13x/17x/19x/空/纯空白/太短/太长/前缀12x/前缀11x/含字母/含空格/首尾空格/+86带空格
• normalize_phone (6个): 纯号码/去+86/去空格/+86带空格/空串/只有+86
• validate_email (16个): 简单/子域名/+后缀/点号/下划线/连字符/大小写/空/纯空白/无@/无域名/无本地/无点/单字符TLD/首尾空格/%特殊字符
• normalize_environment (9个): development/production/staging/None→dev/空→dev/大写转小写/混合大小写/去空格/自定义环境名
• assert_auto_create_schema_allowed (10个): dev启用/dev禁用/staging启用抛错/production启用抛错/staging禁用/prod禁用/None启用/大写仍拦截/带空格staging拦截/自定义环境允许/常量验证

## P3-1 第十六波单元测试 ### 新增 - `tests/unit/test_validation_utils.py` - 57个测试 ### 覆盖范围 - **validate_phone** (16个): 11位大陆号/+86前缀/13x/17x/19x/空/纯空白/太短/太长/前缀12x/前缀11x/含字母/含空格/首尾空格/+86带空格 - **normalize_phone** (6个): 纯号码/去+86/去空格/+86带空格/空串/只有+86 - **validate_email** (16个): 简单/子域名/+后缀/点号/下划线/连字符/大小写/空/纯空白/无@/无域名/无本地/无点/单字符TLD/首尾空格/%特殊字符 - **normalize_environment** (9个): development/production/staging/None→dev/空→dev/大写转小写/混合大小写/去空格/自定义环境名 - **assert_auto_create_schema_allowed** (10个): dev启用/dev禁用/staging启用抛错/production启用抛错/staging禁用/prod禁用/None启用/大写仍拦截/带空格staging拦截/自定义环境允许/常量验证

xiaoxia added 7 commits 2026-07-22 14:49:04 +08:00

test(p3-1): 第十波 - url_security 安全模块单测 72个 ... 911478495a

覆盖 url_security.py 核心安全逻辑：
- validate_url_safety: scheme/端口/主机名/URL长度 等基础校验
- SSRF防护: 内网IP/回环/链路本地/组播/未指定/保留地址
- 内网主机名防护: localhost/.local/.internal/metadata
- 可信域名白名单: 精确匹配+子域名匹配
- is_url_safe 便捷函数
- 文件魔数校验: PNG/JPEG/GIF/WEBP/BMP/WAV/MP3/OGG/FLAC
- safe_download_file / safe_download_bytes (mock网络)
- 大小限制 / MIME类型白名单 / Content-Length预检

test(pagination): 补充通用分页器52个单元测试 ... a9bc314fed

覆盖 PaginationParams 默认值/校验/offset/limit、
PaginationMeta.from_params 边界场景、
PaginatedResponse.create 工厂方法、
paginate 内存分页函数

test(storage): 补充SharedStorageService 55个单元测试 ... 92b8044cb3

覆盖 _normalize_storage_key/URL解码、_is_local_generated_url、
get_url、create_direct_upload_post/policy+HMAC签名验证、
get_download_url fallback、未配置OSS错误处理、
bucket操作调用验证、单例模式、endpoint https前缀处理

test(module_registry): 补充模块注册中心62个单元测试 ... bb83549cc0

覆盖 ModuleStatus枚举/QuotaRule/ModuleCapability/Module数据类、
Module.activate/disable状态转换、ModuleRegistry注册注销/
自动激活/依赖检查/能力发现/状态过滤/全局单例

test(feature_flag_store): 补充FeatureFlag配置与存储43个单元测试 ... 0a23f72bfa

覆盖 FeatureFlagConfig 序列化/is_active判定（全局开关/
白名单/百分比哈希/边界值）、InMemoryFeatureFlagStore CRUD

test(text_splitter): 补充长文本分段工具32个单元测试 ... de5538ad1f

覆盖空文本/短文本/句子边界分段/超长硬切/
过短段落合并/max_chars参数/中英文混合/输出完整性

test(validation): 补充验证工具与schema守卫57个单元测试 ... 1a29decac3

覆盖 validate_phone/normalize_phone/validate_email、
normalize_environment/assert_auto_create_schema_allowed

gitea-actions bot commented 2026-07-22 15:13:23 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-07-22 15:17:46 +08:00

fix(ci): auto-fix lint/format issues 636acaec6d

auto-approve-bot approved these changes 2026-07-22 15:52:33 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-22 15:52:33 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot commented 2026-07-22 15:55:23 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #720

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

1. tests/unit/test_feature_flag_store.py (TestIsActiveHashConsistency.test_different_flags_same_user_can_differ)：测试用例缺少断言，无法验证核心逻辑。
   • 问题描述：该测试函数虽然创建了两个不同的 FeatureFlagConfig 实例，但函数体内部没有任何 assert 语句，仅依赖注释说明意图。这意味着即使实现代码中 is_active 完全忽略了 flag name（导致不同 flag 对同一用户永远返回相同结果），该测试依然会通过，失去了测试的意义。
   • 修改建议：必须添加断言来验证结果。由于哈希结果具有概率性，建议要么 Mock 哈希函数以确保输入不同时输出不同，要么选取已知的会产生不同哈希值的 flag name 进行断言（例如：assert config_a.is_active("user") != config_b.is_active("user")，虽然存在极小概率的 flakiness，但比无断言要好得多）。

💬 建议（1个可选）

1. tests/unit/test_feature_flag_store.py (TestIsActiveGlobalSwitch.test_enabled_no_percentage_no_whitelist_returns_true)：注释存在误导性。
   • 具体内容：代码中的注释 # 按逻辑：全局开了但百分比0且无白名单 → 其实应该是 False？ 表现出开发者对业务逻辑的不确定。如果当前断言 assert config.is_active() is False 是符合预期的设计（即 percentage=0 为硬性关闭），应修改注释为明确的业务规则说明（如 "Percentage 0 overrides global enabled switch"），避免后续维护者产生困惑。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-22 15:55:23 | 模型:}

## 代码审查结果 - PR #720 ### ⚠️ 问题（1个需要修改） 1. **tests/unit/test_feature_flag_store.py (TestIsActiveHashConsistency.test_different_flags_same_user_can_differ)**：测试用例缺少断言，无法验证核心逻辑。 - **问题描述**：该测试函数虽然创建了两个不同的 `FeatureFlagConfig` 实例，但函数体内部没有任何 `assert` 语句，仅依赖注释说明意图。这意味着即使实现代码中 `is_active` 完全忽略了 flag name（导致不同 flag 对同一用户永远返回相同结果），该测试依然会通过，失去了测试的意义。 - **修改建议**：必须添加断言来验证结果。由于哈希结果具有概率性，建议要么 Mock 哈希函数以确保输入不同时输出不同，要么选取已知的会产生不同哈希值的 flag name 进行断言（例如：`assert config_a.is_active("user") != config_b.is_active("user")`，虽然存在极小概率的 flakiness，但比无断言要好得多）。 ### 💬 建议（1个可选） 1. **tests/unit/test_feature_flag_store.py (TestIsActiveGlobalSwitch.test_enabled_no_percentage_no_whitelist_returns_true)**：注释存在误导性。 - **具体内容**：代码中的注释 `# 按逻辑：全局开了但百分比0且无白名单 → 其实应该是 False？` 表现出开发者对业务逻辑的不确定。如果当前断言 `assert config.is_active() is False` 是符合预期的设计（即 percentage=0 为硬性关闭），应修改注释为明确的业务规则说明（如 "Percentage 0 overrides global enabled switch"），避免后续维护者产生困惑。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-22 15:55:23 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit eb4ad569be into develop 2026-07-22 16:28:39 +08:00

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

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