Code review comment for lp:~thumper/juju-core/local-provider-skeleton

Revision history for this message
Tim Penhey (thumper) wrote :

Please take a look.

https://codereview.appspot.com/10679045/diff/2005/environs/local/local_test.go
File environs/local/local_test.go (right):

https://codereview.appspot.com/10679045/diff/2005/environs/local/local_test.go#newcode29
environs/local/local_test.go:29: c.Assert(error, IsNil)
On 2013/06/28 06:05:02, wallyworld wrote:
> Please assert that the returned provider really is a local provider

Done.

https://codereview.appspot.com/10679045/

« Back to merge proposal