From 090b078a1afd081c140cce47775d97e7ba2f4dfe Mon Sep 17 00:00:00 2001
From: Gabriel Weyer <gabrielweyer@gmail.com>
Date: Thu, 21 Nov 2013 16:48:49 +1100
Subject: [PATCH] Added basic integration testing for review comments client

---
 .../Octokit.Tests.Integration.csproj          |   1 +
 .../PullRequestReviewCommentsClientTests.cs   | 125 ++++++++++++++++++
 2 files changed, 126 insertions(+)
 create mode 100644 Octokit.Tests.Integration/PullRequestReviewCommentsClientTests.cs

diff --git a/Octokit.Tests.Integration/Octokit.Tests.Integration.csproj b/Octokit.Tests.Integration/Octokit.Tests.Integration.csproj
index 205f8a4e..4eb79e2f 100644
--- a/Octokit.Tests.Integration/Octokit.Tests.Integration.csproj
+++ b/Octokit.Tests.Integration/Octokit.Tests.Integration.csproj
@@ -65,6 +65,7 @@
     <Compile Include="IntegrationTestAttribute.cs" />
     <Compile Include="IssuesClientTests.cs" />
     <Compile Include="MiscellaneousClientTests.cs" />
+    <Compile Include="PullRequestReviewCommentsClientTests.cs" />
     <Compile Include="Reactive\ObservableIssuesClientTests.cs" />
     <Compile Include="Reactive\ObservableMilestonesClientTests.cs" />
     <Compile Include="Reactive\ObservableRepositoriesClientTests.cs" />
diff --git a/Octokit.Tests.Integration/PullRequestReviewCommentsClientTests.cs b/Octokit.Tests.Integration/PullRequestReviewCommentsClientTests.cs
new file mode 100644
index 00000000..9651d29a
--- /dev/null
+++ b/Octokit.Tests.Integration/PullRequestReviewCommentsClientTests.cs
@@ -0,0 +1,125 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Net.Http.Headers;
+using System.Text;
+using System.Threading.Tasks;
+using Octokit;
+using Octokit.Tests.Integration;
+using Xunit;
+
+public class PullRequestReviewCommentsClientTests : IDisposable
+{
+    IGitHubClient _gitHubClient;
+    Repository _repository;
+    IPullRequestReviewCommentsClient _client;
+
+    string _repoName;
+    string _ownerName;
+
+    int _pullRequestNumber;
+    string _pullRequestCommitId;
+    string _path;
+
+    public PullRequestReviewCommentsClientTests()
+    {
+        _gitHubClient = new GitHubClient(new ProductHeaderValue("OctokitTests"))
+        {
+            Credentials = Helper.Credentials
+        };
+
+        // Creating a repository
+
+        _repoName = Helper.MakeNameWithTimestamp("public-repo");
+        _repository = _gitHubClient.Repository.Create(new NewRepository { Name = _repoName }).Result;
+        _ownerName = _repository.Owner.Name;
+
+        // You can't for a repository that doesn't have any commit
+
+        // Creating a blob
+
+        // TODO Not Implemented: http://developer.github.com/v3/git/blobs/#create-a-blob
+
+        // Creating a tree
+
+        // TODO Not Implemented: http://developer.github.com/v3/git/trees/#create-a-tree
+        var treeSha = String.Empty;
+
+        // Creating a commit
+
+        var commit = new NewCommit("A nice commit message", treeSha, Enumerable.Empty<string>());
+
+        var createdCommit = _gitHubClient.GitDatabase.Commit.Create(_ownerName, _repoName, commit).Result;
+
+        // We can't fork our own repository so we need a second test user
+        // Using the second test user
+
+        _gitHubClient = new GitHubClient(new ProductHeaderValue("OctokitTests"))
+        {
+            Credentials = Helper.Credentials
+        };
+
+        // Creating a fork
+
+        // TODO Not Implemented: http://developer.github.com/v3/repos/forks/#create-a-fork
+        
+        // Creating a blob
+
+        // TODO Not Implemented: http://developer.github.com/v3/git/blobs/#create-a-blob
+
+        // Creating a tree
+
+        // TODO Not Implemented: http://developer.github.com/v3/git/trees/#create-a-tree
+        var pullRequestTreeSha = String.Empty;
+        _path = "";
+
+        // Creating a commit
+
+        var pullRequestcommit = new NewCommit("A pull request commit message", pullRequestTreeSha, Enumerable.Empty<string>());
+
+        var createdPullRequestCommit = _gitHubClient.GitDatabase.Commit.Create("second user name", _repoName, pullRequestcommit).Result;
+        _pullRequestCommitId = createdPullRequestCommit.Sha;
+        
+        // Creating a pull request
+
+        // TODO Not Implemented: http://developer.github.com/v3/pulls/#create-a-pull-request
+        _pullRequestNumber = 0;
+
+        // Using the first user again
+
+        _gitHubClient = new GitHubClient(new ProductHeaderValue("OctokitTests"))
+        {
+            Credentials = Helper.Credentials
+        };
+
+        _client = _gitHubClient.PullRequest.Comment;
+    }
+
+    [IntegrationTest(Skip = "Requires Blob, Tree, Fork and Pull Request Api implementation")]
+    public async Task CanCreateAndRetrieveReviewComment()
+    {
+        var pullRequestReviewComment = new PullRequestReviewCommentCreate
+        {
+            Body = "A review comment message",
+            CommitId = _pullRequestCommitId,
+            Path = _path,
+            Position = 1,
+        };
+
+        var createdComment = await _client.Create(_ownerName, _repoName, _pullRequestNumber, pullRequestReviewComment);
+
+        Assert.NotNull(createdComment);
+        Assert.Equal(pullRequestReviewComment.Body, createdComment.Body);
+
+        var commentFromGitHub = await _client.GetComment(_ownerName, _repoName, createdComment.Id);
+
+        Assert.NotNull(commentFromGitHub);
+        Assert.Equal(pullRequestReviewComment.Body, commentFromGitHub.Body);
+    }
+
+    public void Dispose()
+    {
+        Helper.DeleteRepo(_repository);
+        // TODO Ensure that it deletes the forks too
+    }
+}