Rename list_pending_payments to list_recent_payments

Author: jurvis

lightning/src/ln/channelmanager.rs

@@ -1704,7 +1704,7 @@ where
 	/// This can be useful for figuring out whether or not a payment made needs to be retried.
 	/// In general, if it is not listed here, you should consider retrying it if not
 	/// [`RecentPaymentDetails::Fulfilled`].
-	pub fn list_pending_payments(&self) -> Vec<RecentPaymentDetails> {
+	pub fn list_recent_payments(&self) -> Vec<RecentPaymentDetails> {
 		self.pending_outbound_payments.pending_outbound_payments.lock().unwrap().iter()
 			.filter_map(|(_, pending_outbound_payment)| match pending_outbound_payment {
 				PendingOutboundPayment::Retryable { payment_hash, total_msat, .. } => {

lightning/src/ln/payment_tests.rs

@@ -1296,7 +1296,7 @@ fn test_trivial_inflight_htlc_tracking(){
 		assert_eq!(chan_1_used_liquidity, None);
 		assert_eq!(chan_2_used_liquidity, None);
 
-		let pending_payments = nodes[0].node.list_pending_payments();
+		let pending_payments = nodes[0].node.list_recent_payments();
 		assert_eq!(pending_payments.len(), 1);
 		assert_eq!(pending_payments[0], RecentPaymentDetails::Fulfilled { payment_hash: Some(payment_hash) });
 	}
@@ -1333,7 +1333,7 @@ fn test_trivial_inflight_htlc_tracking(){
 		assert_eq!(chan_1_used_liquidity, Some(501000));
 		assert_eq!(chan_2_used_liquidity, Some(500000));
 
-		let pending_payments = nodes[0].node.list_pending_payments();
+		let pending_payments = nodes[0].node.list_recent_payments();
 		assert_eq!(pending_payments.len(), 1);
 		assert_eq!(pending_payments[0], RecentPaymentDetails::Pending { payment_hash, total_msat: 500000 });
 	}
@@ -1370,7 +1370,7 @@ fn test_trivial_inflight_htlc_tracking(){
 		assert_eq!(chan_1_used_liquidity, None);
 		assert_eq!(chan_2_used_liquidity, None);
 
-		let pending_payments = nodes[0].node.list_pending_payments();
+		let pending_payments = nodes[0].node.list_recent_payments();
 		assert_eq!(pending_payments.len(), 0);
 	}
 }