diff --git a/docs/release-notes/release-notes-0.16.0.md b/docs/release-notes/release-notes-0.16.0.md index 3e54cb3de..4c6f78c01 100644 --- a/docs/release-notes/release-notes-0.16.0.md +++ b/docs/release-notes/release-notes-0.16.0.md @@ -140,6 +140,9 @@ certain large transactions](https://github.com/lightningnetwork/lnd/pull/7100). and documentation to reflect this](https://github.com/lightningnetwork/lnd/pull/7142). +* [Added missing wire tests for Warning + message](https://github.com/lightningnetwork/lnd/pull/7143). + ## `lncli` * [Add an `insecure` flag to skip tls auth as well as a `metadata` string slice flag](https://github.com/lightningnetwork/lnd/pull/6818) that allows the diff --git a/lnwire/fuzz_test.go b/lnwire/fuzz_test.go index ed7088280..4ec35e6dd 100644 --- a/lnwire/fuzz_test.go +++ b/lnwire/fuzz_test.go @@ -264,6 +264,17 @@ func FuzzError(f *testing.F) { }) } +func FuzzWarning(f *testing.F) { + f.Fuzz(func(t *testing.T, data []byte) { + // Prefix with MsgWarning. + data = prefixWithMsgType(data, MsgWarning) + + // Pass the message into our general fuzz harness for wire + // messages! + harness(t, data) + }) +} + func FuzzFundingCreated(f *testing.F) { f.Fuzz(func(t *testing.T, data []byte) { // Prefix with MsgFundingCreated. diff --git a/lnwire/message_test.go b/lnwire/message_test.go index b934b928d..f2b0892d7 100644 --- a/lnwire/message_test.go +++ b/lnwire/message_test.go @@ -259,6 +259,7 @@ func BenchmarkReadMessage(b *testing.B) { func makeAllMessages(t testing.TB, r *rand.Rand) []lnwire.Message { msgAll := []lnwire.Message{} + msgAll = append(msgAll, newMsgWarning(t, r)) msgAll = append(msgAll, newMsgInit(t, r)) msgAll = append(msgAll, newMsgError(t, r)) msgAll = append(msgAll, newMsgPing(t, r)) @@ -293,6 +294,19 @@ func makeAllMessages(t testing.TB, r *rand.Rand) []lnwire.Message { return msgAll } +func newMsgWarning(tb testing.TB, r io.Reader) *lnwire.Warning { + tb.Helper() + + msg := lnwire.NewWarning() + + _, err := r.Read(msg.ChanID[:]) + require.NoError(tb, err, "unable to generate chan id") + + msg.Data = createExtraData(tb, r) + + return msg +} + func newMsgInit(t testing.TB, r io.Reader) *lnwire.Init { t.Helper()