diff options
author | lsylvester <lachlan.sylvester@hypothetical.com.au> | 2018-07-28 23:50:03 +1000 |
---|---|---|
committer | Kasper Timm Hansen <kaspth@gmail.com> | 2018-07-28 15:50:03 +0200 |
commit | 845cbb4bb2869444379354a87f83553faaadc961 (patch) | |
tree | ace7f077dfce904a6a03f92a0ccfb2b91fd9b883 /activerecord/test | |
parent | 45903be1344992b1f0156cd88c351067bdc93726 (diff) | |
download | rails-845cbb4bb2869444379354a87f83553faaadc961.tar.gz rails-845cbb4bb2869444379354a87f83553faaadc961.tar.bz2 rails-845cbb4bb2869444379354a87f83553faaadc961.zip |
Avoid logging ActiveRecord::LogSubscriber as the query source when the source is ignored (#33455)
Diffstat (limited to 'activerecord/test')
-rw-r--r-- | activerecord/test/cases/log_subscriber_test.rb | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/activerecord/test/cases/log_subscriber_test.rb b/activerecord/test/cases/log_subscriber_test.rb index e2742ed33e..3f26eaa88d 100644 --- a/activerecord/test/cases/log_subscriber_test.rb +++ b/activerecord/test/cases/log_subscriber_test.rb @@ -177,11 +177,24 @@ class LogSubscriberTest < ActiveRecord::TestCase logger = TestDebugLogSubscriber.new logger.sql(Event.new(0, sql: "hi mom!")) + assert_equal 2, @logger.logged(:debug).size assert_match(/↳/, @logger.logged(:debug).last) ensure ActiveRecord::Base.verbose_query_logs = false end + def test_verbose_query_with_ignored_callstack + ActiveRecord::Base.verbose_query_logs = true + ActiveRecord::LogSubscriber.ignored_callstack_paths.push("/") + logger = TestDebugLogSubscriber.new + logger.sql(Event.new(0, sql: "hi mom!")) + assert_equal 1, @logger.logged(:debug).size + assert_no_match(/↳/, @logger.logged(:debug).last) + ensure + ActiveRecord::LogSubscriber.ignored_callstack_paths.delete("/") + ActiveRecord::Base.verbose_query_logs = false + end + def test_verbose_query_logs_disabled_by_default logger = TestDebugLogSubscriber.new logger.sql(Event.new(0, sql: "hi mom!")) |