fixup! feat: add flags to specify repository from CLI

Author: GuillaumeLagrange

src/run/run_environment/local/provider.rs

@@ -15,6 +15,8 @@ use crate::run::{
     },
 };
 
+static FAKE_COMMIT_REF: &str = "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa";
+
 #[derive(Debug)]
 pub struct LocalProvider {
     repository_provider: RepositoryProvider,
@@ -31,56 +33,10 @@ impl TryFrom<&Config> for LocalProvider {
     type Error = Error;
     fn try_from(config: &Config) -> Result<Self> {
         let current_dir = std::env::current_dir()?;
-        match find_repository_root(&current_dir) {
-            Some(mut path) => {
-                // Add a trailing slash to the path
-                path.push("");
-                let repository_root_path = path.to_string_lossy().to_string();
-                let git_repository =
-                    Repository::open(repository_root_path.clone()).context(format!(
-                        "Failed to open repository at path: {}",
-                        repository_root_path
-                    ))?;
-
-                let remote = git_repository.find_remote("origin")?;
-
-                let (repository_provider, owner, repository) = if let Some(repo_override) =
-                    config.repository_override.clone()
-                {
-                    (
-                        repo_override.repository_provider,
-                        repo_override.owner,
-                        repo_override.repository,
-                    )
-                } else {
-                    extract_provider_owner_and_repository_from_remote_url(remote.url().unwrap())?
-                };
-
-                let head = git_repository.head().context("Failed to get HEAD")?;
-                let ref_ = head
-                    .peel_to_commit()
-                    .context("Failed to get HEAD commit")?
-                    .id()
-                    .to_string();
-                let head_ref = if head.is_branch() {
-                    let branch = head.shorthand().context("Failed to get HEAD branch name")?;
-                    Some(branch.to_string())
-                } else {
-                    None
-                };
-
-                Ok(Self {
-                    repository_provider,
-                    ref_,
-                    head_ref,
-                    base_ref: None,
-                    owner,
-                    repository,
-                    event: RunEvent::Local,
-                    repository_root_path,
-                })
-            }
-            None => {
+
+        let repository_root_path = {
+            let Some(mut path) = find_repository_root(&current_dir) else {
+                // We are not in a git repository, use the repository_override with very minimal information
                 let RepositoryOverride {
                     owner,
                     repository,
@@ -90,19 +46,64 @@ impl TryFrom<&Config> for LocalProvider {
                     please make sure you are running the command from inside a git repository or provide repository with --repository flag",
                 )?;
 
-                Ok(Self {
+                return Ok(Self {
                     repository_provider,
-                    // TODO: Handle this ?
-                    ref_: "fake_ref".to_string(),
+                    ref_: FAKE_COMMIT_REF.to_string(),
                     head_ref: None,
                     base_ref: None,
                     owner,
                     repository,
                     repository_root_path: current_dir.to_string_lossy().to_string(),
                     event: RunEvent::Local,
-                })
-            }
-        }
+                });
+            };
+
+            // Add a trailing slash to the path
+            path.push("");
+            path.to_string_lossy().to_string()
+        };
+
+        let git_repository = Repository::open(repository_root_path.clone()).context(format!(
+            "Failed to open repository at path: {}",
+            repository_root_path
+        ))?;
+
+        let remote = git_repository.find_remote("origin")?;
+
+        let (repository_provider, owner, repository) =
+            if let Some(repo_override) = config.repository_override.clone() {
+                (
+                    repo_override.repository_provider,
+                    repo_override.owner,
+                    repo_override.repository,
+                )
+            } else {
+                extract_provider_owner_and_repository_from_remote_url(remote.url().unwrap())?
+            };
+
+        let head = git_repository.head().context("Failed to get HEAD")?;
+        let ref_ = head
+            .peel_to_commit()
+            .context("Failed to get HEAD commit")?
+            .id()
+            .to_string();
+        let head_ref = if head.is_branch() {
+            let branch = head.shorthand().context("Failed to get HEAD branch name")?;
+            Some(branch.to_string())
+        } else {
+            None
+        };
+
+        Ok(Self {
+            repository_provider,
+            ref_,
+            head_ref,
+            base_ref: None,
+            owner,
+            repository,
+            event: RunEvent::Local,
+            repository_root_path,
+        })
     }
 }
 
@@ -216,6 +217,11 @@ mod tests {
         }
     }
 
+    #[test]
+    fn fake_commit_hash_ref() {
+        assert_eq!(FAKE_COMMIT_REF.len(), 40);
+    }
+
     // TODO: uncomment later when we have a way to mock git repository
     // #[test]
     // fn test_provider_metadata() {