Skip to content

Commit 50c55dc

Browse files
committed
Use Default::default() for scoring params in tests
In 26c1639 we switched to using `Default::default()` to initialize `()` for scoring parameters in tests. A number of `()`s slipped back in recently, which we replace here.
1 parent ec4395c commit 50c55dc

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

lightning/src/ln/payment_tests.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1871,7 +1871,7 @@ fn do_test_intercepted_payment(test: InterceptTest) {
18711871
let route_params = RouteParameters::from_payment_params_and_value(payment_params, amt_msat);
18721872
let route = get_route(
18731873
&nodes[0].node.get_our_node_id(), &route_params, &nodes[0].network_graph.read_only(), None,
1874-
nodes[0].logger, &scorer, &(), &random_seed_bytes
1874+
nodes[0].logger, &scorer, &Default::default(), &random_seed_bytes
18751875
).unwrap();
18761876

18771877
let (payment_hash, payment_secret) = nodes[2].node.create_inbound_payment(Some(amt_msat), 60 * 60, None).unwrap();

lightning/src/routing/router.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5417,7 +5417,7 @@ mod tests {
54175417
max_total_routing_fee_msat: Some(149_999) };
54185418
if let Err(LightningError{err, action: ErrorAction::IgnoreError}) = get_route(
54195419
&our_id, &route_params, &network_graph.read_only(), None, Arc::clone(&logger),
5420-
&scorer, &(), &random_seed_bytes) {
5420+
&scorer, &Default::default(), &random_seed_bytes) {
54215421
assert_eq!(err, "Failed to find a sufficient route to the given destination");
54225422
} else { panic!(); }
54235423
}
@@ -7357,7 +7357,7 @@ mod tests {
73577357
let route_params = RouteParameters::from_payment_params_and_value(
73587358
payment_params, amt_msat);
73597359
if let Err(LightningError { err, .. }) = get_route(
7360-
&our_id, &route_params, &netgraph, None, Arc::clone(&logger), &scorer, &(), &random_seed_bytes
7360+
&our_id, &route_params, &netgraph, None, Arc::clone(&logger), &scorer, &Default::default(), &random_seed_bytes
73617361
) {
73627362
assert_eq!(err, "Failed to find a path to the given destination");
73637363
} else { panic!() }
@@ -7414,7 +7414,7 @@ mod tests {
74147414
let route_params = RouteParameters::from_payment_params_and_value(
74157415
payment_params, amt_msat);
74167416
if let Err(LightningError { err, .. }) = get_route(
7417-
&our_id, &route_params, &netgraph, None, Arc::clone(&logger), &scorer, &(), &random_seed_bytes
7417+
&our_id, &route_params, &netgraph, None, Arc::clone(&logger), &scorer, &Default::default(), &random_seed_bytes
74187418
) {
74197419
assert_eq!(err, "Failed to find a path to the given destination");
74207420
} else { panic!() }
@@ -7492,7 +7492,7 @@ mod tests {
74927492
payment_params, amt_msat);
74937493
if let Err(LightningError { err, .. }) = get_route(
74947494
&our_id, &route_params, &netgraph, Some(&first_hops.iter().collect::<Vec<_>>()),
7495-
Arc::clone(&logger), &scorer, &(), &random_seed_bytes
7495+
Arc::clone(&logger), &scorer, &Default::default(), &random_seed_bytes
74967496
) {
74977497
assert_eq!(err, "Failed to find a path to the given destination");
74987498
} else { panic!() }
@@ -7578,7 +7578,7 @@ mod tests {
75787578

75797579
let route = get_route(
75807580
&our_id, &route_params, &netgraph, Some(&first_hops.iter().collect::<Vec<_>>()),
7581-
Arc::clone(&logger), &scorer, &(), &random_seed_bytes
7581+
Arc::clone(&logger), &scorer, &Default::default(), &random_seed_bytes
75827582
).unwrap();
75837583
assert_eq!(route.paths.len(), 1);
75847584
assert_eq!(route.get_total_amount(), amt_msat);

0 commit comments

Comments
 (0)