Review: Approve Looks good to me.
Tip: TestCase has an assertItems() method (or something like that) which will test that lists contain the same items, even if they're in a different order, which can be useful at times. -- https://code.launchpad.net/~trb143/openlp/cleanup/+merge/160476 Your team OpenLP Core is subscribed to branch lp:openlp. _______________________________________________ Mailing list: https://launchpad.net/~openlp-core Post to : openlp-core@lists.launchpad.net Unsubscribe : https://launchpad.net/~openlp-core More help : https://help.launchpad.net/ListHelp