diff --git a/Octokit.Tests.Integration/Clients/AuthorizationClientTests.cs b/Octokit.Tests.Integration/Clients/AuthorizationClientTests.cs index c3bac05f..4bb6b6a1 100644 --- a/Octokit.Tests.Integration/Clients/AuthorizationClientTests.cs +++ b/Octokit.Tests.Integration/Clients/AuthorizationClientTests.cs @@ -40,7 +40,7 @@ namespace Octokit.Tests.Integration.Clients Assert.True(error.Result.Message.Contains("username and password Basic Auth")); } - [BasicAuthenticationTest(Skip = "This test seems to be flaky")] + [BasicAuthenticationTest(Skip = "See https://github.com/octokit/octokit.net/issues/1000 for issue to investigate this further")] public async Task CanCreateAndGetAuthorizationWithoutFingerPrint() { var github = Helper.GetBasicAuthClient(); diff --git a/Octokit.Tests.Integration/Clients/RepositoriesClientTests.cs b/Octokit.Tests.Integration/Clients/RepositoriesClientTests.cs index f3ad5c71..6a2299e9 100644 --- a/Octokit.Tests.Integration/Clients/RepositoriesClientTests.cs +++ b/Octokit.Tests.Integration/Clients/RepositoriesClientTests.cs @@ -439,7 +439,7 @@ public class RepositoriesClientTests public class TheDeleteMethod { - [IntegrationTest(Skip = "not sure why this is now failing to delete successfully")] + [IntegrationTest(Skip = "See https://github.com/octokit/octokit.net/issues/1002 for investigating this failing test")] public async Task DeletesRepository() { var github = Helper.GetAuthenticatedClient(); diff --git a/Octokit.Tests.Integration/Clients/RepositoryDeployKeysClientTests.cs b/Octokit.Tests.Integration/Clients/RepositoryDeployKeysClientTests.cs index 3481dc61..998770f5 100644 --- a/Octokit.Tests.Integration/Clients/RepositoryDeployKeysClientTests.cs +++ b/Octokit.Tests.Integration/Clients/RepositoryDeployKeysClientTests.cs @@ -21,7 +21,7 @@ public class RepositoryDeployKeysClientTests : IDisposable _context = github.CreateRepositoryContext("public-repo").Result; } - [IntegrationTest(Skip = "see https://github.com/octokit/octokit.net/issues/533 for the resolution to this failing test")] + [IntegrationTest(Skip = "see https://github.com/octokit/octokit.net/issues/533 for investigating this failing test")] public async Task CanCreateADeployKey() { var deployKey = new NewDeployKey() @@ -36,7 +36,7 @@ public class RepositoryDeployKeysClientTests : IDisposable Assert.Equal(_keyTitle, deployKeyResult.Title); } - [IntegrationTest(Skip = "this test is triggering a validation failure because the key is already in use")] + [IntegrationTest(Skip = "See https://github.com/octokit/octokit.net/issues/1003 for investigating this failing test")] public async Task CanRetrieveAllDeployKeys() { var deployKeys = await _fixture.GetAll(_context.RepositoryOwner, _context.RepositoryName); diff --git a/Octokit.Tests.Integration/Reactive/ObservableRespositoryDeployKeysClientTests.cs b/Octokit.Tests.Integration/Reactive/ObservableRespositoryDeployKeysClientTests.cs index e2f78b70..19fe4a83 100644 --- a/Octokit.Tests.Integration/Reactive/ObservableRespositoryDeployKeysClientTests.cs +++ b/Octokit.Tests.Integration/Reactive/ObservableRespositoryDeployKeysClientTests.cs @@ -1,6 +1,5 @@ using System; using System.Reactive.Linq; -using System.Runtime.Remoting; using System.Threading.Tasks; using Octokit; using Octokit.Reactive; @@ -26,7 +25,7 @@ public class ObservableRespositoryDeployKeysClientTests : IDisposable _owner = _repository.Owner.Login; } - [IntegrationTest(Skip = "see https://github.com/octokit/octokit.net/issues/533 for the resolution to this failing test")] + [IntegrationTest(Skip = "see https://github.com/octokit/octokit.net/issues/533 for investigating this failing test")] public async Task CanCreateADeployKey() { var deployKey = new NewDeployKey() @@ -43,7 +42,7 @@ public class ObservableRespositoryDeployKeysClientTests : IDisposable Assert.Equal(_keyTitle, createdDeployKey.Title); } - [IntegrationTest(Skip = "this test fails validation due to the key existing on the server")] + [IntegrationTest(Skip = "See https://github.com/octokit/octokit.net/issues/1003 for investigating this failing test")] public async Task CanRetrieveAllDeployKeys() { var deployKeys = await _client.GetAll(_owner, _repository.Name).ToList(); @@ -62,7 +61,7 @@ public class ObservableRespositoryDeployKeysClientTests : IDisposable Assert.Equal(_keyTitle, deployKeys[0].Title); } - [IntegrationTest(Skip = "see https://github.com/octokit/octokit.net/issues/533 for the resolution to this failing test")] + [IntegrationTest(Skip = "see https://github.com/octokit/octokit.net/issues/533 for investigating this failing test")] public async Task CanRetrieveADeployKey() { var newDeployKey = new NewDeployKey() @@ -80,7 +79,7 @@ public class ObservableRespositoryDeployKeysClientTests : IDisposable Assert.Equal(_keyTitle, deployKey.Title); } - [IntegrationTest(Skip = "see https://github.com/octokit/octokit.net/issues/533 for the resolution to this failing test")] + [IntegrationTest(Skip = "see https://github.com/octokit/octokit.net/issues/533 for investigating this failing test")] public async Task CanRemoveADeployKey() { var newDeployKey = new NewDeployKey()