Skip to content

itest: tolerate deferred HTLC timeout sweeps #10909

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
yyforyongyu wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

itest: tolerate deferred HTLC timeout sweeps #10909

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 17 additions & 5 deletions itest/lnd_multi-hop_force_close_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -552,10 +552,22 @@ func runLocalClaimOutgoingHTLC(ht *lntest.HarnessTest,
ht.AssertNumPendingSweeps(bob, 2)
ht.AssertNumPendingSweeps(carol, 1)

// Bob's sweeper should sweep his outgoing HTLC immediately since it's
// expired. His to_local output cannot be swept due to the CSV lock.
// Carol's anchor sweep should be failed due to output being dust.
ht.AssertNumTxsInMempool(1)
// Bob's HTLC timeout input is mature, but it may be offered after the
// sweeper has processed the block that made it mature. If so, mine an
// empty block to trigger the next sweep round without confirming any
// txns that may be broadcast concurrently.
assertMaybeDeferredSweep := func() {
mempool := ht.GetRawMempool()
if len(mempool) == 0 {
ht.MineEmptyBlocks(1)
ht.AssertNumTxsInMempool(1)

return
}

require.Len(ht, mempool, 1)
}
assertMaybeDeferredSweep()

// Mine a block to confirm Bob's outgoing HTLC sweeping tx.
ht.MineBlocksAndAssertNumTxes(1, 1)
Expand Down Expand Up @@ -611,7 +623,7 @@ func runLocalClaimOutgoingHTLC(ht *lntest.HarnessTest,

// Bob's sweeper should now broadcast his second layer sweep
// due to the CSV on the HTLC timeout output.
ht.AssertNumTxsInMempool(1)
assertMaybeDeferredSweep()

// Next, we'll mine a final block that should confirm the
// sweeping transactions left.
Expand Down
Loading