refactor: remove slog_scope dependency from mithril-common

As it was only used in tests for easy loger creation and we have since
added tooling to create test loggers that don't use `slog_scope`.

Author: Alenar

Cargo.lock

@@ -84,7 +84,6 @@ pallas-crypto = "0.30.1"
 rand_core = { version = "0.6.4", features = ["getrandom"] }
 reqwest = { version = "0.12.7", features = ["json"] }
 slog-async = "2.8.0"
-slog-scope = "4.4.0"
 slog-term = "2.9.1"
 tokio = { version = "1.40.0", features = ["macros", "rt-multi-thread", "time"] }
 

mithril-common/Cargo.toml

@@ -302,14 +302,13 @@ impl CertificateVerifier for MithrilCertificateVerifier {
 mod tests {
     use async_trait::async_trait;
     use mockall::mock;
-    use slog_scope;
 
     use super::CertificateRetriever;
     use super::*;
 
     use crate::certificate_chain::CertificateRetrieverError;
     use crate::crypto_helper::{tests_setup::*, ProtocolClerk};
-    use crate::test_utils::MithrilFixtureBuilder;
+    use crate::test_utils::{MithrilFixtureBuilder, TestLogger};
 
     mock! {
         pub CertificateRetrieverImpl { }
@@ -348,7 +347,7 @@ mod tests {
             .into();
 
         let verifier = MithrilCertificateVerifier::new(
-            slog_scope::logger(),
+            TestLogger::stdout(),
             Arc::new(MockCertificateRetrieverImpl::new()),
         );
         let message_tampered = message_hash[1..].to_vec();
@@ -387,7 +386,7 @@ mod tests {
             .returning(move |_| Ok(fake_certificate2.clone()))
             .times(1);
         let verifier = MithrilCertificateVerifier::new(
-            slog_scope::logger(),
+            TestLogger::stdout(),
             Arc::new(mock_certificate_retriever),
         );
         let verify = verifier
@@ -410,7 +409,7 @@ mod tests {
             .returning(move |_| Ok(fake_certificate2.clone()))
             .times(1);
         let verifier = MithrilCertificateVerifier::new(
-            slog_scope::logger(),
+            TestLogger::stdout(),
             Arc::new(mock_certificate_retriever),
         );
         let verify = verifier
@@ -435,7 +434,7 @@ mod tests {
             .returning(move |_| Ok(fake_certificate2.clone()))
             .times(1);
         let verifier = MithrilCertificateVerifier::new(
-            slog_scope::logger(),
+            TestLogger::stdout(),
             Arc::new(mock_certificate_retriever),
         );
         let error = verifier
@@ -478,7 +477,7 @@ mod tests {
             .returning(move |_| Ok(fake_certificate2.clone()))
             .times(1);
         let verifier = MithrilCertificateVerifier::new(
-            slog_scope::logger(),
+            TestLogger::stdout(),
             Arc::new(mock_certificate_retriever),
         );
         let error = verifier
@@ -505,7 +504,7 @@ mod tests {
         fake_certificate1.hash = "another-hash".to_string();
         let mock_certificate_retriever = MockCertificateRetrieverImpl::new();
         let verifier = MithrilCertificateVerifier::new(
-            slog_scope::logger(),
+            TestLogger::stdout(),
             Arc::new(mock_certificate_retriever),
         );
         let error = verifier
@@ -537,7 +536,7 @@ mod tests {
                 .times(1);
         }
         let verifier = MithrilCertificateVerifier::new(
-            slog_scope::logger(),
+            TestLogger::stdout(),
             Arc::new(mock_certificate_retriever),
         );
         let verify = verifier
@@ -570,7 +569,7 @@ mod tests {
                 .times(1);
         }
         let verifier = MithrilCertificateVerifier::new(
-            slog_scope::logger(),
+            TestLogger::stdout(),
             Arc::new(mock_certificate_retriever),
         );
         let error = verifier