Merge pull request #1151 from ovh/dev/aamstutz/fix-cloud-alerting

fix(cloud_project_alerting): Email not returned by API anymore

Author: amstuta

ovh/resource_cloud_project_alerting.go

@@ -63,6 +63,9 @@ func (r *cloudProjectAlertingResource) Create(ctx context.Context, req resource.
 
 	responseData.MergeWith(&data)
 
+	// Explicitly set email to the plan value as it's not returned by the API
+	responseData.Email = data.Email
+
 	// Save data into Terraform state
 	resp.Diagnostics.Append(resp.State.Set(ctx, &responseData)...)
 }
@@ -129,6 +132,9 @@ func (r *cloudProjectAlertingResource) Update(ctx context.Context, req resource.
 
 	responseData.MergeWith(&planData)
 
+	// Explicitly set email to the plan value as it's not returned by the API
+	responseData.Email = planData.Email
+
 	// Save updated data into Terraform state
 	resp.Diagnostics.Append(resp.State.Set(ctx, &responseData)...)
 }

ovh/resource_cloud_project_alerting_gen.go

@@ -48,7 +48,7 @@ func TestAccCloudProjectAlerting_basic(t *testing.T) {
 					resource.TestCheckResourceAttr(
 						"ovh_cloud_project_alerting.alert", "formatted_monthly_threshold.currency_code", "EUR"),
 					resource.TestCheckResourceAttr(
-						"ovh_cloud_project_alerting.alert", "formatted_monthly_threshold.text", "3000.00 €"),
+						"ovh_cloud_project_alerting.alert", "formatted_monthly_threshold.text", "3 000,00 €"),
 					resource.TestCheckResourceAttr(
 						"ovh_cloud_project_alerting.alert", "formatted_monthly_threshold.value", "3000"),
 				),
@@ -65,7 +65,7 @@ func TestAccCloudProjectAlerting_basic(t *testing.T) {
 					resource.TestCheckResourceAttr(
 						"ovh_cloud_project_alerting.alert", "formatted_monthly_threshold.currency_code", "EUR"),
 					resource.TestCheckResourceAttr(
-						"ovh_cloud_project_alerting.alert", "formatted_monthly_threshold.text", "100.00 €"),
+						"ovh_cloud_project_alerting.alert", "formatted_monthly_threshold.text", "100,00 €"),
 					resource.TestCheckResourceAttr(
 						"ovh_cloud_project_alerting.alert", "formatted_monthly_threshold.value", "100"),
 				),