joebadmo-rack-test 0.6.1

data/spec/fixtures/foo.txt

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

data/spec/rack/test/cookie_spec.rb

@@ -0,0 +1,211 @@
+require "spec_helper"
+
+describe Rack::Test::Session do
+
+  context "cookies" do
+    it "keeps a cookie jar" do
+      get "/cookies/show"
+      check 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 "cookie path defaults to the uri of the document that was requested" do
+      pending "See issue rack-test github issue #50" do
+        post "/cookies/default-path", "value" => "cookie"
+        get "/cookies/default-path"
+        check last_request.cookies.should == { "simple"=>"cookie" }
+        get "/cookies/show"
+        check last_request.cookies.should == { }
+      end
+    end
+
+    it "escapes cookie values" do
+      jar = Rack::Test::CookieJar.new
+      jar["value"] = "foo;abc"
+      jar["value"].should == "foo;abc"
+    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
+
+    it "respects cookie domains when no domain is explicitly set" do
+      pending "FIXME: www.example.org should not get the first cookie" 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
+    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"
+      check 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"
+      check 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"
+      check last_request.cookies.should == {}
+
+      get "https://example.com/cookies/show"
+      last_request.cookies.should == { "secure-cookie" => "set" }
+      rack_mock_session.cookie_jar['secure-cookie'].should == '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"
+      check last_request.cookies.should == { "value" => "example" }
+
+      get "http://other.example/cookies/set", "value" => "other"
+      get "http://other.example/cookies/show"
+      check last_request.cookies.should == { "value" => "other" }
+
+      get "http://example.com/cookies/show"
+      last_request.cookies.should == { "value" => "example" }
+    end
+
+    it "keeps one cookie jar for domain and its subdomains" do
+      get "http://example.org/cookies/subdomain"
+      get "http://example.org/cookies/subdomain"
+      last_request.cookies.should == { "count" => "1" }
+
+      get "http://foo.example.org/cookies/subdomain"
+      last_request.cookies.should == { "count" => "2" }
+    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 "skips emtpy string cookies" do
+      set_cookie "value=10\n\nfoo=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"
+      check 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,46 @@
+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
+
+    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 GETs with params" do
+      digest_authorize "alice", "correct-password"
+      response = get "/", "foo" => "bar"
+      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,145 @@
+# encoding: UTF-8
+
+require "spec_helper"
+
+describe Rack::Test::Session do
+
+  def test_file_path
+    File.dirname(__FILE__) + "/../../fixtures/foo.txt"
+  end
+
+  def second_test_file_path
+    File.dirname(__FILE__) + "/../../fixtures/bar.txt"
+  end
+
+  def uploaded_file
+    Rack::Test::UploadedFile.new(test_file_path)
+  end
+
+  def second_uploaded_file
+    Rack::Test::UploadedFile.new(second_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
+
+    it "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 encoded as ISO-8859-1" do
+      post "/", "photo" => uploaded_file, "foo" => "bar", "utf8" => "☃"
+      last_request.POST["foo"].should == "bar"
+
+      if Rack::Test.encoding_aware_strings?
+        last_request.POST["utf8"].should == "☃"
+      else
+        last_request.POST["utf8"].should == "\xE2\x98\x83"
+      end
+    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
+
+
+  context "uploading two files" do
+    it "sends the multipart/form-data content type" do
+      post "/", "photos" => [uploaded_file, second_uploaded_file]
+      last_request.env["CONTENT_TYPE"].should include("multipart/form-data;")
+    end
+
+    it "sends files with the filename" do
+      post "/", "photos" => [uploaded_file, second_uploaded_file]
+      last_request.POST["photos"].collect{|photo| photo[:filename]}.should == ["foo.txt", "bar.txt"]
+    end
+
+    it "sends files with the text/plain MIME type by default" do
+      post "/", "photos" => [uploaded_file, second_uploaded_file]
+      last_request.POST["photos"].collect{|photo| photo[:type]}.should == ["text/plain", "text/plain"]
+    end
+
+    it "sends files with the right names" do
+      post "/", "photos" => [uploaded_file, second_uploaded_file]
+      last_request.POST["photos"].all?{|photo| photo[:name].should == "photos[]" }
+    end
+
+    it "allows mixed content types" do
+      image_file = Rack::Test::UploadedFile.new(test_file_path, "image/jpeg")
+
+      post "/", "photos" => [uploaded_file, image_file]
+      last_request.POST["photos"].collect{|photo| photo[:type]}.should == ["text/plain", "image/jpeg"]
+    end
+
+    it "sends files with a Content-Length in the header" do
+      post "/", "photos" => [uploaded_file, second_uploaded_file]
+      last_request.POST["photos"].all?{|photo| photo[:head].should include("Content-Length: 4") }
+    end
+
+    it "sends both files as Tempfiles" do
+      post "/", "photos" => [uploaded_file, second_uploaded_file]
+      last_request.POST["photos"].all?{|photo| photo[:tempfile].should be_a(::Tempfile) }
+    end
+  end
+end

data/spec/rack/test/uploaded_file_spec.rb

@@ -0,0 +1,22 @@
+require "spec_helper"
+
+describe Rack::Test::UploadedFile do
+  def test_file_path
+    File.dirname(__FILE__) + "/../../fixtures/foo.txt"
+  end
+
+  it "responds to things that Tempfile responds to" do
+    uploaded_file = Rack::Test::UploadedFile.new(test_file_path)
+
+    uploaded_file.should respond_to(:close)
+    uploaded_file.should respond_to(:close!)
+    uploaded_file.should respond_to(:delete)
+    uploaded_file.should respond_to(:length)
+    uploaded_file.should respond_to(:open)
+    uploaded_file.should respond_to(:path)
+    uploaded_file.should respond_to(:size)
+    uploaded_file.should respond_to(:unlink)
+    uploaded_file.should respond_to(:read)
+  end
+
+end

data/spec/rack/test/utils_spec.rb

@@ -0,0 +1,122 @@
+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 an empty string" do
+      build_nested_query(nil).should == ""
+    end
+
+    it "converts hashes with nil values" do
+      build_nested_query(:a => nil).should == "a"
+    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 arrays with brackets '[]' in the name" do
+      build_nested_query("a[]" => [1, 2]).should == "a%5B%5D=1&a%5B%5D=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)
+      check params["submit-name"].should == "Larry"
+      check params["files"][:filename].should == "foo.txt"
+      params["files"][:tempfile].read.should == "bar\n"
+    end
+
+   it "builds multipart bodies from array of files" do
+      files = [Rack::Test::UploadedFile.new(multipart_file("foo.txt")), Rack::Test::UploadedFile.new(multipart_file("bar.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)
+      check params["submit-name"].should == "Larry"
+
+      check params["files"][0][:filename].should == "foo.txt"
+      params["files"][0][:tempfile].read.should == "bar\n"
+
+      check params["files"][1][:filename].should == "bar.txt"
+      params["files"][1][:tempfile].read.should == "baz\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}], "foo" => ['1', '2'])
+
+      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)
+      check params["people"][0]["submit-name"].should == "Larry"
+      check params["people"][0]["files"][:filename].should == "foo.txt"
+      params["people"][0]["files"][:tempfile].read.should == "bar\n"
+      check params["foo"].should == ["1", "2"]
+    end
+
+    it "returns 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