test(wave205): path_security 单测重构与补全 +46测 #1172

Merged

xiaoxia merged 2 commits from test/wave205-path-security into develop 2026-07-30 00:26:06 +08:00

Conversation 3 Commits 2 Files Changed 1

+224 -209

xiaoxia commented 2026-07-29 22:56:00 +08:00

Owner

Copy Link

Copy Source

变更内容

• path_security 单测从 unittest 迁移到 pytest 风格
• 修正导入路径为标准包导入（apps.worker.video_processing.path_security）
• 覆盖 sanitize_filename / is_path_safe / is_in_allowed_dirs / safe_resolve_path 核心函数
• 覆盖边界场景: 路径遍历攻击、空路径、超长路径、特殊字符、符号链接、Windows路径等

测试统计

• 新增/重构测试: 46个
• 本地验证: pytest全绿 + black + ruff通过

## 变更内容 - path_security 单测从 unittest 迁移到 pytest 风格 - 修正导入路径为标准包导入（apps.worker.video_processing.path_security） - 覆盖 sanitize_filename / is_path_safe / is_in_allowed_dirs / safe_resolve_path 核心函数 - 覆盖边界场景: 路径遍历攻击、空路径、超长路径、特殊字符、符号链接、Windows路径等 ## 测试统计 - 新增/重构测试: 46个 - 本地验证: pytest全绿 + black + ruff通过

xiaoxia added 1 commit 2026-07-29 22:56:01 +08:00

test(wave205): path_security 单测重构与补全 +46测 ... 92d5b3f26c

- 从 unittest 迁移到 pytest 风格
- 修正导入路径为标准包导入
- 覆盖 sanitize_filename / is_path_safe / is_in_allowed_dirs / safe_resolve_path
- 覆盖边界: 路径遍历、空路径、超长路径、特殊字符、符号链接等

auto-approve-bot added 1 commit 2026-07-29 23:03:18 +08:00

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

gitea-actions bot commented 2026-07-29 23:09:16 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot commented 2026-07-29 23:09:47 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

• 是否存在阻塞级问题：否
• 阻塞级问题数量：0 个

📊 审查概览

• 整体评价：通过
• 建议级问题数量：2 个

🔴 阻塞级问题（必须修复）

无

💡 改进建议（不阻塞合并）

1. [tests/unit/test_path_security.py: 211] 弱断言建议增强

   • 具体内容：在 test_none_allowed_dirs_uses_default 测试中，仅断言返回值为 bool 类型 (isinstance(result, bool))。如果默认允许目录列表被误修改为空列表，导致 /tmp 不再被允许，该测试依然会通过（返回 False）。建议明确断言 assert result is True，以确保默认配置确实包含 /tmp。

2. [tests/unit/test_path_security.py: 100, 103] 平台兼容性建议

   • 具体内容：测试中使用了 /etc/passwd 等 Linux 特有的系统路径来验证“系统路径”拦截逻辑。如果 CI/CD 环境包含 Windows 或其他非 Linux 系统，这些测试可能会无法按预期触发拦截（因为 /etc/passwd 在 Windows 上可能不被视为敏感系统路径）。建议使用 pytest.mark.linux 标记这些测试，或者通过 Mock/注入配置的方式模拟敏感路径列表，以提高测试在不同平台上的通过率和准确性。

✅ 良好实践

• 使用 pytest.fixture 管理临时目录资源，利用上下文管理器自动清理，比旧版的 setUp/tearDown 更简洁可靠。
• 引入 MAX_PATH_LENGTH 常量替代硬编码数字，使测试边界与业务逻辑保持同步。
• 增加了 match 参数校验异常信息，确保抛出的异常原因符合预期，提高了测试的精确度。
• 增加了对 pathlib.Path 对象作为输入的测试用例，增强了代码对现代 Python 路径对象的兼容性验证。
• 移除了 sys.path.insert 的 hack 写法，改用标准的模块导入路径，有利于代码静态分析和维护。

---

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

---

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

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：2 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[tests/unit/test_path_security.py: 211] 弱断言建议增强** - 具体内容：在 `test_none_allowed_dirs_uses_default` 测试中，仅断言返回值为 `bool` 类型 (`isinstance(result, bool)`)。如果默认允许目录列表被误修改为空列表，导致 `/tmp` 不再被允许，该测试依然会通过（返回 False）。建议明确断言 `assert result is True`，以确保默认配置确实包含 `/tmp`。 2. **[tests/unit/test_path_security.py: 100, 103] 平台兼容性建议** - 具体内容：测试中使用了 `/etc/passwd` 等 Linux 特有的系统路径来验证“系统路径”拦截逻辑。如果 CI/CD 环境包含 Windows 或其他非 Linux 系统，这些测试可能会无法按预期触发拦截（因为 `/etc/passwd` 在 Windows 上可能不被视为敏感系统路径）。建议使用 `pytest.mark.linux` 标记这些测试，或者通过 Mock/注入配置的方式模拟敏感路径列表，以提高测试在不同平台上的通过率和准确性。 ### ✅ 良好实践 - 使用 `pytest.fixture` 管理临时目录资源，利用上下文管理器自动清理，比旧版的 `setUp/tearDown` 更简洁可靠。 - 引入 `MAX_PATH_LENGTH` 常量替代硬编码数字，使测试边界与业务逻辑保持同步。 - 增加了 `match` 参数校验异常信息，确保抛出的异常原因符合预期，提高了测试的精确度。 - 增加了对 `pathlib.Path` 对象作为输入的测试用例，增强了代码对现代 Python 路径对象的兼容性验证。 - 移除了 `sys.path.insert` 的 hack 写法，改用标准的模块导入路径，有利于代码静态分析和维护。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无明显隐患 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 15:09:47 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit c4144d9e3f into develop 2026-07-30 00:26:06 +08:00

gitea-actions bot commented 2026-07-30 00:28:25 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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