merb-core 1.0.15 → 1.1.0.pre

data/spec10/public/test/mock_request_helper_spec.rb

@@ -0,0 +1,269 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "spec_helper"))
+
+Merb.start :environment => 'test', :log_level => :fatal
+
+Dir[File.join(File.dirname(__FILE__), "controllers/**/*.rb")].each do |f|
+  require f
+end
+
+describe Merb::Test::RequestHelper do
+  
+  describe Merb::Test::RequestHelper::CookieJar do
+    
+    it "should update its values from a request object" do
+      cookie_jar = Merb::Test::RequestHelper::CookieJar.new
+      cookie_jar.should be_empty
+      request = fake_request
+      request.cookies[:foo] = "bar+baz" # escaped by default
+      cookie_jar.update_from_request request
+      cookie_jar[:foo].should == 'bar baz'
+    end
+    
+  end  
+  
+  describe "#dispatch_to" do
+
+    before(:all) do
+      @controller_klass = Merb::Test::DispatchController
+    end
+
+    it "should dispatch to the given controller and action" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:index)
+
+      dispatch_to(@controller_klass, :index)
+    end
+
+    it "should dispatch to the given controller and action with params" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:show)
+
+      controller = dispatch_to(@controller_klass, :show, :name => "Fred")
+      controller.params[:name].should == "Fred"
+    end
+
+    it "should dispatch to the given controller and action with the query string merged into the params" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:show)
+      controller = dispatch_to(@controller_klass, :show, {:name => "Fred"}, {'QUERY_STRING' => "last_name=Jones&age=42"} )
+      
+      controller.params[:name].should == "Fred"
+      controller.params[:last_name].should == "Jones"
+      controller.params[:age].should == "42"   
+    end
+
+    it "should not hit the router to match its route" do
+      Merb::Router.should_not_receive(:match)
+      dispatch_to(@controller_klass, :index)
+    end
+    
+    it "merges :controller into params" do
+      controller = dispatch_to(@controller_klass, :show, :name => "Fred")
+      
+      controller.params[:controller].should == @controller_klass.name.to_const_path
+    end
+    
+    it "merges :action into params" do
+      controller = dispatch_to(@controller_klass, :show, :name => "Fred")
+      
+      controller.params[:action].should == "show"
+    end
+
+    it "should support setting request.raw_post" do
+      controller = dispatch_to(@controller_klass, :show, {}, {:post_body => 'some XML'})
+      controller.request.raw_post.should == 'some XML'
+    end
+  end
+  
+  describe "#dispatch_with_basic_authentication_to" do
+
+    before(:all) do
+      @controller_klass = Merb::Test::DispatchController
+    end
+
+    it "should dispatch to the given controller and action" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:index)
+
+      dispatch_with_basic_authentication_to(@controller_klass, :index, "Fred", "secret")
+    end
+
+    it "should dispatch to the given controller and action with authentication token" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:show)
+
+      controller = dispatch_with_basic_authentication_to(@controller_klass, :show, "Fred", "secret")
+
+      controller.request.env["X_HTTP_AUTHORIZATION"].should == "Basic #{Base64.encode64("Fred:secret")}"
+    end
+    
+    it "should dispatch to the given controller and action with authentication token and params" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:show)
+
+      controller = dispatch_with_basic_authentication_to(@controller_klass, :show, "Fred", "secret", :name => "Fred")
+
+      controller.request.env["X_HTTP_AUTHORIZATION"].should == "Basic #{Base64.encode64("Fred:secret")}"
+      controller.params[:name].should == "Fred"
+    end
+
+    it "should not hit the router to match its route" do
+      Merb::Router.should_not_receive(:match)
+      dispatch_with_basic_authentication_to(@controller_klass, :index, "Fred", "secret")
+    end
+  end
+
+  describe "#get" do
+    before(:each) do 
+      Merb::Router.prepare do |r| 
+        r.resources :spec_helper_controller
+        r.match("/:controller/:action/:custom").to(:controller => ":controller") 
+      end
+    end
+
+    it "should perform the index action when used with a get" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:index)
+      get("/spec_helper_controller")  
+    end
+
+    it "should perform the index action and have params available" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:index)
+      controller = get("/spec_helper_controller", :name => "Harry")
+      controller.params[:name].should == "Harry"    
+    end
+    
+    it "should perform the index action and have params available from the query string" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:index)
+      controller = get("/spec_helper_controller?last_name=Oswald&age=25", :name => "Harry")
+      controller.params[:name].should == "Harry"
+      controller.params[:last_name].should == "Oswald"
+      controller.params[:age].should == "25"
+    end
+
+    it "should evaluate in the context of the controller in the block" do
+      get("/spec_helper_controller") do |controller|
+        controller.class.should == SpecHelperController
+      end    
+    end
+
+    it "should allow for custom router params" do
+      controller = get("/spec_helper_controller/index/my_custom_stuff")
+      controller.params[:custom].should == "my_custom_stuff"    
+    end   
+
+    it "should get the show action" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:show)
+      controller = get("/spec_helper_controller/my_id")
+      controller.params[:id].should == "my_id"    
+    end
+  end
+
+  describe "#post" do
+    before(:each) do
+      Merb::Router.prepare do |r|
+        r.resources :spec_helper_controller
+      end
+    end
+
+    it "should post to the create action" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:create)
+      post("/spec_helper_controller")
+    end
+
+    it "should post to the create action with params" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:create)
+      controller = post("/spec_helper_controller", :name => "Harry")
+      controller.params[:name].should == "Harry"
+    end
+  end
+
+  describe "#put" do
+    before(:each) do
+      Merb::Router.prepare do |r|
+        r.resources :spec_helper_controller
+      end
+    end
+    it "should put to the update action" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:update)
+      put("/spec_helper_controller/1")
+    end
+
+    it "should put to the update action with params" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:update)
+      controller = put("/spec_helper_controller/my_id", :name => "Harry")
+      controller.params[:name].should == "Harry"
+      controller.params[:id].should   == "my_id"
+    end
+  end
+
+  describe "#delete" do
+    before(:each) do
+      Merb::Router.prepare do |r|
+        r.resources :spec_helper_controller
+      end
+    end
+    it "should put to the update action" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:destroy)
+      delete("/spec_helper_controller/1")
+    end
+
+    it "should put to the update action with params" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:destroy)
+      controller = delete("/spec_helper_controller/my_id", :name => "Harry")
+      controller.params[:name].should == "Harry"
+      controller.params[:id].should   == "my_id"
+    end
+  end
+  
+  describe "#request" do
+    before(:each) do 
+      Merb::Router.prepare do |r| 
+        r.namespace :namespaced do |namespaced|
+          namespaced.resources :spec_helper_controller
+        end
+      end
+    end
+    
+    it "should support setting request.raw_post" do
+      controller = mock_request("/namespaced/spec_helper_controller", {}, {:post_body => 'some XML'})
+      controller.request.raw_post.should == 'some XML'
+    end
+
+    it "should get namespaced index action" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:index)
+      controller = mock_request("/namespaced/spec_helper_controller")
+      controller.class.should == Namespaced::SpecHelperController
+    end
+
+    it "should make the post body available in the request on deferred routing" do
+      Merb::Router.prepare do |r|
+        r.match('/xmlrpc').defer_to do |request, params|
+          request.raw_post.should == 'XMLRPC request body'
+          {:controller => 'spec_helper_controller', :action => :index}
+        end
+      end
+
+      mock_request('/xmlrpc', {}, {:post_body => 'XMLRPC request body'})
+    end
+  end
+  
+end
+
+module Merb::Test::RequestHelper
+  describe FakeRequest, ".new(env = {}, req = StringIO.new)" do
+    it "should create request with default enviroment, minus rack.input" do
+      @mock = FakeRequest.new
+      @mock.env.except('rack.input').should == FakeRequest::DEFAULT_ENV
+    end
+
+    it "should override default env values passed in HTTP format" do
+      @mock = FakeRequest.new('HTTP_ACCEPT' => 'nothing')
+      @mock.env['HTTP_ACCEPT'].should == 'nothing'
+    end
+
+    it "should override default env values passed in symbol format" do
+      @mock = FakeRequest.new(:http_accept => 'nothing')
+      @mock.env['HTTP_ACCEPT'].should == 'nothing'
+    end
+
+    it "should set rack input to an empty StringIO" do
+      @mock = FakeRequest.new
+      @mock.env['rack.input'].should be_kind_of(StringIO)
+      @mock.env['rack.input'].read.should == ''
+    end
+  end
+end

data/spec10/public/test/multipart_request_helper_spec.rb

@@ -0,0 +1,159 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "spec_helper"))
+
+Merb.start :environment => 'test', :log_level => :fatal
+
+Dir[File.join(File.dirname(__FILE__), "controllers/**/*.rb")].each do |f|
+  require f
+end
+
+describe Merb::Test::MultipartRequestHelper do
+
+  describe "#dispatch_multipart_to" do
+
+    before(:all) do 
+      @controller_klass = Merb::Test::DispatchController
+    end
+
+    it "should dispatch to the given controller and action" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:index)
+
+      dispatch_multipart_to(@controller_klass, :index)    
+    end
+
+    it "should dispatch to the given controller and action with params" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:show)
+
+      controller = dispatch_multipart_to(@controller_klass, :show, :name => "Fred")
+      controller.params[:name].should == "Fred"
+    end
+
+    it "should handle a file object when used as a param" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:show)
+      file_name = File.join(File.dirname(__FILE__), "multipart_upload_text_file.txt")
+      File.open( file_name ) do |file|
+        controller = dispatch_multipart_to(@controller_klass, :show, :my_file => file)
+        file_params = controller.params[:my_file]
+        file_params[:content_type].should == "text/plain"
+        file_params[:size].should == File.size(file_name)
+        file_params[:tempfile].should be_a_kind_of(Tempfile)
+        file_params[:filename].should == "multipart_upload_text_file.txt"
+      end
+    end
+  end
+
+  describe "#multipart_post" do
+    before(:each) do
+      Merb::Router.prepare do |r|
+        r.resources :spec_helper_controller
+      end
+    end
+  
+    it "should post to the create action" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:create)
+      multipart_post("/spec_helper_controller")
+    end
+  
+    it "should post to the create action with params" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:create)
+      controller = multipart_post("/spec_helper_controller", :name => "Harry")
+      controller.params[:name].should == "Harry"
+    end
+  
+    it "should upload a file to the action using multipart" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:create)
+      file_name = File.join(File.dirname(__FILE__), "multipart_upload_text_file.txt")
+      File.open( file_name ) do |file|
+        controller = multipart_post("/spec_helper_controller", :my_file => file)
+        file_params = controller.params[:my_file]
+        file_params[:content_type].should == "text/plain"
+        file_params[:size].should == File.size(file_name)
+        file_params[:tempfile].should be_a_kind_of(Tempfile)
+        file_params[:filename].should == "multipart_upload_text_file.txt"
+      end
+    end
+  
+  end
+
+  describe "#multipart_put" do
+    before(:each) do
+      Merb::Router.prepare do |r|
+        r.resources :spec_helper_controller
+      end
+    end
+    it "should put to the update action multipart" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:update)
+      multipart_put("/spec_helper_controller/1")
+    end
+  
+    it "should put to the update action with multipart params" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:update)
+      controller = multipart_put("/spec_helper_controller/my_id", :name => "Harry")
+      controller.params[:name].should == "Harry"
+      controller.params[:id].should   == "my_id"
+    end
+  
+    it "should upload a file to the action using multipart" do
+      Merb::Test::ControllerAssertionMock.should_receive(:called).with(:update)
+      file_name = File.join(File.dirname(__FILE__), "multipart_upload_text_file.txt")
+      File.open( file_name ) do |file|
+        controller = multipart_put("/spec_helper_controller/my_id", :my_file => file)
+        controller.params[:id].should == "my_id"
+        file_params = controller.params[:my_file]
+        file_params[:content_type].should == "text/plain"
+        file_params[:size].should == File.size(file_name)
+        file_params[:tempfile].should be_a_kind_of(Tempfile)
+        file_params[:filename].should == "multipart_upload_text_file.txt"
+      end
+    end
+  end
+end
+
+module Merb::Test::MultipartRequestHelper
+  describe Param, '#to_multipart' do
+    it "should represent the key and value correctly" do
+      param = Param.new('foo', 'bar')
+      param.to_multipart.should == %(Content-Disposition: form-data; name="foo"\r\n\r\nbar\r\n)
+    end
+  end
+
+  describe FileParam, '#to_multipart' do
+    it "should represent the key, filename and content correctly" do
+      param = FileParam.new('foo', '/bar.txt', 'baz')
+      param.to_multipart.should == %(Content-Disposition: form-data; name="foo"; filename="/bar.txt"\r\nContent-Type: text/plain\r\n\r\nbaz\r\n)
+    end
+  end
+
+  describe Post, '#push_params(params) param parsing' do
+    before(:each) do
+      @fake_return_param = mock('fake return_param')
+    end
+
+    it "should create Param from params when param doesn't respond to read" do
+      params = { 'normal' => 'normal_param' }
+      Param.should_receive(:new).with('normal', 'normal_param').and_return(@fake_return_param)
+      Post.new.push_params(params)
+    end
+  
+    it "should create FileParam from params when param does response to read" do
+      file_param = mock('file param')
+      file_param.should_receive(:read).and_return('file contents')
+      file_param.should_receive(:path).and_return('file.txt')
+      params = { 'file' => file_param }
+      FileParam.should_receive(:new).with('file', 'file.txt', 'file contents').and_return(@fake_return_param)
+      Post.new.push_params(params)
+    end
+  end
+  
+  describe Post, '#to_multipart' do
+    it "should create a multipart request from the params" do
+      file_param = mock('file param')
+      file_param.should_receive(:read).and_return('file contents')
+      file_param.should_receive(:path).and_return('file.txt')
+      params = { 'file' => file_param, 'normal' => 'normal_param' }
+      multipart = Post.new(params)
+      query, content_type = multipart.to_multipart
+      content_type.should == "multipart/form-data, boundary=----------0xKhTmLbOuNdArY"
+      query.should == "------------0xKhTmLbOuNdArY\r\nContent-Disposition: form-data; name=\"file\"; filename=\"file.txt\"\r\nContent-Type: text/plain\r\n\r\nfile contents\r\n------------0xKhTmLbOuNdArY\r\nContent-Disposition: form-data; name=\"normal\"\r\n\r\nnormal_param\r\n------------0xKhTmLbOuNdArY--"
+    end
+  end
+end

data/spec10/public/test/multipart_upload_text_file.txt

@@ -0,0 +1 @@
+File With Some Text In It For Testing Multipart Uploads

data/spec10/public/test/request_helper_spec.rb

@@ -0,0 +1,124 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "spec_helper"))
+
+Merb.start :environment => 'test', :log_level => :fatal
+
+require File.dirname(__FILE__) / "controllers/request_controller"
+
+describe Merb::Test::RequestHelper do
+  
+  before(:each) do
+    Merb::Controller._default_cookie_domain = "example.org"
+    
+    Merb::Router.prepare do
+      with(:controller => "merb/test/request_controller") do
+        match("/set/short/long/read").to(:action => "get")
+        match("/:action(/:junk)", :junk => ".*").register
+      end
+    end
+  end
+  
+  it "should dispatch a request using GET by defalt" do
+    request("/request_method").should have_body("Method - GET")
+  end
+  
+  it "should work with have_selector" do
+    request("/document").should have_selector("div div")
+  end
+  
+  it "should work with have_xpath" do
+    request("/document").should have_xpath("//div/div")
+  end
+  
+  it "should work with have_content" do
+    request("/request_method").should contain("Method")
+  end
+  
+  it "should persist cookies across sequential cookie setting requests" do
+    request("/counter").should have_body("1")
+    request("/counter").should have_body("2")
+  end
+  
+  it "should persist cookies across requests that don't return any cookie headers" do
+    request("/counter").should have_body("1")
+    request("/void").should    have_body("Void")
+    request('/counter').should have_body("2")
+  end
+  
+  it "should delete cookies from the jar" do
+    request("/counter").should have_body("1")
+    request("/delete").should  have_body("Delete")
+    request("/counter").should have_body("1")
+  end
+  
+  it "should be able to disable the cookie jar" do
+    request("/counter", :jar => nil).should have_body("1")
+    request("/counter", :jar => nil).should have_body("1")
+    request("/counter").should have_body("1")
+    request("/counter").should have_body("2")
+  end
+  
+  it "should be able to specify separate jars" do
+    request("/counter", :jar => :one).should have_body("1")
+    request("/counter", :jar => :two).should have_body("1")
+    request("/counter", :jar => :one).should have_body("2")
+    request("/counter", :jar => :two).should have_body("2")
+  end
+  
+  it "should respect cookie domains when no domain is explicitly set" do
+    request("http://example.org/counter").should     have_body("1")
+    request("http://www.example.org/counter").should have_body("2")
+    request("http://example.org/counter").should     have_body("3")
+    request("http://www.example.org/counter").should have_body("4")
+  end
+  
+  it "should respect the domain set in the cookie" do
+    request("http://example.org/domain").should     have_body("1")
+    request("http://foo.example.org/domain").should have_body("1")
+    request("http://example.org/domain").should     have_body("1")
+    request("http://foo.example.org/domain").should have_body("2")
+  end
+  
+  it "should respect the path set in the cookie" do
+    request("/path").should      have_body("1")
+    request("/path/zomg").should have_body("1")
+    request("/path").should      have_body("1")
+    request("/path/zomg").should have_body("2")
+  end
+  
+  it "should use the most specific path cookie" do
+    request("/set/short")
+    request("/set/short/long")
+    request("/set/short/long/read").should have_body("/set/short/long")
+  end
+  
+  it "should use the most specific path cookie even if it was defined first" do
+    request("/set/short/long")
+    request("/set/short")
+    request("/set/short/long/read").should have_body("/set/short/long")
+  end
+  
+  it "should leave the least specific cookie intact when specifying a more specific path" do
+    request("/set/short")
+    request("/set/short/long/zomg/what/hi")
+    request("/set/short/long/read").should have_body("/set/short")
+  end
+  
+  it "should use the most specific domain cookie" do
+    request("http://test.com/domain_set")
+    request("http://one.test.com/domain_set")
+    request("http://one.test.com/domain_get").should have_body("one.test.com")
+  end
+  
+  it "should keep the less specific domain cookie" do
+    request("http://test.com/domain_set").should be_successful
+    request("http://one.test.com/domain_set").should be_successful
+    request("http://test.com/domain_get").should have_body("test.com")
+  end
+  
+  it "should respect the expiration" do
+    request("/expires").should have_body("1")
+    sleep(1)
+    request("/expires").should have_body("1")
+  end
+  
+end

data/spec10/public/test/route_helper_spec.rb

@@ -0,0 +1,75 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "spec_helper"))
+
+class TestController < Merb::Controller
+  def get(id = nil); end
+  def post; end
+end
+
+describe Merb::Test::RouteHelper do
+  include Merb::Test::RouteHelper
+  
+  before(:each) do
+    Merb::Router.prepare do
+      match("/").defer_to do |request, params|
+        { :controller => 'test_controller', :action => request.raw_post } unless request.raw_post.blank?
+      end
+      match("/", :method => :get).to(:controller => "test_controller", :action => "get").name(:getter)
+      match("/", :method => :post).to(:controller => "test_controller", :action => "post")
+      match("/:id").to(:controller => "test_controller", :action => "get").name(:with_id)
+      default_routes
+    end
+  end
+  
+  describe "#url" do
+    it "should use Merb::Router" do
+      url(:getter).should == "/"
+    end
+    
+    it "should work with a parameters hash" do
+      url(:with_id, :id => 123).should == "/123"
+    end
+    
+    it "should turn extra hash items into query params" do
+      generated_url = url(:getter, :id => 123, :color => 'red', :size => 'large')
+      lambda {
+        generated_url.match(/\bid=123\b/) &&
+        generated_url.match(/\bsize=large\b/) &&
+        generated_url.match(/\bcolor=red\b/)
+      }.call.should_not be_nil
+    end
+    
+    it "should remove items with nil values from query params" do
+      url(:getter, :color => nil, :size => 'large').should == "/?size=large"
+    end
+    
+    it "should remove items with nil values from query params when named route isn't specified" do
+      url(:controller => 'cont', :action => 'act', :color => nil, :size => 'large').should == "/cont/act?size=large"
+    end
+  end
+  
+  describe "#request_to" do
+    it "should GET if no method is given" do
+      request_to("/")[:action].should == "get"
+    end
+    
+    it "should return a hash" do
+      Hash.should === request_to("/")
+    end
+    
+    it "should contain the controller in the result" do
+      request_to("/")[:controller].should == "test_controller"
+    end
+    
+    it "should contain the action in the result" do
+      request_to("/")[:action].should == "get"
+    end
+    
+    it "should contain any parameters in the result" do
+      request_to("/123")[:id].should == "123"
+    end
+
+    it "should play nice with raw_post in deferred routing" do
+      request_to("/", :post, {:post_body => 'deferred'})[:action].should == 'deferred'
+    end
+  end
+end