rack 1.6.11 → 2.2.0

data/test/spec_lint.rb

@@ -1,550 +0,0 @@
-require 'stringio'
-require 'tempfile'
-require 'rack/lint'
-require 'rack/mock'
-
-describe Rack::Lint do
-  def env(*args)
-    Rack::MockRequest.env_for("/", *args)
-  end
-
-  should "pass valid request" do
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       [200, {"Content-type" => "test/plain", "Content-length" => "3"}, ["foo"]]
-                     }).call(env({}))
-    }.should.not.raise
-  end
-
-  should "notice fatal errors" do
-    lambda { Rack::Lint.new(nil).call }.should.raise(Rack::Lint::LintError).
-      message.should.match(/No env given/)
-  end
-
-  should "notice environment errors" do
-    lambda { Rack::Lint.new(nil).call 5 }.should.raise(Rack::Lint::LintError).
-      message.should.match(/not a Hash/)
-
-    lambda {
-      e = env
-      e.delete("REQUEST_METHOD")
-      Rack::Lint.new(nil).call(e)
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/missing required key REQUEST_METHOD/)
-
-    lambda {
-      e = env
-      e.delete("SERVER_NAME")
-      Rack::Lint.new(nil).call(e)
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/missing required key SERVER_NAME/)
-
-
-    lambda {
-      Rack::Lint.new(nil).call(env("HTTP_CONTENT_TYPE" => "text/plain"))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/contains HTTP_CONTENT_TYPE/)
-
-    lambda {
-      Rack::Lint.new(nil).call(env("HTTP_CONTENT_LENGTH" => "42"))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/contains HTTP_CONTENT_LENGTH/)
-
-    lambda {
-      Rack::Lint.new(nil).call(env("FOO" => Object.new))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/non-string value/)
-
-    lambda {
-      Rack::Lint.new(nil).call(env("rack.version" => "0.2"))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/must be an Array/)
-
-    lambda {
-      Rack::Lint.new(nil).call(env("rack.url_scheme" => "gopher"))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/url_scheme unknown/)
-
-    lambda {
-      Rack::Lint.new(nil).call(env("rack.session" => []))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.equal("session [] must respond to store and []=")
-
-    lambda {
-      Rack::Lint.new(nil).call(env("rack.logger" => []))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.equal("logger [] must respond to info")
-
-    lambda {
-      Rack::Lint.new(nil).call(env("rack.multipart.buffer_size" => 0))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.equal("rack.multipart.buffer_size must be an Integer > 0 if specified")
-
-    lambda {
-      Rack::Lint.new(nil).call(env("rack.multipart.tempfile_factory" => Tempfile))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.equal("rack.multipart.tempfile_factory must respond to #call")
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-        env['rack.multipart.tempfile_factory'].call("testfile", "text/plain")
-      }).call(env("rack.multipart.tempfile_factory" => lambda { |filename, content_type| Object.new }))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.equal("rack.multipart.tempfile_factory return value must respond to #<<")
-
-    lambda {
-      Rack::Lint.new(nil).call(env("REQUEST_METHOD" => "FUCKUP?"))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/REQUEST_METHOD/)
-
-    lambda {
-      Rack::Lint.new(nil).call(env("SCRIPT_NAME" => "howdy"))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/must start with/)
-
-    lambda {
-      Rack::Lint.new(nil).call(env("PATH_INFO" => "../foo"))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/must start with/)
-
-    lambda {
-      Rack::Lint.new(nil).call(env("CONTENT_LENGTH" => "xcii"))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/Invalid CONTENT_LENGTH/)
-
-    lambda {
-      e = env
-      e.delete("PATH_INFO")
-      e.delete("SCRIPT_NAME")
-      Rack::Lint.new(nil).call(e)
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/One of .* must be set/)
-
-    lambda {
-      Rack::Lint.new(nil).call(env("SCRIPT_NAME" => "/"))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/cannot be .* make it ''/)
-  end
-
-  should "notice input errors" do
-    lambda {
-      Rack::Lint.new(nil).call(env("rack.input" => ""))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/does not respond to #gets/)
-
-    lambda {
-      input = Object.new
-      def input.binmode?
-        false
-      end
-      Rack::Lint.new(nil).call(env("rack.input" => input))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/is not opened in binary mode/)
-
-    lambda {
-      input = Object.new
-      def input.external_encoding
-        result = Object.new
-        def result.name
-          "US-ASCII"
-        end
-        result
-      end
-      Rack::Lint.new(nil).call(env("rack.input" => input))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/does not have ASCII-8BIT as its external encoding/)
-  end
-
-  should "notice error errors" do
-    lambda {
-      Rack::Lint.new(nil).call(env("rack.errors" => ""))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/does not respond to #puts/)
-  end
-
-  should "notice status errors" do
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       ["cc", {}, ""]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/must be >=100 seen as integer/)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       [42, {}, ""]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/must be >=100 seen as integer/)
-  end
-
-  should "notice header errors" do
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       [200, Object.new, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.equal("headers object should respond to #each, but doesn't (got Object as headers)")
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       [200, {true=>false}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.equal("header key must be a string, was TrueClass")
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       [200, {"Status" => "404"}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/must not contain Status/)
-
-    # From RFC 7230:<F24><F25>
-    # Most HTTP header field values are defined using common syntax
-    # components (token, quoted-string, and comment) separated by
-    # whitespace or specific delimiting characters.  Delimiters are chosen
-    # from the set of US-ASCII visual characters not allowed in a token
-    # (DQUOTE and "(),/:;<=>?@[\]{}").
-    # 
-    #   token          = 1*tchar
-    # 
-    #   tchar          = "!" / "#" / "$" / "%" / "&" / "'" / "*"
-    #                 / "+" / "-" / "." / "^" / "_" / "`" / "|" / "~"
-    #                 / DIGIT / ALPHA
-    #                 ; any VCHAR, except delimiters
-    invalid_headers = 0.upto(31).map(&:chr) + %W<( ) , / : ; < = > ? @ [ \\ ] { } \x7F>
-    invalid_headers.each do |invalid_header|
-      lambda {
-        Rack::Lint.new(lambda { |env|
-          [200, {invalid_header => "text/plain"}, []]
-        }).call(env({}))
-      }.should.raise(Rack::Lint::LintError, "on invalid header: #{invalid_header}").
-      message.should.equal("invalid header name: #{invalid_header}")
-    end
-    valid_headers = 0.upto(127).map(&:chr) - invalid_headers
-    valid_headers.each do |valid_header|
-      lambda {
-        Rack::Lint.new(lambda { |env|
-          [200, {valid_header => "text/plain"}, []]
-        }).call(env({}))
-      }.should.not.raise(Rack::Lint::LintError, "on valid header: #{valid_header}")
-    end
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       [200, {"Foo" => Object.new}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.equal("a header value must be a String, but the value of 'Foo' is a Object")
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       [200, {"Foo" => [1, 2, 3]}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.equal("a header value must be a String, but the value of 'Foo' is a Array")
-
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       [200, {"Foo-Bar" => "text\000plain"}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/invalid header/)
-
-    # line ends (010) should be allowed in header values.
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       [200, {"Foo-Bar" => "one\ntwo\nthree", "Content-Length" => "0", "Content-Type" => "text/plain" }, []]
-                     }).call(env({}))
-    }.should.not.raise(Rack::Lint::LintError)
-
-    # non-Hash header responses should be allowed
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       [200, [%w(Content-Type text/plain), %w(Content-Length 0)], []]
-                     }).call(env({}))
-    }.should.not.raise(TypeError)
-  end
-
-  should "notice content-type errors" do
-    # lambda {
-    #   Rack::Lint.new(lambda { |env|
-    #                    [200, {"Content-length" => "0"}, []]
-    #                  }).call(env({}))
-    # }.should.raise(Rack::Lint::LintError).
-    #   message.should.match(/No Content-Type/)
-
-    [100, 101, 204, 205, 304].each do |status|
-      lambda {
-        Rack::Lint.new(lambda { |env|
-                         [status, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                       }).call(env({}))
-      }.should.raise(Rack::Lint::LintError).
-        message.should.match(/Content-Type header found/)
-    end
-  end
-
-  should "notice content-length errors" do
-    [100, 101, 204, 205, 304].each do |status|
-      lambda {
-        Rack::Lint.new(lambda { |env|
-                         [status, {"Content-length" => "0"}, []]
-                       }).call(env({}))
-      }.should.raise(Rack::Lint::LintError).
-        message.should.match(/Content-Length header found/)
-    end
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       [200, {"Content-type" => "text/plain", "Content-Length" => "1"}, []]
-                     }).call(env({}))[2].each { }
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/Content-Length header was 1, but should be 0/)
-  end
-
-  should "notice body errors" do
-    lambda {
-      body = Rack::Lint.new(lambda { |env|
-                               [200, {"Content-type" => "text/plain","Content-length" => "3"}, [1,2,3]]
-                             }).call(env({}))[2]
-      body.each { |part| }
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/yielded non-string/)
-  end
-
-  should "notice input handling errors" do
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].gets("\r\n")
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/gets called with arguments/)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].read(1, 2, 3)
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/read called with too many arguments/)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].read("foo")
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/read called with non-integer and non-nil length/)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].read(-1)
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/read called with a negative length/)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].read(nil, nil)
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/read called with non-String buffer/)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].read(nil, 1)
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/read called with non-String buffer/)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].rewind(0)
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/rewind called with arguments/)
-
-    weirdio = Object.new
-    class << weirdio
-      def gets
-        42
-      end
-
-      def read
-        23
-      end
-
-      def each
-        yield 23
-        yield 42
-      end
-
-      def rewind
-        raise Errno::ESPIPE, "Errno::ESPIPE"
-      end
-    end
-
-    eof_weirdio = Object.new
-    class << eof_weirdio
-      def gets
-        nil
-      end
-
-      def read(*args)
-        nil
-      end
-
-      def each
-      end
-
-      def rewind
-      end
-    end
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].gets
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env("rack.input" => weirdio))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/gets didn't return a String/)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].each { |x| }
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env("rack.input" => weirdio))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/each didn't yield a String/)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].read
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env("rack.input" => weirdio))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/read didn't return nil or a String/)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].read
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env("rack.input" => eof_weirdio))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/read\(nil\) returned nil on EOF/)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].rewind
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env("rack.input" => weirdio))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/rewind raised Errno::ESPIPE/)
-
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].close
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/close must not be called/)
-  end
-
-  should "notice error handling errors" do
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.errors"].write(42)
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/write not called with a String/)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.errors"].close
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({}))
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/close must not be called/)
-  end
-
-  should "notice HEAD errors" do
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       [200, {"Content-type" => "test/plain", "Content-length" => "3"}, []]
-                     }).call(env({"REQUEST_METHOD" => "HEAD"}))
-    }.should.not.raise
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       [200, {"Content-type" => "test/plain", "Content-length" => "3"}, ["foo"]]
-                     }).call(env({"REQUEST_METHOD" => "HEAD"}))[2].each { }
-    }.should.raise(Rack::Lint::LintError).
-      message.should.match(/body was given for HEAD/)
-  end
-
-  should "pass valid read calls" do
-    hello_str = "hello world"
-    hello_str.force_encoding("ASCII-8BIT") if hello_str.respond_to? :force_encoding
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].read
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({"rack.input" => StringIO.new(hello_str)}))
-    }.should.not.raise(Rack::Lint::LintError)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].read(0)
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({"rack.input" => StringIO.new(hello_str)}))
-    }.should.not.raise(Rack::Lint::LintError)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].read(1)
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({"rack.input" => StringIO.new(hello_str)}))
-    }.should.not.raise(Rack::Lint::LintError)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].read(nil)
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({"rack.input" => StringIO.new(hello_str)}))
-    }.should.not.raise(Rack::Lint::LintError)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].read(nil, '')
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({"rack.input" => StringIO.new(hello_str)}))
-    }.should.not.raise(Rack::Lint::LintError)
-
-    lambda {
-      Rack::Lint.new(lambda { |env|
-                       env["rack.input"].read(1, '')
-                       [201, {"Content-type" => "text/plain", "Content-length" => "0"}, []]
-                     }).call(env({"rack.input" => StringIO.new(hello_str)}))
-    }.should.not.raise(Rack::Lint::LintError)
-  end
-end
-
-describe "Rack::Lint::InputWrapper" do
-  should "delegate :rewind to underlying IO object" do
-    io = StringIO.new("123")
-    wrapper = Rack::Lint::InputWrapper.new(io)
-    wrapper.read.should.equal "123"
-    wrapper.read.should.equal ""
-    wrapper.rewind
-    wrapper.read.should.equal "123"
-  end
-end

data/test/spec_lobster.rb

@@ -1,58 +0,0 @@
-require 'rack/lobster'
-require 'rack/lint'
-require 'rack/mock'
-
-module LobsterHelpers
-  def lobster
-    Rack::MockRequest.new Rack::Lint.new(Rack::Lobster.new)
-  end
-
-  def lambda_lobster
-    Rack::MockRequest.new Rack::Lint.new(Rack::Lobster::LambdaLobster)
-  end
-end
-
-describe Rack::Lobster::LambdaLobster do
-  extend LobsterHelpers
-  
-  should "be a single lambda" do
-    Rack::Lobster::LambdaLobster.should.be.kind_of Proc
-  end
-
-  should "look like a lobster" do
-    res = lambda_lobster.get("/")
-    res.should.be.ok
-    res.body.should.include "(,(,,(,,,("
-    res.body.should.include "?flip"
-  end
-
-  should "be flippable" do
-    res = lambda_lobster.get("/?flip")
-    res.should.be.ok
-    res.body.should.include "(,,,(,,(,("
-  end
-end
-
-describe Rack::Lobster do
-  extend LobsterHelpers
-  
-  should "look like a lobster" do
-    res = lobster.get("/")
-    res.should.be.ok
-    res.body.should.include "(,(,,(,,,("
-    res.body.should.include "?flip"
-    res.body.should.include "crash"
-  end
-
-  should "be flippable" do
-    res = lobster.get("/?flip=left")
-    res.should.be.ok
-    res.body.should.include "),,,),,),)"
-  end
-
-  should "provide crashing for testing purposes" do
-    lambda {
-      lobster.get("/?flip=crash")
-    }.should.raise
-  end
-end