Testing...

JohnVillalovos · JohnVillalovos · commit 51653544f1df · 2021-02-22T14:05:40.000-08:00
diff --git a/gitlab/client.py b/gitlab/client.py
@@ -23,9 +23,9 @@
 import requests.utils
 
 import gitlab.config
-from gitlab.const import *  # noqa
-from gitlab.exceptions import *  # noqa
-from gitlab import utils  # noqa
+import gitlab.const
+import gitlab.exceptions
+from gitlab import utils
 from requests_toolbelt.multipart.encoder import MultipartEncoder
 
 
@@ -70,7 +70,7 @@ def __init__(
         per_page=None,
         pagination=None,
         order_by=None,
-        user_agent=USER_AGENT,
+        user_agent=gitlab.const.USER_AGENT,
     ):
 
         self._api_version = str(api_version)
@@ -227,7 +227,7 @@ def version(self):
 
         return self._server_version, self._server_revision
 
-    @on_http_error(GitlabVerifyError)
+    @gitlab.exceptions.on_http_error(gitlab.exceptions.GitlabVerifyError)
     def lint(self, content, **kwargs):
         """Validate a gitlab CI configuration.
 
@@ -247,7 +247,7 @@ def lint(self, content, **kwargs):
         data = self.http_post("/ci/lint", post_data=post_data, **kwargs)
         return (data["status"] == "valid", data["errors"])
 
-    @on_http_error(GitlabMarkdownError)
+    @gitlab.exceptions.on_http_error(gitlab.exceptions.GitlabMarkdownError)
     def markdown(self, text, gfm=False, project=None, **kwargs):
         """Render an arbitrary Markdown document.
 
@@ -272,7 +272,7 @@ def markdown(self, text, gfm=False, project=None, **kwargs):
         data = self.http_post("/markdown", post_data=post_data, **kwargs)
         return data["html"]
 
-    @on_http_error(GitlabLicenseError)
+    @gitlab.exceptions.on_http_error(gitlab.exceptions.GitlabLicenseError)
     def get_license(self, **kwargs):
         """Retrieve information about the current license.
 
@@ -288,7 +288,7 @@ def get_license(self, **kwargs):
         """
         return self.http_get("/license", **kwargs)
 
-    @on_http_error(GitlabLicenseError)
+    @gitlab.exceptions.on_http_error(gitlab.exceptions.GitlabLicenseError)
     def set_license(self, license, **kwargs):
         """Add a new license.
 
@@ -426,7 +426,7 @@ def _check_redirects(self, result):
                 # Did we end-up with an https:// URL?
                 location = item.headers.get("Location", None)
                 if location and location.startswith("https://"):
-                    raise RedirectError(REDIRECT_MSG)
+                    raise gitlab.exceptions.RedirectError(REDIRECT_MSG)
 
     def http_request(
         self,
@@ -547,13 +547,13 @@ def http_request(
                 pass
 
             if result.status_code == 401:
-                raise GitlabAuthenticationError(
+                raise gitlab.exceptions.GitlabAuthenticationError(
                     response_code=result.status_code,
                     error_message=error_message,
                     response_body=result.content,
                 )
 
-            raise GitlabHttpError(
+            raise gitlab.exceptions.GitlabHttpError(
                 response_code=result.status_code,
                 error_message=error_message,
                 response_body=result.content,
@@ -592,7 +592,7 @@ def http_get(self, path, query_data=None, streamed=False, raw=False, **kwargs):
             try:
                 return result.json()
             except Exception as e:
-                raise GitlabParsingError(
+                raise gitlab.exceptions.GitlabParsingError(
                     error_message="Failed to parse the server message"
                 ) from e
         else:
@@ -674,7 +674,7 @@ def http_post(self, path, query_data=None, post_data=None, files=None, **kwargs)
             if result.headers.get("Content-Type", None) == "application/json":
                 return result.json()
         except Exception as e:
-            raise GitlabParsingError(
+            raise gitlab.exceptions.GitlabParsingError(
                 error_message="Failed to parse the server message"
             ) from e
         return result
@@ -712,7 +712,7 @@ def http_put(self, path, query_data=None, post_data=None, files=None, **kwargs):
         try:
             return result.json()
         except Exception as e:
-            raise GitlabParsingError(
+            raise gitlab.exceptions.GitlabParsingError(
                 error_message="Failed to parse the server message"
             ) from e
 
@@ -732,7 +732,7 @@ def http_delete(self, path, **kwargs):
         """
         return self.http_request("delete", path, **kwargs)
 
-    @on_http_error(GitlabSearchError)
+    @gitlab.exceptions.on_http_error(gitlab.exceptions.GitlabSearchError)
     def search(self, scope, search, **kwargs):
         """Search GitLab resources matching the provided string.'
 
@@ -792,7 +792,7 @@ def _query(self, url, query_data=None, **kwargs):
         try:
             self._data = result.json()
         except Exception as e:
-            raise GitlabParsingError(
+            raise gitlab.exceptions.GitlabParsingError(
                 error_message="Failed to parse the server message"
             ) from e
 
