ci: add coveralls coverage reporting

- increase test coverage

.github/workflows/ci.yml

@@ -44,7 +44,13 @@ jobs:
         uses: swatinem/rust-cache@v2
 
       - name: Run Tests
-        run: cargo test --locked
+        run: cargo llvm-cov --locked
+
+      - name: Generate Coverage File
+        run: cargo llvm-cov report --lcov --output-path lcov.info --locked
+
+      - name: Coveralls
+        uses: coverallsapp/github-action@v2
 
       - name: Build
         run: cargo build -r --target ${{ matrix.target }} --locked

.gitignore

@@ -1,3 +1,4 @@
 /target
 /result
 *.vdf
+lcov.info

shell.nix

@@ -10,6 +10,8 @@ let
     nixpkgs-fmt
     rustc
     cargo
+    rustc.llvmPackages.llvm
+    cargo-llvm-cov
     gcc
     rustfmt
     clippy
@@ -38,5 +40,7 @@ pkgs.mkShell {
   inherit buildInputs;
 
   RUST_SRC_PATH = "${pkgs.rust.packages.stable.rustPlatform.rustLibSrc}";
+  LLVM_COV = "${pkgs.rustc.llvmPackages.llvm}/bin/llvm-cov";
+  LLVM_PROFDATA = "${pkgs.rustc.llvmPackages.llvm}/bin/llvm-profdata";
   RUST_LOG = "trace";
 }

src/utils/command.rs

@@ -42,3 +42,38 @@ pub fn split(command: &str) -> Result<Vec<String>, String> {
 
     Ok(command_args)
 }
+
+#[cfg(test)]
+mod tests {
+    use crate::utils::command::{join, split};
+
+    fn as_strings(values: Vec<&str>) -> Vec<String> {
+        values.iter().map(|v| v.to_string()).collect()
+    }
+
+    #[test]
+    fn join_tests() {
+        assert_eq!(join(vec!["echo 'hello'"]), Ok("echo 'hello'".to_string()));
+        assert_eq!(
+            join(vec!["echo", "\"this\"", "is", "a", "test"]),
+            Ok("echo '\"this\"' is a test".to_string())
+        );
+        assert_eq!(
+            join(vec!["\0", "\0"]),
+            Err("Failed to parse command!".to_string())
+        );
+    }
+
+    #[test]
+    fn split_tests() {
+        assert_eq!(split("echo 'hello'"), Ok(as_strings(vec!["echo", "hello"])));
+        assert_eq!(
+            split("echo '\"this\"' is a test"),
+            Ok(as_strings(vec!["echo", "\"this\"", "is", "a", "test"]))
+        );
+        assert_eq!(
+            split("echo 'hello"),
+            Err("Failed to parse command!".to_string())
+        );
+    }
+}

src/utils/env.rs

@@ -3,9 +3,25 @@ pub fn convert_bool(value: bool) -> String {
 }
 
 pub fn convert_bool_as(value: bool, false_value: &'static str, true_value: &'static str) -> String {
-    if value {
-        false_value.to_string()
-    } else {
-        true_value.to_string()
+    match value {
+        true => true_value.to_string(),
+        false => false_value.to_string(),
+    }
+}
+
+#[cfg(test)]
+mod tests {
+    use crate::utils::env::{convert_bool, convert_bool_as};
+
+    #[test]
+    fn convert_bool_should_map_to_0_or_1() {
+        assert_eq!(convert_bool(false), "0");
+        assert_eq!(convert_bool(true), "1");
+    }
+
+    #[test]
+    fn convert_bool_as_should_support_custom_values() {
+        assert_eq!(convert_bool_as(false, "false", "true"), "false");
+        assert_eq!(convert_bool_as(true, "false", "true"), "true");
     }
 }