From dee2371738f390050c89f4d601d9199c76ca272c Mon Sep 17 00:00:00 2001 From: f321x Date: Thu, 8 Jan 2026 14:48:44 +0100 Subject: [PATCH] test_lnpeer: use PeerInTests instead if Peer ec65c53 replaces the usage of `PeerInTest` with `Peer` in test_lnpeer.py. PeerInTests sets `Peer.DELAY_INC_MSG_PROCESSING_SLEEP` to 0 so all incoming messages get processed immediately. Because `Peer` instead of `TestInPeer` was used the delay caused `test_reestablish_with_old_state` to fail regularly because bob receives the old channel state and kills the OldTaskGroup of the unittest with GracefulDisconnect before Alice processed the answer of Bob and is still in ChannelState.REESTABLISHING. ``` FAILED tests/test_lnpeer.py::TestPeerDirect::test_reestablish_with_old_state - AssertionError: != ``` --- tests/test_lnpeer.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/test_lnpeer.py b/tests/test_lnpeer.py index afb6f0a77..6107a8b49 100644 --- a/tests/test_lnpeer.py +++ b/tests/test_lnpeer.py @@ -535,7 +535,7 @@ class TestPeer(ElectrumTestCase): # create peers for ab in channels.keys(): - peers[ab] = Peer(workers[ab[0]], keys[ab[1]].pubkey, transports[ab]) + peers[ab] = PeerInTests(workers[ab[0]], keys[ab[1]].pubkey, transports[ab]) # add peers to workers for a, w in workers.items():