From ee7db600d39a67c6bb1c64a519f6d8e201c18e3e Mon Sep 17 00:00:00 2001 From: Hyunjae Woo Date: Tue, 5 Sep 2023 17:21:06 -0700 Subject: [PATCH] Add bi-directional gRPC streaming options for periodic concurrency mode --- src/c++/perf_analyzer/test_command_line_parser.cc | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/src/c++/perf_analyzer/test_command_line_parser.cc b/src/c++/perf_analyzer/test_command_line_parser.cc index 9592472c4..172e9a5a1 100644 --- a/src/c++/perf_analyzer/test_command_line_parser.cc +++ b/src/c++/perf_analyzer/test_command_line_parser.cc @@ -1194,8 +1194,13 @@ TEST_CASE("Testing Command Line Parser") char* option_name = "--periodic-concurrency-range"; // Add required args that specifies where to dump profiled data - args.push_back("--profile-export-file"); - args.push_back("profile.json"); + args.insert( + args.end(), {"-i", "grpc", "--async", "--streaming", + "--profile-export-file", "profile.json"}); + exp->protocol = cb::ProtocolType::GRPC; + exp->async = true; + exp->streaming = true; + exp->url = "localhost:8001"; // gRPC url CheckValidRange( args, option_name, parser, act, exp->using_periodic_concurrency_range, @@ -1229,7 +1234,7 @@ TEST_CASE("Testing Command Line Parser") SUBCASE("no export file specified") { - // Remove the required args + // Remove the export file args args.pop_back(); args.pop_back();