coerce tests in sqlserver gem rather than rails

Author: Anna

Rakefile

@@ -21,7 +21,7 @@ end
 def test_files
   test_setup = ["test/cases/sqlserver_helper.rb"]
 
-  return test_setup+(ENV['TEST_FILES']).split(',') if ENV['TEST_FILES']
+  return test_setup + (ENV['TEST_FILES']).split(',') if ENV['TEST_FILES']
 
   sqlserver_cases = Dir.glob("test/cases/**/*_test_sqlserver.rb")
 

test/cases/adapter_test_sqlserver.rb

@@ -799,3 +799,12 @@ class AdapterTestSqlserver < ActiveRecord::TestCase
   end
 
 end
+
+
+class AdapterTest < ActiveRecord::TestCase
+    COERCED_TESTS = [:test_update_prepared_statement] 
+    # Like PostgreSQL, SQL Server does not support null bytes in strings.
+    # This is not run for PostgreSQL at the rails level and the same should happen for SQL Server
+    # Until that patch is made to rails we are preventing this test from running in this gem.
+  include SqlserverCoercedTest
+end

test/cases/associations_test_sqlserver.rb

@@ -0,0 +1,16 @@
+require 'cases/sqlserver_helper'
+require 'models/owner'
+
+class HasManyThroughAssociationsTest <  ActiveRecord::TestCase
+    COERCED_TESTS = [:test_has_many_through_obeys_order_on_through_association] 
+    # Rails does not do a case-insensive comparison
+    # Until that patch is made to rails we are preventing this test from running in this gem.
+
+  include SqlserverCoercedTest
+  def test_coerced_has_many_through_obeys_order_on_through_association
+    owner = owners(:blackbeard)
+    # assert owner.toys.to_sql.include?("pets.name desc") # What's currently in rails
+    assert owner.toys.to_sql.downcase.include?("pets.name desc")
+    assert_equal ["parrot", "bulbul"], owner.toys.map { |r| r.pet.name }
+  end
+end

test/cases/base_test_sqlserver.rb

@@ -4,7 +4,11 @@
 
 class BasicsTest < ActiveRecord::TestCase
 
-  COERCED_TESTS = [:test_column_names_are_escaped]
+  COERCED_TESTS = [:test_column_names_are_escaped, 
+  :test_respect_internal_encoding]
+  # test_respect_internal_encoding is not run for PostgreSQL at the rails level and the same should happen for SQL Server
+  # Until that patch is made to rails we are preventing this test from running in this gem.
+
 
   include SqlserverCoercedTest
 

test/cases/column_test_sqlserver.rb

@@ -3,10 +3,13 @@
 require 'models_sqlserver/float_data'
 require 'models_sqlserver/numeric_data'
 require 'models_sqlserver/sql_server_chronic'
+require 'models_sqlserver/sql_server_edge_schema'
 require 'models_sqlserver/sql_server_string'
 require 'models_sqlserver/sql_server_unicode'
 require 'models_sqlserver/table_with_real_column'
+
 require 'models_sqlserver/topic'
+require "cases/migration/helper"
 
 class ColumnTestSqlserver < ActiveRecord::TestCase
 
@@ -339,7 +342,37 @@ class ColumnTestSqlserver < ActiveRecord::TestCase
       assert_equal 'tinyint(1)', @tinyint.sql_type
     end
 
-  end
-  
-  
+  end  
 end
+
+module ActiveRecord
+  class Migration
+    class ColumnsTest < ActiveRecord::TestCase
+      include ActiveRecord::Migration::TestHelper
+
+      COERCED_TESTS = [:test_remove_column_with_multi_column_index] 
+        # The if current_adapter? conditional below should also contain :SQLServerAdapter.
+        # Until that patch is made to rails we are preventing this test from running in this gem.
+
+      include SqlserverCoercedTest
+
+      #the only thing we changed in this method was to add :SQLServerAdapter
+      def test_coerced_remove_column_with_multi_column_index
+        add_column "test_models", :hat_size, :integer
+        add_column "test_models", :hat_style, :string, :limit => 100
+        add_index "test_models", ["hat_style", "hat_size"], :unique => true
+
+        assert_equal 1, connection.indexes('test_models').size
+        remove_column("test_models", "hat_size")
+
+        # Every database and/or database adapter has their own behavior
+        # if it drops the multi-column index when any of the indexed columns dropped by remove_column.
+        if current_adapter?(:PostgreSQLAdapter, :OracleAdapter, :SQLServerAdapter)
+          assert_equal [], connection.indexes('test_models').map(&:name)
+        else
+          assert_equal ['index_test_models_on_hat_style_and_hat_size'], connection.indexes('test_models').map(&:name)
+        end
+      end
+    end
+  end
+end