Pass responses appropriately to errors (swaggerpy)

swaggerpy/response.py

@@ -26,9 +26,17 @@ def handle_response_errors(e):
     :raises HTTPError: :class: `swaggerpy.exception.HTTPError`
     """
     args = list(e.args)
-    if hasattr(e, 'response') and hasattr(e.response, 'text'):
-        args[0] += (' : ' + e.response.text)
-    raise HTTPError(*args), None, sys.exc_info()[2]
+    kwargs = {}
+
+    if hasattr(e, 'response'):
+        kwargs['response'] = e.response
+        if hasattr(e.response, 'text'):
+            args[0] += (' : ' + e.response.text)
+
+    if hasattr(e, 'request'):
+        kwargs['request'] = e.request
+
+    raise HTTPError(*args, **kwargs), None, sys.exc_info()[2]
 
 
 class HTTPFuture(object):

tests/resource_response_test.py

@@ -55,16 +55,17 @@
 """
 
 import datetime
-from swaggerpy.compat import json
 import unittest
-from mock import Mock
+from swaggerpy.compat import json
 
 import httpretty
 from dateutil.tz import tzutc
+from mock import Mock
 
 from swaggerpy.client import SwaggerClient
 from swaggerpy.exception import CancelledError, HTTPError
 from swaggerpy.processors import SwaggerError
+from swaggerpy.response import handle_response_errors
 from swaggerpy.response import HTTPFuture
 
 
@@ -87,6 +88,23 @@ def test_cancelled_returns_false_if_called_before_cancel(self):
         self.assertFalse(self.future.cancelled())
 
 
+class HandleResponseErrorsTest(unittest.TestCase):
+
+    def test_pass_response_and_request(self):
+        request = Mock()
+        response = Mock(text='{"foo": "bar"}')
+        response.json.return_value = {'foo': 'bar'}
+        error = Mock(
+            args=('400 Bad Request',), response=response, request=request
+        )
+        try:
+            handle_response_errors(error)
+        except HTTPError as e:
+            self.assertEqual(e.response.json(), {'foo': 'bar'})
+            self.assertEqual(e.response, response)
+            self.assertEqual(e.request, request)
+
+
 class ResourceResponseTest(unittest.TestCase):
     def setUp(self):
         parameter = {