refactor: fix a few details to please flake8

Author: consideRatio

batchspawner/__init__.py

@@ -1,3 +1,3 @@
-from .batchspawner import *
-from ._version import __version__, version_info
-from . import api
+from . import api  # noqa
+from ._version import __version__, version_info  # noqa
+from .batchspawner import *  # noqa

batchspawner/singleuser.py

@@ -27,9 +27,9 @@ def main(argv=None):
     if hub_auth.client_ca:
         kwargs["verify"] = hub_auth.client_ca
 
-    r = requests.post(
+    requests.post(
         url,
-        headers={"Authorization": f"token {hub_auth.api_token}"},
+        headers=headers,
         json={"port": port},
         **kwargs,
     )

batchspawner/tests/conftest.py

@@ -2,9 +2,9 @@
 
 # We only use "db" and "io_loop", but we also need event_loop which is used by
 # io_loop to be available with jupyterhub 1+.
-from jupyterhub.tests.conftest import db, io_loop
+from jupyterhub.tests.conftest import db, io_loop  # noqa
 
 try:
-    from jupyterhub.tests.conftest import event_loop
+    from jupyterhub.tests.conftest import event_loop  # noqa
 except:
     pass

batchspawner/tests/test_spawners.py

@@ -131,7 +131,7 @@ def test_submit_failure(db, io_loop):
     spawner = new_spawner(db=db)
     assert spawner.get_state() == {}
     spawner.batch_submit_cmd = "cat > /dev/null; true"
-    with pytest.raises(RuntimeError) as e_info:
+    with pytest.raises(RuntimeError):
         io_loop.run_sync(spawner.start, timeout=30)
     assert spawner.job_id == ""
     assert spawner.job_status == ""
@@ -142,7 +142,7 @@ def test_submit_pending_fails(db, io_loop):
     spawner = new_spawner(db=db)
     assert spawner.get_state() == {}
     spawner.batch_query_cmd = "echo xyz"
-    with pytest.raises(RuntimeError) as e_info:
+    with pytest.raises(RuntimeError):
         io_loop.run_sync(spawner.start, timeout=30)
     status = io_loop.run_sync(spawner.query_job_status, timeout=30)
     assert status == JobStatus.NOTFOUND