Provide unordered collection diff support #111

Activity

1. bodewig commented on Feb 27, 2018

   

   bodewig

   on Feb 27, 2018

   Member

   More actions

   XMLUnit is not aware of XML-Schema when it calculates differences, although this would be a great enhancement see also #88 .

   If element order is not significant for you, you can tell the difference engine by providing a DifferenceEvaluator. In your example

       .withDifferenceEvaluator(DifferenceEvaluators.chain(
           DifferenceEvaluators.Default,
           DifferenceEvaluators.downgradeDifferencesToSimilar(ComparisonType.CHILD_NODELIST_SEQUENCE)
       ))
   

   should work.
2. izeye

   added a commit that references this issue on Feb 28, 2018

   Try global switch for unordered collection

   ...

   36a469e
3. izeye commented on Feb 28, 2018

   

   izeye

   on Feb 28, 2018

   Author

   More actions

   @bodewig Thanks for the quick feedback!

   Global switch for unordered collection will work for my immediate needs but the suggestion didn't work for me. It might be a fault on my end when applying your suggestion as I'm almost new to XMLUnit.

   This is what I tried with your suggestion: https://github.com/izeye/samples-java-branches/blob/master/src/test/java/learningtest/org/xmlunit/diff/DiffTests.java#L88-L105
4. bodewig commented on Feb 28, 2018

   

   bodewig

   on Feb 28, 2018

   Member

   More actions

   Ah, in your case XMLUnit simply doesn't compare the elements you probably want it to compare.

   The person elements are siblings and you tell XMLUnit to match elements by their names and nested text. The person elements haven't got any nested text themselves, so XMLUnit picks them in order and you end up comparing the wrong persons - and thus see differences. I'd recommend using the Hamcrest matchers or include diff.toString() in your assertThat so that you can better see what kind of differences have been detected.

   In order to make your test pass you need to help XMLUnit matching the correct persons. It looks as if your person element was defined by the nested text of the id child element. In that case something like

   .withNodeMatcher(new DefaultNodeMatcher(conditionalBuilder()
       .whenElementIsNamed("person").thenUse(byXPath("./id", byNameAndText))
       .elseUse(byNameAndText)
       .build())

   should work, I haven't tried it. (just assume there was a static import on ElementSelectors so I can save typing :-))

   See https://github.com/xmlunit/user-guide/wiki/SelectingNodes for all the details.
5. izeye

   added a commit that references this issue on Feb 28, 2018

   Update tests based on @bodewig's advice

   ...

   daedc5e
6. izeye commented on Feb 28, 2018

   

   izeye

   on Feb 28, 2018

   Author

   More actions

   @bodewig Thanks again for the quick response!

   I think I understand your first advice now thanks to your second advice and I updated my tests based on it: https://github.com/izeye/samples-java-branches/blob/master/src/test/java/learningtest/org/xmlunit/diff/DiffTests.java#L89-L123

   Both work like a charm 👍

   I'm happy with the current solution for now but I'd like to keep this issue open for a possible enhancement based on XML schema for ordered and unordered mixed documents diff.
7. bodewig

   added

   enhancement

   help wanted

   on Feb 28, 2018
8. bodewig commented on Feb 28, 2018

   

   bodewig

   on Feb 28, 2018

   Member

   More actions

   Glad it worked.

   Keeping the issue open as an enhancement request is fine with me, I've labeled it accordingly.

   👍React with 👍1Reacted by Johnny Lim
9. bodewig

   added

   core

   on Feb 28, 2018
10. michailangelo commented on Jul 2, 2019

    

    michailangelo

    on Jul 2, 2019

    Last edited by michailangelo

    More actions

    If the strings were like this :

    String xml1 = <persons><person><id>1</id><name><firstName>Johnny</firstName></name></person><person><id>2</id><name><firstName>John</firstName></name></person></persons>
    String xml2 = <persons><person><id>2</id><name><firstName>John</firstName></name></person><person><id>1</id><name><firstName>John</firstName></name></person></persons>

    How would you use multiple selectors to make the comparison?
11. bodewig commented on Jul 2, 2019

    

    bodewig

    on Jul 2, 2019

    Member

    More actions

    @michailangelo I think this is a different question, so please open a new issue.
12. michailangelo commented on Jul 2, 2019

    

    michailangelo

    on Jul 2, 2019

    More actions

    I thought this is relevant to the discussion before but I opened a new issue anyway :)