Merge pull request #8 from joliss/spec

Fix :preserve_host spec

Author: jaswope

spec/rack/reverse_proxy_spec.rb

@@ -38,34 +38,35 @@ def app
       last_response.body.should == "Proxied App2"
     end
 
-		it "the response header should never contain Status" do
-			stub_request(:any, 'example.com/test/stuff').to_return(:headers => {'Status' => '200 OK'})
-			get '/test/stuff'
-			last_response.headers['Status'].should == nil
-		end
-
-		it "the response header should never transfer-encoding" do
-			stub_request(:any, 'example.com/test/stuff').to_return(:headers => {'transfer-encoding' => 'Chunked'})
-			get '/test/stuff'
-			last_response.headers['transfer-encoding'].should == nil
-		end
-
-		it "should set the Host header" do
-			stub_request(:any, 'example.com/test/stuff')
-			get '/test/stuff'
-			a_request(:get, 'http://example.com/test/stuff').with(:headers => {"Host" => "example.com"}).should have_been_made
-		end
+    it "the response header should never contain Status" do
+      stub_request(:any, 'example.com/test/stuff').to_return(:headers => {'Status' => '200 OK'})
+      get '/test/stuff'
+      last_response.headers['Status'].should == nil
+    end
+
+    it "the response header should never transfer-encoding" do
+      stub_request(:any, 'example.com/test/stuff').to_return(:headers => {'transfer-encoding' => 'Chunked'})
+      get '/test/stuff'
+      last_response.headers['transfer-encoding'].should == nil
+    end
+
+    it "should set the Host header" do
+      stub_request(:any, 'example.com/test/stuff')
+      get '/test/stuff'
+      a_request(:get, 'http://example.com/test/stuff').with(:headers => {"Host" => "example.com"}).should have_been_made
+    end
 
     describe "with preserve host turned off" do
       def app
         Rack::ReverseProxy.new(dummy_app) do
-          reverse_proxy '/test', 'http://example.com/'
+          reverse_proxy '/test', 'http://example.com/', {:preserve_host => false}
         end
       end
 
       it "should not set the Host header" do
         stub_request(:any, 'example.com/test/stuff')
         get '/test/stuff'
+        a_request(:get, 'http://example.com/test/stuff').with(:headers => {"Host" => "example.com"}).should_not have_been_made
         a_request(:get, 'http://example.com/test/stuff').should have_been_made
       end
     end
@@ -108,9 +109,9 @@ def app
       end
 
       it "should throw an exception" do
-       stub_request(:get, 'http://example1.com/test').to_return({:body => "Proxied App"})
-       get '/test'
-       last_response.body.should == "Proxied App"
+        stub_request(:get, 'http://example1.com/test').to_return({:body => "Proxied App"})
+        get '/test'
+        last_response.body.should == "Proxied App"
       end
     end
 
@@ -128,20 +129,20 @@ def app
       end
     end
 
-		describe "with a https route" do
+    describe "with a https route" do
       def app
         Rack::ReverseProxy.new(dummy_app) do
           reverse_proxy '/test', 'https://example.com'
         end
       end
 
-			it "should make a secure request" do
+      it "should make a secure request" do
         stub_request(:get, 'https://example.com/test/stuff').to_return({:body => "Proxied Secure App"})
         get '/test/stuff'
         last_response.body.should == "Proxied Secure App"
-			end
+      end
 
-		end
+    end
 
     describe "with a route as a string" do
       def app