From ee2162b5d2fb8a616c2c0327acdf947c282759e7 Mon Sep 17 00:00:00 2001 From: eugene Date: Tue, 5 Oct 2021 15:36:24 -0400 Subject: [PATCH 1/2] htlcswitch: remove 5 second timeout for data race unit tests --- htlcswitch/switch_test.go | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/htlcswitch/switch_test.go b/htlcswitch/switch_test.go index 09c38d030..b68384500 100644 --- a/htlcswitch/switch_test.go +++ b/htlcswitch/switch_test.go @@ -3449,16 +3449,11 @@ func TestSwitchDustForwarding(t *testing.T) { ) require.NoError(t, err) - select { - case result, ok := <-carolResultChan: - require.True(t, ok) - assertFailureCode( - t, result.Error, lnwire.CodeTemporaryChannelFailure, - ) - - case <-time.After(5 * time.Second): - t.Fatal("no result arrived for carol's dust htlc") - } + result, ok := <-carolResultChan + require.True(t, ok) + assertFailureCode( + t, result.Error, lnwire.CodeTemporaryChannelFailure, + ) // Send an HTLC from Alice to Carol and assert that it is failed at the // call to SendHTLC. From 2fbc35ca26e2e9d3ca90dfb03b803b94255e0449 Mon Sep 17 00:00:00 2001 From: eugene Date: Tue, 5 Oct 2021 16:40:13 -0400 Subject: [PATCH 2/2] docs: update release notes for 0.14.0 --- docs/release-notes/release-notes-0.14.0.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/docs/release-notes/release-notes-0.14.0.md b/docs/release-notes/release-notes-0.14.0.md index 6de9e2405..02f91ecb3 100644 --- a/docs/release-notes/release-notes-0.14.0.md +++ b/docs/release-notes/release-notes-0.14.0.md @@ -436,6 +436,8 @@ messages directly. There is no routing/path finding involved. * [Fix missing label on sweep transactions](https://github.com/lightningnetwork/lnd/pull/5895). +* [Fixed flake that occurred with the switch dust forwarding test under the data race unit build.](https://github.com/lightningnetwork/lnd/pull/5828) + ## Database * [Ensure single writer for legacy