f - Address TODO about using different payment secrets

Author: jkczyz

lightning/src/ln/functional_tests.rs

@@ -3983,9 +3983,7 @@ fn do_test_holding_cell_htlc_add_timeouts(forwarded_htlc: bool) {
 	// Now attempt to route a second payment, which should be placed in the holding cell
 	let sending_node = if forwarded_htlc { &nodes[0] } else { &nodes[1] };
 	let (route, second_payment_hash, _, second_payment_secret) = get_route_and_payment_hash!(sending_node, nodes[2], 100000);
-	// TODO: Why use a different payment secret here?
-	let payment_secret = if forwarded_htlc { first_payment_secret } else { second_payment_secret };
-	sending_node.node.send_payment(&route, second_payment_hash, &Some(payment_secret)).unwrap();
+	sending_node.node.send_payment(&route, second_payment_hash, &Some(second_payment_secret)).unwrap();
 	if forwarded_htlc {
 		check_added_monitors!(nodes[0], 1);
 		let payment_event = SendEvent::from_event(nodes[0].node.get_and_clear_pending_msg_events().remove(0));