Broke out bytes_to_hex()

cryspen · Aug 1, 2023 · 6e907ae · 6e907ae
1 parent 17fcf8e
commit 6e907ae
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 18 deletions.
diff --git a/src/jasmin.rs b/src/jasmin.rs
@@ -12,3 +12,16 @@ pub mod x25519;
 
 #[cfg(any(target_os = "linux", target_os = "macos"))]
 pub mod kyber_derand;
+
+#[cfg(test)]
+mod testing {
+    use std::fmt::Write;
+
+    pub(crate) fn bytes_to_hex(bytes: &[u8]) -> String {
+        let mut s = String::with_capacity(2 * bytes.len());
+        for byte in bytes {
+            write!(s, "{:02x}", byte).unwrap();
+        }
+        s
+    }
+}
diff --git a/src/jasmin/sha2.rs b/src/jasmin/sha2.rs
@@ -26,15 +26,7 @@ pub fn sha256(input: &[u8]) -> Result<Sha256Digest, &'static str> {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use std::fmt::Write;
-
-    fn bytes_to_hex(bytes: &[u8]) -> String {
-        let mut s = String::with_capacity(2 * bytes.len());
-        for byte in bytes {
-            write!(s, "{:02x}", byte).unwrap();
-        }
-        s
-    }
+    use crate::jasmin::testing::bytes_to_hex;
 
     #[test]
     fn test_hash() {

diff --git a/src/jasmin/x25519.rs b/src/jasmin/x25519.rs
@@ -82,15 +82,7 @@ pub(crate) mod mulx {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use std::fmt::Write;
-
-    fn bytes_to_hex(bytes: &[u8]) -> String {
-        let mut s = String::with_capacity(2 * bytes.len());
-        for byte in bytes {
-            write!(s, "{:02x}", byte).unwrap();
-        }
-        s
-    }
+    use crate::jasmin::testing::bytes_to_hex;
 
     fn bmi2_and_adx_are_supported() -> bool {
         std::arch::is_x86_feature_detected!("bmi2") && std::arch::is_x86_feature_detected!("adx")