aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAaron Patterson <aaron.patterson@gmail.com>2011-08-04 14:07:23 -0700
committerXavier Noria <fxn@hashref.com>2011-08-13 16:22:29 -0700
commita7a3169cc33aa06b6d3b70889c577d067a6c41ae (patch)
treecedddb976854a1d72dc49e8d0937f43346eff7a6
parent886d0115dc766865eada9308839ee827d9f13cad (diff)
downloadrails-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.rb13
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