Code review comment for lp:~tomasgroth/openlp/bug719514

Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

Some minor fixes to your tests, but otherwise looking good. Technically you also want to test the case where the files are the same (it should be simple to do if you just copy the current test and modify it where necessary)

review: Needs Fixing

« Back to merge proposal