fix: field possible unknown for RequireIfAttributeIsOneOf

FrangipaneTeam · Jun 13, 2023 · 6a4935f · 6a4935f
1 parent 0e449bd
commit 6a4935f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/internal/require_if_attribute_is_one_of.go b/internal/require_if_attribute_is_one_of.go
@@ -46,10 +46,10 @@ func (av RequireIfAttributeIsOneOf) Description(_ context.Context) string {
 	var expectedValueDescritpion string
 	for i, expectedValue := range av.ExceptedValues {
 		if i == len(av.ExceptedValues)-1 {
-			expectedValueDescritpion += fmt.Sprintf("%s, ", expectedValue.String())
+			expectedValueDescritpion += expectedValue.String()
 			break
 		}
-		expectedValueDescritpion += expectedValue.String()
+		expectedValueDescritpion += fmt.Sprintf("%s, ", expectedValue.String())
 	}
 	return fmt.Sprintf("If %s attribute is set and the value is one of %s, this attribute is REQUIRED", av.PathExpression, expectedValueDescritpion)
 }
@@ -110,7 +110,7 @@ func (av RequireIfAttributeIsOneOf) Validate(ctx context.Context, req RequireIfA
 
 		for _, expectedValue := range av.ExceptedValues {
 			if mpVal.Equal(expectedValue) {
-				if req.ConfigValue.IsNull() || req.ConfigValue.IsUnknown() {
+				if req.ConfigValue.IsNull() {
 					res.Diagnostics.AddAttributeError(
 						path,
 						fmt.Sprintf("Invalid configuration for attribute %s", req.Path),