Skip to content

Commit b38c19e

Browse files
committed
f Drop expect_pending_htlcs_forwardable_conditions macro
1 parent 64ba15c commit b38c19e

File tree

2 files changed

+4
-12
lines changed

2 files changed

+4
-12
lines changed

lightning/src/ln/functional_test_utils.rs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1969,14 +1969,6 @@ impl SendEvent {
19691969
}
19701970
}
19711971

1972-
#[macro_export]
1973-
/// Don't use this, use the identically-named function instead.
1974-
macro_rules! expect_pending_htlcs_forwardable_conditions {
1975-
($node: expr, $expected_failures: expr) => {
1976-
$crate::ln::functional_test_utils::expect_pending_htlcs_forwardable_conditions($node.node.get_and_clear_pending_events(), &$expected_failures);
1977-
}
1978-
}
1979-
19801972
#[macro_export]
19811973
macro_rules! expect_htlc_handling_failed_destinations {
19821974
($events: expr, $expected_failures: expr) => {{

lightning/src/ln/monitor_tests.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1219,8 +1219,8 @@ fn test_no_preimage_inbound_htlc_balances() {
12191219
let as_htlc_timeout_claim = nodes[0].tx_broadcaster.txn_broadcasted.lock().unwrap().split_off(0);
12201220
assert_eq!(as_htlc_timeout_claim.len(), 1);
12211221
check_spends!(as_htlc_timeout_claim[0], as_txn[0]);
1222-
expect_pending_htlcs_forwardable_conditions!(nodes[0],
1223-
[HTLCHandlingFailureType::Receive { payment_hash: to_a_failed_payment_hash }]);
1222+
expect_pending_htlcs_forwardable_conditions(nodes[0].node.get_and_clear_pending_events(), &[HTLCHandlingFailureType::Receive { payment_hash: to_a_failed_payment_hash }]);
1223+
nodes[0].node.process_pending_htlc_forwards();
12241224

12251225
assert_eq!(as_pre_spend_claims,
12261226
sorted_vec(nodes[0].chain_monitor.chain_monitor.get_monitor(chan_id).unwrap().get_claimable_balances()));
@@ -1237,8 +1237,8 @@ fn test_no_preimage_inbound_htlc_balances() {
12371237
// The next few blocks for B look the same as for A, though for the opposite HTLC
12381238
nodes[1].tx_broadcaster.txn_broadcasted.lock().unwrap().clear();
12391239
connect_blocks(&nodes[1], TEST_FINAL_CLTV - (ANTI_REORG_DELAY - 1));
1240-
expect_pending_htlcs_forwardable_conditions!(nodes[1],
1241-
[HTLCHandlingFailureType::Receive { payment_hash: to_b_failed_payment_hash }]);
1240+
expect_pending_htlcs_forwardable_conditions(nodes[1].node.get_and_clear_pending_events(), &[HTLCHandlingFailureType::Receive { payment_hash: to_b_failed_payment_hash }]);
1241+
nodes[1].node.process_pending_htlc_forwards();
12421242
let bs_htlc_timeout_claim = nodes[1].tx_broadcaster.txn_broadcasted.lock().unwrap().split_off(0);
12431243
assert_eq!(bs_htlc_timeout_claim.len(), 1);
12441244
check_spends!(bs_htlc_timeout_claim[0], as_txn[0]);

0 commit comments

Comments
 (0)