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

Unified Diff: worker/machineenvironmentworker/machineenvironmentworker_test.go

Issue 92610045: New utils/proxy package.
Patch Set: New utils/proxy package. Created 10 years, 10 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 | « worker/machineenvironmentworker/machineenvironmentworker.go ('k') | worker/uniter/context.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: worker/machineenvironmentworker/machineenvironmentworker_test.go
=== modified file 'worker/machineenvironmentworker/machineenvironmentworker_test.go'
--- worker/machineenvironmentworker/machineenvironmentworker_test.go 2014-05-23 11:23:30 +0000
+++ worker/machineenvironmentworker/machineenvironmentworker_test.go 2014-05-26 09:48:26 +0000
@@ -14,7 +14,6 @@
"launchpad.net/juju-core/agent"
"launchpad.net/juju-core/environs/config"
- "launchpad.net/juju-core/juju/osenv"
jujutesting "launchpad.net/juju-core/juju/testing"
"launchpad.net/juju-core/names"
"launchpad.net/juju-core/provider"
@@ -23,6 +22,7 @@
"launchpad.net/juju-core/state/api/environment"
"launchpad.net/juju-core/testing"
"launchpad.net/juju-core/utils/apt"
+ "launchpad.net/juju-core/utils/proxy"
"launchpad.net/juju-core/worker"
"launchpad.net/juju-core/worker/machineenvironmentworker"
)
@@ -72,13 +72,13 @@
}
}
-func (s *MachineEnvironmentWatcherSuite) waitProxySettings(c *gc.C, expected osenv.ProxySettings) {
+func (s *MachineEnvironmentWatcherSuite) waitProxySettings(c *gc.C, expected proxy.Settings) {
for {
select {
case <-time.After(testing.LongWait):
c.Fatalf("timeout while waiting for proxy settings to change")
case <-time.After(10 * time.Millisecond):
- obtained := osenv.DetectProxies()
+ obtained := proxy.DetectProxies()
if obtained != expected {
c.Logf("proxy settings are %#v, still waiting", obtained)
continue
@@ -118,9 +118,9 @@
c.Assert(worker.Stop(envWorker), gc.IsNil)
}
-func (s *MachineEnvironmentWatcherSuite) updateConfig(c *gc.C) (osenv.ProxySettings, osenv.ProxySettings) {
+func (s *MachineEnvironmentWatcherSuite) updateConfig(c *gc.C) (proxy.Settings, proxy.Settings) {
- proxySettings := osenv.ProxySettings{
+ proxySettings := proxy.Settings{
Http: "http proxy",
Https: "https proxy",
Ftp: "ftp proxy",
@@ -135,7 +135,7 @@
// settings that are used for the apt config, and not just the normal
// proxy settings which is what we would get if we don't explicitly set
// apt values.
- aptProxySettings := osenv.ProxySettings{
+ aptProxySettings := proxy.Settings{
Http: "apt http proxy",
Https: "apt https proxy",
Ftp: "apt ftp proxy",
« no previous file with comments | « worker/machineenvironmentworker/machineenvironmentworker.go ('k') | worker/uniter/context.go » ('j') | no next file with comments »

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