Fix two linter warnings as well

atst/handlers/request_new.py

@@ -6,7 +6,6 @@ from atst.forms.funding import FundingForm
 from atst.forms.readiness import ReadinessForm
 from atst.forms.review import ReviewForm
 import tornado.httputil
-from tornado.httpclient import HTTPError
 
 
 class RequestNew(BaseHandler):

script/test

@@ -6,7 +6,7 @@ set -e
 # Ensure we are in the app root directory (not the /script directory)
 cd "$(dirname "${0}")/.."
 
-pipenv run pylint --errors-only atst
+pipenv run pylint app.py atst/ tests/
 
 # Run unit tests
 pipenv run python -m pytest -s $*

tests/test_auth.py

@@ -47,7 +47,7 @@ def test_login_via_dev_endpoint(app, http_client, base_url):
 @pytest.mark.gen_test
 @pytest.mark.skip(reason="need to work out auth error user paths")
 def test_login_with_invalid_bearer_token(http_client, base_url):
-    response = yield http_client.fetch(
+    _response = yield http_client.fetch(
         base_url + "/home",
         raise_error=False,
         headers={"Cookie": "bearer-token=anything"},