merb-core 0.9.2

data/spec/public/test/controllers/controller_assertion_mock.rb

@@ -0,0 +1,7 @@
+class Merb::Test::ControllerAssertionMock
+  
+  def self.called(action)
+    return action
+  end
+  
+end

data/spec/public/test/controllers/dispatch_controller.rb

@@ -0,0 +1,11 @@
+class Merb::Test::DispatchController < Merb::Controller
+  
+  def index
+    Merb::Test::ControllerAssertionMock.called(:index)
+  end
+  
+  def show
+    Merb::Test::ControllerAssertionMock.called(:show)
+  end
+  
+end

data/spec/public/test/controllers/spec_helper_controller.rb

@@ -0,0 +1,30 @@
+class SpecHelperController < Merb::Controller
+  
+  def index
+    Merb::Test::ControllerAssertionMock.called(:index)
+  end
+  
+  def show
+    Merb::Test::ControllerAssertionMock.called(:show)
+  end
+  
+  def edit
+    Merb::Test::ControllerAssertionMock.called(:edit)
+  end
+  
+  def new
+    Merb::Test::ControllerAssertionMock.called(:new)
+  end
+  
+  def create
+    Merb::Test::ControllerAssertionMock.called(:create)
+  end
+  
+  def update
+    Merb::Test::ControllerAssertionMock.called(:update)
+  end
+  
+  def destroy
+    Merb::Test::ControllerAssertionMock.called(:destroy)
+  end  
+end

data/spec/public/test/multipart_request_helper_spec.rb

@@ -0,0 +1,159 @@
+require 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(File)
+        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(File)
+        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(File)
+        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/spec/public/test/multipart_upload_text_file.txt

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

data/spec/public/test/request_helper_spec.rb

@@ -0,0 +1,153 @@
+require 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 "#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 not hit the router to match it's route" do
+      Merb::Router.should_not_receive(:match)
+      dispatch_to(@controller_klass, :index)
+    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 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
+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/spec/public/test/route_helper_spec.rb

@@ -0,0 +1,54 @@
+require 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
+  before(:each) do
+    Merb::Router.prepare do |r|
+      r.match("/", :method => :get).to(:controller => "test_controller", :action => "get").name(:getter)
+      r.match("/", :method => :post).to(:controller => "test_controller", :action => "post")
+      r.match("/:id").to(:controller => "test_controller", :action => "get").name(:with_id)
+    end
+  end
+  
+  describe "#url" do
+    it "should use Merb::Router" do
+      url(:getter).should == "/"
+    end
+    
+    it "should work with a model as the parameter" do
+      model = mock(:model)
+      model.stub!(:id).and_return("123")
+      url(:with_id, model).should == "/123"
+    end
+    
+    it "should work with a parameters hash" do
+      url(:with_id, :id => 123).should == "/123"
+    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
+  end
+end

data/spec/public/test/route_matchers_spec.rb

@@ -0,0 +1,133 @@
+require File.join(File.dirname(__FILE__), "..", "..", "spec_helper")
+
+Merb.start :environment => 'test', :log_level => :fatal
+
+class TestController < Merb::Controller
+  def get(id = nil); end
+  def post; end
+end
+
+class IDish
+  attr_accessor :id
+  alias_method :to_param, :id
+  
+  def initialize(id)
+    @id = id
+  end
+end
+
+describe Merb::Test::Rspec::RouteMatchers do
+  include Merb::Test::RouteHelper
+  
+  before(:each) do
+    Merb::Router.prepare do |r|
+      r.match("/", :method => :get).to(:controller => "test_controller", :action => "get").name(:getter)
+      r.match("/", :method => :post).to(:controller => "test_controller", :action => "post")
+      r.match("/:id").to(:controller => "test_controller", :action => "get").name(:with_id)
+    end
+  end
+  
+  describe "#route_to" do
+    it "should work with the request_to helper" do
+      request_to("/", :get).should route_to(TestController, :get)
+      request_to("/", :post).should_not route_to(TestController, :get)
+    end
+    
+    it "should work with the url helper and ParamMatcher" do
+      idish = IDish.new(rand(1000).to_s)
+      request_to(url(:with_id, idish)).should route_to(TestController, :get).with(idish)
+    end
+  end
+  
+  module Merb::Test::Rspec::RouteMatchers
+  
+    describe RouteToMatcher do
+      
+      it "should work with snake cased controllers" do
+        RouteToMatcher.new(TestController, :get).matches?(:controller => "test_controller", :action => "get").should be_true
+      end
+      
+      it "should work with camel cased controllers" do
+        RouteToMatcher.new(TestController, :get).matches?(:controller => "TestController", :action => "get").should be_true
+      end
+      
+      it "should work with symbol or string controller name" do
+        RouteToMatcher.new(TestController, :get).matches?(:controller => "test_controller", :action => "get").should be_true
+        RouteToMatcher.new(TestController, :get).matches?(:controller => :test_controller, :action => :get)
+      end
+      
+      it "should not pass if the controllers do not match" do
+        RouteToMatcher.new(TestController, :get).matches?(:controller => "other_controller", :action => "get").should be_false
+      end
+      
+      it "should not pass if the actions do not match" do
+        RouteToMatcher.new(TestController, :get).matches?(:controller => "test_controller", :action => "post").should be_false
+      end
+      
+      it "should not pass if the parameters do not the ParameterMatcher" do
+        route_matcher = RouteToMatcher.new(TestController, :get)
+        route_matcher.with(:id => "123")
+        
+        route_matcher.matches?(:controller => "test_case", :action => "get", :id => "456").should be_false
+      end
+      
+      describe "#with" do
+        it "should add a ParameterMatcher" do
+          ParameterMatcher.should_receive(:new).with(:id => "123")
+          
+          route_matcher = RouteToMatcher.new(TestController, :get)
+          route_matcher.with(:id => "123")
+        end
+      end
+      
+      describe "#failure_message" do
+        it "should include the expected controller and action" do
+          RouteToMatcher.new(TestController, :any_action).failure_message.should include("TestController#any_action")
+        end
+        
+        it "should include the target controller and action in camel case" do
+          matcher = RouteToMatcher.new(TestController, :any_action)
+          matcher.matches?(:controller => "target_controller", :action => "target_action")
+          matcher.failure_message.should include("TargetController#target_action")
+        end
+      end
+      
+      describe "#negative_failure_message" do
+        it "should include the expected controller and action" do
+          RouteToMatcher.new(TestController, :any_action).negative_failure_message.should include("TestController#any_action")
+        end
+      end
+    end
+  
+    describe ParameterMatcher do
+      it "should work with a Hash as the parameter argument" do
+        ParameterMatcher.new(:param => "abc").matches?(:param => "abc").should be_true
+      end
+      
+      it "should work with an object as the parameter argument" do
+        ParameterMatcher.new(IDish.new(1234)).matches?(:id => 1234).should be_true
+      end
+      
+      describe "#failure_message" do
+        it "should include the expected parameters hash" do
+          parameter_hash = {:parent_id => "123", :child_id => "abc"}
+          ParameterMatcher.new(parameter_hash).failure_message.should include(parameter_hash.inspect)
+        end
+        
+        it "should include the actual parameters hash" do
+          parameter_hash = {:parent_id => "123", :child_id => "abc"}
+          matcher = ParameterMatcher.new(:id => 123)
+          matcher.matches?(parameter_hash)
+          matcher.failure_message.should include(parameter_hash.inspect)
+        end
+      end
+      
+      describe "#negative_failure_message" do
+        it "should include the expected parameters hash" do
+          parameter_hash = {:parent_id => "123", :child_id => "abc"}
+          ParameterMatcher.new(parameter_hash).negative_failure_message.should include(parameter_hash.inspect)
+        end
+      end
+    end
+  end
+end