Add support for custom executable_name in config

Author: perryqh

src/config.rs

@@ -128,6 +128,40 @@ mod tests {
             vec!["frontend/**/node_modules/**/*", "frontend/**/__generated__/**/*"]
         );
         assert_eq!(config.vendored_gems_path, "vendored/");
+        assert_eq!(config.executable_name, "codeowners");
+        Ok(())
+    }
+
+    #[test]
+    fn test_parse_config_with_custom_executable_name() -> Result<(), Box<dyn Error>> {
+        let temp_dir = tempdir()?;
+        let config_path = temp_dir.path().join("config.yml");
+        let config_str = indoc! {"
+            ---
+            owned_globs:
+              - \"**/*.rb\"
+            executable_name: my-custom-codeowners
+        "};
+        fs::write(&config_path, config_str)?;
+        let config_file = File::open(&config_path)?;
+        let config: Config = serde_yaml::from_reader(config_file)?;
+        assert_eq!(config.executable_name, "my-custom-codeowners");
+        Ok(())
+    }
+
+    #[test]
+    fn test_executable_name_defaults_when_not_specified() -> Result<(), Box<dyn Error>> {
+        let temp_dir = tempdir()?;
+        let config_path = temp_dir.path().join("config.yml");
+        let config_str = indoc! {"
+            ---
+            owned_globs:
+              - \"**/*.rb\"
+        "};
+        fs::write(&config_path, config_str)?;
+        let config_file = File::open(&config_path)?;
+        let config: Config = serde_yaml::from_reader(config_file)?;
+        assert_eq!(config.executable_name, "codeowners");
         Ok(())
     }
 }

src/ownership/file_owner_resolver.rs

@@ -293,6 +293,7 @@ mod tests {
             vendored_gems_path: vendored_path.to_string(),
             cache_directory: "tmp/cache/codeowners".to_string(),
             ignore_dirs: vec![],
+            executable_name: "codeowners".to_string(),
         }
     }
 

src/ownership/validator.rs

@@ -169,7 +169,7 @@ impl Error {
                 Error::FileWithoutOwner { path: _ } => "Some files are missing ownership".to_owned(),
                 Error::FileWithMultipleOwners { path: _, owners: _ } => "Code ownership should only be defined for each file in one way. The following files have declared ownership in multiple ways".to_owned(),
                 Error::CodeownershipFileIsStale { executable_name } => {
-                    format!("CODEOWNERS out of date. Run `{} generate` to update the CODEOWNERS file", executable_name.to_string())
+                    format!("CODEOWNERS out of date. Run `{} generate` to update the CODEOWNERS file", executable_name)
                 }
                 Error::InvalidTeam { name: _, path: _ } => "Found invalid team annotations".to_owned(),
             }

src/project.rs

@@ -220,6 +220,7 @@ mod tests {
             codeowners_file_path: PathBuf::from(".github/CODEOWNERS"),
             directory_codeowner_files: vec![],
             teams_by_name: HashMap::new(),
+            executable_name: "codeowners".to_string(),
         };
 
         let map = project.vendored_gem_by_name();

tests/custom_executable_name_test.rs

@@ -0,0 +1,33 @@
+use predicates::prelude::*;
+use std::error::Error;
+
+mod common;
+use common::OutputStream;
+use common::run_codeowners;
+
+#[test]
+fn test_validate_uses_custom_executable_name() -> Result<(), Box<dyn Error>> {
+    run_codeowners(
+        "custom_executable_name",
+        &["validate"],
+        false,
+        OutputStream::Stdout,
+        predicate::str::contains("CODEOWNERS out of date. Run `my-custom-tool generate` to update the CODEOWNERS file"),
+    )?;
+    Ok(())
+}
+
+#[test]
+fn test_validate_default_executable_name() -> Result<(), Box<dyn Error>> {
+    // This tests the invalid_project which doesn't specify executable_name
+    // and should use the default "codeowners"
+    run_codeowners(
+        "invalid_project",
+        &["validate"],
+        false,
+        OutputStream::Stdout,
+        predicate::str::contains("CODEOWNERS out of date. Run `codeowners generate` to update the CODEOWNERS file"),
+    )?;
+    Ok(())
+}
+

tests/fixtures/custom_executable_name/.github/CODEOWNERS

@@ -0,0 +1,3 @@
+# This is a stale CODEOWNERS file - it's intentionally wrong to trigger the error
+ruby/app/payments/bar.rb @PaymentTeam
+

tests/fixtures/custom_executable_name/config/code_ownership.yml

@@ -0,0 +1,6 @@
+owned_globs:
+  - "**/*.rb"
+team_file_glob:
+  - config/teams/**/*.yml
+executable_name: my-custom-tool
+

tests/fixtures/custom_executable_name/config/teams/payments.yml

@@ -0,0 +1,6 @@
+name: Payments
+github:
+  team: "@PaymentTeam"
+owned_globs:
+  - ruby/app/payments/**
+

tests/fixtures/custom_executable_name/ruby/app/payments/foo.rb

@@ -0,0 +1,4 @@
+# A file owned by Payments
+class Foo
+end
+