summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKenneth Giusti <kgiusti@gmail.com>2017-06-05 14:07:26 -0400
committerKenneth Giusti <kgiusti@gmail.com>2017-06-05 14:14:56 -0400
commit1642071febdcb474c2a4f41dd2f7fb477eef125f (patch)
tree4b734b167716da93371fe1f0d97ffff099d4cbe6
parent8b44bfd5388dea0d6e2083c7a3502949fbe887d9 (diff)
downloadoslo-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-xoslo_messaging/tests/rpc/test_client.py8
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