fix double unquoting in tests

Author: James Cor

enginetest/queries/json_scripts.go

@@ -627,7 +627,7 @@ var JsonScripts = []ScriptTest{
 		SetUpScript: []string{
 			"create table t (pk int primary key, col1 JSON, col2 JSON);",
 			`insert into t values (1, JSON_OBJECT('key1', 1, 'key2', '"abc"'), JSON_ARRAY(3,10,5,17,"z"));`,
-			`insert into t values (2, JSON_OBJECT('key1', 100, 'key2', '"ghi"'), JSON_ARRAY(3,10,5,17,JSON_ARRAY(22,"y",66)));`,
+			`insert into t values (2, JSON_OBJECT('key1', 100, 'key2', 'ghi'), JSON_ARRAY(3,10,5,17,JSON_ARRAY(22,"y",66)));`,
 			`CREATE TABLE t2 (i INT PRIMARY KEY, j JSON);`,
 			`INSERT INTO t2 VALUES (0, '{"a": "123", "outer": {"inner": 456}}');`,
 		},
@@ -636,16 +636,23 @@ var JsonScripts = []ScriptTest{
 				Query:    `select col1->'$.key1' from t;`,
 				Expected: []sql.Row{{types.MustJSON("1")}, {types.MustJSON("100")}},
 			},
+			{
+				Query: `select col1->'$.key2' from t;`,
+				Expected: []sql.Row{
+					{types.JSONDocument{Val: "\"abc\""}},
+					{types.JSONDocument{Val: "ghi"}},
+				},
+			},
 			{
 				Query:    `select col1->>'$.key2' from t;`,
-				Expected: []sql.Row{{"abc"}, {"ghi"}},
+				Expected: []sql.Row{{"\"abc\""}, {"ghi"}},
 			},
 			{
 				Query:    `select pk, col1 from t where col1->'$.key1' = 1;`,
 				Expected: []sql.Row{{1, types.MustJSON(`{"key1":1, "key2":"\"abc\""}`)}},
 			},
 			{
-				Query:    `select pk, col1 from t where col1->>'$.key2' = 'abc';`,
+				Query:    `select pk, col1 from t where col1->>'$.key2' = '"abc"';`,
 				Expected: []sql.Row{{1, types.MustJSON(`{"key1":1, "key2":"\"abc\""}`)}},
 			},
 			{

enginetest/queries/queries.go

@@ -5412,6 +5412,12 @@ SELECT * FROM cte WHERE  d = 2;`,
 			},
 		},
 	},
+	{
+		Query: `SELECT COALESCE(CAST('{"a": "one \\n two"}' as json), '');`,
+		Expected: []sql.Row{
+			{"{\"a\": \"one \\n two\"}"},
+		},
+	},
 	{
 		Query: "SELECT concat(s, i) FROM mytable",
 		Expected: []sql.Row{

sql/types/strings.go

@@ -415,11 +415,6 @@ func ConvertToBytes(ctx context.Context, v interface{}, t sql.StringType, dest [
 		if err != nil {
 			return nil, err
 		}
-		// TODO: why do we do this?
-		//val, err = strings.UnquoteBytes(val)
-		if err != nil {
-			return nil, err
-		}
 		start = 0
 	case sql.Wrapper[string]:
 		unwrapped, err := s.Unwrap(ctx)