Last week I added PatchClientFacadeCall to api/export_test.go which does
the same thing: https://github.com/juju/juju/blob/master/api/export_test.go
It is used in several tests, e.g. TestShareEnvironmentExistingUser
On 11/09/14 22:30, Matthew Williams wrote:
Thanks Andrew,
I was almost about to reimplement this myself - making use of it right now
Thanks
Matty
On Thu, Sep 11, 2014 at 5:57 AM, Andrew Wilkins
<[email protected] <mailto:[email protected]>>
wrote:
Hi folks,
I'd like to bring a small, recent addition to everyone's attention:
https://github.com/juju/juju/blob/master/api/base/testing/patch.go
PatchFacadeCaller can be used (and is used now in several places)
to avoid calling through to a real API server in api tests.
Ideally we'd just provide mock FacadeCallers to begin with, but
this alleviates some of the pain between now and when we do that
across all the existing tests.
If you're writing tests for api clients, please consider using
PatchFacadeCaller or a mock FacadeCaller, so we can improve test
coverage and avoid slow tests.
Cheers,
Andrew
--
Juju-dev mailing list
[email protected] <mailto:[email protected]>
Modify settings or unsubscribe at:
https://lists.ubuntu.com/mailman/listinfo/juju-dev
--
Juju-dev mailing list
[email protected]
Modify settings or unsubscribe at:
https://lists.ubuntu.com/mailman/listinfo/juju-dev
