refactor: Replacing http_requests return type hint

Liora Milbaum · Liora Milbaum · commit 550263ae4a66 · 2022-12-20T00:33:16.000+02:00
diff --git a/gitlab/client.py b/gitlab/client.py
@@ -653,7 +653,7 @@ def http_request(
         retry_transient_errors: Optional[bool] = None,
         max_retries: int = 10,
         **kwargs: Any,
-    ) -> requests.Response:
+    ) -> http_backends.DefaultResponse:
         """Make an HTTP request to the Gitlab server.
 
         Args:
@@ -749,7 +749,7 @@ def http_request(
             self._check_redirects(result.response)
 
             if 200 <= result.status_code < 300:
-                return result.response
+                return result
 
             def should_retry() -> bool:
                 if result.status_code == 429 and obey_rate_limit:
@@ -827,9 +827,10 @@ def http_get(
             GitlabParsingError: If the json data could not be parsed
         """
         query_data = query_data or {}
-        result = self.http_request(
+        response = self.http_request(
             "get", path, query_data=query_data, streamed=streamed, **kwargs
         )
+        result = response.response
 
         if (
             result.headers["Content-Type"] == "application/json"
@@ -1010,7 +1011,7 @@ def http_post(
         query_data = query_data or {}
         post_data = post_data or {}
 
-        result = self.http_request(
+        response = self.http_request(
             "post",
             path,
             query_data=query_data,
@@ -1019,6 +1020,8 @@ def http_post(
             raw=raw,
             **kwargs,
         )
+        result = response.response
+
         try:
             if result.headers.get("Content-Type", None) == "application/json":
                 json_result = result.json()
@@ -1095,7 +1098,8 @@ def http_delete(self, path: str, **kwargs: Any) -> requests.Response:
         Raises:
             GitlabHttpError: When the return code is not 2xx
         """
-        return self.http_request("delete", path, **kwargs)
+        response = self.http_request("delete", path, **kwargs)
+        return response.response
 
     @gitlab.exceptions.on_http_error(gitlab.exceptions.GitlabSearchError)
     def search(
@@ -1149,7 +1153,9 @@ def _query(
         self, url: str, query_data: Optional[Dict[str, Any]] = None, **kwargs: Any
     ) -> None:
         query_data = query_data or {}
-        result = self._gl.http_request("get", url, query_data=query_data, **kwargs)
+        response = self._gl.http_request("get", url, query_data=query_data, **kwargs)
+        result = response.response
+
         try:
             next_url = result.links["next"]["url"]
         except KeyError:
diff --git a/gitlab/http_backends/requests_backend.py b/gitlab/http_backends/requests_backend.py
@@ -27,6 +27,10 @@ def headers(self) -> CaseInsensitiveDict[str]:
     def content(self) -> bytes:
         return self._response.content
 
+    @property
+    def reason(self) -> str:
+        return self._response.reason
+
     def json(self) -> Any:
         return self._response.json()
 
