fixup: get rid of client unused constructor

Author: martinsaposnic

lightning-liquidity/src/lsps5/client.rs

@@ -17,8 +17,7 @@ use crate::lsps5::msgs::{
 	LSPS5Message, LSPS5Request, LSPS5Response, ListWebhooksRequest, RemoveWebhookRequest,
 	SetWebhookRequest, WebhookNotification,
 };
-#[cfg(feature = "time")]
-use crate::lsps5::service::DefaultTimeProvider;
+
 use crate::message_queue::MessageQueue;
 use crate::prelude::{new_hash_map, HashMap};
 use crate::sync::{Arc, Mutex, RwLock};
@@ -536,27 +535,6 @@ where
 	}
 }
 
-#[cfg(feature = "time")]
-impl<ES: Deref> LSPS5ClientHandler<ES, Arc<DefaultTimeProvider>>
-where
-	ES::Target: EntropySource,
-{
-	/// Constructs a `LSPS5ClientHandler` using [`DefaultTimeProvider`].
-	#[allow(dead_code)]
-	pub(crate) fn new(
-		entropy_source: ES, pending_messages: Arc<MessageQueue>, pending_events: Arc<EventQueue>,
-		config: LSPS5ClientConfig,
-	) -> Self {
-		Self::new_with_time_provider(
-			entropy_source,
-			pending_messages,
-			pending_events,
-			config,
-			Arc::new(DefaultTimeProvider),
-		)
-	}
-}
-
 impl<ES: Deref, TP: Deref + Clone> LSPSProtocolMessageHandler for LSPS5ClientHandler<ES, TP>
 where
 	ES::Target: EntropySource,
@@ -578,7 +556,9 @@ mod tests {
 
 	use super::*;
 	use crate::{
-		lsps0::ser::LSPSRequestId, lsps5::msgs::SetWebhookResponse, tests::utils::TestEntropy,
+		lsps0::ser::LSPSRequestId,
+		lsps5::{msgs::SetWebhookResponse, service::DefaultTimeProvider},
+		tests::utils::TestEntropy,
 	};
 	use bitcoin::{key::Secp256k1, secp256k1::SecretKey};
 
@@ -592,11 +572,12 @@ mod tests {
 		let test_entropy_source = Arc::new(TestEntropy {});
 		let message_queue = Arc::new(MessageQueue::new());
 		let event_queue = Arc::new(EventQueue::new());
-		let client = LSPS5ClientHandler::new(
+		let client = LSPS5ClientHandler::new_with_time_provider(
 			test_entropy_source,
 			Arc::clone(&message_queue),
 			Arc::clone(&event_queue),
 			LSPS5ClientConfig::default(),
+			Arc::new(DefaultTimeProvider),
 		);
 
 		let secp = Secp256k1::new();