activerecord-sqlserver-adapter 6.0.0.rc1 → 6.0.0.rc2

data/test/cases/change_column_null_test_sqlserver.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'cases/helper_sqlserver'
-require 'migrations/create_clients_and_change_column_null'
+require "cases/helper_sqlserver"
+require "migrations/create_clients_and_change_column_null"
 
 class ChangeColumnNullTestSqlServer < ActiveRecord::TestCase
   before do
@@ -19,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,8 @@
 # frozen_string_literal: true
 
-require 'cases/helper_sqlserver'
+require "cases/helper_sqlserver"
 
-
-
-require 'models/event'
+require "models/event"
 class UniquenessValidationTest < ActiveRecord::TestCase
   # So sp_executesql swallows this exception. Run without prepared to see it.
   coerce_tests! :test_validate_uniqueness_with_limit
@@ -30,16 +28,13 @@ class UniquenessValidationTest < ActiveRecord::TestCase
   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_')
+    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 don`t think we can support legacy binds.
@@ -54,7 +49,7 @@ module ActiveRecord
     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
@@ -69,9 +64,6 @@ module ActiveRecord
   end
 end
 
-
-
-
 module ActiveRecord
   class AdapterTestWithoutTransaction < ActiveRecord::TestCase
     # SQL Server does not allow truncation of tables that are referenced by foreign key
@@ -125,15 +117,12 @@ module ActiveRecord
   end
 end
 
-
-
-
-require 'models/topic'
+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_not_predicate topic, :is_test?
   end
@@ -141,21 +130,18 @@ class AttributeMethodsTest < ActiveRecord::TestCase
   # 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_predicate topic, :is_test?
   end
 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.
@@ -175,7 +161,7 @@ class BasicsTest < ActiveRecord::TestCase
   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)
@@ -200,9 +186,6 @@ class BasicsTest < ActiveRecord::TestCase
   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
@@ -227,9 +210,6 @@ class BelongsToAssociationsTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
 module ActiveRecord
   class BindParameterTest < ActiveRecord::TestCase
     # Same as original coerced test except log is found using `EXEC sp_executesql` wrapper.
@@ -257,9 +237,6 @@ module ActiveRecord
   end
 end
 
-
-
-
 module ActiveRecord
   class InstrumentationTest < ActiveRecord::TestCase
     # Fix randomly failing test. The loading of the model's schema was affecting the test.
@@ -271,9 +248,6 @@ module ActiveRecord
   end
 end
 
-
-
-
 class CalculationsTest < ActiveRecord::TestCase
   # Fix randomly failing test. The loading of the model's schema was affecting the test.
   coerce_tests! :test_offset_is_kept
@@ -286,7 +260,7 @@ class CalculationsTest < ActiveRecord::TestCase
   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
@@ -317,9 +291,6 @@ class CalculationsTest < ActiveRecord::TestCase
   coerce_tests! :test_having_with_strong_parameters
 end
 
-
-
-
 module ActiveRecord
   class Migration
     class ChangeSchemaTest < ActiveRecord::TestCase
@@ -351,9 +322,6 @@ module ActiveRecord
   end
 end
 
-
-
-
 module ActiveRecord
   module ConnectionAdapters
     class QuoteARBaseTest < ActiveRecord::TestCase
@@ -361,22 +329,19 @@ module ActiveRecord
       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
@@ -391,16 +356,13 @@ module ActiveRecord
   end
 end
 
-
-
-
 module ActiveRecord
   class Migration
     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"
@@ -416,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`.
@@ -433,9 +395,6 @@ module ActiveRecord
   end
 end
 
-
-
-
 class MigrationTest < ActiveRecord::TestCase
   # We do not have do the DecimalWithoutScale type.
   coerce_tests! :test_add_table_with_decimals
@@ -459,7 +418,7 @@ class MigrationTest < ActiveRecord::TestCase
     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_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
@@ -475,9 +434,6 @@ class MigrationTest < ActiveRecord::TestCase
   coerce_tests! :test_internal_metadata_stores_environment
 end
 
-
-
-
 class CoreTest < ActiveRecord::TestCase
   # 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
@@ -485,9 +441,6 @@ class CoreTest < ActiveRecord::TestCase
   coerce_tests! :test_pretty_print_persisted
 end
 
-
-
-
 module ActiveRecord
   module ConnectionAdapters
     # Just like PostgreSQLAdapter does.
@@ -501,23 +454,26 @@ 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
+        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
@@ -580,7 +536,6 @@ module ActiveRecord
         end
       end
     end
-
   end
 
   class DatabaseTasksCollationTest < ActiveRecord::TestCase
@@ -603,8 +558,8 @@ module ActiveRecord
     def test_sqlserver_structure_dump
       with_stubbed_new do
         assert_called_with(
-            eval("@sqlserver_tasks"), :structure_dump,
-            ["awesome-file.sql", nil]
+          eval("@sqlserver_tasks"), :structure_dump,
+          ["awesome-file.sql", nil]
         ) do
           ActiveRecord::Tasks::DatabaseTasks.structure_dump({ "adapter" => :sqlserver }, "awesome-file.sql")
         end
@@ -619,9 +574,9 @@ module ActiveRecord
     def test_sqlserver_structure_load
       with_stubbed_new do
         assert_called_with(
-            eval("@sqlserver_tasks"),
-            :structure_load,
-            ["awesome-file.sql", nil]
+          eval("@sqlserver_tasks"),
+          :structure_load,
+          ["awesome-file.sql", nil]
         ) do
           ActiveRecord::Tasks::DatabaseTasks.structure_load({ "adapter" => :sqlserver }, "awesome-file.sql")
         end
@@ -631,7 +586,7 @@ 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
@@ -645,9 +600,6 @@ module ActiveRecord
   end
 end
 
-
-
-
 class DefaultScopingTest < ActiveRecord::TestCase
   # We are not doing order duplicate removal anymore.
   coerce_tests! :test_order_in_default_scope_should_not_prevail
@@ -661,16 +613,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
@@ -682,7 +631,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
@@ -692,9 +641,6 @@ class EachTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
 class EagerAssociationTest < ActiveRecord::TestCase
   # Use LEN() vs length() function.
   coerce_tests! :test_count_with_include
@@ -706,10 +652,7 @@ 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},
@@ -737,7 +680,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
@@ -748,7 +691,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
@@ -757,9 +700,6 @@ class FinderTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
 module ActiveRecord
   class Migration
     class ForeignKeyTest < ActiveRecord::TestCase
@@ -777,9 +717,6 @@ 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
@@ -796,9 +733,6 @@ class HasOneAssociationsTest < ActiveRecord::TestCase
   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
@@ -810,16 +744,12 @@ class HasOneThroughAssociationsTest < ActiveRecord::TestCase
   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) }
@@ -828,45 +758,36 @@ class InheritanceTest < ActiveRecord::TestCase
   # 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
 
-
-
-
-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'
+require "models/topic"
 class PersistenceTest < ActiveRecord::TestCase
   # Rails test required updating a identity column.
   coerce_tests! :test_update_columns_changing_id
@@ -890,10 +811,7 @@ class PersistenceTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
-require 'models/author'
+require "models/author"
 class UpdateAllTest < ActiveRecord::TestCase
   # Rails test required updating a identity column.
   coerce_tests! :test_update_all_doesnt_ignore_order
@@ -903,17 +821,14 @@ class UpdateAllTest < 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'
+    _(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.
@@ -930,9 +845,6 @@ module ActiveRecord
   end
 end
 
-
-
-
 class PrimaryKeysTest < ActiveRecord::TestCase
   # SQL Server does not have query for release_savepoint
   coerce_tests! :test_create_without_primary_key_no_extra_query
@@ -945,10 +857,7 @@ class PrimaryKeysTest < ActiveRecord::TestCase
   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
@@ -980,15 +889,12 @@ class QueryCacheTest < ActiveRecord::TestCase
         Task.find(1)
       end
 
-      assert_includes ActiveRecord::SQLCounter.log_all.first , "TC.CONSTRAINT_TYPE = N''PRIMARY KEY''"
+      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
@@ -1029,7 +935,6 @@ class RelationTest < ActiveRecord::TestCase
     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
 
@@ -1055,7 +960,7 @@ class RelationTest < ActiveRecord::TestCase
 
   # 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
@@ -1063,7 +968,7 @@ class RelationTest < ActiveRecord::TestCase
 
   # 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
@@ -1077,10 +982,7 @@ class RelationTest < ActiveRecord::TestCase
   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
@@ -1090,7 +992,7 @@ class SanitizeTest < ActiveRecord::TestCase
         where("title LIKE ?", sanitize_sql_like(term, "!"))
       end
 
-      scope :search_as_scope, -> (term) {
+      scope :search_as_scope, ->(term) {
         where("title LIKE ?", sanitize_sql_like(term, "!"))
       }
     end
@@ -1105,9 +1007,6 @@ class SanitizeTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
 class SchemaDumperTest < ActiveRecord::TestCase
   # We have precision to 38.
   coerce_tests! :test_schema_dump_keeps_large_precision_integer_columns_as_decimal
@@ -1134,26 +1033,17 @@ class SchemaDumperTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
 class SchemaDumperDefaultsTest < ActiveRecord::TestCase
   # These date formats do not match ours. We got these covered in our dumper tests.
   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
@@ -1167,10 +1057,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`.
@@ -1180,7 +1067,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
@@ -1190,10 +1077,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
@@ -1205,13 +1089,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
@@ -1220,23 +1101,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
@@ -1252,7 +1127,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)
@@ -1269,9 +1144,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
@@ -1297,9 +1169,6 @@ class TimePrecisionTest < ActiveRecord::TestCase
   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.
   coerce_tests! :test_default_positive_integer
@@ -1326,9 +1195,6 @@ class DefaultNumbersTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
 module ActiveRecord
   class CollectionCacheKeyTest < ActiveRecord::TestCase
     # Will trust rails has this sorted since you cant offset without a limit.
@@ -1336,9 +1202,6 @@ 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
@@ -1349,9 +1212,6 @@ module ActiveRecord
   end
 end
 
-
-
-
 require "models/book"
 module ActiveRecord
   class StatementCacheTest < ActiveRecord::TestCase
@@ -1370,28 +1230,23 @@ module ActiveRecord
   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
 
-
-
-
 class UnsafeRawSqlTest < ActiveRecord::TestCase
-   # Use LEN() vs length() function.
+  # Use LEN() vs length() function.
   coerce_tests! %r{order: always allows Arel}
-  test 'order: always allows Arel' do
+  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) }
 
@@ -1420,9 +1275,6 @@ class UnsafeRawSqlTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
 class ReservedWordTest < ActiveRecord::TestCase
   coerce_tests! :test_change_columns
   def test_change_columns_coerced
@@ -1433,33 +1285,27 @@ 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
@@ -1469,9 +1315,6 @@ 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
@@ -1481,9 +1324,6 @@ module ActiveRecord
   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.
@@ -1527,13 +1367,8 @@ class EnumTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
-
-require 'models/task'
+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
@@ -1563,9 +1398,7 @@ class QueryCacheExpiryTest < ActiveRecord::TestCase
   end
 end
 
-
-
-require 'models/citation'
+require "models/citation"
 class EagerLoadingTooManyIdsTest < ActiveRecord::TestCase
   # 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)
@@ -1587,9 +1420,6 @@ class EagerLoadingTooManyIdsTest < ActiveRecord::TestCase
   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
@@ -1598,9 +1428,6 @@ class LogSubscriberTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
 class ActiveRecordSchemaTest < ActiveRecord::TestCase
   # Workaround for randomly failing test.
   coerce_tests! :test_has_primary_key
@@ -1610,9 +1437,6 @@ class ActiveRecordSchemaTest < ActiveRecord::TestCase
   end
 end
 
-
-
-
 module ActiveRecord
   module ConnectionAdapters
     class ReaperTest < ActiveRecord::TestCase
@@ -1622,19 +1446,13 @@ module ActiveRecord
   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/
+  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/
+  coerce_tests! :test_has_one_with_reload if RbConfig::CONFIG["host_os"] =~ /mswin|mingw/
 end