X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=blobdiff_plain;f=lightning%2Fsrc%2Fln%2Fpayment_tests.rs;h=c4cd0fc1b09d1ca691654c7fbc32e256bb39ed41;hb=aefe276fcb0f344e02315279da52b3dd464089e5;hp=4dd1b9009763a3bdfae43e3944d9c30f29ea5dc4;hpb=9c08fbd435b097c0aeec2843d8b4a6fdec06a8f0;p=rust-lightning

diff --git a/lightning/src/ln/payment_tests.rs b/lightning/src/ln/payment_tests.rs
index 4dd1b900..c4cd0fc1 100644
--- a/lightning/src/ln/payment_tests.rs
+++ b/lightning/src/ln/payment_tests.rs
@@ -98,7 +98,6 @@ fn mpp_retry() {
 	let mut route_params = RouteParameters {
 		payment_params: route.payment_params.clone().unwrap(),
 		final_value_msat: amt_msat,
-		final_cltv_expiry_delta: TEST_FINAL_CLTV,
 	};
 
 	nodes[0].router.expect_find_route(route_params.clone(), Ok(route.clone()));
@@ -297,7 +296,6 @@ fn do_retry_with_no_persist(confirm_before_reload: bool) {
 	let route_params = RouteParameters {
 		payment_params: route.payment_params.clone().unwrap(),
 		final_value_msat: amt_msat,
-		final_cltv_expiry_delta: TEST_FINAL_CLTV,
 	};
 	nodes[0].node.send_payment_with_retry(payment_hash, &Some(payment_secret), PaymentId(payment_hash.0), route_params, Retry::Attempts(1)).unwrap();
 	check_added_monitors!(nodes[0], 1);
@@ -1388,12 +1386,12 @@ fn do_test_intercepted_payment(test: InterceptTest) {
 	let route_params = RouteParameters {
 		payment_params,
 		final_value_msat: amt_msat,
-		final_cltv_expiry_delta: TEST_FINAL_CLTV,
 	};
 	let route = get_route(
 		&nodes[0].node.get_our_node_id(), &route_params.payment_params,
 		&nodes[0].network_graph.read_only(), None, route_params.final_value_msat,
-		route_params.final_cltv_expiry_delta, nodes[0].logger, &scorer, &random_seed_bytes
+		route_params.payment_params.final_cltv_expiry_delta, nodes[0].logger, &scorer,
+		&random_seed_bytes,
 	).unwrap();
 
 	let (payment_hash, payment_secret) = nodes[2].node.create_inbound_payment(Some(amt_msat), 60 * 60, None).unwrap();
@@ -1578,7 +1576,6 @@ fn do_automatic_retries(test: AutoRetry) {
 	let route_params = RouteParameters {
 		payment_params,
 		final_value_msat: amt_msat,
-		final_cltv_expiry_delta: TEST_FINAL_CLTV,
 	};
 	let (_, payment_hash, payment_preimage, payment_secret) = get_route_and_payment_hash!(nodes[0], nodes[2], amt_msat);
 
@@ -1788,7 +1785,6 @@ fn auto_retry_partial_failure() {
 	let route_params = RouteParameters {
 		payment_params,
 		final_value_msat: amt_msat,
-		final_cltv_expiry_delta: TEST_FINAL_CLTV,
 	};
 
 	// Ensure the first monitor update (for the initial send path1 over chan_1) succeeds, but the
@@ -1861,12 +1857,12 @@ fn auto_retry_partial_failure() {
 	let mut payment_params = route_params.payment_params.clone();
 	payment_params.previously_failed_channels.push(chan_2_id);
 	nodes[0].router.expect_find_route(RouteParameters {
-			payment_params, final_value_msat: amt_msat / 2, final_cltv_expiry_delta: TEST_FINAL_CLTV
+			payment_params, final_value_msat: amt_msat / 2,
 		}, Ok(retry_1_route));
 	let mut payment_params = route_params.payment_params.clone();
 	payment_params.previously_failed_channels.push(chan_3_id);
 	nodes[0].router.expect_find_route(RouteParameters {
-			payment_params, final_value_msat: amt_msat / 4, final_cltv_expiry_delta: TEST_FINAL_CLTV
+			payment_params, final_value_msat: amt_msat / 4,
 		}, Ok(retry_2_route));
 
 	// Send a payment that will partially fail on send, then partially fail on retry, then succeed.
@@ -2000,7 +1996,6 @@ fn auto_retry_zero_attempts_send_error() {
 	let route_params = RouteParameters {
 		payment_params,
 		final_value_msat: amt_msat,
-		final_cltv_expiry_delta: TEST_FINAL_CLTV,
 	};
 
 	chanmon_cfgs[0].persister.set_update_ret(ChannelMonitorUpdateStatus::PermanentFailure);
@@ -2040,7 +2035,6 @@ fn fails_paying_after_rejected_by_payee() {
 	let route_params = RouteParameters {
 		payment_params,
 		final_value_msat: amt_msat,
-		final_cltv_expiry_delta: TEST_FINAL_CLTV,
 	};
 
 	nodes[0].node.send_payment_with_retry(payment_hash, &Some(payment_secret), PaymentId(payment_hash.0), route_params, Retry::Attempts(1)).unwrap();
@@ -2087,7 +2081,6 @@ fn retry_multi_path_single_failed_payment() {
 	let route_params = RouteParameters {
 		payment_params: payment_params.clone(),
 		final_value_msat: amt_msat,
-		final_cltv_expiry_delta: TEST_FINAL_CLTV,
 	};
 
 	let chans = nodes[0].node.list_usable_channels();
@@ -2122,7 +2115,7 @@ fn retry_multi_path_single_failed_payment() {
 			payment_params: pay_params,
 			// Note that the second request here requests the amount we originally failed to send,
 			// not the amount remaining on the full payment, which should be changed.
-			final_value_msat: 100_000_001, final_cltv_expiry_delta: TEST_FINAL_CLTV
+			final_value_msat: 100_000_001,
 		}, Ok(route.clone()));
 
 	{
@@ -2181,7 +2174,6 @@ fn immediate_retry_on_failure() {
 	let route_params = RouteParameters {
 		payment_params,
 		final_value_msat: amt_msat,
-		final_cltv_expiry_delta: TEST_FINAL_CLTV,
 	};
 
 	let chans = nodes[0].node.list_usable_channels();
@@ -2208,7 +2200,6 @@ fn immediate_retry_on_failure() {
 	pay_params.previously_failed_channels.push(chans[0].short_channel_id.unwrap());
 	nodes[0].router.expect_find_route(RouteParameters {
 			payment_params: pay_params, final_value_msat: amt_msat,
-			final_cltv_expiry_delta: TEST_FINAL_CLTV
 		}, Ok(route.clone()));
 
 	nodes[0].node.send_payment_with_retry(payment_hash, &Some(payment_secret), PaymentId(payment_hash.0), route_params, Retry::Attempts(1)).unwrap();
@@ -2271,7 +2262,6 @@ fn no_extra_retries_on_back_to_back_fail() {
 	let route_params = RouteParameters {
 		payment_params,
 		final_value_msat: amt_msat,
-		final_cltv_expiry_delta: TEST_FINAL_CLTV,
 	};
 
 	let mut route = Route {
@@ -2317,7 +2307,7 @@ fn no_extra_retries_on_back_to_back_fail() {
 	route.paths[0][1].fee_msat = amt_msat;
 	nodes[0].router.expect_find_route(RouteParameters {
 			payment_params: second_payment_params,
-			final_value_msat: amt_msat, final_cltv_expiry_delta: TEST_FINAL_CLTV,
+			final_value_msat: amt_msat,
 		}, Ok(route.clone()));
 
 	nodes[0].node.send_payment_with_retry(payment_hash, &Some(payment_secret), PaymentId(payment_hash.0), route_params, Retry::Attempts(1)).unwrap();
@@ -2472,7 +2462,6 @@ fn test_simple_partial_retry() {
 	let route_params = RouteParameters {
 		payment_params,
 		final_value_msat: amt_msat,
-		final_cltv_expiry_delta: TEST_FINAL_CLTV,
 	};
 
 	let mut route = Route {
@@ -2517,7 +2506,7 @@ fn test_simple_partial_retry() {
 	route.paths.remove(0);
 	nodes[0].router.expect_find_route(RouteParameters {
 			payment_params: second_payment_params,
-			final_value_msat: amt_msat / 2, final_cltv_expiry_delta: TEST_FINAL_CLTV,
+			final_value_msat: amt_msat / 2,
 		}, Ok(route.clone()));
 
 	nodes[0].node.send_payment_with_retry(payment_hash, &Some(payment_secret), PaymentId(payment_hash.0), route_params, Retry::Attempts(1)).unwrap();
@@ -2638,7 +2627,6 @@ fn test_threaded_payment_retries() {
 	let mut route_params = RouteParameters {
 		payment_params,
 		final_value_msat: amt_msat,
-		final_cltv_expiry_delta: TEST_FINAL_CLTV,
 	};
 
 	let mut route = Route {