pre-commit/flake8
1
0
Fork 0
mirror of https://github.com/PyCQA/flake8.git synced 2026-03-29 10:36:53 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fixes --exit-zero when empty --diff

Browse source 
fixes pycqa/flake8/merge_requests!137
This commit is contained in:
Kinya TERASAKA 2019-12-02 13:11:28 +09:00
parent 15de413f9e
commit cc037338df
2 changed files with 7 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/flake8/main/application.py
View file

@ -124,7 +124,9 @@ class Application(object):
if self.options.count:
print(self.result_count)
if not self.options.exit_zero:
if self.options.exit_zero:
raise SystemExit(self.catastrophic_failure)
else:
raise SystemExit(
(self.result_count > 0) or self.catastrophic_failure
)

21
tests/unit/test_application.py
View file

@ -23,29 +23,16 @@ def application():
return app.Application()
@pytest.mark.parametrize(
'result_count, catastrophic, exit_zero', [
(0, True, True),
(2, False, True),
(2, True, True),
]
)
def test_exit_does_not_raise(result_count, catastrophic, exit_zero,
application):
"""Verify Application.exit doesn't raise SystemExit."""
application.result_count = result_count
application.catastrophic_failure = catastrophic
application.options = options(exit_zero=exit_zero)
assert application.exit() is None
@pytest.mark.parametrize(
'result_count, catastrophic, exit_zero, value', [
(0, False, False, False),
(0, True, False, True),
(2, False, False, True),
(2, True, False, True),
(0, True, True, True),
(2, False, True, False),
(2, True, True, True),
]
)
def test_exit_does_raise(result_count, catastrophic, exit_zero, value,