qa: run flake8 for test/ (#409)

Author: blueyed

setup.cfg

@@ -2,8 +2,13 @@
 test = pytest
 
 [flake8]
-ignore = D211,E731,D401,W503
+extend-ignore = D211,E731,D401,W503
 max-line-length = 88
+per-file-ignores =
+  test/*:D1
+
+[isort]
+known_first_party = pynvim
 
 [tool:pytest]
 testpaths = test

test/conftest.py

@@ -3,6 +3,7 @@
 import textwrap
 
 import pynvim
+
 import pytest
 
 pynvim.setup_logging("test")

test/test_buffer.py

@@ -160,21 +160,24 @@ def test_get_exceptions(vim):
     except vim.error:
         pass
 
+
 def test_set_items_for_range(vim):
     vim.current.buffer[:] = ['a', 'b', 'c', 'd', 'e']
     r = vim.current.buffer.range(1, 3)
-    r[1:3] = ['foo']*3
+    r[1:3] = ['foo'] * 3
     assert vim.current.buffer[:] == ['a', 'foo', 'foo', 'foo', 'd', 'e']
 
+
 # NB: we can't easily test the effect of this. But at least run the lua
 # function sync, so we know it runs without runtime error with simple args.
 def test_update_highlights(vim):
     vim.current.buffer[:] = ['a', 'b', 'c']
     src_id = vim.new_highlight_source()
-    vim.current.buffer.update_highlights(src_id, [["Comment", 0, 0, -1], ("String", 1, 0, 1)], clear=True, async_=False)
+    vim.current.buffer.update_highlights(
+        src_id, [["Comment", 0, 0, -1], ("String", 1, 0, 1)], clear=True, async_=False
+    )
 
 
 def test_buffer_inequality(vim):
     b = vim.current.buffer
     assert not (b != vim.current.buffer)
-

test/test_client_rpc.py

@@ -4,6 +4,7 @@
 
 def test_call_and_reply(vim):
     cid = vim.channel_id
+
     def setup_cb():
         cmd = 'let g:result = rpcrequest(%d, "client-call", 1, 2, 3)' % cid
         vim.command(cmd)
@@ -20,6 +21,7 @@ def request_cb(name, args):
 
 def test_call_api_before_reply(vim):
     cid = vim.channel_id
+
     def setup_cb():
         cmd = 'let g:result = rpcrequest(%d, "client-call2", 1, 2, 3)' % cid
         vim.command(cmd)
@@ -32,6 +34,7 @@ def request_cb(name, args):
 
     vim.run_loop(request_cb, None, setup_cb)
 
+
 def test_async_call(vim):
 
     def request_cb(name, args):
@@ -52,6 +55,7 @@ def request_cb(name, args):
 
 def test_recursion(vim):
     cid = vim.channel_id
+
     def setup_cb():
         vim.vars['result1'] = 0
         vim.vars['result2'] = 0

test/test_concurrency.py

@@ -4,13 +4,13 @@
 def test_interrupt_from_another_thread(vim):
     timer = Timer(0.5, lambda: vim.async_call(lambda: vim.stop_loop()))
     timer.start()
-    assert vim.next_message() == None
+    assert vim.next_message() is None
 
 
 def test_exception_in_threadsafe_call(vim):
     # an exception in a threadsafe_call shouldn't crash the entire host
     msgs = []
-    vim.async_call(lambda: [vim.eval("3"), undefined_variable])
+    vim.async_call(lambda: [vim.eval("3"), undefined_variable])  # noqa: F821
     timer = Timer(0.5, lambda: vim.async_call(lambda: vim.stop_loop()))
     timer.start()
     vim.run_loop(None, None, err_cb=msgs.append)

test/test_decorators.py

@@ -3,7 +3,7 @@
 
 def test_command_count():
     def function():
-        "A dummy function to decorate."
+        """A dummy function to decorate."""
         return
 
     # ensure absence with default value of None

test/test_host.py

@@ -2,6 +2,7 @@
 
 from pynvim.plugin.host import Host, host_method_spec
 
+
 def test_host_clientinfo(vim):
     h = Host(vim)
     assert h._request_handlers.keys() == host_method_spec.keys()

tox.ini

@@ -18,7 +18,7 @@ deps =
   flake8-import-order
   flake8-docstrings
   pep8-naming
-commands = flake8 {posargs:pynvim}
+commands = flake8 {posargs:pynvim test}
 
 [testenv:docs]
 deps =