test(feature_flag_store): P3-1第十四波 FeatureFlag存储单测 43个 #718

Merged

auto-approve-bot merged 6 commits from feat/p3-1-feature-flag-store-tests into develop 2026-07-22 15:53:51 +08:00

Conversation 5 Commits 6 Files Changed 5

+2423 -252

xiaoxia commented 2026-07-22 14:40:13 +08:00

Owner

Copy Link

Copy Source

P3-1 第十四波单元测试

新增

• tests/unit/test_feature_flag_store.py - 43个测试

覆盖范围

• 常量 (1个): Redis前缀
• FeatureFlagConfig默认值 (5个): name/enabled/percentage/whitelist/完整参数
• 序列化 (6个): to_dict默认值/to_dict带值/from_dict最小/from_dict完整/round_trip/类型强转
• is_active全局开关 (4个): 关闭返回false/带identifier也false/开启0百分比/100百分比
• is_active白名单 (5个): 匹配返回true/不匹配fallthrough/覆盖0百分比/覆盖部分百分比/无identifier不查白名单
• is_active百分比边界 (4个): 0返回false/100返回true/负数按0/超100按100
• is_active哈希一致性 (6个): 同用户结果一致/不同flag可不同/30%覆盖率大致正确/50%大致对半/10%大致十分之一/空identifier不崩溃
• InMemory CRUD (8个): 不存在返回默认禁用/set&get/覆盖/删除存在/删除不存在/空列表/多flag列表/返回副本
• store.is_active便捷方法 (4个): 启用flag/禁用flag/不存在flag/带identifier白名单

## P3-1 第十四波单元测试 ### 新增 - `tests/unit/test_feature_flag_store.py` - 43个测试 ### 覆盖范围 - **常量** (1个): Redis前缀 - **FeatureFlagConfig默认值** (5个): name/enabled/percentage/whitelist/完整参数 - **序列化** (6个): to_dict默认值/to_dict带值/from_dict最小/from_dict完整/round_trip/类型强转 - **is_active全局开关** (4个): 关闭返回false/带identifier也false/开启0百分比/100百分比 - **is_active白名单** (5个): 匹配返回true/不匹配fallthrough/覆盖0百分比/覆盖部分百分比/无identifier不查白名单 - **is_active百分比边界** (4个): 0返回false/100返回true/负数按0/超100按100 - **is_active哈希一致性** (6个): 同用户结果一致/不同flag可不同/30%覆盖率大致正确/50%大致对半/10%大致十分之一/空identifier不崩溃 - **InMemory CRUD** (8个): 不存在返回默认禁用/set&get/覆盖/删除存在/删除不存在/空列表/多flag列表/返回副本 - **store.is_active便捷方法** (4个): 启用flag/禁用flag/不存在flag/带identifier白名单

xiaoxia added 5 commits 2026-07-22 14:40:14 +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

gitea-actions bot commented 2026-07-22 15:02:46 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

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

fix(ci): auto-fix lint/format issues a9daa4b077

auto-approve-bot approved these changes 2026-07-22 15:37:30 +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:37:31 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot commented 2026-07-22 15:40:28 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #718

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

💡 建议（3个可选）

1. 文件权限设置: 新增的测试文件 test_feature_flag_store.py 权限为 100755（可执行）。测试脚本通常不需要执行权限，建议设置为 100644，以符合安全最佳实践。
2. 空值与None的边界测试: test_empty_identifier_treated_as_no_identifier 测试了空字符串 "" 的情况。建议增加对 None 的测试用例。虽然 Python 中 None 和空字符串在布尔上下文均为 False，但在哈希或字符串拼接操作中行为可能不同，确保 is_active(identifier=None) 不会导致崩溃是必要的。
3. 异常类型转换测试: test_from_dict_coerces_types 验证了合法的类型强转（如字符串 "50" 转 int）。建议增加非法输入的测试（如 percentage: "invalid"），验证 from_dict 方法在遇到 ValueError 时是否能优雅降级或抛出明确的异常，而不是直接导致程序崩溃。

---

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

---

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

## 代码审查结果 - PR #718 ### ⚠️ 问题（0个需要修改） ### 💡 建议（3个可选） 1. **文件权限设置**: 新增的测试文件 `test_feature_flag_store.py` 权限为 `100755`（可执行）。测试脚本通常不需要执行权限，建议设置为 `100644`，以符合安全最佳实践。 2. **空值与None的边界测试**: `test_empty_identifier_treated_as_no_identifier` 测试了空字符串 `""` 的情况。建议增加对 `None` 的测试用例。虽然 Python 中 `None` 和空字符串在布尔上下文均为 False，但在哈希或字符串拼接操作中行为可能不同，确保 `is_active(identifier=None)` 不会导致崩溃是必要的。 3. **异常类型转换测试**: `test_from_dict_coerces_types` 验证了合法的类型强转（如字符串 `"50"` 转 int）。建议增加非法输入的测试（如 `percentage: "invalid"`），验证 `from_dict` 方法在遇到 `ValueError` 时是否能优雅降级或抛出明确的异常，而不是直接导致程序崩溃。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-22 15:40:28 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot merged commit 34ea6a866b into develop 2026-07-22 15:53:51 +08:00

auto-approve-bot deleted branch feat/p3-1-feature-flag-store-tests 2026-07-22 15:53:52 +08:00

gitea-actions bot commented 2026-07-22 15:54:57 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #718 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

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