The tests look good to me on a first glance :+1:

I never expected `resolve` to be mocked. How you laid out the tests was exactly 
how I imagined it.

One last thing: it would be nice to add a test that shows that globally 
installed modules will be taken into account too. Not sure how to best test 
this, but maybe you have an idea?

[ Full content available at: https://github.com/apache/cordova-fetch/pull/44 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to