rspec-rails 2.0.0.beta.22 → 2.6.0

data/spec/rspec/rails/mocks/stub_model_spec.rb

@@ -2,100 +2,154 @@ require 'spec_helper'
 require File.dirname(__FILE__) + '/ar_classes'
 
 describe "stub_model" do
-  describe "defaults" do
-    it "has an id" do
-      stub_model(MockableModel).id.should be > 0
+
+  shared_examples_for "stub model" do
+    describe "with a block" do
+      it "yields the model" do
+        model = stub_model(model_class) do |block_arg|
+          @block_arg = block_arg
+        end
+        model.should be(@block_arg)
+      end
     end
 
-    it "says it is not a new record" do
-      stub_model(MockableModel).should_not be_new_record
+    describe "#persisted?" do
+      context "default" do
+        it "returns true" do
+          model = stub_model(model_class)
+          model.should be_persisted
+        end
+      end
+
+      context "with as_new_record" do
+        it "returns false" do
+          model = stub_model(model_class).as_new_record
+          model.should_not be_persisted
+        end
+      end
     end
-  end
 
-  it "accepts a stub id" do
-    stub_model(MockableModel, :id => 37).id.should == 37
-  end
+    it "increments the value returned by to_param" do
+      first = stub_model(model_class)
+      second = stub_model(model_class)
+      second.to_param.to_i.should == (first.to_param.to_i + 1)
+    end
 
-  it "says it is a new record when id is set to nil" do
-    stub_model(MockableModel, :id => nil).should be_new_record
+    describe "#blank?" do
+      it "is false" do
+        stub_model(model_class).should_not be_blank
+      end
+    end
   end
 
-  it "accepts any arbitrary stub" do
-    stub_model(MockableModel, :foo => "bar").foo.should == "bar"
+  context "with ActiveModel (not ActiveRecord)" do
+    it_behaves_like "stub model" do
+      def model_class
+        NonActiveRecordModel
+      end
+    end
   end
 
-  it "accepts a stub for save" do
-    stub_model(MockableModel, :save => false).save.should be(false)
-  end
-  
-  describe "alternate primary key" do
-    it "has the correct primary_key name" do
-      stub_model(AlternatePrimaryKeyModel).class.primary_key.should eql('my_id')
+  context "with an ActiveRecord model" do
+    let(:model_class) { MockableModel }
+
+    it_behaves_like "stub model"
+
+    describe "#new_record?" do
+      context "default" do
+        it "returns false" do
+          model = stub_model(model_class)
+          model.new_record?.should be_false
+        end
+      end
+
+      context "with as_new_record" do
+        it "returns true" do
+          model = stub_model(model_class).as_new_record
+          model.new_record?.should be_true
+        end
+      end
     end
-    
-    it "has a primary_key" do
-      stub_model(AlternatePrimaryKeyModel).my_id.should be > 0
+
+    describe "defaults" do
+      it "has an id" do
+        stub_model(MockableModel).id.should be > 0
+      end
+
+      it "says it is not a new record" do
+        stub_model(MockableModel).should_not be_new_record
+      end
     end
-    
-    it "does not say it is a new record" do
-      stub_model(AlternatePrimaryKeyModel).should_not be_new_record
+
+    describe "#as_new_record" do
+      it "has a nil id" do
+        stub_model(MockableModel).as_new_record.id.should be(nil)
+      end
     end
-    
-    it "says it is a new record if primary_key is nil" do
-      stub_model(AlternatePrimaryKeyModel, :my_id => nil).should be_new_record
+
+    it "raises when hitting the db" do
+      lambda do
+        stub_model(MockableModel).connection
+      end.should raise_error(RSpec::Rails::IllegalDataAccessException, /stubbed models are not allowed to access the database/)
     end
-    
-    it "accepts a stub for the primary_key" do
-      stub_model(AlternatePrimaryKeyModel, :my_id => 5).my_id.should == 5
+
+    it "increments the id" do
+      first = stub_model(model_class)
+      second = stub_model(model_class)
+      second.id.should == (first.id + 1)
     end
-  end
 
-  describe "#as_new_record" do
-    it "says it is a new record" do
-      stub_model(MockableModel).as_new_record.should be_new_record
+    it "accepts a stub id" do
+      stub_model(MockableModel, :id => 37).id.should == 37
     end
 
-    it "has a nil id" do
-      stub_model(MockableModel).as_new_record.id.should be(nil)
+    it "says it is a new record when id is set to nil" do
+      stub_model(MockableModel, :id => nil).should be_new_record
     end
-  end
 
-  it "raises when hitting the db" do
-    lambda do
-      stub_model(ConnectableModel).connection
-    end.should raise_error(RSpec::Rails::IllegalDataAccessException, /stubbed models are not allowed to access the database/)
-  end
+    it "accepts a stub for save" do
+      stub_model(MockableModel, :save => false).save.should be(false)
+    end
 
-  it "increments the id" do
-    first = stub_model(MockableModel)
-    second = stub_model(MockableModel)
-    second.id.should == (first.id + 1)
-  end
+    describe "alternate primary key" do
+      it "has the correct primary_key name" do
+        stub_model(AlternatePrimaryKeyModel).class.primary_key.to_s.should eql('my_id')
+      end
+
+      it "has a primary_key" do
+        stub_model(AlternatePrimaryKeyModel).my_id.should be > 0
+      end
+
+      it "says it is not a new record" do
+        stub_model(AlternatePrimaryKeyModel) do |m|
+          m.should_not be_new_record
+        end
+      end
+
+      it "says it is a new record if primary_key is nil" do
+        stub_model(AlternatePrimaryKeyModel, :my_id => nil).should be_new_record
+      end
 
-  describe "as association" do
-    before(:each) do
-      @real = AssociatedModel.create!
-      @stub_model = stub_model(MockableModel)
-      @real.mockable_model = @stub_model
+      it "accepts a stub for the primary_key" do
+        stub_model(AlternatePrimaryKeyModel, :my_id => 5).my_id.should == 5
+      end
     end
 
-    it "passes associated_model == mock" do
+    describe "as association" do
+      before(:each) do
+        @real = AssociatedModel.create!
+        @stub_model = stub_model(MockableModel)
+        @real.mockable_model = @stub_model
+      end
+
+      it "passes associated_model == mock" do
         @stub_model.should == @real.mockable_model
-    end
+      end
 
-    it "passes mock == associated_model" do
+      it "passes mock == associated_model" do
         @real.mockable_model.should == @stub_model
-    end
-  end
-
-  describe "with a block" do
-    it "yields the model" do
-      model = stub_model(MockableModel) do |block_arg|
-        @block_arg = block_arg
       end
-      model.should be(@block_arg)
     end
+
   end
 end
-
-

data/spec/rspec/rails/view_rendering_spec.rb

@@ -0,0 +1,105 @@
+require "spec_helper"
+
+module RSpec::Rails
+  describe ViewRendering do
+    let(:group) do
+      RSpec::Core::ExampleGroup.describe do
+        def controller
+          ActionController::Base.new
+        end
+        include ViewRendering
+      end
+    end
+
+    context "default" do
+      context "ActionController::Base" do
+        it "does not render views" do
+          group.new.render_views?.should be_false
+        end
+
+        it "does not render views in a nested group" do
+          group.describe{}.new.render_views?.should be_false
+        end
+      end
+
+      context "ActionController::Metal" do
+        it "renders views" do
+          group.new.tap do |example|
+            def example.controller
+              ActionController::Metal.new
+            end
+            example.render_views?.should be_true
+          end
+        end
+      end
+    end
+
+    describe "#render_views" do
+      context "with no args" do
+        it "tells examples to render views" do
+          group.render_views
+          group.new.render_views?.should be_true
+        end
+      end
+
+      context "with true" do
+        it "tells examples to render views" do
+          group.render_views true
+          group.new.render_views?.should be_true
+        end
+      end
+
+      context "with false" do
+        it "tells examples not to render views" do
+          group.render_views false
+          group.new.render_views?.should be_false
+        end
+      end
+
+      context "in a nested group" do
+        let(:nested_group) do
+          group.describe{}
+        end
+
+        context "with no args" do
+          it "tells examples to render views" do
+            nested_group.render_views
+            nested_group.new.render_views?.should be_true
+          end
+        end
+
+        context "with true" do
+          it "tells examples to render views" do
+            nested_group.render_views true
+            nested_group.new.render_views?.should be_true
+          end
+        end
+
+        context "with false" do
+          it "tells examples not to render views" do
+            nested_group.render_views false
+            nested_group.new.render_views?.should be_false
+          end
+        end
+
+        it "leaves the parent group as/is" do
+          group.render_views
+          nested_group.render_views false
+          group.new.render_views?.should be_true
+        end
+
+        it "overrides the value inherited from the parent group" do
+          group.render_views
+          nested_group.render_views false
+          nested_group.new.render_views?.should be_false
+        end
+
+        it "passes override to children" do
+          group.render_views
+          nested_group.render_views false
+          nested_group.describe{}.new.render_views?.should be_false
+        end
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -9,35 +9,13 @@ require 'rspec/rails'
 
 Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each {|f| require f}
 
-# TODO - most of this is borrowed from rspec-core's spec_helper - should
-# be extracted to something we can use here
-def in_editor?
-  ENV.has_key?('TM_MODE') || ENV.has_key?('EMACS') || ENV.has_key?('VIM')
-end
-
 class RSpec::Core::ExampleGroup
   def self.run_all(reporter=nil)
     run(reporter || RSpec::Mocks::Mock.new('reporter').as_null_object)
   end
 end
 
-module MatchesForRSpecRailsSpecs
-  extend RSpec::Matchers::DSL
-
-  matcher :be_included_in_files_in do |path|
-    match do |mod|
-      stub_metadata(
-        :example_group => {:file_path => "#{path}whatever_spec.rb:15"}
-      )
-      group = RSpec::Core::ExampleGroup.describe
-      group.included_modules.include?(mod)
-    end
-  end
-end
-
 RSpec.configure do |c|
-  c.include MatchesForRSpecRailsSpecs
-  c.color_enabled = !in_editor?
   c.before(:each) do
     @real_world = RSpec.world
     RSpec.instance_variable_set(:@world, RSpec::Core::World.new)

data/spec/support/matchers.rb

@@ -0,0 +1,9 @@
+RSpec::Matchers::define :be_included_in_files_in do |path|
+  match do |mod|
+    stub_metadata(
+      :example_group => {:file_path => "#{path}whatever_spec.rb:15"}
+    )
+    group = RSpec::Core::ExampleGroup.describe
+    group.included_modules.include?(mod)
+  end
+end

data/templates/generate_stuff.rb

@@ -5,9 +5,17 @@ generate('integration_test widgets')
 generate('mailer Notifications signup')
 generate('model thing name:string')
 generate('helper things')
-generate('scaffold widget name:string category:string instock:boolean')
+generate('scaffold widget name:string category:string instock:boolean --force')
 generate('observer widget')
-generate('scaffold gadget ') # scaffold with no attributes
+generate('scaffold gadget') # scaffold with no attributes
+generate('scaffold admin/accounts name:string') # scaffold with nested resource
+
+generate('controller things custom_action')
+template_code= <<-TEMPLATE
+  <% raise 'Error from custom_action because we should never render this template....derp derp derp' %>
+TEMPLATE
+
+file "app/views/things/custom_action.html.erb", template_code, {:force=>true}
 
 run('rake db:migrate')
 run('rake db:test:prepare')

data/templates/run_specs.rb

@@ -1,4 +1,4 @@
-run('bundle exec rspec spec -cfdoc')
+run('rspec spec -cfdoc')
 run('rake spec')
 run('rake spec:requests')
 run('rake spec:models')
@@ -6,4 +6,4 @@ run('rake spec:views')
 run('rake spec:controllers')
 run('rake spec:helpers')
 run('rake spec:mailers')
-run('rake stats')
+run("rake stats")