activerecord-sqlserver-adapter 5.2.1 → 6.0.2

data/test/cases/change_column_null_test_sqlserver.rb

@@ -1,5 +1,7 @@
-require 'cases/helper_sqlserver'
-require 'migrations/create_clients_and_change_column_null'
+# frozen_string_literal: true
+
+require "cases/helper_sqlserver"
+require "migrations/create_clients_and_change_column_null"
 
 class ChangeColumnNullTestSqlServer < ActiveRecord::TestCase
   before do
@@ -17,25 +19,25 @@ class ChangeColumnNullTestSqlServer < ActiveRecord::TestCase
     table.find { |column| column.name == name }
   end
 
-  let(:clients_table) { connection.columns('clients') }
-  let(:name_column) { find_column(clients_table, 'name') }
-  let(:code_column) { find_column(clients_table, 'code') }
-  let(:value_column) { find_column(clients_table, 'value') }
+  let(:clients_table) { connection.columns("clients") }
+  let(:name_column) { find_column(clients_table, "name") }
+  let(:code_column) { find_column(clients_table, "code") }
+  let(:value_column) { find_column(clients_table, "value") }
 
-  describe '#change_column_null' do
-    it 'does not change the column limit' do
+  describe "#change_column_null" do
+    it "does not change the column limit" do
       _(name_column.limit).must_equal 15
     end
 
-    it 'does not change the column default' do
-      _(code_column.default).must_equal 'n/a'
+    it "does not change the column default" do
+      _(code_column.default).must_equal "n/a"
     end
 
-    it 'does not change the column precision' do
+    it "does not change the column precision" do
       _(value_column.precision).must_equal 32
     end
 
-    it 'does not change the column scale' do
+    it "does not change the column scale" do
       _(value_column.scale).must_equal 8
     end
   end

data/test/cases/coerced_tests.rb

@@ -1,10 +1,10 @@
-require 'cases/helper_sqlserver'
+# frozen_string_literal: true
 
+require "cases/helper_sqlserver"
 
-
-require 'models/event'
+require "models/event"
 class UniquenessValidationTest < ActiveRecord::TestCase
-  # So sp_executesql swallows this exception. Run without prpared to see it.
+  # So sp_executesql swallows this exception. Run without prepared to see it.
   coerce_tests! :test_validate_uniqueness_with_limit
   def test_validate_uniqueness_with_limit_coerced
     connection.unprepared_statement do
@@ -14,7 +14,7 @@ class UniquenessValidationTest < ActiveRecord::TestCase
     end
   end
 
-  # So sp_executesql swallows this exception. Run without prpared to see it.
+  # So sp_executesql swallows this exception. Run without prepared to see it.
   coerce_tests! :test_validate_uniqueness_with_limit_and_utf8
   def test_validate_uniqueness_with_limit_and_utf8_coerced
     connection.unprepared_statement do
@@ -23,66 +23,125 @@ class UniquenessValidationTest < ActiveRecord::TestCase
       end
     end
   end
-end
-
 
+  # Skip the test if database is case-insensitive.
+  coerce_tests! :test_validate_case_sensitive_uniqueness_by_default
+  def test_validate_case_sensitive_uniqueness_by_default_coerced
+    database_collation = connection.select_one("SELECT collation_name FROM sys.databases WHERE name = 'activerecord_unittest'").values.first
+    skip if database_collation.include?("_CI_")
 
+    original_test_validate_case_sensitive_uniqueness_by_default_coerced
+  end
+end
 
-require 'models/event'
+require "models/event"
 module ActiveRecord
   class AdapterTest < ActiveRecord::TestCase
-    # I really dont think we can support legacy binds.
+    # I really don`t think we can support legacy binds.
     coerce_tests! :test_select_all_with_legacy_binds
     coerce_tests! :test_insert_update_delete_with_legacy_binds
 
     # As far as I can tell, SQL Server does not support null bytes in strings.
     coerce_tests! :test_update_prepared_statement
 
-    # So sp_executesql swallows this exception. Run without prpared to see it.
+    # So sp_executesql swallows this exception. Run without prepared to see it.
     coerce_tests! :test_value_limit_violations_are_translated_to_specific_exception
     def test_value_limit_violations_are_translated_to_specific_exception_coerced
       connection.unprepared_statement do
         error = assert_raises(ActiveRecord::ValueTooLong) do
-          Event.create(title: 'abcdefgh')
+          Event.create(title: "abcdefgh")
         end
         assert_not_nil error.cause
       end
     end
+
+    # Fix randomly failing test. The loading of the model's schema was affecting the test.
+    coerce_tests! :test_errors_when_an_insert_query_is_called_while_preventing_writes
+    def test_errors_when_an_insert_query_is_called_while_preventing_writes_coerced
+      Subscriber.send(:load_schema!)
+      original_test_errors_when_an_insert_query_is_called_while_preventing_writes
+    end
   end
 end
 
+module ActiveRecord
+  class AdapterTestWithoutTransaction < ActiveRecord::TestCase
+    # SQL Server does not allow truncation of tables that are referenced by foreign key
+    # constraints. So manually remove/add foreign keys in test.
+    coerce_tests! :test_truncate_tables
+    def test_truncate_tables_coerced
+      # Remove foreign key constraint to allow truncation.
+      @connection.remove_foreign_key :authors, :author_addresses
+
+      assert_operator Post.count, :>, 0
+      assert_operator Author.count, :>, 0
+      assert_operator AuthorAddress.count, :>, 0
+
+      @connection.truncate_tables("author_addresses", "authors", "posts")
+
+      assert_equal 0, Post.count
+      assert_equal 0, Author.count
+      assert_equal 0, AuthorAddress.count
+    ensure
+      reset_fixtures("posts", "authors", "author_addresses")
+
+      # Restore foreign key constraint.
+      @connection.add_foreign_key :authors, :author_addresses
+    end
+
+    # SQL Server does not allow truncation of tables that are referenced by foreign key
+    # constraints. So manually remove/add foreign keys in test.
+    coerce_tests! :test_truncate_tables_with_query_cache
+    def test_truncate_tables_with_query_cache
+      # Remove foreign key constraint to allow truncation.
+      @connection.remove_foreign_key :authors, :author_addresses
 
+      @connection.enable_query_cache!
 
+      assert_operator Post.count, :>, 0
+      assert_operator Author.count, :>, 0
+      assert_operator AuthorAddress.count, :>, 0
 
-require 'models/topic'
+      @connection.truncate_tables("author_addresses", "authors", "posts")
+
+      assert_equal 0, Post.count
+      assert_equal 0, Author.count
+      assert_equal 0, AuthorAddress.count
+    ensure
+      reset_fixtures("posts", "authors", "author_addresses")
+      @connection.disable_query_cache!
+
+      # Restore foreign key constraint.
+      @connection.add_foreign_key :authors, :author_addresses
+    end
+  end
+end
+
+require "models/topic"
 class AttributeMethodsTest < ActiveRecord::TestCase
+  # Use IFF for boolean statement in SELECT
   coerce_tests! %r{typecast attribute from select to false}
   def test_typecast_attribute_from_select_to_false_coerced
-    Topic.create(:title => 'Budget')
+    Topic.create(:title => "Budget")
     topic = Topic.all.merge!(:select => "topics.*, IIF (1 = 2, 1, 0) as is_test").first
-    assert !topic.is_test?
+    assert_not_predicate topic, :is_test?
   end
 
+  # Use IFF for boolean statement in SELECT
   coerce_tests! %r{typecast attribute from select to true}
   def test_typecast_attribute_from_select_to_true_coerced
-    Topic.create(:title => 'Budget')
+    Topic.create(:title => "Budget")
     topic = Topic.all.merge!(:select => "topics.*, IIF (1 = 1, 1, 0) as is_test").first
-    assert topic.is_test?
+    assert_predicate topic, :is_test?
   end
 end
 
-
-class NumericDataTest < ActiveRecord::TestCase
-  # We do not have do the DecimalWithoutScale type.
-  coerce_tests! :test_numeric_fields
-  coerce_tests! :test_numeric_fields_with_scale
-end
-
 class BasicsTest < ActiveRecord::TestCase
+  # Use square brackets as SQL Server escaped character
   coerce_tests! :test_column_names_are_escaped
   def test_column_names_are_escaped_coerced
     conn = ActiveRecord::Base.connection
-    assert_equal '[t]]]', conn.quote_column_name('t]')
+    assert_equal "[t]]]", conn.quote_column_name("t]")
   end
 
   # Just like PostgreSQLAdapter does.
@@ -96,52 +155,60 @@ class BasicsTest < ActiveRecord::TestCase
     Time.use_zone("Eastern Time (US & Canada)") do
       topic = Topic.find(1)
       time = Time.zone.parse("2017-07-17 10:56")
-      topic.update_attributes!(written_on: time)
+      topic.update!(written_on: time)
       assert_equal(time, topic.written_on)
     end
   end
 
   def test_update_date_time_attributes_with_default_timezone_local
-    with_env_tz 'America/New_York' do
+    with_env_tz "America/New_York" do
       with_timezone_config default: :local do
         Time.use_zone("Eastern Time (US & Canada)") do
           topic = Topic.find(1)
           time = Time.zone.parse("2017-07-17 10:56")
-          topic.update_attributes!(written_on: time)
+          topic.update!(written_on: time)
           assert_equal(time, topic.written_on)
         end
       end
     end
   end
 
-  # Need to escape `quoted_id` once it contains brackets
-  coerce_tests! %r{column names are quoted when using #from clause and model has ignored columns}
-  test "column names are quoted when using #from clause and model has ignored columns coerced" do
-    refute_empty Developer.ignored_columns
-    query = Developer.from("developers").to_sql
-    quoted_id = "#{Developer.quoted_table_name}.#{Developer.quoted_primary_key}"
-
-    assert_match(/SELECT #{Regexp.escape(quoted_id)}.* FROM developers/, query)
+  # SQL Server does not have query for release_savepoint
+  coerce_tests! %r{an empty transaction does not raise if preventing writes}
+  test "an empty transaction does not raise if preventing writes coerced" do
+    ActiveRecord::Base.connection_handler.while_preventing_writes do
+      assert_queries(1, ignore_none: true) do
+        Bird.transaction do
+          ActiveRecord::Base.connection.materialize_transactions
+        end
+      end
+    end
   end
 end
 
-
-
-
 class BelongsToAssociationsTest < ActiveRecord::TestCase
   # Since @client.firm is a single first/top, and we use FETCH the order clause is used.
   coerce_tests! :test_belongs_to_does_not_use_order_by
 
+  # Square brackets around column name
   coerce_tests! :test_belongs_to_with_primary_key_joins_on_correct_column
   def test_belongs_to_with_primary_key_joins_on_correct_column_coerced
     sql = Client.joins(:firm_with_primary_key).to_sql
     assert_no_match(/\[firm_with_primary_keys_companies\]\.\[id\]/, sql)
     assert_match(/\[firm_with_primary_keys_companies\]\.\[name\]/, sql)
   end
-end
-
-
 
+  # Asserted SQL to get one row different from original test.
+  coerce_tests! :test_belongs_to
+  def test_belongs_to_coerced
+    client = Client.find(3)
+    first_firm = companies(:first_firm)
+    assert_sql(/FETCH NEXT @(\d) ROWS ONLY(.)*@\1 = 1/) do
+      assert_equal first_firm, client.firm
+      assert_equal first_firm.name, client.firm.name
+    end
+  end
+end
 
 module ActiveRecord
   class BindParameterTest < ActiveRecord::TestCase
@@ -163,62 +230,53 @@ module ActiveRecord
     # SQL Server adapter does not use a statement cache as query plans are already reused using `EXEC sp_executesql`.
     coerce_tests! :test_statement_cache
     coerce_tests! :test_statement_cache_with_query_cache
+    coerce_tests! :test_statement_cache_with_find
     coerce_tests! :test_statement_cache_with_find_by
     coerce_tests! :test_statement_cache_with_in_clause
     coerce_tests! :test_statement_cache_with_sql_string_literal
   end
 end
 
-
 module ActiveRecord
   class InstrumentationTest < ActiveRecord::TestCase
-    # This fails randomly due to schema cache being lost?
+    # Fix randomly failing test. The loading of the model's schema was affecting the test.
     coerce_tests! :test_payload_name_on_load
     def test_payload_name_on_load_coerced
-      Book.create(name: "test book")
-      Book.first
-      subscriber = ActiveSupport::Notifications.subscribe("sql.active_record") do |*args|
-        event = ActiveSupport::Notifications::Event.new(*args)
-        if event.payload[:sql].match "SELECT"
-          assert_equal "Book Load", event.payload[:name]
-        end
-      end
-      Book.first
-    ensure
-      ActiveSupport::Notifications.unsubscribe(subscriber) if subscriber
+      Book.send(:load_schema!)
+      original_test_payload_name_on_load
     end
   end
 end
 
 class CalculationsTest < ActiveRecord::TestCase
-  # This fails randomly due to schema cache being lost?
+  # Fix randomly failing test. The loading of the model's schema was affecting the test.
   coerce_tests! :test_offset_is_kept
   def test_offset_is_kept_coerced
-    Account.first
-    queries = assert_sql { Account.offset(1).count }
-    assert_equal 1, queries.length
-    assert_match(/OFFSET/, queries.first)
+    Account.send(:load_schema!)
+    original_test_offset_is_kept
   end
 
   # Are decimal, not integer.
   coerce_tests! :test_should_return_decimal_average_of_integer_field
   def test_should_return_decimal_average_of_integer_field_coerced
     value = Account.average(:id)
-    assert_equal BigDecimal('3.0').to_s, BigDecimal(value).to_s
+    assert_equal BigDecimal("3.0").to_s, BigDecimal(value).to_s
   end
 
+  # Match SQL Server limit implementation
   coerce_tests! :test_limit_is_kept
   def test_limit_is_kept_coerced
     queries = capture_sql_ss { Account.limit(1).count }
     assert_equal 1, queries.length
-    _(queries.first).must_match %r{ORDER BY \[accounts\]\.\[id\] ASC OFFSET 0 ROWS FETCH NEXT @0 ROWS ONLY.*@0 = 1}
+    assert_match(/ORDER BY \[accounts\]\.\[id\] ASC OFFSET 0 ROWS FETCH NEXT @0 ROWS ONLY.*@0 = 1/, queries.first)
   end
 
+  # Match SQL Server limit implementation
   coerce_tests! :test_limit_with_offset_is_kept
   def test_limit_with_offset_is_kept_coerced
     queries = capture_sql_ss { Account.limit(1).offset(1).count }
     assert_equal 1, queries.length
-    _(queries.first).must_match %r{ORDER BY \[accounts\]\.\[id\] ASC OFFSET @0 ROWS FETCH NEXT @1 ROWS ONLY.*@0 = 1, @1 = 1}
+    assert_match(/ORDER BY \[accounts\]\.\[id\] ASC OFFSET @0 ROWS FETCH NEXT @1 ROWS ONLY.*@0 = 1, @1 = 1/, queries.first)
   end
 
   # SQL Server needs an alias for the calculated column
@@ -233,49 +291,57 @@ class CalculationsTest < ActiveRecord::TestCase
   coerce_tests! :test_having_with_strong_parameters
 end
 
-
-
 module ActiveRecord
   class Migration
     class ChangeSchemaTest < ActiveRecord::TestCase
-       # We test these.
-      coerce_tests! :test_create_table_with_bigint,
-                    :test_create_table_with_defaults
-    end
+      # Integer.default is a number and not a string
+      coerce_tests! :test_create_table_with_defaults
+      def test_create_table_with_defaults_coerce
+        connection.create_table :testings do |t|
+          t.column :one, :string, default: "hello"
+          t.column :two, :boolean, default: true
+          t.column :three, :boolean, default: false
+          t.column :four, :integer, default: 1
+          t.column :five, :text, default: "hello"
+        end
 
-    class ChangeSchemaWithDependentObjectsTest < ActiveRecord::TestCase
-      # In SQL Server you have to delete the tables yourself in the right order.
-      coerce_tests! :test_create_table_with_force_cascade_drops_dependent_objects
+        columns = connection.columns(:testings)
+        one = columns.detect { |c| c.name == "one" }
+        two = columns.detect { |c| c.name == "two" }
+        three = columns.detect { |c| c.name == "three" }
+        four = columns.detect { |c| c.name == "four" }
+        five = columns.detect { |c| c.name == "five" }
+
+        assert_equal "hello", one.default
+        assert_equal true, connection.lookup_cast_type_from_column(two).deserialize(two.default)
+        assert_equal false, connection.lookup_cast_type_from_column(three).deserialize(three.default)
+        assert_equal 1, four.default
+        assert_equal "hello", five.default
+      end
     end
   end
 end
 
-
-
 module ActiveRecord
   module ConnectionAdapters
     class QuoteARBaseTest < ActiveRecord::TestCase
-
       # Use our date format.
       coerce_tests! :test_quote_ar_object
       def test_quote_ar_object_coerced
         value = DatetimePrimaryKey.new(id: @time)
-        assert_equal "'02-14-2017 12:34:56.79'",  @connection.quote(value)
+        assert_equal "'02-14-2017 12:34:56.79'", @connection.quote(value)
       end
 
       # Use our date format.
       coerce_tests! :test_type_cast_ar_object
       def test_type_cast_ar_object_coerced
         value = DatetimePrimaryKey.new(id: @time)
-        assert_equal "02-14-2017 12:34:56.79",  @connection.type_cast(value)
+        assert_equal "02-14-2017 12:34:56.79", @connection.type_cast(value)
       end
-
     end
   end
 end
 
-
-
 module ActiveRecord
   class Migration
     class ColumnAttributesTest < ActiveRecord::TestCase
@@ -290,16 +356,13 @@ module ActiveRecord
   end
 end
 
-
-
-
 module ActiveRecord
   class Migration
-    class ColumnsTest
+    class ColumnsTest < ActiveRecord::TestCase
       # Our defaults are real 70000 integers vs '70000' strings.
       coerce_tests! :test_rename_column_preserves_default_value_not_null
       def test_rename_column_preserves_default_value_not_null_coerced
-        add_column 'test_models', 'salary', :integer, :default => 70000
+        add_column "test_models", "salary", :integer, :default => 70000
         default_before = connection.columns("test_models").find { |c| c.name == "salary" }.default
         assert_equal 70000, default_before
         rename_column "test_models", "salary", "annual_salary"
@@ -315,9 +378,9 @@ module ActiveRecord
         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
+        assert_equal 1, connection.indexes("test_models").size
         remove_column("test_models", "hat_size")
-        assert_equal [], connection.indexes('test_models').map(&:name)
+        assert_equal [], connection.indexes("test_models").map(&:name)
       end
 
       # Choose `StatementInvalid` vs `ActiveRecordError`.
@@ -332,61 +395,19 @@ module ActiveRecord
   end
 end
 
-
-
-
 class MigrationTest < ActiveRecord::TestCase
-  # We do not have do the DecimalWithoutScale type.
-  coerce_tests! :test_add_table_with_decimals
-  def test_add_table_with_decimals_coerced
-    Person.connection.drop_table :big_numbers rescue nil
-    assert !BigNumber.table_exists?
-    GiveMeBigNumbers.up
-    BigNumber.reset_column_information
-    assert BigNumber.create(
-      :bank_balance => 1586.43,
-      :big_bank_balance => BigDecimal("1000234000567.95"),
-      :world_population => 6000000000,
-      :my_house_population => 3,
-      :value_of_e => BigDecimal("2.7182818284590452353602875")
-    )
-    b = BigNumber.first
-    assert_not_nil b
-    assert_not_nil b.bank_balance
-    assert_not_nil b.big_bank_balance
-    assert_not_nil b.world_population
-    assert_not_nil b.my_house_population
-    assert_not_nil b.value_of_e
-    assert_kind_of BigDecimal, b.world_population
-    assert_equal '6000000000.0', b.world_population.to_s
-    assert_kind_of Integer, b.my_house_population
-    assert_equal 3, b.my_house_population
-    assert_kind_of BigDecimal, b.bank_balance
-    assert_equal BigDecimal("1586.43"), b.bank_balance
-    assert_kind_of BigDecimal, b.big_bank_balance
-    assert_equal BigDecimal("1000234000567.95"), b.big_bank_balance
-    GiveMeBigNumbers.down
-    assert_raise(ActiveRecord::StatementInvalid) { BigNumber.first }
-  end
-
   # For some reason our tests set Rails.@_env which breaks test env switching.
   coerce_tests! :test_internal_metadata_stores_environment_when_other_data_exists
   coerce_tests! :test_internal_metadata_stores_environment
 end
 
-
-
-
 class CoreTest < ActiveRecord::TestCase
-  # I think fixtures are useing the wrong time zone and the `:first`
+  # I think fixtures are using the wrong time zone and the `:first`
   # `topics`.`bonus_time` attribute of 2005-01-30t15:28:00.00+01:00 is
   # getting local EST time for me and set to "09:28:00.0000000".
   coerce_tests! :test_pretty_print_persisted
 end
 
-
-
-
 module ActiveRecord
   module ConnectionAdapters
     # Just like PostgreSQLAdapter does.
@@ -400,27 +421,152 @@ module ActiveRecord
   end
 end
 
+module ActiveRecord
+  # The original module is hardcoded for PostgreSQL/SQLite/MySQL tests.
+  module DatabaseTasksSetupper
+    def setup
+      @sqlserver_tasks =
+        Class.new do
+          def create; end
 
+          def drop; end
 
+          def purge; end
+
+          def charset; end
+
+          def collation; end
+
+          def structure_dump(*); end
+
+          def structure_load(*); end
+        end.new
+
+      $stdout, @original_stdout = StringIO.new, $stdout
+      $stderr, @original_stderr = StringIO.new, $stderr
+    end
+
+    def with_stubbed_new
+      ActiveRecord::Tasks::SQLServerDatabaseTasks.stub(:new, @sqlserver_tasks) do
+        yield
+      end
+    end
+  end
+
+  class DatabaseTasksCreateTest < ActiveRecord::TestCase
+    # Coerce PostgreSQL/SQLite/MySQL tests.
+    coerce_all_tests!
+
+    def test_sqlserver_create
+      with_stubbed_new do
+        assert_called(eval("@sqlserver_tasks"), :create) do
+          ActiveRecord::Tasks::DatabaseTasks.create "adapter" => :sqlserver
+        end
+      end
+    end
+  end
+
+  class DatabaseTasksDropTest < ActiveRecord::TestCase
+    # Coerce PostgreSQL/SQLite/MySQL tests.
+    coerce_all_tests!
+
+    def test_sqlserver_drop
+      with_stubbed_new do
+        assert_called(eval("@sqlserver_tasks"), :drop) do
+          ActiveRecord::Tasks::DatabaseTasks.drop "adapter" => :sqlserver
+        end
+      end
+    end
+  end
+
+  class DatabaseTasksPurgeTest < ActiveRecord::TestCase
+    # Coerce PostgreSQL/SQLite/MySQL tests.
+    coerce_all_tests!
+
+    def test_sqlserver_purge
+      with_stubbed_new do
+        assert_called(eval("@sqlserver_tasks"), :purge) do
+          ActiveRecord::Tasks::DatabaseTasks.purge "adapter" => :sqlserver
+        end
+      end
+    end
+  end
+
+  class DatabaseTasksCharsetTest < ActiveRecord::TestCase
+    # Coerce PostgreSQL/SQLite/MySQL tests.
+    coerce_all_tests!
+
+    def test_sqlserver_charset
+      with_stubbed_new do
+        assert_called(eval("@sqlserver_tasks"), :charset) do
+          ActiveRecord::Tasks::DatabaseTasks.charset "adapter" => :sqlserver
+        end
+      end
+    end
+  end
+
+  class DatabaseTasksCollationTest < ActiveRecord::TestCase
+    # Coerce PostgreSQL/SQLite/MySQL tests.
+    coerce_all_tests!
+
+    def test_sqlserver_collation
+      with_stubbed_new do
+        assert_called(eval("@sqlserver_tasks"), :collation) do
+          ActiveRecord::Tasks::DatabaseTasks.collation "adapter" => :sqlserver
+        end
+      end
+    end
+  end
+
+  class DatabaseTasksStructureDumpTest < ActiveRecord::TestCase
+    # Coerce PostgreSQL/SQLite/MySQL tests.
+    coerce_all_tests!
+
+    def test_sqlserver_structure_dump
+      with_stubbed_new do
+        assert_called_with(
+          eval("@sqlserver_tasks"), :structure_dump,
+          ["awesome-file.sql", nil]
+        ) do
+          ActiveRecord::Tasks::DatabaseTasks.structure_dump({ "adapter" => :sqlserver }, "awesome-file.sql")
+        end
+      end
+    end
+  end
+
+  class DatabaseTasksStructureLoadTest < ActiveRecord::TestCase
+    # Coerce PostgreSQL/SQLite/MySQL tests.
+    coerce_all_tests!
+
+    def test_sqlserver_structure_load
+      with_stubbed_new do
+        assert_called_with(
+          eval("@sqlserver_tasks"),
+          :structure_load,
+          ["awesome-file.sql", nil]
+        ) do
+          ActiveRecord::Tasks::DatabaseTasks.structure_load({ "adapter" => :sqlserver }, "awesome-file.sql")
+        end
+      end
+    end
+  end
 
-module ActiveRecord
   class DatabaseTasksDumpSchemaCacheTest < ActiveRecord::TestCase
     # Skip this test with /tmp/my_schema_cache.yml path on Windows.
-    coerce_tests! :test_dump_schema_cache if RbConfig::CONFIG['host_os'] =~ /mswin|mingw/
+    coerce_tests! :test_dump_schema_cache if RbConfig::CONFIG["host_os"] =~ /mswin|mingw/
   end
+
   class DatabaseTasksCreateAllTest < ActiveRecord::TestCase
     # We extend `local_database?` so that common VM IPs can be used.
     coerce_tests! :test_ignores_remote_databases, :test_warning_for_remote_databases
   end
+
   class DatabaseTasksDropAllTest < ActiveRecord::TestCase
     # We extend `local_database?` so that common VM IPs can be used.
     coerce_tests! :test_ignores_remote_databases, :test_warning_for_remote_databases
   end
 end
 
-
-
-
 class DefaultScopingTest < ActiveRecord::TestCase
   # We are not doing order duplicate removal anymore.
   coerce_tests! :test_order_in_default_scope_should_not_prevail
@@ -434,16 +580,13 @@ class DefaultScopingTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
-require 'models/post'
-require 'models/subscriber'
+require "models/post"
+require "models/subscriber"
 class EachTest < ActiveRecord::TestCase
   # Quoting in tests does not cope with bracket quoting.
   coerce_tests! :test_find_in_batches_should_quote_batch_order
   def test_find_in_batches_should_quote_batch_order_coerced
-    c = Post.connection
+    Post.connection
     assert_sql(/ORDER BY \[posts\]\.\[id\]/) do
       Post.find_in_batches(:batch_size => 1) do |batch|
         assert_kind_of Array, batch
@@ -455,7 +598,7 @@ class EachTest < ActiveRecord::TestCase
   # Quoting in tests does not cope with bracket quoting.
   coerce_tests! :test_in_batches_should_quote_batch_order
   def test_in_batches_should_quote_batch_order_coerced
-    c = Post.connection
+    Post.connection
     assert_sql(/ORDER BY \[posts\]\.\[id\]/) do
       Post.in_batches(of: 1) do |relation|
         assert_kind_of ActiveRecord::Relation, relation
@@ -465,9 +608,6 @@ class EachTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
 class EagerAssociationTest < ActiveRecord::TestCase
   # Use LEN() vs length() function.
   coerce_tests! :test_count_with_include
@@ -479,14 +619,13 @@ class EagerAssociationTest < ActiveRecord::TestCase
   coerce_tests! %r{including association based on sql condition and no database column}
 end
 
-
-
-
-require 'models/topic'
+require "models/topic"
 class FinderTest < ActiveRecord::TestCase
+  # We have implicit ordering, via FETCH.
   coerce_tests! %r{doesn't have implicit ordering},
-                :test_find_doesnt_have_implicit_ordering # We have implicit ordering, via FETCH.
+                :test_find_doesnt_have_implicit_ordering
 
+  # Square brackets around column name
   coerce_tests! :test_exists_does_not_select_columns_without_alias
   def test_exists_does_not_select_columns_without_alias_coerced
     assert_sql(/SELECT\s+1 AS one FROM \[topics\].*OFFSET 0 ROWS FETCH NEXT @0 ROWS ONLY.*@0 = 1/i) do
@@ -494,6 +633,7 @@ class FinderTest < ActiveRecord::TestCase
     end
   end
 
+  # Assert SQL Server limit implementation
   coerce_tests! :test_take_and_first_and_last_with_integer_should_use_sql_limit
   def test_take_and_first_and_last_with_integer_should_use_sql_limit_coerced
     assert_sql(/OFFSET 0 ROWS FETCH NEXT @0 ROWS ONLY.* @0 = 3/) { Topic.take(3).entries }
@@ -507,7 +647,7 @@ class FinderTest < ActiveRecord::TestCase
   # Can not use array condition due to not finding right type and hence fractional second quoting.
   coerce_tests! :test_condition_utc_time_interpolation_with_default_timezone_local
   def test_condition_utc_time_interpolation_with_default_timezone_local_coerced
-    with_env_tz 'America/New_York' do
+    with_env_tz "America/New_York" do
       with_timezone_config default: :local do
         topic = Topic.first
         assert_equal topic, Topic.where(written_on: topic.written_on.getutc).first
@@ -518,7 +658,7 @@ class FinderTest < ActiveRecord::TestCase
   # Can not use array condition due to not finding right type and hence fractional second quoting.
   coerce_tests! :test_condition_local_time_interpolation_with_default_timezone_utc
   def test_condition_local_time_interpolation_with_default_timezone_utc_coerced
-    with_env_tz 'America/New_York' do
+    with_env_tz "America/New_York" do
       with_timezone_config default: :utc do
         topic = Topic.first
         assert_equal topic, Topic.where(written_on: topic.written_on.getlocal).first
@@ -527,9 +667,6 @@ class FinderTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
 module ActiveRecord
   class Migration
     class ForeignKeyTest < ActiveRecord::TestCase
@@ -547,90 +684,103 @@ module ActiveRecord
   end
 end
 
-
-
-
 class HasOneAssociationsTest < ActiveRecord::TestCase
   # We use OFFSET/FETCH vs TOP. So we always have an order.
   coerce_tests! :test_has_one_does_not_use_order_by
-end
-
 
+  # Asserted SQL to get one row different from original test.
+  coerce_tests! :test_has_one
+  def test_has_one_coerced
+    firm = companies(:first_firm)
+    first_account = Account.find(1)
+    assert_sql(/FETCH NEXT @(\d) ROWS ONLY(.)*@\1 = 1/) do
+      assert_equal first_account, firm.account
+      assert_equal first_account.credit_limit, firm.account.credit_limit
+    end
+  end
+end
 
+class HasOneThroughAssociationsTest < ActiveRecord::TestCase
+  # Asserted SQL to get one row different from original test.
+  coerce_tests! :test_has_one_through_executes_limited_query
+  def test_has_one_through_executes_limited_query_coerced
+    boring_club = clubs(:boring_club)
+    assert_sql(/FETCH NEXT @(\d) ROWS ONLY(.)*@\1 = 1/) do
+      assert_equal boring_club, @member.general_club
+    end
+  end
+end
 
-require 'models/company'
+require "models/company"
 class InheritanceTest < ActiveRecord::TestCase
+  # Rails test required inserting to a identity column.
   coerce_tests! :test_a_bad_type_column
   def test_a_bad_type_column_coerced
-    Company.connection.with_identity_insert_enabled('companies') do
+    Company.connection.with_identity_insert_enabled("companies") do
       Company.connection.insert "INSERT INTO companies (id, #{QUOTED_TYPE}, name) VALUES(100, 'bad_class!', 'Not happening')"
     end
     assert_raise(ActiveRecord::SubclassNotFound) { Company.find(100) }
   end
 
+  # Use Square brackets around column name
   coerce_tests! :test_eager_load_belongs_to_primary_key_quoting
   def test_eager_load_belongs_to_primary_key_quoting_coerced
-    con = Account.connection
+    Account.connection
     assert_sql(/\[companies\]\.\[id\] = @0.* @0 = 1/) do
       Account.all.merge!(:includes => :firm).find(1)
     end
   end
 end
 
-
-
-
 class LeftOuterJoinAssociationTest < ActiveRecord::TestCase
   # Uses || operator in SQL. Just trust core gets value out of this test.
   coerce_tests! :test_does_not_override_select
 end
 
-
-
-
-class NamedScopingTest < ActiveRecord::TestCase
-  # This works now because we add an `order(:id)` sort to break the order tie for deterministic results.
-  coerce_tests! :test_scopes_honor_current_scopes_from_when_defined
-  def test_scopes_honor_current_scopes_from_when_defined_coerced
-    assert !Post.ranked_by_comments.order(:id).limit_by(5).empty?
-    assert !authors(:david).posts.ranked_by_comments.order(:id).limit_by(5).empty?
-    assert_not_equal Post.ranked_by_comments.order(:id).limit_by(5), authors(:david).posts.ranked_by_comments.order(:id).limit_by(5)
-    assert_not_equal Post.order(:id).top(5), authors(:david).posts.order(:id).top(5)
-    # Oracle sometimes sorts differently if WHERE condition is changed
-    assert_equal authors(:david).posts.ranked_by_comments.limit_by(5).to_a.sort_by(&:id), authors(:david).posts.top(5).to_a.sort_by(&:id)
-    assert_equal Post.ranked_by_comments.limit_by(5), Post.top(5)
-  end
-end
-
-
-
-
-require 'models/developer'
-require 'models/computer'
+require "models/developer"
+require "models/computer"
 class NestedRelationScopingTest < ActiveRecord::TestCase
+  # Assert SQL Server limit implementation
   coerce_tests! :test_merge_options
   def test_merge_options_coerced
-    Developer.where('salary = 80000').scoping do
+    Developer.where("salary = 80000").scoping do
       Developer.limit(10).scoping do
         devs = Developer.all
         sql = devs.to_sql
-        assert_match '(salary = 80000)', sql
-        assert_match 'FETCH NEXT 10 ROWS ONLY', sql
+        assert_match "(salary = 80000)", sql
+        assert_match "FETCH NEXT 10 ROWS ONLY", sql
       end
     end
   end
 end
 
+require "models/topic"
+class PersistenceTest < ActiveRecord::TestCase
+  # Rails test required updating a identity column.
+  coerce_tests! :test_update_columns_changing_id
 
+  # Rails test required updating a identity column.
+  coerce_tests! :test_update
+  def test_update_coerced
+    topic = Topic.find(1)
+    assert_not_predicate topic, :approved?
+    assert_equal "The First Topic", topic.title
 
+    topic.update("approved" => true, "title" => "The First Topic Updated")
+    topic.reload
+    assert_predicate topic, :approved?
+    assert_equal "The First Topic Updated", topic.title
 
-require 'models/parrot'
-require 'models/topic'
-class PersistenceTest < ActiveRecord::TestCase
-  # We can not UPDATE identity columns.
-  coerce_tests! :test_update_columns_changing_id
+    topic.update(approved: false, title: "The First Topic")
+    topic.reload
+    assert_not_predicate topic, :approved?
+    assert_equal "The First Topic", topic.title
+  end
+end
 
-  # Previous test required updating a identity column.
+require "models/author"
+class UpdateAllTest < ActiveRecord::TestCase
+  # Rails test required updating a identity column.
   coerce_tests! :test_update_all_doesnt_ignore_order
   def test_update_all_doesnt_ignore_order_coerced
     david, mary = authors(:david), authors(:mary)
@@ -638,81 +788,80 @@ class PersistenceTest < ActiveRecord::TestCase
     _(mary.id).must_equal 2
     _(david.name).wont_equal mary.name
     assert_sql(/UPDATE.*\(SELECT \[authors\].\[id\] FROM \[authors\].*ORDER BY \[authors\].\[id\]/i) do
-      Author.where('[id] > 1').order(:id).update_all(name: 'Test')
+      Author.where("[id] > 1").order(:id).update_all(name: "Test")
     end
-    _(david.reload.name).must_equal 'David'
-    _(mary.reload.name).must_equal 'Test'
-  end
-
-  # We can not UPDATE identity columns.
-  coerce_tests! :test_update_attributes
-  def test_update_attributes_coerced
-    topic = Topic.find(1)
-    assert !topic.approved?
-    assert_equal "The First Topic", topic.title
-    topic.update_attributes("approved" => true, "title" => "The First Topic Updated")
-    topic.reload
-    assert topic.approved?
-    assert_equal "The First Topic Updated", topic.title
-    topic.update_attributes(approved: false, title: "The First Topic")
-    topic.reload
-    assert !topic.approved?
-    assert_equal "The First Topic", topic.title
-    # SQLServer: Here is where it breaks down. No exceptions.
-    # assert_raise(ActiveRecord::RecordNotUnique, ActiveRecord::StatementInvalid) do
-    #   topic.update_attributes(id: 3, title: "Hm is it possible?")
-    # end
-    # assert_not_equal "Hm is it possible?", Topic.find(3).title
-    # topic.update_attributes(id: 1234)
-    # assert_nothing_raised { topic.reload }
-    # assert_equal topic.title, Topic.find(1234).title
+    _(david.reload.name).must_equal "David"
+    _(mary.reload.name).must_equal "Test"
   end
 end
 
-
-
-
-require 'models/topic'
+require "models/topic"
 module ActiveRecord
   class PredicateBuilderTest < ActiveRecord::TestCase
+    # Same as original test except string has `N` prefix to indicate unicode string.
     coerce_tests! :test_registering_new_handlers
     def test_registering_new_handlers_coerced
-      Topic.predicate_builder.register_handler(Regexp, proc do |column, value|
-        Arel::Nodes::InfixOperation.new('~', column, Arel.sql(value.source))
-      end)
-      assert_match %r{\[topics\].\[title\] ~ rails}i, Topic.where(title: /rails/).to_sql
-    ensure
-      Topic.reset_column_information
+      assert_match %r{#{Regexp.escape(topic_title)} ~ N'rails'}i, Topic.where(title: /rails/).to_sql
+    end
+
+    # Same as original test except string has `N` prefix to indicate unicode string.
+    coerce_tests! :test_registering_new_handlers_for_association
+    def test_registering_new_handlers_for_association_coerced
+      assert_match %r{#{Regexp.escape(topic_title)} ~ N'rails'}i, Reply.joins(:topic).where(topics: { title: /rails/ }).to_sql
     end
   end
 end
 
-
-
-
 class PrimaryKeysTest < ActiveRecord::TestCase
-  # Gonna trust Rails core for this. We end up with 2 querys vs 3 asserted
-  # but as far as I can tell, this is only one for us anyway.
+  # SQL Server does not have query for release_savepoint
   coerce_tests! :test_create_without_primary_key_no_extra_query
+  def test_create_without_primary_key_no_extra_query_coerced
+    klass = Class.new(ActiveRecord::Base) do
+      self.table_name = "dashboards"
+    end
+    klass.create! # warmup schema cache
+    assert_queries(2, ignore_none: true) { klass.create! }
+  end
 end
 
-
-
-
-require 'models/task'
+require "models/task"
 class QueryCacheTest < ActiveRecord::TestCase
+  # SQL Server adapter not in list of supported adapters in original test.
   coerce_tests! :test_cache_does_not_wrap_results_in_arrays
   def test_cache_does_not_wrap_results_in_arrays_coerced
     Task.cache do
-      assert_kind_of Numeric, Task.connection.select_value("SELECT count(*) AS count_all FROM tasks")
+      assert_equal 2, Task.connection.select_value("SELECT count(*) AS count_all FROM tasks")
     end
   end
-end
 
+  # Same as original test except that we expect one query to be performed to retrieve the table's primary key.
+  # When we generate the SQL for the `find` it includes ordering on the primary key. If we reset the column
+  # information then the primary key needs to be retrieved from the database again to generate the SQL causing the
+  # original test's `assert_no_queries` assertion to fail. Assert that the query was to get the primary key.
+  coerce_tests! :test_query_cached_even_when_types_are_reset
+  def test_query_cached_even_when_types_are_reset_coerced
+    Task.cache do
+      # Warm the cache
+      Task.find(1)
+
+      # Preload the type cache again (so we don't have those queries issued during our assertions)
+      Task.connection.send(:reload_type_map)
+
+      # Clear places where type information is cached
+      Task.reset_column_information
+      Task.initialize_find_by_cache
+      Task.define_attribute_methods
 
+      assert_queries(1, ignore_none: true) do
+        Task.find(1)
+      end
 
+      assert_includes ActiveRecord::SQLCounter.log_all.first, "TC.CONSTRAINT_TYPE = N''PRIMARY KEY''"
+    end
+  end
+end
 
-require 'models/post'
+require "models/post"
 class RelationTest < ActiveRecord::TestCase
   # Use LEN vs LENGTH function.
   coerce_tests! :test_reverse_order_with_function
@@ -733,6 +882,26 @@ class RelationTest < ActiveRecord::TestCase
   # We have implicit ordering, via FETCH.
   coerce_tests! %r{doesn't have implicit ordering}
 
+  # We have implicit ordering, via FETCH.
+  coerce_tests! :test_reorder_with_take
+  def test_reorder_with_take_coerced
+    sql_log = capture_sql do
+      assert Post.order(:title).reorder(nil).take
+    end
+    assert sql_log.none? { |sql| /order by [posts].[title]/i.match?(sql) }, "ORDER BY title was used in the query: #{sql_log}"
+    assert sql_log.all?  { |sql| /order by \[posts\]\.\[id\]/i.match?(sql) }, "default ORDER BY ID was not used in the query: #{sql_log}"
+  end
+
+  # We have implicit ordering, via FETCH.
+  coerce_tests! :test_reorder_with_first
+  def test_reorder_with_first_coerced
+    sql_log = capture_sql do
+      assert Post.order(:title).reorder(nil).first
+    end
+    assert sql_log.none? { |sql| /order by [posts].[title]/i.match?(sql) }, "ORDER BY title was used in the query: #{sql_log}"
+    assert sql_log.all?  { |sql| /order by \[posts\]\.\[id\]/i.match?(sql) }, "default ORDER BY ID was not used in the query: #{sql_log}"
+  end
+
   # We are not doing order duplicate removal anymore.
   coerce_tests! :test_order_using_scoping
 
@@ -756,17 +925,17 @@ class RelationTest < ActiveRecord::TestCase
   # so we are skipping all together.
   coerce_tests! :test_empty_complex_chained_relations
 
-  # Can't apply offset withour ORDER
+  # Can't apply offset without ORDER
   coerce_tests! %r{using a custom table affects the wheres}
-  test 'using a custom table affects the wheres coerced' do
+  test "using a custom table affects the wheres coerced" do
     post = posts(:welcome)
 
     assert_equal post, custom_post_relation.where!(title: post.title).order(:id).take
   end
 
-  # Can't apply offset withour ORDER
+  # Can't apply offset without ORDER
   coerce_tests! %r{using a custom table with joins affects the joins}
-  test 'using a custom table with joins affects the joins coerced' do
+  test "using a custom table with joins affects the joins coerced" do
     post = posts(:welcome)
 
     assert_equal post, custom_post_relation.joins(:author).where!(title: post.title).order(:id).take
@@ -780,45 +949,46 @@ class RelationTest < ActiveRecord::TestCase
   end
 end
 
-class ActiveRecord::RelationTest < ActiveRecord::TestCase
-  coerce_tests! :test_relation_merging_with_merged_symbol_joins_is_aliased
-  def test_relation_merging_with_merged_symbol_joins_is_aliased__coerced
-    categorizations_with_authors = Categorization.joins(:author)
-    queries = capture_sql { Post.joins(:author, :categorizations).merge(Author.select(:id)).merge(categorizations_with_authors).to_a }
-
-    nb_inner_join = queries.sum { |sql| sql.scan(/INNER\s+JOIN/i).size }
-    assert_equal 3, nb_inner_join, "Wrong amount of INNER JOIN in query"
-
-    # using `\W` as the column separator
-    query_matches = queries.any? do |sql|
-      %r[INNER\s+JOIN\s+#{Regexp.escape(Author.quoted_table_name)}\s+\Wauthors_categorizations\W]i.match?(sql)
+module ActiveRecord
+  class RelationTest < ActiveRecord::TestCase
+    # Skipping this test. SQL Server doesn't support optimizer hint as comments
+    coerce_tests! :test_relation_with_optimizer_hints_filters_sql_comment_delimiters
+
+    coerce_tests! :test_does_not_duplicate_optimizer_hints_on_merge
+    def test_does_not_duplicate_optimizer_hints_on_merge_coerced
+      escaped_table = Post.connection.quote_table_name("posts")
+      expected = "SELECT #{escaped_table}.* FROM #{escaped_table} OPTION (OMGHINT)"
+      query = Post.optimizer_hints("OMGHINT").merge(Post.optimizer_hints("OMGHINT")).to_sql
+      assert_equal expected, query
     end
-
-    assert query_matches, "Should be aliasing the child INNER JOINs in query"
   end
 end
 
-
-
-
-require 'models/post'
+require "models/post"
 class SanitizeTest < ActiveRecord::TestCase
+  # Use nvarchar string (N'') in assert
   coerce_tests! :test_sanitize_sql_like_example_use_case
   def test_sanitize_sql_like_example_use_case_coerced
     searchable_post = Class.new(Post) do
-      def self.search(term)
-        where("title LIKE ?", sanitize_sql_like(term, '!'))
+      def self.search_as_method(term)
+        where("title LIKE ?", sanitize_sql_like(term, "!"))
       end
+
+      scope :search_as_scope, ->(term) {
+        where("title LIKE ?", sanitize_sql_like(term, "!"))
+      }
+    end
+
+    assert_sql(/LIKE N'20!% !_reduction!_!!'/) do
+      searchable_post.search_as_method("20% _reduction_!").to_a
     end
-    assert_sql(/\(title LIKE N'20!% !_reduction!_!!'\)/) do
-      searchable_post.search("20% _reduction_!").to_a
+
+    assert_sql(/LIKE N'20!% !_reduction!_!!'/) do
+      searchable_post.search_as_scope("20% _reduction_!").to_a
     end
   end
 end
 
-
-
-
 class SchemaDumperTest < ActiveRecord::TestCase
   # We have precision to 38.
   coerce_tests! :test_schema_dump_keeps_large_precision_integer_columns_as_decimal
@@ -850,19 +1020,14 @@ class SchemaDumperDefaultsTest < ActiveRecord::TestCase
   coerce_tests! :test_schema_dump_defaults_with_universally_supported_types
 end
 
-
-
-
 class TestAdapterWithInvalidConnection < ActiveRecord::TestCase
   # We trust Rails on this since we do not want to install mysql.
   coerce_tests! %r{inspect on Model class does not raise}
 end
 
-
-
-
-require 'models/topic'
+require "models/topic"
 class TransactionTest < ActiveRecord::TestCase
+  # SQL Server does not have query for release_savepoint
   coerce_tests! :test_releasing_named_savepoints
   def test_releasing_named_savepoints_coerced
     Topic.transaction do
@@ -874,10 +1039,7 @@ class TransactionTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
-require 'models/tag'
+require "models/tag"
 class TransactionIsolationTest < ActiveRecord::TestCase
   # SQL Server will lock the table for counts even when both
   # connections are `READ COMMITTED`. So we bypass with `READPAST`.
@@ -887,7 +1049,7 @@ class TransactionIsolationTest < ActiveRecord::TestCase
       assert_equal 0, Tag.count
       Tag2.transaction do
         Tag2.create
-        assert_equal 0, Tag.lock('WITH(READPAST)').count
+        assert_equal 0, Tag.lock("WITH(READPAST)").count
       end
     end
     assert_equal 1, Tag.count
@@ -897,10 +1059,7 @@ class TransactionIsolationTest < ActiveRecord::TestCase
   coerce_tests! %r{repeatable read}
 end
 
-
-
-
-require 'models/book'
+require "models/book"
 class ViewWithPrimaryKeyTest < ActiveRecord::TestCase
   # We have a few view tables. use includes vs equality.
   coerce_tests! :test_views
@@ -912,9 +1071,10 @@ class ViewWithPrimaryKeyTest < ActiveRecord::TestCase
   coerce_tests! :test_does_not_assume_id_column_as_primary_key
   def test_does_not_assume_id_column_as_primary_key_coerced
     model = Class.new(ActiveRecord::Base) { self.table_name = "ebooks" }
-    assert_equal 'id', model.primary_key
+    assert_equal "id", model.primary_key
   end
 end
+
 class ViewWithoutPrimaryKeyTest < ActiveRecord::TestCase
   # We have a few view tables. use includes vs equality.
   coerce_tests! :test_views
@@ -923,23 +1083,17 @@ class ViewWithoutPrimaryKeyTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
-require 'models/author'
+require "models/author"
 class YamlSerializationTest < ActiveRecord::TestCase
   coerce_tests! :test_types_of_virtual_columns_are_not_changed_on_round_trip
   def test_types_of_virtual_columns_are_not_changed_on_round_trip_coerced
-    author = Author.select('authors.*, 5 as posts_count').first
+    author = Author.select("authors.*, 5 as posts_count").first
     dumped = YAML.load(YAML.dump(author))
     assert_equal 5, author.posts_count
     assert_equal 5, dumped.posts_count
   end
 end
 
-
-
-
 class DateTimePrecisionTest < ActiveRecord::TestCase
   # Original test had `7` which we support vs `8` which we use.
   coerce_tests! :test_invalid_datetime_precision_raises_error
@@ -955,7 +1109,7 @@ class DateTimePrecisionTest < ActiveRecord::TestCase
   coerce_tests! :test_datetime_precision_is_truncated_on_assignment
   def test_datetime_precision_is_truncated_on_assignment_coerced
     @connection.create_table(:foos, force: true)
-    @connection.add_column :foos, :created_at,  :datetime, precision: 0
+    @connection.add_column :foos, :created_at, :datetime, precision: 0
     @connection.add_column :foos, :updated_at, :datetime, precision: 6
 
     time = ::Time.now.change(nsec: 123456789)
@@ -972,8 +1126,6 @@ class DateTimePrecisionTest < ActiveRecord::TestCase
   end
 end
 
-
-
 class TimePrecisionTest < ActiveRecord::TestCase
   # datetime is rounded to increments of .000, .003, or .007 seconds
   coerce_tests! :test_time_precision_is_truncated_on_assignment
@@ -994,9 +1146,10 @@ class TimePrecisionTest < ActiveRecord::TestCase
     assert_equal 0, foo.start.nsec
     assert_equal 123457000, foo.finish.nsec
   end
-end
-
 
+  # SQL Server uses default precision for time.
+  coerce_tests! :test_no_time_precision_isnt_truncated_on_assignment
+end
 
 class DefaultNumbersTest < ActiveRecord::TestCase
   # We do better with native types and do not return strings for everything.
@@ -1006,16 +1159,23 @@ class DefaultNumbersTest < ActiveRecord::TestCase
     assert_equal 7, record.positive_integer
     assert_equal 7, record.positive_integer_before_type_cast
   end
+
+  # We do better with native types and do not return strings for everything.
   coerce_tests! :test_default_negative_integer
   def test_default_negative_integer_coerced
     record = DefaultNumber.new
     assert_equal -5, record.negative_integer
     assert_equal -5, record.negative_integer_before_type_cast
   end
-end
-
-
 
+  # We do better with native types and do not return strings for everything.
+  coerce_tests! :test_default_decimal_number
+  def test_default_decimal_number_coerced
+    record = DefaultNumber.new
+    assert_equal BigDecimal("2.78"), record.decimal_number
+    assert_equal 2.78, record.decimal_number_before_type_cast
+  end
+end
 
 module ActiveRecord
   class CollectionCacheKeyTest < ActiveRecord::TestCase
@@ -1024,40 +1184,63 @@ module ActiveRecord
   end
 end
 
+module ActiveRecord
+  class CacheKeyTest < ActiveRecord::TestCase
+    # Like Mysql2 and PostgreSQL, SQL Server doesn't return a string value for updated_at. In the Rails tests
+    # the tests are skipped if adapter is Mysql2 or PostgreSQL.
+    coerce_tests! %r{cache_version is the same when it comes from the DB or from the user}
+    coerce_tests! %r{cache_version does NOT call updated_at when value is from the database}
+    coerce_tests! %r{cache_version does not truncate zeros when timestamp ends in zeros}
+  end
+end
 
-
-
+require "models/book"
 module ActiveRecord
   class StatementCacheTest < ActiveRecord::TestCase
     # Getting random failures.
     coerce_tests! :test_find_does_not_use_statement_cache_if_table_name_is_changed
-  end
-end
-
 
+    # Need to remove index as SQL Server considers NULLs on a unique-index to be equal unlike PostgreSQL/MySQL/SQLite.
+    coerce_tests! :test_statement_cache_values_differ
+    def test_statement_cache_values_differ_coerced
+      Book.connection.remove_index(:books, column: [:author_id, :name])
 
+      original_test_statement_cache_values_differ
+    ensure
+      Book.connection.add_index(:books, [:author_id, :name], unique: true)
+    end
+  end
+end
 
 module ActiveRecord
   module ConnectionAdapters
     class SchemaCacheTest < ActiveRecord::TestCase
       private
+
       # We need to give the full path for this to work.
       def schema_dump_path
-        File.join ARTest::SQLServer.root_activerecord, 'test/assets/schema_dump_5_1.yml'
+        File.join ARTest::SQLServer.root_activerecord, "test/assets/schema_dump_5_1.yml"
       end
     end
   end
 end
 
+require "models/post"
+require "models/comment"
 class UnsafeRawSqlTest < ActiveRecord::TestCase
-  coerce_tests! %r{always allows Arel}
-  test 'order: always allows Arel' do
+  fixtures :posts
+
+  # Use LEN() vs length() function.
+  coerce_tests! %r{order: always allows Arel}
+  test "order: always allows Arel" do
     ids_depr     = with_unsafe_raw_sql_deprecated { Post.order(Arel.sql("len(title)")).pluck(:title) }
     ids_disabled = with_unsafe_raw_sql_disabled   { Post.order(Arel.sql("len(title)")).pluck(:title) }
 
     assert_equal ids_depr, ids_disabled
   end
 
+  # Use LEN() vs length() function.
+  coerce_tests! %r{pluck: always allows Arel}
   test "pluck: always allows Arel" do
     values_depr     = with_unsafe_raw_sql_deprecated { Post.includes(:comments).pluck(:title, Arel.sql("len(title)")) }
     values_disabled = with_unsafe_raw_sql_disabled   { Post.includes(:comments).pluck(:title, Arel.sql("len(title)")) }
@@ -1065,73 +1248,98 @@ class UnsafeRawSqlTest < ActiveRecord::TestCase
     assert_equal values_depr, values_disabled
   end
 
-
-  coerce_tests! %r{order: disallows invalid Array arguments}
-  test "order: disallows invalid Array arguments" do
-    with_unsafe_raw_sql_disabled do
-      assert_raises(ActiveRecord::UnknownAttributeReference) do
-        Post.order(["author_id", "len(title)"]).pluck(:id)
-      end
-    end
-  end
-
+  # Use LEN() vs length() function.
   coerce_tests! %r{order: allows valid Array arguments}
   test "order: allows valid Array arguments" do
     ids_expected = Post.order(Arel.sql("author_id, len(title)")).pluck(:id)
 
-    ids_depr     = with_unsafe_raw_sql_deprecated { Post.order(["author_id", Arel.sql("len(title)")]).pluck(:id) }
-    ids_disabled = with_unsafe_raw_sql_disabled   { Post.order(["author_id", Arel.sql("len(title)")]).pluck(:id) }
+    ids_depr     = with_unsafe_raw_sql_deprecated { Post.order(["author_id", "len(title)"]).pluck(:id) }
+    ids_disabled = with_unsafe_raw_sql_disabled   { Post.order(["author_id", "len(title)"]).pluck(:id) }
 
     assert_equal ids_expected, ids_depr
     assert_equal ids_expected, ids_disabled
   end
 
-  coerce_tests! %r{order: logs deprecation warning for unrecognized column}
-  test "order: logs deprecation warning for unrecognized column" do
-    with_unsafe_raw_sql_deprecated do
-      assert_deprecated(/Dangerous query method/) do
-        Post.order("len(title)")
-      end
-    end
+  test "order: allows string column names that are quoted" do
+    ids_expected = Post.order(Arel.sql("id")).pluck(:id)
+
+    ids_depr     = with_unsafe_raw_sql_deprecated { Post.order("[id]").pluck(:id) }
+    ids_disabled = with_unsafe_raw_sql_disabled   { Post.order("[id]").pluck(:id) }
+
+    assert_equal ids_expected, ids_depr
+    assert_equal ids_expected, ids_disabled
   end
 
-  coerce_tests! %r{pluck: disallows invalid column name}
-  test "pluck: disallows invalid column name" do
-     with_unsafe_raw_sql_disabled do
-       assert_raises(ActiveRecord::UnknownAttributeReference) do
-         Post.pluck("len(title)")
-       end
-     end
-   end
-
-   coerce_tests! %r{pluck: disallows invalid column name amongst valid names}
-   test "pluck: disallows invalid column name amongst valid names" do
-     with_unsafe_raw_sql_disabled do
-       assert_raises(ActiveRecord::UnknownAttributeReference) do
-         Post.pluck(:title, "len(title)")
-       end
-     end
-   end
-
-   coerce_tests! %r{pluck: disallows invalid column names with includes}
-   test "pluck: disallows invalid column names with includes" do
-     with_unsafe_raw_sql_disabled do
-       assert_raises(ActiveRecord::UnknownAttributeReference) do
-         Post.includes(:comments).pluck(:title, "len(title)")
-       end
-     end
-   end
-
-   coerce_tests! %r{pluck: logs deprecation warning}
-   test "pluck: logs deprecation warning" do
-     with_unsafe_raw_sql_deprecated do
-       assert_deprecated(/Dangerous query method/) do
-         Post.includes(:comments).pluck(:title, "len(title)")
-       end
-     end
-   end
-end
+  test "order: allows string column names that are quoted with table" do
+    ids_expected = Post.order(Arel.sql("id")).pluck(:id)
+
+    ids_depr     = with_unsafe_raw_sql_deprecated { Post.order("[posts].[id]").pluck(:id) }
+    ids_disabled = with_unsafe_raw_sql_disabled   { Post.order("[posts].[id]").pluck(:id) }
+
+    assert_equal ids_expected, ids_depr
+    assert_equal ids_expected, ids_disabled
+  end
+
+  test "order: allows string column names that are quoted with table and user" do
+    ids_expected = Post.order(Arel.sql("id")).pluck(:id)
+
+    ids_depr     = with_unsafe_raw_sql_deprecated { Post.order("[dbo].[posts].[id]").pluck(:id) }
+    ids_disabled = with_unsafe_raw_sql_disabled   { Post.order("[dbo].[posts].[id]").pluck(:id) }
+
+    assert_equal ids_expected, ids_depr
+    assert_equal ids_expected, ids_disabled
+  end
+
+  test "order: allows string column names that are quoted with table, user and database" do
+    ids_expected = Post.order(Arel.sql("id")).pluck(:id)
+
+    ids_depr     = with_unsafe_raw_sql_deprecated { Post.order("[activerecord_unittest].[dbo].[posts].[id]").pluck(:id) }
+    ids_disabled = with_unsafe_raw_sql_disabled   { Post.order("[activerecord_unittest].[dbo].[posts].[id]").pluck(:id) }
+
+    assert_equal ids_expected, ids_depr
+    assert_equal ids_expected, ids_disabled
+  end
+
+  test "pluck: allows string column name that are quoted" do
+    titles_expected = Post.pluck(Arel.sql("title"))
+
+    titles_depr     = with_unsafe_raw_sql_deprecated { Post.pluck("[title]") }
+    titles_disabled = with_unsafe_raw_sql_disabled   { Post.pluck("[title]") }
+
+    assert_equal titles_expected, titles_depr
+    assert_equal titles_expected, titles_disabled
+  end
+
+  test "pluck: allows string column name that are quoted with table" do
+    titles_expected = Post.pluck(Arel.sql("title"))
 
+    titles_depr     = with_unsafe_raw_sql_deprecated { Post.pluck("[posts].[title]") }
+    titles_disabled = with_unsafe_raw_sql_disabled   { Post.pluck("[posts].[title]") }
+
+    assert_equal titles_expected, titles_depr
+    assert_equal titles_expected, titles_disabled
+  end
+
+  test "pluck: allows string column name that are quoted with table and user" do
+    titles_expected = Post.pluck(Arel.sql("title"))
+
+    titles_depr     = with_unsafe_raw_sql_deprecated { Post.pluck("[dbo].[posts].[title]") }
+    titles_disabled = with_unsafe_raw_sql_disabled   { Post.pluck("[dbo].[posts].[title]") }
+
+    assert_equal titles_expected, titles_depr
+    assert_equal titles_expected, titles_disabled
+  end
+
+  test "pluck: allows string column name that are quoted with table, user and database" do
+    titles_expected = Post.pluck(Arel.sql("title"))
+
+    titles_depr     = with_unsafe_raw_sql_deprecated { Post.pluck("[activerecord_unittest].[dbo].[posts].[title]") }
+    titles_disabled = with_unsafe_raw_sql_disabled   { Post.pluck("[activerecord_unittest].[dbo].[posts].[title]") }
+
+    assert_equal titles_expected, titles_depr
+    assert_equal titles_expected, titles_disabled
+  end
+end
 
 class ReservedWordTest < ActiveRecord::TestCase
   coerce_tests! :test_change_columns
@@ -1143,32 +1351,29 @@ class ReservedWordTest < ActiveRecord::TestCase
   end
 end
 
-
-
 class OptimisticLockingTest < ActiveRecord::TestCase
   # We do not allow updating identities, but we can test using a non-identity key
   coerce_tests! :test_update_with_dirty_primary_key
   def test_update_with_dirty_primary_key_coerced
     assert_raises(ActiveRecord::RecordNotUnique) do
-      record = StringKeyObject.find('record1')
-      record.id = 'record2'
+      record = StringKeyObject.find("record1")
+      record.id = "record2"
       record.save!
     end
 
-    record = StringKeyObject.find('record1')
-    record.id = 'record42'
+    record = StringKeyObject.find("record1")
+    record.id = "record42"
     record.save!
 
-    assert StringKeyObject.find('record42')
+    assert StringKeyObject.find("record42")
     assert_raises(ActiveRecord::RecordNotFound) do
-      StringKeyObject.find('record1')
+      StringKeyObject.find("record1")
     end
   end
 end
 
-
-
 class RelationMergingTest < ActiveRecord::TestCase
+  # Use nvarchar string (N'') in assert
   coerce_tests! :test_merging_with_order_with_binds
   def test_merging_with_order_with_binds_coerced
     relation = Post.all.merge(Post.order([Arel.sql("title LIKE ?"), "%suffix"]))
@@ -1176,8 +1381,144 @@ class RelationMergingTest < ActiveRecord::TestCase
   end
 end
 
+module ActiveRecord
+  class DatabaseTasksTruncateAllTest < ActiveRecord::TestCase
+    # SQL Server does not allow truncation of tables that are referenced by foreign key
+    # constraints. As this test truncates all tables we would need to remove all foreign
+    # key constraints and then restore them afterwards to get this test to pass.
+    coerce_tests! :test_truncate_tables
+  end
+end
+
+require "models/book"
+class EnumTest < ActiveRecord::TestCase
+  # Need to remove index as SQL Server considers NULLs on a unique-index to be equal unlike PostgreSQL/MySQL/SQLite.
+  coerce_tests! %r{enums are distinct per class}
+  test "enums are distinct per class coerced" do
+    Book.connection.remove_index(:books, column: [:author_id, :name])
+
+    send(:'original_enums are distinct per class')
+  ensure
+    Book.connection.add_index(:books, [:author_id, :name], unique: true)
+  end
+
+  # Need to remove index as SQL Server considers NULLs on a unique-index to be equal unlike PostgreSQL/MySQL/SQLite.
+  coerce_tests! %r{creating new objects with enum scopes}
+  test "creating new objects with enum scopes coerced" do
+    Book.connection.remove_index(:books, column: [:author_id, :name])
+
+    send(:'original_creating new objects with enum scopes')
+  ensure
+    Book.connection.add_index(:books, [:author_id, :name], unique: true)
+  end
+
+  # Need to remove index as SQL Server considers NULLs on a unique-index to be equal unlike PostgreSQL/MySQL/SQLite.
+  coerce_tests! %r{enums are inheritable}
+  test "enums are inheritable coerced" do
+    Book.connection.remove_index(:books, column: [:author_id, :name])
+
+    send(:'original_enums are inheritable')
+  ensure
+    Book.connection.add_index(:books, [:author_id, :name], unique: true)
+  end
+
+  # Need to remove index as SQL Server considers NULLs on a unique-index to be equal unlike PostgreSQL/MySQL/SQLite.
+  coerce_tests! %r{declare multiple enums at a time}
+  test "declare multiple enums at a time coerced" do
+    Book.connection.remove_index(:books, column: [:author_id, :name])
+
+    send(:'original_declare multiple enums at a time')
+  ensure
+    Book.connection.add_index(:books, [:author_id, :name], unique: true)
+  end
+end
+
+require "models/task"
+class QueryCacheExpiryTest < ActiveRecord::TestCase
+  # SQL Server does not support skipping or upserting duplicates.
+  coerce_tests! :test_insert_all
+  def test_insert_all_coerced
+    assert_raises(ArgumentError, /does not support skipping duplicates/) do
+      Task.cache { Task.insert({ starting: Time.now }) }
+    end
+
+    assert_called(ActiveRecord::Base.connection, :clear_query_cache, times: 2) do
+      Task.cache { Task.insert_all!([{ starting: Time.now }]) }
+    end
+
+    assert_called(ActiveRecord::Base.connection, :clear_query_cache, times: 2) do
+      Task.cache { Task.insert!({ starting: Time.now }) }
+    end
+
+    assert_called(ActiveRecord::Base.connection, :clear_query_cache, times: 2) do
+      Task.cache { Task.insert_all!([{ starting: Time.now }]) }
+    end
+
+    assert_raises(ArgumentError, /does not support upsert/) do
+      Task.cache { Task.upsert({ starting: Time.now }) }
+    end
+
+    assert_raises(ArgumentError, /does not support upsert/) do
+      Task.cache { Task.upsert_all([{ starting: Time.now }]) }
+    end
+  end
+end
 
+require "models/citation"
 class EagerLoadingTooManyIdsTest < ActiveRecord::TestCase
-  # Temporarily coerce this test due to https://github.com/rails/rails/issues/34945
+  # Original Rails test fails with SQL Server error message "The query processor ran out of internal resources and
+  # could not produce a query plan". This error goes away if you change database compatibility level to 110 (SQL 2012)
+  # (see https://www.mssqltips.com/sqlservertip/5279/sql-server-error-query-processor-ran-out-of-internal-resources-and-could-not-produce-a-query-plan/).
+  # However, you cannot change the compatibility level during a test. The purpose of the test is to ensure that an
+  # unprepared statement is used if the number of values exceeds the adapter's `bind_params_length`. The coerced test
+  # still does this as there will be 32,768 remaining citation records in the database and the `bind_params_length` of
+  # adapter is 2,098.
   coerce_tests! :test_eager_loading_too_may_ids
+  def test_eager_loading_too_may_ids_coerced
+    # Remove excess records.
+    Citation.limit(32768).order(id: :desc).delete_all
+
+    # Perform test
+    citation_count = Citation.count
+    assert_sql(/WHERE \(\[citations\]\.\[id\] IN \(0, 1/) do
+      assert_equal citation_count, Citation.eager_load(:citations).offset(0).size
+    end
+  end
+end
+
+class LogSubscriberTest < ActiveRecord::TestCase
+  # Call original test from coerced test. Fixes issue on CI with Rails installed as a gem.
+  coerce_tests! :test_vebose_query_logs
+  def test_vebose_query_logs_coerced
+    original_test_vebose_query_logs
+  end
+end
+
+class ActiveRecordSchemaTest < ActiveRecord::TestCase
+  # Workaround for randomly failing test.
+  coerce_tests! :test_has_primary_key
+  def test_has_primary_key_coerced
+    @schema_migration.reset_column_information
+    original_test_has_primary_key
+  end
+end
+
+module ActiveRecord
+  module ConnectionAdapters
+    class ReaperTest < ActiveRecord::TestCase
+      # Coerce can be removed if Rails version > 6.0.3
+      coerce_tests! :test_connection_pool_starts_reaper_in_fork unless Process.respond_to?(:fork)
+    end
+  end
+end
+
+class FixturesTest < ActiveRecord::TestCase
+  # Skip test on Windows. Skip can be removed when Rails PR https://github.com/rails/rails/pull/39234 has been merged.
+  coerce_tests! :test_binary_in_fixtures if RbConfig::CONFIG["host_os"] =~ /mswin|mingw/
+end
+
+class ReloadModelsTest < ActiveRecord::TestCase
+  # Skip test on Windows. The number of arguements passed to `IO.popen` in
+  # `activesupport/lib/active_support/testing/isolation.rb` exceeds what Windows can handle.
+  coerce_tests! :test_has_one_with_reload if RbConfig::CONFIG["host_os"] =~ /mswin|mingw/
 end