From 7f4d1cee6d19172d547f417a58631ea485374591 Mon Sep 17 00:00:00 2001 From: "Johan T. Halseth" Date: Mon, 25 Feb 2019 13:03:56 -0300 Subject: [PATCH] lntest: don't return from ensureConnected on non-error This would casue the method to return before the peer list check was performed. --- lntest/harness.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lntest/harness.go b/lntest/harness.go index 2bf40f0e0..4b3048e68 100644 --- a/lntest/harness.go +++ b/lntest/harness.go @@ -460,11 +460,11 @@ func (n *NetworkHarness) EnsureConnected(ctx context.Context, a, b *HarnessNode) // can exit early. return nil - case aErr != errConnectionRequested: + case aErr != nil && aErr != errConnectionRequested: // Return any critical errors returned by either alice. return aErr - case bErr != errConnectionRequested: + case bErr != nil && bErr != errConnectionRequested: // Return any critical errors returned by either bob. return bErr