apply pr suggestion: trim StringList values from gherkin tests

alexandraoberaigner · alexandraoberaigner · commit 0f4ea572d564 · 2025-11-26T13:27:26.000+01:00
Signed-off-by: Alexandra Oberaigner &lt;alexandra.oberaigner@dynatrace.com&gt;
diff --git a/tests/flagd/testframework/utils.go b/tests/flagd/testframework/utils.go
@@ -79,8 +79,11 @@ func (vc *ValueConverter) ConvertToReflectValue(valueType, value string, fieldTy
 		}
 		return reflect.ValueOf(longVal).Convert(fieldType)
 	case "StringList":
-		arrayVal := strings.Split(value, ",")
-		return reflect.ValueOf(arrayVal).Convert(fieldType)
+		splitVal := strings.Split(value, ",")
+		for i, v := range splitVal {
+			splitVal[i] = strings.TrimSpace(v)
+		}
+		return reflect.ValueOf(splitVal).Convert(fieldType)
 	default:
 		return reflect.ValueOf(value).Convert(fieldType)
 	}

Original file line number	Diff line number	Diff line change
`@@ -79,8 +79,11 @@ func (vc *ValueConverter) ConvertToReflectValue(valueType, value string, fieldTy`
`79`	`79`	`}`
`80`	`80`	`return reflect.ValueOf(longVal).Convert(fieldType)`
`81`	`81`	`case "StringList":`
`82`		`- arrayVal := strings.Split(value, ",")`
`83`		`- return reflect.ValueOf(arrayVal).Convert(fieldType)`
	`82`	`+ splitVal := strings.Split(value, ",")`
	`83`	`+ for i, v := range splitVal {`
	`84`	`+ splitVal[i] = strings.TrimSpace(v)`
	`85`	`+ }`
	`86`	`+ return reflect.ValueOf(splitVal).Convert(fieldType)`
`84`	`87`	`default:`
`85`	`88`	`return reflect.ValueOf(value).Convert(fieldType)`
`86`	`89`	`}`