commit 785d6b656aac4f94d6685166652bc08c2932beda
author AJ Heller <hork@google.com> Tue Jul 19 12:18:02 2022 -0700
committer GitHub <noreply@github.com> Tue Jul 19 12:18:02 2022 -0700
tree 506586ebf47712a2e31b3abb06f506b36b3734f9
parent 4341a810f6ea4993560b31586cac2f1d850ae1ec

Add logging for invalid client PayloadConfig in QPS test (#30261)

* Add invalid PayloadConfig logging to QPS test

* flip conditions

test/cpp/qps/client.h

diff --git a/test/cpp/qps/client.h b/test/cpp/qps/client.h
index 61914e9..1ec0dda 100644
--- a/test/cpp/qps/client.h
+++ b/test/cpp/qps/client.h
@@ -71,6 +71,9 @@
   ClientRequestCreator(SimpleRequest* req,
                        const PayloadConfig& payload_config) {
     if (payload_config.has_bytebuf_params()) {
+      gpr_log(GPR_ERROR,
+              "Invalid PayloadConfig, config cannot have bytebuf_params: %s",
+              payload_config.DebugString().c_str());
       GPR_ASSERT(false);  // not appropriate for this specialization
     } else if (payload_config.has_simple_params()) {
       req->set_response_type(grpc::testing::PayloadType::COMPRESSABLE);
@@ -81,6 +84,9 @@
       std::unique_ptr<char[]> body(new char[size]);
       req->mutable_payload()->set_body(body.get(), size);
     } else if (payload_config.has_complex_params()) {
+      gpr_log(GPR_ERROR,
+              "Invalid PayloadConfig, cannot have complex_params: %s",
+              payload_config.DebugString().c_str());
       GPR_ASSERT(false);  // not appropriate for this specialization
     } else {
       // default should be simple proto without payloads
@@ -104,6 +110,8 @@
       Slice slice(buf.get(), req_sz);
       *req = ByteBuffer(&slice, 1);
     } else {
+      gpr_log(GPR_ERROR, "Invalid PayloadConfig, missing bytebug_params: %s",
+              payload_config.DebugString().c_str());
       GPR_ASSERT(false);  // not appropriate for this specialization
     }
   }