Functionality to perform the edit

Author: StanleyGoldman

src/GitHub.App/Api/ApiClient.cs

@@ -110,6 +110,16 @@ public IObservable<PullRequestReviewComment> CreatePullRequestReviewComment(
             return gitHubClient.PullRequest.ReviewComment.CreateReply(owner, name, number, comment);
         }
 
+        public IObservable<PullRequestReviewComment> EditPullRequestReviewComment(
+            string owner,
+            string name,
+            int number,
+            string body)
+        {
+            var pullRequestReviewCommentEdit = new PullRequestReviewCommentEdit(body);
+            return gitHubClient.PullRequest.ReviewComment.Edit(owner, name, number, pullRequestReviewCommentEdit);
+        }
+
         public IObservable<Unit> DeletePullRequestReviewComment(
             string owner,
             string name,

src/GitHub.Exports.Reactive/Api/IApiClient.cs

@@ -90,14 +90,33 @@ IObservable<PullRequestReviewComment> CreatePullRequestReviewComment(
         /// <returns></returns>
         IObservable<PullRequestReviewComment> CreatePullRequestReviewComment(string owner, string name, int number, string body, int inReplyTo);
 
-        IObservable<Branch> GetBranches(string owner, string repo);
-        IObservable<Repository> GetRepositories();
-        IObservable<Repository> GetRepository(string owner, string repo);
-        IObservable<RepositoryContent> GetFileContents(string owner, string name, string reference, string path);
-
+        /// <summary>
+        /// Delete a PR review comment.
+        /// </summary>
+        /// <param name="owner">The repository owner.</param>
+        /// <param name="name">The repository name.</param>
+        /// <param name="number">The pull request comment number.</param>
         IObservable<Unit> DeletePullRequestReviewComment(
             string owner,
             string name,
             int number);
+
+        /// <summary>
+        /// Edits a PR review comment.
+        /// </summary>
+        /// <param name="owner">The repository owner.</param>
+        /// <param name="name">The repository name.</param>
+        /// <param name="number">The pull request comment number.</param>
+        /// <param name="body">The replacement comment body.</param>
+        IObservable<PullRequestReviewComment> EditPullRequestReviewComment(
+            string owner,
+            string name,
+            int number,
+            string body);
+
+        IObservable<Branch> GetBranches(string owner, string repo);
+        IObservable<Repository> GetRepositories();
+        IObservable<Repository> GetRepository(string owner, string repo);
+        IObservable<RepositoryContent> GetFileContents(string owner, string name, string reference, string path);
     }
 }

src/GitHub.Exports.Reactive/Services/IPullRequestSession.cs

@@ -111,7 +111,7 @@ Task<IPullRequestReviewCommentModel> PostReviewComment(
         /// <param name="body">The comment body.</param>
         /// <param name="inReplyTo">The REST ID of the comment to reply to.</param>
         /// <param name="inReplyToNodeId">The GraphQL ID of the comment to reply to.</param>
-        /// <returns></returns>
+        /// <returns>A comment model.</returns>
         Task<IPullRequestReviewCommentModel> PostReviewComment(
             string body,
             int inReplyTo,
@@ -152,5 +152,13 @@ Task<IPullRequestReviewCommentModel> PostReviewComment(
         /// <param name="number">The number of the pull request comment to delete</param>
         /// <returns>A task which completes when the session has completed updating.</returns>
         Task DeleteComment(int number);
+
+        /// <summary>
+        /// Edit a PR review comment reply.
+        /// </summary>
+        /// <param name="number">The number of the pull request comment</param>
+        /// <param name="body">The replacement comment body.</param>
+        /// <returns>A comment model.</returns>
+        Task<IPullRequestReviewCommentModel> EditComment(int number, string body);
     }
 }

src/GitHub.Exports/Models/UsageModel.cs

@@ -65,6 +65,7 @@ public class MeasuresModel
             public int NumberOfPRReviewDiffViewInlineCommentOpen { get; set; }
             public int NumberOfPRReviewDiffViewInlineCommentPost { get; set; }
             public int NumberOfPRReviewDiffViewInlineCommentDelete { get; set; }
+            public int NumberOfPRReviewDiffViewInlineCommentEdit { get; set; }
             public int NumberOfPRReviewDiffViewInlineCommentStartReview { get; set; }
             public int NumberOfPRReviewPosts { get; set; }
             public int NumberOfShowCurrentPullRequest { get; set; }

src/GitHub.InlineReviews/Services/IPullRequestSessionService.cs

@@ -313,5 +313,20 @@ Task DeleteComment(
             string remoteRepositoryOwner,
             IAccount user,
             int number);
+
+        /// <summary>
+        /// Edit a PR review comment.
+        /// </summary>
+        /// <param name="localRepository">The local repository.</param>
+        /// <param name="remoteRepositoryOwner">The owner of the repository fork to delete from.</param>
+        /// <param name="user">The user deleting the comment.</param>
+        /// <param name="number">The pull request comment number.</param>
+        /// <param name="body">The replacement comment body.</param>
+        /// <returns>A model representing the edited comment.</returns>
+        Task<PullRequestReviewCommentModel> EditComment(ILocalRepositoryModel localRepository,
+            string remoteRepositoryOwner,
+            IAccount user,
+            int number,
+            string body);
     }
 }

src/GitHub.InlineReviews/Services/PullRequestSession.cs

@@ -175,6 +175,20 @@ await service.DeleteComment(
                 number);
         }
 
+        /// <inheritdoc/>
+        public async Task<IPullRequestReviewCommentModel> EditComment(
+            int number, string body)
+        {
+            var model = await service.EditComment(
+                LocalRepository,
+                RepositoryOwner,
+                User,
+                number,
+                body);
+
+            return model;
+        }
+
         /// <inheritdoc/>
         public async Task<IPullRequestReviewCommentModel> PostReviewComment(
             string body,

src/GitHub.InlineReviews/Services/PullRequestSessionService.cs

@@ -600,6 +600,7 @@ public async Task<IPullRequestReviewCommentModel> PostStandaloneReviewCommentRep
             };
         }
 
+        /// <inheritdoc/>
         public async Task DeleteComment(
             ILocalRepositoryModel localRepository,
             string remoteRepositoryOwner,
@@ -617,6 +618,39 @@ await apiClient.DeletePullRequestReviewComment(
             await usageTracker.IncrementCounter(x => x.NumberOfPRReviewDiffViewInlineCommentDelete);
         }
 
+        /// <inheritdoc/>
+        public async Task<PullRequestReviewCommentModel> EditComment(ILocalRepositoryModel localRepository,
+            string remoteRepositoryOwner,
+            IAccount user,
+            int number,
+            string body)
+        {
+            var address = HostAddress.Create(localRepository.CloneUrl.Host);
+            var apiClient = await apiClientFactory.Create(address);
+
+            var result = await apiClient.EditPullRequestReviewComment(
+                remoteRepositoryOwner,
+                localRepository.Name,
+                number,
+                body);
+
+            await usageTracker.IncrementCounter(x => x.NumberOfPRReviewDiffViewInlineCommentEdit);
+
+            return new PullRequestReviewCommentModel
+            {
+                Body = result.Body,
+                CommitId = result.CommitId,
+                DiffHunk = result.DiffHunk,
+                Id = result.Id,
+                OriginalCommitId = result.OriginalCommitId,
+                OriginalPosition = result.OriginalPosition,
+                Path = result.Path,
+                Position = result.Position,
+                CreatedAt = result.CreatedAt,
+                User = user,
+            };
+        }
+
         int GetUpdatedLineNumber(IInlineCommentThreadModel thread, IEnumerable<DiffChunk> diff)
         {
             var line = DiffUtilities.Match(diff, thread.DiffMatch);

src/GitHub.InlineReviews/ViewModels/CommentViewModel.cs

@@ -224,7 +224,7 @@ async Task DoCommitEdit(object unused)
                 ErrorMessage = null;
                 IsSubmitting = true;
 
-                var model = await Thread.EditComment.ExecuteAsyncTask(Body);
+                var model = await Thread.EditComment.ExecuteAsyncTask(new Tuple<int, string>(Id, Body));
                 Id = model.Id;
                 NodeId = model.NodeId;
                 EditState = CommentEditState.None;

src/GitHub.InlineReviews/ViewModels/InlineCommentPeekViewModel.cs

@@ -175,8 +175,7 @@ async Task UpdateThread()
             }
             else
             {
-                var newThread = new NewInlineCommentThreadViewModel(session, file, lineNumber, leftBuffer);
-                Thread = newThread;
+                Thread = new NewInlineCommentThreadViewModel(session, file, lineNumber, leftBuffer);
             }
 
             if (!string.IsNullOrWhiteSpace(placeholderBody))
@@ -189,6 +188,14 @@ async Task UpdateThread()
                     placeholder.Body = placeholderBody;
                 }
             }
+
+            threadSubscription = Thread
+                .WhenAnyValue(model => model.EditComment, model => model.DeleteComment)
+                .Skip(1)
+                .Subscribe(_ =>
+                {
+
+                });
         }
 
         async Task SessionChanged(IPullRequestSession pullRequestSession)

src/GitHub.InlineReviews/ViewModels/InlineCommentThreadViewModel.cs

@@ -77,15 +77,12 @@ async Task<ICommentModel> DoPostComment(object parameter)
             return await Session.PostReviewComment(body, replyId, nodeId);
         }
 
-        Task<ICommentModel> DoEditComment(object parameter)
+        async Task<ICommentModel> DoEditComment(object parameter)
         {
             Guard.ArgumentNotNull(parameter, nameof(parameter));
 
-            var body = (string)parameter;
-            var replyId = Comments[0].Id;
-            var nodeId = Comments[0].NodeId;
-
-            throw new NotImplementedException();
+            var item = (Tuple<int, string>)parameter;
+            return await Session.EditComment(item.Item1, item.Item2);
         }
 
         async Task<object> DoDeleteComment(object parameter)