fix(opensearch): acc tests

Author: marceljk

go.sum

@@ -708,12 +708,8 @@ github.com/stackitcloud/stackit-sdk-go/services/opensearch v0.26.1 h1:AO5Np67/w0
 github.com/stackitcloud/stackit-sdk-go/services/opensearch v0.26.1/go.mod h1:Reo2Nk3yiA686quUMrk0eKwzpzaAZ7KwmzHzScNdG8Q=
 github.com/stackitcloud/stackit-sdk-go/services/postgresflex v1.3.5 h1:H67e3KnHQx954yI8fuQmxXwRf/myqAdLg2KvxImp00g=
 github.com/stackitcloud/stackit-sdk-go/services/postgresflex v1.3.5/go.mod h1:xmAWk9eom8wznvLuLfm0F4xyeiBX8LaggXsKFmos+dw=
-github.com/stackitcloud/stackit-sdk-go/services/rabbitmq v0.26.0 h1:/8lmviszgrB+0Cz7HdhFELyTiTeqIs7LfnI6sNX4rW8=
-github.com/stackitcloud/stackit-sdk-go/services/rabbitmq v0.26.0/go.mod h1:hnhvlLX1Y71R8KIQqLBeoSZqkU5ZJOG0J4wz0LeUdaw=
 github.com/stackitcloud/stackit-sdk-go/services/rabbitmq v0.29.0 h1:I8NF8Q5nRj7/7qDI8Qm4oI0KCNQcOjE6kqyBdWcUoxk=
 github.com/stackitcloud/stackit-sdk-go/services/rabbitmq v0.29.0/go.mod h1:wk2Oz+7pxT9YF1h6elWzdlhU6V7GSQXcGixxlXxNe+U=
-github.com/stackitcloud/stackit-sdk-go/services/redis v0.25.6 h1:CXM9cZ9WeTyJd+Aw/hnJnDsKRVAQi4qgtd0RJ3zoPwo=
-github.com/stackitcloud/stackit-sdk-go/services/redis v0.25.6/go.mod h1:KJNceOHRefjku1oVBoHG7idCS/SeW42WJ+55bN3AxrQ=
 github.com/stackitcloud/stackit-sdk-go/services/redis v0.27.3 h1:wZIGETPQ1aiJwzJMVjatdu/Kv+/yuKVOf5RaMWW9bpU=
 github.com/stackitcloud/stackit-sdk-go/services/redis v0.27.3/go.mod h1:jgWx29zvZP0R3tXAC/qWU5eBsPfLBJFP96mx9hwdFPM=
 github.com/stackitcloud/stackit-sdk-go/services/resourcemanager v0.20.0 h1:bT/qoghsdKvLomCay9Wj4Fu63lx0cD2zr23Fjqzgotc=

stackit/internal/conversion/conversion.go

@@ -156,7 +156,8 @@ func StringListToSlice(list basetypes.ListValue) ([]string, error) {
 		return nil, nil
 	}
 
-	var listStr []string
+	// Instantiate an empty slice to ensure the slice is not nil
+	listStr := []string{}
 	for i, el := range list.Elements() {
 		elStr, ok := el.(types.String)
 		if !ok {

stackit/internal/conversion/conversion_test.go

@@ -469,6 +469,11 @@ func TestStringListToSlice(t *testing.T) {
 			in:   basetypes.NewListNull(types.StringType),
 			want: nil,
 		},
+		{
+			name: "empty list",
+			in:   basetypes.NewListValueMust(types.StringType, []attr.Value{}),
+			want: []string{},
+		},
 		{
 			name:    "invalid type",
 			in:      basetypes.NewListValueMust(types.Int64Type, []attr.Value{types.Int64Value(123)}),

stackit/internal/services/opensearch/instance/resource_test.go

@@ -342,6 +342,57 @@ func TestToUpdatePayload(t *testing.T) {
 			},
 			true,
 		},
+		{
+			"plugins empty slice",
+			&Model{
+				PlanId: types.StringValue("plan"),
+				Parameters: types.ObjectValueMust(parametersTypes, map[string]attr.Value{
+					"sgw_acl":                types.StringValue("acl"),
+					"enable_monitoring":      types.BoolValue(true),
+					"graphite":               types.StringValue("graphite"),
+					"java_garbage_collector": types.StringValue(string(legacyOpensearch.INSTANCEPARAMETERSJAVA_GARBAGE_COLLECTOR_USE_G1_GC)),
+					"java_heapspace":         types.Int32Value(10),
+					"java_maxmetaspace":      types.Int32Value(10),
+					"max_disk_threshold":     types.Int32Value(10),
+					"metrics_frequency":      types.Int32Value(10),
+					"metrics_prefix":         types.StringValue("prefix"),
+					"monitoring_instance_id": types.StringValue("mid"),
+					"plugins":                types.ListValueMust(types.StringType, []attr.Value{}),
+					"syslog": types.ListValueMust(types.StringType, []attr.Value{
+						types.StringValue("syslog"),
+						types.StringValue("syslog2"),
+					}),
+					"tls_ciphers": types.ListValueMust(types.StringType, []attr.Value{
+						types.StringValue("cipher"),
+						types.StringValue("cipher2"),
+					}),
+					"tls_protocols": types.ListValueMust(types.StringType, []attr.Value{
+						types.StringValue("TLSv1.2"),
+						types.StringValue("TLSv1.3"),
+					}),
+				}),
+			},
+			&opensearch.PartialUpdateInstancePayload{
+				Parameters: &opensearch.InstanceParameters{
+					SgwAcl:               new("acl"),
+					EnableMonitoring:     new(true),
+					Graphite:             new("graphite"),
+					JavaGarbageCollector: new(string(legacyOpensearch.INSTANCEPARAMETERSJAVA_GARBAGE_COLLECTOR_USE_G1_GC)),
+					JavaHeapspace:        new(int32(10)),
+					JavaMaxmetaspace:     new(int32(10)),
+					MaxDiskThreshold:     new(int32(10)),
+					MetricsFrequency:     new(int32(10)),
+					MetricsPrefix:        new("prefix"),
+					MonitoringInstanceId: new("mid"),
+					Plugins:              []string{},
+					Syslog:               []string{"syslog", "syslog2"},
+					TlsCiphers:           []string{"cipher", "cipher2"},
+					TlsProtocols:         []string{"TLSv1.2", "TLSv1.3"},
+				},
+				PlanId: new("plan"),
+			},
+			true,
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.description, func(t *testing.T) {