Code review comment for lp:~axwalk/juju-core/lp1186969-add-jujud-tools-help

The attempt to merge lp:~axwalk/juju-core/lp1186969-add-jujud-tools-help into lp:juju-core failed. Below is the output from the failed tests.

ok launchpad.net/juju-core/agent 2.259s
ok launchpad.net/juju-core/agent/tools 21.264s
ok launchpad.net/juju-core/bzr 6.840s
ok launchpad.net/juju-core/cert 3.141s
ok launchpad.net/juju-core/charm 0.543s
? launchpad.net/juju-core/charm/hooks [no test files]
ok launchpad.net/juju-core/cloudinit 0.009s
ok launchpad.net/juju-core/cmd 0.348s
? launchpad.net/juju-core/cmd/builddb [no test files]
? launchpad.net/juju-core/cmd/charmd [no test files]
? launchpad.net/juju-core/cmd/charmload [no test files]

----------------------------------------------------------------------
FAIL: main_test.go:256: MainSuite.TestHelpCommands

main_test.go:271:
    // The names should be output in alphabetical order, so don't sort.
    c.Assert(names, DeepEquals, commandNames)
... obtained []string = []string{"add-machine", "add-relation", "add-unit", "bootstrap", "debug-log", "deploy", "destroy-environment", "destroy-machine", "destroy-relation", "destroy-service", "destroy-unit", "env", "expose", "generate-config", "get", "get-constraints", "get-env", "get-environment", "help", "help-tool", "image-metadata", "init", "publish", "remove-relation", "remove-unit", "resolved", "scp", "set", "set-constraints", "set-env", "set-environment", "ssh", "stat", "status", "switch", "sync-tools", "terminate-machine", "unexpose", "upgrade-charm", "upgrade-juju", "version"}
... expected []string = []string{"add-machine", "add-relation", "add-unit", "bootstrap", "debug-log", "deploy", "destroy-environment", "destroy-machine", "destroy-relation", "destroy-service", "destroy-unit", "env", "expose", "generate-config", "get", "get-constraints", "get-env", "get-environment", "help", "image-metadata", "init", "publish", "remove-relation", "remove-unit", "resolved", "scp", "set", "set-constraints", "set-env", "set-environment", "ssh", "stat", "status", "switch", "sync-tools", "terminate-machine", "unexpose", "upgrade-charm", "upgrade-juju", "version"}

OOPS: 165 passed, 1 FAILED
--- FAIL: TestPackage (102.42 seconds)
FAIL
FAIL launchpad.net/juju-core/cmd/juju 102.730s
ok launchpad.net/juju-core/cmd/jujud 40.914s
ok launchpad.net/juju-core/constraints 0.015s
ok launchpad.net/juju-core/container/lxc 0.314s
? launchpad.net/juju-core/container/lxc/mock [no test files]
ok launchpad.net/juju-core/downloader 5.298s
ok launchpad.net/juju-core/environs 1.798s
? launchpad.net/juju-core/environs/all [no test files]
ok launchpad.net/juju-core/environs/azure 3.640s
ok launchpad.net/juju-core/environs/cloudinit 0.652s
ok launchpad.net/juju-core/environs/config 1.061s
ok launchpad.net/juju-core/environs/dummy 17.147s
ok launchpad.net/juju-core/environs/ec2 175.440s
ok launchpad.net/juju-core/environs/imagemetadata 0.345s
ok launchpad.net/juju-core/environs/instances 0.236s
ok launchpad.net/juju-core/environs/jujutest 0.261s
ok launchpad.net/juju-core/environs/local 1.776s
? launchpad.net/juju-core/environs/local/storage [no test files]
ok launchpad.net/juju-core/environs/localstorage 0.267s
ok launchpad.net/juju-core/environs/maas 3.163s
ok launchpad.net/juju-core/environs/openstack 8.855s
? launchpad.net/juju-core/environs/provider [no test files]
ok launchpad.net/juju-core/environs/testing 0.005s
? launchpad.net/juju-core/errors [no test files]
ok launchpad.net/juju-core/instance 0.011s
ok launchpad.net/juju-core/juju 12.458s
? launchpad.net/juju-core/juju/testing [no test files]
ok launchpad.net/juju-core/log 0.007s
ok launchpad.net/juju-core/log/syslog 0.010s
ok launchpad.net/juju-core/rpc 0.265s
ok launchpad.net/juju-core/rpc/jsoncodec 0.301s
ok launchpad.net/juju-core/schema 0.011s
ok launchpad.net/juju-core/state 63.847s
? launchpad.net/juju-core/state/api [no test files]
? launchpad.net/juju-core/state/api/common [no test files]
ok launchpad.net/juju-core/state/api/deployer 6.765s
ok launchpad.net/juju-core/state/api/machineagent 1.852s
ok launchpad.net/juju-core/state/api/machiner 3.695s
ok launchpad.net/juju-core/state/api/params 0.022s
ok launchpad.net/juju-core/state/api/upgrader 3.750s
ok launchpad.net/juju-core/state/api/watcher 3.280s
ok launchpad.net/juju-core/state/apiserver 3.070s
ok launchpad.net/juju-core/state/apiserver/client 14.579s
ok launchpad.net/juju-core/state/apiserver/common 0.020s
ok launchpad.net/juju-core/state/apiserver/deployer 4.802s
ok launchpad.net/juju-core/state/apiserver/machine 3.981s
? launchpad.net/juju-core/state/apiserver/testing [no test files]
ok launchpad.net/juju-core/state/apiserver/upgrader 3.990s
ok launchpad.net/juju-core/state/multiwatcher 1.161s
ok launchpad.net/juju-core/state/presence 6.848s
ok launchpad.net/juju-core/state/statecmd 7.523s
? launchpad.net/juju-core/state/testing [no test files]
ok launchpad.net/juju-core/state/watcher 8.022s
ok launchpad.net/juju-core/store 24.522s
ok launchpad.net/juju-core/testing 1.436s
ok launchpad.net/juju-core/testing/checkers 0.009s
ok launchpad.net/juju-core/thirdparty/pbkdf2 0.221s
ok launchpad.net/juju-core/upstart 0.331s
ok launchpad.net/juju-core/utils 1.212s
ok launchpad.net/juju-core/utils/fslock 1.043s
ok launchpad.net/juju-core/utils/set 0.008s
ok launchpad.net/juju-core/version 0.099s
ok launchpad.net/juju-core/worker 1.580s
ok launchpad.net/juju-core/worker/cleaner 1.270s
ok launchpad.net/juju-core/worker/deployer 3.402s
ok launchpad.net/juju-core/worker/firewaller 11.617s
ok launchpad.net/juju-core/worker/machiner 2.597s
ok launchpad.net/juju-core/worker/provisioner 19.136s
ok launchpad.net/juju-core/worker/resumer 1.490s
ok launchpad.net/juju-core/worker/uniter 86.612s
ok launchpad.net/juju-core/worker/uniter/charm 2.258s
ok launchpad.net/juju-core/worker/uniter/hook 0.008s
ok launchpad.net/juju-core/worker/uniter/jujuc 0.501s
ok launchpad.net/juju-core/worker/uniter/relation 1.328s

warning: building out-of-date packages:
 launchpad.net/goamz/ec2/ec2test
 launchpad.net/goamz/s3/s3test
 launchpad.net/goose/testservices/hook
 launchpad.net/goose/testservices/identityservice
 launchpad.net/goose/testservices
 launchpad.net/goose/testservices/novaservice
 launchpad.net/goose/testservices/swiftservice
 launchpad.net/goose/testservices/openstackservice
installing these packages with 'go test -i ./...' will speed future tests.