fix(autoswap): use empty chanId as default when dismissing

BoltzExchange · Mar 14, 2024 · 13f2d59 · 13f2d59
1 parent 251dad7
commit 13f2d59
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/autoswap/autoswap_test.go b/autoswap/autoswap_test.go
@@ -421,10 +421,9 @@ func TestDismissedChannels(t *testing.T) {
 			},
 			reverseSwaps: []database.ReverseSwap{
 				{
-					Id:      "TEST",
-					State:   boltzrpc.SwapState_PENDING,
-					IsAuto:  true,
-					ChanIds: []lightning.ChanId{2},
+					Id:     "TEST",
+					State:  boltzrpc.SwapState_PENDING,
+					IsAuto: true,
 				},
 				{
 					Id:      "TEST1",
@@ -434,8 +433,8 @@ func TestDismissedChannels(t *testing.T) {
 				},
 			},
 			dismissed: DismissedChannels{
+				0: []string{ReasonPendingSwap},
 				1: []string{ReasonPendingSwap},
-				2: []string{ReasonPendingSwap},
 			},
 		},
 		{

diff --git a/autoswap/config.go b/autoswap/config.go
@@ -88,6 +88,9 @@ type DismissedChannels map[lightning.ChanId][]string
 type ChannelLimits map[lightning.ChanId]uint64
 
 func (dismissed DismissedChannels) addChannels(chanIds []lightning.ChanId, reason string) {
+	if chanIds == nil {
+		chanIds = []lightning.ChanId{0}
+	}
 	for _, chanId := range chanIds {
 		if !slices.Contains(dismissed[chanId], reason) {
 			dismissed[chanId] = append(dismissed[chanId], reason)