Fixing and adding unit tests for UserEmails...

Client and ObservableUserEmailsClient
This commit is contained in:
Peter MacNaughton
2014-02-09 12:04:36 -07:00
parent 2421201dda
commit 2a9c7c7eff
2 changed files with 18 additions and 12 deletions
+14 -4
View File
@@ -39,17 +39,27 @@ namespace Octokit.Tests.Clients
}
[Fact]
public async Task EnsuresNonNullArgument()
public void EnsuresNonNullArgument()
{
var client = new UserEmailsClient(Substitute.For<IApiConnection>());
await AssertEx.Throws<ArgumentNullException>(async () => await client.Add(null));
Assert.Throws<ArgumentNullException>(() => client.Add(null));
}
[Fact]
public async Task EnsuresNoNullEmails()
public void EnsuresNoNullEmails()
{
var client = new UserEmailsClient(Substitute.For<IApiConnection>());
await AssertEx.Throws<ArgumentException>(async () => await client.Add("octokit@github.com", null));
Assert.Throws<ArgumentException>(() => client.Add("octokit@github.com", null));
}
}
public class TheCtor
{
[Fact]
public void EnsuresArguments()
{
Assert.Throws<ArgumentNullException>(
() => new UserEmailsClient(null));
}
}
}
@@ -54,10 +54,8 @@ namespace Octokit.Tests
{
var client = CreateFixtureWithNonReactiveClient();
Assert.Throws<ArgumentNullException>(
() => client.Add(null));
Assert.Throws<ArgumentException>(
() => client.Add("octo@github.com", null));
Assert.Throws<ArgumentNullException>(() => client.Add(null));
Assert.Throws<ArgumentException>(() => client.Add("octo@github.com", null));
}
[Fact]
@@ -65,10 +63,8 @@ namespace Octokit.Tests
{
var client = CreateFixtureWithNonReactiveClient();
Assert.Throws<ArgumentException>(
() => client.Add(""));
Assert.Throws<ArgumentException>(
() => client.Add("octo@github.com", ""));
Assert.Throws<ArgumentException>(() => client.Add(""));
Assert.Throws<ArgumentException>(() => client.Add("octo@github.com", ""));
}
}