From b3663caf9b571833fa9f09770f6293256533ae7b Mon Sep 17 00:00:00 2001 From: "pre-commit-ci[bot]" <66853113+pre-commit-ci[bot]@users.noreply.github.com> Date: Fri, 7 Apr 2023 12:41:50 +0000 Subject: [PATCH] [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --- binderhub/builder.py | 14 ++++++++++---- binderhub/tests/test_build.py | 1 + 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/binderhub/builder.py b/binderhub/builder.py index 1f2453c2b..ccb0df99e 100644 --- a/binderhub/builder.py +++ b/binderhub/builder.py @@ -19,7 +19,7 @@ from tornado.iostream import StreamClosedError from tornado.log import app_log from tornado.queues import Queue -from tornado.web import Finish, authenticated, HTTPError +from tornado.web import Finish, HTTPError, authenticated from .base import BaseHandler from .build import ProgressEvent @@ -411,15 +411,21 @@ async def get(self, provider_prefix, _unescaped_spec): require_build_only = self.settings.get("require_build_only", False) build_only = False if not require_build_only: - build_only_query_parameter = self.get_query_argument(name="build_only", default="") + build_only_query_parameter = self.get_query_argument( + name="build_only", default="" + ) if build_only_query_parameter.lower() == "true": - raise HTTPError(log_message="Building but not launching is not permitted!") + raise HTTPError( + log_message="Building but not launching is not permitted!" + ) else: # Not setting a default will make the function raise an error # if the `build_only` query parameter is missing from the request build_only_query_parameter = self.get_query_argument(name="build_only") if build_only_query_parameter.lower() != "true": - raise HTTPError(log_message="The `build_only=true` query parameter is required!") + raise HTTPError( + log_message="The `build_only=true` query parameter is required!" + ) # If we're here, it means a build only deployment is required build_only = True diff --git a/binderhub/tests/test_build.py b/binderhub/tests/test_build.py index 40b4802f6..20dedbe9e 100644 --- a/binderhub/tests/test_build.py +++ b/binderhub/tests/test_build.py @@ -138,6 +138,7 @@ async def test_build_no_launch(app): r.raise_for_status() assert r.reason == "Building but not launching is not permitted!" + def _list_image_builder_pods_mock(): """Mock list of DIND pods""" mock_response = mock.MagicMock()