From 88078a555b1a1659f1ea537204f837966bfa75bb Mon Sep 17 00:00:00 2001 From: mprahl Date: Tue, 23 Apr 2019 13:07:42 -0400 Subject: [PATCH] Define a generic error to use in MBSResolver --- module_build_service/resolver/MBSResolver.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/module_build_service/resolver/MBSResolver.py b/module_build_service/resolver/MBSResolver.py index e7aff688..7d775ee1 100644 --- a/module_build_service/resolver/MBSResolver.py +++ b/module_build_service/resolver/MBSResolver.py @@ -48,6 +48,7 @@ class MBSResolver(GenericResolver): self.session = requests.Session() adapter = requests.adapters.HTTPAdapter(max_retries=3) self.session.mount(self.mbs_prod_url, adapter) + self._generic_error = "Failed to query MBS with query %r returned HTTP status %s" def _query_from_nsvc(self, name, stream, version=None, context=None, state="ready"): """ @@ -96,9 +97,7 @@ class MBSResolver(GenericResolver): while True: res = self.session.get(self.mbs_prod_url, params=query) if not res.ok: - raise RuntimeError("Failed to query MBS with query %r returned HTTP status %s" % - (query, res.status_code)) - break + raise RuntimeError(self._generic_error % (query, res.status_code)) data = res.json() modules_per_page = data["items"]