From 486b1b24818e696795fd28f2d686bb22c478ddcb Mon Sep 17 00:00:00 2001 From: Rusty Russell Date: Wed, 23 Mar 2022 09:31:36 +1030 Subject: [PATCH] pytest: fix flake in test_funding_fail We may not see a disconnect instantly: ``` > assert len(l2.rpc.listpeers()['peers']) == 0 E assert 1 == 0 E +1 E -0 ``` Signed-off-by: Rusty Russell --- tests/test_connection.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/test_connection.py b/tests/test_connection.py index 60fe64850..9e2b284ab 100644 --- a/tests/test_connection.py +++ b/tests/test_connection.py @@ -999,8 +999,8 @@ def test_funding_fail(node_factory, bitcoind): l1.rpc.fundchannel(l2.info['id'], int(funds / 10)) # channels disconnect on failure - assert len(l1.rpc.listpeers()['peers']) == 0 - assert len(l2.rpc.listpeers()['peers']) == 0 + wait_for(lambda: len(l1.rpc.listpeers()['peers']) == 0) + wait_for(lambda: len(l2.rpc.listpeers()['peers']) == 0) # Restart l2 without ridiculous locktime. del l2.daemon.opts['watchtime-blocks']