highperfocused/lnd
1
0
Fork 0
mirror of https://github.com/lightningnetwork/lnd.git synced 2025-11-10 06:07:16 +01:00
Code Issues Packages Projects Releases Wiki Activity

chancloser: use separate broadcast channels

Browse Source 
This doesn't change the behavior but makes the use of the broadcast
channels more clear.
This commit is contained in:
Oliver Gugger
2025-07-08 16:20:53 +02:00
parent 899131f7be
commit 8023583414
1 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
lnwallet/chancloser/chancloser_test.go
View File

@@ -509,7 +509,10 @@ func TestTaprootFastClose(t *testing.T) {
aliceChan := newMockTaprootChan(t, true)
bobChan := newMockTaprootChan(t, false)
broadcastSignal := make(chan struct{}, 2)
// We'll create two distinct broadcast signals to ensure that each party
// broadcasts at the correct time.
aliceBroadcast := make(chan struct{}, 1)
bobBroadcast := make(chan struct{}, 1)
idealFee := chainfee.SatPerKWeight(506)
@@ -520,7 +523,7 @@ func TestTaprootFastClose(t *testing.T) {
Channel: aliceChan,
MusigSession: newMockMusigSession(),
BroadcastTx: func(_ *wire.MsgTx, _ string) error {
broadcastSignal <- struct{}{}
aliceBroadcast <- struct{}{}
return nil
},
MaxFee: chainfee.SatPerKWeight(1000),
@@ -538,7 +541,7 @@ func TestTaprootFastClose(t *testing.T) {
MusigSession: newMockMusigSession(),
MaxFee: chainfee.SatPerKWeight(1000),
BroadcastTx: func(_ *wire.MsgTx, _ string) error {
broadcastSignal <- struct{}{}
bobBroadcast <- struct{}{}
return nil
},
FeeEstimator: &SimpleCoopFeeEstimator{},
@@ -591,7 +594,7 @@ func TestTaprootFastClose(t *testing.T) {
// At this point, Bob has accepted the offer, so he can broadcast the
// closing transaction, and considers the channel closed.
_, err = lnutils.RecvOrTimeout(broadcastSignal, time.Second*1)
_, err = lnutils.RecvOrTimeout(bobBroadcast, time.Second*1)
require.NoError(t, err)
// Bob's fee proposal should exactly match Alice's initial fee.
@@ -623,7 +626,7 @@ func TestTaprootFastClose(t *testing.T) {
aliceClosingSigned = oClosingSigned.UnwrapOrFail(t)
// Alice should now also broadcast her closing transaction.
_, err = lnutils.RecvOrTimeout(broadcastSignal, time.Second*1)
_, err = lnutils.RecvOrTimeout(aliceBroadcast, time.Second*1)
require.NoError(t, err)
// Finally, Bob will process Alice's echo message, and conclude.