Merge pull request #369 from github/fixes/360-validate-clone-directory-exists

Check if the clone target directory already exists.

Author: shana

src/GitHub.App/Extensions/ValidationExtensions.cs

@@ -124,7 +124,6 @@
     <Compile Include="Caches\ImageCache.cs" />
     <Compile Include="Extensions\AkavacheExtensions.cs" />
     <Compile Include="Extensions\EnvironmentExtensions.cs" />
-    <Compile Include="Extensions\ValidationExtensions.cs" />
     <Compile Include="Factories\UIFactory.cs" />
     <Compile Include="GlobalSuppressions.cs" />
     <Compile Include="Infrastructure\LoggingConfiguration.cs" />

src/GitHub.App/GitHub.App.csproj

@@ -377,7 +377,11 @@ public RepositoryCloneViewModelDesigner()
                 x => x
             );
 
-            BaseRepositoryPathValidator = this.CreateBaseRepositoryPathValidator();
+            BaseRepositoryPathValidator = ReactivePropertyValidator.ForObservable(this.WhenAny(x => x.BaseRepositoryPath, x => x.Value))
+                .IfNullOrEmpty("Please enter a repository path")
+                .IfTrue(x => x.Length > 200, "Path too long")
+                .IfContainsInvalidPathChars("Path contains invalid characters")
+                .IfPathNotRooted("Please enter a valid path");
         }
 
         public IReactiveCommand<Unit> CloneCommand

src/GitHub.App/SampleData/SampleViewModels.cs

@@ -3,8 +3,11 @@
 using System.ComponentModel.Composition;
 using System.Diagnostics;
 using System.Globalization;
+using System.IO;
+using System.Linq;
 using System.Reactive;
 using System.Reactive.Linq;
+using System.Text.RegularExpressions;
 using System.Windows.Input;
 using GitHub.App;
 using GitHub.Exports;
@@ -80,7 +83,17 @@ public RepositoryCloneViewModel(
                 signalReset: filterResetSignal
             );
 
-            BaseRepositoryPathValidator = this.CreateBaseRepositoryPathValidator();
+            var baseRepositoryPath = this.WhenAny(
+                x => x.BaseRepositoryPath,
+                x => x.SelectedRepository,
+                (x, y) => x.Value);
+
+            BaseRepositoryPathValidator = ReactivePropertyValidator.ForObservable(baseRepositoryPath)
+                .IfNullOrEmpty("Please enter a repository path")
+                .IfTrue(x => x.Length > 200, "Path too long")
+                .IfContainsInvalidPathChars("Path contains invalid characters")
+                .IfPathNotRooted("Please enter a valid path")
+                .IfTrue(IsAlreadyRepoAtPath, Resources.RepositoryNameValidatorAlreadyExists);
 
             var canCloneObservable = this.WhenAny(
                 x => x.SelectedRepository,
@@ -140,6 +153,23 @@ IObservable<Unit> OnCloneRepository(object state)
             });
         }
 
+        bool IsAlreadyRepoAtPath(string path)
+        {
+            bool isAlreadyRepoAtPath = false;
+
+            if (SelectedRepository != null)
+            {
+                var validationResult = BaseRepositoryPathValidator.ValidationResult;
+                if (validationResult != null && validationResult.IsValid)
+                {
+                    string potentialPath = Path.Combine(path, SelectedRepository.Name);
+                    isAlreadyRepoAtPath = operatingSystem.Directory.Exists(potentialPath);
+                }
+            }
+
+            return isAlreadyRepoAtPath;
+        }
+
         /// <summary>
         /// Path to clone repositories into
         /// </summary>

src/GitHub.App/ViewModels/RepositoryCloneViewModel.cs

@@ -78,7 +78,11 @@ public RepositoryCreationViewModel(
 
             browseForDirectoryCommand.Subscribe(_ => ShowBrowseForDirectoryDialog());
 
-            BaseRepositoryPathValidator = this.CreateBaseRepositoryPathValidator();
+            BaseRepositoryPathValidator = ReactivePropertyValidator.ForObservable(this.WhenAny(x => x.BaseRepositoryPath, x => x.Value))
+                .IfNullOrEmpty("Please enter a repository path")
+                .IfTrue(x => x.Length > 200, "Path too long")
+                .IfContainsInvalidPathChars("Path contains invalid characters")
+                .IfPathNotRooted("Please enter a valid path");
 
             var nonNullRepositoryName = this.WhenAny(
                 x => x.RepositoryName,
@@ -251,23 +255,11 @@ bool IsAlreadyRepoAtPath(string potentialRepositoryName)
                 if (Path.GetInvalidPathChars().Any(chr => BaseRepositoryPath.Contains(chr)))
                     return false;
                 string potentialPath = Path.Combine(BaseRepositoryPath, safeRepositoryName);
-                isAlreadyRepoAtPath = IsGitRepo(potentialPath);
+                isAlreadyRepoAtPath = operatingSystem.Directory.Exists(potentialPath);
             }
             return isAlreadyRepoAtPath;
         }
 
-        bool IsGitRepo(string path)
-        {
-            try
-            {
-                return operatingSystem.File.Exists(Path.Combine(path, ".git", "HEAD"));
-            }
-            catch (PathTooLongException)
-            {
-                return false;
-            }
-        }
-
         IObservable<Unit> OnCreateRepository(object state)
         {
             var newRepository = GatherRepositoryInfo();

src/GitHub.App/ViewModels/RepositoryCreationViewModel.cs

@@ -5,6 +5,7 @@
 using System.Threading.Tasks;
 using GitHub.Models;
 using GitHub.Services;
+using GitHub.Validation;
 using GitHub.ViewModels;
 using NSubstitute;
 using Rothko;
@@ -196,6 +197,33 @@ public void IsTrueIfLoadingReposFails()
         }
     }
 
+    public class TheBaseRepositoryPathValidator
+    {
+        [Fact]
+        public void IsInvalidWhenDestinationRepositoryExists()
+        {
+            var repo = Substitute.For<IRepositoryModel>();
+            repo.Name.Returns("bar");
+            var repositoryHost = Substitute.For<IRepositoryHost>();
+            repositoryHost.ModelService.GetRepositories().Returns(Observable.Return(new[] { repo }));
+            var cloneService = Substitute.For<IRepositoryCloneService>();
+            var os = Substitute.For<IOperatingSystem>();
+            var directories = Substitute.For<IDirectoryFacade>();
+            os.Directory.Returns(directories);
+            directories.Exists(@"c:\foo\bar").Returns(true);
+            var vm = new RepositoryCloneViewModel(
+                repositoryHost,
+                cloneService,
+                os,
+                Substitute.For<INotificationService>());
+
+            vm.BaseRepositoryPath = @"c:\foo";
+            vm.SelectedRepository = repo;
+
+            Assert.Equal(ValidationStatus.Invalid, vm.BaseRepositoryPathValidator.ValidationResult.Status);
+        }
+    }
+
     public class TheCloneCommand : TestBaseClass
     {
         [Fact]

src/UnitTests/GitHub.App/ViewModels/RepositoryCloneViewModelTests.cs

@@ -283,7 +283,7 @@ public void IsFalseWhenRepositoryAlreadyExists(bool exists, bool expected)
         {
             var provider = Substitutes.ServiceProvider;
             var operatingSystem = provider.GetOperatingSystem();
-            operatingSystem.File.Exists(@"c:\fake\foo\.git\HEAD").Returns(exists);
+            operatingSystem.Directory.Exists(@"c:\fake\foo").Returns(exists);
             var vm = GetMeAViewModel(provider);
             vm.BaseRepositoryPath = @"c:\fake\";