Fixing #2234 - fixing all of the tests broken by my bindaddress fix

Signed-off-by: Luke Kanies <luke@madstop.com>

2 files changed, 36 insertions, 2 deletions

diff --git a/spec/integration/network/server/mongrel.rb b/spec/integration/network/server/mongrel.rb
index 180fdf7ad..d621220e2 100755
--- a/spec/integration/network/server/mongrel.rb
+++ b/spec/integration/network/server/mongrel.rb
@@ -10,10 +10,13 @@ describe Puppet::Network::Server do
         
         before :each do
             Puppet[:servertype] = 'mongrel'
-            @params = { :address => "127.0.0.1", :port => 34346, :handlers => [ :node ] }
+            Puppet[:server] = '127.0.0.1'
+            @params = { :port => 34346, :handlers => [ :node ] }
             @server = Puppet::Network::Server.new(@params)            
         end
 
+        after { Puppet.settings.clear }
+
         describe "before listening" do
             it "should not be reachable at the specified address and port" do
                 lambda { TCPSocket.new('127.0.0.1', 34346) }.should raise_error(Errno::ECONNREFUSED)
@@ -26,6 +29,21 @@ describe Puppet::Network::Server do
                 lambda { TCPSocket.new('127.0.0.1', 34346) }.should_not raise_error            
             end
 
+            it "should default to '127.0.0.1' as its bind address" do
+                @server = Puppet::Network::Server.new(@params.merge(:port => 34343))            
+                @server.stubs(:unlisten) # we're breaking listening internally, so we have to keep it from unlistening
+                @server.send(:http_server).expects(:listen).with { |args| args[:address] == "127.0.0.1" }
+                @server.listen
+            end
+
+            it "should use any specified bind address" do
+                Puppet[:bindaddress] = "0.0.0.0"
+                @server = Puppet::Network::Server.new(@params.merge(:port => 34343))            
+                @server.stubs(:unlisten) # we're breaking listening internally, so we have to keep it from unlistening
+                @server.send(:http_server).expects(:listen).with { |args| args[:address] == "0.0.0.0" }
+                @server.listen
+            end
+
             it "should not allow multiple servers to listen on the same address and port" do
                 @server.listen
                 @server2 = Puppet::Network::Server.new(@params)
diff --git a/spec/integration/network/server/webrick.rb b/spec/integration/network/server/webrick.rb
index 5cd573141..06916573f 100755
--- a/spec/integration/network/server/webrick.rb
+++ b/spec/integration/network/server/webrick.rb
@@ -9,7 +9,8 @@ describe Puppet::Network::Server do
     describe "when using webrick" do
         before :each do
             Puppet[:servertype] = 'webrick'
-            @params = { :address => "127.0.0.1", :port => 34343, :handlers => [ :node ], :xmlrpc_handlers => [ :status ] }
+            Puppet[:server] = '127.0.0.1'
+            @params = { :port => 34343, :handlers => [ :node ], :xmlrpc_handlers => [ :status ] }
 
             # Get a safe temporary file
             @tmpfile = Tempfile.new("webrick_integration_testing")
@@ -46,6 +47,21 @@ describe Puppet::Network::Server do
                 lambda { TCPSocket.new('127.0.0.1', 34343) }.should_not raise_error            
             end
 
+            it "should default to '0.0.0.0' as its bind address" do
+                @server = Puppet::Network::Server.new(@params.merge(:port => 34343))            
+                @server.stubs(:unlisten) # we're breaking listening internally, so we have to keep it from unlistening
+                @server.send(:http_server).expects(:listen).with { |args| args[:address] == "0.0.0.0" }
+                @server.listen
+            end
+
+            it "should use any specified bind address" do
+                Puppet[:bindaddress] = "127.0.0.1"
+                @server = Puppet::Network::Server.new(@params.merge(:port => 34343))            
+                @server.stubs(:unlisten) # we're breaking listening internally, so we have to keep it from unlistening
+                @server.send(:http_server).expects(:listen).with { |args| args[:address] == "127.0.0.1" }
+                @server.listen
+            end
+
             it "should not allow multiple servers to listen on the same address and port" do
                 @server = Puppet::Network::Server.new(@params.merge(:port => 34343))            
                 @server.listen