diff --git a/lnwallet/channel.go b/lnwallet/channel.go index 9b5421f8..cfe0c20a 100644 --- a/lnwallet/channel.go +++ b/lnwallet/channel.go @@ -998,7 +998,7 @@ func (lc *LightningChannel) ExtendRevocationWindow() (*lnwire.CommitRevocation, // AddPayment adds a new HTLC to either the local or remote HTLC log depending // on the value of 'incoming'. -func (lc *LightningChannel) AddHTLC(htlc *lnwire.HTLCAddRequest, incoming bool) error { +func (lc *LightningChannel) AddHTLC(htlc *lnwire.HTLCAddRequest, incoming bool) uint32 { pd := &PaymentDescriptor{ entryType: Add, RHash: PaymentHash(htlc.RedemptionHashes[0]), @@ -1019,7 +1019,7 @@ func (lc *LightningChannel) AddHTLC(htlc *lnwire.HTLCAddRequest, incoming bool) pd.Index = index lc.stateUpdateLog.PushBack(pd) - return nil + return index } // SettleHTLC attempts to settle an existing outstanding HTLC with an htlc diff --git a/lnwallet/channel_test.go b/lnwallet/channel_test.go index 4402b5d5..969a7294 100644 --- a/lnwallet/channel_test.go +++ b/lnwallet/channel_test.go @@ -225,15 +225,11 @@ func TestSimpleAddSettleWorkflow(t *testing.T) { // First Alice adds the outgoing HTLC to her local channel's state // update log. - if err := aliceChannel.AddHTLC(htlc, false); err != nil { - t.Fatalf("unable to add htlc to alice's channel: %v", err) - } + aliceChannel.AddHTLC(htlc, false) // Then Alice sends this wire message over to Bob who also adds this // htlc to his local state update log. - if err := bobChannel.AddHTLC(htlc, true); err != nil { - t.Fatalf("unable to add htlc bob's channel: %v", err) - } + bobChannel.AddHTLC(htlc, true) // Next alice commits this change by sending a signature message. aliceSig, bobLogIndex, err := aliceChannel.SignNextCommitment()