openlp-core team mailing list archive

[Tomas Groth <tomasgroth@xxxxxxxx>]

> 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.