activerecord-sqlserver-adapter 5.2.0 → 5.2.1

data/test/cases/specific_schema_test_sqlserver.rb

@@ -11,8 +11,8 @@ class SpecificSchemaTestSQLServer < ActiveRecord::TestCase
 
   it 'models can use tinyint pk tables' do
     obj = SSTestTinyintPk.create! name: '1'
-    ['Fixnum', 'Integer'].must_include obj.id.class.name
-    SSTestTinyintPk.find(obj.id).must_equal obj
+    _(['Fixnum', 'Integer']).must_include obj.id.class.name
+    _(SSTestTinyintPk.find(obj.id)).must_equal obj
   end
 
   it 'be able to complex count tables with no primary key' do
@@ -58,17 +58,17 @@ class SpecificSchemaTestSQLServer < ActiveRecord::TestCase
 
   it 'default objects work' do
     obj = SSTestObjectDefault.create! name: 'MetaSkills'
-    obj.date.must_be_nil 'since this is set on insert'
-    obj.reload.date.must_be_instance_of Date
+    _(obj.date).must_be_nil 'since this is set on insert'
+    _(obj.reload.date).must_be_instance_of Date
   end
 
   it 'allows datetime2 as timestamps' do
-    SSTestBooking.columns_hash['created_at'].sql_type.must_equal 'datetime2(7)'
-    SSTestBooking.columns_hash['updated_at'].sql_type.must_equal 'datetime2(7)'
+    _(SSTestBooking.columns_hash['created_at'].sql_type).must_equal 'datetime2(7)'
+    _(SSTestBooking.columns_hash['updated_at'].sql_type).must_equal 'datetime2(7)'
     obj1 = SSTestBooking.new name: 'test1'
     obj1.save!
-    obj1.created_at.must_be_instance_of Time
-    obj1.updated_at.must_be_instance_of Time
+    _(obj1.created_at).must_be_instance_of Time
+    _(obj1.updated_at).must_be_instance_of Time
   end
 
   # Natural primary keys.
@@ -124,10 +124,10 @@ class SpecificSchemaTestSQLServer < ActiveRecord::TestCase
     o = SSTestDatatypeMigration.create!
     o.varchar_col = "O'Reilly"
     o.save!
-    o.reload.varchar_col.must_equal "O'Reilly"
+    _(o.reload.varchar_col).must_equal "O'Reilly"
     o.varchar_col = nil
     o.save!
-    o.reload.varchar_col.must_be_nil
+    _(o.reload.varchar_col).must_be_nil
   end
 
   # With column names that have spaces
@@ -156,7 +156,7 @@ class SpecificSchemaTestSQLServer < ActiveRecord::TestCase
   it 'returns a new id via connection newid_function' do
     acceptable_uuid = ActiveRecord::ConnectionAdapters::SQLServer::Type::Uuid::ACCEPTABLE_UUID
     db_uuid = ActiveRecord::Base.connection.newid_function
-    db_uuid.must_match(acceptable_uuid)
+    _(db_uuid).must_match(acceptable_uuid)
   end
 
   # with similar table definition in two schemas

data/test/cases/transaction_test_sqlserver.rb

@@ -34,22 +34,22 @@ class TransactionTestSQLServer < ActiveRecord::TestCase
   it 'can use an isolation level and reverts back to starting isolation level' do
     in_level = nil
     begin_level = connection.user_options_isolation_level
-    begin_level.must_match %r{read committed}i
+    _(begin_level).must_match %r{read committed}i
     Ship.transaction(isolation: :serializable) do
       Ship.create! name: 'Black Pearl'
       in_level = connection.user_options_isolation_level
     end
     after_level = connection.user_options_isolation_level
-    in_level.must_match %r{serializable}i
-    after_level.must_match %r{read committed}i
+    _(in_level).must_match %r{serializable}i
+    _(after_level).must_match %r{read committed}i
   end
 
   it 'can use an isolation level and reverts back to starting isolation level under exceptions' do
-    connection.user_options_isolation_level.must_match %r{read committed}i
-    lambda {
+    _(connection.user_options_isolation_level).must_match %r{read committed}i
+    _(lambda {
       Ship.transaction(isolation: :serializable) { Ship.create! }
-    }.must_raise(ActiveRecord::RecordInvalid)
-    connection.user_options_isolation_level.must_match %r{read committed}i
+    }).must_raise(ActiveRecord::RecordInvalid)
+    _(connection.user_options_isolation_level).must_match %r{read committed}i
   end
 
   describe 'when READ_COMMITTED_SNAPSHOT is set' do
@@ -64,7 +64,7 @@ class TransactionTestSQLServer < ActiveRecord::TestCase
     end
 
     it 'should use READ COMMITTED as an isolation level' do
-      connection.user_options_isolation_level.must_match "read committed snapshot"
+      _(connection.user_options_isolation_level).must_match "read committed snapshot"
 
       Ship.transaction(isolation: :serializable) do
         Ship.create! name: 'Black Pearl'
@@ -73,7 +73,7 @@ class TransactionTestSQLServer < ActiveRecord::TestCase
       # We're actually testing that the isolation level was correctly reset to
       # "READ COMMITTED", and that no exception was raised (it's reported back
       # by SQL Server as "read committed snapshot").
-      connection.user_options_isolation_level.must_match "read committed snapshot"
+      _(connection.user_options_isolation_level).must_match "read committed snapshot"
     end
   end
 

data/test/cases/trigger_test_sqlserver.rb

@@ -12,19 +12,19 @@ class SQLServerTriggerTest < ActiveRecord::TestCase
     exclude_output_inserted_table_names['sst_table_with_trigger'] = true
     assert SSTestTriggerHistory.all.empty?
     obj = SSTestTrigger.create! event_name: 'test trigger'
-    ['Fixnum', 'Integer'].must_include obj.id.class.name
-    obj.event_name.must_equal 'test trigger'
-    obj.id.must_be :present?
-    obj.id.to_s.must_equal SSTestTriggerHistory.first.id_source
+    _(['Fixnum', 'Integer']).must_include obj.id.class.name
+    _(obj.event_name).must_equal 'test trigger'
+    _(obj.id).must_be :present?
+    _(obj.id.to_s).must_equal SSTestTriggerHistory.first.id_source
   end
 
   it 'can insert into a table with output inserted - with a uniqueidentifier value' do
     exclude_output_inserted_table_names['sst_table_with_uuid_trigger'] = 'uniqueidentifier'
     assert SSTestTriggerHistory.all.empty?
     obj = SSTestTriggerUuid.create! event_name: 'test uuid trigger'
-    obj.id.class.name.must_equal 'String'
-    obj.event_name.must_equal 'test uuid trigger'
-    obj.id.must_be :present?
-    obj.id.to_s.must_equal SSTestTriggerHistory.first.id_source
+    _(obj.id.class.name).must_equal 'String'
+    _(obj.event_name).must_equal 'test uuid trigger'
+    _(obj.id).must_be :present?
+    _(obj.id.to_s).must_equal SSTestTriggerHistory.first.id_source
   end
 end

data/test/cases/utils_test_sqlserver.rb

@@ -3,15 +3,15 @@ require 'cases/helper_sqlserver'
 class UtilsTestSQLServer < ActiveRecord::TestCase
 
   it '.quote_string' do
-    SQLServer::Utils.quote_string("I'll store this in C:\\Users").must_equal "I''ll store this in C:\\Users"
+    _(SQLServer::Utils.quote_string("I'll store this in C:\\Users")).must_equal "I''ll store this in C:\\Users"
   end
 
   it '.unquote_string' do
-    SQLServer::Utils.unquote_string("I''ll store this in C:\\Users").must_equal "I'll store this in C:\\Users"
+    _(SQLServer::Utils.unquote_string("I''ll store this in C:\\Users")).must_equal "I'll store this in C:\\Users"
   end
 
   it '.quoted_raw' do
-    SQLServer::Utils.quoted_raw("some.Name").must_equal "[some.Name]"
+    _(SQLServer::Utils.quoted_raw("some.Name")).must_equal "[some.Name]"
   end
 
   describe '.extract_identifiers constructor and thus SQLServer::Utils::Name value object' do
@@ -47,8 +47,8 @@ class UtilsTestSQLServer < ActiveRecord::TestCase
     it 'extracts and returns #object identifier unquoted by default or quoted as needed' do
       valid_names.each do |n|
         name = extract_identifiers(n)
-        name.object.must_equal 'object', "With #{n.inspect} for #object"
-        name.object_quoted.must_equal '[object]', "With #{n.inspect} for #object_quoted"
+        _(name.object).must_equal 'object', "With #{n.inspect} for #object"
+        _(name.object_quoted).must_equal '[object]', "With #{n.inspect} for #object_quoted"
       end
     end
 
@@ -58,64 +58,64 @@ class UtilsTestSQLServer < ActiveRecord::TestCase
         present, blank = send(:"#{part}_names")
         present.each do |n|
           name = extract_identifiers(n)
-          name.send(:"#{part}").must_equal "#{part}", "With #{n.inspect} for ##{part} method"
-          name.send(:"#{part}_quoted").must_equal "[#{part}]", "With #{n.inspect} for ##{part}_quoted method"
+          _(name.send(:"#{part}")).must_equal "#{part}", "With #{n.inspect} for ##{part} method"
+          _(name.send(:"#{part}_quoted")).must_equal "[#{part}]", "With #{n.inspect} for ##{part}_quoted method"
         end
         blank.each do |n|
           name = extract_identifiers(n)
-          name.send(:"#{part}").must_be_nil "With #{n.inspect} for ##{part} method"
-          name.send(:"#{part}_quoted").must_be_nil "With #{n.inspect} for ##{part}_quoted method"
+          _(name.send(:"#{part}")).must_be_nil "With #{n.inspect} for ##{part} method"
+          _(name.send(:"#{part}_quoted")).must_be_nil "With #{n.inspect} for ##{part}_quoted method"
         end
       end
 
     end
 
     it 'does not blow up on nil or blank string name' do
-      extract_identifiers(nil).object.must_be_nil
-      extract_identifiers(' ').object.must_be_nil
+      _(extract_identifiers(nil).object).must_be_nil
+      _(extract_identifiers(' ').object).must_be_nil
     end
 
     it 'has a #quoted that returns a fully quoted name with all identifiers as orginially passed in' do
-      extract_identifiers('object').quoted.must_equal '[object]'
-      extract_identifiers('server.database..object').quoted.must_equal '[server].[database]..[object]'
-      extract_identifiers('[server]...[object]').quoted.must_equal '[server]...[object]'
+      _(extract_identifiers('object').quoted).must_equal '[object]'
+      _(extract_identifiers('server.database..object').quoted).must_equal '[server].[database]..[object]'
+      _(extract_identifiers('[server]...[object]').quoted).must_equal '[server]...[object]'
     end
 
     it 'can take a symbol argument' do
-      extract_identifiers(:object).object.must_equal 'object'
+      _(extract_identifiers(:object).object).must_equal 'object'
     end
 
     it 'allows identifiers with periods to work' do
-      extract_identifiers('[obj.name]').quoted.must_equal '[obj.name]'
-      extract_identifiers('[obj.name].[foo]').quoted.must_equal '[obj.name].[foo]'
+      _(extract_identifiers('[obj.name]').quoted).must_equal '[obj.name]'
+      _(extract_identifiers('[obj.name].[foo]').quoted).must_equal '[obj.name].[foo]'
     end
 
     it 'should indicate if a name is fully qualitified' do
-      extract_identifiers('object').fully_qualified?.must_equal false
-      extract_identifiers('schema.object').fully_qualified?.must_equal false
-      extract_identifiers('database.schema.object').fully_qualified?.must_equal false
-      extract_identifiers('database.object').fully_qualified?.must_equal false
-      extract_identifiers('server...object').fully_qualified?.must_equal false
-      extract_identifiers('server.database..object').fully_qualified?.must_equal false
-      extract_identifiers('server.database.schema.object').fully_qualified?.must_equal true
-      extract_identifiers('server.database.schema.').fully_qualified?.must_equal true
-      extract_identifiers('[obj.name]').fully_qualified?.must_equal false
-      extract_identifiers('[schema].[obj.name]').fully_qualified?.must_equal false
-      extract_identifiers('[database].[schema].[obj.name]').fully_qualified?.must_equal false
-      extract_identifiers('[database].[obj.name]').fully_qualified?.must_equal false
-      extract_identifiers('[server.name]...[obj.name]').fully_qualified?.must_equal false
-      extract_identifiers('[server.name].[database]..[obj.name]').fully_qualified?.must_equal false
-      extract_identifiers('[server.name].[database].[schema].[obj.name]').fully_qualified?.must_equal true
-      extract_identifiers('[server.name].[database].[schema].').fully_qualified?.must_equal true
+      _(extract_identifiers('object').fully_qualified?).must_equal false
+      _(extract_identifiers('schema.object').fully_qualified?).must_equal false
+      _(extract_identifiers('database.schema.object').fully_qualified?).must_equal false
+      _(extract_identifiers('database.object').fully_qualified?).must_equal false
+      _(extract_identifiers('server...object').fully_qualified?).must_equal false
+      _(extract_identifiers('server.database..object').fully_qualified?).must_equal false
+      _(extract_identifiers('server.database.schema.object').fully_qualified?).must_equal true
+      _(extract_identifiers('server.database.schema.').fully_qualified?).must_equal true
+      _(extract_identifiers('[obj.name]').fully_qualified?).must_equal false
+      _(extract_identifiers('[schema].[obj.name]').fully_qualified?).must_equal false
+      _(extract_identifiers('[database].[schema].[obj.name]').fully_qualified?).must_equal false
+      _(extract_identifiers('[database].[obj.name]').fully_qualified?).must_equal false
+      _(extract_identifiers('[server.name]...[obj.name]').fully_qualified?).must_equal false
+      _(extract_identifiers('[server.name].[database]..[obj.name]').fully_qualified?).must_equal false
+      _(extract_identifiers('[server.name].[database].[schema].[obj.name]').fully_qualified?).must_equal true
+      _(extract_identifiers('[server.name].[database].[schema].').fully_qualified?).must_equal true
     end
 
     it 'can return fully qualified quoted table name' do
       name = extract_identifiers('[my.server].db.schema.')
-      name.fully_qualified_database_quoted.must_equal '[my.server].[db]'
+      _(name.fully_qualified_database_quoted).must_equal '[my.server].[db]'
       name = extract_identifiers('[server.name].[database].[schema].[object]')
-      name.fully_qualified_database_quoted.must_equal '[server.name].[database]'
+      _(name.fully_qualified_database_quoted).must_equal '[server.name].[database]'
       name = extract_identifiers('server.database.schema.object')
-      name.fully_qualified_database_quoted.must_equal '[server].[database]'
+      _(name.fully_qualified_database_quoted).must_equal '[server].[database]'
     end
 
   end

data/test/cases/uuid_test_sqlserver.rb

@@ -6,24 +6,24 @@ class SQLServerUuidTest < ActiveRecord::TestCase
   let(:acceptable_uuid) { ActiveRecord::ConnectionAdapters::SQLServer::Type::Uuid::ACCEPTABLE_UUID }
 
   it 'has a uuid primary key' do
-    SSTestUuid.columns_hash['id'].type.must_equal :uuid
+    _(SSTestUuid.columns_hash['id'].type).must_equal :uuid
     assert SSTestUuid.primary_key
   end
 
   it 'can create with a new pk' do
     obj = SSTestUuid.create!
-    obj.id.must_be :present?
-    obj.id.must_match acceptable_uuid
+    _(obj.id).must_be :present?
+    _(obj.id).must_match acceptable_uuid
   end
 
   it 'can create other uuid column on reload' do
     obj = SSTestUuid.create!
     obj.reload
-    obj.other_uuid.must_match acceptable_uuid
+    _(obj.other_uuid).must_match acceptable_uuid
   end
 
   it 'can find uuid pk via connection' do
-    connection.primary_key(SSTestUuid.table_name).must_equal 'id'
+    _(connection.primary_key(SSTestUuid.table_name)).must_equal 'id'
   end
 
   it 'changing column default' do
@@ -31,17 +31,17 @@ class SQLServerUuidTest < ActiveRecord::TestCase
     connection.add_column table_name, :thingy, :uuid, null: false, default: "NEWSEQUENTIALID()"
     SSTestUuid.reset_column_information
     column = SSTestUuid.columns_hash['thingy']
-    column.default_function.must_equal "newsequentialid()"
+    _(column.default_function).must_equal "newsequentialid()"
     # Now to a different function.
     connection.change_column table_name, :thingy, :uuid, null: false, default: "NEWID()"
     SSTestUuid.reset_column_information
     column = SSTestUuid.columns_hash['thingy']
-    column.default_function.must_equal "newid()"
+    _(column.default_function).must_equal "newid()"
   end
 
   it 'can insert even when use_output_inserted to false ' do
     obj = with_use_output_inserted_disabled { SSTestUuid.create!(name: "😢") }
-    obj.id.must_be :nil?
+    _(obj.id).must_be :nil?
   end
 
 end

data/test/schema/datatypes/2012.sql

@@ -35,6 +35,7 @@ CREATE TABLE [sst_datatypes] (
   [smalldatetime] [smalldatetime] NULL DEFAULT '1901-01-01T15:45:00.000Z',
   [time_7] [time](7) NULL DEFAULT '04:20:00.2883215',
   [time_2] [time](2) NULL,
+  [time_default] [time] NULL DEFAULT '15:03:42.0621978',
   -- Character Strings
   [char_10] [char](10) NULL DEFAULT '1234567890',
   [varchar_50] [varchar](50) NULL DEFAULT 'test varchar_50',

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-sqlserver-adapter
 version: !ruby/object:Gem::Version
-  version: 5.2.0
+  version: 5.2.1
 platform: ruby
 authors:
 - Ken Collins
@@ -14,7 +14,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-03-19 00:00:00.000000000 Z
+date: 2020-03-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -54,7 +54,6 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".travis.yml"
-- BACKERS.md
 - CHANGELOG.md
 - CODE_OF_CONDUCT.md
 - Dockerfile
@@ -67,13 +66,14 @@ files:
 - VERSION
 - activerecord-sqlserver-adapter.gemspec
 - appveyor.yml
-- circle.yml
 - docker-compose.ci.yml
 - lib/active_record/connection_adapters/sqlserver/core_ext/active_record.rb
 - lib/active_record/connection_adapters/sqlserver/core_ext/attribute_methods.rb
 - lib/active_record/connection_adapters/sqlserver/core_ext/calculations.rb
 - lib/active_record/connection_adapters/sqlserver/core_ext/explain.rb
 - lib/active_record/connection_adapters/sqlserver/core_ext/explain_subscriber.rb
+- lib/active_record/connection_adapters/sqlserver/core_ext/finder_methods.rb
+- lib/active_record/connection_adapters/sqlserver/core_ext/query_methods.rb
 - lib/active_record/connection_adapters/sqlserver/database_limits.rb
 - lib/active_record/connection_adapters/sqlserver/database_statements.rb
 - lib/active_record/connection_adapters/sqlserver/database_tasks.rb
@@ -222,7 +222,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.1
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: ActiveRecord SQL Server Adapter.

data/BACKERS.md

@@ -1,32 +0,0 @@
-# Backers
-
-You can join in supporting TinyTDS and the Rails SQL Server Adapter development by [pledging on Patreon](https://www.patreon.com/metaskills)! Backers in the same pledge level appear in the order of pledge date.
-
-### $2000
-
-[It could be you!](https://www.patreon.com/bePatron?c=765225&rid=1611218)
-
-
-### $500
-
-[It could be you!](https://www.patreon.com/bePatron?c=765225&rid=1611209)
-
-
-### $250
-
-[It could be you!](https://www.patreon.com/bePatron?c=765225&rid=1611199)
-
-
-### $100
-
-[It could be you!](https://www.patreon.com/bePatron?c=765225&rid=1611196)
-
-
-### $50+
-
-[It could be you!](https://www.patreon.com/bePatron?c=765225&rid=1611186)
-
-
-### $10+
-
-[It could be you!](https://www.patreon.com/bePatron?c=765225&rid=1611149)

data/circle.yml

@@ -1,38 +0,0 @@
-general:
-  branches:
-    ignore:
-      - /dev.*/
-
-machine:
-  environment:
-    PATH: /opt/local/bin:${PATH}
-    TINYTDS_VERSION: 2.1.0
-    ACTIVERECORD_UNITTEST_HOST: localhost
-    ACTIVERECORD_UNITTEST_DATASERVER: localhost
-  services:
-    - docker
-
-dependencies:
-  override:
-    - sudo ./test/bin/install-openssl.sh
-    - openssl version
-    - sudo ./test/bin/install-freetds.sh
-    - tsql -C
-    - rvm-exec 2.3.8 bundle install
-    - rvm-exec 2.4.5 bundle install
-    - rvm-exec 2.5.3 bundle install
-    - rvm-exec 2.6.0 bundle install
-
-database:
-  override:
-    - echo "Hello"
-  post:
-    - docker info
-    - ./test/bin/setup.sh
-
-test:
-  override:
-    - rvm-exec 2.3.8 bundle exec rake test
-    - rvm-exec 2.4.5 bundle exec rake test
-    - rvm-exec 2.5.3 bundle exec rake test
-    - rvm-exec 2.6.0 bundle exec rake test