rack 2.0.4 → 2.1.0

data/test/spec_chunked.rb

@@ -1,103 +0,0 @@
-require 'minitest/autorun'
-require 'rack/chunked'
-require 'rack/lint'
-require 'rack/mock'
-
-describe Rack::Chunked do
-  def chunked(app)
-    proc do |env|
-      app = Rack::Chunked.new(app)
-      response = Rack::Lint.new(app).call(env)
-      # we want to use body like an array, but it only has #each
-      response[2] = response[2].to_enum.to_a
-      response
-    end
-  end
-
-  before do
-    @env = Rack::MockRequest.
-      env_for('/', 'HTTP_VERSION' => '1.1', 'REQUEST_METHOD' => 'GET')
-  end
-
-  it 'chunk responses with no Content-Length' do
-    app = lambda { |env| [200, {"Content-Type" => "text/plain"}, ['Hello', ' ', 'World!']] }
-    response = Rack::MockResponse.new(*chunked(app).call(@env))
-    response.headers.wont_include 'Content-Length'
-    response.headers['Transfer-Encoding'].must_equal 'chunked'
-    response.body.must_equal "5\r\nHello\r\n1\r\n \r\n6\r\nWorld!\r\n0\r\n\r\n"
-  end
-
-  it 'chunks empty bodies properly' do
-    app = lambda { |env| [200, {"Content-Type" => "text/plain"}, []] }
-    response = Rack::MockResponse.new(*chunked(app).call(@env))
-    response.headers.wont_include 'Content-Length'
-    response.headers['Transfer-Encoding'].must_equal 'chunked'
-    response.body.must_equal "0\r\n\r\n"
-  end
-
-  it 'chunks encoded bodies properly' do
-    body = ["\uFFFEHello", " ", "World"].map {|t| t.encode("UTF-16LE") }
-    app  = lambda { |env| [200, {"Content-Type" => "text/plain"}, body] }
-    response = Rack::MockResponse.new(*chunked(app).call(@env))
-    response.headers.wont_include 'Content-Length'
-    response.headers['Transfer-Encoding'].must_equal 'chunked'
-    response.body.encoding.to_s.must_equal "ASCII-8BIT"
-    response.body.must_equal "c\r\n\xFE\xFFH\x00e\x00l\x00l\x00o\x00\r\n2\r\n \x00\r\na\r\nW\x00o\x00r\x00l\x00d\x00\r\n0\r\n\r\n".force_encoding("BINARY")
-    response.body.must_equal "c\r\n\xFE\xFFH\x00e\x00l\x00l\x00o\x00\r\n2\r\n \x00\r\na\r\nW\x00o\x00r\x00l\x00d\x00\r\n0\r\n\r\n".force_encoding(Encoding::BINARY)
-  end
-
-  it 'not modify response when Content-Length header present' do
-    app = lambda { |env|
-      [200, {"Content-Type" => "text/plain", 'Content-Length'=>'12'}, ['Hello', ' ', 'World!']]
-    }
-    status, headers, body = chunked(app).call(@env)
-    status.must_equal 200
-    headers.wont_include 'Transfer-Encoding'
-    headers.must_include 'Content-Length'
-    body.join.must_equal 'Hello World!'
-  end
-
-  it 'not modify response when client is HTTP/1.0' do
-    app = lambda { |env| [200, {"Content-Type" => "text/plain"}, ['Hello', ' ', 'World!']] }
-    @env['HTTP_VERSION'] = 'HTTP/1.0'
-    status, headers, body = chunked(app).call(@env)
-    status.must_equal 200
-    headers.wont_include 'Transfer-Encoding'
-    body.join.must_equal 'Hello World!'
-  end
-
-  it 'not modify response when client is ancient, pre-HTTP/1.0' do
-    app = lambda { |env| [200, {"Content-Type" => "text/plain"}, ['Hello', ' ', 'World!']] }
-    check = lambda do
-      status, headers, body = chunked(app).call(@env.dup)
-      status.must_equal 200
-      headers.wont_include 'Transfer-Encoding'
-      body.join.must_equal 'Hello World!'
-    end
-
-    @env.delete('HTTP_VERSION') # unicorn will do this on pre-HTTP/1.0 requests
-    check.call
-
-    @env['HTTP_VERSION'] = 'HTTP/0.9' # not sure if this happens in practice
-    check.call
-  end
-
-  it 'not modify response when Transfer-Encoding header already present' do
-    app = lambda { |env|
-      [200, {"Content-Type" => "text/plain", 'Transfer-Encoding' => 'identity'}, ['Hello', ' ', 'World!']]
-    }
-    status, headers, body = chunked(app).call(@env)
-    status.must_equal 200
-    headers['Transfer-Encoding'].must_equal 'identity'
-    body.join.must_equal 'Hello World!'
-  end
-
-  [100, 204, 304].each do |status_code|
-    it "not modify response when status code is #{status_code}" do
-      app = lambda { |env| [status_code, {}, []] }
-      status, headers, _ = chunked(app).call(@env)
-      status.must_equal status_code
-      headers.wont_include 'Transfer-Encoding'
-    end
-  end
-end

data/test/spec_common_logger.rb

@@ -1,95 +0,0 @@
-require 'minitest/autorun'
-require 'rack/common_logger'
-require 'rack/lint'
-require 'rack/mock'
-
-require 'logger'
-
-describe Rack::CommonLogger do
-  obj = 'foobar'
-  length = obj.size
-
-  app = Rack::Lint.new lambda { |env|
-    [200,
-     {"Content-Type" => "text/html", "Content-Length" => length.to_s},
-     [obj]]}
-  app_without_length = Rack::Lint.new lambda { |env|
-    [200,
-     {"Content-Type" => "text/html"},
-     []]}
-  app_with_zero_length = Rack::Lint.new lambda { |env|
-    [200,
-     {"Content-Type" => "text/html", "Content-Length" => "0"},
-     []]}
-
-  it "log to rack.errors by default" do
-    res = Rack::MockRequest.new(Rack::CommonLogger.new(app)).get("/")
-
-    res.errors.wont_be :empty?
-    res.errors.must_match(/"GET \/ " 200 #{length} /)
-  end
-
-  it "log to anything with +write+" do
-    log = StringIO.new
-    Rack::MockRequest.new(Rack::CommonLogger.new(app, log)).get("/")
-
-    log.string.must_match(/"GET \/ " 200 #{length} /)
-  end
-
-  it "work with standartd library logger" do
-    logdev = StringIO.new
-    log = Logger.new(logdev)
-    Rack::MockRequest.new(Rack::CommonLogger.new(app, log)).get("/")
-
-    logdev.string.must_match(/"GET \/ " 200 #{length} /)
-  end
-
-  it "log - content length if header is missing" do
-    res = Rack::MockRequest.new(Rack::CommonLogger.new(app_without_length)).get("/")
-
-    res.errors.wont_be :empty?
-    res.errors.must_match(/"GET \/ " 200 - /)
-  end
-
-  it "log - content length if header is zero" do
-    res = Rack::MockRequest.new(Rack::CommonLogger.new(app_with_zero_length)).get("/")
-
-    res.errors.wont_be :empty?
-    res.errors.must_match(/"GET \/ " 200 - /)
-  end
-
-  def with_mock_time(t = 0)
-    mc = class << Time; self; end
-    mc.send :alias_method, :old_now, :now
-    mc.send :define_method, :now do
-      at(t)
-    end
-    yield
-  ensure
-    mc.send :undef_method, :now
-    mc.send :alias_method, :now, :old_now
-  end
-
-  it "log in common log format" do
-    log = StringIO.new
-    with_mock_time do
-      Rack::MockRequest.new(Rack::CommonLogger.new(app, log)).get("/")
-    end
-
-    md = /- - - \[([^\]]+)\] "(\w+) \/ " (\d{3}) \d+ ([\d\.]+)/.match(log.string)
-    md.wont_equal nil
-    time, method, status, duration = *md.captures
-    time.must_equal Time.at(0).strftime("%d/%b/%Y:%H:%M:%S %z")
-    method.must_equal "GET"
-    status.must_equal "200"
-    (0..1).must_include duration.to_f
-  end
-
-  def length
-    123
-  end
-
-  def self.obj
-    "hello world"
-  end
-end

data/test/spec_conditional_get.rb

@@ -1,103 +0,0 @@
-require 'minitest/autorun'
-require 'time'
-require 'rack/conditional_get'
-require 'rack/mock'
-
-describe Rack::ConditionalGet do
-  def conditional_get(app)
-    Rack::Lint.new Rack::ConditionalGet.new(app)
-  end
-
-  it "set a 304 status and truncate body when If-Modified-Since hits" do
-    timestamp = Time.now.httpdate
-    app = conditional_get(lambda { |env|
-      [200, {'Last-Modified'=>timestamp}, ['TEST']] })
-
-    response = Rack::MockRequest.new(app).
-      get("/", 'HTTP_IF_MODIFIED_SINCE' => timestamp)
-
-    response.status.must_equal 304
-    response.body.must_be :empty?
-  end
-
-  it "set a 304 status and truncate body when If-Modified-Since hits and is higher than current time" do
-    app = conditional_get(lambda { |env|
-      [200, {'Last-Modified'=>(Time.now - 3600).httpdate}, ['TEST']] })
-
-    response = Rack::MockRequest.new(app).
-      get("/", 'HTTP_IF_MODIFIED_SINCE' => Time.now.httpdate)
-
-    response.status.must_equal 304
-    response.body.must_be :empty?
-  end
-
-  it "set a 304 status and truncate body when If-None-Match hits" do
-    app = conditional_get(lambda { |env|
-      [200, {'ETag'=>'1234'}, ['TEST']] })
-
-    response = Rack::MockRequest.new(app).
-      get("/", 'HTTP_IF_NONE_MATCH' => '1234')
-
-    response.status.must_equal 304
-    response.body.must_be :empty?
-  end
-
-  it "not set a 304 status if If-Modified-Since hits but Etag does not" do
-    timestamp = Time.now.httpdate
-    app = conditional_get(lambda { |env|
-      [200, {'Last-Modified'=>timestamp, 'Etag'=>'1234', 'Content-Type' => 'text/plain'}, ['TEST']] })
-
-    response = Rack::MockRequest.new(app).
-      get("/", 'HTTP_IF_MODIFIED_SINCE' => timestamp, 'HTTP_IF_NONE_MATCH' => '4321')
-
-    response.status.must_equal 200
-    response.body.must_equal 'TEST'
-  end
-
-  it "set a 304 status and truncate body when both If-None-Match and If-Modified-Since hits" do
-    timestamp = Time.now.httpdate
-    app = conditional_get(lambda { |env|
-      [200, {'Last-Modified'=>timestamp, 'ETag'=>'1234'}, ['TEST']] })
-
-    response = Rack::MockRequest.new(app).
-      get("/", 'HTTP_IF_MODIFIED_SINCE' => timestamp, 'HTTP_IF_NONE_MATCH' => '1234')
-
-    response.status.must_equal 304
-    response.body.must_be :empty?
-  end
-
-  it "not affect non-GET/HEAD requests" do
-    app = conditional_get(lambda { |env|
-      [200, {'Etag'=>'1234', 'Content-Type' => 'text/plain'}, ['TEST']] })
-
-    response = Rack::MockRequest.new(app).
-      post("/", 'HTTP_IF_NONE_MATCH' => '1234')
-
-    response.status.must_equal 200
-    response.body.must_equal 'TEST'
-  end
-
-  it "not affect non-200 requests" do
-    app = conditional_get(lambda { |env|
-      [302, {'Etag'=>'1234', 'Content-Type' => 'text/plain'}, ['TEST']] })
-
-    response = Rack::MockRequest.new(app).
-      get("/", 'HTTP_IF_NONE_MATCH' => '1234')
-
-    response.status.must_equal 302
-    response.body.must_equal 'TEST'
-  end
-
-  it "not affect requests with malformed HTTP_IF_NONE_MATCH" do
-    bad_timestamp = Time.now.strftime('%Y-%m-%d %H:%M:%S %z')
-    app = conditional_get(lambda { |env|
-      [200,{'Last-Modified'=>(Time.now - 3600).httpdate, 'Content-Type' => 'text/plain'}, ['TEST']] })
-
-    response = Rack::MockRequest.new(app).
-      get("/", 'HTTP_IF_MODIFIED_SINCE' => bad_timestamp)
-
-    response.status.must_equal 200
-    response.body.must_equal 'TEST'
-  end
-
-end

data/test/spec_config.rb

@@ -1,23 +0,0 @@
-require 'minitest/autorun'
-require 'rack/builder'
-require 'rack/config'
-require 'rack/content_length'
-require 'rack/lint'
-require 'rack/mock'
-
-describe Rack::Config do
-  it "accept a block that modifies the environment" do
-    app = Rack::Builder.new do
-      use Rack::Lint
-      use Rack::Config do |env|
-        env['greeting'] = 'hello'
-      end
-      run lambda { |env|
-        [200, {'Content-Type' => 'text/plain'}, [env['greeting'] || '']]
-      }
-    end
-
-    response = Rack::MockRequest.new(app).get('/')
-    response.body.must_equal 'hello'
-  end
-end

data/test/spec_content_length.rb

@@ -1,86 +0,0 @@
-require 'minitest/autorun'
-require 'rack/content_length'
-require 'rack/lint'
-require 'rack/mock'
-
-describe Rack::ContentLength do
-  def content_length(app)
-    Rack::Lint.new Rack::ContentLength.new(app)
-  end
-
-  def request
-    Rack::MockRequest.env_for
-  end
-
-  it "set Content-Length on Array bodies if none is set" do
-    app = lambda { |env| [200, {'Content-Type' => 'text/plain'}, ["Hello, World!"]] }
-    response = content_length(app).call(request)
-    response[1]['Content-Length'].must_equal '13'
-  end
-
-  it "not set Content-Length on variable length bodies" do
-    body = lambda { "Hello World!" }
-    def body.each ; yield call ; end
-
-    app = lambda { |env| [200, {'Content-Type' => 'text/plain'}, body] }
-    response = content_length(app).call(request)
-    response[1]['Content-Length'].must_be_nil
-  end
-
-  it "not change Content-Length if it is already set" do
-    app = lambda { |env| [200, {'Content-Type' => 'text/plain', 'Content-Length' => '1'}, "Hello, World!"] }
-    response = content_length(app).call(request)
-    response[1]['Content-Length'].must_equal '1'
-  end
-
-  it "not set Content-Length on 304 responses" do
-    app = lambda { |env| [304, {}, []] }
-    response = content_length(app).call(request)
-    response[1]['Content-Length'].must_be_nil
-  end
-
-  it "not set Content-Length when Transfer-Encoding is chunked" do
-    app = lambda { |env| [200, {'Content-Type' => 'text/plain', 'Transfer-Encoding' => 'chunked'}, []] }
-    response = content_length(app).call(request)
-    response[1]['Content-Length'].must_be_nil
-  end
-
-  # Using "Connection: close" for this is fairly contended. It might be useful
-  # to have some other way to signal this.
-  #
-  # should "not force a Content-Length when Connection:close" do
-  #   app = lambda { |env| [200, {'Connection' => 'close'}, []] }
-  #   response = content_length(app).call({})
-  #   response[1]['Content-Length'].must_be_nil
-  # end
-
-  it "close bodies that need to be closed" do
-    body = Struct.new(:body) do
-      attr_reader :closed
-      def each; body.join; end
-      def close; @closed = true; end
-      def to_ary; end
-    end.new(%w[one two three])
-
-    app = lambda { |env| [200, {'Content-Type' => 'text/plain'}, body] }
-    response = content_length(app).call(request)
-    body.closed.must_be_nil
-    response[2].close
-    body.closed.must_equal true
-  end
-
-  it "support single-execute bodies" do
-    body = Struct.new(:body) do
-      def each
-        yield body.shift until body.empty?
-      end
-      def to_ary; end
-    end.new(%w[one two three])
-
-    app = lambda { |env| [200, {'Content-Type' => 'text/plain'}, body] }
-    response = content_length(app).call(request)
-    expected = %w[one two three]
-    response[1]['Content-Length'].must_equal expected.join.size.to_s
-    response[2].to_enum.to_a.must_equal expected
-  end
-end

data/test/spec_content_type.rb

@@ -1,46 +0,0 @@
-require 'minitest/autorun'
-require 'rack/content_type'
-require 'rack/lint'
-require 'rack/mock'
-
-describe Rack::ContentType do
-  def content_type(app, *args)
-    Rack::Lint.new Rack::ContentType.new(app, *args)
-  end
-
-  def request
-    Rack::MockRequest.env_for
-  end
-
-  it "set Content-Type to default text/html if none is set" do
-    app = lambda { |env| [200, {}, "Hello, World!"] }
-    headers = content_type(app).call(request)[1]
-    headers['Content-Type'].must_equal 'text/html'
-  end
-
-  it "set Content-Type to chosen default if none is set" do
-    app = lambda { |env| [200, {}, "Hello, World!"] }
-    headers =
-      content_type(app, 'application/octet-stream').call(request)[1]
-    headers['Content-Type'].must_equal 'application/octet-stream'
-  end
-
-  it "not change Content-Type if it is already set" do
-    app = lambda { |env| [200, {'Content-Type' => 'foo/bar'}, "Hello, World!"] }
-    headers = content_type(app).call(request)[1]
-    headers['Content-Type'].must_equal 'foo/bar'
-  end
-
-  it "detect Content-Type case insensitive" do
-    app = lambda { |env| [200, {'CONTENT-Type' => 'foo/bar'}, "Hello, World!"] }
-    headers = content_type(app).call(request)[1]
-    headers.to_a.select { |k,v| k.downcase == "content-type" }.
-      must_equal [["CONTENT-Type","foo/bar"]]
-  end
-
-  it "not set Content-Type on 304 responses" do
-    app = lambda { |env| [304, {}, []] }
-    response = content_type(app, "text/html").call(request)
-    response[1]['Content-Type'].must_be_nil
-  end
-end