Fixed wrong behaivor on registries with non default ports

- Added unit test which covers imagename creation with registries on non default ports.
- Added fix for BuildImageName on registries on non default ports

Signed-off-by: Markus Hartmann <[email protected]>

Author: aidun

schema/image.go

@@ -65,7 +65,18 @@ func (i *BuildFormat) Set(value string) error {
 // BuildImageName builds a Docker image tag for build, push or deploy
 func BuildImageName(format BuildFormat, image string, version string, branch string) string {
 	imageVal := image
-	if strings.Contains(image, ":") == false {
+
+	colon_count := strings.Count(image, ":")
+	if colon_count > 0 {
+
+		slash_index := strings.Index(image, "/")
+		colon_index := strings.Index(image, ":")
+
+		if slash_index-colon_index > 0 && colon_count == 1 {
+			imageVal += ":latest"
+		}
+
+	} else {
 		imageVal += ":latest"
 	}
 

schema/image_test.go

@@ -37,3 +37,21 @@ func Test_BuildImageName_BranchAndSHAFormat(t *testing.T) {
 		t.Errorf("BuildImageName want: \"%s\", got: \"%s\"", want, got)
 	}
 }
+
+func Test_BuildImageName_RegistryWithPort(t *testing.T) {
+	want := "registry.domain:8080/image:latest"
+	got := BuildImageName(DefaultFormat, "registry.domain:8080/image", "ef384", "master")
+
+	if got != want {
+		t.Errorf("BuildImageName want: \"%s\", got: \"%s\"", want, got)
+	}
+}
+
+func Test_BuildImageName_RegistryWithPortAndTag(t *testing.T) {
+	want := "registry.domain:8080/image:foo"
+	got := BuildImageName(DefaultFormat, "registry.domain:8080/image:foo", "ef384", "master")
+
+	if got != want {
+		t.Errorf("BuildImageName want: \"%s\", got: \"%s\"", want, got)
+	}
+}