itest: remove old TODOs

As they are fixed now.

itest/lnd_hold_invoice_force_test.go

@@ -89,13 +89,6 @@ func testHoldInvoiceForceClose(ht *lntest.HarnessTest) {
 	blocksTillCancel := blocksTillExpiry -
 		lncfg.DefaultHoldInvoiceExpiryDelta
 
-	// When using ht.MineBlocks, for bitcoind backend, the block height
-	// synced differ significantly among subsystems. From observation, the
-	// LNWL syncs much faster than other subsystems, with more than 10
-	// blocks ahead. For this test case, CRTR may be lagging behind for
-	// more than 20 blocks. Thus we use slow mining instead.
-	// TODO(yy): fix block height asymmetry among all the subsystems.
-	//
 	// We first mine enough blocks to trigger an invoice cancelation.
 	ht.MineBlocks(int(blocksTillCancel))
 

itest/lnd_multi-hop_force_close_test.go

@@ -1932,8 +1932,6 @@ func testLocalClaimIncomingHTLCLeased(ht *lntest.HarnessTest) {
 // we force close a channel with an incoming HTLC, and later find out the
 // preimage via the witness beacon, we properly settle the HTLC on-chain using
 // the HTLC success transaction in order to ensure we don't lose any funds.
-//
-// TODO(yy): simplify or remove this test as it's too complicated.
 func runLocalClaimIncomingHTLCLeased(ht *lntest.HarnessTest,
 	cfgs [][]string, params lntest.OpenChannelParams) {