From 6c42dd2c09a588bc5eb85c9e265aa27062c539be Mon Sep 17 00:00:00 2001 From: Conner Fromknecht Date: Wed, 14 Mar 2018 04:01:57 -0700 Subject: [PATCH] lnd_test: removes unnecessary disconnects from switch itest This commit removes two unnecessary disconnect/reconnect attempts from the switch itest. Both occur after the primary test has been completed, and were being executed before doing a final sanity check that the path is still usable. Nothing about the test behavior should change. --- lnd_test.go | 20 -------------------- 1 file changed, 20 deletions(-) diff --git a/lnd_test.go b/lnd_test.go index 3ecadcb1..f37b64fb 100644 --- a/lnd_test.go +++ b/lnd_test.go @@ -7208,16 +7208,6 @@ func testSwitchOfflineDelivery(net *lntest.NetworkHarness, t *harnessTest) { assertAmountPaid(t, ctxb, "Bob(local) => Alice(remote)", net.Bob, aliceFundPoint, amountPaid+(baseFee*numPayments)*2, int64(0)) - ctxt, _ = context.WithTimeout(ctxb, timeout) - if err := net.DisconnectNodes(ctxt, dave, net.Alice); err != nil { - t.Fatalf("unable to disconnect alice from dave: %v", err) - } - - ctxt, _ = context.WithTimeout(ctxb, timeout) - if err := net.ConnectNodes(ctxt, dave, net.Alice); err != nil { - t.Fatalf("unable to reconnect alice to dave: %v", err) - } - // Lastly, we will send one more payment to ensure all channels are // still functioning properly. finalInvoice := &lnrpc.Invoice{ @@ -7524,16 +7514,6 @@ func testSwitchOfflineDeliveryPersistence(net *lntest.NetworkHarness, t *harness assertAmountPaid(t, ctxb, "Bob(local) => Alice(remote)", net.Bob, aliceFundPoint, amountPaid+(baseFee*numPayments)*2, int64(0)) - ctxt, _ = context.WithTimeout(ctxb, timeout) - if err := net.DisconnectNodes(ctxt, dave, net.Alice); err != nil { - t.Fatalf("unable to disconnect alice from dave: %v", err) - } - - ctxt, _ = context.WithTimeout(ctxb, timeout) - if err := net.ConnectNodes(ctxt, dave, net.Alice); err != nil { - t.Fatalf("unable to reconnect alice to dave: %v", err) - } - // Lastly, we will send one more payment to ensure all channels are // still functioning properly. finalInvoice := &lnrpc.Invoice{