f nits/sp

Author: TheBlueMatt

lightning/src/ln/functional_tests.rs

@@ -120,8 +120,8 @@ fn test_insane_channel_opens() {
 fn do_test_counterparty_no_reserve(send_from_initiator: bool) {
 	// A peer providing a channel_reserve_satoshis of 0 (or less than our dust limit) is insecure,
 	// but only for them. Because some LSPs do it with some level of trust of the clients (for a
-	// substantial UX improvement), we explicitly allow it. Because its unlikely to happen often in
-	// normal testing, we test it explicitly here.
+	// substantial UX improvement), we explicitly allow it. Because it's unlikely to happen often
+	// in normal testing, we test it explicitly here.
 	let chanmon_cfgs = create_chanmon_cfgs(2);
 	let node_cfgs = create_node_cfgs(2, &chanmon_cfgs);
 	let node_chanmgrs = create_node_chanmgrs(2, &node_cfgs, &[None, None]);
@@ -166,7 +166,7 @@ fn do_test_counterparty_no_reserve(send_from_initiator: bool) {
 			// Note that for outbound channels we have to consider the commitment tx fee and the
 			// "fee spike buffer", which is currently a multiple of the total commitment tx fee as
 			// well as an additional HTLC.
-			- FEE_SPIKE_BUFFER_FEE_INCREASE_MULTIPLE * feerate_per_kw * (COMMITMENT_TX_BASE_WEIGHT + 2 * COMMITMENT_TX_WEIGHT_PER_HTLC) / 1000 * 1000);
+			- FEE_SPIKE_BUFFER_FEE_INCREASE_MULTIPLE * commit_tx_fee_msat(feerate_per_kw, 2));
 	} else {
 		send_payment(&nodes[1], &[&nodes[0]], push_amt);
 	}