Merge branch 'features/check-suite-annotations-inline' into donokuda/update-check-icon

# Conflicts:
#	src/GitHub.VisualStudio.UI/Views/GitHubPane/PullRequestCheckView.xaml

Author: StanleyGoldman

src/GitHub.App/SampleData/CommentThreadViewModelDesigner.cs

@@ -1,5 +1,7 @@
-using System.Diagnostics.CodeAnalysis;
+using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
 using System.Threading.Tasks;
+using GitHub.Models;
 using GitHub.ViewModels;
 using ReactiveUI;
 
@@ -8,9 +10,73 @@ namespace GitHub.SampleData
     [SuppressMessage("Microsoft.Performance", "CA1812:AvoidUninstantiatedInternalClasses")]
     public class CommentThreadViewModelDesigner : ViewModelBase, ICommentThreadViewModel
     {
+        public CommentThreadViewModelDesigner()
+        {
+
+            var checkRunAnnotationModel1 = new CheckRunAnnotationModel
+            {
+                AnnotationLevel = CheckAnnotationLevel.Failure,
+                Path = "SomeFile.cs",
+                EndLine = 12,
+                StartLine = 12,
+                Message = "CS12345: ; expected",
+                Title = "CS12345"
+            };
+
+            var checkRunAnnotationModel2 = new CheckRunAnnotationModel
+            {
+                AnnotationLevel = CheckAnnotationLevel.Warning,
+                Path = "SomeFile.cs",
+                EndLine = 12,
+                StartLine = 12,
+                Message = "CS12345: ; expected",
+                Title = "CS12345"
+            };
+
+            var checkRunAnnotationModel3 = new CheckRunAnnotationModel
+            {
+                AnnotationLevel = CheckAnnotationLevel.Notice,
+                Path = "SomeFile.cs",
+                EndLine = 12,
+                StartLine = 12,
+                Message = "CS12345: ; expected",
+                Title = "CS12345"
+            };
+
+            var checkRunModel =
+                new CheckRunModel
+                {
+                    Annotations = new List<CheckRunAnnotationModel> { checkRunAnnotationModel1, checkRunAnnotationModel2 },
+                    Name = "MSBuildLog Analyzer"
+                };
+
+            var checkSuiteModel = new CheckSuiteModel()
+            {
+                HeadSha = "ed6198c37b13638e902716252b0a17d54bd59e4a",
+                CheckRuns = new List<CheckRunModel> { checkRunModel },
+                ApplicationName = "My Awesome Check Suite"
+            };
+
+            Annotations = new[]
+            {
+                new InlineAnnotationViewModel(new InlineAnnotationModel(checkSuiteModel, checkRunModel, checkRunAnnotationModel1)),
+                new InlineAnnotationViewModel(new InlineAnnotationModel(checkSuiteModel, checkRunModel, checkRunAnnotationModel2)),
+                new InlineAnnotationViewModel(new InlineAnnotationModel(checkSuiteModel, checkRunModel, checkRunAnnotationModel3)),
+            };
+
+            Comments = new ReactiveList<ICommentViewModel>(){new CommentViewModelDesigner()
+            {
+                Author = new ActorViewModel{ Login = "shana"},
+                Body = "You can use a `CompositeDisposable` type here, it's designed to handle disposables in an optimal way (you can just call `Dispose()` on it and it will handle disposing everything it holds)."
+            }};
+
+        }
+
         public IReadOnlyReactiveList<ICommentViewModel> Comments { get; }
             = new ReactiveList<ICommentViewModel>();
 
+        public IReadOnlyList<IInlineAnnotationViewModel> Annotations { get; }
+
         public IActorViewModel CurrentUser { get; set; }
             = new ActorViewModel { Login = "shana" };
 

src/GitHub.App/SampleData/InlineAnnotationViewModelDesigner.cs

@@ -0,0 +1,40 @@
+using System.Collections.Generic;
+using GitHub.Models;
+using GitHub.ViewModels;
+
+namespace GitHub.SampleData
+{
+    public class InlineAnnotationViewModelDesigner : IInlineAnnotationViewModel
+    {
+        public InlineAnnotationViewModelDesigner()
+        {
+            var checkRunAnnotationModel = new CheckRunAnnotationModel
+            {
+                AnnotationLevel = CheckAnnotationLevel.Failure,
+                Path = "SomeFile.cs",
+                EndLine = 12,
+                StartLine = 12,
+                Message = "Some Error Message",
+                Title = "CS12345"
+            };
+
+            var checkRunModel =
+                new CheckRunModel
+                {
+                    Annotations = new List<CheckRunAnnotationModel> {checkRunAnnotationModel},
+                    Name = "Fake Check Run"
+                };
+
+            var checkSuiteModel = new CheckSuiteModel()
+            {
+                ApplicationName = "Fake Check Suite",
+                HeadSha = "ed6198c37b13638e902716252b0a17d54bd59e4a",
+                CheckRuns = new List<CheckRunModel> { checkRunModel}
+            };
+
+            Model= new InlineAnnotationModel(checkSuiteModel, checkRunModel, checkRunAnnotationModel);
+        }
+
+        public InlineAnnotationModel Model { get; }
+    }
+}

src/GitHub.App/SampleData/PullRequestAnnotationItemViewModelDesigner.cs

@@ -12,7 +12,7 @@ public sealed class PullRequestAnnotationItemViewModelDesigner : IPullRequestAnn
         public CheckRunAnnotationModel Annotation { get; set; }
         public bool IsExpanded { get; set; }
         public string LineDescription => $"{Annotation.StartLine}:{Annotation.EndLine}";
-        public bool IsFileInPullRequest { get; }
+        public bool IsFileInPullRequest { get; set;  }
         public ReactiveCommand<Unit, Unit> OpenAnnotation { get; }
     }
 }

src/GitHub.App/SampleData/PullRequestAnnotationsViewModelDesigner.cs

@@ -38,6 +38,7 @@ public sealed class PullRequestAnnotationsViewModelDesigner : PanePageViewModelB
                                 Title = "CS 12345"
                             },
                             IsExpanded = true,
+                            IsFileInPullRequest = true
                         },
                         new PullRequestAnnotationItemViewModelDesigner
                         {
@@ -51,6 +52,7 @@ public sealed class PullRequestAnnotationsViewModelDesigner : PanePageViewModelB
                                 Title = "CS 12345"
                             },
                             IsExpanded = true,
+                            IsFileInPullRequest = true
                         },
                     }
                 },

src/GitHub.App/SampleData/PullRequestCheckViewModelDesigner.cs

@@ -1,6 +1,5 @@
 using System;
 using System.Reactive;
-using System.Windows.Media.Imaging;
 using GitHub.Models;
 using GitHub.ViewModels;
 using GitHub.ViewModels.GitHubPane;

src/GitHub.App/SampleData/PullRequestFilesViewModelDesigner.cs

@@ -36,6 +36,9 @@ public PullRequestFilesViewModelDesigner()
         public ReactiveCommand<IPullRequestFileNode, Unit> DiffFileWithWorkingDirectory { get; }
         public ReactiveCommand<IPullRequestFileNode, Unit> OpenFileInWorkingDirectory { get; }
         public ReactiveCommand<IPullRequestFileNode, Unit> OpenFirstComment { get; }
+        public ReactiveCommand<IPullRequestFileNode, Unit> OpenFirstAnnotationNotice { get; }
+        public ReactiveCommand<IPullRequestFileNode, Unit> OpenFirstAnnotationWarning { get; }
+        public ReactiveCommand<IPullRequestFileNode, Unit> OpenFirstAnnotationFailure { get; }
 
         public Task InitializeAsync(
             IPullRequestSession session,

src/GitHub.App/Services/PullRequestEditorService.cs

@@ -285,7 +285,7 @@ await pullRequestService.ExtractToTempFile(
         }
 
         /// <inheritdoc/>
-        public async Task<IDifferenceViewer> OpenDiff(
+        public Task<IDifferenceViewer> OpenDiff(
             IPullRequestSession session,
             string relativePath,
             IInlineCommentThreadModel thread)
@@ -294,11 +294,17 @@ public async Task<IDifferenceViewer> OpenDiff(
             Guard.ArgumentNotEmptyString(relativePath, nameof(relativePath));
             Guard.ArgumentNotNull(thread, nameof(thread));
 
-            var diffViewer = await OpenDiff(session, relativePath, thread.CommitSha, scrollToFirstDraftOrDiff: false);
+            return OpenDiff(session, relativePath, thread.CommitSha, thread.LineNumber - 1);
+        }
+
+        /// <inheritdoc/>
+        public async Task<IDifferenceViewer> OpenDiff(IPullRequestSession session, string relativePath, string headSha, int fromLine)
+        {
+            var diffViewer = await OpenDiff(session, relativePath, headSha, scrollToFirstDraftOrDiff: false);
 
-            var param = (object)new InlineCommentNavigationParams
+            var param = (object) new InlineCommentNavigationParams
             {
-                FromLine = thread.LineNumber - 1,
+                FromLine = fromLine,
             };
 
             // HACK: We need to wait here for the inline comment tags to initialize so we can find the next inline comment.

src/GitHub.App/ViewModels/CommentThreadViewModel.cs

@@ -57,6 +57,9 @@ public CommentThreadViewModel(
         /// <inheritdoc/>
         public IActorViewModel CurrentUser { get; private set; }
 
+        /// <inheritdoc/>
+        public IReadOnlyList<IInlineAnnotationViewModel> Annotations { get; private set; }
+
         /// <inheritdoc/>
         IReadOnlyReactiveList<ICommentViewModel> ICommentThreadViewModel.Comments => comments;
 
@@ -92,8 +95,10 @@ protected IDisposable AddPlaceholder(ICommentViewModel placeholder)
         /// Intializes a new instance of the <see cref="CommentThreadViewModel"/> class.
         /// </summary>
         /// <param name="currentUser">The current user.</param>
-        protected Task InitializeAsync(ActorModel currentUser)
+        /// <param name="annotations"></param>
+        protected Task InitializeAsync(ActorModel currentUser, IReadOnlyList<IInlineAnnotationViewModel> annotations)
         {
+            Annotations = annotations;
             Guard.ArgumentNotNull(currentUser, nameof(currentUser));
             CurrentUser = new ActorViewModel(currentUser);
             return Task.CompletedTask;

src/GitHub.App/ViewModels/GitHubPane/PullRequestAnnotationItemViewModel.cs

@@ -1,5 +1,7 @@
 using System.Reactive;
+using System.Reactive.Linq;
 using GitHub.Models;
+using GitHub.Services;
 using ReactiveUI;
 
 namespace GitHub.ViewModels.GitHubPane
@@ -14,12 +16,22 @@ public class PullRequestAnnotationItemViewModel : ViewModelBase, IPullRequestAnn
         /// </summary>
         /// <param name="annotation">The check run annotation model.</param>
         /// <param name="isFileInPullRequest">A flag that denotes if the annotation is part of the pull request's changes.</param>
-        public PullRequestAnnotationItemViewModel(CheckRunAnnotationModel annotation, bool isFileInPullRequest)
+        /// <param name="checkSuite">The check suite model.</param>
+        /// <param name="session">The pull request session.</param>
+        /// <param name="editorService">The pull request editor service.</param>
+        public PullRequestAnnotationItemViewModel(
+            CheckRunAnnotationModel annotation,
+            bool isFileInPullRequest,
+            CheckSuiteModel checkSuite,
+            IPullRequestSession session,
+            IPullRequestEditorService editorService)
         {
             Annotation = annotation;
             IsFileInPullRequest = isFileInPullRequest;
 
-            OpenAnnotation = ReactiveCommand.Create(() => { });
+            OpenAnnotation = ReactiveCommand.CreateFromTask<Unit>(
+                async _ => await editorService.OpenDiff(session, annotation.Path, checkSuite.HeadSha, annotation.EndLine - 1),
+                Observable.Return(IsFileInPullRequest));
         }
 
         /// <inheritdoc />

src/GitHub.App/ViewModels/GitHubPane/PullRequestAnnotationsViewModel.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.ComponentModel.Composition;
 using System.Linq;
@@ -15,7 +15,8 @@ namespace GitHub.ViewModels.GitHubPane
     [PartCreationPolicy(CreationPolicy.NonShared)]
     public class PullRequestAnnotationsViewModel : PanePageViewModelBase, IPullRequestAnnotationsViewModel
     {
-        private readonly IPullRequestSessionManager sessionManager;
+        readonly IPullRequestSessionManager sessionManager;
+        readonly IPullRequestEditorService pullRequestEditorService;
 
         IPullRequestSession session;
         string title;
@@ -27,10 +28,12 @@ public class PullRequestAnnotationsViewModel : PanePageViewModelBase, IPullReque
         /// Initializes a new instance of the <see cref="PullRequestAnnotationsViewModel"/> class.
         /// </summary>
         /// <param name="sessionManager">The pull request session manager.</param>
+        /// <param name="pullRequestEditorService">The pull request editor service.</param>
         [ImportingConstructor]
-        public PullRequestAnnotationsViewModel(IPullRequestSessionManager sessionManager)
+        public PullRequestAnnotationsViewModel(IPullRequestSessionManager sessionManager, IPullRequestEditorService pullRequestEditorService)
         {
             this.sessionManager = sessionManager;
+            this.pullRequestEditorService = pullRequestEditorService;
             NavigateToPullRequest = ReactiveCommand.Create(() => {
                     NavigateTo(FormattableString.Invariant(
                         $"{LocalRepository.Owner}/{LocalRepository.Name}/pull/{PullRequestNumber}"));
@@ -132,7 +135,7 @@ void Load(PullRequestDetailModel pullRequest)
                     .ToDictionary(
                         path => path,
                         path => annotationsLookup[path]
-                            .Select(annotation => new PullRequestAnnotationItemViewModel(annotation, changedFiles.Contains(path)))
+                            .Select(annotation => new PullRequestAnnotationItemViewModel(annotation, changedFiles.Contains(path), checkSuiteRun.checkSuite, session, pullRequestEditorService))
                             .Cast<IPullRequestAnnotationItemViewModel>()
                             .ToArray()
                         );