diff options
author | Kenneth Giusti <kgiusti@gmail.com> | 2017-06-05 14:07:26 -0400 |
---|---|---|
committer | Kenneth Giusti <kgiusti@gmail.com> | 2017-06-05 14:14:56 -0400 |
commit | 1642071febdcb474c2a4f41dd2f7fb477eef125f (patch) | |
tree | 4b734b167716da93371fe1f0d97ffff099d4cbe6 | |
parent | 8b44bfd5388dea0d6e2083c7a3502949fbe887d9 (diff) | |
download | oslo-messaging-1642071febdcb474c2a4f41dd2f7fb477eef125f.tar.gz |
Fix serializer tests
The serializer tests occasionally fail since the order of a dict() is
not guaranteed. Explicitly compute the serializer instead of
pre-computing it.
Change-Id: I74f8fa227e6508491b5982fe7e2841683724540c
-rwxr-xr-x | oslo_messaging/tests/rpc/test_client.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/oslo_messaging/tests/rpc/test_client.py b/oslo_messaging/tests/rpc/test_client.py index 6123087..e955167 100755 --- a/oslo_messaging/tests/rpc/test_client.py +++ b/oslo_messaging/tests/rpc/test_client.py @@ -348,13 +348,13 @@ class TestSerializer(test_utils.BaseTestCase): serializer.deserialize_entity = mock.Mock() serializer.serialize_context = mock.Mock() + def _stub(ctxt, arg): + return 's' + arg + msg = dict(method='foo', args=dict()) - expected_side_effect = [] for k, v in self.args.items(): msg['args'][k] = 's' + v - expected_side_effect.append('s' + v) - - serializer.serialize_entity.side_effect = expected_side_effect + serializer.serialize_entity.side_effect = _stub if self.call: serializer.deserialize_entity.return_value = 'd' + self.retval |