diff --git a/spec/unit/network/http_pool_spec.rb b/spec/unit/network/http_pool_spec.rb
index c86f7a660..0bbd1da15 100755
--- a/spec/unit/network/http_pool_spec.rb
+++ b/spec/unit/network/http_pool_spec.rb
@@ -1,135 +1,135 @@
 #!/usr/bin/env rspec
 require 'spec_helper'
 require 'puppet/network/http_pool'
 
 describe Puppet::Network::HttpPool do
   after do
     Puppet::Network::HttpPool.instance_variable_set("@ssl_host", nil)
   end
 
   it "should use the global SSL::Host instance to get its certificate information" do
     host = mock 'host'
     Puppet::SSL::Host.expects(:localhost).with.returns host
     Puppet::Network::HttpPool.ssl_host.should equal(host)
   end
 
   describe "when managing http instances" do
-    def stub_settings(settings)
-      settings.each do |param, value|
-        Puppet.settings.stubs(:value).with(param).returns(value)
-      end
-    end
-
-    before do
+    before :each do
       # All of the cert stuff is tested elsewhere
       Puppet::Network::HttpPool.stubs(:cert_setup)
     end
 
     it "should return an http instance created with the passed host and port" do
-      http = stub 'http', :use_ssl= => nil, :read_timeout= => nil, :open_timeout= => nil, :started? => false
-      Net::HTTP.expects(:new).with("me", 54321, nil, nil).returns(http)
-      Puppet::Network::HttpPool.http_instance("me", 54321).should equal(http)
+      http = Puppet::Network::HttpPool.http_instance("me", 54321)
+      http.should be_an_instance_of Net::HTTP
+      http.address.should == 'me'
+      http.port.should    == 54321
     end
 
     it "should enable ssl on the http instance" do
-      Puppet::Network::HttpPool.http_instance("me", 54321).instance_variable_get("@use_ssl").should be_true
+      Puppet::Network::HttpPool.http_instance("me", 54321).should be_use_ssl
     end
 
-    it "should set the read timeout" do
-      Puppet::Network::HttpPool.http_instance("me", 54321).read_timeout.should == 120
-    end
+    context "proxy and timeout settings should propagate" do
+      subject { Puppet::Network::HttpPool.http_instance("me", 54321) }
+      before :each do
+        Puppet[:http_proxy_host] = "myhost"
+        Puppet[:http_proxy_port] = 432
+        Puppet[:configtimeout]   = 120
+      end
 
-    it "should set the open timeout" do
-      Puppet::Network::HttpPool.http_instance("me", 54321).open_timeout.should == 120
+      its(:open_timeout)  { should == Puppet[:configtimeout] }
+      its(:read_timeout)  { should == Puppet[:configtimeout] }
+      its(:proxy_address) { should == Puppet[:http_proxy_host] }
+      its(:proxy_port)    { should == Puppet[:http_proxy_port] }
     end
 
-    it "should create the http instance with the proxy host and port set if the http_proxy is not set to 'none'" do
-      stub_settings :http_proxy_host => "myhost", :http_proxy_port => 432, :configtimeout => 120
-      Puppet::Network::HttpPool.http_instance("me", 54321).open_timeout.should == 120
+    it "should not set a proxy if the value is 'none'" do
+      Puppet[:http_proxy_host] = 'none'
+      Puppet::Network::HttpPool.http_instance("me", 54321).proxy_address.should be_nil
     end
 
     it "should not cache http instances" do
-      stub_settings :http_proxy_host => "myhost", :http_proxy_port => 432, :configtimeout => 120
-      old = Puppet::Network::HttpPool.http_instance("me", 54321)
-      Puppet::Network::HttpPool.http_instance("me", 54321).should_not equal(old)
+      Puppet::Network::HttpPool.http_instance("me", 54321).
+        should_not equal Puppet::Network::HttpPool.http_instance("me", 54321)
     end
   end
 
-  describe "when adding certificate information to http instances" do
-    before do
-      @http = mock 'http'
-      [:cert_store=, :verify_mode=, :ca_file=, :cert=, :key=].each { |m| @http.stubs(m) }
-      @store = stub 'store'
-
-      @cert = stub 'cert', :content => "real_cert"
-      @key = stub 'key', :content => "real_key"
-      @host = stub 'host', :certificate => @cert, :key => @key, :ssl_store => @store
+  describe "when doing SSL setup for http instances" do
+    let :http do
+      http = Net::HTTP.new('localhost', 443)
+      http.use_ssl = true
+      http
+    end
 
-      Puppet[:confdir] = "/sometthing/else"
-      Puppet.settings.stubs(:value).returns "/some/file"
-      Puppet.settings.stubs(:value).with(:hostcert).returns "/host/cert"
-      Puppet.settings.stubs(:value).with(:localcacert).returns "/local/ca/cert"
+    let :store do stub('store') end
 
-      FileTest.stubs(:exist?).with("/host/cert").returns true
-      FileTest.stubs(:exist?).with("/local/ca/cert").returns true
+    before :each do
+      Puppet[:hostcert]    = '/host/cert'
+      Puppet[:localcacert] = '/local/ca/cert'
 
-      Puppet::Network::HttpPool.stubs(:ssl_host).returns @host
-    end
-
-    after do
-      Puppet.settings.clear
+      cert  = stub 'cert', :content => 'real_cert'
+      key   = stub 'key',  :content => 'real_key'
+      host  = stub 'host', :certificate => cert, :key => key, :ssl_store => store
+      Puppet::Network::HttpPool.stubs(:ssl_host).returns(host)
     end
 
     it "should do nothing if no host certificate is on disk" do
-      FileTest.expects(:exist?).with("/host/cert").returns false
-      @http.expects(:cert=).never
-      Puppet::Network::HttpPool.cert_setup(@http)
+      FileTest.expects(:exist?).with(Puppet[:hostcert]).returns false
+      http.expects(:cert=).never
+      Puppet::Network::HttpPool.cert_setup(http)
     end
 
     it "should do nothing if no local certificate is on disk" do
-      FileTest.expects(:exist?).with("/local/ca/cert").returns false
-      @http.expects(:cert=).never
-      Puppet::Network::HttpPool.cert_setup(@http)
+      FileTest.expects(:exist?).with(Puppet[:hostcert]).returns true
+      FileTest.expects(:exist?).with(Puppet[:localcacert]).returns false
+      http.expects(:cert=).never
+      Puppet::Network::HttpPool.cert_setup(http)
     end
 
     it "should add a certificate store from the ssl host" do
-      @http.expects(:cert_store=).with(@store)
+      FileTest.expects(:exist?).with(Puppet[:hostcert]).returns true
+      FileTest.expects(:exist?).with(Puppet[:localcacert]).returns true
+      http.expects(:cert_store=).with(store)
 
-      Puppet::Network::HttpPool.cert_setup(@http)
+      Puppet::Network::HttpPool.cert_setup(http)
     end
 
     it "should add the client certificate" do
-      @http.expects(:cert=).with("real_cert")
+      FileTest.expects(:exist?).with(Puppet[:hostcert]).returns true
+      FileTest.expects(:exist?).with(Puppet[:localcacert]).returns true
+      http.expects(:cert=).with("real_cert")
 
-      Puppet::Network::HttpPool.cert_setup(@http)
+      Puppet::Network::HttpPool.cert_setup(http)
     end
 
     it "should add the client key" do
-      @http.expects(:key=).with("real_key")
+      FileTest.expects(:exist?).with(Puppet[:hostcert]).returns true
+      FileTest.expects(:exist?).with(Puppet[:localcacert]).returns true
+      http.expects(:key=).with("real_key")
 
-      Puppet::Network::HttpPool.cert_setup(@http)
+      Puppet::Network::HttpPool.cert_setup(http)
     end
 
     it "should set the verify mode to OpenSSL::SSL::VERIFY_PEER" do
-      @http.expects(:verify_mode=).with(OpenSSL::SSL::VERIFY_PEER)
+      FileTest.expects(:exist?).with(Puppet[:hostcert]).returns true
+      FileTest.expects(:exist?).with(Puppet[:localcacert]).returns true
+      http.expects(:verify_mode=).with(OpenSSL::SSL::VERIFY_PEER)
 
-      Puppet::Network::HttpPool.cert_setup(@http)
+      Puppet::Network::HttpPool.cert_setup(http)
     end
 
     it "should set the ca file" do
-      Puppet.settings.stubs(:value).returns "/some/file"
-      FileTest.stubs(:exist?).with(Puppet[:hostcert]).returns true
-
-      Puppet.settings.stubs(:value).with(:localcacert).returns "/ca/cert/file"
-      FileTest.stubs(:exist?).with("/ca/cert/file").returns true
-      @http.expects(:ca_file=).with("/ca/cert/file")
+      FileTest.expects(:exist?).with(Puppet[:hostcert]).returns(true)
+      FileTest.expects(:exist?).with(Puppet[:localcacert]).returns(true)
 
-      Puppet::Network::HttpPool.cert_setup(@http)
+      Puppet::Network::HttpPool.cert_setup(http)
+      http.ca_file.should == Puppet[:localcacert]
     end
 
     it "should set up certificate information when creating http instances" do
       Puppet::Network::HttpPool.expects(:cert_setup).with { |i| i.is_a?(Net::HTTP) }
       Puppet::Network::HttpPool.http_instance("one", "two")
     end
   end
 end