#4125 - fix version and help run options to not depend on .git directory

Author: arturcic

src/GitVersion.App/GitVersionExecutor.cs

@@ -9,26 +9,29 @@ namespace GitVersion;
 internal class GitVersionExecutor(
     ILog log,
     IConsole console,
+    IVersionWriter versionWriter,
+    IHelpWriter helpWriter,
     IConfigurationFileLocator configurationFileLocator,
     IConfigurationProvider configurationProvider,
     IConfigurationSerializer configurationSerializer,
     IGitVersionCalculateTool gitVersionCalculateTool,
     IGitVersionOutputTool gitVersionOutputTool,
-    IVersionWriter versionWriter,
-    IHelpWriter helpWriter,
+    IGitRepository gitRepository,
     IGitRepositoryInfo repositoryInfo)
     : IGitVersionExecutor
 {
     private readonly ILog log = log.NotNull();
     private readonly IConsole console = console.NotNull();
+    private readonly IVersionWriter versionWriter = versionWriter.NotNull();
+    private readonly IHelpWriter helpWriter = helpWriter.NotNull();
+
     private readonly IConfigurationFileLocator configurationFileLocator = configurationFileLocator.NotNull();
     private readonly IConfigurationProvider configurationProvider = configurationProvider.NotNull();
     private readonly IConfigurationSerializer configurationSerializer = configurationSerializer.NotNull();
 
     private readonly IGitVersionCalculateTool gitVersionCalculateTool = gitVersionCalculateTool.NotNull();
     private readonly IGitVersionOutputTool gitVersionOutputTool = gitVersionOutputTool.NotNull();
-    private readonly IVersionWriter versionWriter = versionWriter.NotNull();
-    private readonly IHelpWriter helpWriter = helpWriter.NotNull();
+    private readonly IGitRepository gitRepository = gitRepository.NotNull();
     private readonly IGitRepositoryInfo repositoryInfo = repositoryInfo.NotNull();
 
     public int Execute(GitVersionOptions gitVersionOptions)
@@ -49,6 +52,7 @@ public int Execute(GitVersionOptions gitVersionOptions)
 
     private int RunGitVersionTool(GitVersionOptions gitVersionOptions)
     {
+        this.gitRepository.DiscoverRepository(gitVersionOptions.WorkingDirectory);
         var mutexName = this.repositoryInfo.DotGitDirectory?.Replace(Path.DirectorySeparatorChar.ToString(), "") ?? string.Empty;
         using var mutex = new Mutex(true, $@"Global\gitversion{mutexName}", out var acquired);
 

src/GitVersion.Core.Tests/Core/DynamicRepositoryTests.cs

@@ -74,6 +74,8 @@ public void FindsVersionInDynamicRepo(string name, string url, string targetBran
 
         var sp = ConfigureServices(services => services.AddSingleton(options));
 
+        sp.DiscoverRepository();
+
         var gitPreparer = sp.GetRequiredService<IGitPreparer>();
         gitPreparer.Prepare();
 

src/GitVersion.Core.Tests/Core/GitVersionExecutorTests.cs

@@ -34,6 +34,8 @@ public void CacheKeySameAfterReNormalizing()
 
         this.sp = GetServiceProvider(gitVersionOptions, environment: environment);
 
+        sp.DiscoverRepository();
+
         var preparer = this.sp.GetRequiredService<IGitPreparer>();
 
         preparer.Prepare();
@@ -78,6 +80,8 @@ public void CacheKeyForWorktree()
 
             this.sp = GetServiceProvider(gitVersionOptions);
 
+            sp.DiscoverRepository();
+
             var preparer = this.sp.GetRequiredService<IGitPreparer>();
             preparer.Prepare();
             var cacheKeyFactory = this.sp.GetRequiredService<IGitVersionCacheKeyFactory>();
@@ -516,6 +520,9 @@ public void CalculateVersionVariables_TwoBranchHasSameCommitHeadDetachedAndNotTa
         environment.SetEnvironmentVariable(AzurePipelines.EnvironmentVariableName, "true");
 
         this.sp = GetServiceProvider(gitVersionOptions, environment: environment);
+
+        sp.DiscoverRepository();
+
         var sut = sp.GetRequiredService<IGitVersionCalculateTool>();
 
         // Execute & Verify
@@ -543,6 +550,9 @@ public void CalculateVersionVariables_TwoBranchHasSameCommitHeadDetachedAndTagge
         environment.SetEnvironmentVariable(AzurePipelines.EnvironmentVariableName, "true");
 
         this.sp = GetServiceProvider(gitVersionOptions, environment: environment);
+
+        sp.DiscoverRepository();
+
         var sut = sp.GetRequiredService<IGitVersionCalculateTool>();
 
         // Execute
@@ -568,6 +578,9 @@ public void CalculateVersionVariables_ShallowFetch_ThrowException()
         environment.SetEnvironmentVariable(AzurePipelines.EnvironmentVariableName, "true");
 
         this.sp = GetServiceProvider(gitVersionOptions, environment: environment);
+
+        sp.DiscoverRepository();
+
         var sut = sp.GetRequiredService<IGitVersionCalculateTool>();
 
         // Execute & Verify
@@ -583,6 +596,8 @@ private IGitVersionCalculateTool GetGitVersionCalculator(GitVersionOptions gitVe
         this.log = this.sp.GetRequiredService<ILog>();
         this.gitVersionCacheProvider = (GitVersionCacheProvider)this.sp.GetRequiredService<IGitVersionCacheProvider>();
 
+        sp.DiscoverRepository();
+
         return this.sp.GetRequiredService<IGitVersionCalculateTool>();
     }
 

src/GitVersion.Core.Tests/Extensions/GitToolsTestingExtensions.cs

@@ -46,6 +46,13 @@ public static IBranch CreateMockBranch(string name, params ICommit[] commits)
         return branch;
     }
 
+    public static void DiscoverRepository(this IServiceProvider sp)
+    {
+        var gitRepository = sp.GetRequiredService<IGitRepository>();
+        var gitRepositoryInfo = sp.GetRequiredService<IGitRepositoryInfo>();
+        gitRepository.DiscoverRepository(gitRepositoryInfo.GitRootPath);
+    }
+
     public static IBranch FindBranch(this IGitRepository repository, string branchName)
         => repository.Branches.FirstOrDefault(branch => branch.Name.WithoutOrigin == branchName)
             ?? throw new GitVersionException($"Branch {branchName} not found");
@@ -84,6 +91,9 @@ public static GitVersionVariables GetVersion(this RepositoryFixtureBase fixture,
                 services.AddSingleton(options);
                 services.AddSingleton(configurationProviderMock);
             });
+
+            sp.DiscoverRepository();
+
             var variableProvider = sp.GetRequiredService<IVariableProvider>();
             var nextVersionCalculator = sp.GetRequiredService<INextVersionCalculator>();
             var contextOptions = sp.GetRequiredService<Lazy<GitVersionContext>>();

src/GitVersion.Core.Tests/IntegrationTests/RemoteRepositoryScenarios.cs

@@ -63,6 +63,8 @@ public void GivenARemoteGitRepositoryWithCommitsAndBranchesThenClonedLocalShould
             services.AddSingleton<IEnvironment>(environment);
         });
 
+        sp.DiscoverRepository();
+
         var gitPreparer = sp.GetRequiredService<IGitPreparer>();
 
         gitPreparer.Prepare();

src/GitVersion.LibGit2Sharp/Git/GitRepository.extended.cs

@@ -8,12 +8,14 @@ namespace GitVersion.Git;
 
 internal partial class GitRepository(ILog log) : IMutatingGitRepository
 {
+    private readonly ILog log = log.NotNull();
+
     public void Clone(string? sourceUrl, string? workdirPath, AuthenticationInfo auth)
     {
         try
         {
             var path = Repository.Clone(sourceUrl, workdirPath, GetCloneOptions(auth));
-            log.Info($"Returned path after repository clone: {path}");
+            this.log.Info($"Returned path after repository clone: {path}");
         }
         catch (LibGit2Sharp.LockedFileException ex)
         {
@@ -48,7 +50,7 @@ public void Fetch(string remote, IEnumerable<string> refSpecs, AuthenticationInf
             Commands.Fetch((Repository)RepositoryInstance, remote, refSpecs, GetFetchOptions(auth), logMessage));
     public void CreateBranchForPullRequestBranch(AuthenticationInfo auth) => RepositoryExtensions.RunSafe(() =>
     {
-        log.Info("Fetching remote refs to see if there is a pull request ref");
+        this.log.Info("Fetching remote refs to see if there is a pull request ref");
 
         // FIX ME: What to do when Tip is null?
         if (Head.Tip == null)
@@ -59,21 +61,21 @@ public void CreateBranchForPullRequestBranch(AuthenticationInfo auth) => Reposit
         var reference = GetPullRequestReference(auth, remote, headTipSha);
         var canonicalName = reference.CanonicalName;
         var referenceName = ReferenceName.Parse(reference.CanonicalName);
-        log.Info($"Found remote tip '{canonicalName}' pointing at the commit '{headTipSha}'.");
+        this.log.Info($"Found remote tip '{canonicalName}' pointing at the commit '{headTipSha}'.");
 
         if (referenceName.IsTag)
         {
-            log.Info($"Checking out tag '{canonicalName}'");
+            this.log.Info($"Checking out tag '{canonicalName}'");
             Checkout(reference.Target.Sha);
         }
         else if (referenceName.IsPullRequest)
         {
             var fakeBranchName = canonicalName.Replace("refs/pull/", "refs/heads/pull/").Replace("refs/pull-requests/", "refs/heads/pull-requests/");
 
-            log.Info($"Creating fake local branch '{fakeBranchName}'.");
+            this.log.Info($"Creating fake local branch '{fakeBranchName}'.");
             Refs.Add(fakeBranchName, headTipSha);
 
-            log.Info($"Checking local branch '{fakeBranchName}' out.");
+            this.log.Info($"Checking local branch '{fakeBranchName}' out.");
             Checkout(fakeBranchName);
         }
         else
@@ -91,7 +93,7 @@ private DirectReference GetPullRequestReference(AuthenticationInfo auth, LibGit2
                 : network.ListReferences(remote))
             .Select(r => r.ResolveToDirectReference()).ToList();
 
-        log.Info($"Remote Refs:{PathHelper.NewLine}" + string.Join(PathHelper.NewLine, remoteTips.Select(r => r.CanonicalName)));
+        this.log.Info($"Remote Refs:{PathHelper.NewLine}" + string.Join(PathHelper.NewLine, remoteTips.Select(r => r.CanonicalName)));
         var refs = remoteTips.Where(r => r.TargetIdentifier == headTipSha).ToList();
 
         switch (refs.Count)

src/GitVersion.LibGit2Sharp/GitVersionLibGit2SharpModule.cs

@@ -1,5 +1,4 @@
 using GitVersion.Git;
-using GitVersion.Logging;
 using Microsoft.Extensions.DependencyInjection;
 
 namespace GitVersion;
@@ -8,15 +7,7 @@ public class GitVersionLibGit2SharpModule : IGitVersionModule
 {
     public void RegisterTypes(IServiceCollection services)
     {
-        services.AddSingleton<IGitRepository>(sp =>
-        {
-            var repositoryInfo = sp.GetRequiredService<IGitRepositoryInfo>();
-            var log = sp.GetRequiredService<ILog>();
-            IGitRepository gitRepository = new GitRepository(log);
-            gitRepository.DiscoverRepository(repositoryInfo.GitRootPath);
-            return gitRepository;
-        });
-
+        services.AddSingleton<IGitRepository, GitRepository>();
         services.AddSingleton<IMutatingGitRepository>(sp => (IMutatingGitRepository)sp.GetRequiredService<IGitRepository>());
         services.AddSingleton<IGitRepositoryInfo, GitRepositoryInfo>();
     }