@pevans This applies some cleanup to the code to help it through the CI pipeline. The tests are now passing for me, so I think this should be sufficient to be able to merge the feature into the main code.
I have removed the lines that write to a temporary file (tmp.xml) because I couldn't see this actually being used at any point and it was leaving unnecessary files in the code.