[FAB-10163] Detect trailing args for 'peer version'

Change-Id: I9fb340a163ea7e5244025d4f6fc3e4554f60183e
Signed-off-by: Zhenguo Niu <Niu.ZGlinux@gmail.com>

Author: niuzhenguo

peer/version/version.go

@@ -26,8 +26,14 @@ var cobraCommand = &cobra.Command{
 	Use:   "version",
 	Short: "Print fabric peer version.",
 	Long:  `Print current version of the fabric peer server.`,
-	Run: func(cmd *cobra.Command, args []string) {
+	RunE: func(cmd *cobra.Command, args []string) error {
+		if len(args) != 0 {
+			return fmt.Errorf("trailing args detected")
+		}
+		// Parsing of the command line is done so silence cmd usage
+		cmd.SilenceUsage = true
 		fmt.Print(GetInfo())
+		return nil
 	},
 }
 

peer/version/version_test.go

@@ -26,3 +26,10 @@ func TestCmd(t *testing.T) {
 	cmd := Cmd()
 	assert.NoError(t, cmd.Execute(), "expected version command to succeed")
 }
+
+func TestCmdWithTrailingArgs(t *testing.T) {
+	cmd := Cmd()
+	args := []string{"trailingargs"}
+	cmd.SetArgs(args)
+	assert.EqualError(t, cmd.Execute(), "trailing args detected")
+}