Pytest 代码审查指南

快速参考

审查关卡（Review gates）

请按顺序执行。在每一项适用的关卡通过之前，不得指出与 pytest 相关的具体问题。

1. 限定文件范围 — 通过条件： 明确列出所有将被引用的 test_*.py 文件及任意涉及的 conftest.py；对列表之外的文件不提出任何问题。
2. 异步 vs 同步 — 通过条件： 对每个限定范围内的文件，注明其是否使用 async def test_* 或 await；若存在，则在批评异步用法前，须先查阅 references/async-testing.md。
3. Fixture — 通过条件： 若共享初始化逻辑重要，请明确指出所涉 conftest.py 的路径，或声明无相关文件；对于 yield 类型的 fixture，须确认其已包含 finally 块中的清理逻辑，方可认定存在资源泄漏。
4. patch / mock — 通过条件： 针对任何关于 patch 或 mock 的批评意见，必须提供该符号被调用处（call site）的完整导入路径；若 mock 是本次审查的核心内容，则需同步查阅 references/mocking.md。
5. 问题描述 — 通过条件： 每个问题均需包含具体文件路径与行号（或测试节点 ID），不可仅复述通用规则。

审查清单

• 测试函数针对被测异步代码应定义为 async def test_*
• 异步依赖应使用 AsyncMock，而非 Mock
• 所有异步 mock 和协程均已被正确 await
• 共享初始化逻辑应提取至 conftest.py 中定义的 fixture
• Fixture 的作用域设置恰当（function / class / module / session）
• yield 类型的 fixture 在 finally 块中实现了正确的清理逻辑
• 使用 @pytest.mark.parametrize 处理结构相似的测试用例
• 多个测试函数之间不存在重复的测试逻辑
• Mock 能准确追踪调用（例如使用 assert_called_once_with）
• patch() 的目标位置正确（即符号被使用的位置，而非定义位置）
• 未对本应被测试的内部实现进行 mock
• 测试相互隔离（测试间无共享的可变状态）

何时查阅参考文档

• 审查异步测试函数 → 查阅 async-testing.md
• 审查 fixture 或 conftest.py → 查阅 fixtures.md
• 审查结构相似的测试用例 → 查阅 parametrize.md
• 审查 mock 和 patch → 查阅 mocking.md

审查关键问题

1. 所有异步函数是否均由 async def test_* 形式的测试函数覆盖？
2. fixture 的作用域设置是否合理，且具备适当的清理机制？
3. 结构相似的测试用例是否可通过 parametrize 实现去重？
4. mock 是否能准确追踪调用，并应用于正确的位置？