diff --git a/Octokit.Tests/Reactive/ObservableFollowersTest.cs b/Octokit.Tests/Reactive/ObservableFollowersTest.cs index fcaa1b28..b7bd269d 100644 --- a/Octokit.Tests/Reactive/ObservableFollowersTest.cs +++ b/Octokit.Tests/Reactive/ObservableFollowersTest.cs @@ -6,6 +6,7 @@ using Octokit.Tests.Helpers; using System; using System.Collections.Generic; using System.Reactive.Linq; +using System.Reactive.Threading.Tasks; using System.Threading.Tasks; using Xunit; @@ -47,8 +48,8 @@ namespace Octokit.Tests.Reactive { var client = new ObservableFollowersClient(Substitute.For()); - await AssertEx.Throws(async () => await client.GetAll(null)); - await AssertEx.Throws(async () => await client.GetAll("")); + await Assert.ThrowsAsync(() => client.GetAll(null).ToTask()); + await Assert.ThrowsAsync(() => client.GetAll("").ToTask()); } } @@ -86,8 +87,8 @@ namespace Octokit.Tests.Reactive { var client = new ObservableFollowersClient(Substitute.For()); - await AssertEx.Throws(async () => await client.GetAllFollowing(null)); - await AssertEx.Throws(async () => await client.GetAllFollowing("")); + await Assert.ThrowsAsync(() => client.GetAllFollowing(null).ToTask()); + await Assert.ThrowsAsync(() => client.GetAllFollowing("").ToTask()); } } @@ -110,8 +111,8 @@ namespace Octokit.Tests.Reactive { var client = new ObservableFollowersClient(Substitute.For()); - await AssertEx.Throws(async () => await client.IsFollowingForCurrent(null)); - await AssertEx.Throws(async () => await client.IsFollowingForCurrent("")); + await Assert.ThrowsAsync(() => client.IsFollowingForCurrent(null).ToTask()); + await Assert.ThrowsAsync(() => client.IsFollowingForCurrent("").ToTask()); } } @@ -134,10 +135,10 @@ namespace Octokit.Tests.Reactive { var client = new ObservableFollowersClient(Substitute.For()); - await AssertEx.Throws(async () => await client.IsFollowing(null, "alfhenrik-test")); - await AssertEx.Throws(async () => await client.IsFollowing("", "alfhenrik-test")); - await AssertEx.Throws(async () => await client.IsFollowing("alfhenrik", null)); - await AssertEx.Throws(async () => await client.IsFollowing("alfhenrik", "")); + await Assert.ThrowsAsync(() => client.IsFollowing(null, "alfhenrik-test").ToTask()); + await Assert.ThrowsAsync(() => client.IsFollowing("", "alfhenrik-test").ToTask()); + await Assert.ThrowsAsync(() => client.IsFollowing("alfhenrik", null).ToTask()); + await Assert.ThrowsAsync(() => client.IsFollowing("alfhenrik", "").ToTask()); } } @@ -160,8 +161,8 @@ namespace Octokit.Tests.Reactive { var client = new ObservableFollowersClient(Substitute.For()); - await AssertEx.Throws(async () => await client.Follow(null)); - await AssertEx.Throws(async () => await client.Follow("")); + await Assert.ThrowsAsync(() => client.Follow(null).ToTask()); + await Assert.ThrowsAsync(() => client.Follow("").ToTask()); } } @@ -184,8 +185,8 @@ namespace Octokit.Tests.Reactive { var client = new ObservableFollowersClient(Substitute.For()); - await AssertEx.Throws(async () => await client.Unfollow(null)); - await AssertEx.Throws(async () => await client.Unfollow("")); + await Assert.ThrowsAsync(() => client.Unfollow(null).ToTask()); + await Assert.ThrowsAsync(() => client.Unfollow("").ToTask()); } } } diff --git a/Octokit.Tests/Reactive/ObservableGistsTests.cs b/Octokit.Tests/Reactive/ObservableGistsTests.cs index 3057f021..0f6b7bab 100644 --- a/Octokit.Tests/Reactive/ObservableGistsTests.cs +++ b/Octokit.Tests/Reactive/ObservableGistsTests.cs @@ -1,6 +1,7 @@ using System; using System.Collections.Generic; using System.Reactive.Linq; +using System.Reactive.Threading.Tasks; using System.Threading.Tasks; using NSubstitute; using Octokit; @@ -22,11 +23,11 @@ namespace Octokit.Tests.Reactive { var client = new ObservableGistsClient(Substitute.For()); - await AssertEx.Throws(async () => await client.GetAllCommits(null)); - await AssertEx.Throws(async () => await client.GetAllCommits("")); + await Assert.ThrowsAsync(() => client.GetAllCommits(null).ToTask()); + await Assert.ThrowsAsync(() => client.GetAllCommits("").ToTask()); - await AssertEx.Throws(async () => await client.GetAllForks(null)); - await AssertEx.Throws(async () => await client.GetAllForks("")); + await Assert.ThrowsAsync(() => client.GetAllForks(null).ToTask()); + await Assert.ThrowsAsync(() => client.GetAllForks("").ToTask()); } [Fact] diff --git a/Octokit.Tests/Reactive/ObservableIssueCommentsClientTests.cs b/Octokit.Tests/Reactive/ObservableIssueCommentsClientTests.cs index 2ad5fb87..2c6cb302 100644 --- a/Octokit.Tests/Reactive/ObservableIssueCommentsClientTests.cs +++ b/Octokit.Tests/Reactive/ObservableIssueCommentsClientTests.cs @@ -1,6 +1,7 @@ using System; using System.Collections.Generic; using System.Reactive.Linq; +using System.Reactive.Threading.Tasks; using System.Threading.Tasks; using NSubstitute; using Octokit.Reactive; @@ -29,10 +30,10 @@ namespace Octokit.Tests.Reactive { var client = new ObservableIssueCommentsClient(Substitute.For()); - await AssertEx.Throws(async () => await client.Get(null, "name", 1)); - await AssertEx.Throws(async () => await client.Get("", "name", 1)); - await AssertEx.Throws(async () => await client.Get("owner", null, 1)); - await AssertEx.Throws(async () => await client.Get("owner", "", 1)); + await Assert.ThrowsAsync(() => client.Get(null, "name", 1).ToTask()); + await Assert.ThrowsAsync(() => client.Get("", "name", 1).ToTask()); + await Assert.ThrowsAsync(() => client.Get("owner", null, 1).ToTask()); + await Assert.ThrowsAsync(() => client.Get("owner", "", 1).ToTask()); } } @@ -57,10 +58,10 @@ namespace Octokit.Tests.Reactive var gitHubClient = Substitute.For(); var client = new ObservableIssueCommentsClient(gitHubClient); - await AssertEx.Throws(async () => await client.GetAllForRepository(null, "name")); - await AssertEx.Throws(async () => await client.GetAllForRepository("", "name")); - await AssertEx.Throws(async () => await client.GetAllForRepository("owner", null)); - await AssertEx.Throws(async () => await client.GetAllForRepository("owner", "")); + await Assert.ThrowsAsync(() => client.GetAllForRepository(null, "name").ToTask()); + await Assert.ThrowsAsync(() => client.GetAllForRepository("", "name").ToTask()); + await Assert.ThrowsAsync(() => client.GetAllForRepository("owner", null).ToTask()); + await Assert.ThrowsAsync(() => client.GetAllForRepository("owner", "").ToTask()); } } @@ -84,10 +85,10 @@ namespace Octokit.Tests.Reactive var gitHubClient = Substitute.For(); var client = new ObservableIssueCommentsClient(gitHubClient); - await AssertEx.Throws(async () => await client.GetAllForIssue(null, "name", 1)); - await AssertEx.Throws(async () => await client.GetAllForIssue("", "name", 1)); - await AssertEx.Throws(async () => await client.GetAllForIssue("owner", null, 1)); - await AssertEx.Throws(async () => await client.GetAllForIssue("owner", "", 1)); + await Assert.ThrowsAsync(() => client.GetAllForIssue(null, "name", 1).ToTask()); + await Assert.ThrowsAsync(() => client.GetAllForIssue("", "name", 1).ToTask()); + await Assert.ThrowsAsync(() => client.GetAllForIssue("owner", null, 1).ToTask()); + await Assert.ThrowsAsync(() => client.GetAllForIssue("owner", "", 1).ToTask()); } } @@ -111,11 +112,11 @@ namespace Octokit.Tests.Reactive var gitHubClient = Substitute.For(); var client = new ObservableIssueCommentsClient(gitHubClient); - await AssertEx.Throws(async () => await client.Create(null, "name", 1, "title")); - await AssertEx.Throws(async () => await client.Create("", "name", 1, "x")); - await AssertEx.Throws(async () => await client.Create("owner", null, 1, "x")); - await AssertEx.Throws(async () => await client.Create("owner", "", 1, "x")); - await AssertEx.Throws(async () => await client.Create("owner", "name", 1, null)); + await Assert.ThrowsAsync(() => client.Create(null, "name", 1, "title").ToTask()); + await Assert.ThrowsAsync(() => client.Create("", "name", 1, "x").ToTask()); + await Assert.ThrowsAsync(() => client.Create("owner", null, 1, "x").ToTask()); + await Assert.ThrowsAsync(() => client.Create("owner", "", 1, "x").ToTask()); + await Assert.ThrowsAsync(() => client.Create("owner", "name", 1, null).ToTask()); } } @@ -139,11 +140,11 @@ namespace Octokit.Tests.Reactive var gitHubClient = Substitute.For(); var client = new ObservableIssueCommentsClient(gitHubClient); - await AssertEx.Throws(async () => await client.Update(null, "name", 42, "title")); - await AssertEx.Throws(async () => await client.Update("", "name", 42, "x")); - await AssertEx.Throws(async () => await client.Update("owner", null, 42, "x")); - await AssertEx.Throws(async () => await client.Update("owner", "", 42, "x")); - await AssertEx.Throws(async () => await client.Update("owner", "name", 42, null)); + await Assert.ThrowsAsync(() => client.Update(null, "name", 42, "title").ToTask()); + await Assert.ThrowsAsync(() => client.Update("", "name", 42, "x").ToTask()); + await Assert.ThrowsAsync(() => client.Update("owner", null, 42, "x").ToTask()); + await Assert.ThrowsAsync(() => client.Update("owner", "", 42, "x").ToTask()); + await Assert.ThrowsAsync(() => client.Update("owner", "name", 42, null).ToTask()); } }