multi: add default conf targt in SendCoins/SendMany/OpenChannel/CloseChannel

itest/lnd_misc_test.go

@@ -815,13 +815,18 @@ func testSweepAllCoins(ht *lntest.HarnessTest) {
 		TargetConf: 6,
 	})
 
+	// TODO(yy): we still allow default values to be used when neither conf
+	// target or fee rate is set in 0.18.0. When future release forbidden
+	// this behavior, we should revive the test below, which asserts either
+	// conf target or fee rate is set.
+	//
 	// Send coins to a compatible address without specifying fee rate or
 	// conf target.
-	ainz.RPC.SendCoinsAssertErr(&lnrpc.SendCoinsRequest{
-		Addr:    ht.Miner.NewMinerAddress().String(),
-		SendAll: true,
-		Label:   sendCoinsLabel,
-	})
+	// ainz.RPC.SendCoinsAssertErr(&lnrpc.SendCoinsRequest{
+	// 	Addr:    ht.Miner.NewMinerAddress().String(),
+	// 	SendAll: true,
+	// 	Label:   sendCoinsLabel,
+	// })
 
 	// Send coins to a compatible address.
 	ainz.RPC.SendCoins(&lnrpc.SendCoinsRequest{