Fix build failures with istio test (#30074)

* Fix ubsan failure

* Fix build failures with istio test

* Fix other status code
pull/30047/head
Alisha Nanda 3 years ago committed by GitHub
parent a285704aaf
commit e085568c88
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 6
      test/cpp/interop/istio_echo_server_test.cc

@ -49,7 +49,7 @@ class SimpleEchoTestServerImpl : public proto::EchoTestService::Service {
explicit SimpleEchoTestServerImpl() {} explicit SimpleEchoTestServerImpl() {}
grpc::Status Echo(grpc::ServerContext* /* context */, grpc::Status Echo(grpc::ServerContext* /* context */,
const proto::EchoRequest* request, const proto::EchoRequest* /* request */,
proto::EchoResponse* /* response */) override { proto::EchoResponse* /* response */) override {
GPR_ASSERT(false); GPR_ASSERT(false);
return Status(StatusCode::INVALID_ARGUMENT, "Unexpected"); return Status(StatusCode::INVALID_ARGUMENT, "Unexpected");
@ -59,7 +59,7 @@ class SimpleEchoTestServerImpl : public proto::EchoTestService::Service {
const proto::ForwardEchoRequest* request, const proto::ForwardEchoRequest* request,
proto::ForwardEchoResponse* response) override { proto::ForwardEchoResponse* response) override {
if (fail_rpc_) { if (fail_rpc_) {
return Status(UNAVAILABLE, "fail rpc"); return Status(StatusCode::UNAVAILABLE, "fail rpc");
} }
response->add_output(request->message()); response->add_output(request->message());
return Status::OK; return Status::OK;
@ -184,7 +184,7 @@ TEST_F(EchoTest, ForwardEchoFailure) {
request.set_url(absl::StrCat("http://", server_address_)); request.set_url(absl::StrCat("http://", server_address_));
request.set_message("hello"); request.set_message("hello");
auto status = stub_->ForwardEcho(&context, request, &response); auto status = stub_->ForwardEcho(&context, request, &response);
ASSERT_EQ(status.error_code(), UNAVAILABLE); ASSERT_EQ(status.error_code(), StatusCode::UNAVAILABLE);
} }
} // namespace } // namespace

Loading…
Cancel
Save