Skip to content

Commit d09e2a7

Browse files
Parameterize blinded test util with intro node min/max htlc.
We're currently not setting each node's min/max htlc correctly, see next commit.
1 parent 9520f7e commit d09e2a7

File tree

2 files changed

+15
-14
lines changed

2 files changed

+15
-14
lines changed

lightning/src/ln/blinded_payment_tests.rs

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,9 @@ use crate::util::config::UserConfig;
2727
use crate::util::test_utils;
2828

2929
pub fn get_blinded_route_parameters(
30-
amt_msat: u64, payment_secret: PaymentSecret, node_ids: Vec<PublicKey>,
31-
channel_upds: &[&msgs::UnsignedChannelUpdate], keys_manager: &test_utils::TestKeysInterface
30+
amt_msat: u64, payment_secret: PaymentSecret, intro_node_min_htlc: u64, intro_node_max_htlc: u64,
31+
node_ids: Vec<PublicKey>, channel_upds: &[&msgs::UnsignedChannelUpdate],
32+
keys_manager: &test_utils::TestKeysInterface
3233
) -> RouteParameters {
3334
let mut intermediate_nodes = Vec::new();
3435
for (node_id, chan_upd) in node_ids.iter().zip(channel_upds) {
@@ -183,13 +184,13 @@ fn mpp_to_three_hop_blinded_paths() {
183184
let (payment_preimage, payment_hash, payment_secret) = get_payment_preimage_hash(&nodes[5], Some(amt_msat), None);
184185
let route_params = {
185186
let path_1_params = get_blinded_route_parameters(
186-
amt_msat, payment_secret, vec![
187+
amt_msat, payment_secret, 1, 100000000, vec![
187188
nodes[1].node.get_our_node_id(), nodes[3].node.get_our_node_id(),
188189
nodes[5].node.get_our_node_id()
189190
], &[&chan_upd_1_3, &chan_upd_3_5], &chanmon_cfgs[5].keys_manager
190191
);
191192
let path_2_params = get_blinded_route_parameters(
192-
amt_msat, payment_secret, vec![
193+
amt_msat, payment_secret, 1, 100000000, vec![
193194
nodes[2].node.get_our_node_id(), nodes[4].node.get_our_node_id(),
194195
nodes[5].node.get_our_node_id()
195196
], &[&chan_upd_2_4, &chan_upd_4_5], &chanmon_cfgs[5].keys_manager
@@ -258,7 +259,7 @@ fn do_forward_checks_failure(check: ForwardCheckFail, intro_fails: bool) {
258259

259260
let amt_msat = 5000;
260261
let (_, payment_hash, payment_secret) = get_payment_preimage_hash(&nodes[3], Some(amt_msat), None);
261-
let route_params = get_blinded_route_parameters(amt_msat, payment_secret,
262+
let route_params = get_blinded_route_parameters(amt_msat, payment_secret, 1, 100000000,
262263
nodes.iter().skip(1).map(|n| n.node.get_our_node_id()).collect(),
263264
&[&chan_upd_1_2, &chan_upd_2_3], &chanmon_cfgs[3].keys_manager);
264265

@@ -358,7 +359,7 @@ fn failed_backwards_to_intro_node() {
358359

359360
let amt_msat = 5000;
360361
let (_, payment_hash, payment_secret) = get_payment_preimage_hash(&nodes[2], Some(amt_msat), None);
361-
let route_params = get_blinded_route_parameters(amt_msat, payment_secret,
362+
let route_params = get_blinded_route_parameters(amt_msat, payment_secret, 1, 100000000,
362363
nodes.iter().skip(1).map(|n| n.node.get_our_node_id()).collect(), &[&chan_upd_1_2],
363364
&chanmon_cfgs[2].keys_manager);
364365

@@ -438,7 +439,7 @@ fn do_forward_fail_in_process_pending_htlc_fwds(check: ProcessPendingHTLCsCheck,
438439

439440
let amt_msat = 5000;
440441
let (_, payment_hash, payment_secret) = get_payment_preimage_hash(&nodes[2], Some(amt_msat), None);
441-
let route_params = get_blinded_route_parameters(amt_msat, payment_secret,
442+
let route_params = get_blinded_route_parameters(amt_msat, payment_secret, 1, 100000000,
442443
nodes.iter().skip(1).map(|n| n.node.get_our_node_id()).collect(), &[&chan_upd_1_2, &chan_upd_2_3],
443444
&chanmon_cfgs[2].keys_manager);
444445

@@ -555,7 +556,7 @@ fn do_blinded_intercept_payment(intercept_node_fails: bool) {
555556
let intercept_scid = nodes[1].node.get_intercept_scid();
556557
let mut intercept_chan_upd = chan_upd;
557558
intercept_chan_upd.short_channel_id = intercept_scid;
558-
let route_params = get_blinded_route_parameters(amt_msat, payment_secret,
559+
let route_params = get_blinded_route_parameters(amt_msat, payment_secret, 1, 100000000,
559560
nodes.iter().skip(1).map(|n| n.node.get_our_node_id()).collect(), &[&intercept_chan_upd],
560561
&chanmon_cfgs[2].keys_manager);
561562

@@ -632,7 +633,7 @@ fn two_hop_blinded_path_success() {
632633

633634
let amt_msat = 5000;
634635
let (payment_preimage, payment_hash, payment_secret) = get_payment_preimage_hash(&nodes[2], Some(amt_msat), None);
635-
let route_params = get_blinded_route_parameters(amt_msat, payment_secret,
636+
let route_params = get_blinded_route_parameters(amt_msat, payment_secret, 1, 100000000,
636637
nodes.iter().skip(1).map(|n| n.node.get_our_node_id()).collect(), &[&chan_upd_1_2],
637638
&chanmon_cfgs[2].keys_manager);
638639

@@ -655,7 +656,7 @@ fn three_hop_blinded_path_success() {
655656

656657
let amt_msat = 5000;
657658
let (payment_preimage, payment_hash, payment_secret) = get_payment_preimage_hash(&nodes[4], Some(amt_msat), None);
658-
let route_params = get_blinded_route_parameters(amt_msat, payment_secret,
659+
let route_params = get_blinded_route_parameters(amt_msat, payment_secret, 1, 100000000,
659660
nodes.iter().skip(2).map(|n| n.node.get_our_node_id()).collect(),
660661
&[&chan_upd_2_3, &chan_upd_3_4], &chanmon_cfgs[4].keys_manager);
661662

@@ -677,7 +678,7 @@ fn three_hop_blinded_path_fail() {
677678

678679
let amt_msat = 5000;
679680
let (_, payment_hash, payment_secret) = get_payment_preimage_hash(&nodes[3], Some(amt_msat), None);
680-
let route_params = get_blinded_route_parameters(amt_msat, payment_secret,
681+
let route_params = get_blinded_route_parameters(amt_msat, payment_secret, 1, 100000000,
681682
nodes.iter().skip(1).map(|n| n.node.get_our_node_id()).collect(),
682683
&[&chan_upd_1_2, &chan_upd_2_3], &chanmon_cfgs[3].keys_manager);
683684

@@ -768,7 +769,7 @@ fn do_multi_hop_receiver_fail(check: ReceiveCheckFail) {
768769
Some(TEST_FINAL_CLTV as u16 - 2)
769770
} else { None };
770771
let (_, payment_hash, payment_secret) = get_payment_preimage_hash(&nodes[2], Some(amt_msat), excess_final_cltv_delta_opt);
771-
let mut route_params = get_blinded_route_parameters(amt_msat, payment_secret,
772+
let mut route_params = get_blinded_route_parameters(amt_msat, payment_secret, 1, 100000000,
772773
nodes.iter().skip(1).map(|n| n.node.get_our_node_id()).collect(), &[&chan_upd_1_2],
773774
&chanmon_cfgs[2].keys_manager);
774775

@@ -785,7 +786,7 @@ fn do_multi_hop_receiver_fail(check: ReceiveCheckFail) {
785786
let high_htlc_min_bp = {
786787
let mut high_htlc_minimum_upd = chan_upd_1_2.clone();
787788
high_htlc_minimum_upd.htlc_minimum_msat = amt_msat + 1000;
788-
let high_htlc_min_params = get_blinded_route_parameters(amt_msat, payment_secret,
789+
let high_htlc_min_params = get_blinded_route_parameters(amt_msat, payment_secret, 1, 100000000,
789790
nodes.iter().skip(1).map(|n| n.node.get_our_node_id()).collect(), &[&high_htlc_minimum_upd],
790791
&chanmon_cfgs[2].keys_manager);
791792
if let Payee::Blinded { route_hints, .. } = high_htlc_min_params.payment_params.payee {

lightning/src/ln/shutdown_tests.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -434,7 +434,7 @@ fn do_htlc_fail_async_shutdown(blinded_recipient: bool) {
434434
let (_, our_payment_hash, our_payment_secret) = get_payment_preimage_hash(&nodes[2], Some(amt_msat), None);
435435
let route_params = if blinded_recipient {
436436
crate::ln::blinded_payment_tests::get_blinded_route_parameters(
437-
amt_msat, our_payment_secret,
437+
amt_msat, our_payment_secret, 1, 100000000,
438438
nodes.iter().skip(1).map(|n| n.node.get_our_node_id()).collect(), &[&chan_2.0.contents],
439439
&chanmon_cfgs[2].keys_manager)
440440
} else {

0 commit comments

Comments
 (0)