Merge pull request #1559 from buildtesters/alias_for_buildtest_config…

…_validate alias for buildtest config validate command
buildtesters · Jul 17, 2023 · bec8564 · bec8564
2 parents 5acc622 + 858142c
commit bec8564
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/bash_completion.sh b/bash_completion.sh
@@ -237,7 +237,7 @@ _buildtest ()
     ;;
 
     config|cg)
-      local cmds="-h --help co compilers e edit ex executors p path systems validate v view"
+      local cmds="-h --help co compilers e edit ex executors p path systems val validate v view"
 
       COMPREPLY=( $( compgen -W "${cmds}" -- $cur ) )
       # handle completion logic for 'buildtest config <subcommand>' based on subcommands
@@ -312,7 +312,7 @@ _buildtest ()
       ;;
 
     buildspec|bc)
-      local cmds="-h --help ef edit-file et edit-test f find maintainers s show sf show-fail sm summary val validate"
+      local cmds="-h --help ef edit-file et edit-test f find m maintainers s show sf show-fail sm summary val validate"
       COMPREPLY=( $( compgen -W "${cmds}" -- $cur ) )
 
       # switch based on 2nd word 'buildtest buildspec <subcommand>'

diff --git a/buildtest/cli/__init__.py b/buildtest/cli/__init__.py
@@ -991,7 +991,9 @@ def config_menu(subparsers, parent_parser):
     subparsers_config.add_parser("systems", help="List all available systems")
     # buildtest config validate
     subparsers_config.add_parser(
-        "validate", help="Validate buildtest settings file with schema."
+        "validate",
+        aliases=["val"],
+        help="Validate buildtest settings file with schema.",
     )
     # buildtest config view
     subparsers_config.add_parser(

diff --git a/buildtest/cli/config.py b/buildtest/cli/config.py
@@ -34,7 +34,7 @@ def config_cmd(args, configuration, editor, system):
             args.invalid,
         )
 
-    elif args.config == "validate":
+    elif args.config in ["validate", "val"]:
         validate_config(configuration, system.system["moduletool"])
 
     elif args.config == "systems":