Viewing all changes in revision 1939.

  • Committer: Tarmac
  • Author(s): Tim Penhey
  • Date: 2013-10-03 04:00:43 UTC
  • mfrom: (1932.3.3 verify-storage-move)
  • Revision ID: tarmac-20131003040043-xb0lnqkiji4vw8b0
[r=thumper],[bug=1228992] Fail earlier when bootstrapping if up already.

I broke apart the VerifyBootstrap method into two:
Well I didn't create the second, just didn't call it
from the EnsureNotBootstrapped call.

The ensure method is now called in the command bootstrap
prior to any upload tools checks.

However the implication of this was much futher reaching
than I expected. Many tests assumed that bootstrap.Bootstrap
was enough to bootstrap and environment, and that it would
complain if already bootstrapped. Since the complaining was
moved into the command function, some changes had to be made.
Since some of the tests were refactored to check the storage,
this required making the dummy provider actually write the
provider-state into storage like a normal provider.

I moved the VerifyStorage out of emptystorage.go and into open.go
as it seemed better fitting there (and the comment actually
makes sense now).

Also used the environs constant for the VerificationFilename.


expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: