diff options
author | Aaron Patterson <aaron.patterson@gmail.com> | 2011-08-04 14:07:23 -0700 |
---|---|---|
committer | Xavier Noria <fxn@hashref.com> | 2011-08-13 16:22:29 -0700 |
commit | a7a3169cc33aa06b6d3b70889c577d067a6c41ae (patch) | |
tree | cedddb976854a1d72dc49e8d0937f43346eff7a6 | |
parent | 886d0115dc766865eada9308839ee827d9f13cad (diff) | |
download | rails-a7a3169cc33aa06b6d3b70889c577d067a6c41ae.tar.gz rails-a7a3169cc33aa06b6d3b70889c577d067a6c41ae.tar.bz2 rails-a7a3169cc33aa06b6d3b70889c577d067a6c41ae.zip |
fixing assert_difference issues on ruby 1.8
-rw-r--r-- | activesupport/lib/active_support/testing/assertions.rb | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/activesupport/lib/active_support/testing/assertions.rb b/activesupport/lib/active_support/testing/assertions.rb index ba34e9853c..f3629ada5b 100644 --- a/activesupport/lib/active_support/testing/assertions.rb +++ b/activesupport/lib/active_support/testing/assertions.rb @@ -45,18 +45,19 @@ module ActiveSupport # post :delete, :id => ... # end def assert_difference(expression, difference = 1, message = nil, &block) - exps = Array.wrap(expression).map { |e| - callee = e.respond_to?(:call) ? e : lambda { eval(e, block.binding) } - [e, callee] + expressions = Array.wrap expression + + exps = expressions.map { |e| + e.respond_to?(:call) ? e : lambda { eval(e, block.binding) } } - before = exps.map { |_, block| block.call } + before = exps.map { |e| e.call } yield - exps.each_with_index do |(code, block), i| + expressions.zip(exps).each_with_index do |(code, e), i| error = "#{code.inspect} didn't change by #{difference}" error = "#{message}.\n#{error}" if message - assert_equal(before[i] + difference, block.call, error) + assert_equal(before[i] + difference, e.call, error) end end |