Fixes

Author: aidanharan

lib/active_record/connection_adapters/sqlserver/schema_creation.rb

@@ -50,7 +50,7 @@ def visit_CreateIndexDefinition(o)
         end
 
         def add_column_options!(sql, options)
-          sql << " DEFAULT #{quote_default_expression(options[:default], options[:column])}" if options_include_default?(options)
+          sql << " DEFAULT #{quote_default_expression_for_column_definition(options[:default], options[:column])}" if options_include_default?(options)
           if options[:collation].present?
             sql << " COLLATE #{options[:collation]}"
           end

test/cases/coerced_tests.rb

@@ -570,8 +570,8 @@ def test_create_table_with_defaults_coerce
         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 true, two.fetch_cast_type(connection).deserialize(two.default)
+        assert_equal false, three.fetch_cast_type(connection).deserialize(three.default)
         assert_equal 1, four.default
         assert_equal "hello", five.default
       end
@@ -1971,12 +1971,17 @@ def with_marshable_time_defaults
         # Revert changes
         @connection.change_column_default(:sst_datatypes, :datetime, current_default) if current_default.present?
       end
-
-      # We need to give the full path for this to work.
-      undef_method :schema_dump_path
-      def schema_dump_path
+      
+      # We need to give the full paths for this to work.
+      undef_method :schema_dump_5_1_path
+      def schema_dump_5_1_path
         File.join(ARTest::SQLServer.root_activerecord, "test/assets/schema_dump_5_1.yml")
       end
+
+      undef_method :schema_dump_8_0_path
+      def schema_dump_8_0_path
+        File.join(ARTest::SQLServer.root_activerecord, "test/assets/schema_dump_8_0.yml")
+      end
     end
   end
 end