diff options
author | Yash Tibrewal <yashkt@google.com> | 2018-03-22 11:44:13 -0700 |
---|---|---|
committer | Yash Tibrewal <yashkt@google.com> | 2018-03-22 11:44:13 -0700 |
commit | e7ef1e5968cf81f622843f2712a9eeec434d1189 (patch) | |
tree | e7665160fd67b51b8a4a44c821a0efbecd52d565 /test/core/end2end/h2_ssl_cert_test.cc | |
parent | 7ed7beef6820a6de545c1af4c219663e0045449a (diff) |
Combine cq draining and server destroy for h2_ssl_cert_test
Diffstat (limited to 'test/core/end2end/h2_ssl_cert_test.cc')
-rw-r--r-- | test/core/end2end/h2_ssl_cert_test.cc | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/test/core/end2end/h2_ssl_cert_test.cc b/test/core/end2end/h2_ssl_cert_test.cc index 9addfff232..0741e7c39b 100644 --- a/test/core/end2end/h2_ssl_cert_test.cc +++ b/test/core/end2end/h2_ssl_cert_test.cc @@ -268,13 +268,13 @@ static void drain_cq(grpc_completion_queue* cq) { } while (ev.type != GRPC_QUEUE_SHUTDOWN); } +// Shuts down the server. +// Side effect - Also shuts down and drains the completion queue. static void shutdown_server(grpc_end2end_test_fixture* f) { if (!f->server) return; grpc_server_shutdown_and_notify(f->server, f->cq, tag(1000)); - grpc_event ev = grpc_completion_queue_next( - f->cq, grpc_timeout_seconds_to_deadline(5), nullptr); - ASSERT_EQ(ev.type, GRPC_OP_COMPLETE); - ASSERT_EQ(ev.tag, tag(1000)); + grpc_completion_queue_shutdown(f->cq); + drain_cq(f->cq); grpc_server_destroy(f->server); f->server = nullptr; } @@ -288,9 +288,6 @@ static void shutdown_client(grpc_end2end_test_fixture* f) { static void end_test(grpc_end2end_test_fixture* f) { shutdown_client(f); shutdown_server(f); - - grpc_completion_queue_shutdown(f->cq); - drain_cq(f->cq); grpc_completion_queue_destroy(f->cq); } |