cli: ensure scaling policy target doesn't have trailing comma

hashicorp · Oct 6, 2020 · 53b240b · 53b240b
1 parent 13466a9
commit 53b240b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/command/scaling_policy.go b/command/scaling_policy.go
@@ -75,5 +75,5 @@ func formatScalingPolicyTarget(t map[string]string) string {
 	if len(other) > 0 {
 		out = append(out, other...)
 	}
-	return strings.Join(out, ",")
+	return strings.Trim(strings.Join(out, ","), ",")
 }
diff --git a/command/scaling_policy_test.go b/command/scaling_policy_test.go
@@ -31,6 +31,13 @@ func Test_formatScalingPolicyTarget(t *testing.T) {
 			expectedOutput: "Namespace:default,Job:example,Group:cache,Unknown:alien",
 			name:           "extra key in input mapping",
 		},
+		{
+			inputMap: map[string]string{
+				"Namespace": "default",
+			},
+			expectedOutput: "Namespace:default",
+			name:           "single entry in map",
+		},
 	}
 
 	for _, tc := range testCases {