Ensure that '--version' flag works properly for root command

Make it so that, in the case that the root command is not runnable
but has subcommands, specifying a '--version' flag will still
run the "version" behavior.
This commit is contained in:
Nick Miyake 2017-12-06 23:08:30 -08:00
parent c156af3984
commit f8d9e63479
2 changed files with 6 additions and 2 deletions

View file

@ -685,7 +685,7 @@ func (c *Command) execute(a []string) (err error) {
return err
}
if helpVal || !c.Runnable() {
if helpVal {
return flag.ErrHelp
}
@ -705,6 +705,10 @@ func (c *Command) execute(a []string) (err error) {
}
}
if !c.Runnable() {
return flag.ErrHelp
}
c.preRun()
argWoFlags := c.Flags().Args()

View file

@ -867,7 +867,7 @@ func TestVersionTemplate(t *testing.T) {
}
func TestVersionFlagExecutedOnSubcommand(t *testing.T) {
rootCmd := &Command{Use: "root", Version: "1.0.0", Run: emptyRun}
rootCmd := &Command{Use: "root", Version: "1.0.0"}
rootCmd.AddCommand(&Command{Use: "sub", Run: emptyRun})
output, err := executeCommand(rootCmd, "--version", "sub")