openlp-core team mailing list archive

[Raoul Snyman <raoul@xxxxxxxxxxx>]

Review: Needs Fixing

Could you provide some tests for your "private" functions too please? If we only rely on the tests of one function, but we're using other functions within the function which are not tested. Without tests around those functions we cannot be sure our overall tests can really be trusted.
-- 
https://code.launchpad.net/~patrick-zakweb/openlp/duplicate-removal-review/+merge/149430
Your team OpenLP Core is subscribed to branch lp:openlp.