diff options
author | José Valim <jose.valim@gmail.com> | 2012-05-17 08:52:40 +0200 |
---|---|---|
committer | José Valim <jose.valim@gmail.com> | 2012-05-17 08:52:40 +0200 |
commit | 313f292e27bfb093a59b8aee4d606f2c63391c7c (patch) | |
tree | 48e19dea03f3a6ef726d0ea2752c1ed6ec466a36 | |
parent | 6057a1d9d2522e27e8c9f2f44aab8cd2815f24da (diff) | |
download | rails-313f292e27bfb093a59b8aee4d606f2c63391c7c.tar.gz rails-313f292e27bfb093a59b8aee4d606f2c63391c7c.tar.bz2 rails-313f292e27bfb093a59b8aee4d606f2c63391c7c.zip |
Ensure blocks given to subscriber are properly yield
-rw-r--r-- | activesupport/lib/active_support/log_subscriber/test_helper.rb | 9 | ||||
-rw-r--r-- | activesupport/test/log_subscriber_test.rb | 2 |
2 files changed, 6 insertions, 5 deletions
diff --git a/activesupport/lib/active_support/log_subscriber/test_helper.rb b/activesupport/lib/active_support/log_subscriber/test_helper.rb index 6e1502989b..9d01dc85c0 100644 --- a/activesupport/lib/active_support/log_subscriber/test_helper.rb +++ b/activesupport/lib/active_support/log_subscriber/test_helper.rb @@ -61,11 +61,12 @@ module ActiveSupport @logged = Hash.new { |h,k| h[k] = [] } end - def debug - end - def method_missing(level, message) - @logged[level] << message + if block_given? + @logged[level] << yield + else + @logged[level] << message + end end def logged(level) diff --git a/activesupport/test/log_subscriber_test.rb b/activesupport/test/log_subscriber_test.rb index 8e160714b1..2a0e8d20ed 100644 --- a/activesupport/test/log_subscriber_test.rb +++ b/activesupport/test/log_subscriber_test.rb @@ -11,7 +11,7 @@ class MyLogSubscriber < ActiveSupport::LogSubscriber def foo(event) debug "debug" - info "info" + info { "info" } warn "warn" end |