Add timeout retry strategy to outbound payment

rename InvoicePayment to InvoicePaymentUsingTime with Time trait to allow manual time advance in test.
impl Display for PaymentAttempts

Author: kafaichoi

lightning-background-processor/src/lib.rs

@@ -365,7 +365,7 @@ mod tests {
 	use lightning::util::logger::Logger;
 	use lightning::util::ser::Writeable;
 	use lightning::util::test_utils;
-	use lightning_invoice::payment::{InvoicePayer, RetryAttempts};
+	use lightning_invoice::payment::{InvoicePayer, Retry};
 	use lightning_invoice::utils::DefaultRouter;
 	use lightning_persister::FilesystemPersister;
 	use std::fs;
@@ -755,7 +755,7 @@ mod tests {
 		let persister = Persister::new(data_dir);
 		let scorer = Arc::new(Mutex::new(test_utils::TestScorer::with_penalty(0)));
 		let router = DefaultRouter::new(Arc::clone(&nodes[0].network_graph), Arc::clone(&nodes[0].logger), random_seed_bytes);
-		let invoice_payer = Arc::new(InvoicePayer::new(Arc::clone(&nodes[0].node), router, scorer, Arc::clone(&nodes[0].logger), |_: &_| {}, RetryAttempts(2)));
+		let invoice_payer = Arc::new(InvoicePayer::new(Arc::clone(&nodes[0].node), router, scorer, Arc::clone(&nodes[0].logger), |_: &_| {}, Retry::Attempts(2)));
 		let event_handler = Arc::clone(&invoice_payer);
 		let bg_processor = BackgroundProcessor::start(persister, event_handler, nodes[0].chain_monitor.clone(), nodes[0].node.clone(), nodes[0].net_graph_msg_handler.clone(), nodes[0].peer_manager.clone(), nodes[0].logger.clone());
 		assert!(bg_processor.stop().is_ok());