pytest和flake8职责分离:前者动态执行测试,后者静态检查代码,应并列作为ci质量门禁独立运行,禁止强行集成。

pytest 和 flake8 本质不兼容,硬集成会破坏各自职责
flake8 是静态代码检查工具,只看源码文本;pytest 是动态测试执行器,要 import 模块、运行函数。两者工作阶段、输入输出、失败含义完全不同。强行让 flake8 “调用 pytest” 或把测试用例当普通 Python 文件喂给 flake8,会导致 ImportError、NameError、甚至测试逻辑被误判为未使用变量(比如 assert 后面的表达式在静态分析里可能被当成无副作用语句)。
常见错误现象:flake8 test_*.py 报一堆 undefined name 'pytest' 或 unused variable 'client' —— 这不是代码有问题,是 flake8 在不该分析的地方强行分析了。
- 真正该做的:把 flake8 和 pytest 当作并列的质量门禁,各自独立运行,统一接入 CI 流程
- 不要写
flake8 --load-plugin pytest_flake8这类伪解决方案,社区已弃用且不维护 - 如果想“一次命令跑全部”,用
make check或just check封装,而不是让 flake8 承担测试职责
如何让 flake8 跳过测试文件但保留对 conftest.py 的检查
测试文件(test_*.py、*_test.py)通常含大量 fixture、assert、yield 等 flake8 不理解的模式,直接跳过最省心。但 conftest.py 是普通模块,含 fixture 定义和工具函数,应参与 lint。
推荐做法是在 .flake8 或 setup.cfg 中配置排除规则:
[flake8] ignore = E501,W503 exclude = .git,__pycache__,build,dist,venv,.venv,*.egg-info,*/migrations/*,*/tests/*,test_*.py,*_test.py # 注意:conftest.py 不在 exclude 列表里,所以仍会被检查
-
*/tests/*排除整个 tests/ 目录(含子目录中的conftest.py) - 若项目把
conftest.py放在根目录或src/下,它就不会被排除,这是预期行为 - 避免用
--extend-exclude临时加排除,CI 中易遗漏;固定写进配置更可靠
CI 中串联 flake8 和 pytest 的最小可行命令
GitHub Actions / GitLab CI / pre-commit 都支持顺序执行多个命令,失败即中断。关键是别用 && 拼接(shell 错误码可能被吞),而用明确的分步写法。
示例(GitHub Actions):
- name: Run linter run: flake8 src/ --max-line-length=88
- name: Run tests
run: pytest tests/ --tb=short -xvs
-
src/和tests/显式指定路径,避免 flake8 扫到测试文件,也避免 pytest 导入时触发 lint 报错 -
--max-line-length=88和--tb=short是实用参数差异:前者适配团队风格,后者让失败信息不刷屏 - 不用
pytest --flake8插件——它只是把 flake8 调用包装进 pytest 生命周期,没解决根本冲突,还增加调试复杂度
-
pre-commit hook 怎么同时管代码风格和测试入口
pre-commit 可以定义多个 hook,每个职责单一。flake8 和 pytest 应作为两个独立 hook 存在,而非合并成一个。
.pre-commit-config.yaml 示例片段:
- repo: https://github.com/pycqa/flake8
rev: 6.1.0
hooks:
- id: flake8
files: ^src/.*\.py$
- repo: https://www.php.cn/link/62be10fcc6aa7b1ac58ba2fedbb6ff2f
rev: v7.4.4
hooks:
- id: pytest
files: ^tests/.*.py$
注意:pytest hook 默认不运行,需加 args 触发
args: [--tb=short, -x, tests/]
-
files正则严格限定作用范围:flake8 只扫src/,pytest 只扫tests/ - pytest hook 的
args必须显式传入路径,否则它会尝试扫描整个仓库,可能导入失败 - 不要用
pytest-check或pytest-flake8这类混合型插件——它们让 hook 行为变得不可预测,出问题时难定位是 lint 失败还是测试失败
真正容易被忽略的是:测试文件里的辅助函数(比如
create_test_user())如果放在tests/utils.py,它既不是生产代码也不是测试入口,flake8 不该管,pytest 也不该直接执行——这种边界文件最好挪到src/下或明确加# noqa注释。质量工具不是越严越好,而是越准越好。 -
- id: pytest
files: ^tests/.*.py$
Python免费学习笔记(深入):立即使用
在学习笔记中,你将探索 Python 的核心概念和高级技巧!











