Skip to content

Commit a571695

Browse files
author
Bret Ambrose
committed
Widen failure possibility based on CI
1 parent a1b918a commit a571695

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

eventstream_rpc/tests/EventStreamClientTest.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -166,8 +166,7 @@ static int s_TestEventStreamConnectFailureNoAuthHeader(struct aws_allocator *all
166166
auto future = connection.Connect(connectionConfig, &lifecycleHandler, *testContext.clientBootstrap);
167167
EventStreamRpcStatusCode clientStatus = future.get().baseStatus;
168168

169-
// TOFIX: this isn't reliably true on Windows over TCP due to RSTs blocking final data reads
170-
ASSERT_INT_EQUALS(EVENT_STREAM_RPC_CONNECTION_ACCESS_DENIED, clientStatus);
169+
ASSERT_TRUE(clientStatus == EVENT_STREAM_RPC_CRT_ERROR || clientStatus == EVENT_STREAM_RPC_CONNECTION_ACCESS_DENIED);
171170
}
172171

173172
return AWS_OP_SUCCESS;
@@ -200,8 +199,7 @@ static int s_TestEventStreamConnectFailureBadAuthHeader(struct aws_allocator *al
200199
auto future = connection.Connect(connectionConfig, &lifecycleHandler, *testContext.clientBootstrap);
201200
EventStreamRpcStatusCode clientStatus = future.get().baseStatus;
202201

203-
// TOFIX: this isn't reliably true on Windows over TCP due to RSTs blocking final data reads
204-
ASSERT_INT_EQUALS(EVENT_STREAM_RPC_CONNECTION_ACCESS_DENIED, clientStatus);
202+
ASSERT_TRUE(clientStatus == EVENT_STREAM_RPC_CRT_ERROR || clientStatus == EVENT_STREAM_RPC_CONNECTION_ACCESS_DENIED);
205203
}
206204

207205
return AWS_OP_SUCCESS;

0 commit comments

Comments
 (0)