Merge "Make remote error handling more robust"
This commit is contained in:
commit
987f1502b6
|
@ -36,7 +36,7 @@ class NoUniqueMatch(Base):
|
|||
|
||||
class RemoteError(Base):
|
||||
def __init__(self, message=None, code=None, type=None, errors=None,
|
||||
request_id=None):
|
||||
request_id=None, **ignore):
|
||||
err_message = self._get_error_message(message, type, errors)
|
||||
self.message = err_message
|
||||
self.code = code
|
||||
|
|
|
@ -54,3 +54,10 @@ class RemoteErrorTestCase(base.TestCase):
|
|||
self.response_dict['type'] = expected_msg
|
||||
remote_err = exceptions.RemoteError(**self.response_dict)
|
||||
self.assertEqual(expected_msg, remote_err.message)
|
||||
|
||||
def test_get_error_message_with_unknown_response(self):
|
||||
expected_msg = 'invalid_object'
|
||||
self.response_dict['message'] = expected_msg
|
||||
self.response_dict['unknown'] = 'fake'
|
||||
remote_err = exceptions.RemoteError(**self.response_dict)
|
||||
self.assertEqual(expected_msg, remote_err.message)
|
||||
|
|
Loading…
Reference in New Issue