ISSUE-16 - Add further testing:

* Update existing testing to use rstest
* Add cases for tests where appropriate
* Pulling out sub-module from format/table
* Improve general test coverage

cargo-geiger/Cargo.toml

@@ -19,9 +19,11 @@ cargo-platform = "0.1.1"
 colored = "2.0.0"
 console = "0.11.3"
 env_logger = "0.7.1"
+fake = { version = "2.2", features=["derive"] }
 geiger = { path = "../geiger", version = "0.4.5" }
 petgraph = "0.5.1"
 pico-args = "0.3.3"
+rand = "0.7.3"
 serde = { version = "1.0.116", features = ["derive"] }
 serde_json = "1.0.57"
 strum = "0.19.2"

cargo-geiger/src/args.rs

@@ -1,6 +1,8 @@
-use crate::format::print::OutputFormat;
+use crate::format::print_config::OutputFormat;
 use crate::format::Charset;
 
+use fake::{Dummy, Fake};
+use pico_args::Arguments;
 use std::path::PathBuf;
 
 pub const HELP: &str =
@@ -54,6 +56,7 @@ OPTIONS:
     -V, --version                 Prints version information.
 ";
 
+#[derive(Debug, Dummy)]
 pub struct Args {
     pub all: bool,
     pub all_deps: bool,
@@ -86,51 +89,52 @@ pub struct Args {
 }
 
 impl Args {
-    pub fn parse_args() -> Result<Args, Box<dyn std::error::Error>> {
-        let mut args = pico_args::Arguments::from_env();
+    pub fn parse_args(
+        mut raw_args: Arguments,
+    ) -> Result<Args, Box<dyn std::error::Error>> {
         let args = Args {
-            all: args.contains(["-a", "--all"]),
-            all_deps: args.contains("--all-dependencies"),
-            all_features: args.contains("--all-features"),
-            all_targets: args.contains("--all-targets"),
-            build_deps: args.contains("--build-dependencies"),
-            charset: args
+            all: raw_args.contains(["-a", "--all"]),
+            all_deps: raw_args.contains("--all-dependencies"),
+            all_features: raw_args.contains("--all-features"),
+            all_targets: raw_args.contains("--all-targets"),
+            build_deps: raw_args.contains("--build-dependencies"),
+            charset: raw_args
                 .opt_value_from_str("--charset")?
                 .unwrap_or(Charset::Utf8),
-            color: args.opt_value_from_str("--color")?,
-            dev_deps: args.contains("--dev-dependencies"),
-            features: args.opt_value_from_str("--features")?,
-            forbid_only: args.contains(["-f", "--forbid-only"]),
-            format: args
+            color: raw_args.opt_value_from_str("--color")?,
+            dev_deps: raw_args.contains("--dev-dependencies"),
+            features: raw_args.opt_value_from_str("--features")?,
+            forbid_only: raw_args.contains(["-f", "--forbid-only"]),
+            format: raw_args
                 .opt_value_from_str("--format")?
                 .unwrap_or_else(|| "{p}".to_string()),
-            frozen: args.contains("--frozen"),
-            help: args.contains(["-h", "--help"]),
-            include_tests: args.contains("--include-tests"),
-            invert: args.contains(["-i", "--invert"]),
-            locked: args.contains("--locked"),
-            manifest_path: args.opt_value_from_str("--manifest-path")?,
-            no_default_features: args.contains("--no-default-features"),
-            no_indent: args.contains("--no-indent"),
-            offline: args.contains("--offline"),
-            package: args.opt_value_from_str("--manifest-path")?,
-            prefix_depth: args.contains("--prefix-depth"),
-            quiet: args.contains(["-q", "--quiet"]),
-            target: args.opt_value_from_str("--target")?,
-            unstable_flags: args
+            frozen: raw_args.contains("--frozen"),
+            help: raw_args.contains(["-h", "--help"]),
+            include_tests: raw_args.contains("--include-tests"),
+            invert: raw_args.contains(["-i", "--invert"]),
+            locked: raw_args.contains("--locked"),
+            manifest_path: raw_args.opt_value_from_str("--manifest-path")?,
+            no_default_features: raw_args.contains("--no-default-features"),
+            no_indent: raw_args.contains("--no-indent"),
+            offline: raw_args.contains("--offline"),
+            package: raw_args.opt_value_from_str("--manifest-path")?,
+            prefix_depth: raw_args.contains("--prefix-depth"),
+            quiet: raw_args.contains(["-q", "--quiet"]),
+            target: raw_args.opt_value_from_str("--target")?,
+            unstable_flags: raw_args
                 .opt_value_from_str("-Z")?
                 .map(|s: String| s.split(' ').map(|s| s.to_owned()).collect())
                 .unwrap_or_else(Vec::new),
             verbose: match (
-                args.contains("-vv"),
-                args.contains(["-v", "--verbose"]),
+                raw_args.contains("-vv"),
+                raw_args.contains(["-v", "--verbose"]),
             ) {
                 (false, false) => 0,
                 (false, true) => 1,
                 (true, _) => 2,
             },
-            version: args.contains(["-V", "--version"]),
-            output_format: if args.contains("--json") {
+            version: raw_args.contains(["-V", "--version"]),
+            output_format: if raw_args.contains("--json") {
                 Some(OutputFormat::Json)
             } else {
                 None
@@ -139,3 +143,65 @@ impl Args {
         Ok(args)
     }
 }
+
+#[cfg(test)]
+pub mod args_tests {
+    use super::*;
+
+    use rstest::*;
+    use std::ffi::OsString;
+
+    #[rstest(
+        input_argument_vector,
+        expected_all,
+        expected_charset,
+        expected_verbose,
+        case(
+            vec![],
+            false,
+            Charset::Utf8,
+            0
+        ),
+        case(
+            vec![OsString::from("--all")],
+            true,
+            Charset::Utf8,
+            0,
+        ),
+        case(
+            vec![OsString::from("--charset"), OsString::from("ascii")],
+            false,
+            Charset::Ascii,
+            0
+        ),
+        case(
+            vec![OsString::from("-v")],
+            false,
+            Charset::Utf8,
+            1
+        ),
+        case(
+            vec![OsString::from("-vv")],
+            false,
+            Charset::Utf8,
+            2
+        )
+    )]
+    fn parse_args_test(
+        input_argument_vector: Vec<OsString>,
+        expected_all: bool,
+        expected_charset: Charset,
+        expected_verbose: u32,
+    ) {
+        let args_result =
+            Args::parse_args(Arguments::from_vec(input_argument_vector));
+
+        assert!(args_result.is_ok());
+
+        let args = args_result.unwrap();
+
+        assert_eq!(args.all, expected_all);
+        assert_eq!(args.charset, expected_charset);
+        assert_eq!(args.verbose, expected_verbose)
+    }
+}

cargo-geiger/src/cli.rs

@@ -108,7 +108,9 @@ pub fn resolve<'a, 'cfg>(
 mod cli_tests {
     use super::*;
 
-    #[test]
+    use rstest::*;
+
+    #[rstest]
     fn get_cfgs_test() {
         let config = Config::default().unwrap();
 
@@ -135,11 +137,11 @@ mod cli_tests {
             .filter(|cfg| matches!(cfg, Cfg::KeyPair(_, _)))
             .collect();
 
-        assert!(names.len() > 0);
-        assert!(key_pairs.len() > 0);
+        assert!(!names.is_empty());
+        assert!(!key_pairs.is_empty());
     }
 
-    #[test]
+    #[rstest]
     fn get_registry_test() {
         let config = Config::default().unwrap();
         let workspace = Workspace::new(
@@ -162,7 +164,7 @@ mod cli_tests {
         assert_eq!(package_set.sources().len(), 1);
     }
 
-    #[test]
+    #[rstest]
     fn get_workspace_test() {
         let config = Config::default().unwrap();
         let manifest_path: Option<PathBuf> = None;

cargo-geiger/src/format.rs

@@ -1,22 +1,24 @@
 pub mod emoji_symbols;
 pub mod pattern;
-pub mod print;
+pub mod print_config;
 pub mod table;
 
 mod display;
 mod parse;
 
 use cargo::core::dependency::DepKind;
+use fake::Dummy;
 use std::fmt;
 use std::str::{self, FromStr};
 use strum_macros::EnumIter;
 
-#[derive(Clone, Copy, Debug, PartialEq)]
+#[derive(Clone, Copy, Debug, Dummy, PartialEq)]
 pub enum Charset {
     Ascii,
     Utf8,
 }
 
+#[derive(Debug, PartialEq)]
 pub enum Chunk {
     License,
     Package,
@@ -43,6 +45,7 @@ pub enum CrateDetectionStatus {
     UnsafeDetected,
 }
 
+#[derive(Debug, PartialEq)]
 pub enum RawChunk<'a> {
     Argument(&'a str),
     Error(&'static str),
@@ -83,14 +86,16 @@ pub fn get_kind_group_name(dep_kind: DepKind) -> Option<&'static str> {
 mod format_tests {
     use super::*;
 
-    #[test]
+    use rstest::*;
+
+    #[rstest]
     fn charset_from_str_test() {
         assert_eq!(Charset::from_str("ascii"), Ok(Charset::Ascii));
         assert_eq!(Charset::from_str("utf8"), Ok(Charset::Utf8));
         assert_eq!(Charset::from_str("invalid_str"), Err("invalid charset"));
     }
 
-    #[test]
+    #[rstest]
     fn get_kind_group_name_test() {
         assert_eq!(
             get_kind_group_name(DepKind::Build),