consolidate more tests

Author: taowen

jsoniter_optional_test.go

@@ -1,75 +1,29 @@
-package jsoniter
+package misc_tests
 
 import (
 	"encoding/json"
 	"github.com/stretchr/testify/require"
 	"strings"
 	"testing"
+	"github.com/json-iterator/go"
 )
 
-func Test_json_RawMessage(t *testing.T) {
-	should := require.New(t)
-	var data json.RawMessage
-	should.Nil(Unmarshal([]byte(`[1,2,3]`), &data))
-	should.Equal(`[1,2,3]`, string(data))
-	str, err := MarshalToString(data)
-	should.Nil(err)
-	should.Equal(`[1,2,3]`, str)
-}
-
 func Test_jsoniter_RawMessage(t *testing.T) {
 	should := require.New(t)
-	var data RawMessage
-	should.Nil(Unmarshal([]byte(`[1,2,3]`), &data))
+	var data jsoniter.RawMessage
+	should.Nil(jsoniter.Unmarshal([]byte(`[1,2,3]`), &data))
 	should.Equal(`[1,2,3]`, string(data))
-	str, err := MarshalToString(data)
+	str, err := jsoniter.MarshalToString(data)
 	should.Nil(err)
 	should.Equal(`[1,2,3]`, str)
 }
 
-func Test_json_RawMessage_in_struct(t *testing.T) {
-	type TestObject struct {
-		Field1 string
-		Field2 json.RawMessage
-	}
-	should := require.New(t)
-	var data TestObject
-	should.Nil(Unmarshal([]byte(`{"field1": "hello", "field2": [1,2,3]}`), &data))
-	should.Equal(` [1,2,3]`, string(data.Field2))
-	should.Equal(`hello`, data.Field1)
-}
-
-func Test_decode_map_of_raw_message(t *testing.T) {
-	should := require.New(t)
-	type RawMap map[string]*json.RawMessage
-	b := []byte("{\"test\":[{\"key\":\"value\"}]}")
-	var rawMap RawMap
-	should.Nil(Unmarshal(b, &rawMap))
-	should.Equal(`[{"key":"value"}]`, string(*rawMap["test"]))
-	type Inner struct {
-		Key string `json:"key"`
-	}
-	var inner []Inner
-	Unmarshal(*rawMap["test"], &inner)
-	should.Equal("value", inner[0].Key)
-}
-
-func Test_encode_map_of_raw_message(t *testing.T) {
-	should := require.New(t)
-	type RawMap map[string]*json.RawMessage
-	value := json.RawMessage("[]")
-	rawMap := RawMap{"hello": &value}
-	output, err := MarshalToString(rawMap)
-	should.Nil(err)
-	should.Equal(`{"hello":[]}`, output)
-}
-
 func Test_encode_map_of_jsoniter_raw_message(t *testing.T) {
 	should := require.New(t)
-	type RawMap map[string]*RawMessage
-	value := RawMessage("[]")
+	type RawMap map[string]*jsoniter.RawMessage
+	value := jsoniter.RawMessage("[]")
 	rawMap := RawMap{"hello": &value}
-	output, err := MarshalToString(rawMap)
+	output, err := jsoniter.MarshalToString(rawMap)
 	should.Nil(err)
 	should.Equal(`{"hello":[]}`, output)
 }
@@ -82,8 +36,8 @@ func Test_marshal_invalid_json_raw_message(t *testing.T) {
 
 	a := A{}
 	should := require.New(t)
-	should.Nil(ConfigCompatibleWithStandardLibrary.Unmarshal(message, &a))
-	aout, aouterr := ConfigCompatibleWithStandardLibrary.Marshal(&a)
+	should.Nil(jsoniter.ConfigCompatibleWithStandardLibrary.Unmarshal(message, &a))
+	aout, aouterr := jsoniter.ConfigCompatibleWithStandardLibrary.Marshal(&a)
 	should.Equal(`{"raw":null}`, string(aout))
 	should.Nil(aouterr)
 }
@@ -99,14 +53,14 @@ func Test_raw_message_memory_not_copied_issue(t *testing.T) {
 		BiddingMax  *float32    `json:"bidding_max"`
 		BiddingMin  *float32    `json:"bidding_min"`
 		BiddingType *string     `json:"bidding_type"`
-		Freq        *RawMessage `json:"freq"`
-		Targeting   *RawMessage `json:"targeting"`
-		Url         *RawMessage `json:"url"`
+		Freq        *jsoniter.RawMessage `json:"freq"`
+		Targeting   *jsoniter.RawMessage `json:"targeting"`
+		Url         *jsoniter.RawMessage `json:"url"`
 		Speed       *int        `json:"speed" db:"speed"`
 	}
 
 	obj := &IteratorObject{}
-	decoder := NewDecoder(strings.NewReader(jsonStream))
+	decoder := jsoniter.NewDecoder(strings.NewReader(jsonStream))
 	err := decoder.Decode(obj)
 	should := require.New(t)
 	should.Nil(err)

jsoniter_raw_message_test.go misc_tests/jsoniter_raw_message_test.gojsoniter_raw_message_test.go renamed to misc_tests/jsoniter_raw_message_test.go

@@ -1,8 +1,14 @@
 package test
 
-import "math/big"
+import (
+	"math/big"
+	"encoding/json"
+)
 
 func init() {
+	var pRawMessage = func(val json.RawMessage) *json.RawMessage {
+		return &val
+	}
 	nilMap := map[string]string(nil)
 	marshalCases = append(marshalCases,
 		map[string]interface{}{"abc": 1},
@@ -20,13 +26,17 @@ func init() {
 		},
 		nilMap,
 		&nilMap,
+		map[string]*json.RawMessage{"hello":pRawMessage(json.RawMessage("[]"))},
 	)
 	unmarshalCases = append(unmarshalCases, unmarshalCase{
 		ptr: (*map[string]string)(nil),
 		input: `{"k\"ey": "val"}`,
 	}, unmarshalCase{
 		ptr: (*map[string]string)(nil),
 		input: `null`,
+	}, unmarshalCase{
+		ptr: (*map[string]*json.RawMessage)(nil),
+		input: "{\"test\":[{\"key\":\"value\"}]}",
 	})
 }
 

value_tests/map_test.go

@@ -4,6 +4,9 @@ func init() {
 	var pEFace = func(val interface{}) *interface{} {
 		return &val
 	}
+	var pInt = func(val int) *int {
+		return &val
+	}
 	unmarshalCases = append(unmarshalCases, unmarshalCase{
 		ptr: (**interface{})(nil),
 		input: `"hello"`,
@@ -16,5 +19,7 @@ func init() {
 	})
 	marshalCases = append(marshalCases,
 		pEFace("hello"),
+		(*int)(nil),
+		pInt(100),
 	)
-}
+}

value_tests/ptr_test.go

@@ -6,4 +6,8 @@ func init() {
 	marshalCases = append(marshalCases,
 		json.RawMessage("{}"),
 	)
+	unmarshalCases = append(unmarshalCases, unmarshalCase{
+		ptr: (*json.RawMessage)(nil),
+		input: `[1,2,3]`,
+	})
 }

value_tests/raw_message_test.go

@@ -7,6 +7,9 @@ import (
 )
 
 func init() {
+	var pString = func(val string) *string {
+		return &val
+	}
 	unmarshalCases = append(unmarshalCases, unmarshalCase{
 		ptr: (*struct {
 			Field interface{}
@@ -34,6 +37,18 @@ func init() {
 			Field1 string
 		})(nil),
 		input: `{"\u0046ield1":"hello"}`,
+	}, unmarshalCase{
+		ptr: (*struct {
+			Field1 *string
+			Field2 *string
+		})(nil),
+		input: `{"field1": null, "field2": "world"}`,
+	}, unmarshalCase{
+		ptr: (*struct {
+			Field1 string
+			Field2 json.RawMessage
+		})(nil),
+		input: `{"field1": "hello", "field2":[1,2,3]}`,
 	})
 	marshalCases = append(marshalCases,
 		struct {
@@ -116,6 +131,10 @@ func init() {
 			MaxAge: 20,
 		},
 		structOrder{},
+		struct {
+			Field1 *string
+			Field2 *string
+		}{Field2: pString("world")},
 	)
 }