From 6e3a46ee9116797752d14592f31d3fa85221cb42 Mon Sep 17 00:00:00 2001 From: Carla Kirk-Cohen Date: Sat, 27 Apr 2024 11:28:42 -0400 Subject: [PATCH] itest: add more comprehensive assertions before HTLC cleared check We mine quite a few blocks in this test to trigger a htlc timeout, so it can be pretty slow. This fix adds assertions for additional "state steps" that happen in between the failing of the htlc on-chain and asserting that we're fully cleared out so that slower running machines won't timeout. --- itest/lnd_route_blinding_test.go | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/itest/lnd_route_blinding_test.go b/itest/lnd_route_blinding_test.go index 1cae46a0c..4b0eef859 100644 --- a/itest/lnd_route_blinding_test.go +++ b/itest/lnd_route_blinding_test.go @@ -1018,10 +1018,14 @@ func testErrorHandlingOnChainFailure(ht *lntest.HarnessTest) { // suspended Carol we don't need to account for her commitment output // claim. ht.Miner.MineBlocksAndAssertNumTxes(1, 1) + ht.AssertNumPendingSweeps(ht.Bob, 0) // Assert that the HTLC has cleared. - ht.AssertHTLCNotActive(ht.Alice, testCase.channels[0], hash[:]) + ht.WaitForBlockchainSync(ht.Bob) + ht.WaitForBlockchainSync(ht.Alice) + ht.AssertHTLCNotActive(ht.Bob, testCase.channels[0], hash[:]) + ht.AssertHTLCNotActive(ht.Alice, testCase.channels[0], hash[:]) // Wait for the HTLC to reflect as failed for Alice. paymentStream := ht.Alice.RPC.TrackPaymentV2(hash[:])