Fix #21 by dropping ctor

Cargo.toml

@@ -20,8 +20,6 @@ base64 = "0.21.0"
 once_cell = "1.17.1"
 
 [dev-dependencies]
-# Simplifies test setup and the `waitid` hack.
-ctor = "0.2.0"
 quickcheck = "1.0.3"
 quickcheck_macros = "1.0.0"
 tempfile = "3.4.0"

src/parent/ipc/test_utils.rs

@@ -74,6 +74,7 @@ impl StaticState {
     }
 
     pub fn init_test_state_with_key_dir(&'static self, key_dir: std::path::PathBuf) {
+        init_logger();
         self.state.init_dynamic(ParentIpcDynamic {
             port: std::num::NonZeroU16::new(1).unwrap(),
             child_user_group: UserGroup {

src/state/byte_count_map.rs

@@ -296,6 +296,7 @@ mod tests {
 
     #[test]
     fn works_on_one_thread() {
+        init_logger();
         static S: PromState = PromState::new();
 
         S.add_message_line_ingested(&map_key(b"one"), 123);
@@ -372,6 +373,7 @@ mod tests {
 
     #[test]
     fn works_on_two_contending_threads() {
+        init_logger();
         // Pre-allocate everything so it won't take a long time to run with all the allocations.
         // This is one of the slowest tests in Miri, so it's particularly helpful here.
         static S: PromState = PromState::new();
@@ -468,6 +470,7 @@ mod tests {
     // Extremely slow in Miri (might very well take over an hour). Just skip it.
     #[cfg_attr(miri, ignore)]
     fn works_on_a_hundred_contending_threads() {
+        init_logger();
         // Pre-allocate everything so it won't take a long time to run with all the allocations.
         static S: PromState = PromState::new();
 

src/test_utils/spy/logger_spy.rs

@@ -162,21 +162,23 @@ impl std::ops::Deref for LoggerCaptureGuard {
     }
 }
 
-// Initialize this for all tests. Easier than trying to pump it through literally everything, and
-// I can also use this to track down any/all stray logs.
-#[ctor::ctor]
-fn init_logger() {
-    // Don't log `trace!(...)` stuff.
-    if std::env::var("RUST_TRACE") == Ok("1".into()) {
-        log::set_max_level(log::LevelFilter::Trace);
-    } else {
-        log::set_max_level(log::LevelFilter::Debug);
-    }
-    log::set_logger(&TEST_LOGGER).unwrap();
+pub fn init_logger() {
+    static ONCE: std::sync::Once = std::sync::Once::new();
+    ONCE.call_once(|| {
+        // Don't log `trace!(...)` stuff.
+        let level = if std::env::var("RUST_TRACE") == Ok("1".into()) {
+            log::LevelFilter::Trace
+        } else {
+            log::LevelFilter::Debug
+        };
+        log::set_max_level(level);
+        log::set_logger(&TEST_LOGGER).unwrap();
+    });
 }
 
 #[must_use = "The returned guard must be live for the whole test to ensure all logs are captured."]
 pub fn setup_capture_logger() -> LoggerCaptureGuard {
+    init_logger();
     LoggerCaptureGuard(LoggerGuard::new(LoggerKind::LoggerVec(LogCapture {
         inner: Mutex::new(Vec::new()),
     })))