From b4019d5080181a911f8652cabd184794963911b6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jos=C3=A9=20Valim?= Date: Thu, 14 Jan 2010 22:03:47 +0100 Subject: Rails::Subscriber is now responsible for flushing all loggers it's responsible for. --- railties/test/subscriber_test.rb | 23 ++++++++++++++++++++++- 1 file changed, 22 insertions(+), 1 deletion(-) (limited to 'railties/test') diff --git a/railties/test/subscriber_test.rb b/railties/test/subscriber_test.rb index db953bb041..7e91b7333f 100644 --- a/railties/test/subscriber_test.rb +++ b/railties/test/subscriber_test.rb @@ -76,10 +76,31 @@ module SubscriberTest def test_does_not_send_the_event_if_logger_is_nil Rails.logger = nil + @subscriber.expects(:some_event).never Rails::Subscriber.add :my_subscriber, @subscriber instrument "my_subscriber.some_event" wait - assert_equal [], @logger.logged(:info) + end + + def test_flushes_loggers + Rails::Subscriber.add :my_subscriber, @subscriber + Rails::Subscriber.flush_all! + assert_equal 1, @logger.flush_count + end + + def test_flushes_loggers_when_action_dispatch_callback_is_received + Rails::Subscriber.add :my_subscriber, @subscriber + instrument "action_dispatch.callback" + wait + assert_equal 1, @logger.flush_count + end + + def test_flushes_the_same_logger_just_once + Rails::Subscriber.add :my_subscriber, @subscriber + Rails::Subscriber.add :another, @subscriber + instrument "action_dispatch.callback" + wait + assert_equal 1, @logger.flush_count end class SyncSubscriberTest < ActiveSupport::TestCase -- cgit v1.2.3