fix(mapstr): cast inner maps to M on Clone (#283)

Author: mauri870

mapstr/mapstr.go

@@ -169,7 +169,7 @@ func cloneMap(dst, src M) {
 			dst[k] = d
 			cloneMap(d, v)
 		case map[string]interface{}:
-			d := make(map[string]interface{}, len(v))
+			d := make(M, len(v))
 			dst[k] = d
 			cloneMap(d, v)
 		case []M:
@@ -181,9 +181,9 @@ func cloneMap(dst, src M) {
 			}
 			dst[k] = a
 		case []map[string]interface{}:
-			a := make([]map[string]interface{}, 0, len(v))
+			a := make([]M, 0, len(v))
 			for _, m := range v {
-				d := make(map[string]interface{}, len(m))
+				d := make(M, len(m))
 				cloneMap(d, m)
 				a = append(a, d)
 			}

mapstr/mapstr_test.go

@@ -349,6 +349,8 @@ func TestClone(t *testing.T) {
 			"c32": 2,
 		},
 		"c4": []M{{"c41": 1}},
+		"c5": map[string]interface{}{"c51": 1},
+		"c6": []map[string]interface{}{{"c61": 1}},
 	}
 
 	// Clone the original mapstr and then increment every value in it. Ensures the test will fail if
@@ -366,6 +368,8 @@ func TestClone(t *testing.T) {
 				"c32": 2,
 			},
 			"c4": []M{{"c41": 1}},
+			"c5": M{"c51": 1},
+			"c6": []M{{"c61": 1}},
 		},
 		cloned,
 	)