francois-shoulda 2.0.5.4 → 2.10.1

data/test/functional/users_controller_test.rb

@@ -16,23 +16,4 @@ class UsersControllerTest < Test::Unit::TestCase
   
   should_filter_params :ssn
 
-  should_be_restful do |resource|
-    resource.identifier = :id
-    resource.klass      = User
-    resource.object     = :user
-    resource.parent     = []
-    resource.actions    = [:index, :show, :new, :edit, :update, :create, :destroy]
-    resource.formats    = [:html, :xml]
-
-    resource.create.params = { :name => "bob", :email => 'bob@bob.com', :age => 13, :ssn => "123456789"}
-    resource.update.params = { :name => "sue" }
-
-    resource.create.redirect  = "user_url(@user)"
-    resource.update.redirect  = "user_url(@user)"
-    resource.destroy.redirect = "users_url"
-
-    resource.create.flash  = /created/i
-    resource.update.flash  = /updated/i
-    resource.destroy.flash = /removed/i
-  end
 end

data/test/matchers/active_record/allow_mass_assignment_of_matcher_test.rb

@@ -0,0 +1,68 @@
+require File.join(File.dirname(__FILE__), '..', '..', 'test_helper')
+
+class AllowMassAssignmentOfMatcherTest < Test::Unit::TestCase # :nodoc:
+
+  context "an attribute that is blacklisted from mass-assignment" do
+    setup do
+      define_model :example, :attr => :string do
+        attr_protected :attr
+      end
+      @model = Example.new
+    end
+
+    should "reject being mass-assignable" do
+      assert_rejects allow_mass_assignment_of(:attr), @model
+    end
+  end
+
+  context "an attribute that is not whitelisted for mass-assignment" do
+    setup do
+      define_model :example, :attr => :string, :other => :string do
+        attr_accessible :other
+      end
+      @model = Example.new
+    end
+
+    should "reject being mass-assignable" do
+      assert_rejects allow_mass_assignment_of(:attr), @model
+    end
+  end
+
+  context "an attribute that is whitelisted for mass-assignment" do
+    setup do
+      define_model :example, :attr => :string do
+        attr_accessible :attr
+      end
+      @model = Example.new
+    end
+
+    should "accept being mass-assignable" do
+      assert_accepts allow_mass_assignment_of(:attr), @model
+    end
+  end
+
+  context "an attribute not included in the mass-assignment blacklist" do
+    setup do
+      define_model :example, :attr => :string, :other => :string do
+        attr_protected :other
+      end
+      @model = Example.new
+    end
+
+    should "accept being mass-assignable" do
+      assert_accepts allow_mass_assignment_of(:attr), @model
+    end
+  end
+
+  context "an attribute on a class with no protected attributes" do
+    setup do
+      define_model :example, :attr => :string
+      @model = Example.new
+    end
+
+    should "accept being mass-assignable" do
+      assert_accepts allow_mass_assignment_of(:attr), @model
+    end
+  end
+
+end

data/test/matchers/active_record/allow_value_matcher_test.rb

@@ -0,0 +1,41 @@
+require File.join(File.dirname(__FILE__), '..', '..', 'test_helper')
+
+class AllowValueMatcherTest < Test::Unit::TestCase # :nodoc:
+
+  context "an attribute with a format validation" do
+    setup do
+      define_model :example, :attr => :string do
+        validates_format_of :attr, :with => /abc/
+      end
+      @model = Example.new
+    end
+
+    should "allow a good value" do
+      assert_accepts allow_value("abcde").for(:attr), @model
+    end
+    
+    should "not allow a bad value" do
+      assert_rejects allow_value("xyz").for(:attr), @model
+    end
+  end
+
+  context "an attribute with a format validation and a custom message" do
+    setup do
+      define_model :example, :attr => :string do
+        validates_format_of :attr, :with => /abc/, :message => 'bad value'
+      end
+      @model = Example.new
+    end
+
+    should "allow a good value" do
+      assert_accepts allow_value('abcde').for(:attr).with_message(/bad/),
+                     @model
+    end
+    
+    should "not allow a bad value" do
+      assert_rejects allow_value('xyz').for(:attr).with_message(/bad/),
+                     @model
+    end
+  end
+
+end

data/test/matchers/active_record/association_matcher_test.rb

@@ -0,0 +1,258 @@
+require File.join(File.dirname(__FILE__), '..', '..', 'test_helper')
+
+class AssociationMatcherTest < Test::Unit::TestCase # :nodoc:
+
+  context "belong_to" do
+    setup do
+      @matcher = belong_to(:parent)
+    end
+
+    should "accept a good association with the default foreign key" do
+      define_model :parent
+      define_model :child, :parent_id => :integer do
+        belongs_to :parent
+      end
+      assert_accepts @matcher, Child.new
+    end
+
+    should "reject a nonexistent association" do
+      define_model :child
+      assert_rejects @matcher, Child.new
+    end
+
+    should "reject an association of the wrong type" do
+      define_model :parent, :child_id => :integer
+      child_class = define_model :child do
+        has_one :parent
+      end
+      assert_rejects @matcher, Child.new
+    end
+
+    should "reject an association that has a nonexistent foreign key" do
+      define_model :parent
+      define_model :child do
+        belongs_to :parent
+      end
+      assert_rejects @matcher, Child.new
+    end
+
+    should "accept an association with an existing custom foreign key" do
+      define_model :parent
+      define_model :child, :guardian_id => :integer do
+        belongs_to :parent, :foreign_key => 'guardian_id'
+      end
+      assert_accepts @matcher, Child.new
+    end
+
+    should "accept a polymorphic association" do
+      define_model :child, :parent_type => :string,
+                                :parent_id   => :integer do
+        belongs_to :parent, :polymorphic => true
+      end
+      assert_accepts @matcher, Child.new
+    end
+
+    should "accept an association with a valid :dependent option" do
+      define_model :parent
+      define_model :child, :parent_id => :integer do
+        belongs_to :parent, :dependent => :destroy
+      end
+      assert_accepts @matcher.dependent(:destroy), Child.new
+    end
+
+    should "reject an association with a bad :dependent option" do
+      define_model :parent
+      define_model :child, :parent_id => :integer do
+        belongs_to :parent
+      end
+      assert_rejects @matcher.dependent(:destroy), Child.new
+    end
+  end
+
+  context "have_many" do
+    setup do
+      @matcher = have_many(:children)
+    end
+
+    should "accept a valid association without any options" do
+      define_model :child, :parent_id => :integer
+      define_model :parent do
+        has_many :children
+      end
+      assert_accepts @matcher, Parent.new
+    end
+
+    should "accept a valid association with a :through option" do
+      define_model :child
+      define_model :conception, :child_id  => :integer,
+                                     :parent_id => :integer do
+        belongs_to :child
+      end
+      define_model :parent do
+        has_many :conceptions
+        has_many :children, :through => :conceptions
+      end
+      assert_accepts @matcher, Parent.new
+    end
+
+    should "accept a valid association with an :as option" do
+      define_model :child, :guardian_type => :string,
+                                :guardian_id   => :integer
+      define_model :parent do
+        has_many :children, :as => :guardian
+      end
+      assert_accepts @matcher, Parent.new
+    end
+
+    should "reject an association that has a nonexistent foreign key" do
+      define_model :child
+      define_model :parent do
+        has_many :children
+      end
+      assert_rejects @matcher, Parent.new
+    end
+
+    should "reject an association with a bad :as option" do
+      define_model :child, :caretaker_type => :string,
+                                :caretaker_id   => :integer
+      define_model :parent do
+        has_many :children, :as => :guardian
+      end
+      assert_rejects @matcher, Parent.new
+    end
+
+    should "reject an association that has a bad :through option" do
+      define_model :child, :parent_id => :integer
+      define_model :parent do
+        has_many :children
+      end
+      assert_rejects @matcher.through(:conceptions), Parent.new
+    end
+
+    should "reject an association that has the wrong :through option" do
+      define_model :child
+      define_model :conception, :child_id  => :integer,
+                                     :parent_id => :integer do
+        belongs_to :child
+      end
+      define_model :parent do
+        has_many :conceptions
+        has_many :children, :through => :conceptions
+      end
+      assert_rejects @matcher.through(:relationships), Parent.new
+    end
+
+    should "accept an association with a valid :dependent option" do
+      define_model :child, :parent_id => :integer
+      define_model :parent do
+        has_many :children, :dependent => :destroy
+      end
+      assert_accepts @matcher.dependent(:destroy), Parent.new
+    end
+
+    should "reject an association with a bad :dependent option" do
+      define_model :child, :parent_id => :integer
+      define_model :parent do
+        has_many :children
+      end
+      assert_rejects @matcher.dependent(:destroy), Parent.new
+    end
+  end
+
+  context "have_one" do
+    setup do
+      @matcher = have_one(:profile)
+    end
+
+    should "accept a valid association without any options" do
+      define_model :profile, :person_id => :integer
+      define_model :person do
+        has_one :profile
+      end
+      assert_accepts @matcher, Person.new
+    end
+
+    should "accept a valid association with an :as option" do
+      define_model :profile, :profilable_id   => :integer,
+                                  :profilable_type => :string
+      define_model :person do
+        has_one :profile, :as => :profilable
+      end
+      assert_accepts @matcher, Person.new
+    end
+
+    should "reject an association that has a nonexistent foreign key" do
+      define_model :profile
+      define_model :person do
+        has_one :profile
+      end
+      assert_rejects @matcher, Person.new
+    end
+
+    should "reject an association with a bad :as option" do
+      define_model :profile, :profilable_id   => :integer,
+                                  :profilable_type => :string
+      define_model :person do
+        has_one :profile, :as => :describable
+      end
+      assert_rejects @matcher, Person.new
+    end
+
+    should "accept an association with a valid :dependent option" do
+      define_model :profile, :person_id => :integer
+      define_model :person do
+        has_one :profile, :dependent => :destroy
+      end
+      assert_accepts @matcher.dependent(:destroy), Person.new
+    end
+
+    should "reject an association with a bad :dependent option" do
+      define_model :profile, :person_id => :integer
+      define_model :person do
+        has_one :profile
+      end
+      assert_rejects @matcher.dependent(:destroy), Person.new
+    end
+  end
+
+  context "have_and_belong_to_many" do
+    setup do
+      @matcher = have_and_belong_to_many(:relatives)
+    end
+
+    should "accept a valid association" do
+      define_model :relatives
+      define_model :person do
+        has_and_belongs_to_many :relatives
+      end
+      define_model :people_relative, :person_id   => :integer,
+                                          :relative_id => :integer
+      assert_accepts @matcher, Person.new
+    end
+
+    should "reject a nonexistent association" do
+      define_model :relatives
+      define_model :person
+      define_model :people_relative, :person_id   => :integer,
+                                          :relative_id => :integer
+      assert_rejects @matcher, Person.new
+    end
+
+    should "reject an association with a nonexistent join table" do
+      define_model :relatives
+      define_model :person do
+        has_and_belongs_to_many :relatives
+      end
+      assert_rejects @matcher, Person.new
+    end
+
+    should "reject an association of the wrong type" do
+      define_model :relatives, :person_id => :integer
+      define_model :person do
+        has_many :relatives
+      end
+      assert_rejects @matcher, Person.new
+    end
+  end
+
+end

data/test/matchers/active_record/ensure_inclusion_of_matcher_test.rb

@@ -0,0 +1,80 @@
+require File.join(File.dirname(__FILE__), '..', '..', 'test_helper')
+
+class EnsureInclusionOfMatcherTest < Test::Unit::TestCase # :nodoc:
+
+  context "an attribute which must be included in a range" do
+    setup do
+      @model = define_model(:example, :attr => :integer) do
+        validates_inclusion_of :attr, :in => 2..5
+      end.new
+    end
+
+    should "accept ensuring the correct range" do
+      assert_accepts ensure_inclusion_of(:attr).in_range(2..5), @model
+    end
+
+    should "reject ensuring a lower minimum value" do
+      assert_rejects ensure_inclusion_of(:attr).in_range(1..5), @model
+    end
+
+    should "reject ensuring a higher minimum value" do
+      assert_rejects ensure_inclusion_of(:attr).in_range(3..5), @model
+    end
+
+    should "reject ensuring a lower maximum value" do
+      assert_rejects ensure_inclusion_of(:attr).in_range(2..4), @model
+    end
+
+    should "reject ensuring a higher maximum value" do
+      assert_rejects ensure_inclusion_of(:attr).in_range(2..6), @model
+    end
+
+    should "not override the default message with a blank" do
+      assert_accepts ensure_inclusion_of(:attr).
+                       in_range(2..5).
+                       with_message(nil),
+                     @model
+    end
+  end
+
+  context "an attribute with a custom ranged value validation" do
+    setup do
+      @model = define_model(:example, :attr => :string) do
+        validates_inclusion_of :attr, :in => 2..4, :message => 'not good'
+
+      end.new
+    end
+
+    should "accept ensuring the correct range" do
+      assert_accepts ensure_inclusion_of(:attr).
+                       in_range(2..4).
+                       with_message(/not good/),
+                     @model
+    end
+  end
+
+  context "an attribute with custom range validations" do
+    setup do
+      define_model :example, :attr => :integer do
+        def validate
+          if attr < 2
+            errors.add(:attr, 'too low')
+          elsif attr > 5
+            errors.add(:attr, 'too high')
+          end
+        end
+      end
+      @model = Example.new
+    end
+
+    should "accept ensuring the correct range and messages" do
+      assert_accepts ensure_inclusion_of(:attr).
+                       in_range(2..5).
+                       with_low_message(/low/).
+                       with_high_message(/high/),
+                     @model
+    end
+
+  end
+
+end