SUMO-0 removing the validation

Author: priyanshgoel

sumologic/resource_sumologic_field.go

@@ -137,10 +137,6 @@ func resourceSumologicFieldUpdate(d *schema.ResourceData, meta interface{}) erro
 		return err
 	}
 
-	if f.FieldName != name || f.DataType != tpe {
-		return errors.New("Only state field is updatable")
-	}
-
 	if state == "Enabled" {
 		err := c.EnableField(id)
 		if err != nil {

sumologic/resource_sumologic_field_test.go

@@ -92,53 +92,6 @@ func TestAccSumologicField_update(t *testing.T) {
 	})
 }
 
-func TestAccSumologicField_OnlyStateFieldIsUpdatable(t *testing.T) {
-
-	var field Field
-
-	resourceName := "sumologic_field.test"
-	testFieldName := "fields_provider_test"
-	testDataType := "String"
-	testState := "Enabled"
-	updatedDataType := "int"
-
-	resource.Test(t, resource.TestCase{
-		PreCheck:     func() { testAccPreCheck(t) },
-		Providers:    testAccProviders,
-		CheckDestroy: testAccCheckFieldDestroy(field),
-		Steps: []resource.TestStep{
-			{
-				Config: fmt.Sprintf(`
-					resource "sumologic_field" "test" {
-						field_name = "%s"
-						data_type  = "%s"
-						state      = "%s"
-					}
-				`, testFieldName, testDataType, testState),
-				Check: resource.ComposeTestCheckFunc(
-					resource.TestCheckResourceAttr(resourceName, "field_name", testFieldName),
-					resource.TestCheckResourceAttr(resourceName, "data_type", testDataType),
-					resource.TestCheckResourceAttr(resourceName, "state", testState),
-				),
-			},
-
-			{
-				Config: fmt.Sprintf(`
-					resource "sumologic_field" "test" {
-						field_name = "%s"
-						data_type  = "%s"
-						state      = "%s"
-					}
-				`, testFieldName, updatedDataType, testState),
-				ExpectError: regexp.MustCompile("Only state field is updatable"),
-				Check: resource.ComposeTestCheckFunc(
-                    resource.TestCheckResourceAttr(resourceName, "data_type", testDataType),
-                ),
-			},
-		},
-	})
-}
-
 func testAccCheckFieldDestroy(field Field) resource.TestCheckFunc {
 	return func(s *terraform.State) error {
 		client := testAccProvider.Meta().(*Client)