* lib/webrick/httpresponse.rb (setup_header): 1xx responses

  are allowed to have Keep-Alive connections.

* test/webrick/test_httpresponse.rb: corresponding test.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@34024 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

1 files changed, 17 insertions, 14 deletions

diff --git a/test/webrick/test_httpresponse.rb b/test/webrick/test_httpresponse.rb
index 3967e2c3e8..d5d5552796 100644
--- a/test/webrick/test_httpresponse.rb
+++ b/test/webrick/test_httpresponse.rb
@@ -15,29 +15,32 @@ module WEBrick
       end
     end
 
-    def test_304_does_not_log_warning
-      logger          = FakeLogger.new
-      config          = Config::HTTP
-      config[:Logger] = logger
+    attr_reader :config, :logger, :res
+
+    def setup
+      super
+      @logger          = FakeLogger.new
+      @config          = Config::HTTP
+      @config[:Logger] = logger
+      @res             = HTTPResponse.new config
+      @res.keep_alive  = true
+    end
 
-      res             = HTTPResponse.new config
+    def test_304_does_not_log_warning
       res.status      = 304
-      res.keep_alive  = true
-
       res.setup_header
-
       assert_equal 0, logger.messages.length
     end
 
     def test_204_does_not_log_warning
-      logger          = FakeLogger.new
-      config          = Config::HTTP
-      config[:Logger] = logger
-
-      res             = HTTPResponse.new config
       res.status      = 204
-      res.keep_alive  = true
+      res.setup_header
+
+      assert_equal 0, logger.messages.length
+    end
 
+    def test_1xx_does_not_log_warnings
+      res.status      = 105
       res.setup_header
 
       assert_equal 0, logger.messages.length