openstack/python-cinderclient
Code Issues Proposed changes

Merge "When there is no error body return the HTTP reason"

Browse Source
This commit is contained in:
Jenkins 2014-04-18 19:30:36 +00:00 committed by Gerrit Code Review
commit 04eaa48eb6
2 changed files with 3 additions and 1 deletions
cinderclient
exceptions.py
tests
utils.py

3
cinderclient/exceptions.py

@ -181,4 +181,5 @@ def from_response(response, body):
return cls(code=response.status_code, message=message, details=details, return cls(code=response.status_code, message=message, details=details,
request_id=request_id) request_id=request_id)
else: else:
return cls(code=response.status_code, request_id=request_id) return cls(code=response.status_code, request_id=request_id,
message=response.reason)

1
cinderclient/tests/utils.py

@ -46,6 +46,7 @@ class TestResponse(requests.Response):
if isinstance(data, dict): if isinstance(data, dict):
self.status_code = data.get('status_code', None) self.status_code = data.get('status_code', None)
self.headers = data.get('headers', None) self.headers = data.get('headers', None)
self.reason = data.get('reason', '')
# Fake the text attribute to streamline Response creation # Fake the text attribute to streamline Response creation
self._text = data.get('text', None) self._text = data.get('text', None)
else: else: