itest: use coop closes instead of force closes

Author: ellemouton

itest/litd_firewall_test.go

@@ -161,7 +161,7 @@ func testFirewallRules(ctx context.Context, net *NetworkHarness,
 			Amt: 100000,
 		},
 	)
-	defer closeChannelAndAssert(t, net, net.Alice, channelOp, true)
+	defer closeChannelAndAssert(t, net, net.Alice, channelOp, false)
 
 	t.t.Run("history limit rule", func(_ *testing.T) {
 		testHistoryLimitRule(net, t)
@@ -199,14 +199,14 @@ func testSessionLinking(net *NetworkHarness, t *harnessTest) {
 			Amt: 100000,
 		},
 	)
-	defer closeChannelAndAssert(t, net, net.Alice, channelAB2Op, true)
+	defer closeChannelAndAssert(t, net, net.Alice, channelAB2Op, false)
 
 	channelAB3Op := openChannelAndAssert(
 		t, net, net.Alice, net.Bob, lntest.OpenChannelParams{
 			Amt: 100000,
 		},
 	)
-	defer closeChannelAndAssert(t, net, net.Alice, channelAB3Op, true)
+	defer closeChannelAndAssert(t, net, net.Alice, channelAB3Op, false)
 
 	// List Alice's channels so that we can extract the channel points for
 	// the three channels.
@@ -1243,22 +1243,22 @@ func testPeerAndChannelRestrictRules(net *NetworkHarness, t *harnessTest) {
 			Amt: 100000,
 		},
 	)
-	defer closeChannelAndAssert(t, net, net.Alice, channelAB2Op, true)
+	defer closeChannelAndAssert(t, net, net.Alice, channelAB2Op, false)
 
 	// Open two channels between Alice and Charlie.
 	channelAC1Op := openChannelAndAssert(
 		t, net, net.Alice, charlie, lntest.OpenChannelParams{
 			Amt: 100000,
 		},
 	)
-	defer closeChannelAndAssert(t, net, net.Alice, channelAC1Op, true)
+	defer closeChannelAndAssert(t, net, net.Alice, channelAC1Op, false)
 
 	channelAC2Op := openChannelAndAssert(
 		t, net, net.Alice, charlie, lntest.OpenChannelParams{
 			Amt: 100000,
 		},
 	)
-	defer closeChannelAndAssert(t, net, net.Alice, channelAC2Op, true)
+	defer closeChannelAndAssert(t, net, net.Alice, channelAC2Op, false)
 
 	// List Alice's channels so that we can extract the channel points
 	// for the four channels.