aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJosé Valim <jose.valim@gmail.com>2012-05-17 08:52:40 +0200
committerJosé Valim <jose.valim@gmail.com>2012-05-17 08:52:40 +0200
commit313f292e27bfb093a59b8aee4d606f2c63391c7c (patch)
tree48e19dea03f3a6ef726d0ea2752c1ed6ec466a36
parent6057a1d9d2522e27e8c9f2f44aab8cd2815f24da (diff)
downloadrails-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.rb9
-rw-r--r--activesupport/test/log_subscriber_test.rb2
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