Adding missing asserts to HTTP/2 tests
Motivation: Some tests do not properly assert that all requests have been sent/received, so the failures messages may be misleading. Modifications: Adding missing asserts to HTTP/2 tests for awaiting requests and responses. Result: HTTP/2 tests properly assert message counts.
This commit is contained in:
parent
b785128349
commit
f8890768be
@ -224,7 +224,7 @@ public class DefaultHttp2ToHttpConnectionHandlerTest {
|
|||||||
}
|
}
|
||||||
|
|
||||||
private void awaitRequests() throws Exception {
|
private void awaitRequests() throws Exception {
|
||||||
requestLatch.await(2, SECONDS);
|
assertTrue(requestLatch.await(2, SECONDS));
|
||||||
}
|
}
|
||||||
|
|
||||||
private ChannelHandlerContext ctx() {
|
private ChannelHandlerContext ctx() {
|
||||||
|
@ -367,7 +367,7 @@ public class Http2FrameRoundtripTest {
|
|||||||
}
|
}
|
||||||
|
|
||||||
private void awaitRequests(long seconds) throws InterruptedException {
|
private void awaitRequests(long seconds) throws InterruptedException {
|
||||||
requestLatch.await(seconds, SECONDS);
|
assertTrue(requestLatch.await(seconds, SECONDS));
|
||||||
}
|
}
|
||||||
|
|
||||||
private void awaitRequests() throws InterruptedException {
|
private void awaitRequests() throws InterruptedException {
|
||||||
|
@ -670,11 +670,11 @@ public class InboundHttp2ToHttpAdapterTest {
|
|||||||
}
|
}
|
||||||
|
|
||||||
private void awaitRequests() throws Exception {
|
private void awaitRequests() throws Exception {
|
||||||
serverLatch.await(2, SECONDS);
|
assertTrue(serverLatch.await(2, SECONDS));
|
||||||
}
|
}
|
||||||
|
|
||||||
private void awaitResponses() throws Exception {
|
private void awaitResponses() throws Exception {
|
||||||
clientLatch.await(2, SECONDS);
|
assertTrue(clientLatch.await(2, SECONDS));
|
||||||
}
|
}
|
||||||
|
|
||||||
private ChannelHandlerContext ctxClient() {
|
private ChannelHandlerContext ctxClient() {
|
||||||
|
Loading…
x
Reference in New Issue
Block a user