Skip to content

Commit 2b07d0b

Browse files
rustyrussellcdecker
authored andcommitted
pytest: fix test_channel_lease_unilat_closes flake
We fail waiting for 'Resolved OUR_UNILATERAL/DELAYED_OUTPUT_TO_US by our proposal OUR_DELAYED_RETURN_TO_WALLET' because we close *two* channels, but only wait for one transaction before mining a block. This means we might only have one, and we immediately mine the next wait_for_mempool=1, so the OUR_DELAYED_RETURN_TO_WALLET isn't mined. Signed-off-by: Rusty Russell <[email protected]>
1 parent 94e3e2f commit 2b07d0b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tests/test_closing.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -910,7 +910,7 @@ def test_channel_lease_unilat_closes(node_factory, bitcoind):
910910
l3.rpc.close(l2.info['id'], 1, force_lease_closed=True)
911911

912912
# Wait til to_self_delay expires, l1 should claim to_local back
913-
bitcoind.generate_block(10, wait_for_mempool=1)
913+
bitcoind.generate_block(10, wait_for_mempool=2)
914914
l1.daemon.wait_for_log('Broadcasting OUR_DELAYED_RETURN_TO_WALLET')
915915
bitcoind.generate_block(1, wait_for_mempool=1)
916916
l1.daemon.wait_for_log('Resolved OUR_UNILATERAL/DELAYED_OUTPUT_TO_US by our proposal OUR_DELAYED_RETURN_TO_WALLET')

0 commit comments

Comments
 (0)