summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlan Conway <aconway@apache.org>2012-02-17 14:17:46 +0000
committerAlan Conway <aconway@apache.org>2012-02-17 14:17:46 +0000
commitd8b41424f905ba6ab7a83c3da4e658ce4dab9f52 (patch)
tree482ebb2abfcbd30ab5fbd33c9529ed3b780d9e7d
parentad62fb6af2adf01d4815bf2612c1a605ecd2d200 (diff)
downloadqpid-python-d8b41424f905ba6ab7a83c3da4e658ce4dab9f52.tar.gz
QPID-3603: Fix faulty tests in cluster_tests.py.
git-svn-id: https://svn.apache.org/repos/asf/qpid/branches/qpid-3603-7@1245559 13f79535-47bb-0310-9956-ffa450edef68
-rwxr-xr-xqpid/cpp/src/tests/cluster_tests.py13
1 files changed, 6 insertions, 7 deletions
diff --git a/qpid/cpp/src/tests/cluster_tests.py b/qpid/cpp/src/tests/cluster_tests.py
index 36f5b24c02..354f9d540b 100755
--- a/qpid/cpp/src/tests/cluster_tests.py
+++ b/qpid/cpp/src/tests/cluster_tests.py
@@ -241,6 +241,7 @@ acl deny all all
retry(lambda: find_in_file("brokerLinkUp", qp.outfile("out")))
broker1.ready()
broker2.ready()
+ qr.wait()
def test_queue_cleaner(self):
""" Regression test to ensure that cleanup of expired messages works correctly """
@@ -1220,25 +1221,23 @@ class LongTests(BrokerTest):
receiver = NumberedReceiver(cluster[0])
receiver.start()
- senders = [NumberedSender(cluster[0]) for i in range(1,3)]
- for s in senders:
- s.start()
+ sender = NumberedSender(cluster[0])
+ sender.start()
# Wait for senders & receiver to get up and running
- retry(lambda: receiver.received > 2*senders)
+ retry(lambda: receiver.received > 10)
# Kill original brokers, start new ones for the duration.
endtime = time.time() + self.duration();
i = 0
while time.time() < endtime:
- for s in senders: s.sender.assert_running()
+ sender.sender.assert_running()
receiver.receiver.assert_running()
for b in cluster[i:]: b.ready() # Check if any broker crashed.
cluster[i].kill()
i += 1
b = cluster.start(expect=EXPECT_EXIT_FAIL)
time.sleep(5)
- for s in senders:
- s.stop()
+ sender.stop()
receiver.stop()
for i in range(i, len(cluster)): cluster[i].kill()