stubbing out teams_for_files_from_codeowners

Author: perryqh

src/ownership.rs

@@ -13,7 +13,7 @@ mod file_generator;
 mod file_owner_finder;
 pub mod for_file_fast;
 pub(crate) mod mapper;
-pub(crate) mod parser;
+pub(crate) mod codeowners_file_parser;
 mod validator;
 
 use crate::{
@@ -26,7 +26,7 @@ pub use validator::Errors as ValidatorErrors;
 use self::{
     file_generator::FileGenerator,
     mapper::{JavascriptPackageMapper, Mapper, RubyPackageMapper, TeamFileMapper, TeamGemMapper, TeamGlobMapper, TeamYmlMapper},
-    parser::parse_for_team,
+    codeowners_file_parser::parse_for_team,
     validator::Validator,
 };
 

src/ownership/parser.rs src/ownership/codeowners_file_parser.rssrc/ownership/parser.rs renamed to src/ownership/codeowners_file_parser.rs

@@ -22,6 +22,10 @@ pub struct Parser {
 }
 
 impl Parser {
+    pub fn teams_from_files_paths(&self, file_paths: &[PathBuf]) -> Result<HashMap<String, Team>, Box<dyn Error>> {
+        todo!()
+    }
+    
     pub fn team_from_file_path(&self, file_path: &Path) -> Result<Option<Team>, Box<dyn Error>> {
         let file_path_str = file_path
             .to_str()

src/runner.rs

@@ -1,5 +1,6 @@
 use core::fmt;
 use std::{
+    collections::HashMap,
     fs::File,
     path::{Path, PathBuf},
     process::Command,
@@ -280,17 +281,36 @@ fn for_file_codeowners_only(run_config: &RunConfig, file_path: &str) -> RunResul
         },
     }
 }
-pub fn team_for_file_from_codeowners(run_config: &RunConfig, file_path: &str) -> Result<Option<Team>, Error> {
+
+// For an array of file paths, return a map of file path to its owning team
+pub fn teams_for_files_from_codeowners(run_config: &RunConfig, file_paths: &[String]) -> Result<HashMap<String, Team>, Error> {
+    let relative_file_paths: Vec<PathBuf> = file_paths
+        .iter()
+        .map(|path| Path::new(path).strip_prefix(&run_config.project_root).unwrap_or(Path::new(path)))
+        .map(|path| path.to_path_buf())
+        .collect();
+
+    let parser = build_codeowners_parser(run_config)?;
+    Ok(parser
+        .teams_from_files_paths(&relative_file_paths)
+        .map_err(|e| Error::Io(e.to_string()))?)
+}
+
+fn build_codeowners_parser(run_config: &RunConfig) -> Result<crate::ownership::codeowners_file_parser::Parser, Error> {
     let config = config_from_path(&run_config.config_path)?;
+    Ok(crate::ownership::codeowners_file_parser::Parser {
+        codeowners_file_path: run_config.codeowners_file_path.clone(),
+        project_root: run_config.project_root.clone(),
+        team_file_globs: config.team_file_glob.clone(),
+    })
+}
+
+pub fn team_for_file_from_codeowners(run_config: &RunConfig, file_path: &str) -> Result<Option<Team>, Error> {
     let relative_file_path = Path::new(file_path)
         .strip_prefix(&run_config.project_root)
         .unwrap_or(Path::new(file_path));
 
-    let parser = crate::ownership::parser::Parser {
-        project_root: run_config.project_root.clone(),
-        codeowners_file_path: run_config.codeowners_file_path.clone(),
-        team_file_globs: config.team_file_glob.clone(),
-    };
+    let parser = build_codeowners_parser(run_config)?;
     Ok(parser
         .team_from_file_path(Path::new(relative_file_path))
         .map_err(|e| Error::Io(e.to_string()))?)