diff options
| author | Ran Benita <ran@unusedvar.com> | 2020-04-24 21:51:32 +0300 |
|---|---|---|
| committer | Ran Benita <ran@unusedvar.com> | 2020-05-12 09:29:40 +0300 |
| commit | d1534181c0bd165f354179d1ac131d874b71a81b (patch) | |
| tree | 0a46a695feba1b0eb7bf8ee61fab8cf8ae49f3ee /testing/test_setupplan.py | |
| parent | 15d5e8cd97c621cf558d154a5760519468770199 (diff) | |
| download | pytest-d1534181c0bd165f354179d1ac131d874b71a81b.tar.gz | |
pre-commit: upgrade flake8 3.7.7 -> 3.8.1
New errors:
testing/test_setupplan.py:104:15: E741 ambiguous variable name 'l'
testing/test_setupplan.py:107:15: E741 ambiguous variable name 'l'
extra/get_issues.py:48:29: E741 ambiguous variable name 'l'
testing/test_error_diffs.py:270:32: E741 ambiguous variable name 'l'
Not so sure about it but easier to just fix.
But more importantly, is a large amount of typing-related issues there
were fixed which necessitated noqa's which can now be removed.
Diffstat (limited to 'testing/test_setupplan.py')
| -rw-r--r-- | testing/test_setupplan.py | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/testing/test_setupplan.py b/testing/test_setupplan.py index a44474dd1..64b464b32 100644 --- a/testing/test_setupplan.py +++ b/testing/test_setupplan.py @@ -101,10 +101,14 @@ def test_show_multi_test_fixture_setup_and_teardown_same_as_setup_show(testdir): # the number and text of these lines should be identical plan_lines = [ - l for l in plan_result.stdout.lines if "SETUP" in l or "TEARDOWN" in l + line + for line in plan_result.stdout.lines + if "SETUP" in line or "TEARDOWN" in line ] show_lines = [ - l for l in show_result.stdout.lines if "SETUP" in l or "TEARDOWN" in l + line + for line in show_result.stdout.lines + if "SETUP" in line or "TEARDOWN" in line ] assert plan_lines == show_lines |