Request for adding ignore order feature for verifyXml

[wuh3]

Hello developers,

Due to some code stability requirements, I may need the verifyXml(String, Option) in Approvals.java to pass if the fields in the XML string are shuffled. I wonder if you could add a new feature that can satisfy this. By stability requirement, I mean specifically when I apply the tool NonDex, which will shuffle implementation-dependent operations. The NonDex tool will shuffle the elements inside of the xml string which will cause the verifyXml method to fail.

I saw some similar issues were raised previously, and the closest one I found was https://github.com/approvals/ApprovalTests.Java/issues/301. However, I think the proposed solution Option.withComparator() may not fit my demand that well since I am testing an xml string instead of a File. It would be great if you could help!

[LarsEckart]

hello,

I'm not sure I can follow. Any chance you can add an example for your ask?

do you want to compare
<xml attributeA=B attributeC=D> with <xml attributeC=D attributeA=B> and assert that they are still the same?

[wuh3]

Hello @LarsEckart ,

Yes, this is exactly what I meant. I know this is just a trivial order issue, and regular runs will not cause error. But, under certain circumstances with very high stability requirements, such order issue will cause trouble. Thank you for your understanding.

[LarsEckart]

hello,

been a while but thank you for your patience. We looked at the issue and think we can do it. We already have a similar solution for JSON. We're curious though, what XML library do you use, jaxb?