Collect metrics --help flag

Signed-off-by: Guillaume Tardif <guillaume.tardif@docker.com>

metrics/commands.go

@@ -18,7 +18,7 @@ package metrics
 
 var commandFlags = []string{
 	//added to catch scan details
-	"--version", "--login",
+	"--version", "--login", "--help",
 }
 
 // Generated with generatecommands/main.go

metrics/metrics.go

@@ -51,9 +51,6 @@ func GetCommand(args []string) string {
 	result := ""
 	onlyFlags := false
 	for _, arg := range args {
-		if arg == "--help" {
-			return ""
-		}
 		if arg == "--" {
 			break
 		}
@@ -63,7 +60,7 @@ func GetCommand(args []string) string {
 			} else {
 				result = result + " " + arg
 			}
-			if !isManagementCommand(arg) {
+			if isCommand(arg) && !isManagementCommand(arg) {
 				onlyFlags = true
 			}
 		}

metrics/metrics_test.go

@@ -153,26 +153,26 @@ func TestGetCommand(t *testing.T) {
 	}
 }
 
-func TestIgnoreHelpCommands(t *testing.T) {
+func TestKeepHelpCommands(t *testing.T) {
 	testCases := []struct {
 		name     string
 		args     []string
 		expected string
 	}{
 		{
-			name:     "help",
-			args:     []string{"--help"},
-			expected: "",
-		},
-		{
-			name:     "help on run",
+			name:     "run with help flag",
 			args:     []string{"run", "--help"},
-			expected: "",
+			expected: "run --help",
 		},
 		{
-			name:     "help on compose up",
-			args:     []string{"compose", "up", "--help"},
-			expected: "",
+			name:     "with help flag before-after commands",
+			args:     []string{"compose", "--help", "up"},
+			expected: "compose --help up",
+		},
+		{
+			name:     "help flag",
+			args:     []string{"--help"},
+			expected: "--help",
 		},
 	}