diff options
| author | Vishal Lal <vish61@gmail.com> | 2014-03-22 22:50:55 +0000 |
|---|---|---|
| committer | Vishal Lal <vish61@gmail.com> | 2014-03-22 22:58:37 +0000 |
| commit | 0ebae1dbc59c34f6b5ae87bf96b8848051a0254c (patch) | |
| tree | 0b31e059da70d8dcbb9243fd89b8f634e6a330ed | |
| parent | 9d44b3f886847eee9c70097165863f8e37d3d1d8 (diff) | |
| download | rails-0ebae1dbc59c34f6b5ae87bf96b8848051a0254c.tar.gz rails-0ebae1dbc59c34f6b5ae87bf96b8848051a0254c.tar.bz2 rails-0ebae1dbc59c34f6b5ae87bf96b8848051a0254c.zip | |
Swapped parameters of assert_equal in assert_select
| -rw-r--r-- | actionpack/CHANGELOG.md | 7 | ||||
| -rw-r--r-- | actionpack/lib/action_dispatch/testing/assertions/selector.rb | 2 |
2 files changed, 8 insertions, 1 deletions
diff --git a/actionpack/CHANGELOG.md b/actionpack/CHANGELOG.md index 59460cbe82..fe5b38d29f 100644 --- a/actionpack/CHANGELOG.md +++ b/actionpack/CHANGELOG.md @@ -1,3 +1,10 @@ +* Swapped the parameters of assert_equal in `assert_select` so that the + proper values were printed correctly + + Fixes #14422. + + *Vishal Lal* + * The method `shallow?` returns false if the parent resource is a singleton so we need to check if we're not inside a nested scope before copying the :path and :as options to their shallow equivalents. diff --git a/actionpack/lib/action_dispatch/testing/assertions/selector.rb b/actionpack/lib/action_dispatch/testing/assertions/selector.rb index 3253a3d424..8a128427bf 100644 --- a/actionpack/lib/action_dispatch/testing/assertions/selector.rb +++ b/actionpack/lib/action_dispatch/testing/assertions/selector.rb @@ -291,7 +291,7 @@ module ActionDispatch # so is this custom message really needed? message = message || %(Expected #{count_description(min, max, count)} matching "#{selector.to_s}", found #{matches.size}.) if count - assert_equal matches.size, count, message + assert_equal count, matches.size, message else assert_operator matches.size, :>=, min, message if min assert_operator matches.size, :<=, max, message if max |