Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1015)

Unified Diff: environs/openstack/local_test.go

Issue 6923056: Fix OpenStack config issues (Closed)
Patch Set: Fix OpenStack config issues Created 12 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « environs/openstack/config_test.go ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: environs/openstack/local_test.go
=== modified file 'environs/openstack/local_test.go'
--- environs/openstack/local_test.go 2012-12-07 16:19:40 +0000
+++ environs/openstack/local_test.go 2012-12-19 04:09:51 +0000
@@ -2,48 +2,17 @@
import (
. "launchpad.net/gocheck"
- "launchpad.net/juju-core/environs"
"launchpad.net/juju-core/environs/openstack"
- coretesting "launchpad.net/juju-core/testing"
- "os"
)
-func init() {
- // HEADS UP: Please do not break trunk tests. Before committing changes,
- // make sure that tests in trunk are actually passing against whatever
- // revision of the packages depended upon that are *currently* public,
- // so that other people can continue to rely on trunk for their work.
- //
- // Note that this is just an ugly hack to get trunk building again. Tests
- // shouldn't depend on external variables like this, and should also not
- // hack the environment permanently as done here. It is fine to depend on
- // the environment when running with a flag that enables specific tests
- // though (such as -amazon).
- os.Setenv("OS_AUTH_URL", "PLEASE FIX ME")
- os.Setenv("OS_REGION_NAME", "PLEASE FIX ME")
- os.Setenv("OS_TENANT_NAME", "PLEASE FIX ME")
- os.Setenv("OS_USERNAME", "PLEASE FIX ME")
- os.Setenv("OS_PASSWORD", "PLEASE FIX ME")
-}
-
func registerLocalTests() {
Suite(&LocalSuite{})
}
type LocalSuite struct {
- env environs.Environ
}
func (s *LocalSuite) SetUpSuite(c *C) {
- env, err := environs.NewFromAttrs(map[string]interface{}{
- "name": "test",
- "type": "openstack",
- "authorized-keys": "foo",
- "ca-cert": coretesting.CACert,
- "ca-private-key": coretesting.CAKey,
- })
- c.Assert(err, IsNil)
- s.env = env
openstack.UseTestMetadata(true)
openstack.ShortTimeouts(true)
}
@@ -51,19 +20,6 @@
func (s *LocalSuite) TearDownSuite(c *C) {
openstack.UseTestMetadata(false)
openstack.ShortTimeouts(false)
- s.env = nil
-}
-
-func (s *LocalSuite) TestPrivateAddress(c *C) {
- p := s.env.Provider()
- addr, err := p.PrivateAddress()
- c.Assert(err, IsNil)
- c.Assert(addr, Equals, "private.dummy.address.example.com")
-}
-
-func (s *LocalSuite) TestPublicAddress(c *C) {
- p := s.env.Provider()
- addr, err := p.PublicAddress()
- c.Assert(err, IsNil)
- c.Assert(addr, Equals, "public.dummy.address.example.com")
-}
+}
+
+//TODO(wallyworld) - add any necessary tests
« no previous file with comments | « environs/openstack/config_test.go ('k') | no next file » | no next file with comments »

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b