Add in acceptance tests

Author: helenjw

internal/provider/resource_tfe_hyok_configuration.go

@@ -220,18 +220,23 @@ func (r *resourceTFEHYOKConfiguration) Create(ctx context.Context, req resource.
 	}
 
 	var azureOIDCConfig *tfe.AzureOIDCConfiguration
-	if plan.VaultOIDCConfigurationID.ValueString() != "" {
+	if plan.AzureOIDCConfigurationID.ValueString() != "" {
 		azureOIDCConfig = &tfe.AzureOIDCConfiguration{ID: plan.AzureOIDCConfigurationID.ValueString()}
 	}
 
-	options := tfe.HYOKConfigurationsCreateOptions{
-		KEKID: plan.KEKID.ValueString(),
-		Name:  plan.Name.ValueString(),
-		KMSOptions: &tfe.KMSOptions{
+	var kmsOptions *tfe.KMSOptions
+	if plan.KMSOptions != nil {
+		kmsOptions = &tfe.KMSOptions{
 			KeyRegion:   plan.KMSOptions.KeyRegion.ValueString(),
 			KeyLocation: plan.KMSOptions.KeyLocation.ValueString(),
 			KeyRingID:   plan.KMSOptions.KeyRingID.ValueString(),
-		},
+		}
+	}
+
+	options := tfe.HYOKConfigurationsCreateOptions{
+		KEKID:      plan.KEKID.ValueString(),
+		Name:       plan.Name.ValueString(),
+		KMSOptions: kmsOptions,
 		OIDCConfiguration: &tfe.OIDCConfigurationTypeChoice{
 			AWSOIDCConfiguration:   awsOIDCConfig,
 			GCPOIDCConfiguration:   gcpOIDCConfig,
@@ -295,15 +300,20 @@ func (r *resourceTFEHYOKConfiguration) Update(ctx context.Context, req resource.
 		return
 	}
 
-	options := tfe.HYOKConfigurationsUpdateOptions{
-		Name:  plan.Name.ValueStringPointer(),
-		KEKID: plan.KEKID.ValueStringPointer(),
-		KMSOptions: &tfe.KMSOptions{
+	var kmsOptions *tfe.KMSOptions
+	if plan.KMSOptions != nil {
+		kmsOptions = &tfe.KMSOptions{
 			KeyRegion:   plan.KMSOptions.KeyRegion.ValueString(),
 			KeyLocation: plan.KMSOptions.KeyLocation.ValueString(),
 			KeyRingID:   plan.KMSOptions.KeyRingID.ValueString(),
-		},
-		AgentPool: &tfe.AgentPool{ID: plan.AgentPoolID.ValueString()},
+		}
+	}
+
+	options := tfe.HYOKConfigurationsUpdateOptions{
+		Name:       plan.Name.ValueStringPointer(),
+		KEKID:      plan.KEKID.ValueStringPointer(),
+		KMSOptions: kmsOptions,
+		AgentPool:  &tfe.AgentPool{ID: plan.AgentPoolID.ValueString()},
 	}
 
 	hyokID := state.ID.ValueString()
@@ -340,10 +350,13 @@ func (r *resourceTFEHYOKConfiguration) Delete(ctx context.Context, req resource.
 }
 
 func modelFromTFEHYOKConfiguration(p *tfe.HYOKConfiguration) modelTFEHYOKConfiguration {
-	kmsOptions := modelTFEKMSOptions{
-		KeyRegion:   types.StringValue(p.KMSOptions.KeyRegion),
-		KeyLocation: types.StringValue(p.KMSOptions.KeyLocation),
-		KeyRingID:   types.StringValue(p.KMSOptions.KeyRingID),
+	var kmsOptions *modelTFEKMSOptions
+	if p.KMSOptions != nil {
+		kmsOptions = &modelTFEKMSOptions{
+			KeyRegion:   types.StringValue(p.KMSOptions.KeyRegion),
+			KeyLocation: types.StringValue(p.KMSOptions.KeyLocation),
+			KeyRingID:   types.StringValue(p.KMSOptions.KeyRingID),
+		}
 	}
 
 	model := modelTFEHYOKConfiguration{
@@ -352,7 +365,7 @@ func modelFromTFEHYOKConfiguration(p *tfe.HYOKConfiguration) modelTFEHYOKConfigu
 		KEKID:        types.StringValue(p.KEKID),
 		Organization: types.StringValue(p.Organization.Name),
 		AgentPoolID:  types.StringValue(p.AgentPool.ID),
-		KMSOptions:   &kmsOptions,
+		KMSOptions:   kmsOptions,
 	}
 
 	if p.OIDCConfiguration.AWSOIDCConfiguration != nil {