From 17aa047b17a69e502df4d9f843d88769f5a56d2c Mon Sep 17 00:00:00 2001 From: Rusty Russell Date: Wed, 18 Jan 2023 15:34:28 +1030 Subject: [PATCH] pytest: fix output order assumption in test_setchannel_all Hash table coming up makes it random! Signed-off-by: Rusty Russell --- tests/test_pay.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/test_pay.py b/tests/test_pay.py index 1d9686844..7d478caaf 100644 --- a/tests/test_pay.py +++ b/tests/test_pay.py @@ -2413,7 +2413,10 @@ def test_setchannel_all(node_factory, bitcoind): wait_for(lambda: [c['base_fee_millisatoshi'] for c in l1.rpc.listchannels(scid3)['channels']] == [0xDEAD, DEF_BASE]) wait_for(lambda: [c['fee_per_millionth'] for c in l1.rpc.listchannels(scid3)['channels']] == [0xBEEF, DEF_PPM]) + # Don't assume order! assert len(result['channels']) == 2 + if result['channels'][0]['peer_id'] == l3.info['id']: + result['channels'] = [result['channels'][1], result['channels'][0]] assert result['channels'][0]['peer_id'] == l2.info['id'] assert result['channels'][0]['short_channel_id'] == scid2 assert result['channels'][0]['fee_base_msat'] == 0xDEAD