akovalenko/lnd.xprv
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

test: update error propagation test for new onion error change

Browse Source 
In this commit, we update the error propagation test to ensure that the
proper HTLC amount is included for the `CodeUnknownPaymentHash` error.
This commit is contained in:
Olaoluwa Osuntokun 2019-01-28 17:41:56 -08:00
parent c8518140ee
commit b1b42dcfae
No known key found for this signature in database
GPG Key ID: CE58F7F8E20FD9A2
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
lnd_test.go
View File

@ -7878,10 +7878,11 @@ out:
// Next, we'll test the case of a recognized payHash but, an incorrect // Next, we'll test the case of a recognized payHash but, an incorrect
// value on the extended HTLC. // value on the extended HTLC.
htlcAmt := lnwire.NewMSatFromSatoshis(1000)
sendReq = &lnrpc.SendRequest{ sendReq = &lnrpc.SendRequest{
PaymentHashString: hex.EncodeToString(carolInvoice.RHash), PaymentHashString: hex.EncodeToString(carolInvoice.RHash),
DestString: hex.EncodeToString(carol.PubKey[:]), DestString: hex.EncodeToString(carol.PubKey[:]),
Amt: 1000, // 10k satoshis are expected. Amt: int64(htlcAmt.ToSatoshis()), // 10k satoshis are expected.
} }
ctxt, _ = context.WithTimeout(ctxb, defaultTimeout) ctxt, _ = context.WithTimeout(ctxb, defaultTimeout)
resp, err = net.Alice.SendPaymentSync(ctxt, sendReq) resp, err = net.Alice.SendPaymentSync(ctxt, sendReq)
@ -7895,12 +7896,19 @@ out:
t.Fatalf("payment should have been rejected due to wrong " + t.Fatalf("payment should have been rejected due to wrong " +
"HTLC amount") "HTLC amount")
} }
expectedErrorCode = lnwire.CodeIncorrectPaymentAmount.String() expectedErrorCode = lnwire.CodeUnknownPaymentHash.String()
if !strings.Contains(resp.PaymentError, expectedErrorCode) { if !strings.Contains(resp.PaymentError, expectedErrorCode) {
t.Fatalf("payment should have failed due to wrong amount, "+ t.Fatalf("payment should have failed due to wrong amount, "+
"instead failed due to: %v", resp.PaymentError) "instead failed due to: %v", resp.PaymentError)
} }
// We'll also ensure that the encoded error includes the invlaid HTLC
// amount.
if !strings.Contains(resp.PaymentError, htlcAmt.String()) {
t.Fatalf("error didn't include expected payment amt of %v: "+
"%v", htlcAmt, resp.PaymentError)
}
// The balances of all parties should be the same as initially since // The balances of all parties should be the same as initially since
// the HTLC was cancelled. // the HTLC was cancelled.
assertBaseBalance() assertBaseBalance()