shoulda-matchers 1.4.2 → 1.5.0

data/spec/shoulda/matchers/action_controller/filter_param_matcher_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe Shoulda::Matchers::ActionController::FilterParamMatcher do
+  it 'accepts filtering a filtered parameter' do
+    filter(:secret)
+
+    nil.should filter_param(:secret)
+  end
+
+  it 'rejects filtering an unfiltered parameter' do
+    filter(:secret)
+    matcher = filter_param(:other)
+
+    matcher.matches?(nil).should be_false
+
+    matcher.failure_message_for_should.should =~ /Expected other to be filtered.*secret/
+  end
+
+  def filter(param)
+    Rails.application.config.filter_parameters = [param]
+  end
+end

data/spec/shoulda/matchers/action_controller/redirect_to_matcher_spec.rb

@@ -0,0 +1,42 @@
+require 'spec_helper'
+
+describe Shoulda::Matchers::ActionController::RedirectToMatcher do
+  context 'a controller that redirects' do
+    it 'accepts redirecting to that url' do
+      controller_redirecting_to('/some/url').should redirect_to('/some/url')
+    end
+
+    it 'rejects redirecting to a different url' do
+      controller_redirecting_to('/some/url').
+        should_not redirect_to('/some/other/url')
+    end
+
+    it 'accepts redirecting to that url in a block' do
+      controller_redirecting_to('/some/url').
+        should redirect_to('somewhere') { '/some/url' }
+    end
+
+    it 'rejects redirecting to a different url in a block' do
+      controller_redirecting_to('/some/url').
+        should_not redirect_to('somewhere else') { '/some/other/url' }
+    end
+
+    def controller_redirecting_to(url)
+      build_response { redirect_to url }
+    end
+  end
+
+  context 'a controller that does not redirect' do
+    it 'rejects redirecting to a url' do
+      controller = build_response { render :text => 'hello' }
+
+      controller.should_not redirect_to('/some/url')
+    end
+  end
+
+  it 'provides the correct description when provided a block' do
+    matcher = redirect_to('somewhere else') { '/some/other/url' }
+
+    matcher.description.should == 'redirect to somewhere else'
+  end
+end

data/spec/shoulda/matchers/action_controller/render_template_matcher_spec.rb

@@ -0,0 +1,78 @@
+require 'spec_helper'
+
+describe Shoulda::Matchers::ActionController::RenderTemplateMatcher do
+  include ActionController::TemplateAssertions
+
+  context 'a controller that renders a template' do
+    it 'accepts rendering that template' do
+      controller_with_show.should render_template(:show)
+    end
+
+    it 'rejects rendering a different template' do
+      controller_with_show.should_not render_template(:index)
+    end
+
+    it 'accepts rendering that template in the given context' do
+      controller_with_show.should render_template(:show).in_context(self)
+    end
+
+    it 'rejects rendering a different template in the given context' do
+      controller_with_show.should_not render_template(:index).in_context(self)
+    end
+
+    def controller_with_show
+      build_response(:action => 'show') { render }
+    end
+  end
+
+  context 'a controller that renders a partial' do
+    it 'accepts rendering that partial' do
+      controller_with_customer_partial.
+        should render_template(:partial => '_customer')
+    end
+
+    it 'rejects rendering a different template' do
+      controller_with_customer_partial.
+        should_not render_template(:partial => '_client')
+    end
+
+    it 'accepts rendering that template in the given context' do
+      controller_with_customer_partial.
+        should render_template(:partial => '_customer').in_context(self)
+    end
+
+    it 'rejects rendering a different template in the given context' do
+      controller_with_customer_partial.
+        should_not render_template(:partial => '_client').in_context(self)
+    end
+
+    def controller_with_customer_partial
+      build_response(:partial => '_customer') { render :partial => 'customer' }
+    end
+  end
+
+  context 'a controller that does not render partials' do
+    it 'accepts not rendering a partial' do
+      controller = build_response(:action => 'show') { render }
+
+      controller.should render_template(:partial => false)
+    end
+  end
+
+  context 'a controller that renders a partial several times' do
+    it 'accepts rendering that partial twice' do
+      controller = build_response(:partial => '_customer') do
+        render :partial => 'customer', :collection => [1,2]
+      end
+
+      controller.should render_template(:partial => '_customer', :count => 2)
+    end
+  end
+
+  context 'a controller that does not render a template' do
+    it 'rejects rendering a template' do
+      build_response { render :nothing => true }.
+        should_not render_template(:show)
+    end
+  end
+end

data/spec/shoulda/matchers/action_controller/render_with_layout_matcher_spec.rb

@@ -0,0 +1,58 @@
+require 'spec_helper'
+
+describe Shoulda::Matchers::ActionController::RenderWithLayoutMatcher do
+  include ActionController::TemplateAssertions
+
+  context 'a controller that renders with a layout' do
+    it 'accepts rendering with any layout' do
+      controller_with_wide_layout.should render_with_layout
+    end
+
+    it 'accepts rendering with that layout' do
+      controller_with_wide_layout.should render_with_layout(:wide)
+    end
+
+    it 'rejects rendering with another layout' do
+      controller_with_wide_layout.should_not render_with_layout(:other)
+    end
+
+    def controller_with_wide_layout
+      create_view('layouts/wide.html.erb', 'some content, <%= yield %>')
+      build_response { render :layout => 'wide' }
+    end
+  end
+
+  context 'a controller that renders without a layout' do
+
+    it 'rejects rendering with a layout' do
+      controller_without_layout = build_response { render :layout => false }
+
+      controller_without_layout.should_not render_with_layout
+    end
+  end
+
+  context 'a controller that renders a partial' do
+    it 'rejects rendering with a layout' do
+      controller_with_partial = build_response { render :partial => 'partial' }
+
+      controller_with_partial.should_not render_with_layout
+    end
+  end
+
+  context 'given a context with layouts' do
+    it 'accepts that layout in that context' do
+      set_in_context_layout('happy')
+
+      controller_without_layout.should render_with_layout('happy').in_context(self)
+    end
+
+    def set_in_context_layout(layout)
+      @layouts = Hash.new(0)
+      @layouts[layout] = 1
+    end
+
+    def controller_without_layout
+      build_response { render :layout => false }
+    end
+  end
+end

data/spec/shoulda/matchers/action_controller/respond_with_content_type_matcher_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe Shoulda::Matchers::ActionController::RespondWithContentTypeMatcher do
+  it 'generates the correct description' do
+    expected = 'respond with content type of application/xml'
+
+    respond_with_content_type(:xml).description.should == expected
+  end
+
+  it 'accepts responding with content type as symbol' do
+    xml_controller.should respond_with_content_type(:xml)
+  end
+
+  it 'accepts responding with qualified MIME-style content type' do
+    xml_controller.should respond_with_content_type('application/xml')
+  end
+
+  it 'accepts responding with a regex matching the content type' do
+    xml_controller.should respond_with_content_type(/xml/)
+  end
+
+  it 'rejects responding with another content type' do
+    xml_controller.should_not respond_with_content_type(:json)
+  end
+
+  def xml_controller
+    build_response do
+      render :xml => { :user => 'thoughtbot' }.to_xml
+    end
+  end
+end

data/spec/shoulda/matchers/action_controller/respond_with_matcher_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe Shoulda::Matchers::ActionController::RespondWithMatcher do
+  statuses = { :success => 200, :redirect => 301, :missing => 404, :error => 500,
+    :not_implemented => 501 }
+
+  statuses.each do |human_name, numeric_code|
+    context "a controller responding with #{human_name}" do
+      it 'accepts responding with a numeric response code' do
+        controller_with_status(numeric_code).should respond_with(numeric_code)
+      end
+
+      it 'accepts responding with a symbol response code' do
+        controller_with_status(numeric_code).should respond_with(human_name)
+      end
+
+      it 'rejects responding with another status' do
+        another_status = statuses.except(human_name).keys.first
+
+        controller_with_status(numeric_code).
+          should_not respond_with(another_status)
+      end
+    end
+  end
+
+  def controller_with_status(status)
+    build_response do
+      render :text => 'text', :status => status
+    end
+  end
+end

data/spec/shoulda/matchers/action_controller/route_matcher_spec.rb

@@ -0,0 +1,65 @@
+require 'spec_helper'
+
+describe Shoulda::Matchers::ActionController::RouteMatcher do
+  context 'given a controller with a defined glob url' do
+    it 'accepts glob route' do
+      controller = define_controller('Examples').new
+
+      define_routes do
+        match 'examples/*id', :to => 'examples#example'
+      end
+
+      controller.should route(:get, '/examples/foo/bar').
+        to(:action => 'example', :id => 'foo/bar')
+    end
+  end
+
+  context 'given a controller with a defined route' do
+
+    it 'accepts routing the correct path to the correct parameters' do
+      route_examples_to_examples.should route(:get, '/examples/1').
+        to(:action => 'example', :id => '1')
+    end
+
+    it 'accepts a symbol controller' do
+      route_examples_to_examples
+      Object.new.should route(:get, '/examples/1').
+        to(:controller => :examples, :action => 'example', :id => '1')
+    end
+
+    it 'accepts a symbol action' do
+      route_examples_to_examples.should route(:get, '/examples/1').
+        to(:action => :example, :id => '1')
+    end
+
+    it 'accepts a non-string parameter' do
+      route_examples_to_examples.should route(:get, '/examples/1').
+        to(:action => 'example', :id => 1)
+    end
+
+    it 'rejects an undefined route' do
+      route_examples_to_examples.
+        should_not route(:get, '/bad_route').to(:var => 'value')
+    end
+
+    it 'rejects a route for another controller' do
+      route_examples_to_examples
+      other = define_controller('Other').new
+      other.should_not route(:get, '/examples/1').
+        to(:action => 'example', :id => '1')
+    end
+
+    it 'rejects a route for different parameters' do
+      route_examples_to_examples.should_not route(:get, '/examples/1').
+        to(:action => 'other', :id => '1')
+    end
+
+    def route_examples_to_examples
+      define_routes do
+        get 'examples/:id', :to => 'examples#example'
+      end
+
+      define_controller('Examples').new
+    end
+  end
+end

data/spec/shoulda/matchers/action_controller/set_session_matcher_spec.rb

@@ -0,0 +1,51 @@
+require 'spec_helper'
+
+describe Shoulda::Matchers::ActionController::SetSessionMatcher do
+  context 'a controller that sets a session variable' do
+    it 'accepts assigning to that variable' do
+      controller_with_session(:var => 'hi').should set_session(:var)
+    end
+
+    it 'accepts assigning the correct value to that variable' do
+      controller_with_session(:var => 'hi').should set_session(:var).to('hi')
+    end
+
+    it 'rejects assigning another value to that variable' do
+      controller_with_session(:var => 'hi').should_not set_session(:var).to('other')
+    end
+
+    it 'rejects assigning to another variable' do
+      controller_with_session(:var => 'hi').should_not set_session(:other)
+    end
+
+    it 'accepts assigning nil to another variable' do
+      controller_with_session(:var => 'hi').should set_session(:other).to(nil)
+    end
+
+    it 'accepts assigning false to that variable' do
+      controller_with_session(:var => false).should set_session(:var).to(false)
+    end
+
+    it 'accepts assigning to the same value in the test context' do
+      expected = 'value'
+
+      controller_with_session(:var => expected).
+        should set_session(:var).in_context(self).to { expected }
+    end
+
+    it 'rejects assigning to the another value in the test context' do
+      expected = 'other'
+
+      controller_with_session(:var => 'unexpected').
+        should_not set_session(:var).in_context(self).to { expected }
+    end
+
+    def controller_with_session(session_hash)
+      build_response do
+        session_hash.each do |key, value|
+          session[key] = value
+        end
+      end
+    end
+  end
+end

data/spec/shoulda/matchers/action_controller/set_the_flash_matcher_spec.rb

@@ -0,0 +1,153 @@
+require 'spec_helper'
+
+describe Shoulda::Matchers::ActionController::SetTheFlashMatcher do
+  it 'fails with unmatchable #to' do
+    expect { set_the_flash.to(1) }.to raise_error('cannot match against 1')
+  end
+
+  context 'a controller that sets a flash message' do
+    it 'accepts setting any flash message' do
+      controller_with_flash(:notice => 'hi').should set_the_flash
+    end
+
+    it 'accepts setting the exact flash message' do
+      controller_with_flash(:notice => 'hi').should set_the_flash.to('hi')
+    end
+
+    it 'accepts setting a matched flash message' do
+      controller_with_flash(:notice => 'hello').should set_the_flash.to(/he/)
+    end
+
+    it 'rejects setting a different flash message' do
+      controller_with_flash(:notice => 'hi').
+        should_not set_the_flash.to('other')
+    end
+
+    it 'rejects setting a different pattern' do
+      controller_with_flash(:notice => 'hi').
+        should_not set_the_flash.to(/other/)
+    end
+  end
+
+  context 'a controller that sets a flash.now message' do
+    it 'rejects setting any flash message' do
+      controller_with_flash_now.should_not set_the_flash
+    end
+
+    it 'accepts setting any flash.now message' do
+      controller_with_flash_now.should set_the_flash.now
+    end
+
+    it 'accepts setting the exact flash.now message' do
+      controller_with_flash_now(:notice => 'hi').
+        should set_the_flash.now.to('hi')
+    end
+
+    it 'accepts setting a matched flash.now message' do
+      controller_with_flash_now(:notice => 'flasher').
+        should set_the_flash.now.to(/lash/)
+    end
+
+    it 'rejects setting a different flash.now message' do
+      controller_with_flash_now(:notice => 'hi').
+        should_not set_the_flash.now.to('other')
+    end
+
+    it 'rejects setting a different flash.now pattern' do
+      controller_with_flash_now(:notice => 'hi').
+        should_not set_the_flash.now.to(/other/)
+    end
+  end
+
+  context 'a controller that sets flash messages for multiple keys' do
+    it 'accepts flash message for either key' do
+      controller = controller_with_flash(:notice => 'one', :alert => 'two')
+
+      controller.should set_the_flash[:notice]
+      controller.should set_the_flash[:alert]
+    end
+
+    it 'rejects a flash message that is not one of the set keys' do
+      controller_with_flash(:notice => 'one', :alert => 'two').
+        should_not set_the_flash[:warning]
+    end
+
+    it 'accepts exact flash message of notice' do
+      controller_with_flash(:notice => 'one', :alert => 'two').
+        should set_the_flash[:notice].to('one')
+    end
+
+    it 'accepts setting a matched flash message of notice' do
+      controller_with_flash(:notice => 'one', :alert => 'two').
+        should set_the_flash[:notice].to(/on/)
+    end
+
+    it 'rejects setting a different flash message of notice' do
+      controller_with_flash(:notice => 'one', :alert => 'two').
+        should_not set_the_flash[:notice].to('other')
+    end
+
+    it 'rejects setting a different pattern' do
+      controller_with_flash(:notice => 'one', :alert => 'two').
+        should_not set_the_flash[:notice].to(/other/)
+    end
+  end
+
+  context 'a controller that sets flash and flash.now' do
+    it 'accepts setting any flash.now message' do
+      controller = build_response do
+        flash.now[:notice] = 'value'
+        flash[:success] = 'great job'
+      end
+
+      controller.should set_the_flash.now
+      controller.should set_the_flash
+    end
+
+    it 'accepts setting a matched flash.now message' do
+      controller = build_response do
+        flash.now[:notice] = 'value'
+        flash[:success] = 'great job'
+      end
+
+      controller.should set_the_flash.now.to(/value/)
+      controller.should set_the_flash.to(/great/)
+    end
+
+    it 'rejects setting a different flash.now message' do
+      controller = build_response do
+        flash.now[:notice] = 'value'
+        flash[:success] = 'great job'
+      end
+
+      controller.should_not set_the_flash.now.to('other')
+      controller.should_not set_the_flash.to('other')
+    end
+  end
+
+  context 'a controller that does not set a flash message' do
+    it 'rejects setting any flash message' do
+      controller_with_no_flashes.should_not set_the_flash
+    end
+  end
+
+  def controller_with_no_flashes
+    build_response
+  end
+
+  def controller_with_flash(flash_hash)
+    build_response do
+      flash_hash.each do |key, value|
+        flash[key] = value
+      end
+    end
+  end
+
+  def controller_with_flash_now(flash_hash = { :notice => 'hi' })
+    build_response do
+      flash_hash.each do |key, value|
+        flash.now[key] = value
+      end
+    end
+  end
+end