josh-rack-test 0.4.1

data/spec/fixtures/config.ru

@@ -0,0 +1,3 @@
+require "fake_app"
+
+run Rack::Test::FakeApp

data/spec/fixtures/fake_app.rb

@@ -0,0 +1,115 @@
+require "sinatra/base"
+
+module Rack
+  module Test
+
+    class FakeApp < Sinatra::Default
+      head "/" do
+        "meh"
+      end
+
+      get "/" do
+        "Hello, GET: #{params.inspect}"
+      end
+
+      get "/redirect" do
+        redirect "/redirected"
+      end
+
+      get "/redirected" do
+        "You've been redirected"
+      end
+
+      get "/void" do
+        [200, {}, ""]
+      end
+
+      get "/cookies/show" do
+        request.cookies.inspect
+      end
+
+      get "/COOKIES/show" do
+        request.cookies.inspect
+      end
+
+      get "/not-cookies/show" do
+        request.cookies.inspect
+      end
+
+      get "/cookies/set-secure" do
+        raise if params["value"].nil?
+
+        response.set_cookie("secure-cookie", :value => params["value"], :secure => true)
+        "Set"
+      end
+
+      get "/cookies/set-simple" do
+        raise if params["value"].nil?
+
+        response.set_cookie "simple", params["value"]
+        "Set"
+      end
+
+      get "/cookies/delete" do
+        response.delete_cookie "value"
+      end
+
+      get "/cookies/count" do
+        old_value = request.cookies["count"].to_i || 0
+        new_value = (old_value + 1).to_s
+
+        response.set_cookie("count", :value => new_value)
+        new_value
+      end
+
+      get "/cookies/set" do
+        raise if params["value"].nil?
+
+        response.set_cookie("value", {
+          :value => params["value"],
+          :path => "/cookies",
+          :expires => Time.now + 10
+        })
+        "Set"
+      end
+
+      get "/cookies/domain" do
+        old_value = request.cookies["count"].to_i || 0
+        new_value = (old_value + 1).to_s
+
+        response.set_cookie("count", :value => new_value, :domain => "localhost.com")
+        new_value
+      end
+
+      get "/cookies/set-uppercase" do
+        raise if params["value"].nil?
+
+        response.set_cookie("VALUE", {
+          :value => params["value"],
+          :path => "/cookies",
+          :expires => Time.now + 10
+        })
+        "Set"
+      end
+
+      get "/cookies/set-multiple" do
+        response.set_cookie("key1", :value => "value1")
+        response.set_cookie("key2", :value => "value2")
+        "Set"
+      end
+
+      post "/" do
+        "Hello, POST: #{params.inspect}"
+      end
+
+      put "/" do
+        "Hello, PUT: #{params.inspect}"
+      end
+
+      delete "/" do
+        "Hello, DELETE: #{params.inspect}"
+      end
+    end
+
+  end
+end

data/spec/fixtures/foo.txt

@@ -0,0 +1 @@
+bar

data/spec/rack/test/cookie_spec.rb

@@ -0,0 +1,182 @@
+require "spec_helper"
+
+describe Rack::Test::Session do
+  def have_body(string)
+    simple_matcher "have body #{string.inspect}" do |response|
+      response.body.should == string
+    end
+  end
+
+  context "cookies" do
+    it "keeps a cookie jar" do
+      get "/cookies/show"
+      last_request.cookies.should == {}
+
+      get "/cookies/set", "value" => "1"
+      get "/cookies/show"
+      last_request.cookies.should == { "value" => "1" }
+    end
+
+    it "doesn't send expired cookies" do
+      get "/cookies/set", "value" => "1"
+      now = Time.now
+      Time.stub!(:now => now + 60)
+      get "/cookies/show"
+      last_request.cookies.should == {}
+    end
+
+    it "doesn't send cookies with the wrong domain" do
+      get "http://www.example.com/cookies/set", "value" => "1"
+      get "http://www.other.example/cookies/show"
+      last_request.cookies.should == {}
+    end
+
+    it "doesn't send cookies with the wrong path" do
+      get "/cookies/set", "value" => "1"
+      get "/not-cookies/show"
+      last_request.cookies.should == {}
+    end
+
+    it "persists cookies across requests that don't return any cookie headers" do
+      get "/cookies/set", "value" => "1"
+      get "/void"
+      get "/cookies/show"
+      last_request.cookies.should == { "value" => "1" }
+    end
+
+    it "deletes cookies" do
+      get "/cookies/set", "value" => "1"
+      get "/cookies/delete"
+      get "/cookies/show"
+      last_request.cookies.should == { }
+    end
+
+    xit "respects cookie domains when no domain is explicitly set" do
+      request("http://example.org/cookies/count").should     have_body("1")
+      request("http://www.example.org/cookies/count").should have_body("1")
+      request("http://example.org/cookies/count").should     have_body("2")
+      request("http://www.example.org/cookies/count").should have_body("2")
+    end
+
+    it "treats domains case insensitively" do
+      get "http://example.com/cookies/set", "value" => "1"
+      get "http://EXAMPLE.COM/cookies/show"
+      last_request.cookies.should == { "value" => "1" }
+    end
+
+    it "treats paths case sensitively" do
+      get "/cookies/set", "value" => "1"
+      get "/COOKIES/show"
+      last_request.cookies.should == {}
+    end
+
+    it "prefers more specific cookies" do
+      get "http://example.com/cookies/set",     "value" => "domain"
+      get "http://sub.example.com/cookies/set", "value" => "sub"
+
+      get "http://sub.example.com/cookies/show"
+      last_request.cookies.should == { "value" => "sub" }
+
+      get "http://example.com/cookies/show"
+      last_request.cookies.should == { "value" => "domain" }
+    end
+
+    it "treats cookie names case insensitively" do
+      get "/cookies/set", "value" => "lowercase"
+      get "/cookies/set-uppercase", "value" => "UPPERCASE"
+      get "/cookies/show"
+      last_request.cookies.should == { "VALUE" => "UPPERCASE" }
+    end
+
+    it "defaults the domain to the request domain" do
+      get "http://example.com/cookies/set-simple", "value" => "cookie"
+      get "http://example.com/cookies/show"
+      last_request.cookies.should == { "simple" => "cookie" }
+
+      get "http://other.example/cookies/show"
+      last_request.cookies.should == {}
+    end
+
+    it "defaults the domain to the request path up to the last slash" do
+      get "/cookies/set-simple", "value" => "1"
+      get "/not-cookies/show"
+      last_request.cookies.should == {}
+    end
+
+    it "supports secure cookies" do
+      get "https://example.com/cookies/set-secure", "value" => "set"
+      get "http://example.com/cookies/show"
+      last_request.cookies.should == {}
+
+      get "https://example.com/cookies/show"
+      last_request.cookies.should == { "secure-cookie" => "set" }
+    end
+
+    it "keeps separate cookie jars for different domains" do
+      get "http://example.com/cookies/set", "value" => "example"
+      get "http://example.com/cookies/show"
+      last_request.cookies.should == { "value" => "example" }
+
+      get "http://other.example/cookies/set", "value" => "other"
+      get "http://other.example/cookies/show"
+      last_request.cookies.should == { "value" => "other" }
+
+      get "http://example.com/cookies/show"
+      last_request.cookies.should == { "value" => "example" }
+    end
+
+    it "allows cookies to be cleared" do
+      get "/cookies/set", "value" => "1"
+      clear_cookies
+      get "/cookies/show"
+      last_request.cookies.should == {}
+    end
+
+    it "allow cookies to be set" do
+      set_cookie "value=10"
+      get "/cookies/show"
+      last_request.cookies.should == { "value" => "10" }
+    end
+
+    it "allows an array of cookies to be set" do
+      set_cookie ["value=10", "foo=bar"]
+      get "/cookies/show"
+      last_request.cookies.should == { "value" => "10", "foo" => "bar" }
+    end
+
+    it "parses multiple cookies properly" do
+      get "/cookies/set-multiple"
+      get "/cookies/show"
+      last_request.cookies.should == { "key1" => "value1", "key2" => "value2" }
+    end
+
+    it "supports multiple sessions" do
+      with_session(:first) do
+        get "/cookies/set", "value" => "1"
+        get "/cookies/show"
+        last_request.cookies.should == { "value" => "1" }
+      end
+
+      with_session(:second) do
+        get "/cookies/show"
+        last_request.cookies.should == { }
+      end
+    end
+
+    it "uses :default as the default session name" do
+      get "/cookies/set", "value" => "1"
+      get "/cookies/show"
+      last_request.cookies.should == { "value" => "1" }
+
+      with_session(:default) do
+        get "/cookies/show"
+        last_request.cookies.should == { "value" => "1" }
+      end
+    end
+
+    it "accepts explicitly provided cookies" do
+      request "/cookies/show", :cookie => "value=1"
+      last_request.cookies.should == { "value" => "1" }
+    end
+  end
+end

data/spec/rack/test/digest_auth_spec.rb

@@ -0,0 +1,48 @@
+require "spec_helper"
+
+describe Rack::Test::Session do
+  context "HTTP Digest authentication" do
+
+    def app
+      app = Rack::Auth::Digest::MD5.new(Rack::Test::FakeApp.new) do |username|
+        { 'alice' => 'correct-password' }[username]
+      end
+      app.realm = 'WallysWorld'
+      app.opaque = 'this-should-be-secret'
+      app
+    end
+
+    def be_challenge
+      simple_matcher "a HTTP Digest challenge response" do |response|
+        response.status == 401 &&
+        response['WWW-Authenticate'] =~ /^Digest / &&
+        response.body.empty?
+      end
+    end
+
+    it 'incorrectly authenticates GETs' do
+      digest_authorize 'foo', 'bar'
+      get '/'
+      last_response.should be_challenge
+    end
+
+    it "correctly authenticates GETs" do
+      digest_authorize "alice", "correct-password"
+      response = get "/"
+      response.should be_ok
+    end
+
+    it "correctly authenticates POSTs" do
+      digest_authorize "alice", "correct-password"
+      response = post "/"
+      response.should be_ok
+    end
+
+    it "returns a re-challenge if authenticating incorrectly" do
+      digest_authorize "alice", "incorrect-password"
+      response = get "/"
+      response.should be_challenge
+    end
+
+  end
+end

data/spec/rack/test/multipart_spec.rb

@@ -0,0 +1,85 @@
+require "spec_helper"
+
+describe Rack::Test::Session do
+
+  def test_file_path
+    File.dirname(__FILE__) + "/../../fixtures/foo.txt"
+  end
+
+  def uploaded_file
+    Rack::Test::UploadedFile.new(test_file_path)
+  end
+
+  context "uploading a file" do
+    it "sends the multipart/form-data content type" do
+      post "/", "photo" => uploaded_file
+      last_request.env["CONTENT_TYPE"].should include("multipart/form-data;")
+    end
+
+    it "sends regular params" do
+      post "/", "photo" => uploaded_file, "foo" => "bar"
+      last_request.POST["foo"].should == "bar"
+    end
+
+    it "sends nested params" do
+      post "/", "photo" => uploaded_file, "foo" => {"bar" => "baz"}
+      last_request.POST["foo"]["bar"].should == "baz"
+    end
+
+    it "sends multiple nested params" do
+      post "/", "photo" => uploaded_file, "foo" => {"bar" => {"baz" => "bop"}}
+      last_request.POST["foo"]["bar"]["baz"].should == "bop"
+    end
+
+    xit "sends params with arrays" do
+      post "/", "photo" => uploaded_file, "foo" => ["1", "2"]
+      last_request.POST["foo[]"].should == ["1", "2"]
+    end
+
+    it "sends params with encoding sensitive values" do
+      post "/", "photo" => uploaded_file, "foo" => "bar? baz"
+      last_request.POST["foo"].should == "bar? baz"
+    end
+
+    it "sends params with parens in names" do
+      post "/", "photo" => uploaded_file, "foo(1i)" => "bar"
+      last_request.POST["foo(1i)"].should == "bar"
+    end
+
+    it "sends params with encoding sensitive names" do
+      post "/", "photo" => uploaded_file, "foo bar" => "baz"
+      last_request.POST["foo bar"].should == "baz"
+    end
+
+    it "sends files with the filename" do
+      post "/", "photo" => uploaded_file
+      last_request.POST["photo"][:filename].should == "foo.txt"
+    end
+
+    it "sends files with the text/plain MIME type by default" do
+      post "/", "photo" => uploaded_file
+      last_request.POST["photo"][:type].should == "text/plain"
+    end
+
+    it "sends files with the right name" do
+      post "/", "photo" => uploaded_file
+      last_request.POST["photo"][:name].should == "photo"
+    end
+
+    it "allows overriding the content type" do
+      post "/", "photo" => Rack::Test::UploadedFile.new(test_file_path, "image/jpeg")
+      last_request.POST["photo"][:type].should == "image/jpeg"
+    end
+
+    it "sends files with a Content-Length in the header" do
+      post "/", "photo" => uploaded_file
+      last_request.POST["photo"][:head].should include("Content-Length: 4")
+    end
+
+    it "sends files as Tempfiles" do
+      post "/", "photo" => uploaded_file
+      last_request.POST["photo"][:tempfile].should be_a(::Tempfile)
+    end
+  end
+
+end

data/spec/rack/test/utils_spec.rb

@@ -0,0 +1,93 @@
+require "spec_helper"
+
+describe Rack::Test::Utils do
+  include Rack::Test::Utils
+
+  describe "build_nested_query" do
+    it "converts empty strings to =" do
+      build_nested_query("").should == "="
+    end
+
+    it "converts nil to =" do
+      build_nested_query(nil).should == "="
+    end
+
+    it "converts hashes" do
+      build_nested_query(:a => 1).should == "a=1"
+    end
+
+    it "converts hashes with multiple keys" do
+      hash = { :a => 1, :b => 2 }
+      ["a=1&b=2", "b=2&a=1"].should include(build_nested_query(hash))
+    end
+
+    it "converts arrays with one element" do
+      build_nested_query(:a => [1]).should == "a[]=1"
+    end
+
+    it "converts arrays with multiple elements" do
+      build_nested_query(:a => [1, 2]).should == "a[]=1&a[]=2"
+    end
+
+    it "converts nested hashes" do
+      build_nested_query(:a => { :b => 1 }).should == "a[b]=1"
+    end
+
+    it "converts arrays nested in a hash" do
+      build_nested_query(:a => { :b => [1, 2] }).should == "a[b][]=1&a[b][]=2"
+    end
+
+    it "converts arrays of hashes" do
+      build_nested_query(:a => [{ :b => 2}, { :c => 3}]).should == "a[][b]=2&a[][c]=3"
+    end
+  end
+
+  describe "build_multipart" do
+    it "builds multipart bodies" do
+      files = Rack::Test::UploadedFile.new(multipart_file("foo.txt"))
+      data  = build_multipart("submit-name" => "Larry", "files" => files)
+
+      options = {
+        "CONTENT_TYPE" => "multipart/form-data; boundary=#{Rack::Test::MULTIPART_BOUNDARY}",
+        "CONTENT_LENGTH" => data.length.to_s,
+        :input => StringIO.new(data)
+      }
+      env = Rack::MockRequest.env_for("/", options)
+      params = Rack::Utils::Multipart.parse_multipart(env)
+      params["submit-name"].should == "Larry"
+      params["files"][:filename].should == "foo.txt"
+      params["files"][:tempfile].read.should == "bar\n"
+    end
+
+    it "builds nested multipart bodies" do
+      files = Rack::Test::UploadedFile.new(multipart_file("foo.txt"))
+      data  = build_multipart("people" => [{"submit-name" => "Larry", "files" => files}])
+
+      options = {
+        "CONTENT_TYPE" => "multipart/form-data; boundary=#{Rack::Test::MULTIPART_BOUNDARY}",
+        "CONTENT_LENGTH" => data.length.to_s,
+        :input => StringIO.new(data)
+      }
+      env = Rack::MockRequest.env_for("/", options)
+      params = Rack::Utils::Multipart.parse_multipart(env)
+      params["people"][0]["submit-name"].should == "Larry"
+      params["people"][0]["files"][:filename].should == "foo.txt"
+      params["people"][0]["files"][:tempfile].read.should == "bar\n"
+    end
+
+    it "should return nil if no UploadedFiles were used" do
+      data = build_multipart("people" => [{"submit-name" => "Larry", "files" => "contents"}])
+      data.should be_nil
+    end
+
+    it "raises ArgumentErrors if params is not a Hash" do
+      lambda {
+        build_multipart("foo=bar")
+      }.should raise_error(ArgumentError, "value must be a Hash")
+    end
+
+    def multipart_file(name)
+      File.join(File.dirname(__FILE__), "..", "..", "fixtures", name.to_s)
+    end
+  end
+end