Fix pre-commits

Author: PaulKMueller

src/lib.rs

@@ -29,11 +29,9 @@ pub type CliInput<'a> = (
 );
 pub type CheckOutput = (Vec<LicenseInfo>, Vec<LicenseInfo>);
 
-pub fn fetch_license_infos(
-    cli_input: CliInput,
-) -> Result<LicenseInfos> {
-
-    let (conda_deny_config, cli_lockfiles, cli_platforms, cli_environments, conda_prefixes, _) = cli_input;
+pub fn fetch_license_infos(cli_input: CliInput) -> Result<LicenseInfos> {
+    let (conda_deny_config, cli_lockfiles, cli_platforms, cli_environments, conda_prefixes, _) =
+        cli_input;
 
     if conda_prefixes.is_empty() {
         LicenseInfos::get_license_infos_from_config(
@@ -44,22 +42,23 @@ pub fn fetch_license_infos(
         )
         .with_context(|| "Getting license information from config file failed.")
     } else {
-        LicenseInfos::from_conda_prefixes(conda_prefixes).with_context(|| {
-            "Getting license information from conda prefixes failed."
-        })
+        LicenseInfos::from_conda_prefixes(conda_prefixes)
+            .with_context(|| "Getting license information from conda prefixes failed.")
     }
 }
 
 pub fn list(cli_input: CliInput) -> Result<()> {
-    let license_infos = fetch_license_infos(cli_input).with_context(|| "Fetching license information failed.")?;
+    let license_infos =
+        fetch_license_infos(cli_input).with_context(|| "Fetching license information failed.")?;
     license_infos.list();
     Ok(())
 }
 
 pub fn check_license_infos(cli_input: CliInput) -> Result<CheckOutput> {
     let (conda_deny_config, _, _, _, _, osi) = cli_input;
 
-    let license_infos = fetch_license_infos(cli_input).with_context(|| "Fetching license information failed.")?;
+    let license_infos =
+        fetch_license_infos(cli_input).with_context(|| "Fetching license information failed.")?;
 
     if osi {
         debug!("Checking licenses for OSI compliance");

src/license_info.rs

@@ -217,17 +217,17 @@ impl LicenseInfos {
 
     pub fn get_license_infos_from_config(
         config: &CondaDenyConfig,
-        cli_lockfiles: &Vec<String>,
-        cli_platforms: &Vec<String>,
-        cli_environments: &Vec<String>,
+        cli_lockfiles: &[String],
+        cli_platforms: &[String],
+        cli_environments: &[String],
     ) -> Result<LicenseInfos> {
         let mut platforms = config.get_platform_spec().map_or(vec![], |p| p);
         let mut lockfiles = config.get_lockfile_spec();
         let mut environment_specs = config.get_environment_spec().map_or(vec![], |e| e);
 
-        platforms.extend(cli_platforms.clone());
-        lockfiles.extend(cli_lockfiles.clone());
-        environment_specs.extend(cli_environments.clone());
+        platforms.extend(cli_platforms.to_owned());
+        lockfiles.extend(cli_lockfiles.to_owned());
+        environment_specs.extend(cli_environments.to_owned());
 
         LicenseInfos::from_pixi_lockfiles(lockfiles, platforms, environment_specs)
     }
@@ -469,8 +469,7 @@ mod tests {
             "tests/test_pyproject_toml_files/"
         );
         let config = CondaDenyConfig::from_path(&test_file_path).expect("Failed to read config");
-        let license_infos =
-            LicenseInfos::get_license_infos_from_config(&config, &vec![], &vec![], &vec![]);
+        let license_infos = LicenseInfos::get_license_infos_from_config(&config, &[], &[], &[]);
         assert_eq!(license_infos.unwrap().license_infos.len(), 396);
     }
 }

src/main.rs

@@ -74,7 +74,6 @@ fn main() -> Result<()> {
             include_safe,
             osi: _,
         } => {
-
             debug!("Check command called.");
 
             if include_safe {

tests/integration_tests.rs

@@ -99,9 +99,9 @@ mod tests {
     fn test_prefix_list() {
         // When --prefix is specified, only the license information for the conda-meta directory in the specified prefix should be listed
         // License information from pixi.lock should not be listed
-    
+
         let test_dir = Path::new("tests/test_end_to_end/test_prefix_list");
-    
+
         let output = Command::cargo_bin("conda-deny")
             .unwrap()
             .arg("list")
@@ -110,16 +110,22 @@ mod tests {
             .current_dir(test_dir)
             .output()
             .expect("Failed to execute command");
-    
-        assert!(output.status.success(), "Command did not execute successfully");
-    
+
+        assert!(
+            output.status.success(),
+            "Command did not execute successfully"
+        );
+
         let stdout = str::from_utf8(&output.stdout).expect("Failed to convert output to string");
-    
+
         let line_count = stdout.lines().count();
 
         let expected_line_count = 50;
-        assert_eq!(line_count, expected_line_count, "Unexpected number of output lines");
-    
+        assert_eq!(
+            line_count, expected_line_count,
+            "Unexpected number of output lines"
+        );
+
         println!("Output has {} lines", line_count);
     }
 }