Add support for error and error_description (#79)

Author: blairworkos

tests/utils/test_requests.py

@@ -42,7 +42,7 @@ def test_request_exceptions_include_expected_request_data(
         for status_code, exception in STATUS_CODE_TO_EXCEPTION_MAPPING.items():
             mock_request_method(
                 "get",
-                {"message": response_message,},
+                {"message": response_message},
                 status_code,
                 headers={"X-Request-ID": request_id},
             )
@@ -56,6 +56,32 @@ def test_request_exceptions_include_expected_request_data(
                 # This'll fail for sure here but... just using the nice error that'd come up
                 assert ex.__class__ == exception
 
+    def test_bad_request_exceptions_include_expected_request_data(
+        self, mock_request_method
+    ):
+        request_helper = RequestHelper()
+
+        request_id = "request-123"
+        error = "example_error"
+        error_description = "Example error description"
+
+        mock_request_method(
+            "get",
+            {"error": error, "error_description": error_description},
+            400,
+            headers={"X-Request-ID": request_id},
+        )
+
+        try:
+            request_helper.request("bad_place")
+        except ServerException as ex:
+            assert ex.request_id == request_id
+            assert ex.error == error
+            assert ex.error_description == error_description
+        except Exception as ex:
+            # This'll fail for sure here but... just using the nice error that'd come up
+            assert ex.__class__ == BadRequestException
+
     def test_request_bad_body_raises_expected_exception_with_request_data(
         self, mock_request_method
     ):

workos/exceptions.py

@@ -4,7 +4,7 @@ class ConfigurationException(Exception):
 
 # Request related exceptions
 class BaseRequestException(Exception):
-    def __init__(self, response, message=None, error=None):
+    def __init__(self, response, message=None, error=None, error_description=None):
         super(BaseRequestException, self).__init__(message)
 
         self.error = error
@@ -35,6 +35,9 @@ def __str__(self):
         if self.error is not None:
             exception += ", error=%s" % self.error
 
+        if self.error_description is not None:
+            exception += ", error_description=%s" % self.error_description
+
         return exception + ")"
 
 

workos/utils/request.py

@@ -79,7 +79,10 @@ def request(
             elif status_code == 403:
                 raise AuthorizationException(response)
             error = response_json.get("error")
-            raise BadRequestException(response, error=error)
+            error_description = response_json.get("error_description")
+            raise BadRequestException(
+                response, error=error, error_description=error_description
+            )
         elif status_code >= 500 and status_code < 600:
             raise ServerException(response)