diff options
author | Zuul <zuul@review.opendev.org> | 2020-07-06 16:47:58 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2020-07-06 16:47:58 +0000 |
commit | 621872561087695ec8bb5f93d00384b691b0c366 (patch) | |
tree | 3de21f95718330593c992ab089307771ef5c4e50 | |
parent | a08029a44ea4872d1c7afd51311a15be4f6a257f (diff) | |
parent | a4855c544cac35e92ce7c20143536ca7a3bb847a (diff) | |
download | ironic-python-agent-621872561087695ec8bb5f93d00384b691b0c366.tar.gz |
Merge "Fix serializing ironic-lib exceptions"
-rw-r--r-- | ironic_python_agent/encoding.py | 12 | ||||
-rw-r--r-- | ironic_python_agent/tests/unit/test_encoding.py | 21 | ||||
-rw-r--r-- | releasenotes/notes/lib-exc-41ee122eb4a04bc4.yaml | 6 |
3 files changed, 39 insertions, 0 deletions
diff --git a/ironic_python_agent/encoding.py b/ironic_python_agent/encoding.py index 715d1341..b1b44cd1 100644 --- a/ironic_python_agent/encoding.py +++ b/ironic_python_agent/encoding.py @@ -15,6 +15,8 @@ import json import uuid +from ironic_lib import exception as lib_exc + class Serializable(object): """Base class for things that can be serialized.""" @@ -43,6 +45,14 @@ class SerializableComparable(Serializable): return self.serialize() != other.serialize() +def serialize_lib_exc(exc): + """Serialize an ironic-lib exception.""" + return {'type': exc.__class__.__name__, + 'code': exc.code, + 'message': str(exc), + 'details': ''} + + class RESTJSONEncoder(json.JSONEncoder): """A slightly customized JSON encoder.""" def encode(self, o): @@ -68,5 +78,7 @@ class RESTJSONEncoder(json.JSONEncoder): return o.serialize() elif isinstance(o, uuid.UUID): return str(o) + elif isinstance(o, lib_exc.IronicException): + return serialize_lib_exc(o) else: return json.JSONEncoder.default(self, o) diff --git a/ironic_python_agent/tests/unit/test_encoding.py b/ironic_python_agent/tests/unit/test_encoding.py index 64cd9b63..862cd9e8 100644 --- a/ironic_python_agent/tests/unit/test_encoding.py +++ b/ironic_python_agent/tests/unit/test_encoding.py @@ -12,6 +12,10 @@ # License for the specific language governing permissions and limitations # under the License. +import json + +from ironic_lib import exception as lib_exc + from ironic_python_agent import encoding from ironic_python_agent.tests.unit import base @@ -59,3 +63,20 @@ class TestSerializableComparable(base.IronicAgentTest): # Ensure __hash__ is None obj = SerializableComparableTesting('hello', 'world') self.assertIsNone(obj.__hash__) + + +class TestEncoder(base.IronicAgentTest): + + encoder = encoding.RESTJSONEncoder() + + def test_encoder(self): + expected = {'jack': 'hello', 'jill': 'world'} + obj = SerializableTesting('hello', 'world') + self.assertEqual(expected, json.loads(self.encoder.encode(obj))) + + def test_ironic_lib(self): + obj = lib_exc.InstanceDeployFailure(reason='boom') + encoded = json.loads(self.encoder.encode(obj)) + self.assertEqual(500, encoded['code']) + self.assertEqual('InstanceDeployFailure', encoded['type']) + self.assertIn('boom', encoded['message']) diff --git a/releasenotes/notes/lib-exc-41ee122eb4a04bc4.yaml b/releasenotes/notes/lib-exc-41ee122eb4a04bc4.yaml new file mode 100644 index 00000000..cb3e8c2f --- /dev/null +++ b/releasenotes/notes/lib-exc-41ee122eb4a04bc4.yaml @@ -0,0 +1,6 @@ +--- +fixes: + - | + Fixes serializing exceptions originating from ironic-lib. Previously an + attempt to do so would result in a ``TypeError``, for example: + `Object of type 'InstanceDeployFailure' is not JSON serializable`. |