tests + benches: adapt

Author: Sk7Str1p3

benches/repo.rs

@@ -1,6 +1,6 @@
 use criterion::{black_box, criterion_group, criterion_main, Criterion};
 use gix::{open, ThreadSafeRepository};
-use onefetch::{cli::CliOptions, info::build_info};
+use onefetch::{cli::CliOptions, config::ConfigOptions, info::build_info};
 
 fn bench_repo_info(c: &mut Criterion) {
     let name = "make_repo.sh".to_string();
@@ -10,10 +10,11 @@ fn bench_repo_info(c: &mut Criterion) {
         input: repo.path().to_path_buf(),
         ..Default::default()
     };
+    let toml = ConfigOptions::default();
 
     c.bench_function("get repo information", |b| {
         b.iter(|| {
-            let result = black_box(build_info(&config));
+            let result = black_box(build_info(&config, &toml));
             assert!(result.is_ok());
         });
     });

src/cli.rs

@@ -417,7 +417,7 @@ mod test {
         let config: CliOptions = CliOptions {
             input: PathBuf::from("/tmp/folder"),
             info: InfoCliOptions {
-                number_of_authors: 4,
+                number_of_authors: Some(4),
                 no_merges: Some(true),
                 disabled_fields: vec![InfoType::Version, InfoType::URL].into(),
                 ..Default::default()

tests/repo.rs

@@ -1,6 +1,7 @@
 use anyhow::Result;
 use gix::{open, Repository, ThreadSafeRepository};
 use onefetch::cli::{CliOptions, InfoCliOptions, TextFormattingCliOptions};
+use onefetch::config::ConfigOptions;
 use onefetch::info::{build_info, get_work_dir};
 
 fn repo(name: &str) -> Result<Repository> {
@@ -43,12 +44,13 @@ fn test_repo() -> Result<()> {
             ..Default::default()
         },
         text_formatting: TextFormattingCliOptions {
-            iso_time: true,
+            iso_time: Some(true),
             ..Default::default()
         },
         ..Default::default()
     };
-    let info = build_info(&config)?;
+    let toml = ConfigOptions::default();
+    let info = build_info(&config, &toml)?;
     insta::assert_json_snapshot!(
         info,
         {
@@ -67,7 +69,8 @@ fn test_repo_without_remote() -> Result<()> {
         input: repo.path().to_path_buf(),
         ..Default::default()
     };
-    let info = build_info(&config);
+    let toml = ConfigOptions::default();
+    let info = build_info(&config, &toml);
     assert!(info.is_ok());
 
     Ok(())
@@ -80,7 +83,8 @@ fn test_partial_repo() -> Result<()> {
         input: repo.path().to_path_buf(),
         ..Default::default()
     };
-    let _info = build_info(&config).expect("no error");
+    let toml = ConfigOptions::default();
+    let _info = build_info(&config, &toml).expect("no error");
     Ok(())
 }
 
@@ -91,6 +95,7 @@ fn test_repo_with_pre_epoch_dates() -> Result<()> {
         input: repo.path().to_path_buf(),
         ..Default::default()
     };
-    let _info = build_info(&config).expect("no error");
+    let toml = ConfigOptions::default();
+    let _info = build_info(&config, &toml).expect("no error");
     Ok(())
 }