Move is_api_error_response out of SeamHttpClient class

Author: andrii-balitskyi

seam/client.py

@@ -61,7 +61,7 @@ def _handle_error_response(self, response: requests.Response):
         if status_code == 401:
             raise SeamHttpUnauthorizedError(request_id)
 
-        if not self._is_api_error_response(response):
+        if not is_api_error_response(response):
             response.raise_for_status()
 
         error = response.json().get("error", {})
@@ -80,30 +80,31 @@ def _handle_error_response(self, response: requests.Response):
 
         raise SeamHttpApiError(error_details, status_code, request_id)
 
-    def _is_api_error_response(self, response: requests.Response) -> bool:
-        try:
-            content_type = response.headers.get("content-type", "")
 
-            if not isinstance(content_type, str) or not content_type.startswith(
-                "application/json"
-            ):
-                return False
+def is_api_error_response(response: requests.Response) -> bool:
+    try:
+        content_type = response.headers.get("content-type", "")
 
-            data = response.json()
-        except (ValueError, requests.exceptions.JSONDecodeError):
+        if not isinstance(content_type, str) or not content_type.startswith(
+            "application/json"
+        ):
             return False
 
-        if not isinstance(data, dict):
-            return False
+        data = response.json()
+    except (ValueError, requests.exceptions.JSONDecodeError):
+        return False
 
-        error = data.get("error")
+    if not isinstance(data, dict):
+        return False
 
-        if not isinstance(error, dict):
-            return False
+    error = data.get("error")
 
-        if not isinstance(error.get("type"), str) or not isinstance(
-            error.get("message"), str
-        ):
-            return False
+    if not isinstance(error, dict):
+        return False
+
+    if not isinstance(error.get("type"), str) or not isinstance(
+        error.get("message"), str
+    ):
+        return False
 
-        return True
+    return True