Skip to content

Commit

Permalink
interop-testing: adds assertions to empty_stream test case (#7030)
Browse files Browse the repository at this point in the history
If the server fails (e.g.: with an unimplemented status), the test still succeeds.
This change adds assertions to check the status code as well as the expected number of responses.
  • Loading branch information
plaflamme committed May 13, 2020
1 parent f9353e0 commit cc2d376
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -790,6 +790,8 @@ public void emptyStream() throws Exception {
= asyncStub.fullDuplexCall(responseObserver);
requestObserver.onCompleted();
responseObserver.awaitCompletion(operationTimeoutMillis(), TimeUnit.MILLISECONDS);
assertSuccess(responseObserver);
assertTrue("Expected an empty stream", responseObserver.getValues().isEmpty());
}

@Test
Expand Down

0 comments on commit cc2d376

Please sign in to comment.