From 125c5ccf383bb9b131ff87ec5dae0a34dd0bbbea Mon Sep 17 00:00:00 2001 From: Justin George Date: Thu, 28 Oct 2010 11:50:25 -0700 Subject: [PATCH] Add tests to demonstrate lack of proper operation naming in sql --- activerecord/test/cases/log_subscriber_test.rb | 26 ++++++++++++++++++++++++ 1 files changed, 26 insertions(+), 0 deletions(-) diff --git a/activerecord/test/cases/log_subscriber_test.rb b/activerecord/test/cases/log_subscriber_test.rb index cbaaca7..0635db0 100644 --- a/activerecord/test/cases/log_subscriber_test.rb +++ b/activerecord/test/cases/log_subscriber_test.rb @@ -30,6 +30,32 @@ class LogSubscriberTest < ActiveRecord::TestCase assert_match(/SELECT .*?FROM .?developers.?/i, @logger.logged(:debug).last) end + def test_creation_query_logging + Developer.create + wait + assert_equal 1, @logger.logged(:debug).size + assert_match(/Developer Create/, @logger.logged(:debug).last) + assert_match(/Insert .*?INTO .?developers.?/i, @logger.logged(:debug).last) + end + + def test_updating_query_logging + x = Developer.create + x.update_attributes({}) + wait + assert_equal 2, @logger.logged(:debug).size + assert_match(/Developer Update/, @logger.logged(:debug).last) + assert_match(/UPDATE .*?developers.?/i, @logger.logged(:debug).last) + end + + def test_deleting_query_logging + x = Developer.create + x.destroy + wait + assert_equal 2, @logger.logged(:debug).size + assert_match(/Developer Destroy/, @logger.logged(:debug).last) + assert_match(/DELETE .*?FROM .?developers.?/, @logger.logged(:debug).last) + end + def test_cached_queries ActiveRecord::Base.cache do Developer.all -- 1.7.0.3