Skip to content

Commit

Permalink
Merge #1423: Keep trying to receive websocket notification in test
Browse files Browse the repository at this point in the history
06c29c2 Keep trying to receive websocket notification in test (Adam Gibson)
  • Loading branch information
AdamISZ committed Jan 22, 2023
2 parents 901e106 + 06c29c2 commit d2a4551
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions jmclient/test/test_wallet_rpc.py
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,8 @@ def test_notif(self):
# create a small delay between the instruction to send
# the notification, and the checking of its receipt,
# otherwise the client will be queried before the notification
# arrived:
# arrived. We will try a few times before giving up.
self.attempt_receipt_counter = 0
d.addCallback(self.wait_to_receive)
return d

Expand All @@ -178,7 +179,13 @@ def wait_to_receive(self, res):
return d

def checkNotifs(self):
assert self.client_factory.notifs == 1
if self.attempt_receipt_counter > 10:
assert False
if not self.client_factory.notifs == 1:
jlog.info("Failed to receive notification, waiting and trying again")
self.attempt_receipt_counter += 1
d = task.deferLater(reactor, 0.2, self.checkNotifs)
return d

def fire_tx_notif(self):
self.daemon.wss_factory.sendTxNotification(self.test_tx,
Expand Down

0 comments on commit d2a4551

Please sign in to comment.