file: impl fn fn get_magic_files

Author: fox0

file/file.rs

@@ -547,27 +547,18 @@ fn get_type_from_magic_file_dbs(test_file: &PathBuf, magic_file_dbs: &[PathBuf])
     })
 }
 
-/// Get the default raw(text based) magic file
-fn get_default_magic_file() -> PathBuf {
-    #[cfg(target_os = "macos")]
-    {
-        PathBuf::from("/usr/share/file/magic/magic")
-    }
-    #[cfg(not(target_os = "macos"))]
-    {
-        PathBuf::from("/etc/magic")
-    }
-}
-
-fn analyze_file(mut path: String, args: &Args) {
+#[cfg(target_os = "macos")]
+/// Default raw (text based) magic file
+const DEFAULT_MAGIC_FILE: &str = "/usr/share/file/magic/magic";
+#[cfg(not(target_os = "macos"))]
+/// Default raw (text based) magic file
+const DEFAULT_MAGIC_FILE: &str = "/etc/magic";
+
+fn get_magic_files(args: &Args) -> Vec<PathBuf> {
     // set priority according to the occurence of flags in the args lowest index will get highest priority
     let mut magic_files: Vec<PathBuf> = Vec::new();
 
-    if path == "-" {
-        path = String::new();
-        io::stdin().read_line(&mut path).unwrap();
-        path = path.trim().to_string();
-    }
+    let default_magic_file = PathBuf::from(DEFAULT_MAGIC_FILE);
 
     if let Some(test_file2) = &args.test_file2 {
         magic_files.push(test_file2.clone());
@@ -579,24 +570,34 @@ fn analyze_file(mut path: String, args: &Args) {
 
             if m_index > h_index {
                 magic_files.push(args.test_file1.as_ref().unwrap().clone());
-                magic_files.push(get_default_magic_file());
+                magic_files.push(default_magic_file);
             } else {
-                magic_files.push(get_default_magic_file());
+                magic_files.push(default_magic_file);
                 magic_files.push(args.test_file1.as_ref().unwrap().clone());
             }
         } else if args.test_file1.is_some() {
             magic_files.push(args.test_file1.as_ref().unwrap().clone());
         } else if args.default_tests {
-            magic_files.push(get_default_magic_file());
+            magic_files.push(default_magic_file);
         }
     } else if let Some(test_file1) = &args.test_file1 {
         magic_files.push(test_file1.clone());
 
         if args.test_file2.is_none() && !args.default_tests {
-            magic_files.push(get_default_magic_file());
+            magic_files.push(default_magic_file);
         }
     } else {
-        magic_files.push(get_default_magic_file());
+        magic_files.push(default_magic_file);
+    }
+
+    magic_files
+}
+
+fn analyze_file(mut path: String, args: &Args, magic_files: &Vec<PathBuf>) {
+    if path == "-" {
+        path = String::new();
+        io::stdin().read_line(&mut path).unwrap();
+        path = path.trim().to_string();
     }
 
     let met = match fs::symlink_metadata(&path) {
@@ -682,8 +683,10 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
     textdomain(PROJECT_NAME).unwrap();
     bind_textdomain_codeset(PROJECT_NAME, "UTF-8").unwrap();
 
+    let magic_files = get_magic_files(&args);
+
     for file in &args.files {
-        analyze_file(file.clone(), &args);
+        analyze_file(file.clone(), &args, &magic_files);
     }
 
     Ok(())

file/tests/file/mod.rs

@@ -38,6 +38,13 @@ fn file_is_a_directory() {
     file_test(&[file], &format!("{file}: directory\n"), "");
 }
 
+#[test]
+fn file_is_a_character_special() {
+    let file = "/dev/null";
+
+    file_test(&[file], &format!("{file}: character special\n"), "");
+}
+
 #[test]
 fn file_is_an_empty_file() {
     let file = "tests/file/empty_file.txt";
@@ -120,17 +127,6 @@ fn file_file_is_a_broken_sym_link() {
     remove_file(broken_sym_link).unwrap()
 }
 
-#[test]
-fn file_is_a_character_special() {
-    let file = PathBuf::from("/dev/null");
-
-    file_test(
-        &[file.to_str().unwrap()],
-        &format!("{}: character special\n", file.to_str().unwrap()),
-        "",
-    );
-}
-
 #[test]
 fn file_symlink_with_h_flag_for_both_valid_and_broken_symlink() {
     use std::env;