clean up test_utils

Author: cowlicks

src/noise.rs

@@ -318,7 +318,9 @@ where
                 }
                 Ok(vec![])
             }
-            Step::Established((..)) => todo!(),
+            Step::Established(_) => {
+                unreachable!("`handle_setup_message` should never be called when Step::Established")
+            }
         }
     }
     #[instrument(skip_all)]

src/test_utils.rs

@@ -97,27 +97,6 @@ pub(crate) fn log() {
     });
 }
 
-#[tokio::test]
-async fn way_one() {
-    use futures::SinkExt;
-    let mut a = Io::default();
-    let _ = a.send(b"hello".into()).await;
-    let Some(res) = a.next().await else { panic!() };
-    assert_eq!(res, b"hello");
-}
-
-#[tokio::test]
-async fn split() {
-    use futures::SinkExt;
-    let (mut left, mut right) = (TwoWay::default()).split_sides();
-
-    left.send(b"hello".to_vec()).await.unwrap();
-    let Some(res) = right.next().await else {
-        panic!();
-    };
-    assert_eq!(res, b"hello");
-}
-
 pub(crate) struct Moo<Rx, Tx> {
     receiver: Rx,
     sender: Tx,
@@ -199,3 +178,26 @@ pub(crate) fn create_result_connected() -> (
     let b = Moo::from(result_channel());
     a.connect(b)
 }
+
+#[cfg(test)]
+mod test_test_utils {
+    use super::*;
+    use futures::SinkExt;
+    #[tokio::test]
+    async fn way_one() {
+        let mut a = Io::default();
+        let _ = a.send(b"hello".into()).await;
+        let Some(res) = a.next().await else { panic!() };
+        assert_eq!(res, b"hello");
+    }
+
+    #[tokio::test]
+    async fn split() {
+        let (mut left, mut right) = (TwoWay::default()).split_sides();
+        left.send(b"hello".to_vec()).await.unwrap();
+        let Some(res) = right.next().await else {
+            panic!();
+        };
+        assert_eq!(res, b"hello");
+    }
+}