Browse Source

pytest: Stabilize test_penalty_htlc_tx_timeout

lightning-5 can sometimes see itself sweeping the unilateral output resulting
in this weird line:

```log
HTLC already resolved by SELF when we found preimage
```
travis-experimental
Christian Decker 4 years ago
committed by Rusty Russell
parent
commit
ee0ee046ff
  1. 43
      tests/test_closing.py

43
tests/test_closing.py

@ -864,22 +864,33 @@ def test_penalty_htlc_tx_timeout(node_factory, bitcoind, chainparams):
# We track channel balances, to verify that accounting is ok. # We track channel balances, to verify that accounting is ok.
coin_mvt_plugin = os.path.join(os.getcwd(), 'tests/plugins/coin_movements.py') coin_mvt_plugin = os.path.join(os.getcwd(), 'tests/plugins/coin_movements.py')
l1, l2, l3, l4, l5 = node_factory.get_nodes(5, l1, l2, l3, l4, l5 = node_factory.get_nodes(
opts=[{'disconnect': ['-WIRE_UPDATE_FULFILL_HTLC'], 5,
'may_reconnect': True, opts=[
'dev-no-reconnect': None}, {
{'plugin': coin_mvt_plugin, 'disconnect': ['-WIRE_UPDATE_FULFILL_HTLC'],
'dev-no-reconnect': None, 'may_reconnect': True,
'may_reconnect': True, 'dev-no-reconnect': None,
'allow_broken_log': True}, }, {
{'plugin': coin_mvt_plugin, 'plugin': coin_mvt_plugin,
'dev-no-reconnect': None, 'dev-no-reconnect': None,
'may_reconnect': True, 'may_reconnect': True,
'allow_broken_log': True}, 'allow_broken_log': True,
{'dev-no-reconnect': None}, }, {
{'disconnect': ['-WIRE_UPDATE_FULFILL_HTLC'], 'plugin': coin_mvt_plugin,
'may_reconnect': True, 'dev-no-reconnect': None,
'dev-no-reconnect': None}]) 'may_reconnect': True,
'allow_broken_log': True,
}, {
'dev-no-reconnect': None,
}, {
'disconnect': ['-WIRE_UPDATE_FULFILL_HTLC'],
'may_reconnect': True,
'dev-no-reconnect': None,
'allow_broken_log': True,
}
]
)
node_factory.join_nodes([l1, l2, l3, l4], wait_for_announce=True) node_factory.join_nodes([l1, l2, l3, l4], wait_for_announce=True)
node_factory.join_nodes([l3, l5], wait_for_announce=True) node_factory.join_nodes([l3, l5], wait_for_announce=True)

Loading…
Cancel
Save