Add support for order in action (#62)

Author: dvirsegev

docs/resources/port_action.md

@@ -32,6 +32,7 @@ Action resource
 - `gitlab_method` (Attributes) The invocation method of the action (see [below for nested schema](#nestedatt--gitlab_method))
 - `icon` (String) Icon
 - `kafka_method` (Object) The invocation method of the action (see [below for nested schema](#nestedatt--kafka_method))
+- `order_properties` (List of String) Order properties
 - `required_approval` (Boolean) Require approval before invoking the action
 - `user_properties` (Attributes) User properties (see [below for nested schema](#nestedatt--user_properties))
 - `webhook_method` (Attributes) The invocation method of the action (see [below for nested schema](#nestedatt--webhook_method))

internal/cli/models.go

@@ -158,6 +158,7 @@ type (
 	ActionUserInputs = struct {
 		Properties map[string]ActionProperty `json:"properties"`
 		Required   []string                  `json:"required,omitempty"`
+		Order      []string                  `json:"order,omitempty"`
 	}
 
 	Blueprint struct {

port/action/actionStateToPortBody.go

@@ -5,6 +5,7 @@ import (
 
 	"github.com/port-labs/terraform-provider-port-labs/internal/cli"
 	"github.com/port-labs/terraform-provider-port-labs/internal/consts"
+	"github.com/port-labs/terraform-provider-port-labs/internal/utils"
 )
 
 func actionDataSetToPortBody(dataSet *DatasetModel) *cli.Dataset {
@@ -85,6 +86,15 @@ func actionStateToPortBody(ctx context.Context, data *ActionModel, bp *cli.Bluep
 		action.UserInputs.Properties = make(map[string]cli.ActionProperty)
 	}
 
+	if !data.OrderProperties.IsNull() {
+		order, err := utils.TerraformListToGoArray(ctx, data.OrderProperties, "string")
+		if err != nil {
+			return nil, err
+		}
+		orderString := utils.InterfaceToStringArray(order)
+		action.UserInputs.Order = orderString
+	}
+
 	return action, nil
 }
 

port/action/model.go

@@ -165,4 +165,5 @@ type ActionModel struct {
 	UserProperties              *UserPropertiesModel              `tfsdk:"user_properties"`
 	ApprovalWebhookNotification *ApprovalWebhookNotificationModel `tfsdk:"approval_webhook_notification"`
 	ApprovalEmailNotification   types.Object                      `tfsdk:"approval_email_notification"`
+	OrderProperties             types.List                        `tfsdk:"order_properties"`
 }

port/action/refreshActionState.go

@@ -190,6 +190,9 @@ func writeInputsToResource(ctx context.Context, a *cli.Action, state *ActionMode
 			}
 		}
 		state.UserProperties = properties
+		if len(a.UserInputs.Order) > 0 {
+			state.OrderProperties = flex.GoArrayStringToTerraformList(ctx, a.UserInputs.Order)
+		}
 	}
 	return nil
 }

port/action/resource_test.go

@@ -585,3 +585,50 @@ func TestAccPortActionJqDefault(t *testing.T) {
 	})
 
 }
+
+func TestAccPortActionOrderProperties(t *testing.T) {
+	identifier := utils.GenID()
+	actionIdentifier := utils.GenID()
+	var testAccActionConfigCreate = testAccCreateBlueprintConfig(identifier) + fmt.Sprintf(`
+	resource "port_action" "action1" {
+		title = "Action 1"
+		identifier = "%s"
+		icon = "Terraform"
+		blueprint = port_blueprint.microservice.id
+		trigger = "DAY-2"
+		kafka_method = {}
+		order_properties = ["myStringIdentifier2", "myStringIdentifier1"]
+		user_properties = {
+			string_props = {
+				myStringIdentifier1 = {
+					title      = "myStringIdentifier1"
+					required   = false
+				}
+				myStringIdentifier2 = {
+					title      = "myStringIdentifier2"
+					required   = false
+				}
+			}
+		}
+	}`, actionIdentifier)
+
+	resource.Test(t, resource.TestCase{
+		PreCheck:                 func() { acctest.TestAccPreCheck(t) },
+		ProtoV6ProviderFactories: acctest.TestAccProtoV6ProviderFactories,
+
+		Steps: []resource.TestStep{
+			{
+				Config: acctest.ProviderConfig + testAccActionConfigCreate,
+				Check: resource.ComposeTestCheckFunc(
+					resource.TestCheckResourceAttr("port_action.action1", "title", "Action 1"),
+					resource.TestCheckResourceAttr("port_action.action1", "identifier", actionIdentifier),
+					resource.TestCheckResourceAttr("port_action.action1", "trigger", "DAY-2"),
+					resource.TestCheckResourceAttr("port_action.action1", "user_properties.string_props.myStringIdentifier1.title", "myStringIdentifier1"),
+					resource.TestCheckResourceAttr("port_action.action1", "user_properties.string_props.myStringIdentifier2.title", "myStringIdentifier2"),
+					resource.TestCheckResourceAttr("port_action.action1", "order_properties.0", "myStringIdentifier2"),
+					resource.TestCheckResourceAttr("port_action.action1", "order_properties.1", "myStringIdentifier1"),
+				),
+			},
+		},
+	})
+}

port/action/schema.go

@@ -230,6 +230,11 @@ func ActionSchema() map[string]schema.Attribute {
 				},
 			},
 		},
+		"order_properties": schema.ListAttribute{
+			MarkdownDescription: "Order properties",
+			Optional:            true,
+			ElementType:         types.StringType,
+		},
 		"user_properties": schema.SingleNestedAttribute{
 			MarkdownDescription: "User properties",
 			Optional:            true,