refactor(tests): use define_test_logger instead of manual creation

And move its definition to a `test` module instead of a `test_utils` to
be in line with the latest naming scheme (only for crates that already
have a `test` module).

Author: Alenar

internal/cardano-node/mithril-cardano-node-chain/src/test/mod.rs

@@ -8,22 +8,4 @@ pub mod double;
 #[cfg(test)]
 pub(crate) mod test_cli_runner;
 
-#[cfg(test)]
-pub(crate) struct TestLogger;
-
-#[cfg(test)]
-impl TestLogger {
-    fn from_writer<W: std::io::Write + Send + 'static>(writer: W) -> slog::Logger {
-        use slog::Drain;
-        use std::sync::Arc;
-
-        let decorator = slog_term::PlainDecorator::new(writer);
-        let drain = slog_term::CompactFormat::new(decorator).build().fuse();
-        let drain = slog_async::Async::new(drain).build().fuse();
-        slog::Logger::root(Arc::new(drain), slog::o!())
-    }
-
-    pub(crate) fn stdout() -> slog::Logger {
-        Self::from_writer(slog_term::TestStdoutWriter)
-    }
-}
+mithril_common::define_test_logger!();

internal/cardano-node/mithril-cardano-node-internal-database/src/test/mod.rs

@@ -10,22 +10,4 @@ pub mod fake_data;
 
 pub use dummy_cardano_db::{DummyCardanoDb, DummyCardanoDbBuilder};
 
-#[cfg(test)]
-pub(crate) struct TestLogger;
-
-#[cfg(test)]
-impl TestLogger {
-    fn from_writer<W: std::io::Write + Send + 'static>(writer: W) -> slog::Logger {
-        use slog::Drain;
-        use std::sync::Arc;
-
-        let decorator = slog_term::PlainDecorator::new(writer);
-        let drain = slog_term::CompactFormat::new(decorator).build().fuse();
-        let drain = slog_async::Async::new(drain).build().fuse();
-        slog::Logger::root(Arc::new(drain), slog::o!())
-    }
-
-    pub(crate) fn stdout() -> slog::Logger {
-        Self::from_writer(slog_term::TestStdoutWriter)
-    }
-}
+mithril_common::define_test_logger!();

internal/mithril-dmq/src/consumer/pallas.rs

@@ -151,7 +151,7 @@ mod tests {
     };
     use tokio::{net::UnixListener, task::JoinHandle, time::sleep};
 
-    use crate::{test::payload::DmqMessageTestPayload, test_tools::TestLogger};
+    use crate::test::{TestLogger, payload::DmqMessageTestPayload};
 
     use super::*;
 

internal/mithril-dmq/src/lib.rs

@@ -9,28 +9,3 @@ pub mod test;
 pub use consumer::{DmqConsumer, DmqConsumerPallas};
 pub use message::DmqMessageBuilder;
 pub use publisher::{DmqPublisher, DmqPublisherPallas};
-
-#[cfg(test)]
-pub(crate) mod test_tools {
-    use std::io;
-    use std::sync::Arc;
-
-    use slog::{Drain, Logger};
-    use slog_async::Async;
-    use slog_term::{CompactFormat, PlainDecorator};
-
-    pub struct TestLogger;
-
-    impl TestLogger {
-        fn from_writer<W: io::Write + Send + 'static>(writer: W) -> Logger {
-            let decorator = PlainDecorator::new(writer);
-            let drain = CompactFormat::new(decorator).build().fuse();
-            let drain = Async::new(drain).build().fuse();
-            Logger::root(Arc::new(drain), slog::o!())
-        }
-
-        pub fn stdout() -> Logger {
-            Self::from_writer(slog_term::TestStdoutWriter)
-        }
-    }
-}

internal/mithril-dmq/src/publisher/pallas.rs

@@ -96,7 +96,7 @@ mod tests {
         test::{TempDir, double::Dummy},
     };
 
-    use crate::{test::payload::DmqMessageTestPayload, test_tools::TestLogger};
+    use crate::test::{TestLogger, payload::DmqMessageTestPayload};
 
     use super::*;
 

internal/mithril-dmq/src/test/mod.rs

@@ -8,3 +8,5 @@ pub mod double;
 
 #[cfg(test)]
 pub(crate) mod payload;
+
+mithril_common::define_test_logger!();

internal/mithril-metric/src/helper.rs

@@ -106,26 +106,7 @@ macro_rules! build_metrics_service {
 
 #[cfg(test)]
 pub(crate) mod test_tools {
-    use std::{io, sync::Arc};
-
-    use slog::{Drain, Logger};
-    use slog_async::Async;
-    use slog_term::{CompactFormat, PlainDecorator};
-
-    pub struct TestLogger;
-
-    impl TestLogger {
-        fn from_writer<W: io::Write + Send + 'static>(writer: W) -> Logger {
-            let decorator = PlainDecorator::new(writer);
-            let drain = CompactFormat::new(decorator).build().fuse();
-            let drain = Async::new(drain).build().fuse();
-            Logger::root(Arc::new(drain), slog::o!())
-        }
-
-        pub fn stdout() -> Logger {
-            Self::from_writer(slog_term::TestStdoutWriter)
-        }
-    }
+    mithril_common::define_test_logger!();
 }
 
 #[cfg(test)]

internal/signed-entity/mithril-signed-entity-preloader/src/lib.rs

@@ -9,25 +9,5 @@ pub use cardano_transactions_preloader::*;
 
 #[cfg(test)]
 pub(crate) mod test_tools {
-    use std::io;
-    use std::sync::Arc;
-
-    use slog::{Drain, Logger};
-    use slog_async::Async;
-    use slog_term::{CompactFormat, PlainDecorator};
-
-    pub struct TestLogger;
-
-    impl TestLogger {
-        fn from_writer<W: io::Write + Send + 'static>(writer: W) -> Logger {
-            let decorator = PlainDecorator::new(writer);
-            let drain = CompactFormat::new(decorator).build().fuse();
-            let drain = Async::new(drain).build().fuse();
-            Logger::root(Arc::new(drain), slog::o!())
-        }
-
-        pub fn stdout() -> Logger {
-            Self::from_writer(slog_term::TestStdoutWriter)
-        }
-    }
+    mithril_common::define_test_logger!();
 }

mithril-aggregator/src/artifact_builder/cardano_database.rs

@@ -125,7 +125,7 @@ mod tests {
         immutable_file_digest_mapper::MockImmutableFileDigestMapper,
         services::CompressedArchiveSnapshotter,
         services::ancillary_signer::MockAncillarySigner,
-        test_tools::TestLogger,
+        test::TestLogger,
         tools::{file_archiver::FileArchiver, url_sanitizer::SanitizedUrlWithTrailingSlash},
     };
 

mithril-aggregator/src/artifact_builder/cardano_database_artifacts/ancillary.rs

@@ -209,7 +209,7 @@ mod tests {
     use mithril_common::test::{TempDir, assert_equivalent, double::Dummy};
 
     use crate::services::{DumbSnapshotter, MockSnapshotter};
-    use crate::test_tools::TestLogger;
+    use crate::test::TestLogger;
 
     use super::*;