fix(cmd) Fix proctype regex for scaling

This disallows upper case characters to match deis/controller checks.

Author: felixbuenemann

cmd/ps.go

@@ -142,13 +142,13 @@ func parseType(target string, appID string) (string, string) {
 
 func parsePsTargets(targets []string) (map[string]int, error) {
 	targetMap := make(map[string]int)
-	regex := regexp.MustCompile(`^([a-zA-Z0-9]+(\-[a-zA-Z0-9]+)*)=([0-9]+)$`)
+	regex := regexp.MustCompile(`^([a-z0-9]+(?:-[a-z0-9]+)*)=([0-9]+)$`)
 	var err error
 
 	for _, target := range targets {
 		if regex.MatchString(target) {
 			captures := regex.FindStringSubmatch(target)
-			targetMap[captures[1]], err = strconv.Atoi(captures[3])
+			targetMap[captures[1]], err = strconv.Atoi(captures[2])
 
 			if err != nil {
 				return nil, err

cmd/ps_test.go

@@ -119,9 +119,10 @@ func TestParsePsTargets(t *testing.T) {
 		{[]string{"test"}, true, nil, "'test' does not match the pattern 'type=num', ex: web=2\n"},
 		{[]string{"test=a"}, true, nil, "'test=a' does not match the pattern 'type=num', ex: web=2\n"},
 		{[]string{"test="}, true, nil, "'test=' does not match the pattern 'type=num', ex: web=2\n"},
+		{[]string{"Test=2"}, true, nil, "'Test=2' does not match the pattern 'type=num', ex: web=2\n"},
 		{[]string{"test=2"}, false, map[string]int{"test": 2}, ""},
 		{[]string{"test-proc=2"}, false, map[string]int{"test-proc": 2}, ""},
-		{[]string{"Test1=2"}, false, map[string]int{"Test1": 2}, ""},
+		{[]string{"test1=2"}, false, map[string]int{"test1": 2}, ""},
 	}
 
 	for _, check := range cases {