Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle different attribute and element ordering #1592

Merged

Conversation

electrofelix
Copy link
Contributor

@electrofelix electrofelix commented Sep 9, 2022

Normalise the XML to ensure the attributes for both documents have the
same ordering to prevent excessive noise when differences are detected.
Additionally sort various elements based on attributes that make
ordering irrelevant to allow for simpler comparison using xmlsimple.

Closes: #1583

@coveralls
Copy link

coveralls commented Sep 12, 2022

Coverage Status

Coverage increased (+0.03%) to 79.573% when pulling 97b2590 on electrofelix:sort-xml-before-comparing into 5e0b169 on vagrant-libvirt:main.

@electrofelix electrofelix force-pushed the sort-xml-before-comparing branch 2 times, most recently from c0b52a6 to 09d214c Compare September 22, 2022 15:48
@electrofelix electrofelix marked this pull request as ready for review September 22, 2022 15:53
Normalise the XML to ensure the attributes for both documents have the
same ordering to prevent excessive noise when differences are detected.
Additionally sort various elements based on attributes that make
ordering irrelevant to allow for simpler comparison using xmlsimple.
@electrofelix electrofelix merged commit f111842 into vagrant-libvirt:main Sep 22, 2022
@electrofelix electrofelix deleted the sort-xml-before-comparing branch September 22, 2022 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants