> My only comment, is that I don't like the "assert_desktop_file_contents"
> method. Instead of generating a long regular expression for the desktop file
> and check that there's a match for every key value pair in the expected
> contents dictionary, I'd parse the desktop file as a dictionary and run an
> assertion on both the dictionaries:
> My only comment, is that I don't like the "assert_ desktop_ file_contents"
> method. Instead of generating a long regular expression for the desktop file
> and check that there's a match for every key value pair in the expected
> contents dictionary, I'd parse the desktop file as a dictionary and run an
> assertion on both the dictionaries:
Right you are! It looks much better now.