97.68% region coverage is good enough (it's just error handling missing)

Author: funnyboy-roks

src/context/builder.rs

@@ -379,10 +379,10 @@ mod test {
         let ctx = TestContext::builder()
             .run_command(["echo", "foo"])
             .test("hello", "world", ())
-            .file(Path::new("./foo/bar.txt"), "/bar.txt")
+            .file(Path::new("./foo/bar.txt"), "/foo/bar.txt")
             .build();
 
-        assert_eq!(ctx.files[0].dest(), Path::new("bar.txt"));
+        assert_eq!(ctx.files[0].dest(), Path::new("foo/bar.txt"));
     }
 
     #[test]

src/context/mod.rs

@@ -29,7 +29,7 @@ impl FileConfig {
             dest.to_path_buf()
         };
 
-        FileConfig {
+        Self {
             src: src.into(),
             dest,
         }
@@ -91,6 +91,12 @@ impl From<&[u8]> for FileContent {
     }
 }
 
+impl<const N: usize> From<&[u8; N]> for FileContent {
+    fn from(value: &[u8; N]) -> Self {
+        Self::bytes(value)
+    }
+}
+
 impl FileContent {
     /// Construct a `FileContent::Path` from something that's like a path
     ///
@@ -311,6 +317,39 @@ mod test {
         assert_eq!(content, FileContent::Bytes(bytes));
     }
 
+    #[test]
+    fn commandconfig_into() {
+        let mut cfg = CommandConfig::<&str>::default();
+
+        assert_eq!(cfg.into(), CommandConfig::<String>::None);
+
+        cfg.with_run("run");
+        assert_eq!(cfg.into(), CommandConfig::<String>::Run("run".to_string()));
+
+        cfg.with_compile("compile");
+        assert_eq!(
+            cfg.into(),
+            CommandConfig::<String>::Different {
+                run: "run".to_string(),
+                compile: "compile".to_string()
+            }
+        );
+
+        let mut cfg = CommandConfig::<&str>::default();
+
+        cfg.with_compile("compile");
+        assert_eq!(
+            cfg.into(),
+            CommandConfig::<String>::Compile("compile".to_string())
+        );
+
+        cfg.with_both("both");
+        assert_eq!(
+            cfg.into(),
+            CommandConfig::<String>::Equal("both".to_string())
+        );
+    }
+
     #[test]
     fn commandconfig_run_only() {
         let mut cfg = CommandConfig::default();

src/lib.rs

@@ -111,7 +111,7 @@ mod test {
     fn bytes_from_string() {
         let string = "hello".to_string();
         let bytes = Bytes::from(string.clone());
-        assert!(matches!(bytes, Bytes::String(_)));
+        assert_eq!(bytes, Bytes::String(string.clone()));
         assert_eq!(bytes.bytes(), string.as_bytes());
         assert_eq!(bytes.as_str(), Some(&*string));
         assert_eq!(bytes.to_str_lossy(), string);
@@ -123,7 +123,7 @@ mod test {
     fn bytes_from_vec() {
         const BYTES: &[u8] = &[0xc3, 0x00, b'h', b'i'];
         let bytes = Bytes::from(BYTES.to_vec()); // not a valid string
-        assert!(matches!(bytes, Bytes::Bytes(_)));
+        assert_eq!(bytes, Bytes::Bytes(BYTES.to_vec()));
         assert_eq!(bytes.bytes(), BYTES);
         assert!(bytes.as_str().is_none());
         assert_eq!(bytes.to_str_lossy(), "�\0hi");
@@ -136,7 +136,7 @@ mod test {
         const STRING: &str = "hello";
         const BYTES: &[u8] = STRING.as_bytes();
         let bytes = Bytes::from(BYTES.to_vec());
-        assert!(matches!(bytes, Bytes::String(_)));
+        assert_eq!(bytes, Bytes::String(STRING.to_string()));
         assert_eq!(bytes.bytes(), BYTES);
         assert_eq!(bytes.as_str(), Some(STRING));
         assert_eq!(bytes.to_str_lossy(), STRING);