Add test gh action (#16)

* add test gh action

* oops

* clippy

* fmt

Author: seatedro

.github/workflows/test.yml

@@ -0,0 +1,32 @@
+name: Tests
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+env:
+  CARGO_TERM_COLOR: always
+
+jobs:
+  test:
+    name: Run tests
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      
+      - name: Install Rust toolchain
+        uses: dtolnay/rust-toolchain@stable
+        
+      - name: Cache dependencies
+        uses: Swatinem/rust-cache@v2
+        
+      - name: Run tests
+        run: cargo test --verbose
+        
+      - name: Run clippy
+        run: cargo clippy -- -D warnings
+        
+      - name: Check formatting
+        run: cargo fmt -- --check 

build.rs

@@ -38,7 +38,7 @@ fn main() {
     code.push_str("pub static SOURCE_EXTENSIONS: Lazy<HashSet<&'static str>> = Lazy::new(|| {\n");
     code.push_str("    let mut set = HashSet::new();\n\n");
 
-    for (_, lang) in &languages {
+    for lang in languages.values() {
         for ext in &lang.extensions {
             let ext = ext.trim_start_matches('.');
             code.push_str(&format!("    set.insert(\"{}\");\n", ext));
@@ -52,7 +52,7 @@ fn main() {
     code.push_str("pub static KNOWN_FILENAMES: Lazy<HashSet<&'static str>> = Lazy::new(|| {\n");
     code.push_str("    let mut set = HashSet::new();\n\n");
 
-    for (_, lang) in &languages {
+    for lang in languages.values() {
         for filename in &lang.filenames {
             code.push_str(&format!("    set.insert(\"{}\");\n", filename));
         }
@@ -65,7 +65,7 @@ fn main() {
     code.push_str("pub static INTERPRETER_NAMES: Lazy<HashSet<&'static str>> = Lazy::new(|| {\n");
     code.push_str("    let mut set = HashSet::new();\n\n");
 
-    for (_, lang) in &languages {
+    for lang in languages.values() {
         for interpreter in &lang.interpreters {
             code.push_str(&format!("    set.insert(\"{}\");\n", interpreter));
         }

src/analyzer.rs

@@ -33,7 +33,7 @@ pub fn process_directory(args: &Cli) -> Result<()> {
         .output
         .clone()
         .expect("output format should be set from config");
-    let entries = process_entries(&args)?;
+    let entries = process_entries(args)?;
     pb.finish();
 
     if let Some(pdf_path) = &args.pdf {

src/cli.rs

@@ -178,7 +178,6 @@ impl Cli {
     }
 }
 
-
 fn parse_exclude(value: &str) -> Result<Exclude, String> {
     let path = PathBuf::from(value);
     if path.exists() {

src/config.rs

@@ -36,7 +36,8 @@ impl<'de> Deserialize<'de> for BackwardsCompatOutputFormat {
                 let format = match s.to_lowercase().as_str() {
                     "tree" => OutputFormat::Tree,
                     "files" => OutputFormat::Files,
-                    "both" | _ => OutputFormat::Both, // Default to Both for unknown values
+                    "both" => OutputFormat::Both,
+                    _ => OutputFormat::Both, // Default to Both for unknown values
                 };
                 Ok(BackwardsCompatOutputFormat(format))
             }

src/file_picker.rs

@@ -208,7 +208,7 @@ impl FilePicker {
             if !self.show_hidden
                 && path
                     .file_name()
-                    .map_or(false, |n| n.to_string_lossy().starts_with('.'))
+                    .is_some_and(|n| n.to_string_lossy().starts_with('.'))
             {
                 continue;
             }

src/git_processor.rs

@@ -19,7 +19,7 @@ impl GitProcessor {
         let parsed_url = Url::parse(url)?;
         let repo_name = parsed_url
             .path_segments()
-            .and_then(|segments| segments.last())
+            .and_then(|mut segments| segments.next_back())
             .map(|name| name.trim_end_matches(".git"))
             .unwrap_or("repo")
             .to_string();

src/main.rs

@@ -66,12 +66,15 @@ fn main() -> anyhow::Result<()> {
             } else {
                 // Process only the specified subpaths inside the repo
                 let mut new_args = args.clone();
-                new_args.paths = subpaths.iter().map(|sub| {
-                    // Join with repo_path
-                    let mut joined = std::path::PathBuf::from(&repo_path);
-                    joined.push(sub);
-                    joined.to_string_lossy().to_string()
-                }).collect();
+                new_args.paths = subpaths
+                    .iter()
+                    .map(|sub| {
+                        // Join with repo_path
+                        let mut joined = std::path::PathBuf::from(&repo_path);
+                        joined.push(sub);
+                        joined.to_string_lossy().to_string()
+                    })
+                    .collect();
                 new_args
             };
             process_directory(&process_args)?;

src/output.rs

@@ -201,7 +201,6 @@ pub fn generate_pdf(entries: &[FileEntry], format: OutputFormat) -> Result<Vec<u
             // New page for files
             let (next_page, next_layer) = doc.add_page(Mm(210.0), Mm(297.0), "New Layer");
             current_layer = doc.get_page(next_page).get_layer(next_layer);
-            y_position = 280.0;
         }
         _ => {}
     }
@@ -212,7 +211,7 @@ pub fn generate_pdf(entries: &[FileEntry], format: OutputFormat) -> Result<Vec<u
 
         // Add file path as header
         current_layer.use_text(
-            &format!("File: {}", entry.path.display()),
+            format!("File: {}", entry.path.display()),
             14.0,
             Mm(10.0),
             Mm(y_position),
@@ -221,7 +220,7 @@ pub fn generate_pdf(entries: &[FileEntry], format: OutputFormat) -> Result<Vec<u
         y_position -= 10.0;
 
         // Add separator line
-        current_layer.use_text(&"=".repeat(48), 12.0, Mm(10.0), Mm(y_position), &font);
+        current_layer.use_text("=".repeat(48), 12.0, Mm(10.0), Mm(y_position), &font);
         y_position -= 10.0;
 
         // Add file content in smaller font

src/source_detection.rs

@@ -8,7 +8,7 @@ include!(concat!(env!("OUT_DIR"), "/languages.rs"));
 fn extract_interpreter(data: &str) -> Option<String> {
     let lines: Vec<&str> = data.lines().take(2).collect();
 
-    if !lines.get(0).map_or(false, |l| l.starts_with("#!")) {
+    if !lines.first().is_some_and(|l| l.starts_with("#!")) {
         return None;
     }
 
@@ -33,7 +33,7 @@ fn extract_interpreter(data: &str) -> Option<String> {
         return None;
     }
 
-    let mut script = first_part.split('/').last()?.to_string();
+    let mut script = first_part.split('/').next_back()?.to_string();
 
     // Handle /usr/bin/env
     if script == "env" {