Merge branch 'refactor/clone' into refactor/clone-include-owner-in-path

Author: jcansdale

.gitignore

@@ -232,6 +232,9 @@ $RECYCLE.BIN/
 *.msm
 *.msp
 
+#OpenCover output
+coverage.xml
+
 AkavacheSqliteLinkerOverride.cs
 NuGetBuild
 WiX.Toolset.DummyFile.txt

appveyor.yml

@@ -29,3 +29,5 @@ on_success:
     if ($full_build) {
       script\Sign-Package -AppVeyor
     }
+on_finish:
+- ps: scripts\Run-CodeCoverage.ps1 -AppVeyor -Configuration:Release

scripts/Run-CodeCoverage.ps1

@@ -0,0 +1,68 @@
+<#
+.SYNOPSIS
+    Runs NUnit
+#>
+
+[CmdletBinding()]
+Param(
+    [string]
+    $Configuration
+    ,
+    [switch]
+    $AppVeyor = $false
+)
+
+$scriptsDirectory = $PSScriptRoot
+$rootDirectory = Split-Path ($scriptsDirectory)
+. $scriptsDirectory\modules.ps1 | out-null
+
+$nunitDirectory = Join-Path $rootDirectory packages\NUnit.ConsoleRunner.3.7.0\tools
+$nunitConsoleRunner = Join-Path $nunitDirectory nunit3-console.exe
+
+$testAssemblies = @(
+    "test\GitHub.Api.UnitTests\bin\$Configuration\GitHub.Api.UnitTests.dll",
+    "test\GitHub.App.UnitTests\bin\$Configuration\GitHub.App.UnitTests.dll",
+    "test\GitHub.Exports.Reactive.UnitTests\bin\$Configuration\GitHub.Exports.Reactive.UnitTests.dll",
+    "test\GitHub.Exports.UnitTests\bin\$Configuration\GitHub.Exports.UnitTests.dll",
+    "test\GitHub.Extensions.UnitTests\bin\$Configuration\GitHub.Extensions.UnitTests.dll",
+    "test\GitHub.InlineReviews.UnitTests\bin\$Configuration\GitHub.InlineReviews.UnitTests.dll",
+    "test\GitHub.Primitives.UnitTests\bin\$Configuration\GitHub.Primitives.UnitTests.dll",
+    "test\GitHub.TeamFoundation.UnitTests\bin\$Configuration\GitHub.TeamFoundation.UnitTests.dll",
+    "test\GitHub.UI.UnitTests\bin\$Configuration\GitHub.UI.UnitTests.dll",
+    "test\GitHub.VisualStudio.UnitTests\bin\$Configuration\GitHub.VisualStudio.UnitTests.dll",
+    "test\MetricsTests\MetricsTests\bin\$Configuration\MetricsTests.dll",
+    "test\TrackingCollectionTests\bin\$Configuration\TrackingCollectionTests.dll"
+)
+
+$opencoverTargetArgs = ($testAssemblies -join " ") + " --where \`"cat!=Timings and cat!=CodeCoverageFlake\`" --inprocess --noresult"
+
+$opencoverDirectory = Join-Path $rootDirectory packages\OpenCover.4.6.519\tools
+$opencover = Join-Path $opencoverDirectory OpenCover.Console.exe
+$opencoverArgs = @(
+    "-target:`"$nunitConsoleRunner`"",
+    "-targetargs:`"$opencoverTargetArgs`"",
+    "-filter:`"+[GitHub*]* -[GitHub*UnitTests]*`"",
+    "-register:user -output:$rootDirectory\coverage.xml"
+) -join " "
+
+$codecovDirectory = Join-Path $rootDirectory packages\Codecov.1.0.5\tools
+$codecov = Join-Path $codecovDirectory codecov.exe
+$codecovArgs = "-f $rootDirectory\coverage.xml"
+
+& {
+    Trap {
+        Write-Output "$Project tests failed"
+        exit 0
+    }
+
+    Run-Process 600 $opencover $opencoverArgs
+
+    if($AppVeyor) {
+        Run-Process 60 $codecov $codecovArgs
+    }
+
+    if (!$?) {
+        Write-Output "$Project tests failed"
+        exit 0
+    }
+}

scripts/modules.ps1

@@ -73,7 +73,7 @@ New-Module -ScriptBlock {
             $output += Get-Content $outputPath
             $exitCode = $process.ExitCode
         } else {
-            $output += "Tests timed out. Backtrace:"
+            $output += "Process timed out. Backtrace:"
             $output += Get-DotNetStack $process.Id
             $exitCode = 9999
         }

src/GitHub.App/SampleData/Dialog/Clone/RepositoryCloneViewModelDesigner.cs

@@ -23,6 +23,7 @@ public RepositoryCloneViewModelDesigner()
         public IRepositorySelectViewModel GitHubTab { get; }
         public IRepositorySelectViewModel EnterpriseTab { get; }
         public IRepositoryUrlViewModel UrlTab { get; }
+        public ReactiveCommand<object> Browse { get; }
         public ReactiveCommand<CloneDialogResult> Clone { get; }
 
         public Task InitializeAsync(IConnection connection)

src/GitHub.App/ViewModels/Dialog/Clone/RepositoryCloneViewModel.cs

@@ -10,6 +10,7 @@
 using GitHub.Models;
 using GitHub.Services;
 using ReactiveUI;
+using Rothko;
 using Serilog;
 
 namespace GitHub.ViewModels.Dialog.Clone
@@ -19,6 +20,7 @@ namespace GitHub.ViewModels.Dialog.Clone
     public class RepositoryCloneViewModel : ViewModelBase, IRepositoryCloneViewModel
     {
         static readonly ILogger log = LogManager.ForContext<RepositoryCloneViewModel>();
+        readonly IOperatingSystem os;
         readonly IConnectionManager connectionManager;
         readonly IRepositoryCloneService service;
         readonly IReadOnlyList<IRepositoryCloneTabViewModel> tabs;
@@ -29,12 +31,14 @@ public class RepositoryCloneViewModel : ViewModelBase, IRepositoryCloneViewModel
 
         [ImportingConstructor]
         public RepositoryCloneViewModel(
+            IOperatingSystem os,
             IConnectionManager connectionManager,
             IRepositoryCloneService service,
             IRepositorySelectViewModel gitHubTab,
             IRepositorySelectViewModel enterpriseTab,
             IRepositoryUrlViewModel urlTab)
         {
+            this.os = os;
             this.connectionManager = connectionManager;
             this.service = service;
 
@@ -61,6 +65,7 @@ public RepositoryCloneViewModel(
                 (repo, error) => (repo, error))
                 .Select(x => x.repo != null && x.error == null);
 
+            Browse = ReactiveCommand.Create().OnExecuteCompleted(_ => BrowseForDirectory());
             Clone = ReactiveCommand.CreateAsyncObservable(
                 canClone,
                 _ => repository.Select(x => new CloneDialogResult(Path, x)));
@@ -88,6 +93,8 @@ public int SelectedTabIndex
 
         public IObservable<object> Done => Clone;
 
+        public ReactiveCommand<object> Browse { get; }
+
         public ReactiveCommand<CloneDialogResult> Clone { get; }
 
         public async Task InitializeAsync(IConnection connection)
@@ -114,6 +121,24 @@ public async Task InitializeAsync(IConnection connection)
             this.WhenAnyValue(x => x.SelectedTabIndex).Subscribe(x => tabs[x].Activate().Forget());
         }
 
+        void BrowseForDirectory()
+        {
+            var result = os.Dialog.BrowseForDirectory(Path, Resources.BrowseForDirectory);
+
+            if (result != BrowseDirectoryResult.Failed)
+            {
+                var path = result.DirectoryPath;
+                var selected = tabs[SelectedTabIndex].Repository;
+
+                if (selected != null)
+                {
+                    path = System.IO.Path.Combine(path, selected.Name);
+                }
+
+                Path = path;
+            }
+        }
+
         void UpdatePath(IRepositoryModel repository)
         {
             if (repository != null)

src/GitHub.Exports.Reactive/ViewModels/Dialog/Clone/IRepositoryCloneViewModel.cs

@@ -42,6 +42,11 @@ public interface IRepositoryCloneViewModel : IDialogContentViewModel, IConnectio
         /// </remarks>
         int SelectedTabIndex { get; }
 
+        /// <summary>
+        /// Gets the command executed when the user clicks "Browse".
+        /// </summary>
+        ReactiveCommand<object> Browse { get; }
+
         /// <summary>
         /// Gets the command executed when the user clicks "Clone".
         /// </summary>

src/GitHub.VisualStudio/Views/Dialog/Clone/RepositoryCloneView.xaml

@@ -36,6 +36,7 @@
                    Margin="16">
             <Label DockPanel.Dock="Left">Local Path</Label>
             <Button DockPanel.Dock="Right"
+                    Command="{Binding Browse}"
                     Style="{DynamicResource GitHubBlueLinkButton}"
                     VerticalContentAlignment="Center">
                 Browse

test/GitHub.App.UnitTests/ViewModels/Dialog/Clone/RepositoryCloneViewModelTests.cs

@@ -8,6 +8,7 @@
 using GitHub.ViewModels.Dialog.Clone;
 using NSubstitute;
 using NUnit.Framework;
+using Rothko;
 
 namespace GitHub.App.UnitTests.ViewModels.Dialog.Clone
 {
@@ -269,19 +270,22 @@ static IRepositoryCloneService CreateRepositoryCloneService()
         }
 
         static RepositoryCloneViewModel CreateTarget(
+            IOperatingSystem os = null,
             IConnectionManager connectionManager = null,
             IRepositoryCloneService service = null,
             IRepositorySelectViewModel gitHubTab = null,
             IRepositorySelectViewModel enterpriseTab = null,
             IRepositoryUrlViewModel urlTab = null)
         {
+            os = os ?? Substitute.For<IOperatingSystem>();
             connectionManager = connectionManager ?? CreateConnectionManager("https://github.com");
             service = service ?? CreateRepositoryCloneService();
             gitHubTab = gitHubTab ?? CreateSelectViewModel();
             enterpriseTab = enterpriseTab ?? CreateSelectViewModel();
             urlTab = urlTab ?? Substitute.For<IRepositoryUrlViewModel>();
 
             return new RepositoryCloneViewModel(
+                os,
                 connectionManager,
                 service,
                 gitHubTab,

test/GitHub.InlineReviews.UnitTests/Services/PullRequestSessionManagerTests.cs

@@ -442,7 +442,7 @@ Line 2
                 }
             }
 
-            [Test]
+            [Test, NUnit.Framework.Category("CodeCoverageFlake")]
             public async Task UpdatesInlineCommentThreadsFromEditorContent()
             {
                 var baseContents = @"Line 1
@@ -497,7 +497,7 @@ Line 2
                 }
             }
 
-            [Test]
+            [Test, NUnit.Framework.Category("CodeCoverageFlake")]
             public async Task UpdatesReviewCommentWithNewBody()
             {
                 var baseContents = @"Line 1