From 29d564add926307fea8cd3f77caf88d735e252a1 Mon Sep 17 00:00:00 2001 From: Xavier Noria Date: Mon, 26 Nov 2012 07:43:43 -0800 Subject: Merge pull request #6197 from blakesmith/connection_adapters_without_explain_support Don't run explain on slow queries for database adapters that don't support it --- activerecord/test/cases/explain_test.rb | 10 ++++++++++ 1 file changed, 10 insertions(+) (limited to 'activerecord/test/cases/explain_test.rb') diff --git a/activerecord/test/cases/explain_test.rb b/activerecord/test/cases/explain_test.rb index cb7781f8e7..bdeb0a033c 100644 --- a/activerecord/test/cases/explain_test.rb +++ b/activerecord/test/cases/explain_test.rb @@ -98,6 +98,16 @@ if ActiveRecord::Base.connection.supports_explain? assert_equal expected, base.exec_explain(queries) end + def test_unsupported_connection_adapter + connection.stubs(:supports_explain?).returns(false) + + base.logger.expects(:warn).never + + with_threshold(0) do + Car.where(:name => 'honda').to_a + end + end + def test_silence_auto_explain base.expects(:collecting_sqls_for_explain).never base.logger.expects(:warn).never -- cgit v1.2.3