diff options
| author | Eric N. Vander Weele <ericvw@gmail.com> | 2020-05-11 23:43:39 -0400 |
|---|---|---|
| committer | Anthony Sottile <asottile@umich.edu> | 2020-05-11 22:17:52 -0700 |
| commit | e6d8a90e5d6e9f3156a4adfcdb65a9fb1d12788f (patch) | |
| tree | f1e7b606906bb9bf4cacb1f65caad3621a7bcd08 /tests/integration | |
| parent | b4d285019210ccdb5d526e64c281db64e5316d6c (diff) | |
| download | flake8-e6d8a90e5d6e9f3156a4adfcdb65a9fb1d12788f.tar.gz | |
options: Forward `--output-file` to be reparsed for BaseFormatter
This fixes a regression introduced in daca2c8 and b14d47b.
The --output-file` option was consumed by the preliminary option parser.
However, the `BaseFormatter` class needs the option for setting the
output filename. This special cases this option to ensure it gets
re-parsed and respected when specified on the CLI.
Diffstat (limited to 'tests/integration')
| -rw-r--r-- | tests/integration/test_main.py | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/tests/integration/test_main.py b/tests/integration/test_main.py index b67992b..54254ef 100644 --- a/tests/integration/test_main.py +++ b/tests/integration/test_main.py @@ -246,3 +246,17 @@ def test_file_not_found(tmpdir, capsys): out, err = capsys.readouterr() assert out.startswith("i-do-not-exist:0:1: E902") assert err == "" + + +def test_output_file(tmpdir, capsys): + """Ensure that --output-file is honored.""" + tmpdir.join('t.py').write('import os\n') + + with tmpdir.as_cwd(): + _call_main(['t.py', '--output-file=f'], retv=1) + + out, err = capsys.readouterr() + assert out == err == "" + + expected = "t.py:1:1: F401 'os' imported but unused\n" + assert tmpdir.join('f').read() == expected |