@@ -79,25 +79,31 @@ module Shoulda # :nodoc: # * <tt>:message</tt> - value the test expects to find in <tt>errors.on(:attribute)</tt>. # Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages.taken')</tt> # * <tt>:scoped_to</tt> - field(s) to scope the uniqueness to. + # * <tt>:case_sensitive</tt> - whether or not the value should match case sensitively. + # Use <tt>true</tt> to explicitly test for a case sensitive match. + # Use <tt>false</tt> to explicitly test for a case insensitive match. + # Default = <tt>true</tt>. # # Examples: # should_require_unique_attributes :keyword, :username # should_require_unique_attributes :name, :message => "O NOES! SOMEONE STOELED YER NAME!" # should_require_unique_attributes :email, :scoped_to => :name # should_require_unique_attributes :address, :scoped_to => [:first_name, :last_name] + # should_require_unique_attributes :email, :case_sensitive => false # def should_require_unique_attributes(*attributes) - message, scope = get_options!(attributes, :message, :scoped_to) + message, scope, case_sensitive = get_options!(attributes, :message, :scoped_to, :case_sensitive) scope = [*scope].compact message ||= default_error_message(:taken) klass = model_class attributes.each do |attribute| attribute = attribute.to_sym - should "require unique value for #{attribute}#{" scoped to #{scope.join(', ')}" unless scope.blank?}" do + should "require#{' case insensitive' if case_sensitive == false} unique value for #{attribute}#{" scoped to #{scope.join(', ')}" unless scope.blank?}" do assert existing = klass.find(:first), "Can't find first #{klass}" object = klass.new existing_value = existing.send(attribute) + existing_value.swapcase! if case_sensitive == false && existing_value.respond_to?(:swapcase!) if !scope.blank? scope.each do |s| @@ -105,6 +111,7 @@ module Shoulda # :nodoc: object.send("#{s}=", existing.send(s)) end end + assert_bad_value(object, attribute, existing_value, message) # Now test that the object is valid when changing the scoped attribute lib/shoulda/active_record/macros.rb @@ -6,6 +6,7 @@ class Post < ActiveRecord::Base has_many :through_tags, :through => :taggings, :source => :tag validates_uniqueness_of :title + validates_uniqueness_of :title, :case_sensitive => false validates_presence_of :title validates_presence_of :body, :message => 'Seriously... wtf' validates_numericality_of :user_id test/rails_root/app/models/post.rb @@ -9,6 +9,7 @@ class PostTest < Test::Unit::TestCase should_have_many :through_tags, :through => :taggings should_require_unique_attributes :title + should_require_unique_attributes :title, :case_sensitive => false should_require_attributes :body, :message => /wtf/ should_require_attributes :title should_only_allow_numeric_values_for :user_id test/unit/post_test.rb 1e74f34536b49a02be919e3fdba062b6d719edfa Lawrence Pit lawrence.pit@gmail.com http://github.com/thoughtbot/shoulda/commit/d8fe1caeb9626ccba20d4c936910dfd4e6fbd557 d8fe1caeb9626ccba20d4c936910dfd4e6fbd557 2009-01-10T11:49:37-08:00 2009-01-10T04:15:04-08:00 Added case_sensitive option to should_require_unique_attributes call [#130] c312f67c75961ee7e57cd2786d71dc7aa164f795 Ryan McGeary ryanongit@mcgeary.org