Update serialize_json.rs

Orange-OpenSource · Dec 1, 2024 · e0da84a · e0da84a
1 parent 1f2ea65
commit e0da84a
Showing 1 changed file with 19 additions and 1 deletion.
diff --git a/packages/hurlfmt/src/format/serialize_json.rs b/packages/hurlfmt/src/format/serialize_json.rs
@@ -50,7 +50,13 @@ impl JValue {
             JValue::Object(key_values) => {
                 let s = key_values
                     .iter()
-                    .map(|(k, v)| format!("\"{}\":{}", k, v.format()))
+                    .map(|(k, v)| {
+                        format!(
+                            "\"{}\":{}",
+                            k.chars().map(format_char).collect::<String>(),
+                            v.format()
+                        )
+                    })
                     .collect::<Vec<String>>()
                     .join(",");
                 format!("{{{s}}}")
@@ -119,6 +125,18 @@ pub mod tests {
             "[1,2,3]"
         );
     }
+    #[test]
+    pub fn test_format_special_characters() {
+        let value = JValue::Object(vec![(
+            "sp\"ecial\\key".to_string(),
+            JValue::String("sp\nvalue\twith\x08control".to_string()),
+        )]);
+
+        assert_eq!(
+            value.format(),
+            r#"{"sp\"ecial\\key":"sp\nvalue\twith\bcontrol"}"#
+        );
+    }
 
     #[test]
     pub fn format_object() {