Regression XML parsing null with keepStrings

[babita-verma]

null used to be {"t":"null"} but now its returning {"t":null} with useStrings

I expected everything to String as it was in the previous version(20250107

[stleary]

@babita-verma Sorry for not getting to this sooner. A complete code fragment will be helpful; otherwise, I will try to work with your initial post.

[babita-verma]

We have this test which fails after the version upgrade. What you think think should happen in this case? when we use useStrings as true ?

void testValueConversion() {
// This test case does not cover functionality in XMLUtil.
// Instead, it covers the ASSUMED configuration of the conversion method.

final var convertedJson = XML.toJSONObject("<t>null</t>", true);
// Parse expected value first to ensure semantic equality, not format equality.
final var expectedJson = new JSONObject("{\"t\": \"null\"}");

assertThat(convertedJson.toString()).isEqualTo(expectedJson.toString());

}

[stleary]

@babita-verma Thanks for catching this. I think the previous behavior was per design, the current behavior is a regression. Looks like we forgot to check for null, when adding support for numberAsString, and booleanAsString. This will be fixed in a future commit. FYI: @marilynel

[babita-verma]

Thanks for confirming. The code confused me whether it was intentional to only stringify number and Boolean. Wait for the fix