Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16790 )
Change subject: [tools] add missing top-level entries into TestModeHelp ...................................................................... [tools] add missing top-level entries into TestModeHelp This patch adds missing entries into the ToolTest.TestModeHelp scenario for the existing top-level commands of the kudu CLI tool, extending the coverage of the scenario to all currently existing top-level kudu CLI commands. I also did a minor clean-up on the code: rearranged the entries in the scenario's code to match the sequence of those in the output of the tool when run with no arguments or with --help flag. Change-Id: I04c6744429e7a6a87ed6e87c1f5cab9c67d0f327 Reviewed-on: http://gerrit.cloudera.org:8080/16790 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Grant Henke <[email protected]> --- M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_master.cc M src/kudu/tools/tool_action_tserver.cc 3 files changed, 48 insertions(+), 42 deletions(-) Approvals: Alexey Serbin: Verified Grant Henke: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/16790 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I04c6744429e7a6a87ed6e87c1f5cab9c67d0f327 Gerrit-Change-Number: 16790 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
