Fix proposal for parse_xml in case of unique child: (#849)

* Fix proposal for parse_xml in case of unique child:
- Modify single node map insert behaviour -> process_node instead of recurse
- Add tests for unique child situation

* apply fix to latest main

---------

Co-authored-by: Pavlos Rontidis <[email protected]>

changelog.d/849.fix.md

@@ -0,0 +1 @@
+The `parse_xml` function now doesn't add an unnecessary `text` key when processing single nodes.

src/parsing/xml.rs

@@ -171,7 +171,7 @@ fn process_node(node: Node, config: &ParseXmlConfig) -> Value {
 
                             map.insert(
                                 node.tag_name().name().to_string().into(),
-                                Value::Object(recurse(node)),
+                                process_node(node, config),
                             );
 
                             Value::Object(map)

src/stdlib/parse_xml.rs

@@ -467,6 +467,18 @@ mod tests {
             )),
             tdef: type_def(),
         }
+
+        if_no_sibling {
+            args: func_args![ value: "<root><a>test</a></root>"],
+            want: Ok(value!({ "root": { "a": "test" } })),
+            tdef: type_def(),
+        }
+
+        if_no_sibling2 {
+            args: func_args![ value: "<root><a><a1>test</a1></a><b>test2</b></root>"],
+            want: Ok(value!({ "root": { "a": { "a1": "test" }, "b" : "test2" } })),
+            tdef: type_def(),
+        }
     ];
 
     #[test]