LGTM with the AttemptStrategy stuff dropped.
https://codereview.appspot.com/7346052/diff/12001/environs/local/storage_test.go File environs/local/storage_test.go (right):
https://codereview.appspot.com/7346052/diff/12001/environs/local/storage_test.go#newcode18 environs/local/storage_test.go:18: var noRetry = trivial.AttemptStrategy{} There's no need for an AttemptStrategy here.
https://codereview.appspot.com/7346052/diff/12001/environs/local/storage_test.go#newcode22 environs/local/storage_test.go:22: // be removed again. These tests should be written properly, and should not be earmarked for deletion. Do we really write these sorts of tests against an *Environ* in the other cases? If so, what have we been smoking?
https://codereview.appspot.com/7346052/diff/12001/environs/local/storage_test.go#newcode85 environs/local/storage_test.go:85: func checkFileDoesNotExist(c *C, storage environs.StorageReader, name string, attempt trivial.AttemptStrategy) { AttemptStrategy not needed.
https://codereview.appspot.com/7346052/diff/12001/environs/local/storage_test.go#newcode99 environs/local/storage_test.go:99: func checkFileHasContents(c *C, storage environs.StorageReader, name string, contents []byte, attempt trivial.AttemptStrategy) { AttemptStrategy not needed.
https://codereview.appspot.com/7346052/
« Back to merge proposal
LGTM with the AttemptStrategy stuff dropped.
https:/ /codereview. appspot. com/7346052/ diff/12001/ environs/ local/storage_ test.go local/storage_ test.go (right):
File environs/
https:/ /codereview. appspot. com/7346052/ diff/12001/ environs/ local/storage_ test.go# newcode18 local/storage_ test.go: 18: var noRetry = AttemptStrategy {}
environs/
trivial.
There's no need for an AttemptStrategy here.
https:/ /codereview. appspot. com/7346052/ diff/12001/ environs/ local/storage_ test.go# newcode22 local/storage_ test.go: 22: // be removed again.
environs/
These tests should be written properly, and should not be earmarked for
deletion. Do we really write these sorts of tests against an *Environ*
in the other cases? If so, what have we been smoking?
https:/ /codereview. appspot. com/7346052/ diff/12001/ environs/ local/storage_ test.go# newcode85 local/storage_ test.go: 85: func checkFileDoesNo tExist( c *C, StorageReader, name string, attempt AttemptStrategy ) {
environs/
storage environs.
trivial.
AttemptStrategy not needed.
https:/ /codereview. appspot. com/7346052/ diff/12001/ environs/ local/storage_ test.go# newcode99 local/storage_ test.go: 99: func checkFileHasCon tents(c *C, StorageReader, name string, contents []byte, attempt AttemptStrategy ) {
environs/
storage environs.
trivial.
AttemptStrategy not needed.
https:/ /codereview. appspot. com/7346052/