Fix a few test from ActiveRecord 3.2.6 upgrade.

Author: metaskills

CHANGELOG

@@ -1,4 +1,9 @@
 
+* 3.2.5 *
+
+* Fix a few test from ActiveRecord 3.2.6 upgrade.
+
+
 * 3.2.4 *
 
 * Fixed schema reflection for identity columns using ODBC. Fixes #193.

lib/active_record/connection_adapters/sqlserver/schema_statements.rb

@@ -50,6 +50,7 @@ def rename_table(table_name, new_name)
 
         def remove_column(table_name, *column_names)
           raise ArgumentError.new("You must specify at least one column name.  Example: remove_column(:people, :first_name)") if column_names.empty?
+          ActiveSupport::Deprecation.warn 'Passing array to remove_columns is deprecated, please use multiple arguments, like: `remove_columns(:posts, :foo, :bar)`', caller if column_names.flatten!
           column_names.flatten.each do |column_name|
             remove_check_constraints(table_name, column_name)
             remove_default_constraint(table_name, column_name)

lib/active_record/connection_adapters/sqlserver/version.rb

@@ -3,7 +3,7 @@ module ConnectionAdapters
     module Sqlserver
       module Version
 
-        VERSION = '3.2.4'
+        VERSION = '3.2.5'
 
       end
     end

test/cases/finder_test_sqlserver.rb

@@ -7,12 +7,19 @@ class FinderTestSqlserver < ActiveRecord::TestCase
 class FinderTest < ActiveRecord::TestCase
 
   COERCED_TESTS = [
+    :test_exists_does_not_select_columns_without_alias,
     :test_string_sanitation,
     :test_first_and_last_with_integer_should_use_sql_limit
   ]
 
   include SqlserverCoercedTest
 
+  def test_coerced_exists_does_not_select_columns_without_alias
+    assert_sql(/SELECT TOP \(1\) 1 AS one FROM \[topics\]/i) do
+      Topic.exists?
+    end
+  end
+
   def test_coerced_string_sanitation
     assert_not_equal "N'something ' 1=1'", ActiveRecord::Base.sanitize("something ' 1=1")
     assert_equal "N'something; select table'", ActiveRecord::Base.sanitize("something; select table")