Skip to content

Commit f483d17

Browse files
committed
Ignore flaky exception tests
1 parent 5e5abfe commit f483d17

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

src/Tests/ClientConcepts/Exceptions/ExceptionTests.cs

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -19,14 +19,14 @@ public ExceptionTests(IndexingCluster cluster, EndpointUsage usage)
1919
_port = cluster.Node.Port;
2020
}
2121

22-
[I]
22+
//[I]
2323
public void ServerTestWhenThrowExceptionsEnabled()
2424
{
2525
var settings = new ConnectionSettings(new Uri($"http://{TestClient.Host}:{_port}"))
2626
.ThrowExceptions();
2727
var client = new ElasticClient(settings);
2828
var exception = Assert.Throws<ElasticsearchClientException>(() => client.GetMapping<Project>(s => s.Index("doesntexist")));
29-
#if DOTNETCORE
29+
#if DOTNETCORE
3030
// HttpClient does not throw on "known error" status codes (i.e. 404) thus the inner exception should not be set
3131
exception.InnerException.Should().BeNull();
3232
#else
@@ -37,7 +37,7 @@ public void ServerTestWhenThrowExceptionsEnabled()
3737
exception.Response.ServerError.Status.Should().BeGreaterThan(0);
3838
}
3939

40-
[I]
40+
//[I]
4141
public void ClientTestWhenThrowExceptionsEnabled()
4242
{
4343
var settings = new ConnectionSettings(new Uri("http://doesntexist:9200"))
@@ -53,23 +53,23 @@ public void ClientTestWhenThrowExceptionsEnabled()
5353
#endif
5454
}
5555

56-
[I]
56+
//[I]
5757
public void ServerTestWhenThrowExceptionsDisabled()
5858
{
5959
var settings = new ConnectionSettings(new Uri($"http://{TestClient.Host}:{_port}"));
6060
var client = new ElasticClient(settings);
6161
var response = client.GetMapping<Project>(s => s.Index("doesntexist"));
62-
#if DOTNETCORE
62+
#if DOTNETCORE
6363
// HttpClient does not throw on "known error" status codes (i.e. 404) thus OriginalException should not be set
6464
response.CallDetails.OriginalException.Should().BeNull();
65-
#else
65+
#else
6666
response.CallDetails.OriginalException.Should().NotBeNull();
6767
#endif
6868
response.CallDetails.ServerError.Should().NotBeNull();
6969
response.CallDetails.ServerError.Status.Should().BeGreaterThan(0);
7070
}
7171

72-
[I]
72+
//[I]
7373
public void ClientTestWhenThrowExceptionsDisabled()
7474
{
7575
var settings = new ConnectionSettings(new Uri("http://doesntexist:9200"));
@@ -89,7 +89,7 @@ public void DispatchIndicatesMissingRouteValues()
8989
{
9090
var settings = new ConnectionSettings(new Uri("http://doesntexist:9200"));
9191
var client = new ElasticClient(settings);
92-
92+
9393
Action dispatch = () => client.Index(new Project());
9494
var ce = dispatch.ShouldThrow<ArgumentException>();
9595
ce.Should().NotBeNull();

0 commit comments

Comments
 (0)