change log and refactor

tableize.go

@@ -15,7 +15,7 @@ type Input struct {
 	// Optional
 	HintSize      int
 	Substitutions map[string]string
-	StringifyArr  bool
+	StringifyArrays  bool
 }
 
 // Tableize the given map by flattening and normalizing all
@@ -26,7 +26,7 @@ func Tableize(in *Input) map[string]interface{} {
 	}
 
 	ret := make(map[string]interface{}, in.HintSize)
-	visit(ret, in.Value, "", in.Substitutions, in.StringifyArr)
+	visit(ret, in.Value, "", in.Substitutions, in.StringifyArrays)
 	return ret
 }
 
@@ -57,7 +57,7 @@ func visit(ret map[string]interface{}, m map[string]interface{}, prefix string,
 			if stringifyArr {
 				valByteArr, err := json.Marshal(val)
 				if err != nil {
-					log.Printf("[Error] Unable to marshall value `%v` err: %v", val, err)
+					log.Printf("go-tableize: dropping array value %+v that could not be converted to string: %s\n", val, err)
 				} else {
 					ret[key] = string(valByteArr)
 				}

tableize_test.go

@@ -126,7 +126,7 @@ func TestTableizeArray(t *testing.T) {
 	}{
 		{
 			desc:  "array inside big JSON",
-			input: Input{Value: prop, StringifyArr: true},
+			input: Input{Value: prop, StringifyArrays: true},
 			expected: map[string]interface{}{
 				"context_latitude":                    nil,
 				"context_location":                    nil,
@@ -156,7 +156,7 @@ func TestTableizeArray(t *testing.T) {
 			desc: "simple arr StringifyArr=true",
 			input: Input{Value: map[string]interface{}{
 				"colors": []interface{}{"red", "blue"},
-			}, StringifyArr: true,
+			}, StringifyArrays: true,
 			},
 			expected: map[string]interface{}{
 				"colors": "[\"red\",\"blue\"]",
@@ -166,7 +166,7 @@ func TestTableizeArray(t *testing.T) {
 			desc: "simple arr StringifyArr=false",
 			input: Input{Value: map[string]interface{}{
 				"colors": []interface{}{"red", "blue"},
-			}, StringifyArr: false,
+			}, StringifyArrays: false,
 			},
 			expected: map[string]interface{}{
 				"colors": []interface{}{"red", "blue"},