Merge branch 'features/check-suite-annotations' into features/check-suite-annotations-inline

# Conflicts:
#	src/GitHub.App/ViewModels/GitHubPane/PullRequestAnnotationItemViewModel.cs
#	src/GitHub.Exports.Reactive/ViewModels/GitHubPane/IPullRequestAnnotationItemViewModel.cs
#	src/GitHub.Exports/Models/CheckRunModel.cs

Author: StanleyGoldman

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

@@ -9,9 +9,7 @@
 
 namespace GitHub.App.ViewModels.GitHubPane
 {
-    /// <summary>
-    /// The viewmodel for a single annotation item in a list
-    /// </summary>
+    /// <inheritdoc cref="IPullRequestAnnotationItemViewModel"/>
     public class PullRequestAnnotationItemViewModel : ViewModelBase, IPullRequestAnnotationItemViewModel
     {
         readonly CheckSuiteModel checkSuite;
@@ -21,6 +19,10 @@ public class PullRequestAnnotationItemViewModel : ViewModelBase, IPullRequestAnn
 
         bool isExpanded;
 
+        /// <summary>
+        /// Initializes the <see cref="PullRequestAnnotationItemViewModel"/>.
+        /// </summary>
+        /// <param name="annotation">The check run annotation model.</param>
         public PullRequestAnnotationItemViewModel(CheckSuiteModel checkSuite, 
             CheckRunModel checkRun,
             CheckRunAnnotationModel annotation,
@@ -43,21 +45,15 @@ public PullRequestAnnotationItemViewModel(CheckSuiteModel checkSuite,
 
         public bool IsFileInPullRequest { get; }
 
-        /// <summary>
-        /// Gets the annotation model.
-        /// </summary>
+        /// <inheritdoc />
         public CheckRunAnnotationModel Annotation { get; }
 
-        /// <summary>
-        /// Gets a formatted descriptor of the line(s) the annotation is about.
-        /// </summary>
+        /// <inheritdoc />
         public string LineDescription => $"{Annotation.StartLine}:{Annotation.EndLine}";
 
         public ReactiveCommand<Unit> OpenAnnotation { get; }
 
-        /// <summary>
-        /// Gets or sets a flag to control the expanded state.
-        /// </summary>
+        /// <inheritdoc />
         public bool IsExpanded
         {
             get { return isExpanded; }

src/GitHub.Exports.Reactive/ViewModels/GitHubPane/IPullRequestAnnotationItemViewModel.cs

@@ -4,11 +4,26 @@
 
 namespace GitHub.ViewModels.GitHubPane
 {
+    /// <summary>
+    /// The viewmodel for a single annotation item in a list
+    /// </summary>
     public interface IPullRequestAnnotationItemViewModel
     {
+        /// <summary>
+        /// Gets the annotation model.
+        /// </summary>
         CheckRunAnnotationModel Annotation { get; }
-        bool IsExpanded { get; set; }
+
+        /// <summary>
+        /// Gets a formatted descriptor of the line(s) the annotation is about.
+        /// </summary>
         string LineDescription { get; }
+
+        /// <summary>
+        /// Gets or sets a flag to control the expanded state.
+        /// </summary>
+        bool IsExpanded { get; set; }
+
         bool IsFileInPullRequest { get; }
         ReactiveCommand<Unit> OpenAnnotation { get; }
     }

src/GitHub.Exports/Models/CheckRunModel.cs

@@ -23,10 +23,14 @@ public class CheckRunModel
         /// </summary>
         public DateTimeOffset? CompletedAt { get; set; }
 
-        /// <summary>The check run's annotations.</summary>
+        /// <summary>
+        /// The check run's annotations.
+        /// </summary>
         public List<CheckRunAnnotationModel> Annotations { get; set; }
 
-        /// <summary>The name of the check for this check run.</summary>
+        /// <summary>
+        /// The name of the check for this check run.
+        /// </summary>
         public string Name { get; set; }
 
         /// <summary>
@@ -40,7 +44,7 @@ public class CheckRunModel
         public string Summary { get; set; }
 
         /// <summary>
-        /// The check run Id
+        /// The database id of a Check Run.
         /// </summary>
         public int DatabaseId { get; set; }
     }

test/GitHub.App.UnitTests/ViewModels/GitHubPane/PullRequestFilesViewModelTests.cs

@@ -57,7 +57,7 @@ public async Task ShouldCreateChangesTreeAsync()
             Assert.That(readme.FileName, Is.EqualTo("readme.md"));
         }
 
-        [Test, Ignore("I broke it")]
+        [Test]
         public async Task FileCommentCountShouldTrackSessionInlineCommentsAsync()
         {
             var outdatedThread = CreateThread(-1);
@@ -88,7 +88,7 @@ public async Task FileCommentCountShouldTrackSessionInlineCommentsAsync()
             RaisePropertyChanged(file, nameof(file.InlineCommentThreads));
             Assert.That(((IPullRequestFileNode)target.Items[0]).CommentCount, Is.EqualTo(2));
 
-            file.Received(1).PropertyChanged += Arg.Any<PropertyChangedEventHandler>();
+            file.Received(2).PropertyChanged += Arg.Any<PropertyChangedEventHandler>();
         }
 
         static PullRequestFilesViewModel CreateTarget()