Skip to content

Commit f10c7bf

Browse files
authored
Merge pull request #15566 from rabbitmq/fix-python_SUITE-test_broadcast-case-in-rabbitmq_stomp
python_SUITE: Wait for bindings to be active before publishing messages
2 parents 0eb579f + 88b1684 commit f10c7bf

2 files changed

Lines changed: 14 additions & 1 deletion

File tree

deps/rabbitmq_stomp/test/python_SUITE_data/src/parsing.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
import time
1313
import sys
1414
import os
15+
import test_util
1516

1617
def connect(cnames):
1718
''' Decorator that creates stomp connections and issues CONNECT '''
@@ -212,7 +213,11 @@ def test_broadcast(self):
212213
for cd in [self.cd1, self.cd2]:
213214
cd.sendall(subscribe.encode('utf-8'))
214215

215-
time.sleep(0.1)
216+
bindings_count = 0
217+
while bindings_count != 2:
218+
time.sleep(0.1)
219+
output = test_util.rabbitmqctl_output(['list_bindings'])
220+
bindings_count = output.split().count('da9d4779')
216221

217222
cmd = ('SEND\n'
218223
'content-type:text/plain\n'

deps/rabbitmq_stomp/test/python_SUITE_data/src/test_util.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,3 +50,11 @@ def rabbitmqctl(args):
5050
cmdline = [ctl, '-n', os.getenv('RABBITMQ_NODENAME')]
5151
cmdline.extend(args)
5252
subprocess.check_call(cmdline)
53+
54+
def rabbitmqctl_output(args):
55+
ctl = os.getenv('RABBITMQCTL')
56+
cmdline = [ctl, '-n', os.getenv('RABBITMQ_NODENAME')]
57+
cmdline.extend(args)
58+
output = subprocess.check_output(cmdline).decode('utf-8')
59+
print(output)
60+
return output

0 commit comments

Comments
 (0)