Feat: add auto approve field to sub environment configuration

client/environment.go

@@ -62,6 +62,7 @@ type SubEnvironment struct {
 	Revision             string               `json:"revision,omitempty"`
 	Workspace            string               `json:"workspaceName,omitempty"`
 	ConfigurationChanges ConfigurationChanges `json:"configurationChanges"`
+	UserRequiresApproval bool                 `json:"userRequiresApproval"`
 }
 
 type DeployRequest struct {

client/remote_state_access_test.go

@@ -8,7 +8,7 @@ import (
 )
 
 var _ = Describe("RemoteStateAccess", func() {
-	environmentId := "environmnet_id"
+	environmentId := "environment_id"
 
 	remoteStateAccess := RemoteStateAccessConfiguration{
 		EnvironmentId:                    environmentId,

client/user_environment_assignment_test.go

@@ -9,7 +9,7 @@ import (
 	"go.uber.org/mock/gomock"
 )
 
-var _ = Describe("User Envrionment Assignment", func() {
+var _ = Describe("User Environment Assignment", func() {
 	environmentId := "environmentId"
 	userId := "userId"
 

env0/provider.go

@@ -199,7 +199,7 @@ func configureProvider(version string, p *schema.Provider) schema.ConfigureConte
 			AddRetryAfterErrorCondition().
 			AddRetryCondition(func(r *resty.Response, err error) bool {
 				if r == nil {
-					// No response. Possiblly a networking issue (E.g. DNS lookup failure).
+					// No response. Possibly a networking issue (E.g. DNS lookup failure).
 					tflog.SubsystemWarn(subCtx, "env0_api_client", "No response, retrying request")
 					return true
 				}

env0/resource_environment.go

@@ -16,12 +16,13 @@ import (
 )
 
 type SubEnvironment struct {
-	Id            string
-	Alias         string
-	Revision      string
-	Workflow      string
-	Workspace     string
-	Configuration client.ConfigurationChanges `tfschema:"-"`
+	Id                       string
+	Alias                    string
+	Revision                 string
+	Workflow                 string
+	Workspace                string
+	Configuration            client.ConfigurationChanges `tfschema:"-"`
+	ApprovePlanAutomatically bool
 }
 
 func getSubEnvironments(d *schema.ResourceData) ([]SubEnvironment, error) {
@@ -290,7 +291,7 @@ func resourceEnvironment() *schema.Resource {
 			},
 			"sub_environment_configuration": {
 				Type:        schema.TypeList,
-				Description: "the subenvironments for a workflow enviornment. Template type must be 'workflow'. Must match the configuration as defined in 'env0.workflow.yml'",
+				Description: "the subenvironments for a workflow environment. Template type must be 'workflow'. Must match the configuration as defined in 'env0.workflow.yml'",
 				Optional:    true,
 				Elem: &schema.Resource{
 					Schema: map[string]*schema.Schema{
@@ -320,6 +321,12 @@ func resourceEnvironment() *schema.Resource {
 							Optional:    true,
 							Elem:        configurationSchema,
 						},
+						"approve_plan_automatically": {
+							Type:        schema.TypeBool,
+							Description: "when 'true' (default) plans are approved automatically, otherwise ('false') deployment require manual approval",
+							Optional:    true,
+							Default:     true,
+						},
 					},
 				},
 			},
@@ -399,16 +406,16 @@ func setEnvironmentSchema(ctx context.Context, d *schema.ResourceData, environme
 			var newSubEnvironments []interface{}
 
 			for i, iSubEnvironment := range iSubEnvironments.([]interface{}) {
-				subEnviornment := iSubEnvironment.(map[string]interface{})
+				subEnvironment := iSubEnvironment.(map[string]interface{})
 
 				alias := d.Get(fmt.Sprintf("sub_environment_configuration.%d.alias", i)).(string)
 
 				workkflowSubEnvironment, ok := environment.LatestDeploymentLog.WorkflowFile.Environments[alias]
 				if ok {
-					subEnviornment["id"] = workkflowSubEnvironment.EnvironmentId
+					subEnvironment["id"] = workkflowSubEnvironment.EnvironmentId
 				}
 
-				newSubEnvironments = append(newSubEnvironments, subEnviornment)
+				newSubEnvironments = append(newSubEnvironments, subEnvironment)
 			}
 
 			d.Set("sub_environment_configuration", newSubEnvironments)
@@ -744,6 +751,7 @@ func deploy(d *schema.ResourceData, apiClient client.ApiClientInterface) diag.Di
 				Revision:             subEnvironment.Revision,
 				Workspace:            subEnvironment.Workspace,
 				ConfigurationChanges: configurationChanges,
+				UserRequiresApproval: !subEnvironment.ApprovePlanAutomatically,
 			}
 		}
 	}
@@ -885,6 +893,7 @@ func getCreatePayload(d *schema.ResourceData, apiClient client.ApiClientInterfac
 				Revision:             subEnvironment.Revision,
 				ConfigurationChanges: subEnvironment.Configuration,
 				Workspace:            subEnvironment.Workspace,
+				UserRequiresApproval: !subEnvironment.ApprovePlanAutomatically,
 			}
 		}
 	}

env0/resource_environment_test.go

@@ -2353,6 +2353,7 @@ func TestUnitEnvironmentWithSubEnvironment(t *testing.T) {
 	}
 
 	updatedSubEnvironment := subEnvironment
+	updatedSubEnvironment.ApprovePlanAutomatically = false
 	updatedSubEnvironment.Configuration = append(updatedSubEnvironment.Configuration, client.ConfigurationVariable{
 		Name:        "name2",
 		Value:       "value2",
@@ -2437,6 +2438,7 @@ func TestUnitEnvironmentWithSubEnvironment(t *testing.T) {
 				Revision:             subEnvironment.Revision,
 				Workspace:            updatedSubEnvironment.Workspace,
 				ConfigurationChanges: updatedSubEnvironment.Configuration,
+				UserRequiresApproval: true,
 			},
 		},
 	}
@@ -2499,6 +2501,7 @@ func TestUnitEnvironmentWithSubEnvironment(t *testing.T) {
 							alias = "%s"
 							revision = "%s"
 							workspace = "%s"
+							approve_plan_automatically = false
 							configuration {
 								name = "%s"
 								value = "%s"
@@ -2530,6 +2533,7 @@ func TestUnitEnvironmentWithSubEnvironment(t *testing.T) {
 						resource.TestCheckResourceAttr(accessor, "sub_environment_configuration.0.alias", updatedSubEnvironment.Alias),
 						resource.TestCheckResourceAttr(accessor, "sub_environment_configuration.0.revision", updatedSubEnvironment.Revision),
 						resource.TestCheckResourceAttr(accessor, "sub_environment_configuration.0.workspace", updatedSubEnvironment.Workspace),
+						resource.TestCheckResourceAttr(accessor, "sub_environment_configuration.0.approve_plan_automatically", "false"),
 						resource.TestCheckResourceAttr(accessor, "sub_environment_configuration.0.configuration.0.name", updatedSubEnvironment.Configuration[0].Name),
 						resource.TestCheckResourceAttr(accessor, "sub_environment_configuration.0.configuration.0.value", updatedSubEnvironment.Configuration[0].Value),
 						resource.TestCheckResourceAttr(accessor, "sub_environment_configuration.0.configuration.1.name", updatedSubEnvironment.Configuration[1].Name),

env0/resource_project_test.go

@@ -170,7 +170,7 @@ func TestUnitProjectResourceDestroyWithEnvironments(t *testing.T) {
 		Name: "name1",
 	}
 
-	t.Run("Success With Force Destory", func(t *testing.T) {
+	t.Run("Success With Force Destroy", func(t *testing.T) {
 		testCase := resource.TestCase{
 			Steps: []resource.TestStep{
 				{
@@ -199,7 +199,7 @@ func TestUnitProjectResourceDestroyWithEnvironments(t *testing.T) {
 		})
 	})
 
-	t.Run("Failure Without Force Destory", func(t *testing.T) {
+	t.Run("Failure Without Force Destroy", func(t *testing.T) {
 		testCase := resource.TestCase{
 			Steps: []resource.TestStep{
 				{

env0/resource_variable_set.go

@@ -39,7 +39,7 @@ var variableSetVariableSchema *schema.Resource = &schema.Resource{
 		},
 		"is_sensitive": {
 			Type:        schema.TypeBool,
-			Description: "is the value sensitive (defaults to 'false'). Note: 'dropdown' value format cannot be senstive.",
+			Description: "is the value sensitive (defaults to 'false'). Note: 'dropdown' value format cannot be sensitive.",
 			Optional:    true,
 			Default:     false,
 		},
@@ -293,15 +293,15 @@ type mergedVariables struct {
 func mergeVariables(schema []client.ConfigurationVariable, api []client.ConfigurationVariable) *mergedVariables {
 	var res mergedVariables
 
-	// To avoid false drifts, keep the order of the 'currentVariables' list similiar to the schema as much as possible.
+	// To avoid false drifts, keep the order of the 'currentVariables' list similar to the schema as much as possible.
 	for _, svariable := range schema {
 		found := false
 
 		for _, avariable := range api {
 			if svariable.Name == avariable.Name && *svariable.Type == *avariable.Type {
 				found = true
 				if avariable.IsSensitive != nil && *avariable.IsSensitive {
-					// Senstive - to avoid drift use the value from the schema
+					// Sensitive - to avoid drift use the value from the schema
 					avariable.Value = svariable.Value
 				}
 				res.currentVariables = append(res.currentVariables, avariable)

tests/integration/012_environment/main.tf

@@ -225,8 +225,9 @@ resource "env0_environment" "workflow-environment" {
   }
 
   sub_environment_configuration {
-    alias    = "rootService1"
-    revision = "master"
+    alias                      = "rootService1"
+    revision                   = "master"
+    approve_plan_automatically = var.second_run ? true : false
     configuration {
       name  = "sub_env1_var1"
       value = "hello"