From b1aea41d20db987b488d5883c5afda7299d3c380 Mon Sep 17 00:00:00 2001 From: Olaoluwa Osuntokun Date: Mon, 5 Aug 2019 21:36:15 -0700 Subject: [PATCH] lntest/itest: force dave to use the legacy onion payload in multi-hop pay test In this commit, we force Dave to use the legacy onion payload for the multi-hop test to ensure that we're able to properly mix the old and new formats, and have all nodes properly decode+forward the HTLC. --- lntest/itest/lnd_test.go | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/lntest/itest/lnd_test.go b/lntest/itest/lnd_test.go index 9905c56b..d6a16a9a 100644 --- a/lntest/itest/lnd_test.go +++ b/lntest/itest/lnd_test.go @@ -4013,13 +4013,15 @@ func testMultiHopPayments(net *lntest.NetworkHarness, t *harnessTest) { } // As preliminary setup, we'll create two new nodes: Carol and Dave, - // such that we now have a 4 ndoe, 3 channel topology. Dave will make - // a channel with Alice, and Carol with Dave. After this setup, the + // such that we now have a 4 node, 3 channel topology. Dave will make a + // channel with Alice, and Carol with Dave. After this setup, the // network topology should now look like: // Carol -> Dave -> Alice -> Bob // - // First, we'll create Dave and establish a channel to Alice. - dave, err := net.NewNode("Dave", nil) + // First, we'll create Dave and establish a channel to Alice. Dave will + // be running an older node that requires the legacy onion payload. + daveArgs := []string{"--legacyprotocol.onion"} + dave, err := net.NewNode("Dave", daveArgs) if err != nil { t.Fatalf("unable to create new nodes: %v", err) }