remarkable_rails 3.1.8 → 3.1.9

data/spec/action_controller/macro_stubs_spec.rb

@@ -1,54 +1,54 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-
-# Define a metaclass in the Object because we are going to need it.
-class Object; def metaclass; class << self; self; end; end; end
-
-describe 'MacroStubs' do
-  controller_name 'tasks'
-  mock_models :task
-
-  def current_id; '37'; end
-
-  describe 'mock_models' do
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+# Define a metaclass in the Object because we are going to need it.
+class Object; def metaclass; class << self; self; end; end; end
+
+describe 'MacroStubs' do
+  controller_name 'tasks'
+  mock_models :task
+
+  def current_id; '37'; end
+
+  describe 'mock_models' do
     before(:each) do
-      self.class.metaclass.send(:undef_method, :projects_proc) if self.class.respond_to?(:projects_proc)
-      self.class.metaclass.send(:undef_method, :project_proc)  if self.class.respond_to?(:project_proc)
-      self.class.send(:undef_method, :mock_project)            if self.respond_to?(:mock_project)
-    end
+      self.class.metaclass.send(:undef_method, :projects_proc) if self.class.respond_to?(:projects_proc)
+      self.class.metaclass.send(:undef_method, :project_proc)  if self.class.respond_to?(:project_proc)
+      self.class.send(:undef_method, :mock_project)            if self.respond_to?(:mock_project)
+    end
 
     it 'should alias model_proc to mock_model' do
       self.class.respond_to?(:mock_project).should be_false
-      self.class.respond_to?(:mock_projects).should be_false
-      self.class.mock_models :project
+      self.class.respond_to?(:mock_projects).should be_false
+      self.class.mock_models :project
       self.class.respond_to?(:mock_project).should be_true
       self.class.respond_to?(:mock_projects).should be_true
     end
-
-    it 'should create a class singular proc method' do
-      self.class.respond_to?(:project_proc).should be_false
-      self.class.mock_models :project
-      self.class.respond_to?(:project_proc).should be_true
+
+    it 'should create a class singular proc method' do
+      self.class.respond_to?(:project_proc).should be_false
+      self.class.mock_models :project
+      self.class.respond_to?(:project_proc).should be_true
     end
 
-    it 'should create a class plural proc method' do
-      self.class.respond_to?(:projects_proc).should be_false
-      self.class.mock_models :project
-      self.class.respond_to?(:projects_proc).should be_true
-    end
-
-    it 'should create an instance mock method' do
-      self.respond_to?(:mock_project).should be_false
-      self.class.mock_models :project
-      self.respond_to?(:mock_project).should be_true
-    end
-
-    it 'should create just an instance method when :class_method is false' do
-      self.class.respond_to?(:project_proc).should be_false
-      self.respond_to?(:mock_project).should be_false
-      self.class.mock_models :project, :class_method => false
-      self.class.respond_to?(:project_proc).should be_false
-      self.respond_to?(:mock_project).should be_true
-    end
+    it 'should create a class plural proc method' do
+      self.class.respond_to?(:projects_proc).should be_false
+      self.class.mock_models :project
+      self.class.respond_to?(:projects_proc).should be_true
+    end
+
+    it 'should create an instance mock method' do
+      self.respond_to?(:mock_project).should be_false
+      self.class.mock_models :project
+      self.respond_to?(:mock_project).should be_true
+    end
+
+    it 'should create just an instance method when :class_method is false' do
+      self.class.respond_to?(:project_proc).should be_false
+      self.respond_to?(:mock_project).should be_false
+      self.class.mock_models :project, :class_method => false
+      self.class.respond_to?(:project_proc).should be_false
+      self.respond_to?(:mock_project).should be_true
+    end
 
     it 'should allow the mock class to be set' do
       self.class.mock_model :project, :as => "::Admin::Project"
@@ -56,24 +56,24 @@ describe 'MacroStubs' do
         mock_project
       }.should raise_error(NameError, "uninitialized constant Admin")
     end
-
-    it 'should create procs which evals to a mock' do
-      proc = self.class.task_proc
-      proc.should be_kind_of(Proc)
-
-      @task.should be_nil
-      instance_eval(&proc).should == mock_task
-      @task.should_not be_nil
+
+    it 'should create procs which evals to a mock' do
+      proc = self.class.task_proc
+      proc.should be_kind_of(Proc)
+
+      @task.should be_nil
+      instance_eval(&proc).should == mock_task
+      @task.should_not be_nil
     end
 
-    it 'should create procs which evals to an array of mocks' do
-      proc = self.class.tasks_proc
-      proc.should be_kind_of(Proc)
-
-      @task.should be_nil
+    it 'should create procs which evals to an array of mocks' do
+      proc = self.class.tasks_proc
+      proc.should be_kind_of(Proc)
+
+      @task.should be_nil
       instance_eval(&proc).should == [ mock_task ]
-      @task.should == mock_task
-    end
+      @task.should == mock_task
+    end
   end
 
   describe 'failures' do
@@ -121,126 +121,152 @@ describe 'MacroStubs' do
     after(:each) do
       teardown_mocks_for_rspec
     end
-  end
-
-  describe 'when extending describe group behavior' do
+  end
+  
+  describe 'with array in options' do
+    expects :find, :on => Task, :with => [proc{ current_id }, 1], :returns => [proc{ current_id }, 2]
+    
+    it 'should evaluate all procs in :with option' do
+      run_expectations!
+      
+      lambda {
+        Task.find(1)
+      }.should raise_error(Spec::Mocks::MockExpectationError, /expected :find with \("37"\, 1\) but received it with \(1\)/)
+      
+      lambda {
+        Task.find("37", 1)
+      }.should_not raise_error
+    end
+    
+    it 'should evaluate all procs in :returns option' do
+      run_expectations!
+      
+      Task.find("37", 1).should eql(["37", 2])
+    end
+    
+    after(:each) do
+      teardown_mocks_for_rspec
+    end
+  end
+
+  describe 'when extending describe group behavior' do
     expects :find, :on => Task, :with => proc{ current_id }, :returns => task_proc
-    expects :count, :max, :min, :on => Task
-
-    get :show, :id => 37
-    params :special_task_id => 42
-    mime Mime::HTML
-
-    it 'should run action declared in a class method' do
-      @controller.send(:performed?).should_not be_true
-
-      run_action!(false)
-
-      @controller.action_name.should == 'show'
-      @controller.request.method.should == :get
-      @controller.send(:performed?).should be_true
-    end
+    expects :count, :max, :min, :on => Task
+
+    get :show, :id => 37
+    params :special_task_id => 42
+    mime Mime::HTML
+
+    it 'should run action declared in a class method' do
+      @controller.send(:performed?).should_not be_true
+
+      run_action!(false)
+
+      @controller.action_name.should == 'show'
+      @controller.request.method.should == :get
+      @controller.send(:performed?).should be_true
+    end
 
     it 'should raise an error if an invalid key is supplied' do
       lambda {
         self.class.expects :find, :on => Task, :and_return => true
       }.should raise_error(ArgumentError, "Unknown key(s): and_return")
     end
-
-    it 'should use parameters given in params on request' do
-      self.should_receive(:current_id).once.and_return('37')
-      run_action!
-      @request.parameters[:special_task_id].should == '42'
-    end
-
-    it 'should respond with the supplied mime type' do
-      self.should_receive(:current_id).once.and_return('37')
-      run_action!
-      @response.content_type.should == Mime::HTML.to_s
-    end
-
-    it 'should run action with expectations' do
-      self.should_receive(:current_id).once.and_return('37')
-      run_action!
-      @controller.send(:performed?).should be_true
-    end
-
-    it 'should not run action twice' do
-      run_action!
-      @controller.send(:performed?).should be_true
-      proc{ run_action!.should be_false }.should_not raise_error
-    end
-
-    it 'should run expectations without performing an action' do
-      self.should_receive(:current_id).once.and_return('37')
-      run_expectations!
+
+    it 'should use parameters given in params on request' do
+      self.should_receive(:current_id).once.and_return('37')
+      run_action!
+      @request.parameters[:special_task_id].should == '42'
+    end
+
+    it 'should respond with the supplied mime type' do
+      self.should_receive(:current_id).once.and_return('37')
+      run_action!
+      @response.content_type.should == Mime::HTML.to_s
+    end
+
+    it 'should run action with expectations' do
+      self.should_receive(:current_id).once.and_return('37')
+      run_action!
+      @controller.send(:performed?).should be_true
+    end
+
+    it 'should not run action twice' do
+      run_action!
+      @controller.send(:performed?).should be_true
+      proc{ run_action!.should be_false }.should_not raise_error
+    end
+
+    it 'should run expectations without performing an action' do
+      self.should_receive(:current_id).once.and_return('37')
+      run_expectations!
       @controller.send(:performed?).should_not be_true
-      get :show, :id => '37' # Execute the action to match expectations
-    end
-
-    it 'should run action with stubs' do
-      self.should_receive(:current_id).never
-      run_action!(false)
-      @controller.send(:performed?).should be_true
-    end
-
-    it 'should run stubs without performing an action' do
-      self.should_receive(:current_id).never
-      run_stubs!
-      @controller.send(:performed?).should_not be_true
-    end
-
-    describe Mime::XML do
-      expects :to_xml, :on => task_proc, :returns => 'XML'
-
-      it 'should provide a description based on the mime given in describe' do
-        self.class.description.should =~ /with xml$/
-      end
-
-      it 'should run action based on inherited declarations' do
-        @controller.send(:performed?).should_not be_true
-
-        run_action!
-
-        @controller.action_name.should == 'show'
-        @controller.request.method.should == :get
-        @controller.send(:performed?).should be_true
-        @controller.response.body.should == 'XML'
-        @request.parameters[:special_task_id].should == '42'
-      end
-    end
-
-    describe 'and running actions in a before(:all) filter' do
-      get :show, :id => 37
-
-      get! do
-        @request.should_not be_nil
-      end
-
-      get! do
-        @flag = true
-      end
-
-      get! do
-        @controller.should_not be_nil
-      end
-
-      it 'should run the action before each example' do
-        @controller.send(:performed?).should be_true
-      end
-
-      it 'should execute the given block' do
-        @flag.should be_true
-      end
-    end
-  end
-
-  describe 'with matcher macros' do
-
-    [:delete, :delete!].each do |method|
-
-      describe method => :destroy, :id => '37' do
-        expects :find,    :on => Task, :with => '37', :returns => task_proc
+      get :show, :id => '37' # Execute the action to match expectations
+    end
+
+    it 'should run action with stubs' do
+      self.should_receive(:current_id).never
+      run_action!(false)
+      @controller.send(:performed?).should be_true
+    end
+
+    it 'should run stubs without performing an action' do
+      self.should_receive(:current_id).never
+      run_stubs!
+      @controller.send(:performed?).should_not be_true
+    end
+
+    describe Mime::XML do
+      expects :to_xml, :on => task_proc, :returns => 'XML'
+
+      it 'should provide a description based on the mime given in describe' do
+        self.class.description.should =~ /with xml$/
+      end
+
+      it 'should run action based on inherited declarations' do
+        @controller.send(:performed?).should_not be_true
+
+        run_action!
+
+        @controller.action_name.should == 'show'
+        @controller.request.method.should == :get
+        @controller.send(:performed?).should be_true
+        @controller.response.body.should == 'XML'
+        @request.parameters[:special_task_id].should == '42'
+      end
+    end
+
+    describe 'and running actions in a before(:all) filter' do
+      get :show, :id => 37
+
+      get! do
+        @request.should_not be_nil
+      end
+
+      get! do
+        @flag = true
+      end
+
+      get! do
+        @controller.should_not be_nil
+      end
+
+      it 'should run the action before each example' do
+        @controller.send(:performed?).should be_true
+      end
+
+      it 'should execute the given block' do
+        @flag.should be_true
+      end
+    end
+  end
+
+  describe 'with matcher macros' do
+
+    [:delete, :delete!].each do |method|
+
+      describe method => :destroy, :id => '37' do
+        expects :find,    :on => Task, :with => '37', :returns => task_proc
         expects :destroy, :on => task_proc
         expects :title,   :on => task_proc, :with => false do |boolean|
           if boolean
@@ -248,47 +274,47 @@ describe 'MacroStubs' do
           else
             'My favourite task'
           end
-        end
-
-        xhr!
-        subject { controller }
-
-        should_assign_to :task
-        should_assign_to :task, :with => task_proc
-        should_assign_to :task, :with_kind_of => Task
-
-        should_set_the_flash
-        should_set_the_flash :notice
-        should_set_the_flash :notice, :to => %{"My favourite task" was removed}
-
-        should_set_session
-        should_set_session :last_task_id
-        should_set_session :last_task_id, :to => 37
-
-        should_redirect_to{ project_tasks_url(10) }
-        should_redirect_to proc{ project_tasks_url(10) }, :with => 302
-
-        it 'should run action declared in describe' do
-          @controller.send(:performed?).should_not be_true unless method == :delete!
-
-          run_action!
-
-          @controller.action_name.should == 'destroy'
-          @controller.request.method.should == :delete
-          @controller.send(:performed?).should be_true
-        end
-
-        it 'should provide a description based on parameters given in describe' do
-          self.class.description.should =~ /responding to #DELETE destroy$/
+        end
+
+        xhr!
+        subject { controller }
+
+        should_assign_to :task
+        should_assign_to :task, :with => task_proc
+        should_assign_to :task, :with_kind_of => Task
+
+        should_set_the_flash
+        should_set_the_flash :notice
+        should_set_the_flash :notice, :to => %{"My favourite task" was removed}
+
+        should_set_session
+        should_set_session :last_task_id
+        should_set_session :last_task_id, :to => 37
+
+        should_redirect_to{ project_tasks_url(10) }
+        should_redirect_to proc{ project_tasks_url(10) }, :with => 302
+
+        it 'should run action declared in describe' do
+          @controller.send(:performed?).should_not be_true unless method == :delete!
+
+          run_action!
+
+          @controller.action_name.should == 'destroy'
+          @controller.request.method.should == :delete
+          @controller.send(:performed?).should be_true
+        end
+
+        it 'should provide a description based on parameters given in describe' do
+          self.class.description.should =~ /responding to #DELETE destroy$/
         end
 
         it 'should perform a XmlHttpRequest' do
           run_action!
           request.env['HTTP_X_REQUESTED_WITH'].should == 'XMLHttpRequest'
-        end
-      end
-
-    end
-
-  end
-end
+        end
+      end
+
+    end
+
+  end
+end