feat. mapping Optimized error stack

core/mapping/fieldoptions.go

@@ -1,7 +1,5 @@
 package mapping
 
-import "fmt"
-
 const notSymbol = '!'
 
 type (
@@ -76,21 +74,21 @@ func (o *fieldOptions) toOptionsWithContext(key string, m Valuer, fullName strin
 		} else if dep[0] == notSymbol {
 			dep = dep[1:]
 			if len(dep) == 0 {
-				return nil, fmt.Errorf("wrong optional value for %q in %q", key, fullName)
+				return nil, newError("wrong optional value for %q in %q", key, fullName)
 			}
 
 			_, baseOn := m.Value(dep)
 			_, selfOn := m.Value(key)
 			if baseOn == selfOn {
-				return nil, fmt.Errorf("set value for either %q or %q in %q", dep, key, fullName)
+				return nil, newError("set value for either %q or %q in %q", dep, key, fullName)
 			}
 
 			optional = baseOn
 		} else {
 			_, baseOn := m.Value(dep)
 			_, selfOn := m.Value(key)
 			if baseOn != selfOn {
-				return nil, fmt.Errorf("values for %q and %q should be both provided or both not in %q",
+				return nil, newError("values for %q and %q should be both provided or both not in %q",
 					dep, key, fullName)
 			}
 

core/mapping/jsonunmarshaler_test.go

@@ -856,7 +856,7 @@ func TestUnmarshalBytesError(t *testing.T) {
 	}
 
 	err := UnmarshalJsonBytes([]byte(payload), &v)
-	assert.Equal(t, errTypeMismatch, err)
+	assert.Equal(t, ErrTypeMismatch, err)
 }
 
 func TestUnmarshalReaderError(t *testing.T) {
@@ -866,7 +866,7 @@ func TestUnmarshalReaderError(t *testing.T) {
 		Any string
 	}
 
-	assert.Equal(t, errTypeMismatch, UnmarshalJsonReader(reader, &v))
+	assert.Equal(t, ErrTypeMismatch, UnmarshalJsonReader(reader, &v))
 }
 
 func TestUnmarshalMap(t *testing.T) {

core/mapping/marshaler.go

@@ -116,11 +116,11 @@ func validateOptional(field reflect.StructField, value reflect.Value) error {
 	switch field.Type.Kind() {
 	case reflect.Ptr:
 		if value.IsNil() {
-			return fmt.Errorf("field %q is nil", field.Name)
+			return newError("field %q is nil", field.Name)
 		}
 	case reflect.Array, reflect.Slice, reflect.Map:
 		if value.IsNil() || value.Len() == 0 {
-			return fmt.Errorf("field %q is empty", field.Name)
+			return newError("field %q is empty", field.Name)
 		}
 	}
 
@@ -137,7 +137,7 @@ func validateOptions(value reflect.Value, opt *fieldOptions) error {
 		}
 	}
 	if !found {
-		return fmt.Errorf("field %q not in options", val)
+		return newError("field %q not in options", val)
 	}
 
 	return nil
@@ -171,15 +171,15 @@ func validateRange(value reflect.Value, opt *fieldOptions) error {
 	case float64:
 		val = v
 	default:
-		return fmt.Errorf("unknown support type for range %q", value.Type().String())
+		return newError("unknown support type for range %q", value.Type().String())
 	}
 
 	// validates [left, right], [left, right), (left, right], (left, right)
 	if val < opt.Range.left ||
 		(!opt.Range.leftInclude && val == opt.Range.left) ||
 		val > opt.Range.right ||
 		(!opt.Range.rightInclude && val == opt.Range.right) {
-		return fmt.Errorf("%v out of range", value.Interface())
+		return newError("%v out of range", value.Interface())
 	}
 
 	return nil

core/mapping/marshaler_test.go

@@ -277,7 +277,7 @@ func TestMarshal_RangeIllegal(t *testing.T) {
 
 	for _, test := range tests {
 		_, err := Marshal(test)
-		assert.Equal(t, err, errNumberRange)
+		assert.Equal(t, err, ErrNumberRange)
 	}
 }
 
@@ -302,7 +302,7 @@ func TestMarshal_RangeLeftEqualsToRight(t *testing.T) {
 			}{
 				Int: 2,
 			},
-			err: errNumberRange,
+			err: ErrNumberRange,
 		},
 		{
 			name: "left exclusive, right inclusive",
@@ -311,7 +311,7 @@ func TestMarshal_RangeLeftEqualsToRight(t *testing.T) {
 			}{
 				Int: 2,
 			},
-			err: errNumberRange,
+			err: ErrNumberRange,
 		},
 		{
 			name: "left exclusive, right exclusive",
@@ -320,7 +320,7 @@ func TestMarshal_RangeLeftEqualsToRight(t *testing.T) {
 			}{
 				Int: 2,
 			},
-			err: errNumberRange,
+			err: ErrNumberRange,
 		},
 	}