From 24a99ade66c567312b667f85c880867c0c345ef6 Mon Sep 17 00:00:00 2001 From: bryanvu Date: Thu, 5 Jan 2017 13:00:40 -0800 Subject: [PATCH] test: remove concurrent payment sends from testMultiHopPayments testMultiHopPayments was failing intermittently due to a lack of thread-safety in the gRPC stream. This commit takes payment sending out of goroutines so that they execute serially. --- lnd_test.go | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/lnd_test.go b/lnd_test.go index 6f3f8c24..6222f2b9 100644 --- a/lnd_test.go +++ b/lnd_test.go @@ -795,16 +795,12 @@ func testMultiHopPayments(net *networkHarness, t *harnessTest) { Amt: paymentAmt, } - wg.Add(1) - go func() { - if err := carolPayStream.Send(sendReq); err != nil { - t.Fatalf("unable to send payment: %v", err) - } - if _, err := carolPayStream.Recv(); err != nil { - t.Fatalf("unable to recv pay resp: %v", err) - } - wg.Done() - }() + if err := carolPayStream.Send(sendReq); err != nil { + t.Fatalf("unable to send payment: %v", err) + } + if _, err := carolPayStream.Recv(); err != nil { + t.Fatalf("unable to recv pay resp: %v", err) + } } finClear := make(chan struct{})