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

Merge branch 'fix/exit-zero' into 'master'

Browse source 
Fix --exit-zero when --diff is empty

See merge request pycqa/flake8!391
This commit is contained in:
Anthony Sottile 2019-12-02 06:25:27 +00:00
commit 1062be2643
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,