Skip to content

Commit 9333d43

Browse files
committed
Clean up send_payment_with_route calls in async_signer_tests.rs
1 parent 8cdb255 commit 9333d43

File tree

1 file changed

+12
-24
lines changed

1 file changed

+12
-24
lines changed

lightning/src/ln/async_signer_tests.rs

Lines changed: 12 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -296,13 +296,10 @@ fn do_test_async_commitment_signature_for_commitment_signed_revoke_and_ack(
296296
let dst = &nodes[1];
297297
let (route, our_payment_hash, _our_payment_preimage, our_payment_secret) =
298298
get_route_and_payment_hash!(src, dst, 8000000);
299+
let recipient_fields = RecipientOnionFields::secret_only(our_payment_secret);
300+
let payment_id = PaymentId(our_payment_hash.0);
299301
src.node
300-
.send_payment_with_route(
301-
route,
302-
our_payment_hash,
303-
RecipientOnionFields::secret_only(our_payment_secret),
304-
PaymentId(our_payment_hash.0),
305-
)
302+
.send_payment_with_route(route, our_payment_hash, recipient_fields, payment_id)
306303
.unwrap();
307304
check_added_monitors!(src, 1);
308305

@@ -538,13 +535,10 @@ fn do_test_async_raa_peer_disconnect(
538535
let dst = &nodes[1];
539536
let (route, our_payment_hash, _our_payment_preimage, our_payment_secret) =
540537
get_route_and_payment_hash!(src, dst, 8000000);
538+
let recipient_fields = RecipientOnionFields::secret_only(our_payment_secret);
539+
let payment_id = PaymentId(our_payment_hash.0);
541540
src.node
542-
.send_payment_with_route(
543-
route,
544-
our_payment_hash,
545-
RecipientOnionFields::secret_only(our_payment_secret),
546-
PaymentId(our_payment_hash.0),
547-
)
541+
.send_payment_with_route(route, our_payment_hash, recipient_fields, payment_id)
548542
.unwrap();
549543
check_added_monitors!(src, 1);
550544

@@ -702,13 +696,10 @@ fn do_test_async_commitment_signature_peer_disconnect(
702696
let dst = &nodes[1];
703697
let (route, our_payment_hash, _our_payment_preimage, our_payment_secret) =
704698
get_route_and_payment_hash!(src, dst, 8000000);
699+
let recipient_fields = RecipientOnionFields::secret_only(our_payment_secret);
700+
let payment_id = PaymentId(our_payment_hash.0);
705701
src.node
706-
.send_payment_with_route(
707-
route,
708-
our_payment_hash,
709-
RecipientOnionFields::secret_only(our_payment_secret),
710-
PaymentId(our_payment_hash.0),
711-
)
702+
.send_payment_with_route(route, our_payment_hash, recipient_fields, payment_id)
712703
.unwrap();
713704
check_added_monitors!(src, 1);
714705

@@ -871,14 +862,11 @@ fn do_test_async_commitment_signature_ordering(monitor_update_failure: bool) {
871862
// to the peer.
872863
let (route, payment_hash_2, payment_preimage_2, payment_secret_2) =
873864
get_route_and_payment_hash!(nodes[0], nodes[1], 1000000);
865+
let recipient_fields = RecipientOnionFields::secret_only(payment_secret_2);
866+
let payment_id = PaymentId(payment_hash_2.0);
874867
nodes[0]
875868
.node
876-
.send_payment_with_route(
877-
route,
878-
payment_hash_2,
879-
RecipientOnionFields::secret_only(payment_secret_2),
880-
PaymentId(payment_hash_2.0),
881-
)
869+
.send_payment_with_route(route, payment_hash_2, recipient_fields, payment_id)
882870
.unwrap();
883871
check_added_monitors!(nodes[0], 1);
884872

0 commit comments

Comments
 (0)