…into develop-rss

The previous test compared the xml as a string to an expected string. This worked occasionally because the xml writer will sometimes place attributes in a random order.

Also a bunch of formatting changes while I tried to eliminate warnings

…into develop-rss

Change import from ET to ElemTree and change whitespace location in expected string to satisfy Flake8. This doesn't affect the test, but unclogs the test output a little.

Recursive test fails on assertion rather than returning bool to bubble up through the nested generators.

Strip newlines from expected string before parsing in ElementTree