Getting the right objects in place to run git config tasks during project setup

Author: StanleyGoldman

src/GitHub.Api/ApplicationManagerBase.cs

@@ -103,7 +103,7 @@ private async Task RunInternal()
         {
             await ThreadingHelper.SwitchToThreadAsync();
 
-            var gitSetup = new GitSetup(Environment, FileSystem, CancellationToken);
+            var gitSetup = new GitSetup(Environment, ProcessManager, FileSystem, CancellationToken);
             var expectedPath = gitSetup.GitInstallationPath;
 
             var setupDone = await gitSetup.SetupIfNeeded(

src/GitHub.Api/GitSetup.cs

@@ -6,24 +6,34 @@ namespace GitHub.Unity
 {
     class GitSetup
     {
-        private readonly IEnvironment environment;
         private readonly CancellationToken cancellationToken;
+        private readonly IEnvironment environment;
         private readonly GitInstaller gitInstaller;
-        public NPath GitInstallationPath { get; private set; }
-        public NPath GitExecutablePath { get; private set; }
+        private readonly IProcessManager processManager;
 
-        public GitSetup(IEnvironment environment, IFileSystem fileSystem, CancellationToken cancellationToken)
+        public GitSetup(IEnvironment environment, IProcessManager processManager, IFileSystem fileSystem,
+            CancellationToken cancellationToken)
         {
             this.environment = environment;
+            this.processManager = processManager;
             this.cancellationToken = cancellationToken;
             gitInstaller = new GitInstaller(environment, fileSystem, cancellationToken);
             GitInstallationPath = gitInstaller.PackageDestinationDirectory;
             GitExecutablePath = gitInstaller.GitDestinationPath;
         }
 
-        public Task<bool> SetupIfNeeded(IProgress<float> percentage = null, IProgress<long> timeRemaining = null)
+        public async Task<bool> SetupIfNeeded(IProgress<float> percentage = null, IProgress<long> timeRemaining = null)
         {
-            return gitInstaller.SetupIfNeeded(percentage, timeRemaining);
+            var setupIfNeeded = await gitInstaller.SetupIfNeeded(percentage, timeRemaining);
+
+            var gitConfigGetTask = new GitConfigGetTask(environment, processManager,
+                new TaskResultDispatcher<string>(s => {  }), "credential.helper",
+                GitConfigSource.Global);
+
+            return setupIfNeeded;
         }
+
+        public NPath GitInstallationPath { get; private set; }
+        public NPath GitExecutablePath { get; private set; }
     }
-}
+}

src/IntegrationTests/BaseGitEnvironmentTest.cs

@@ -14,16 +14,16 @@ protected void InitializeEnvironment(NPath repoPath)
                 UnityProjectPath = repoPath
             };
 
-            var gitSetup = new GitSetup(Environment, FileSystem, CancellationToken.None);
+            Platform = new Platform(Environment, FileSystem, new TestUIDispatcher());
+            GitEnvironment = Platform.GitEnvironment;
+            ProcessManager = new ProcessManager(Environment, GitEnvironment);
+
+            var gitSetup = new GitSetup(Environment, ProcessManager, FileSystem, CancellationToken.None);
             gitSetup.SetupIfNeeded().Wait();
 
             Environment.GitExecutablePath = gitSetup.GitExecutablePath;
 
             FileSystem.SetCurrentDirectory(repoPath);
-
-            Platform = new Platform(Environment, FileSystem, new TestUIDispatcher());
-            GitEnvironment = Platform.GitEnvironment;
-            ProcessManager = new ProcessManager(Environment, GitEnvironment);
             Platform.Initialize(ProcessManager);
 
             Environment.UnityProjectPath = repoPath;

src/IntegrationTests/Git/GitSetupTests.cs

@@ -14,7 +14,12 @@ public void InstallGit()
         {
             var environmentPath = NPath.CreateTempDirectory("integration-test-environment");
             var environment = new IntegrationTestEnvironment(SolutionDirectory, environmentPath);
-            var gitSetup = new GitSetup(environment, FileSystem, CancellationToken.None);
+
+            var platform = new Platform(environment, FileSystem, new TestUIDispatcher());
+            var gitEnvironment = platform.GitEnvironment;
+            var processManager = new ProcessManager(environment, gitEnvironment);
+
+            var gitSetup = new GitSetup(environment, processManager, FileSystem, CancellationToken.None);
             var expectedPath = gitSetup.GitInstallationPath;
 
             var setupDone = false;
@@ -51,10 +56,6 @@ public void InstallGit()
                 setupDone.Should().BeFalse ();
             }
 
-            var platform = new Platform(environment, FileSystem, new TestUIDispatcher());
-            var gitEnvironment = platform.GitEnvironment;
-            var processManager = new ProcessManager(environment, gitEnvironment);
-
             var gitBranches = processManager.GetGitBranches(TestRepoMasterDirtyUnsynchronized, environment.GitExecutablePath);
 
             gitBranches.Should().BeEquivalentTo(