Final code cleanup

Author: StanleyGoldman

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

@@ -5,31 +5,27 @@
 
 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
     {
         bool isExpanded;
 
+        /// <summary>
+        /// Initializes the <see cref="PullRequestAnnotationItemViewModel"/>.
+        /// </summary>
+        /// <param name="annotation">The check run annotation model.</param>
         public PullRequestAnnotationItemViewModel(CheckRunAnnotationModel annotation)
         {
             this.Annotation = annotation;
         }
 
-        /// <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}";
 
-        /// <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

@@ -2,10 +2,24 @@
 
 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; }
     }
 }

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>
@@ -39,6 +43,9 @@ public class CheckRunModel
         /// </summary>
         public string Summary { get; set; }
 
+        /// <summary>
+        /// 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()