fix(builder): the value could be a number

Author: aledbf

builder/types.go

@@ -38,15 +38,15 @@ type BuildHookResponse struct {
 
 // Config represents a Deis application's configuration.
 type Config struct {
-	Owner   string            `json:"owner"`
-	App     string            `json:"app"`
-	Values  map[string]string `json:"values"`
-	Memory  map[string]string `json:"memory"`
-	CPU     map[string]string `json:"cpu"`
-	Tags    map[string]string `json:"tags"`
-	UUID    string            `json:"uuid"`
-	Created DeisTime          `json:"created"`
-	Updated DeisTime          `json:"updated"`
+	Owner   string                 `json:"owner"`
+	App     string                 `json:"app"`
+	Values  map[string]interface{} `json:"values"`
+	Memory  map[string]string      `json:"memory"`
+	CPU     map[string]int         `json:"cpu"`
+	Tags    map[string]string      `json:"tags"`
+	UUID    string                 `json:"uuid"`
+	Created DeisTime               `json:"created"`
+	Updated DeisTime               `json:"updated"`
 }
 
 // DeisTime represents the standard datetime format used across the platform.

builder/utils_test.go

@@ -59,7 +59,7 @@ func TestParseConfigGood(t *testing.T) {
 	resp := &http.Response{
 		Body: &ClosingBuffer{bytes.NewBufferString(`{"owner": "test",
 			"app": "example-go",
-			"values": {"FOO": "bar"},
+			"values": {"FOO": "bar", "CAR": 1234},
 			"memory": {},
 			"cpu": {},
 			"tags": {},
@@ -76,7 +76,15 @@ func TestParseConfigGood(t *testing.T) {
 	}
 
 	if config.Values["FOO"] != "bar" {
-		t.Errorf("expected FOO='bar', got FOO='%s'", config.Values["FOO"])
+		t.Errorf("expected FOO='bar', got FOO='%v'", config.Values["FOO"])
+	}
+
+	if car, ok := config.Values["CAR"].(float64); ok {
+		if car != 1234 {
+			t.Errorf("expected CAR=1234, got CAR=%d", config.Values["CAR"])
+		}
+	} else {
+		t.Error("expected CAR to be of type float64")
 	}
 }