feat: add regional schema helper for Plugin Framework actions

Author: jremy42

internal/locality/regional/schemas_framework.go

@@ -0,0 +1,29 @@
+package regional
+
+import (
+	"github.com/hashicorp/terraform-plugin-framework-validators/stringvalidator"
+	"github.com/hashicorp/terraform-plugin-framework/action/schema"
+	"github.com/hashicorp/terraform-plugin-framework/schema/validator"
+	"github.com/scaleway/scaleway-sdk-go/scw"
+)
+
+// AllRegions returns all valid Scaleway regions as strings
+func AllRegions() []string {
+	regions := make([]string, 0, len(scw.AllRegions))
+	for _, r := range scw.AllRegions {
+		regions = append(regions, r.String())
+	}
+
+	return regions
+}
+
+// SchemaAttribute returns a Plugin Framework schema attribute for a region field
+func SchemaAttribute() schema.StringAttribute {
+	return schema.StringAttribute{
+		Optional:    true,
+		Description: "The region you want to attach the resource to",
+		Validators: []validator.String{
+			stringvalidator.OneOf(AllRegions()...),
+		},
+	}
+}

internal/services/cockpit/action_trigger_test_alert_action.go

@@ -9,6 +9,7 @@ import (
 	"github.com/hashicorp/terraform-plugin-framework/types"
 	"github.com/scaleway/scaleway-sdk-go/api/cockpit/v1"
 	"github.com/scaleway/scaleway-sdk-go/scw"
+	"github.com/scaleway/terraform-provider-scaleway/v2/internal/locality/regional"
 	"github.com/scaleway/terraform-provider-scaleway/v2/internal/meta"
 )
 
@@ -62,10 +63,7 @@ func (a *TriggerTestAlertAction) Schema(ctx context.Context, req action.SchemaRe
 				Required:    true,
 				Description: "ID of the Project",
 			},
-			"region": schema.StringAttribute{
-				Optional:    true,
-				Description: "Region to target. If not provided, will use the default region from the provider configuration",
-			},
+			"region": regional.SchemaAttribute(),
 		},
 	}
 }
@@ -99,8 +97,19 @@ func (a *TriggerTestAlertAction) Invoke(ctx context.Context, req action.InvokeRe
 	}
 
 	var region scw.Region
+
 	if !data.Region.IsNull() && data.Region.ValueString() != "" {
-		region = scw.Region(data.Region.ValueString())
+		parsedRegion, err := scw.ParseRegion(data.Region.ValueString())
+		if err != nil {
+			resp.Diagnostics.AddError(
+				"Invalid region",
+				fmt.Sprintf("The region attribute must be a valid Scaleway region. Got %q: %s", data.Region.ValueString(), err),
+			)
+
+			return
+		}
+
+		region = parsedRegion
 	} else {
 		// Use default region from provider configuration
 		defaultRegion, exists := a.meta.ScwClient().GetDefaultRegion()