diff options
author | Paco Guzman <fjguzman@aspgems.com> | 2010-05-15 13:26:33 +0200 |
---|---|---|
committer | José Valim <jose.valim@gmail.com> | 2010-05-15 16:31:36 +0200 |
commit | cdf700147c5b3ec5e4d1e7de1c2d08134568c2d1 (patch) | |
tree | 86dc957b6048393fe1c83ff3e7a9bf02536e0dce /actionpack/lib/action_dispatch | |
parent | 6f0ed7aa521e55bb2c12a29b86c0e8e68468cef8 (diff) | |
download | rails-cdf700147c5b3ec5e4d1e7de1c2d08134568c2d1.tar.gz rails-cdf700147c5b3ec5e4d1e7de1c2d08134568c2d1.tar.bz2 rails-cdf700147c5b3ec5e4d1e7de1c2d08134568c2d1.zip |
fix assert_select messages to its declaration behaviour
Signed-off-by: José Valim <jose.valim@gmail.com>
Diffstat (limited to 'actionpack/lib/action_dispatch')
-rw-r--r-- | actionpack/lib/action_dispatch/testing/assertions/selector.rb | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/actionpack/lib/action_dispatch/testing/assertions/selector.rb b/actionpack/lib/action_dispatch/testing/assertions/selector.rb index a6b1126e2b..9deabf5b3c 100644 --- a/actionpack/lib/action_dispatch/testing/assertions/selector.rb +++ b/actionpack/lib/action_dispatch/testing/assertions/selector.rb @@ -298,10 +298,14 @@ module ActionDispatch # found one but expecting two. message ||= content_mismatch if matches.empty? # Test minimum/maximum occurrence. - min, max = equals[:minimum], equals[:maximum] - message = message || %(Expected #{count_description(min, max)} matching "#{selector.to_s}", found #{matches.size}.) - assert matches.size >= min, message if min - assert matches.size <= max, message if max + min, max, count = equals[:minimum], equals[:maximum], equals[:count] + message = message || %(Expected #{count_description(min, max, count)} matching "#{selector.to_s}", found #{matches.size}.) + if count + assert matches.size == count, message + else + assert matches.size >= min, message if min + assert matches.size <= max, message if max + end # If a block is given call that block. Set @selected to allow # nested assert_select, which can be nested several levels deep. @@ -318,11 +322,13 @@ module ActionDispatch matches end - def count_description(min, max) #:nodoc: + def count_description(min, max, count) #:nodoc: pluralize = lambda {|word, quantity| word << (quantity == 1 ? '' : 's')} if min && max && (max != min) "between #{min} and #{max} elements" + elsif min && max && max == min && count + "exactly #{count} #{pluralize['element', min]}" elsif min && !(min == 1 && max == 1) "at least #{min} #{pluralize['element', min]}" elsif max |