From d98961188d4bbb8e8338908fdad76e9a5abb0d81 Mon Sep 17 00:00:00 2001
From: pltaylor <pltaylor3@gmail.com>
Date: Sat, 2 Nov 2013 11:57:13 -0400
Subject: [PATCH] Fix some of the tests

---
 Octokit.Tests/Clients/IssueCommentsClientTests.cs | 7 +++----
 Octokit/Clients/IssueCommentsClient.cs            | 2 +-
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/Octokit.Tests/Clients/IssueCommentsClientTests.cs b/Octokit.Tests/Clients/IssueCommentsClientTests.cs
index afe19bdd..450102da 100644
--- a/Octokit.Tests/Clients/IssueCommentsClientTests.cs
+++ b/Octokit.Tests/Clients/IssueCommentsClientTests.cs
@@ -44,7 +44,7 @@ public class IssueCommentsClientTests
 
             client.GetForRepository("fake", "repo");
 
-            connection.Received().GetAll<Issue>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/comments"),
+            connection.Received().GetAll<IssueComment>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/comments"),
                 Arg.Any<Dictionary<string, string>>());
         }
 
@@ -58,7 +58,6 @@ public class IssueCommentsClientTests
             await AssertEx.Throws<ArgumentException>(async () => await client.GetForRepository("", "name"));
             await AssertEx.Throws<ArgumentNullException>(async () => await client.GetForRepository("owner", null));
             await AssertEx.Throws<ArgumentException>(async () => await client.GetForRepository("owner", ""));
-            await AssertEx.Throws<ArgumentNullException>(async () => await client.GetForRepository("owner", "name"));
         }
     }
 
@@ -83,9 +82,9 @@ public class IssueCommentsClientTests
             var client = new IssueCommentsClient(connection);
 
             await AssertEx.Throws<ArgumentNullException>(async () => await client.GetForIssue(null, "name", 1));
-            await AssertEx.Throws<ArgumentNullException>(async () => await client.GetForIssue("", "name", 1));
+            await AssertEx.Throws<ArgumentException>(async () => await client.GetForIssue("", "name", 1));
             await AssertEx.Throws<ArgumentNullException>(async () => await client.GetForIssue("owner", null, 1));
-            await AssertEx.Throws<ArgumentNullException>(async () => await client.GetForIssue("owner", "", 1));
+            await AssertEx.Throws<ArgumentException>(async () => await client.GetForIssue("owner", "", 1));
         }
     }
 
diff --git a/Octokit/Clients/IssueCommentsClient.cs b/Octokit/Clients/IssueCommentsClient.cs
index db97ac44..228210d1 100644
--- a/Octokit/Clients/IssueCommentsClient.cs
+++ b/Octokit/Clients/IssueCommentsClient.cs
@@ -80,7 +80,7 @@ namespace Octokit
             Ensure.ArgumentNotNullOrEmptyString(name, "name");
             Ensure.ArgumentNotNull(newComment, "newComment");
 
-            return ApiConnection.Post<IssueComment>(ApiUrls.IssueComment(owner, name, number), newComment);
+            return ApiConnection.Post<IssueComment>(ApiUrls.IssueComments(owner, name, number), newComment);
         }
 
         /// <summary>