pr review: added function for generating name of workload of an app

Signed-off-by: Florian Bacher <[email protected]>

operator/api/v1alpha1/keptnappversion_types.go

@@ -355,3 +355,7 @@ func (a KeptnAppVersion) GetSpanAttributes() []attribute.KeyValue {
 		common.AppNamespace.String(a.Namespace),
 	}
 }
+
+func (v KeptnAppVersion) GetWorkloadNameOfApp(workloadName string) string {
+	return fmt.Sprintf("%s-%s", v.Spec.AppName, workloadName)
+}

operator/api/v1alpha1/keptnappversion_types_test.go

@@ -0,0 +1,41 @@
+package v1alpha1
+
+import (
+	"testing"
+)
+
+func TestKeptnAppVersion_GetWorkloadNameOfApp(t *testing.T) {
+	type fields struct {
+		Spec KeptnAppVersionSpec
+	}
+	type args struct {
+		workloadName string
+	}
+	tests := []struct {
+		name   string
+		fields fields
+		args   args
+		want   string
+	}{
+		{
+			name: "",
+			fields: fields{
+				Spec: KeptnAppVersionSpec{AppName: "my-app"},
+			},
+			args: args{
+				workloadName: "my-workload",
+			},
+			want: "my-app-my-workload",
+		},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			v := KeptnAppVersion{
+				Spec: tt.fields.Spec,
+			}
+			if got := v.GetWorkloadNameOfApp(tt.args.workloadName); got != tt.want {
+				t.Errorf("GetWorkloadNameOfApp() = %v, want %v", got, tt.want)
+			}
+		})
+	}
+}

operator/controllers/keptnworkloadinstance/controller.go

@@ -299,7 +299,7 @@ func getLatestAppVersion(apps *klcv1alpha1.KeptnAppVersionList, wli *klcv1alpha1
 	for _, app := range apps.Items {
 		if app.Spec.AppName == wli.Spec.AppName {
 			for _, appWorkload := range app.Spec.Workloads {
-				if appWorkload.Version == wli.Spec.Version && fmt.Sprintf("%s-%s", app.Spec.AppName, appWorkload.Name) == wli.Spec.WorkloadName {
+				if appWorkload.Version == wli.Spec.Version && app.GetWorkloadNameOfApp(appWorkload.Name) == wli.Spec.WorkloadName {
 					workloadFound = true
 					newVersion, err := version.NewVersion(app.Spec.Version)
 					if err != nil {