Make dst slice consistent with the empty src slice. (#30)

Author: mennanov

copy.go

@@ -361,6 +361,10 @@ func structToMap(filter FieldFilter, src interface{}, dst map[string]interface{}
 				continue
 			}
 			var newValue []map[string]interface{}
+			if srcField.Kind() == reflect.Slice && !srcField.IsNil() {
+				// If the source slice is not nil then the dst should not be nil either even if the src slice is empty.
+				newValue = make([]map[string]interface{}, 0, srcField.Len())
+			}
 			existingValue, ok := dst[dstName]
 			if ok {
 				v := reflect.ValueOf(existingValue)

copy_test.go

@@ -1,6 +1,7 @@
 package fieldmask_utils_test
 
 import (
+	"encoding/json"
 	"fmt"
 	"reflect"
 	"testing"
@@ -1990,7 +1991,7 @@ func TestStructToStruct_WithMultiTagComma(t *testing.T) {
 	}, dst)
 }
 
-func TestStructToMap_DiffentTypeWithSameDstKey(t *testing.T) {
+func TestStructToMap_DifferentTypeWithSameDstKey(t *testing.T) {
 	type BB struct {
 		Field int
 	}
@@ -2015,6 +2016,40 @@ func TestStructToMap_DiffentTypeWithSameDstKey(t *testing.T) {
 	require.Error(t, err)
 }
 
+func TestStructToMap_EmptySrcSlice_JsonEncode(t *testing.T) {
+	type A struct{}
+	type B struct {
+		As []*A
+	}
+
+	src := &B{[]*A{}}
+	dst := make(map[string]interface{})
+
+	mask := fieldmask_utils.MaskFromString("As")
+	err := fieldmask_utils.StructToMap(mask, src, dst)
+	require.NoError(t, err)
+
+	jsonStr, _ := json.Marshal(dst)
+	assert.Equal(t, string(jsonStr), "{\"As\":[]}")
+}
+
+func TestStructToMap_NilSrcSlice_JsonEncode(t *testing.T) {
+	type A struct{}
+	type B struct {
+		As []*A
+	}
+
+	src := &B{}
+	dst := make(map[string]interface{})
+
+	mask := fieldmask_utils.MaskFromString("As")
+	err := fieldmask_utils.StructToMap(mask, src, dst)
+	require.NoError(t, err)
+
+	jsonStr, _ := json.Marshal(dst)
+	assert.Equal(t, string(jsonStr), "{\"As\":null}")
+}
+
 func TestStructToStruct_CopySlice_WithDiffentAddr_WithDifferentFieldName(t *testing.T) {
 	type A struct {
 		Field1 []int