From 301a7d945a69a72bfd46deee45e15a8bae2b76ea Mon Sep 17 00:00:00 2001
From: Mehdi Abaakouk <sileht@redhat.com>
Date: Fri, 16 Dec 2016 11:12:21 +0100
Subject: [PATCH] tests: make rabbit failover failure more helpful
Change-Id: Ic50b9996e8b58d5b174b9e188d70a384240499ea
---
oslo_messaging/tests/functional/test_rabbitmq.py | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/oslo_messaging/tests/functional/test_rabbitmq.py b/oslo_messaging/tests/functional/test_rabbitmq.py
index fb81f2191..ffeaf0a92 100644
--- a/oslo_messaging/tests/functional/test_rabbitmq.py
+++ b/oslo_messaging/tests/functional/test_rabbitmq.py
@@ -138,6 +138,9 @@ class RabbitMQFailoverTests(test_utils.BaseTestCase):
self.client.client.transport._driver._reply_q_conn,
]
- for cctxt in connection_contexts:
- socket = cctxt.connection.channel.connection.sock
- self.assertEqual(port, socket.getpeername()[1])
+ ports = [cctxt.connection.channel.connection.sock.getpeername()[1]
+ for cctxt in connection_contexts]
+
+ self.assertEqual([port] * len(ports), ports,
+ "expected: %s, rpc-server: %s, rpc-client: %s, "
+ "rpc-replies: %s" % tuple([port] + ports))