activerecord-sqlserver-adapter 8.1.2 → 8.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9f4b1a613792ffe35e1f406460eb03e52e8cc6957dee6d3e3ced571d053d1e35
-  data.tar.gz: e861052889a6bec7c0757bdf1df7a2e5a49d875bde8e5dea6ae2000800a3839e
+  metadata.gz: b449fb9569a0dcfc91cc7e0109a29a145dc0c11b8488a3eb486650fa037eb2ad
+  data.tar.gz: ca39fe0127a4ab8e507b49c5a49f59ae02f193d6d54ae3f0d2e9f7731c6f4b08
 SHA512:
-  metadata.gz: 722c00f02c04e5a901e3985eaf317cacb708d627e7bc4ba2f84d7b2f3a242f4a495f9f24f97517ef847ae81093923b87424b2e8cf6e2231b686eb53544c0c2e2
-  data.tar.gz: 9a351724f2af7ec2c069907207b48c787423d2ceb40147d1573d19a65e5d4b24d4e3ec01f7208b77afc3761300d2972c07d28cda2c5c3ae9d35723a1e1e8f036
+  metadata.gz: d5a25f54332ba7e594ceef724da573ef3add3820e7f7de4dcc3a59622c77b5a5978a433efa42894f9467d8940024e11345022d556522a2000289a5f2046cd012
+  data.tar.gz: ed563ea7651fb586b42c3d59f17d8459b73245ebf014b9121a85baff1584a405a4ec72e8a1a005cbf57c9e0fa84f8a5c8ad4b20c3d451a5490e58b19f34af394

data/.github/workflows/ci.yml

@@ -1,12 +1,6 @@
 name: CI
 
-on:
-  push:
-    branches: [ main ]
-  pull_request:
-    branches: [ main ]
-  schedule:
-    - cron: '0 4 * * 0'
+on: [push, pull_request]
 
 jobs:
   test:

data/CHANGELOG.md

@@ -1,3 +1,24 @@
+## v8.1.4
+
+#### Fixed
+
+- [#1418](https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/1418) Fix `change_column_null` to re-declare datetime2 columns correctly.
+
+#### Changed
+
+- [#1405](https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/1405) Fix `insert_all`/`upsert_all` for single character, temporary, non-ASCII and three part table names, and stop an aliased target being included in the `MERGE` table name.
+
+## v8.1.3
+
+#### Added
+
+- [#1383](https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/1383) Support trust_server_certificate option in dbconsole.
+
+#### Changed
+
+- [#1393](https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/1393) Treat TinyTDS `failed dbsqlsend() function` errors as `ConnectionNotEstablished` so they're retried instead of surfaced as `StatementInvalid`.
+- [#1397](https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/1397) Treat "DBPROCESS is dead or not enabled" as ConnectionNotEstablished.
+
 ## v8.1.2
 
 #### Fixed

data/Gemfile

@@ -9,9 +9,10 @@ gemspec
 gem "bcrypt"
 gem "pg", "1.5.9"
 gem "sqlite3", ">= 2.1"
-gem "tzinfo-data", platforms: [:mingw, :mswin, :x64_mingw, :jruby]
+gem "tzinfo-data"
 gem "benchmark-ips"
-gem "minitest", ">= 5.15.0"
+gem "minitest", "~> 6.0"
+gem "minitest-mock"
 gem "msgpack", ">= 1.7.0"
 
 if ENV["RAILS_SOURCE"]
@@ -61,7 +62,6 @@ end
 
 group :development do
   gem "minitest-spec-rails"
-  gem "mocha"
   gem "pry-byebug", platform: [:mri, :mingw, :x64_mingw]
 end
 

data/VERSION

@@ -1 +1 @@
-8.1.2
+8.1.4

data/lib/active_record/connection_adapters/sqlserver/schema_statements.rb

@@ -4,6 +4,9 @@ module ActiveRecord
   module ConnectionAdapters
     module SQLServer
       module SchemaStatements
+        MERGE_TARGET_IDENTIFIER = /(?:\[[^\]]+\]|[a-z0-9_-]+)/i # :nodoc:
+        MERGE_TARGET_TABLE_NAME = /\A\s*MERGE\s+INTO\s+(#{MERGE_TARGET_IDENTIFIER}(?:\s*\.#{MERGE_TARGET_IDENTIFIER}){0,2})\s+(?:AS|WITH|USING)/i # :nodoc:
+
         def create_table(table_name, **options)
           res = super
           clear_cache!
@@ -411,7 +414,12 @@ module ActiveRecord
           if !null.nil? && null == false && !default.nil?
             execute("UPDATE #{table_id} SET #{column_id}=#{quote(default)} WHERE #{column_id} IS NULL")
           end
-          sql = "ALTER TABLE #{table_id} ALTER COLUMN #{column_id} #{type_to_sql column.type, limit: column.limit, precision: column.precision, scale: column.scale}"
+
+          # If datetime column has precision then it should be datetime2 column.
+          column_type = column.type
+          column_type = :datetime2 if column.type.to_s == "datetime" && !column.precision.nil?
+
+          sql = "ALTER TABLE #{table_id} ALTER COLUMN #{column_id} #{type_to_sql column_type, limit: column.limit, precision: column.precision, scale: column.scale}"
           sql += " NOT NULL" if !null.nil? && null == false
 
           execute sql
@@ -726,7 +734,7 @@ module ActiveRecord
           elsif s.match?(/^\s*UPDATE\s+.*/i)
             s.match(/UPDATE\s+([^(\s]+)\s*/i)[1]
           elsif s.match?(/^\s*MERGE INTO.*/i)
-            s.match(/^\s*MERGE\s+INTO\s+(\[?[a-z0-9_ -]+\]?\.?\[?[a-z0-9_ -]+\]?)\s+(AS|WITH|USING)/i)[1]
+            s.match(MERGE_TARGET_TABLE_NAME)[1]
           else
             s.match(/FROM[\s|(]+((\[[^(\]]+\])|[^(\s]+)\s*/i)[1]
           end.strip

data/lib/active_record/connection_adapters/sqlserver_adapter.rb

@@ -104,6 +104,7 @@ module ActiveRecord
           args += ["-d", config.database.to_s] if config.database
           args += ["-U", sqlserver_config[:username].to_s] if sqlserver_config[:username]
           args += ["-P", sqlserver_config[:password].to_s] if sqlserver_config[:password]
+          args += ["-C"] if sqlserver_config[:trust_server_certificate]
 
           if sqlserver_config[:host]
             host_arg = "tcp:#{sqlserver_config[:host]}"
@@ -482,7 +483,7 @@ module ActiveRecord
 
       def translate_exception(exception, message:, sql:, binds:)
         case message
-        when /(SQL Server client is not connected)|(failed to execute statement)/i
+        when /(SQL Server client is not connected)|(failed to execute statement)|(failed dbsqlsend)|(DBPROCESS is dead or not enabled)/i
           ConnectionNotEstablished.new(message, connection_pool: @pool)
         when /(cannot insert duplicate key .* with unique index) | (violation of (unique|primary) key constraint)/i
           RecordNotUnique.new(message, sql: sql, binds: binds, connection_pool: @pool)

data/test/cases/adapter_test_sqlserver.rb

@@ -230,6 +230,7 @@ class AdapterTestSQLServer < ActiveRecord::TestCase
       @identity_merge_sql = "MERGE INTO [ships] WITH (UPDLOCK, HOLDLOCK) AS target USING ( SELECT * FROM ( SELECT [id], [name], ROW_NUMBER() OVER ( PARTITION BY [id] ORDER BY [id] DESC ) AS rn_0 FROM ( VALUES (101, N'RSS Sir David Attenborough') ) AS t1 ([id], [name]) ) AS ranked_source WHERE rn_0 = 1 ) AS source ON (target.[id] = source.[id]) WHEN MATCHED THEN UPDATE SET target.[name] = source.[name] WHEN NOT MATCHED BY TARGET THEN INSERT ([id], [name]) VALUES (source.[id], source.[name]) OUTPUT INSERTED.[id]"
       @identity_merge_sql_unquoted = "MERGE INTO ships WITH (UPDLOCK, HOLDLOCK) AS target USING ( SELECT * FROM ( SELECT id, name, ROW_NUMBER() OVER ( PARTITION BY id ORDER BY id DESC ) AS rn_0 FROM ( VALUES (101, N'RSS Sir David Attenborough') ) AS t1 (id, name) ) AS ranked_source WHERE rn_0 = 1 ) AS source ON (target.id = source.id) WHEN MATCHED THEN UPDATE SET target.name = source.name WHEN NOT MATCHED BY TARGET THEN INSERT (id, name) VALUES (source.id, source.name) OUTPUT INSERTED.id"
       @identity_merge_sql_unordered = "MERGE INTO [ships] WITH (UPDLOCK, HOLDLOCK) AS target USING ( SELECT * FROM ( SELECT [name], [id], ROW_NUMBER() OVER ( PARTITION BY [id] ORDER BY [id] DESC ) AS rn_0 FROM ( VALUES (101, N'RSS Sir David Attenborough') ) AS t1 ([name], [id]) ) AS ranked_source WHERE rn_0 = 1 ) AS source ON (target.[id] = source.[id]) WHEN MATCHED THEN UPDATE SET target.[name] = source.[name] WHEN NOT MATCHED BY TARGET THEN INSERT ([name], [id]) VALUES (source.[name], source.[id]) OUTPUT INSERTED.[id]"
+      @identity_merge_sql_aliased_target = @identity_merge_sql.sub(" WITH (UPDLOCK, HOLDLOCK)", "")
 
       @identity_insert_sql_non_dbo = "INSERT INTO [test].[aliens] ([id],[name]) VALUES(420,'Mork')"
       @identity_insert_sql_non_dbo_unquoted = "INSERT INTO test.aliens ([id],[name]) VALUES(420,'Mork')"
@@ -253,6 +254,7 @@ class AdapterTestSQLServer < ActiveRecord::TestCase
       assert_equal "[ships]", connection.send(:query_requires_identity_insert?, @identity_merge_sql)
       assert_equal "[ships]", connection.send(:query_requires_identity_insert?, @identity_merge_sql_unquoted)
       assert_equal "[ships]", connection.send(:query_requires_identity_insert?, @identity_merge_sql_unordered)
+      assert_equal "[ships]", connection.send(:query_requires_identity_insert?, @identity_merge_sql_aliased_target)
 
       assert_equal "[test].[aliens]", connection.send(:query_requires_identity_insert?, @identity_insert_sql_non_dbo)
       assert_equal "[test].[aliens]", connection.send(:query_requires_identity_insert?, @identity_insert_sql_non_dbo_unquoted)

data/test/cases/change_column_null_test_sqlserver.rb

@@ -2,29 +2,31 @@
 
 require "cases/helper_sqlserver"
 require "migrations/create_clients_and_change_column_null"
+require "cases/migration/helper"
 
 class ChangeColumnNullTestSqlServer < ActiveRecord::TestCase
-  before do
-    @old_verbose = ActiveRecord::Migration.verbose
-    ActiveRecord::Migration.verbose = false
-    CreateClientsAndChangeColumnNull.new.up
-  end
+  include ::ActiveRecord::Migration::TestHelper
 
-  after do
-    CreateClientsAndChangeColumnNull.new.down
-    ActiveRecord::Migration.verbose = @old_verbose
+  def find_column(table, column_name)
+    connection.columns(table).find { |column| column.name == column_name.to_s }
   end
 
-  def find_column(table, name)
-    table.find { |column| column.name == name }
-  end
+  describe "using migration" do
+    before do
+      @old_verbose = ActiveRecord::Migration.verbose
+      ActiveRecord::Migration.verbose = false
+      CreateClientsAndChangeColumnNull.new.up
+    end
+
+    after do
+      CreateClientsAndChangeColumnNull.new.down
+      ActiveRecord::Migration.verbose = @old_verbose
+    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(:name_column) { find_column("clients", "name") }
+    let(:code_column) { find_column("clients", "code") }
+    let(:value_column) { find_column("clients", "value") }
 
-  describe "#change_column_null" do
     it "does not change the column limit" do
       _(name_column.limit).must_equal 15
     end
@@ -41,4 +43,32 @@ class ChangeColumnNullTestSqlServer < ActiveRecord::TestCase
       _(value_column.scale).must_equal 8
     end
   end
+
+  it "datetime2 column" do
+    add_column :test_models, :expiry_date, :datetime, precision: 6, null: true
+
+    _(find_column(:test_models, :expiry_date).sql_type).must_equal "datetime2(6)"
+    _(find_column(:test_models, :expiry_date).null).must_equal true
+
+    connection.change_column_null(:test_models, :expiry_date, false)
+
+    _(find_column(:test_models, :expiry_date).sql_type).must_equal "datetime2(6)"
+    _(find_column(:test_models, :expiry_date).null).must_equal false
+  ensure
+    remove_column("test_models", "expiry_date")
+  end
+
+  it "datetime column" do
+    add_column :test_models, :expiry_date, :datetime, precision: nil, null: true
+
+    _(find_column(:test_models, :expiry_date).sql_type).must_equal "datetime"
+    _(find_column(:test_models, :expiry_date).null).must_equal true
+
+    connection.change_column_null(:test_models, :expiry_date, false)
+
+    _(find_column(:test_models, :expiry_date).sql_type).must_equal "datetime"
+    _(find_column(:test_models, :expiry_date).null).must_equal false
+  ensure
+    remove_column("test_models", "expiry_date")
+  end
 end

data/test/cases/coerced_tests.rb

@@ -77,6 +77,26 @@ class UniquenessValidationWithIndexTest < ActiveRecord::TestCase
   end
 end
 
+module ActiveRecord
+  class Migration::IndexTest < ActiveRecord::TestCase
+    # SQL Server requires an explicit truthy comparison for BIT columns.
+    coerce_tests! :test_rename_index_preserves_where_clause
+    def test_rename_index_preserves_where_clause_coerced
+      skip unless connection.supports_partial_index?
+
+      # Keep the names short to make Oracle and similar behave.
+      connection.add_index(table_name, [:foo], name: "old_idx", where: "administrator = 1")
+      old_where = connection.indexes(table_name).find { |i| i.name == "old_idx" }.where
+
+      connection.rename_index(table_name, "old_idx", "new_idx")
+
+      new_index = connection.indexes(table_name).find { |i| i.name == "new_idx" }
+      assert_not_nil new_index
+      assert_equal old_where, new_index.where
+    end
+  end
+end
+
 module ActiveRecord
   class AdapterTest < ActiveRecord::TestCase
     # Legacy binds are not supported.
@@ -163,7 +183,7 @@ module ActiveRecord
     # 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
+    def test_truncate_tables_with_query_cache_coerced
       # Remove foreign key constraint to allow truncation.
       @connection.remove_foreign_key :authors, :author_addresses
 
@@ -518,8 +538,8 @@ module ActiveRecord
         five = columns.detect { |c| c.name == "five" }
 
         assert_equal "hello", one.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 true, two.cast_type.deserialize(two.default)
+        assert_equal false, three.cast_type.deserialize(three.default)
         assert_equal 1, four.default
         assert_equal "hello", five.default
       end
@@ -2053,7 +2073,6 @@ class UnsafeRawSqlTest < ActiveRecord::TestCase
   test "order: allows nested functions" do
     ids_expected = Post.order(Arel.sql("author_id, len(trim(title))")).pluck(:id)
 
-    # $DEBUG = true
     ids = Post.order("author_id, len(trim(title))").pluck(:id)
 
     assert_equal ids_expected, ids
@@ -2442,6 +2461,50 @@ class FieldOrderedValuesTest < ActiveRecord::TestCase
     Book.where(author_id: nil, name: nil).delete_all
     Book.lease_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! :test_in_order_of_with_out_of_bound_integer_does_not_match_nulls
+  def test_in_order_of_with_out_of_bound_integer_does_not_match_nulls_coerced
+    Book.lease_connection.remove_index(:books, column: [:author_id, :name])
+
+    original_test_in_order_of_with_out_of_bound_integer_does_not_match_nulls
+  ensure
+    Book.where(author_id: nil, name: nil).delete_all
+    Book.lease_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! :test_in_order_of_with_only_unrepresentable_values_does_not_build_empty_case
+  def test_in_order_of_with_only_unrepresentable_values_does_not_build_empty_case_coerced
+    Book.lease_connection.remove_index(:books, column: [:author_id, :name])
+
+    original_test_in_order_of_with_only_unrepresentable_values_does_not_build_empty_case
+  ensure
+    Book.where(author_id: nil, name: nil).delete_all
+    Book.lease_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! :test_in_order_of_with_unknown_enum_key_does_not_match_nulls
+  def test_in_order_of_with_unknown_enum_key_does_not_match_nulls_coerced
+    Book.lease_connection.remove_index(:books, column: [:author_id, :name])
+
+    original_test_in_order_of_with_unknown_enum_key_does_not_match_nulls
+  ensure
+    Book.where(author_id: nil, name: nil).delete_all
+    Book.lease_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! :test_in_order_of_with_column_from_cte
+  def test_in_order_of_with_column_from_cte_coerced
+    connection.remove_index(:books, column: [:author_id, :name])
+
+    original_test_in_order_of_with_column_from_cte
+  ensure
+    Book.where(author_id: nil, name: nil).delete_all
+    Book.lease_connection.add_index(:books, [:author_id, :name], unique: true)
+  end
 end
 
 class QueryLogsTest < ActiveRecord::TestCase
@@ -2837,3 +2900,13 @@ class EachTest < ActiveRecord::TestCase
     end
   end
 end
+
+class ActiveRecord::Encryption::EncryptableFixtureTest < ActiveRecord::EncryptionTestCase
+  # Set the attribute as JSON type.
+  ::EncryptedBookWithJson.attribute :metadata, ActiveRecord::Type::SQLServer::Json.new
+
+  coerce_tests! %r{fixtures for json columns get encrypted automatically}
+  def fixtures_for_json_columns_get_encrypted_automatically
+    send(:"original_fixtures for json columns get encrypted automatically")
+  end
+end

data/test/cases/dbconsole_test_sqlserver.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+class DbConsole < ActiveRecord::TestCase
+  subject { ActiveRecord::ConnectionAdapters::SQLServerAdapter }
+
+  it "uses sqlcmd to connect to database" do
+    assert_called_with(subject, :find_cmd_and_exec, ["sqlcmd", "-d", "db", "-U", "user", "-P", "secret", "-C", "-S", "tcp:localhost,1433"]) do
+      config = make_db_config(adapter: "sqlserver", database: "db", username: "user", password: "secret", host: "localhost", port: 1433, trust_server_certificate: true)
+
+      subject.dbconsole(config)
+    end
+  end
+
+  private
+
+  def make_db_config(config)
+    ActiveRecord::DatabaseConfigurations::HashConfig.new("test", "primary", config)
+  end
+end

data/test/cases/helper_sqlserver.rb

@@ -14,7 +14,6 @@ require "support/load_schema_sqlserver"
 require "support/coerceable_test_sqlserver"
 require "support/connection_reflection"
 require "support/query_assertions"
-require "mocha/minitest"
 
 Minitest.after_run do
   puts "\n\n"

data/test/cases/schema_test_sqlserver.rb

@@ -122,6 +122,38 @@ class SchemaTestSQLServer < ActiveRecord::TestCase
       it do
         assert_equal "[with_numbers_1234]", connection.send(:get_raw_table_name, "MERGE INTO [with_numbers_1234] AS target")
       end
+
+      it do
+        assert_equal "[dashboards]", connection.send(:get_raw_table_name, "MERGE INTO [dashboards] AS target USING (VALUES (1)) AS source ([id]) ON (target.[id] = source.[id]) WHEN NOT MATCHED THEN INSERT ([id]) VALUES (source.[id]);")
+      end
+
+      it do
+        assert_equal "lock_without_defaults", connection.send(:get_raw_table_name, "MERGE INTO lock_without_defaults AS target USING (VALUES (1)) AS source ([id]) ON (target.[id] = source.[id]) WHEN NOT MATCHED THEN INSERT ([id]) VALUES (source.[id]);")
+      end
+
+      it do
+        assert_equal "[WITH - SPACES]", connection.send(:get_raw_table_name, "MERGE INTO [WITH - SPACES] AS target USING (VALUES (1)) AS source ([id]) ON (target.[id] = source.[id]) WHEN NOT MATCHED THEN INSERT ([id]) VALUES (source.[id]);")
+      end
+
+      it do
+        assert_equal "[with].[select notation]", connection.send(:get_raw_table_name, "MERGE INTO [with].[select notation] AS target USING (VALUES (1)) AS source ([id]) ON (target.[id] = source.[id]) WHEN NOT MATCHED THEN INSERT ([id]) VALUES (source.[id]);")
+      end
+
+      it do
+        assert_equal "[x]", connection.send(:get_raw_table_name, "MERGE INTO [x] WITH (UPDLOCK, HOLDLOCK) AS target USING (VALUES (1)) AS source ([id]) ON (target.[id] = source.[id]) WHEN NOT MATCHED THEN INSERT ([id]) VALUES (source.[id]);")
+      end
+
+      it do
+        assert_equal "[with].[select notation]", connection.send(:get_raw_table_name, "MERGE INTO [with].[select notation] WITH (UPDLOCK, HOLDLOCK) AS target USING (VALUES (1)) AS source ([id]) ON (target.[id] = source.[id]) WHEN NOT MATCHED THEN INSERT ([id]) VALUES (source.[id]);")
+      end
+
+      it do
+        assert_equal "[a].[dbo].[dashboards]", connection.send(:get_raw_table_name, "MERGE INTO [a].[dbo].[dashboards] WITH (UPDLOCK, HOLDLOCK) AS target USING (VALUES (1)) AS source ([id]) ON (target.[id] = source.[id]) WHEN NOT MATCHED THEN INSERT ([id]) VALUES (source.[id]);")
+      end
+
+      it do
+        assert_equal "with .dashboards", connection.send(:get_raw_table_name, "MERGE INTO with .dashboards WITH (UPDLOCK, HOLDLOCK) AS target USING (VALUES (1)) AS source ([id]) ON (target.[id] = source.[id]) WHEN NOT MATCHED THEN INSERT ([id]) VALUES (source.[id]);")
+      end
     end
 
     describe "CREATE VIEW statements" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-sqlserver-adapter
 version: !ruby/object:Gem::Version
-  version: 8.1.2
+  version: 8.1.4
 platform: ruby
 authors:
 - Ken Collins
@@ -154,7 +154,7 @@ files:
 - test/cases/coerced_tests.rb
 - test/cases/column_test_sqlserver.rb
 - test/cases/connection_test_sqlserver.rb
-- test/cases/dbconsole.rb
+- test/cases/dbconsole_test_sqlserver.rb
 - test/cases/disconnected_test_sqlserver.rb
 - test/cases/eager_load_too_many_ids_test_sqlserver.rb
 - test/cases/enum_test_sqlserver.rb
@@ -245,8 +245,8 @@ licenses:
 - MIT
 metadata:
   bug_tracker_uri: https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/issues
-  changelog_uri: https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/blob/v8.1.2/CHANGELOG.md
-  source_code_uri: https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/tree/v8.1.2
+  changelog_uri: https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/blob/v8.1.4/CHANGELOG.md
+  source_code_uri: https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/tree/v8.1.4
 rdoc_options: []
 require_paths:
 - lib
@@ -261,7 +261,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.7
+rubygems_version: 4.0.16
 specification_version: 4
 summary: ActiveRecord SQL Server Adapter.
 test_files: []

data/test/cases/dbconsole.rb

@@ -1,19 +0,0 @@
-# frozen_string_literal: true
-
-class DbConsole < ActiveRecord::TestCase
-  subject { ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter }
-
-  it "uses sqlplus to connect to database" do
-    subject.expects(:find_cmd_and_exec).with("sqlcmd", "-d", "db", "-U", "user", "-P", "secret", "-S", "tcp:localhost,1433")
-
-    config = make_db_config(adapter: "sqlserver", database: "db", username: "user", password: "secret", host: "localhost", port: 1433)
-
-    subject.dbconsole(config)
-  end
-
-  private
-
-  def make_db_config(config)
-    ActiveRecord::DatabaseConfigurations::HashConfig.new("test", "primary", config)
-  end
-end