diff --git a/test/cpp/util/cli_credentials.h b/test/cpp/util/cli_credentials.h index a351eaaeb1cda023cccb5f0466f2f5dd28694e6e..299b8ff6cfbbcc7c906be304b8b583d70a0df491 100644 --- a/test/cpp/util/cli_credentials.h +++ b/test/cpp/util/cli_credentials.h @@ -42,6 +42,7 @@ namespace testing { class CliCredentials { public: + virtual ~CliCredentials() {}; virtual std::shared_ptr<grpc::ChannelCredentials> GetCredentials() const; virtual const grpc::string GetCredentialUsage() const; }; diff --git a/test/cpp/util/proto_file_parser.h b/test/cpp/util/proto_file_parser.h index 300b5789bc49c21918f3376f8d4a79bb36e9dabc..b9e4a1ec2b9399e1b20756eb1c1cabdf7a54f833 100644 --- a/test/cpp/util/proto_file_parser.h +++ b/test/cpp/util/proto_file_parser.h @@ -48,15 +48,18 @@ class ErrorPrinter; // Find method and associated request/response types. class ProtoFileParser { public: - // The given proto file_name will be searched in a source tree rooted from - // proto_path. The method could be a partial string such as Service.Method or - // even just Method. It will log an error if there is ambiguity. - ProtoFileParser(std::shared_ptr<grpc::Channel> channel, - const grpc::string& proto_path, - const grpc::string& protofiles); + // The parser will search proto files using the server reflection service + // provided on the given channel. The given protofiles in a source tree rooted + // from proto_path will also be searched. + explicit ProtoFileParser(std::shared_ptr<grpc::Channel> channel, + const grpc::string& proto_path, + const grpc::string& protofiles); ~ProtoFileParser(); + // The input method name in the following four functions could be a partial + // string such as Service.Method or even just Method. It will log an error if + // there is ambiguity. // Full method name is in the form of Service.Method, it's good to be used in // descriptor database queries. grpc::string GetFullMethodName(const grpc::string& method);