From c33bbb2639cd56fc1d03f9cea80a3d5b0414c5c0 Mon Sep 17 00:00:00 2001 From: Rusty Russell Date: Wed, 9 May 2018 14:33:52 +0930 Subject: [PATCH] test_lightningd.py: wait longer test_permfail for l2 to notice new blocks. Reproduced this failure locally: l2 hadn't seen the block yet. Timeout was too aggressive. Signed-off-by: Rusty Russell --- tests/test_lightningd.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/tests/test_lightningd.py b/tests/test_lightningd.py index e419b3b16..d1f902899 100644 --- a/tests/test_lightningd.py +++ b/tests/test_lightningd.py @@ -1527,7 +1527,6 @@ class LightningDTests(BaseLightningDTests): wait_for(lambda: p.rpc.listpeers(l1.info['id'])['peers'][0]['channels'][0]['status'][1] == 'ONCHAIN:Tracking mutual close transaction') l1.daemon.wait_for_logs([' to ONCHAIN'] * num_peers) - @flaky @unittest.skipIf(not DEVELOPER, "needs DEVELOPER=1") def test_permfail(self): l1, l2 = self.connect() @@ -1583,7 +1582,7 @@ class LightningDTests(BaseLightningDTests): bitcoind.generate_block(95) wait_forget_channels(l1) - wait_for(lambda: l2.rpc.listpeers(l1.info['id'])['peers'][0]['channels'][0]['status'] == ['ONCHAIN:Tracking our own unilateral close', 'ONCHAIN:All outputs resolved: waiting 5 more blocks before forgetting channel'], timeout=1) + wait_for(lambda: l2.rpc.listpeers(l1.info['id'])['peers'][0]['channels'][0]['status'] == ['ONCHAIN:Tracking our own unilateral close', 'ONCHAIN:All outputs resolved: waiting 5 more blocks before forgetting channel']) # Now, 100 blocks l2 should be done. bitcoind.generate_block(5)