chore: add test for getPrimitiveValue util

errors.go

@@ -20,12 +20,12 @@ var (
 
 // ErrUnsupportedType describes an unsupported field type
 type ErrUnsupportedType struct {
-	typ reflect.Type
+	typ reflect.Kind
 }
 
 // Error returns the UnsupportedType error text
 func (e *ErrUnsupportedType) Error() string {
-	return fmt.Sprintf("mold: unsupported field type: %s", e.typ.Kind())
+	return fmt.Sprintf("mold: unsupported field type: %s", e.typ)
 }
 
 // ErrUndefinedTag defines a tag that does not exist

modifiers/multi.go

@@ -132,7 +132,7 @@ func setValue(ctx context.Context, fl mold.FieldLevel) error {
 		// 2. Attempting to set its underlying value
 		// Try to convert the parameter string to the appropriate primitive type
 		// that the pointer references (e.g., *string, *int, *bool)
-		value, err := mold.GetPrimitiveValue(fl.Field().Type().Elem(), fl.Param())
+		value, err := mold.GetPrimitiveValue(fl.Field().Type().Elem().Kind(), fl.Param())
 		if err != nil {
 			// If ErrUnsupportedType: leave as zero value
 			if _, isUnsupportedType := err.(*mold.ErrUnsupportedType); isUnsupportedType {

util.go

@@ -38,8 +38,8 @@ func HasValue(field reflect.Value) bool {
 	}
 }
 
-func GetPrimitiveValue(typ reflect.Type, value string) (reflect.Value, error) {
-	switch typ.Kind() {
+func GetPrimitiveValue(typ reflect.Kind, value string) (reflect.Value, error) {
+	switch typ {
 
 	case reflect.String:
 		return reflect.ValueOf(value), nil

util_test.go

@@ -0,0 +1,198 @@
+package mold
+
+import (
+	"math"
+	"reflect"
+	"testing"
+
+	. "github.com/go-playground/assert/v2"
+)
+
+func TestGetPrimitiveValue(t *testing.T) {
+	tests := []struct {
+		name        string
+		typ         reflect.Kind
+		value       string
+		expected    reflect.Value
+		expectError bool
+	}{
+		{
+			name:     "string",
+			typ:      reflect.String,
+			value:    "test",
+			expected: reflect.ValueOf("test"),
+		},
+		{
+			name:     "int",
+			typ:      reflect.Int,
+			value:    "123",
+			expected: reflect.ValueOf(123),
+		},
+		{
+			name:     "int8",
+			typ:      reflect.Int8,
+			value:    "123",
+			expected: reflect.ValueOf(int8(123)),
+		},
+		{
+			name:     "int16",
+			typ:      reflect.Int16,
+			value:    "123",
+			expected: reflect.ValueOf(int16(123)),
+		},
+		{
+			name:     "bool",
+			typ:      reflect.Bool,
+			value:    "true",
+			expected: reflect.ValueOf(true),
+		},
+		{
+			name:        "error while parsing int",
+			typ:         reflect.Int,
+			value:       "abc",
+			expectError: true,
+		},
+		{
+			name:        "error while parsing int8",
+			typ:         reflect.Int8,
+			value:       "abc",
+			expectError: true,
+		},
+		{
+			name:        "error while parsing int16",
+			typ:         reflect.Int16,
+			value:       "abc",
+			expectError: true,
+		},
+		{
+			name:        "error while parsing int32",
+			typ:         reflect.Int32,
+			value:       "abc",
+			expectError: true,
+		},
+		{
+			name:        "error while parsing int64",
+			typ:         reflect.Int64,
+			value:       "abc",
+			expectError: true,
+		},
+		{
+			name:        "error while parsing uint",
+			typ:         reflect.Uint,
+			value:       "abc",
+			expectError: true,
+		},
+		{
+			name:        "error while parsing uint8",
+			typ:         reflect.Uint8,
+			value:       "abc",
+			expectError: true,
+		},
+		{
+			name:        "error while parsing uint16",
+			typ:         reflect.Uint16,
+			value:       "12.34",
+			expectError: true,
+		},
+		{
+			name:        "error while parsing uint32",
+			typ:         reflect.Uint32,
+			value:       "12.34",
+			expectError: true,
+		},
+		{
+			name:        "error while parsing uint64",
+			typ:         reflect.Uint64,
+			value:       "12.34",
+			expectError: true,
+		},
+		{
+			name:        "error while parsing float32",
+			typ:         reflect.Float32,
+			value:       "abc",
+			expectError: true,
+		},
+		{
+			name:        "error while parsing bool",
+			typ:         reflect.Bool,
+			value:       "invalid bool",
+			expectError: true,
+		},
+		{
+			name:        "unsupported type",
+			typ:         reflect.Struct,
+			value:       "abc",
+			expectError: true,
+		},
+		{
+			name:     "uint",
+			typ:      reflect.Uint,
+			value:    "123",
+			expected: reflect.ValueOf(uint(123)),
+		},
+		{
+			name:     "uint8",
+			typ:      reflect.Uint8,
+			value:    "123",
+			expected: reflect.ValueOf(uint8(123)),
+		},
+		{
+			name:     "uint16",
+			typ:      reflect.Uint16,
+			value:    "123",
+			expected: reflect.ValueOf(uint16(123)),
+		},
+		{
+			name:     "uint32",
+			typ:      reflect.Uint32,
+			value:    "123",
+			expected: reflect.ValueOf(uint32(123)),
+		},
+		{
+			name:     "uint64",
+			typ:      reflect.Uint64,
+			value:    "123",
+			expected: reflect.ValueOf(uint64(123)),
+		},
+		{
+			name:     "float32",
+			typ:      reflect.Float32,
+			value:    "123.45",
+			expected: reflect.ValueOf(float32(123.45)),
+		},
+		{
+			name:     "float64",
+			typ:      reflect.Float64,
+			value:    "123.45",
+			expected: reflect.ValueOf(float64(123.45)),
+		},
+	}
+
+	for _, tc := range tests {
+		t.Run(tc.name, func(t *testing.T) {
+			actual, err := GetPrimitiveValue(tc.typ, tc.value)
+			if tc.expectError {
+				NotEqual(t, nil, err)
+			} else {
+				Equal(t, nil, err)
+				switch tc.typ {
+				case reflect.String:
+					Equal(t, tc.expected.String(), actual.String())
+				case reflect.Int, reflect.Int8, reflect.Int16, reflect.Int32, reflect.Int64:
+					Equal(t, tc.expected.Int(), actual.Int())
+				case reflect.Uint, reflect.Uint8, reflect.Uint16, reflect.Uint32, reflect.Uint64:
+					Equal(t, tc.expected.Uint(), actual.Uint())
+				case reflect.Float32, reflect.Float64:
+					// could not assert equal float because of precision issues
+					// so we just check in 4 decimal places
+					decimalMask := math.Pow(10, 4)
+					Equal(t, math.Round(tc.expected.Float()*decimalMask), math.Round(actual.Float()*decimalMask))
+				case reflect.Bool:
+					Equal(t, tc.expected.Bool(), actual.Bool())
+				default:
+					Equal(t, tc.expected.Interface(), actual.Interface())
+				}
+			}
+		})
+	}
+}