diff --git a/activerecord/test/base_test.rb b/activerecord/test/base_test.rb index 5e20cbe127..103f9f7271 100755 --- a/activerecord/test/base_test.rb +++ b/activerecord/test/base_test.rb @@ -569,29 +569,26 @@ class BasicsTest < Test::Unit::TestCase assert_equal -2, Topic.find(2).replies_count end - # The ADO library doesn't support the number of affected rows - unless current_adapter?(:SQLServerAdapter) - def test_update_all - assert_equal 2, Topic.update_all("content = 'bulk updated!'") - assert_equal "bulk updated!", Topic.find(1).content - assert_equal "bulk updated!", Topic.find(2).content + def test_update_all + assert_equal 2, Topic.update_all("content = 'bulk updated!'") + assert_equal "bulk updated!", Topic.find(1).content + assert_equal "bulk updated!", Topic.find(2).content - assert_equal 2, Topic.update_all(['content = ?', 'bulk updated again!']) - assert_equal "bulk updated again!", Topic.find(1).content - assert_equal "bulk updated again!", Topic.find(2).content + assert_equal 2, Topic.update_all(['content = ?', 'bulk updated again!']) + assert_equal "bulk updated again!", Topic.find(1).content + assert_equal "bulk updated again!", Topic.find(2).content - assert_equal 2, Topic.update_all(['content = ?', nil]) - assert_nil Topic.find(1).content - end + assert_equal 2, Topic.update_all(['content = ?', nil]) + assert_nil Topic.find(1).content + end - def test_update_all_with_hash - assert_not_nil Topic.find(1).last_read - assert_equal 2, Topic.update_all(:content => 'bulk updated with hash!', :last_read => nil) - assert_equal "bulk updated with hash!", Topic.find(1).content - assert_equal "bulk updated with hash!", Topic.find(2).content - assert_nil Topic.find(1).last_read - assert_nil Topic.find(2).last_read - end + def test_update_all_with_hash + assert_not_nil Topic.find(1).last_read + assert_equal 2, Topic.update_all(:content => 'bulk updated with hash!', :last_read => nil) + assert_equal "bulk updated with hash!", Topic.find(1).content + assert_equal "bulk updated with hash!", Topic.find(2).content + assert_nil Topic.find(1).last_read + assert_nil Topic.find(2).last_read end if current_adapter?(:MysqlAdapter) @@ -610,9 +607,6 @@ class BasicsTest < Test::Unit::TestCase end def test_delete_all - # The ADO library doesn't support the number of affected rows - return true if current_adapter?(:SQLServerAdapter) - assert_equal 2, Topic.delete_all end