Review: Needs Fixing

180     +TEST_PATH = os.path.abspath(os.path.join(os.path.dirname(__file__), 
u'../../../resources/easyworshipsongs'))

You need to use os.path.join with each part of that path separately:

TEST_PATH = os.path.abspath(os.path.join(os.path.dirname(__file__), u'..', 
u'..', u'..', u'resources', u'easyworshipsongs'))

This ensures that these paths work in all compatible operating systems.
-- 
https://code.launchpad.net/~phill-ridout/openlp/1114457/+merge/172162
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