From 1c0d435f5e4367586632b6841504b754728dedc0 Mon Sep 17 00:00:00 2001 From: Rusty Russell Date: Fri, 6 Sep 2019 15:39:39 +0930 Subject: [PATCH] pytest: remove flaky part of test_gossip's test_gossip_no_empty_announcements This "wait_for" failed on Travis, but it's unnecessary anyway. Signed-off-by: Rusty Russell --- tests/test_gossip.py | 5 ----- 1 file changed, 5 deletions(-) diff --git a/tests/test_gossip.py b/tests/test_gossip.py index 7e15b484d..8e15eb9e5 100644 --- a/tests/test_gossip.py +++ b/tests/test_gossip.py @@ -506,11 +506,6 @@ def test_gossip_no_empty_announcements(node_factory, bitcoind): # 0x0100 = channel_announcement, which goes to l2 before l3 dies. l2.daemon.wait_for_log(r'\[IN\] 0100') - # l3 actually disconnects from l4 *and* l2! That means we never see - # the (delayed) channel_update from l4. - wait_for(lambda: not l3.rpc.listpeers(l4.info['id'])['peers'][0]['connected']) - l3.rpc.connect(l4.info['id'], 'localhost', l4.port) - # But it never goes to l1, as there's no channel_update. time.sleep(2) assert not l1.daemon.is_in_log(r'\[IN\] 0100')