> 2. I disagree, this is the unit test for this method, and this method will be 
> called with
> OS specific paths and I think it should be tested for them. Please comment.

If it's operating system specific or dependent, you're doing it wrong. Please 
fix.
-- 
https://code.launchpad.net/~felipe-q/openlp/fix_httprouter/+merge/194693
Your team OpenLP Core is subscribed to branch lp:openlp.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to