Code review comment for lp://staging/~wallyworld/juju-core/fix-utopic-tools-tests

Reviewers: mp+221655_code.launchpad.net,

Message:
Please take a look.

Description:
Fix unit tests on Utopic

Fix uploadFakeTools so that current series as well
as lts series are uploaded.

https://code.launchpad.net/~wallyworld/juju-core/fix-utopic-tools-tests/+merge/221655

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/102920049/

Affected files (+6, -1 lines):
   A [revision details]
   M environs/testing/tools.go

Index: [revision details]
=== added file '[revision details]'
--- [revision details] 2012-01-01 00:00:00 +0000
+++ [revision details] 2012-01-01 00:00:00 +0000
@@ -0,0 +1,2 @@
+Old revision: tarmac-20140530163551-b1m9k0br0iaxwqim
+New revision: <email address hidden>

Index: environs/testing/tools.go
=== modified file 'environs/testing/tools.go'
--- environs/testing/tools.go 2014-05-27 01:32:17 +0000
+++ environs/testing/tools.go 2014-06-02 02:40:18 +0000
@@ -21,6 +21,7 @@
   coretesting "launchpad.net/juju-core/testing"
   coretools "launchpad.net/juju-core/tools"
   "launchpad.net/juju-core/utils"
+ "launchpad.net/juju-core/utils/set"
   "launchpad.net/juju-core/version"
   "launchpad.net/juju-core/worker/upgrader"
  )
@@ -185,8 +186,10 @@
  }

  func uploadFakeTools(stor storage.Storage) error {
+ toolsSeries := set.NewStrings(bootstrap.ToolsLtsSeries...)
+ toolsSeries.Add(version.Current.Series)
   var versions []version.Binary
- for _, series := range bootstrap.ToolsLtsSeries {
+ for _, series := range toolsSeries.Values() {
    vers := version.Current
    vers.Series = series
    versions = append(versions, vers)