https://codereview.appspot.com/13660047/diff/7001/environs/sshstorage/storage_test.go File environs/sshstorage/storage_test.go (right):
https://codereview.appspot.com/13660047/diff/7001/environs/sshstorage/storage_test.go#newcode4 environs/sshstorage/storage_test.go:4: package sshstorage Normally the tests would be in sshstorage_test package.
Is there any reason why this is a whitebox test?
https://codereview.appspot.com/13660047/diff/7001/environs/sshstorage/storage_test.go#newcode74 environs/sshstorage/storage_test.go:74: func (s *storageSuite) makeStorage(c *gc.C) (storage *SSHStorage, storageDir string) { Why name the return parameters here? You aren't really gaining much. Is it just to provide more assistance to the reader?
https://codereview.appspot.com/13660047/
« Back to merge proposal
https:/ /codereview. appspot. com/13660047/ diff/7001/ environs/ sshstorage/ storage_ test.go sshstorage/ storage_ test.go (right):
File environs/
https:/ /codereview. appspot. com/13660047/ diff/7001/ environs/ sshstorage/ storage_ test.go# newcode4 sshstorage/ storage_ test.go: 4: package sshstorage
environs/
Normally the tests would be in sshstorage_test package.
Is there any reason why this is a whitebox test?
https:/ /codereview. appspot. com/13660047/ diff/7001/ environs/ sshstorage/ storage_ test.go# newcode74 sshstorage/ storage_ test.go: 74: func (s *storageSuite)
environs/
makeStorage(c *gc.C) (storage *SSHStorage, storageDir string) {
Why name the return parameters here? You aren't really gaining much. Is
it just to provide more assistance to the reader?
https:/ /codereview. appspot. com/13660047/