diff options
author | Sree Kuchibhotla <sreek@google.com> | 2016-02-01 10:53:13 -0800 |
---|---|---|
committer | Sree Kuchibhotla <sreek@google.com> | 2016-02-01 10:53:13 -0800 |
commit | 369a04ace686d2db7fff8a39473680566da6700a (patch) | |
tree | 1910641c472f2435b6b10b096dc798b19c6ad2df /test/cpp/end2end | |
parent | 3075c810280b797b5c932c38e0382633cfe3774c (diff) |
Address code review comments
Diffstat (limited to 'test/cpp/end2end')
-rw-r--r-- | test/cpp/end2end/async_end2end_test.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/test/cpp/end2end/async_end2end_test.cc b/test/cpp/end2end/async_end2end_test.cc index 0deb2eff95..252bda3798 100644 --- a/test/cpp/end2end/async_end2end_test.cc +++ b/test/cpp/end2end/async_end2end_test.cc @@ -884,13 +884,12 @@ class AsyncEnd2endServerTryCancelTest : public AsyncEnd2endTest { // Server sends the final message and cancelled status (but the RPC is // already cancelled at this point. So we expect the operation to fail) - send_response.set_message("Pong"); srv_stream.Finish(send_response, Status::CANCELLED, tag(9)); Verifier(GetParam()).Expect(9, false).Verify(cq_.get()); // Client will see the cancellation cli_stream->Finish(&recv_status, tag(10)); - // TODO: sreek: The expectation here should be true. This is a bug (github + // TODO(sreek): The expectation here should be true. This is a bug (github // issue #4972) Verifier(GetParam()).Expect(10, false).Verify(cq_.get()); EXPECT_FALSE(recv_status.ok()); |