activerecord-rescue_from_duplicate 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: ad1109d6b48c9d16624a06e7262f80571d261b5e
-  data.tar.gz: 7fb9e9970bc575794559358c82be53d8667c5c92
+SHA256:
+  metadata.gz: 57e38eb0a3e8bc47bc4c1e7e299c4ca6b30a4ffa14d9982db17d88086f3dc0ae
+  data.tar.gz: d963ba4a9a08bd3c6ba193bff5ca84f57074e3e0f68afd7a43ff6c02b02accb3
 SHA512:
-  metadata.gz: 7c77fd19ff2d451ad3351c7c2058e76048df727dc5932648668687449960299fe9fa98ce44c1cf5ef0d36bc7955553810fc02240a3e43ccb31e5a87a37403d28
-  data.tar.gz: 52cc5da74eb6c2993042645e679af11209011a86392c4e2863e33af4919dcc1ffb087d5bcdfcad31f0874534c780a80f83288aa34fb5bf66851898e406a211a4
+  metadata.gz: e703b89e8bcb6a4b722ca14fda626f837fe49cfca11cd1ced6023b2d54b0fde559e26e87e808ae5918ab9c180e27142ada04be181c1690122a16774906bee28f
+  data.tar.gz: 3c87f3a1f4f8c5e3233a2a603075bf62f9c0e49dcf6f9257c0e8b98de4bf16c5078b9c718537dd6aea0de704712381d833b7a3b1e544c87eed3a57103426524f

data/.travis.yml

@@ -1,22 +1,16 @@
 sudo: false
 rvm:
-  - 2.1
-  - 2.2
-  - 2.3.1
-gemfile:
-  - gemfiles/Gemfile.ar-3.2
-  - gemfiles/Gemfile.ar-4.0
-  - gemfiles/Gemfile.ar-4.1
-  - gemfiles/Gemfile.ar-4.2
-  - gemfiles/Gemfile.ar-edge
-
-before_script:
-  - mysql -e 'create database rescue_from_duplicate;'
-  - psql -c 'create database rescue_from_duplicate;' -U postgres
+  - 2.5.0
 
 env:
-  - POSTGRES=1 MYSQL=1
+  - DOCKER_COMPOSE_VERSION=1.20.1
+
+before_script:
+  - docker-compose --version
+  - docker-compose pull
+  - docker-compose build
+  - docker-compose up --no-start
+  - docker-compose start
+  - sleep 30 # Wait for databases to come online
+  - docker ps
 
-matrix:
-  allow_failures:
-    - gemfile: gemfiles/Gemfile.ar-edge

data/README.md

@@ -69,6 +69,21 @@ And then execute:
 Or install it yourself as:
 
     $ gem install activerecord-rescue_from_duplicate
+    
+## Development Setup
+
+Install:
+
+- Have Docker installed
+- Clone the repo
+- `docker-compose up -d`
+
+Run tests:
+
+```
+rspec
+```
+
 
 ## Contributing
 

data/Rakefile

@@ -5,13 +5,3 @@ RSpec::Core::RakeTask.new(:spec)
 
 task :default => :spec
 
-namespace :spec do
-  task :all do
-    Dir[File.expand_path('../gemfiles/*.lock', __FILE__)].each { |f| File.delete(f) }
-    Dir[File.expand_path('../gemfiles/*', __FILE__)].each do |gemfile|
-      env = {'BUNDLE_GEMFILE' => gemfile, 'MYSQL' => '1'}
-      system(env, 'bundle install')
-      system(env, 'bundle exec rspec')
-    end
-  end
-end

data/activerecord-rescue_from_duplicate.gemspec

@@ -1,7 +1,7 @@
 # coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path("../lib", __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'rescue_from_duplicate/active_record/version'
+require "rescue_from_duplicate/active_record/version"
 
 Gem::Specification.new do |spec|
   spec.name          = "activerecord-rescue_from_duplicate"
@@ -18,14 +18,14 @@ Gem::Specification.new do |spec|
   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.2"
 
-  spec.add_development_dependency "bundler", "~> 1.3"
+  spec.add_development_dependency "bundler"
   spec.add_development_dependency "rake"
-  spec.add_development_dependency 'simplecov'
-  spec.add_development_dependency 'sqlite3'
-  spec.add_development_dependency 'mysql2'
-  spec.add_development_dependency 'rspec'
-  spec.add_development_dependency 'pg'
-  spec.add_development_dependency 'pry'
+  spec.add_development_dependency "simplecov"
+  spec.add_development_dependency "sqlite3"
+  spec.add_development_dependency "mysql2"
+  spec.add_development_dependency "rspec"
+  spec.add_development_dependency "pg"
+  spec.add_development_dependency "pry"
 end

data/docker-compose.yml

@@ -0,0 +1,18 @@
+mysql:
+  image: mysql:5.7
+  environment:
+    MYSQL_USER: "test"
+    MYSQL_PASSWORD: "test"
+    MYSQL_ALLOW_EMPTY_PASSWORD: "true"
+    MYSQL_DATABASE: "rescue_from_duplicate"
+  ports:
+    - "29291:3306"
+
+postgres:
+  image: postgres:10.3
+  environment:
+    POSTGRES_USER: "test"
+    POSTGRES_PASSWORD: "test"
+    POSTGRES_DB: "rescue_from_duplicate"
+  ports:
+    - "29292:5432"

data/lib/rescue_from_duplicate/active_record/extension.rb

@@ -45,7 +45,7 @@ module RescueFromDuplicate::ActiveRecord
     def exception_handler(exception)
       columns = exception_columns(exception)
       return unless columns
-      columns.sort!
+      columns = columns.sort
 
       self.class._rescue_from_duplicate_handlers.detect do |handler|
         handler.rescue? && columns == handler.columns

data/lib/rescue_from_duplicate/active_record/version.rb

@@ -1,5 +1,5 @@
 module Activerecord
   module RescueFromDuplicate
-    VERSION = '0.1.2'
+    VERSION = '0.1.3'
   end
 end

data/spec/rescue_from_duplicate_spec.rb

@@ -2,16 +2,16 @@ require 'spec_helper'
 include RescueFromDuplicate
 
 shared_examples 'database error rescuing' do
-  let(:uniqueness_exception) { ::ActiveRecord::RecordNotUnique.new(message, nil) }
+  let(:uniqueness_exception) { ::ActiveRecord::RecordNotUnique.new(message) }
 
   subject { Rescuable.new }
 
   before do
-    allow(Rescuable).to receive(:connection).and_return(double(indexes: [Rescuable.index]))
+    allow(Rescuable).to(receive(:connection).and_return(double(indexes: [Rescuable.index])))
   end
 
   describe "#create_or_update when the validation fails" do
-    before { Base.stub(exception: uniqueness_exception) }
+    before { allow(Base).to(receive(:exception).and_return(uniqueness_exception)) }
 
     context "when the validator is present" do
       it "adds an error to the model" do
@@ -21,19 +21,19 @@ shared_examples 'database error rescuing' do
     end
 
     context "when the validator is not present" do
-      before { Rescuable.stub(validators: [Rescuable.presence_validator]) }
+      before { allow(Rescuable).to(receive(:validators).and_return([Rescuable.presence_validator])) }
 
       it "raises an exception" do
-        expect{ subject.create_or_update }.to raise_error(ActiveRecord::RecordNotUnique)
+        expect { subject.create_or_update }.to raise_error(ActiveRecord::RecordNotUnique)
       end
     end
   end
 
   describe "#create_or_update when using rescuer without validation" do
     before {
-      Rescuable.stub(_validators: {})
-      Rescuable.stub(_rescue_from_duplicates: [Rescuable.uniqueness_rescuer])
-      Base.stub(exception: uniqueness_exception)
+      allow(Rescuable).to(receive(:_validators).and_return({}))
+      allow(Rescuable).to(receive(:_rescue_from_duplicates).and_return([Rescuable.uniqueness_rescuer]))
+      allow(Base).to(receive(:exception).and_return(uniqueness_exception))
     }
 
     it "adds an error to the model" do
@@ -44,18 +44,14 @@ shared_examples 'database error rescuing' do
 end
 
 describe RescueFromDuplicate::ActiveRecord do
-  if defined?(MysqlModel)
-    context 'mysql' do
-      let(:message) { "Duplicate entry '1-Rescuable-toto' for key 'index_rescuable_on_shop_id_and_type_and_name'" }
-      it_behaves_like 'database error rescuing'
-    end
+  context 'mysql' do
+    let(:message) { "Duplicate entry '1-Rescuable-toto' for key 'index_rescuable_on_shop_id_and_type_and_name'" }
+    it_behaves_like 'database error rescuing'
   end
 
-  if defined?(PostgresqlModel)
-    context 'pgsql' do
-      let(:message) { "PG::UniqueViolation: ERROR:  duplicate key value violates unique constraint \"index_rescuable_on_shop_id_and_type_and_name\"\nDETAIL:  Key (shop_id, type, name)=(1, Rescuable, toto) already exists.\n: INSERT INTO \"postgresql_models\" (\"shop_id\", \"type\", \"name\") VALUES ($1, $2, $3) RETURNING \"id\"" }
-      it_behaves_like 'database error rescuing'
-    end
+  context 'pgsql' do
+    let(:message) { "PG::UniqueViolation: ERROR:  duplicate key value violates unique constraint \"index_rescuable_on_shop_id_and_type_and_name\"\nDETAIL:  Key (shop_id, type, name)=(1, Rescuable, toto) already exists.\n: INSERT INTO \"postgresql_models\" (\"shop_id\", \"type\", \"name\") VALUES ($1, $2, $3) RETURNING \"id\"" }
+    it_behaves_like 'database error rescuing'
   end
 
   context 'sqlite3' do

data/spec/rescuer_spec.rb

@@ -35,14 +35,10 @@ describe Sqlite3Model do
   it_behaves_like 'a model with rescued unique error without validator'
 end
 
-if defined?(MysqlModel)
-  describe MysqlModel do
-    it_behaves_like 'a model with rescued unique error without validator'
-  end
+describe MysqlModel do
+  it_behaves_like 'a model with rescued unique error without validator'
 end
 
-if defined?(PostgresqlModel)
-  describe PostgresqlModel do
-    it_behaves_like 'a model with rescued unique error without validator'
-  end
+describe PostgresqlModel do
+  it_behaves_like 'a model with rescued unique error without validator'
 end

data/spec/spec_helper.rb

@@ -83,8 +83,6 @@ module RescueFromDuplicate
         "index_rescuable_on_shop_id_and_type_and_name",
         true,
         ["shop_id", "type", "name"],
-        [nil, nil, nil],
-        nil
       )
     end
   end

data/spec/support/model.rb

@@ -1,18 +1,30 @@
-require 'active_record'
-require 'json'
-require 'yaml'
-
-AR_VERSION = Gem::Version.new(ActiveRecord::VERSION::STRING)
-AR_4_0 = Gem::Version.new('4.0')
-AR_4_1 = Gem::Version.new('4.1.0.beta')
-
-ActiveRecord::Base.configurations = {
-  'test_sqlite3' => {adapter: 'sqlite3', database: "/tmp/rescue_from_duplicate.db"},
-  'test_postgresql' => {adapter: 'postgresql', database: 'rescue_from_duplicate', username: 'postgres'},
-  'test_mysql' => {adapter: 'mysql2', database: 'rescue_from_duplicate', username: 'travis'},
+require "active_record"
+require "json"
+require "yaml"
+
+CONNECTIONS = {
+  test_sqlite3: {
+    adapter: "sqlite3",
+    database: "/tmp/rescue_from_duplicate.db",
+  },
+  test_postgresql: {
+    adapter: "postgresql",
+    database: "rescue_from_duplicate",
+    username: "test",
+    password: "test",
+    host: "127.0.0.1",
+    port: "29292",
+  },
+  test_mysql: {
+    adapter: "mysql2",
+    database: "rescue_from_duplicate",
+    username: "test",
+    password: "test",
+    host: "127.0.0.1",
+    port: "29291",
+  },
 }
-
-class CreateAllTables < ActiveRecord::Migration
+class CreateAllTables < ActiveRecord::Migration[5.2]
   def self.recreate_table(name, *args, &block)
     execute "drop table if exists #{name}"
 
@@ -30,17 +42,13 @@ class CreateAllTables < ActiveRecord::Migration
   end
 
   def self.up
-    if ENV['MYSQL']
-      ActiveRecord::Base.establish_connection('test_mysql')
-      recreate_table(:mysql_models)
-    end
+    ActiveRecord::Base.establish_connection(CONNECTIONS.fetch(:test_mysql))
+    recreate_table(:mysql_models)
 
-    if ENV['POSTGRES']
-      ActiveRecord::Base.establish_connection(ENV['POSTGRES_URL'] || 'test_postgresql')
-      recreate_table(:postgresql_models)
-    end
+    ActiveRecord::Base.establish_connection(CONNECTIONS.fetch(:test_postgresql))
+    recreate_table(:postgresql_models)
 
-    ActiveRecord::Base.establish_connection('test_sqlite3')
+    ActiveRecord::Base.establish_connection(CONNECTIONS.fetch(:test_sqlite3))
     recreate_table(:sqlite3_models)
   end
 end
@@ -50,43 +58,37 @@ CreateAllTables.up
 
 
 module TestModel
-  extend ActiveSupport::Concern
-
-  included do
-    rescue_from_duplicate :handle, scope: :relation_id, message: "handle must be unique for this relation"
+  def self.included(base)
+    base.rescue_from_duplicate(:handle, scope: :relation_id, message: "handle must be unique for this relation")
 
-    validates_uniqueness_of :name, rescue_from_duplicate: true, allow_nil: true
-    validates_uniqueness_of :size, allow_nil: true
+    base.validates(:name, uniqueness: { rescue_from_duplicate: true }, allow_nil: true)
+    base.validates(:size, uniqueness: { rescue_from_duplicate: true }, allow_nil: true)
   end
 end
 
-if ENV['MYSQL']
-  class MysqlModel < ActiveRecord::Base
-    include TestModel
+class MysqlModel < ActiveRecord::Base
+  include TestModel
 
-    establish_connection 'test_mysql'
-  end
+  establish_connection(CONNECTIONS.fetch(:test_mysql))
 end
 
-if ENV['POSTGRES']
-  class PostgresqlModel < ActiveRecord::Base
-    include TestModel
+class PostgresqlModel < ActiveRecord::Base
+  include TestModel
 
-    establish_connection ENV['POSTGRES_URL'] || 'test_postgresql'
-  end
+  establish_connection(CONNECTIONS.fetch(:test_postgresql))
 end
 
 class Sqlite3Model < ActiveRecord::Base
   include TestModel
 
-  establish_connection 'test_sqlite3'
+  establish_connection(CONNECTIONS.fetch(:test_sqlite3))
 end
 
 Models = [
-  Sqlite3Model
+  Sqlite3Model,
+  MysqlModel,
+  PostgresqlModel,
 ]
-Models << MysqlModel if defined?(MysqlModel)
-Models << PostgresqlModel if defined?(PostgresqlModel)
 
 
 RSpec.configure do |config|

data/spec/validator_spec.rb

@@ -5,7 +5,8 @@ shared_examples 'a model with rescued uniqueness validator' do
     context 'when catching a race condition' do
 
       before(:each) {
-        ActiveRecord::Validations::UniquenessValidator.any_instance.stub(validate_each: nil)
+        allow_any_instance_of(ActiveRecord::Validations::UniquenessValidator)
+          .to(receive(:validate_each)).and_return(nil)
         described_class.create!(name: 'toto', size: 5)
       }
 
@@ -55,7 +56,7 @@ shared_examples 'missing index finding' do
 
     context 'indexes are missing' do
       before {
-        described_class.stub(_rescue_from_duplicate_handlers: [
+        allow(described_class).to(receive(:_rescue_from_duplicate_handlers).and_return([
           RescueFromDuplicate::UniquenessRescuer.new(
             ::ActiveRecord::Validations::UniquenessValidator.new(
               attributes: [:name],
@@ -63,7 +64,7 @@ shared_examples 'missing index finding' do
             )
           ),
           RescueFromDuplicate::Rescuer.new(:name, scope: [:hello])
-        ])
+        ]))
       }
 
       it 'returns the missing indexes' do
@@ -88,16 +89,12 @@ describe Sqlite3Model do
   it_behaves_like 'missing index finding'
 end
 
-if defined?(MysqlModel)
-  describe MysqlModel do
-    it_behaves_like 'a model with rescued uniqueness validator'
-    it_behaves_like 'missing index finding'
-  end
+describe MysqlModel do
+  it_behaves_like 'a model with rescued uniqueness validator'
+  it_behaves_like 'missing index finding'
 end
 
-if defined?(PostgresqlModel)
-  describe PostgresqlModel do
-    it_behaves_like 'a model with rescued uniqueness validator'
-    it_behaves_like 'missing index finding'
-  end
+describe PostgresqlModel do
+  it_behaves_like 'a model with rescued uniqueness validator'
+  it_behaves_like 'missing index finding'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-rescue_from_duplicate
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Guillaume Malette
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-07-22 00:00:00.000000000 Z
+date: 2019-04-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: '5.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: '5.2'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -151,11 +151,7 @@ files:
 - README.md
 - Rakefile
 - activerecord-rescue_from_duplicate.gemspec
-- gemfiles/Gemfile.ar-3.2
-- gemfiles/Gemfile.ar-4.0
-- gemfiles/Gemfile.ar-4.1
-- gemfiles/Gemfile.ar-4.2
-- gemfiles/Gemfile.ar-edge
+- docker-compose.yml
 - lib/activerecord-rescue_from_duplicate.rb
 - lib/rescue_from_duplicate/active_record.rb
 - lib/rescue_from_duplicate/active_record/extension.rb
@@ -190,7 +186,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.7.6
 signing_key: 
 specification_version: 4
 summary: Rescue from MySQL and Sqlite duplicate errors when trying to insert records

data/gemfiles/Gemfile.ar-3.2

@@ -1,10 +0,0 @@
-source 'https://rubygems.org'
-
-gem 'activerecord', '~> 3.2.15'
-gem 'bundler', '~> 1.3'
-gem 'rake'
-gem 'rspec'
-gem 'sqlite3'
-gem 'pg', '~> 0.11'
-gem 'mysql2', '~> 0.3.20'
-gem 'simplecov', require: false

data/gemfiles/Gemfile.ar-4.0

@@ -1,10 +0,0 @@
-source 'https://rubygems.org'
-
-gem 'activerecord', '~> 4.0.0'
-gem 'bundler', '~> 1.3'
-gem 'rake'
-gem 'rspec'
-gem 'sqlite3'
-gem 'pg', '~> 0.11'
-gem 'mysql2', '~> 0.3.20'
-gem 'simplecov', require: false

data/gemfiles/Gemfile.ar-4.1

@@ -1,10 +0,0 @@
-source 'https://rubygems.org'
-
-gem 'activerecord', '~> 4.1.0'
-gem 'bundler', '~> 1.3'
-gem 'rake'
-gem 'rspec'
-gem 'sqlite3'
-gem 'pg', '~> 0.11'
-gem 'mysql2', '~> 0.3.20'
-gem 'simplecov', require: false

data/gemfiles/Gemfile.ar-4.2

@@ -1,10 +0,0 @@
-source 'https://rubygems.org'
-
-gem 'activerecord', '~> 4.2.0'
-gem 'bundler', '~> 1.3'
-gem 'rake'
-gem 'rspec'
-gem 'sqlite3'
-gem 'pg', '~> 0.11'
-gem 'mysql2', '~> 0.3.20'
-gem 'simplecov', require: false

data/gemfiles/Gemfile.ar-edge

@@ -1,12 +0,0 @@
-source 'https://rubygems.org'
-
-gem 'rails', github: 'rails/rails'
-gem 'arel', github: 'rails/arel'
-
-gem 'bundler', '~> 1.3'
-gem 'rake'
-gem 'rspec'
-gem 'sqlite3'
-gem 'pg', '~> 0.11'
-gem 'mysql2', '~> 0.3.20'
-gem 'simplecov', require: false