Skip to content

Commit

Permalink
Merge branch 'master' of git@github.com:thoughtbot/shoulda
Browse files Browse the repository at this point in the history
  • Loading branch information
Dan Croak committed Oct 7, 2008
2 parents 0b615a4 + 2a13424 commit 97fef43
Show file tree
Hide file tree
Showing 6 changed files with 45 additions and 32 deletions.
2 changes: 1 addition & 1 deletion init.rb
View file
@@ -1 +1 @@
require 'shoulda/rails'
require File.join(File.dirname(__FILE__), 'rails', 'init')
2 changes: 1 addition & 1 deletion lib/shoulda/active_record/assertions.rb
View file
Expand Up @@ -57,7 +57,7 @@ def assert_good_value(object_or_klass, attribute, value, error_message_to_avoid
# @product = Product.new(:tangible => true)
# assert_bad_value(Product, :price, "0")
def assert_bad_value(object_or_klass, attribute, value,
error_message_to_expect = DEFAULT_ERROR_MESSAGES[:invalid])
error_message_to_expect = self.class.default_error_message(:invalid))
object = get_instance_of(object_or_klass)
object.send("#{attribute}=", value)
assert !object.valid?, "#{object.class} allowed #{value.inspect} as a value for #{attribute}"
Expand Down
67 changes: 39 additions & 28 deletions lib/shoulda/active_record/macros.rb
View file
@@ -1,12 +1,22 @@
module ThoughtBot # :nodoc:
module Shoulda # :nodoc:
module ActiveRecord # :nodoc:
DEFAULT_ERROR_MESSAGES =
if Object.const_defined?(:I18n)
I18n.translate('activerecord.errors.messages')
else
::ActiveRecord::Errors.default_error_messages
module MacroHelpers # :nodoc:
# Helper method that determines the default error message used by Active
# Record. Works for both existing Rails 2.1 and Rails 2.2 with the newly
# introduced I18n module used for localization.
#
# default_error_message(:blank)
# default_error_message(:too_short, :count => 5)
# default_error_message(:too_long, :count => 60)
def default_error_message(key, values = {})
if Object.const_defined?(:I18n) # Rails >= 2.2
I18n.translate("activerecord.errors.messages.#{key}", values)
else # Rails <= 2.1.x
::ActiveRecord::Errors.default_error_messages[key] % values[:count]
end
end
end

# = Macro test helpers for your active record models
#
Expand All @@ -28,6 +38,8 @@ module ActiveRecord # :nodoc:
# For all of these helpers, the last parameter may be a hash of options.
#
module Macros
include MacroHelpers

# <b>DEPRECATED:</b> Use <tt>fixtures :all</tt> instead
#
# Loads all fixture files (<tt>test/fixtures/*.yml</tt>)
Expand All @@ -44,14 +56,14 @@ def load_all_fixtures
#
# Options:
# * <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')[:blank]</tt>
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages.blank')</tt>
#
# Example:
# should_require_attributes :name, :phone_number
#
def should_require_attributes(*attributes)
message = get_options!(attributes, :message)
message ||= DEFAULT_ERROR_MESSAGES[:blank]
message ||= default_error_message(:blank)
klass = model_class

attributes.each do |attribute|
Expand All @@ -66,7 +78,7 @@ def should_require_attributes(*attributes)
#
# Options:
# * <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>
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages.taken')</tt>
# * <tt>:scoped_to</tt> - field(s) to scope the uniqueness to.
#
# Examples:
Expand All @@ -78,7 +90,7 @@ def should_require_attributes(*attributes)
def should_require_unique_attributes(*attributes)
message, scope = get_options!(attributes, :message, :scoped_to)
scope = [*scope].compact
message ||= DEFAULT_ERROR_MESSAGES[:taken]
message ||= default_error_message(:taken)

klass = model_class
attributes.each do |attribute|
Expand Down Expand Up @@ -164,14 +176,14 @@ def should_have_readonly_attributes(*attributes)
#
# Options:
# * <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')[:invalid]</tt>
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages.invalid')</tt>
#
# Example:
# should_not_allow_values_for :isbn, "bad 1", "bad 2"
#
def should_not_allow_values_for(attribute, *bad_values)
message = get_options!(bad_values, :message)
message ||= DEFAULT_ERROR_MESSAGES[:invalid]
message ||= default_error_message(:invalid)
klass = model_class
bad_values.each do |v|
should "not allow #{attribute} to be set to #{v.inspect}" do
Expand Down Expand Up @@ -207,17 +219,17 @@ def should_allow_values_for(attribute, *good_values)
#
# Options:
# * <tt>:short_message</tt> - value the test expects to find in <tt>errors.on(:attribute)</tt>.
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages')[:too_short] % range.first</tt>
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages.too_short') % range.first</tt>
# * <tt>:long_message</tt> - value the test expects to find in <tt>errors.on(:attribute)</tt>.
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages')[:too_long] % range.last</tt>
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages.too_long') % range.last</tt>
#
# Example:
# should_ensure_length_in_range :password, (6..20)
#
def should_ensure_length_in_range(attribute, range, opts = {})
short_message, long_message = get_options!([opts], :short_message, :long_message)
short_message ||= DEFAULT_ERROR_MESSAGES[:too_short] % range.first
long_message ||= DEFAULT_ERROR_MESSAGES[:too_long] % range.last
short_message ||= default_error_message(:too_short, :count => range.first)
long_message ||= default_error_message(:too_long, :count => range.last)

klass = model_class
min_length = range.first
Expand Down Expand Up @@ -259,14 +271,14 @@ def should_ensure_length_in_range(attribute, range, opts = {})
#
# Options:
# * <tt>:short_message</tt> - value the test expects to find in <tt>errors.on(:attribute)</tt>.
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages')[:too_short] % min_length</tt>
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages.too_short') % min_length</tt>
#
# Example:
# should_ensure_length_at_least :name, 3
#
def should_ensure_length_at_least(attribute, min_length, opts = {})
short_message = get_options!([opts], :short_message)
short_message ||= DEFAULT_ERROR_MESSAGES[:too_short] % min_length
short_message ||= default_error_message(:too_short, :count => min_length)

klass = model_class

Expand All @@ -290,14 +302,14 @@ def should_ensure_length_at_least(attribute, min_length, opts = {})
#
# Options:
# * <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')[:wrong_length] % length</tt>
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages.wrong_length') % length</tt>
#
# Example:
# should_ensure_length_is :ssn, 9
#
def should_ensure_length_is(attribute, length, opts = {})
message = get_options!([opts], :message)
message ||= DEFAULT_ERROR_MESSAGES[:wrong_length] % length
message ||= default_error_message(:wrong_length, :count => length)

klass = model_class

Expand Down Expand Up @@ -325,17 +337,17 @@ def should_ensure_length_is(attribute, length, opts = {})
#
# Options:
# * <tt>:low_message</tt> - value the test expects to find in <tt>errors.on(:attribute)</tt>.
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages')[:inclusion]</tt>
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages.inclusion')</tt>
# * <tt>:high_message</tt> - value the test expects to find in <tt>errors.on(:attribute)</tt>.
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages')[:inclusion]</tt>
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages.inclusion')</tt>
#
# Example:
# should_ensure_value_in_range :age, (0..100)
#
def should_ensure_value_in_range(attribute, range, opts = {})
low_message, high_message = get_options!([opts], :low_message, :high_message)
low_message ||= DEFAULT_ERROR_MESSAGES[:inclusion]
high_message ||= DEFAULT_ERROR_MESSAGES[:inclusion]
low_message ||= default_error_message(:inclusion)
high_message ||= default_error_message(:inclusion)

klass = model_class
min = range.first
Expand Down Expand Up @@ -370,14 +382,14 @@ def should_ensure_value_in_range(attribute, range, opts = {})
#
# Options:
# * <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')[:not_a_number]</tt>
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages.not_a_number')</tt>
#
# Example:
# should_only_allow_numeric_values_for :age
#
def should_only_allow_numeric_values_for(*attributes)
message = get_options!(attributes, :message)
message ||= DEFAULT_ERROR_MESSAGES[:not_a_number]
message ||= default_error_message(:not_a_number)
klass = model_class
attributes.each do |attribute|
attribute = attribute.to_sym
Expand Down Expand Up @@ -624,14 +636,14 @@ def should_have_indices(*columns)
#
# Options:
# * <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')[:accepted]</tt>
# Regexp or string. Default = <tt>I18n.translate('activerecord.errors.messages.accepted')</tt>
#
# Example:
# should_require_acceptance_of :eula
#
def should_require_acceptance_of(*attributes)
message = get_options!(attributes, :message)
message ||= DEFAULT_ERROR_MESSAGES[:accepted]
message ||= default_error_message(:accepted)
klass = model_class

attributes.each do |attribute|
Expand Down Expand Up @@ -697,7 +709,6 @@ def should_have_named_scope(scope_call, *args)
end
end
end

end
end
end
Expand Down
3 changes: 2 additions & 1 deletion lib/shoulda/controller/macros.rb
View file
Expand Up @@ -253,7 +253,8 @@ def should_render_without_layout
# Example:
#
# should_redirect_to '"/"'
# should_redirect_to "users_url(@user)"
# should_redirect_to "user_url(@user)"
# should_redirect_to "users_url"
def should_redirect_to(url)
should "redirect to #{url.inspect}" do
instantiate_variables_from_assigns do
Expand Down
1 change: 1 addition & 0 deletions rails/init.rb
View file
@@ -0,0 +1 @@
require 'shoulda/rails'

0 comments on commit 97fef43

Please sign in to comment.