Bash completion for flags

CMakeLists.txt

@@ -136,6 +136,7 @@ set(IGN_MATH_VER ${ignition-math6_VERSION_MAJOR})
 ign_find_package(ignition-tools
  REQUIRED
  PKGCONFIG "ignition-tools")
+set(IGN_TOOLS_VER 1)
 
 #--------------------------------------
 # Find protobuf

src/cmd/CMakeLists.txt

@@ -117,3 +117,17 @@ set(ign_model_ruby_path "${cmd_model_script_generated_test}")
 configure_file(
  "model.yaml.in"
  "${CMAKE_BINARY_DIR}/test/conf/model${PROJECT_VERSION_MAJOR}.yaml" @ONLY)
+
+
+#===============================================================================
+# Bash completion
+
+# Tack version onto and install the bash completion script
+configure_file(
+ "gazebo.bash_completion.sh"
+ "${CMAKE_CURRENT_BINARY_DIR}/gazebo${PROJECT_VERSION_MAJOR}.bash_completion.sh" @ONLY)
+install(
+ FILES
+ ${CMAKE_CURRENT_BINARY_DIR}/gazebo${PROJECT_VERSION_MAJOR}.bash_completion.sh
+ DESTINATION
+ ${CMAKE_INSTALL_PREFIX}/${CMAKE_INSTALL_DATAROOTDIR}/gz/gz${IGN_TOOLS_VER}.completion.d)

src/cmd/gazebo.bash_completion.sh

@@ -0,0 +1,49 @@
+#!/usr/bin/env bash
+
+# bash tab-completion
+
+# This is a per-library function definition, used in conjunction with the
+# top-level entry point in ign-tools.
+
+# NOTE: In Garden+, replace `gazebo` in function name to align with subcommand,
+# for ign-tools/etc/ign.bash_completion.sh to find this function.
+function _gz_gazebo
+{
+ if [[ ${COMP_WORDS[COMP_CWORD]} == -* ]]; then
+ # Specify options (-*) word list for this subcommand
+ COMPREPLY=($(compgen -W "
+ -g
+ --iterations
+ --levels
+ --network-role
+ --network-secondaries
+ --record
+ --record-path
+ --record-resources
+ --record-topic
+ --log-overwrite
+ --log-compress
+ --playback
+ -r
+ -s
+ -v --verbose
+ --gui-config
+ --physics-engine
+ --render-engine
+ --render-engine-gui
+ --render-engine-server
+ --version
+ -z
+ -h --help
+ --force-version
+ --versions
+ " -- "${COMP_WORDS[COMP_CWORD]}" ))
+ return
+ else
+ # Just use bash default auto-complete, because we never have two
+ # subcommands in the same line. If that is ever needed, change here to
+ # detect subsequent subcommands
+ COMPREPLY=($(compgen -o default -- "${COMP_WORDS[COMP_CWORD]}"))
+ return
+ fi
+}

src/ign_TEST.cc

@@ -19,6 +19,7 @@
 #include <cstdio>
 #include <cstdlib>
 
+#include <fstream>
 #include <string>
 #include <ignition/common/Util.hh>
 #include <ignition/utilities/ExtraTestMacros.hh>
@@ -160,3 +161,68 @@ TEST(CmdLine, ResourcePath)
  EXPECT_EQ(output.find("Unable to find file plugins.sdf"), std::string::npos)
  << output;
 }
+
+//////////////////////////////////////////////////
+/// \brief Check --help message and bash completion script for consistent flags
+TEST(CmdLine, GazeboHelpVsCompletionFlags)
+{
+ // Flags in help message
+ std::string output = customExecStr(kIgnCommand + " gazebo --help");
+ EXPECT_NE(std::string::npos, output.find("-g")) << output;
+ EXPECT_NE(std::string::npos, output.find("--iterations")) << output;
+ EXPECT_NE(std::string::npos, output.find("--levels")) << output;
+ EXPECT_NE(std::string::npos, output.find("--network-role")) << output;
+ EXPECT_NE(std::string::npos, output.find("--network-secondaries")) << output;
+ EXPECT_NE(std::string::npos, output.find("--record")) << output;
+ EXPECT_NE(std::string::npos, output.find("--record-path")) << output;
+ EXPECT_NE(std::string::npos, output.find("--record-resources")) << output;
+ EXPECT_NE(std::string::npos, output.find("--record-topic")) << output;
+ EXPECT_NE(std::string::npos, output.find("--log-overwrite")) << output;
+ EXPECT_NE(std::string::npos, output.find("--log-compress")) << output;
+ EXPECT_NE(std::string::npos, output.find("--playback")) << output;
+ EXPECT_NE(std::string::npos, output.find("-r")) << output;
+ EXPECT_NE(std::string::npos, output.find("-s")) << output;
+ EXPECT_NE(std::string::npos, output.find("--verbose")) << output;
+ EXPECT_NE(std::string::npos, output.find("--gui-config")) << output;
+ EXPECT_NE(std::string::npos, output.find("--physics-engine")) << output;
+ EXPECT_NE(std::string::npos, output.find("--render-engine")) << output;
+ EXPECT_NE(std::string::npos, output.find("--render-engine-gui")) << output;
+ EXPECT_NE(std::string::npos, output.find("--render-engine-server")) << output;
+ EXPECT_NE(std::string::npos, output.find("--version")) << output;
+ EXPECT_NE(std::string::npos, output.find("-z")) << output;
+ EXPECT_NE(std::string::npos, output.find("--help")) << output;
+ EXPECT_NE(std::string::npos, output.find("--force-version")) << output;
+ EXPECT_NE(std::string::npos, output.find("--versions")) << output;
+
+ // Flags in bash completion
+ std::ifstream scriptFile(std::string(PROJECT_SOURCE_PATH) +
+ "/src/cmd/gazebo.bash_completion.sh");
+ std::string script((std::istreambuf_iterator<char>(scriptFile)),
+ std::istreambuf_iterator<char>());
+
+ EXPECT_NE(std::string::npos, script.find("-g")) << script;
+ EXPECT_NE(std::string::npos, script.find("--iterations")) << script;
+ EXPECT_NE(std::string::npos, script.find("--levels")) << script;
+ EXPECT_NE(std::string::npos, script.find("--network-role")) << script;
+ EXPECT_NE(std::string::npos, script.find("--network-secondaries")) << script;
+ EXPECT_NE(std::string::npos, script.find("--record")) << script;
+ EXPECT_NE(std::string::npos, script.find("--record-path")) << script;
+ EXPECT_NE(std::string::npos, script.find("--record-resources")) << script;
+ EXPECT_NE(std::string::npos, script.find("--record-topic")) << script;
+ EXPECT_NE(std::string::npos, script.find("--log-overwrite")) << script;
+ EXPECT_NE(std::string::npos, script.find("--log-compress")) << script;
+ EXPECT_NE(std::string::npos, script.find("--playback")) << script;
+ EXPECT_NE(std::string::npos, script.find("-r")) << script;
+ EXPECT_NE(std::string::npos, script.find("-s")) << script;
+ EXPECT_NE(std::string::npos, script.find("--verbose")) << script;
+ EXPECT_NE(std::string::npos, script.find("--gui-config")) << script;
+ EXPECT_NE(std::string::npos, script.find("--physics-engine")) << script;
+ EXPECT_NE(std::string::npos, script.find("--render-engine")) << script;
+ EXPECT_NE(std::string::npos, script.find("--render-engine-gui")) << script;
+ EXPECT_NE(std::string::npos, script.find("--render-engine-server")) << script;
+ EXPECT_NE(std::string::npos, script.find("--version")) << script;
+ EXPECT_NE(std::string::npos, script.find("-z")) << script;
+ EXPECT_NE(std::string::npos, script.find("--help")) << script;
+ EXPECT_NE(std::string::npos, script.find("--force-version")) << script;
+ EXPECT_NE(std::string::npos, script.find("--versions")) << script;
+}