activerecord-pedantmysql2-adapter 1.1.3 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: fb193accc432a0fed85a527c520afef1f5bf3c43
-  data.tar.gz: b11fe06882e5086459d26f52cd59bcfb6ead0327
+SHA256:
+  metadata.gz: 0cdc8bc77345e0ed3428828757da42af2180846edbb81327efbad5a0ab28a48e
+  data.tar.gz: 6e90b36c37754b0accdaac56ff69d125b61262f7d79be7884fa79afd835dc010
 SHA512:
-  metadata.gz: e8146d26e36fa99f37a3c08277da4db95210a3fabc0ad6764dde8de9cefa3751ac1a4f79ee97733cc058ab704878100459c7dc3f5c95b612c803c5a16b5f1fd8
-  data.tar.gz: f950e21b77652402beb3ebd2e7d5786f59f47d97a1568120d7fe19e6b9a84abccdb0fb9f4c53ff0c3eae82f2b4222ebf6a2fab5198128bd69343824c15150027
+  metadata.gz: 20edfa621dd97dbd80a2b3a5cc3f31711a85074228a5a848b4bc9c987ace3e5d92ad979995560b220526e572aea9f6c3e586e89825e8386fbab6bb4d68578a72
+  data.tar.gz: 79fa48c21ae537ef22c88692c59ea5eaa6332953bc68f3666134373043b1eeb4d1e1e86160b4a7cb6442a0bf757cd9af206026da145460c9c9058ab75b417ec2

data/.travis.yml

@@ -1,27 +1,24 @@
+services:
+  - mysql
+
 rvm:
-  - 1.9.3
-  - 2.0.0
-  - 2.1.2
-  - 2.2.2
-  - 2.3.0
+  - '2.5'
+  - '2.6'
+  - '2.7'
 
 gemfile:
-  - Gemfile.activerecord32
-  - Gemfile.activerecord40
-  - Gemfile.activerecord41
-  - Gemfile.activerecord42
-  - Gemfile.activerecord50
+  - gemfiles/Gemfile.activerecord52
+  - gemfiles/Gemfile.activerecord60
+  - gemfiles/Gemfile.activerecord-edge
 
 matrix:
   exclude:
-    - rvm: 1.9.3
-      gemfile: Gemfile.activerecord50
-    - rvm: 2.0.0
-      gemfile: Gemfile.activerecord50
-    - rvm: 2.1.2
-      gemfile: Gemfile.activerecord50
+    - rvm: '2.5'
+      gemfile: gemfiles/Gemfile.activerecord-edge
+    - rvm: '2.6'
+      gemfile: gemfiles/Gemfile.activerecord-edge
 
 before_script:
-  - mysql -e 'create database pedant_mysql2_test;'
+  - mysql -u root -h 127.0.0.1 -e 'create database pedant_mysql2_test;'
 
 sudo: false

data/README.md

@@ -52,7 +52,7 @@ and to restore it to raising warnings as errors:
   PedantMysql2.raise_warnings!
 ```
 
-You can easilly whitelist some types of warnings:
+You can easily whitelist some types of warnings:
 
 ```ruby
 PedantMysql2.ignore(/Some warning I don't care about/i)

data/activerecord-pedantmysql2-adapter.gemspec

@@ -13,14 +13,18 @@ Gem::Specification.new do |spec|
   spec.homepage      = 'https://github.com/Shopify/activerecord-pedantmysql2-adapter'
   spec.license       = 'MIT'
 
+  spec.metadata['allowed_push_host'] = 'https://rubygems.org'
+
+  spec.required_ruby_version = '>= 2.2.2'
+
   spec.files         = `git ls-files -z`.split("\x0")
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'activerecord', '>= 3.2'
+  spec.add_dependency 'activerecord', '>= 5.0'
   spec.add_dependency 'mysql2', '>= 0.3.12'
-  spec.add_development_dependency 'bundler', '~> 1.5'
+  spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'rspec', '>= 3.0'
   spec.add_development_dependency 'rspec-its'

data/gemfiles/Gemfile.activerecord-edge

@@ -0,0 +1,4 @@
+source 'https://rubygems.org'
+gemspec path: '..'
+
+gem 'activerecord', github: 'rails/rails', branch: 'master'

data/gemfiles/Gemfile.activerecord52

@@ -0,0 +1,5 @@
+source 'https://rubygems.org'
+gemspec path: '..'
+
+gem 'activerecord', '~> 5.2.0'
+gem 'mysql2'

data/gemfiles/Gemfile.activerecord60

@@ -0,0 +1,5 @@
+source 'https://rubygems.org'
+gemspec path: '..'
+
+gem 'activerecord', '~> 6.0.0'
+gem 'mysql2'

data/lib/active_record/connection_adapters/pedant_mysql2_adapter.rb

@@ -2,24 +2,44 @@ require 'active_record/connection_adapters/mysql2_adapter'
 
 module ActiveRecord
   module ConnectionHandling
-    def pedant_mysql2_connection(config)
-      config = config.symbolize_keys
-
-      config[:username] = 'root' if config[:username].nil?
-
-      if Mysql2::Client.const_defined? :FOUND_ROWS
-        config[:flags] = Mysql2::Client::FOUND_ROWS
+    if ConnectionAdapters::Mysql2Adapter.respond_to?(:new_client)
+      def pedant_mysql2_connection(config)
+        config = config.symbolize_keys
+        config[:flags] ||= 0
+
+        if config[:flags].kind_of? Array
+          config[:flags].push "FOUND_ROWS"
+        else
+          config[:flags] |= Mysql2::Client::FOUND_ROWS
+        end
+
+        ConnectionAdapters::PedantMysql2Adapter.new(
+          ConnectionAdapters::Mysql2Adapter.new_client(config),
+          logger,
+          nil,
+          config,
+        )
       end
-
-      client = Mysql2::Client.new(config)
-
-      options = [config[:host], config[:username], config[:password], config[:database], config[:port], config[:socket], 0]
-      ActiveRecord::ConnectionAdapters::PedantMysql2Adapter.new(client, logger, options, config)
-    rescue Mysql2::Error => error
-      if error.message.include?("Unknown database") && defined?(ActiveRecord::NoDatabaseError)
-        raise ActiveRecord::NoDatabaseError.new(error.message)
-      else
-        raise
+    else
+      def pedant_mysql2_connection(config)
+        config = config.symbolize_keys
+
+        config[:username] = 'root' if config[:username].nil?
+
+        if Mysql2::Client.const_defined? :FOUND_ROWS
+          config[:flags] = Mysql2::Client::FOUND_ROWS
+        end
+
+        client = Mysql2::Client.new(config)
+
+        options = [config[:host], config[:username], config[:password], config[:database], config[:port], config[:socket], 0]
+        ActiveRecord::ConnectionAdapters::PedantMysql2Adapter.new(client, logger, options, config)
+      rescue Mysql2::Error => error
+        if error.message.include?("Unknown database") && defined?(ActiveRecord::NoDatabaseError)
+          raise ActiveRecord::NoDatabaseError.new(error.message)
+        else
+          raise
+        end
       end
     end
   end

data/lib/pedant_mysql2.rb

@@ -46,11 +46,11 @@ module PedantMysql2
     end
 
     def ignored?(warning)
-      whitelist.any? { |matcher| matcher =~ warning.message } || drop_table_warning(warning)
+      whitelist.any? { |matcher| warning.message.match?(matcher) } || drop_table_warning(warning)
     end
 
     def drop_table_warning(warning)
-      warning.query =~ /\ADROP TABLE IF EXISTS/ || warning.message =~ /\AUnknown table/
+      warning.query.match?(/\ADROP TABLE IF EXISTS/) || warning.message.match?(/\AUnknown table/)
     end
 
     def setup_capture

data/lib/pedant_mysql2/version.rb

@@ -1,3 +1,3 @@
 module PedantMysql2
-  VERSION = '1.1.3'
+  VERSION = '1.2.0'
 end

data/spec/adapter_spec.rb

@@ -7,6 +7,7 @@ describe PedantMysql2 do
   before :each do
     PedantMysql2.raise_warnings!
     PedantMysql2.instance_variable_set(:@whitelist, nil)
+    PedantMysql2.ignore(/They will be merged with strict mode in a future release/)
     connection.execute('SET SESSION binlog_format = "STATEMENT"')
     if connection.execute('SHOW TABLES LIKE "comment"').size == 0
       connection.execute('CREATE TABLE comment (id int)')

data/spec/support/database.rb

@@ -1,11 +1,9 @@
-ActiveRecord::Base.configurations = {
-  'test' => {
-    adapter: 'pedant_mysql2',
-    database: 'pedant_mysql2_test',
-    username: 'travis',
-    encoding: 'utf8',
-    strict: false,
-    pool: 5,
-  },
-}
-ActiveRecord::Base.establish_connection(:test)
+ActiveRecord::Base.establish_connection(
+  'adapter' => 'pedant_mysql2',
+  'database' => 'pedant_mysql2_test',
+  'username' => 'root',
+  'encoding' => 'utf8',
+  'host' => 'localhost',
+  'strict' => false,
+  'pool' => 5,
+)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-pedantmysql2-adapter
 version: !ruby/object:Gem::Version
-  version: 1.1.3
+  version: 1.2.0
 platform: ruby
 authors:
 - Jean Boussier
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-07-27 00:00:00.000000000 Z
+date: 2020-08-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: mysql2
   requirement: !ruby/object:Gem::Requirement
@@ -42,16 +42,16 @@ dependencies:
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -120,20 +120,17 @@ files:
 - ".rspec"
 - ".travis.yml"
 - Gemfile
-- Gemfile.activerecord32
-- Gemfile.activerecord40
-- Gemfile.activerecord41
-- Gemfile.activerecord42
-- Gemfile.activerecord50
 - LICENSE.txt
 - README.md
 - Rakefile
 - activerecord-pedantmysql2-adapter.gemspec
+- gemfiles/Gemfile.activerecord-edge
+- gemfiles/Gemfile.activerecord52
+- gemfiles/Gemfile.activerecord60
 - lib/active_record/connection_adapters/pedant_mysql2_adapter.rb
 - lib/activerecord-pedantmysql2-adapter.rb
 - lib/pedant_mysql2.rb
 - lib/pedant_mysql2/version.rb
-- shipit.rubygems.yml
 - spec/adapter_spec.rb
 - spec/connection_spec.rb
 - spec/spec_helper.rb
@@ -141,7 +138,8 @@ files:
 homepage: https://github.com/Shopify/activerecord-pedantmysql2-adapter
 licenses:
 - MIT
-metadata: {}
+metadata:
+  allowed_push_host: https://rubygems.org
 post_install_message: 
 rdoc_options: []
 require_paths:
@@ -150,15 +148,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.2.2
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.5.2
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: ActiveRecord adapter for MySQL that report warnings.

data/Gemfile.activerecord32

@@ -1,4 +0,0 @@
-source 'https://rubygems.org'
-gemspec
-
-gem 'activerecord', '~> 3.2.0'

data/Gemfile.activerecord40

@@ -1,4 +0,0 @@
-source 'https://rubygems.org'
-gemspec
-
-gem 'activerecord', '~> 4.0.0'

data/Gemfile.activerecord41

@@ -1,4 +0,0 @@
-source 'https://rubygems.org'
-gemspec
-
-gem 'activerecord', '~> 4.1.0'

data/Gemfile.activerecord42

@@ -1,4 +0,0 @@
-source 'https://rubygems.org'
-gemspec
-
-gem 'activerecord', '~> 4.2.0.rc1'

data/Gemfile.activerecord50

@@ -1,4 +0,0 @@
-source 'https://rubygems.org'
-gemspec
-
-gem 'activerecord', '~> 5.0.0'

data/shipit.rubygems.yml

@@ -1,2 +0,0 @@
-fetch:
-  - fetch-gem-version activerecord-pedantmysql2-adapter Shopify/activerecord-pedantmysql2-adapter