bugfix: skip validating the float number in the body (#469)

* bugfix: skip validating the float number in the body

* update comments

* fix goimports

* fix tests

CHANGELOG.md

@@ -4,6 +4,7 @@ BUG FIXES:
 - Fix a bug when upgrading from previous provider `azapi_resource` resource will set `tags` for resources that don't have `tags` in the configuration.
 - Fix a bug that `azapi_resource` resource cannot handle tags with unknown values.
 - Fix a bug that `null` value can't pass the schema validation.
+- Fix a bug that schema validation fails to validate the float number in the body.
 
 ## v1.13.0
 BREAKING CHANGES:

internal/azure/types/integer_type.go

@@ -2,7 +2,6 @@ package types
 
 import (
 	"fmt"
-	"math"
 
 	"github.com/Azure/terraform-provider-azapi/internal/azure/utils"
 )
@@ -26,16 +25,10 @@ func (t *IntegerType) Validate(body interface{}, path string) []error {
 	}
 	var v int
 	switch input := body.(type) {
-	case float64:
-		if math.Round(input) != input {
-			return []error{utils.ErrorMismatch(path, "integer", fmt.Sprintf("%T", body))}
-		}
-		v = int(input)
-	case float32:
-		if math.Round(float64(input)) != float64(input) {
-			return []error{utils.ErrorMismatch(path, "integer", fmt.Sprintf("%T", body))}
-		}
-		v = int(input)
+	case float64, float32:
+		// TODO: skip validation for now because of the following issue:
+		// the bicep-types-az parses float as integer type and it should be fixed: https://github.com/Azure/bicep-types-az/issues/1404
+		return nil
 	case int64:
 		v = int(input)
 	case int32:

internal/azure/validator_test.go

@@ -187,7 +187,10 @@ func Test_BodyValidation(t *testing.T) {
     }
 }
 `,
-			Error: true,
+			// TODO: change the error to true once the validation is enabled
+			// the validation is disabled for now, because of the following issue:
+			// the bicep-types-az parses float as integer type and it should be fixed: https://github.com/Azure/bicep-types-az/issues/1404
+			Error: false,
 		},
 	}