Merge pull request #8423 from ncteisen/interop_client_naming

Interop Client Naming Conventions
pull/8367/head
Noah Eisen 8 years ago committed by GitHub
commit 74b731e42f
  1. 2
      src/proto/grpc/testing/test.proto
  2. 2
      test/cpp/interop/client.cc
  3. 2
      test/cpp/interop/interop_client.cc

@ -77,7 +77,7 @@ service TestService {
// The test server will not implement this method. It will be used // The test server will not implement this method. It will be used
// to test the behavior when clients call unimplemented methods. // to test the behavior when clients call unimplemented methods.
rpc UnimplementedMethod(grpc.testing.Empty) returns (grpc.testing.Empty); rpc UnimplementedCall(grpc.testing.Empty) returns (grpc.testing.Empty);
} }
// A simple service NOT implemented at servers so clients can test for // A simple service NOT implemented at servers so clients can test for

@ -80,7 +80,7 @@ DEFINE_string(test_case, "large_unary",
"slow client consumer;\n" "slow client consumer;\n"
"status_code_and_message: verify status code & message;\n" "status_code_and_message: verify status code & message;\n"
"timeout_on_sleeping_server: deadline exceeds on stream;\n" "timeout_on_sleeping_server: deadline exceeds on stream;\n"
"unimplemented_method: client calls an unimplemented_method;\n"); "unimplemented_method: client calls an unimplemented method;\n");
DEFINE_string(default_service_account, "", DEFINE_string(default_service_account, "",
"Email of GCE default service account"); "Email of GCE default service account");
DEFINE_string(service_account_key_file, "", DEFINE_string(service_account_key_file, "",

@ -1010,7 +1010,7 @@ bool InteropClient::DoUnimplementedMethod() {
ClientContext context; ClientContext context;
Status s = Status s =
serviceStub_.Get()->UnimplementedMethod(&context, request, &response); serviceStub_.Get()->UnimplementedCall(&context, request, &response);
if (!AssertStatusCode(s, StatusCode::UNIMPLEMENTED)) { if (!AssertStatusCode(s, StatusCode::UNIMPLEMENTED)) {
return false; return false;

Loading…
Cancel
Save