[Rails5] Conform to table_exists decprecations.

Author: metaskills

lib/active_record/connection_adapters/sqlserver/schema_statements.rb

@@ -11,6 +11,15 @@ def tables(table_type = 'BASE TABLE')
           select_values "SELECT #{lowercase_schema_reflection_sql('TABLE_NAME')} FROM INFORMATION_SCHEMA.TABLES #{"WHERE TABLE_TYPE = '#{table_type}'" if table_type} ORDER BY TABLE_NAME", 'SCHEMA'
         end
 
+        def table_exists?(table_name)
+          ActiveSupport::Deprecation.warn(<<-MSG.squish)
+            #table_exists? currently checks both tables and views.
+            This behavior is deprecated and will be changed with Rails 5.1 to only check tables.
+            Use #data_source_exists? instead.
+          MSG
+          data_source_exists?(table_name)
+        end
+
         def data_source_exists?(table_name)
           return false if table_name.blank?
           unquoted_table_name = SQLServer::Utils.extract_identifiers(table_name).object

test/cases/coerced_tests.rb

@@ -578,9 +578,6 @@ def test_views_coerced
     assert_includes @connection.views, Ebook.table_name
   end
 
-  # This is deprecated and we dont care in this version to pass it.
-  coerce_tests! :test_table_exists
-
   # We do better than ActiveRecord and find the views PK.
   coerce_tests! :test_does_not_assume_id_column_as_primary_key
   def test_does_not_assume_id_column_as_primary_key_coerced
@@ -594,9 +591,6 @@ class ViewWithoutPrimaryKeyTest < ActiveRecord::TestCase
   def test_views_coerced
     assert_includes @connection.views, Paperback.table_name
   end
-
-  # This is deprecated and we dont care in this version to pass it.
-  coerce_tests! :test_table_exists
 end