fix interval_min plan modifier

Author: oarbusi

internal/service/alertconfiguration/resource_alert_configuration.go

@@ -283,9 +283,6 @@ func (r *alertConfigurationRS) Schema(ctx context.Context, req resource.SchemaRe
 						"interval_min": schema.Int64Attribute{
 							Optional: true,
 							Computed: true,
-							PlanModifiers: []planmodifier.Int64{
-								int64planmodifier.UseStateForUnknown(),
-							},
 						},
 						"mobile_number": schema.StringAttribute{
 							Optional: true,

internal/service/alertconfiguration/resource_alert_configuration_test.go

@@ -430,7 +430,6 @@ func datadogTestCase(t *testing.T) *resource.TestCase {
 		},
 	}
 }
-
 func TestAccConfigRSAlertConfiguration_withPagerDuty(t *testing.T) {
 	proxyPort := replay.SetupReplayProxy(t)
 
@@ -464,6 +463,46 @@ func TestAccConfigRSAlertConfiguration_withPagerDuty(t *testing.T) {
 	})
 }
 
+func TestAccConfigRSAlertConfiguration_withEmailToPagerDuty(t *testing.T) {
+	proxyPort := replay.SetupReplayProxy(t)
+
+	var (
+		projectID  = replay.ManageProjectID(t, acc.ProjectIDExecution)
+		serviceKey = dummy32CharKey
+	)
+
+	resource.ParallelTest(t, resource.TestCase{
+		PreCheck:                 func() { acc.PreCheckBasic(t) },
+		ProtoV6ProviderFactories: acc.TestAccProviderV6FactoriesWithProxy(proxyPort),
+		CheckDestroy:             checkDestroyUsingProxy(proxyPort),
+		Steps: []resource.TestStep{
+			{
+				Config: configWithEmail(projectID, true),
+				Check: resource.ComposeAggregateTestCheckFunc(
+					checkExistsUsingProxy(proxyPort, resourceName),
+					resource.TestCheckResourceAttr(resourceName, "project_id", projectID),
+				),
+			},
+			{
+				Config: configWithPagerDuty(projectID, serviceKey, true),
+				Check: resource.ComposeAggregateTestCheckFunc(
+					checkExistsUsingProxy(proxyPort, resourceName),
+					resource.TestCheckResourceAttr(resourceName, "project_id", projectID),
+				),
+			},
+			{
+				ResourceName:      resourceName,
+				ImportStateIdFunc: importStateProjectIDFunc(resourceName),
+				ImportState:       true,
+				ImportStateVerify: true,
+				// service key is not returned by api in import operation
+				// integration_id is not returned during Create
+				ImportStateVerifyIgnore: []string{"updated", "notification.0.service_key", "notification.0.integration_id"},
+			},
+		},
+	})
+}
+
 func TestAccConfigAlertConfiguration_PagerDutyUsingIntegrationID(t *testing.T) {
 	// create a new project as it need to ensure no third party integration has already been created
 	var (
@@ -848,6 +887,22 @@ func configWithPagerDuty(projectID, serviceKey string, enabled bool) string {
 	`, projectID, serviceKey, enabled)
 }
 
+func configWithEmail(projectID string, enabled bool) string {
+	return fmt.Sprintf(`
+		resource "mongodbatlas_alert_configuration" "test" {
+			project_id = %[1]q
+			enabled    = %[2]t
+			event_type = "NO_PRIMARY"
+
+			notification {
+				type_name     = "EMAIL"
+				interval_min  = 60
+				email_address = "test@mongodbtest.com"
+			}
+		}
+	`, projectID, enabled)
+}
+
 func configWithPagerDutyIntegrationID(orgID, projectName, serviceKey string) string {
 	return fmt.Sprintf(`
 		resource "mongodbatlas_project" "test" {