> Why do you have 2 context managers, you only need 1. Also, you don't specify a
> return value, so the method always returns a mock._patch object, so this will
> always evaluate to true.
Right your are. I'll change to only use one context manager, and use the 
TEST_PATH, like some of the other tests.
-- 
https://code.launchpad.net/~tomasgroth/openlp/json-service-format/+merge/171168
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