f refcell, not mutex

Author: TheBlueMatt

lightning-invoice/src/payment.rs

@@ -462,7 +462,6 @@ mod tests {
 	use std::cell::RefCell;
 	use std::collections::VecDeque;
 	use std::time::{SystemTime, Duration};
-	use std::sync::Mutex;
 
 	fn invoice(payment_preimage: PaymentPreimage) -> Invoice {
 		let payment_hash = Sha256::hash(&payment_preimage.0);
@@ -1188,12 +1187,12 @@ mod tests {
 	}
 
 	// *** Full Featured Functional Tests with a Real ChannelManager ***
-	struct ManualRouter(Mutex<VecDeque<Result<Route, LightningError>>>);
+	struct ManualRouter(RefCell<VecDeque<Result<Route, LightningError>>>);
 
 	impl<S: routing::Score> Router<S> for ManualRouter {
 		fn find_route(&self, _payer: &PublicKey, _params: &RouteParameters, _first_hops: Option<&[&ChannelDetails]>, _scorer: &S)
 		-> Result<Route, LightningError> {
-			self.0.lock().unwrap().pop_front().unwrap()
+			self.0.borrow_mut().pop_front().unwrap()
 		}
 	}
 
@@ -1202,7 +1201,7 @@ mod tests {
 			if std::thread::panicking() {
 				return;
 			}
-			assert!(self.0.lock().unwrap().is_empty());
+			assert!(self.0.borrow_mut().is_empty());
 		}
 	}
 
@@ -1244,7 +1243,7 @@ mod tests {
 		route.paths[0][0].fee_msat = 50_000_001;
 		route.paths[1][0].fee_msat = 50_000_000;
 		routes.push_back(Ok(route.clone()));
-		let route_res = ManualRouter(Mutex::new(routes));
+		let route_res = ManualRouter(RefCell::new(routes));
 
 		let event_handled = core::cell::RefCell::new(false);
 		let event_handler = |_: &_| { *event_handled.borrow_mut() = true; };