schema_plus_indexes 0.3.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 59c1cc31f6d11ef0482b7592f398503e4de3081b
-  data.tar.gz: cb27adde0cb695f05991c39275bb3219fc41bffa
+SHA256:
+  metadata.gz: 01e90c31ef7cb567d005ad99b6c5532096e1d1128f2427686f7b4024ea49c156
+  data.tar.gz: 240e9d3ca09e91d1b0a3c04946876427310b19ec243c491a40b672de41287d0b
 SHA512:
-  metadata.gz: d51dd7ccb1d59af210e3ab66a1e32dc138586e54547b88cfad701c001d3401ce2c1b7c9b781484286a8ddd13059112a801285aa50ab5bc9ae3a5431c27e89b93
-  data.tar.gz: 6d7744f9e799a81d8e23bdc7793b9563070c33dc9b7f2b9d09e1d8661b8d52b0278c94f512ca891b1951cbca4c94970f2feb9b4f265724a163784262543f6998
+  metadata.gz: a404889345f7a83ee71d6b9f25582ae78d492b919f447834c6934bffafe535bf2ecee76da798423fc549f56e3e1399554fe71bcf821c28a2a3876ac5605ac568
+  data.tar.gz: 575d9ef92201129e21f1f789c88edf42f29a5dd65d26a5c296e56477a43ce12f5d738ed4d0096b199321732ec42fc71b255410cf62b501535578e12243615fb8

data/.travis.yml

@@ -16,6 +16,9 @@ gemfile:
 - gemfiles/activerecord-5.1/Gemfile.mysql2
 - gemfiles/activerecord-5.1/Gemfile.postgresql
 - gemfiles/activerecord-5.1/Gemfile.sqlite3
+- gemfiles/activerecord-5.2/Gemfile.mysql2
+- gemfiles/activerecord-5.2/Gemfile.postgresql
+- gemfiles/activerecord-5.2/Gemfile.sqlite3
 env: POSTGRESQL_DB_USER=postgres MYSQL_DB_USER=travis
 addons:
   postgresql: '9.4'

data/README.md

@@ -108,12 +108,17 @@ schema_plus_indexes is tested on
 * ruby **2.3.1** with activerecord **4.2**, using **mysql2**, **sqlite3** or **postgresql**
 * ruby **2.3.1** with activerecord **5.0**, using **mysql2**, **sqlite3** or **postgresql**
 * ruby **2.3.1** with activerecord **5.1**, using **mysql2**, **sqlite3** or **postgresql**
+* ruby **2.3.1** with activerecord **5.2**, using **mysql2**, **sqlite3** or **postgresql**
 
 <!-- SCHEMA_DEV: MATRIX - end -->
 
 ## Release Notes
 
-### v0.2.5
+### v0.3.1
+
+* Support AR 5.2.   Thanks to [@jeremyyap](https://github.com/jeremyyap)
+
+### v0.3.0
 
 * Supports AR 5.1.  Thanks to [@iagopiimenta](https://github.com/iagopiimenta)
 

data/gemfiles/activerecord-4.2/Gemfile.postgresql

@@ -2,9 +2,9 @@ require "pathname"
 eval(Pathname.new(__FILE__).dirname.join("Gemfile.base").read, binding)
 
 platform :ruby do
-  gem "pg"
+  gem "pg", "< 1"
 end
 
 platform :jruby do
   gem 'activerecord-jdbcpostgresql-adapter'
-end
+end

data/gemfiles/activerecord-5.0/Gemfile.postgresql

@@ -2,9 +2,9 @@ require "pathname"
 eval(Pathname.new(__FILE__).dirname.join("Gemfile.base").read, binding)
 
 platform :ruby do
-  gem "pg"
+  gem "pg", "< 1"
 end
 
 platform :jruby do
   gem 'activerecord-jdbcpostgresql-adapter'
-end
+end

data/gemfiles/activerecord-5.1/Gemfile.postgresql

@@ -2,9 +2,9 @@ require "pathname"
 eval(Pathname.new(__FILE__).dirname.join("Gemfile.base").read, binding)
 
 platform :ruby do
-  gem "pg"
+  gem "pg", "< 1"
 end
 
 platform :jruby do
   gem 'activerecord-jdbcpostgresql-adapter'
-end
+end

data/gemfiles/activerecord-5.2/Gemfile.base

@@ -0,0 +1,3 @@
+eval File.read File.expand_path('../../Gemfile.base', __FILE__)
+
+gem "activerecord", ">= 5.2.0.beta0", "< 5.3"

data/gemfiles/activerecord-5.2/Gemfile.mysql2

@@ -0,0 +1,10 @@
+require "pathname"
+eval(Pathname.new(__FILE__).dirname.join("Gemfile.base").read, binding)
+
+platform :ruby do
+  gem "mysql2"
+end
+
+platform :jruby do
+  gem 'activerecord-jdbcmysql-adapter'
+end

data/gemfiles/activerecord-5.2/Gemfile.postgresql

@@ -0,0 +1,10 @@
+require "pathname"
+eval(Pathname.new(__FILE__).dirname.join("Gemfile.base").read, binding)
+
+platform :ruby do
+  gem "pg"
+end
+
+platform :jruby do
+  gem 'activerecord-jdbcpostgresql-adapter'
+end

data/gemfiles/activerecord-5.2/Gemfile.sqlite3

@@ -0,0 +1,10 @@
+require "pathname"
+eval(Pathname.new(__FILE__).dirname.join("Gemfile.base").read, binding)
+
+platform :ruby do
+  gem "sqlite3"
+end
+
+platform :jruby do
+  gem 'activerecord-jdbcsqlite3-adapter', '>=1.3.0.beta2'
+end

data/lib/schema_plus/indexes.rb

@@ -1,5 +1,6 @@
 require 'schema_plus/core'
 require 'its-it'
+require 'active_record'
 
 require_relative 'indexes/remove_if_exists'
 require_relative 'indexes/active_record/base'

data/lib/schema_plus/indexes/active_record/base.rb

@@ -6,7 +6,7 @@ module SchemaPlus::Indexes
         public
 
         # Returns a list of IndexDefinition objects, for each index
-        # defind on this model's table.
+        # defined on this model's table.
         #
         def indexes
           @indexes ||= connection.indexes(table_name)

data/lib/schema_plus/indexes/active_record/connection_adapters/index_definition.rb

@@ -3,19 +3,42 @@ module SchemaPlus::Indexes
     module ConnectionAdapters
 
       module IndexDefinition
+        if Gem::Version.new(::ActiveRecord::VERSION::STRING) >= Gem::Version.new('5.2')
+          attr_accessor :orders
 
-        def initialize(*args) #:nodoc:
-          # same args as add_index(table_name, column_names, options)
-          if args.length == 3 and Hash === args.last
-            table_name, column_names, options = args + [{}]
-            super table_name, options[:name], options[:unique], column_names, options[:length], options[:orders], options[:where], options[:type], options[:using]
-          else # backwards compatibility
-            super
+          def initialize(*args) #:nodoc:
+            # same args as add_index(table_name, column_names, options)
+            if args.length == 3 and Hash === args.last
+              table_name, column_names, options = args + [{}]
+
+              super table_name, options[:name], options[:unique], column_names, options.except(:name, :unique)
+            else # backwards compatibility
+              super
+            end
+            unless orders.blank?
+              # fill out orders with :asc when undefined.  make sure hash ordering
+              # follows column ordering.
+              if self.orders.is_a?(Hash)
+                self.orders = Hash[columns.map{|column| [column, orders[column] || :asc]}]
+              else
+                self.orders = Hash[columns.map{|column| [column, orders || :asc]}]
+              end
+            end
           end
-          unless orders.blank?
-            # fill out orders with :asc when undefined.  make sure hash ordering
-            # follows column ordering.
-            self.orders = Hash[columns.map{|column| [column, orders[column] || :asc]}]
+        else
+          def initialize(*args) #:nodoc:
+            # same args as add_index(table_name, column_names, options)
+            if args.length == 3 and Hash === args.last
+              table_name, column_names, options = args + [{}]
+              super table_name, options[:name], options[:unique], column_names, options[:length], options[:orders], options[:where], options[:type], options[:using]
+            else # backwards compatibility
+              super
+            end
+            unless orders.blank?
+              # fill out orders with :asc when undefined.  make sure hash ordering
+              # follows column ordering.
+              self.orders = Hash[columns.map{|column| [column, orders[column] || :asc]}]
+            end
           end
         end
 

data/lib/schema_plus/indexes/remove_if_exists.rb

@@ -2,12 +2,22 @@ module SchemaPlus::Indexes
   module RemoveIfExists
     # Extends rails' remove_index to include this options:
     #   :if_exists
-    def remove_index(table_name, *args)
-      options = args.extract_options!
-      if_exists = options.delete(:if_exists)
-      args << options if options.any?
-      return if if_exists && !index_name_exists?(table_name, options[:name] || index_name(table_name, *args), nil)
-      super table_name, *args
+    if Gem::Version.new(::ActiveRecord::VERSION::STRING) >= Gem::Version.new('5.1')
+      def remove_index(table_name, *args)
+        options = args.extract_options!
+        if_exists = options.delete(:if_exists)
+        args << options if options.any?
+        return if if_exists && !index_name_exists?(table_name, options[:name] || index_name(table_name, *args))
+        super table_name, *args
+      end
+    else
+      def remove_index(table_name, *args)
+        options = args.extract_options!
+        if_exists = options.delete(:if_exists)
+        args << options if options.any?
+        return if if_exists && !index_name_exists?(table_name, options[:name] || index_name(table_name, *args), nil)
+        super table_name, *args
+      end
     end
   end
 end

data/lib/schema_plus/indexes/version.rb

@@ -1,5 +1,5 @@
 module SchemaPlus
   module Indexes
-    VERSION = "0.3.0"
+    VERSION = "0.3.1"
   end
 end

data/schema_dev.yml

@@ -4,6 +4,7 @@ activerecord:
     - 4.2
     - 5.0
     - 5.1
+    - 5.2
 db:
     - mysql2
     - sqlite3

data/schema_plus_indexes.gemspec

@@ -17,13 +17,13 @@ Gem::Specification.new do |gem|
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.require_paths = ["lib"]
 
-  gem.add_dependency "activerecord", ">= 4.2", "< 5.2"
+  gem.add_dependency "activerecord", ">= 4.2", "< 5.3"
   gem.add_dependency "schema_plus_core"
   gem.add_dependency "its-it", "~> 1.2"
 
   gem.add_development_dependency "bundler", "~> 1.7"
   gem.add_development_dependency "rake", "~> 10.0"
-  gem.add_development_dependency "rspec", "~> 3.0.0"
+  gem.add_development_dependency "rspec", "~> 3.0"
   gem.add_development_dependency "schema_dev", "~> 3.6"
   gem.add_development_dependency "simplecov"
   gem.add_development_dependency "simplecov-gem-profile"

data/spec/index_definition_spec.rb

@@ -23,8 +23,14 @@ describe "Index definition" do
     class Post < ::ActiveRecord::Base ; end
   end
 
-  after(:each) do
-    migration.remove_index :users, :name => 'users_login_index' if migration.index_name_exists? :users, 'users_login_index', nil
+  if Gem::Version.new(ActiveRecord::VERSION::STRING) >= Gem::Version.new('5.1')
+    after(:each) do
+      migration.remove_index :users, :name => 'users_login_index' if migration.index_name_exists?(:users, 'users_login_index')
+    end
+  else
+    after(:each) do
+      migration.remove_index :users, :name => 'users_login_index' if migration.index_name_exists?(:users, 'users_login_index', nil)
+    end
   end
 
   context "when index is multicolumn" do

data/spec/index_spec.rb

@@ -53,6 +53,11 @@ describe "index" do
       expect(index_for([:login, :deleted_at]).orders).to eq({"login" => :desc, "deleted_at" => :asc})
     end
 
+    it "should assign order (all same direction)", :mysql => :skip do
+      add_index(:users, [:login, :deleted_at], :order => {:login => :desc, :deleted_at => :desc})
+      expect(index_for([:login, :deleted_at]).orders).to eq({"login" => :desc, "deleted_at" => :desc})
+    end
+
     context "for duplicate index" do
       it "should not complain if the index is the same" do
         add_index(:users, :login)
@@ -64,7 +69,7 @@ describe "index" do
       it "should complain if the index is different" do
         add_index(:users, :login, :unique => true)
         expect(index_for(:login)).not_to be_nil
-        expect { add_index(:users, :login) }.to raise_error
+        expect { add_index(:users, :login) }.to raise_error(ArgumentError, /already exists/)
         expect(index_for(:login)).not_to be_nil
       end
     end
@@ -150,7 +155,7 @@ describe "index" do
     it "raises exception if doesn't exist" do
       expect {
         remove_index :users, :login
-      }.to raise_error
+      }.to raise_error(ArgumentError)
     end
 
     it "doesn't raise exception with :if_exists" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: schema_plus_indexes
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors:
 - ronen barzel
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-07-24 00:00:00.000000000 Z
+date: 2018-12-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -19,7 +19,7 @@ dependencies:
         version: '4.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '5.3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -29,7 +29,7 @@ dependencies:
         version: '4.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '5.3'
 - !ruby/object:Gem::Dependency
   name: schema_plus_core
   requirement: !ruby/object:Gem::Requirement
@@ -92,14 +92,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: schema_dev
   requirement: !ruby/object:Gem::Requirement
@@ -168,6 +168,10 @@ files:
 - gemfiles/activerecord-5.1/Gemfile.mysql2
 - gemfiles/activerecord-5.1/Gemfile.postgresql
 - gemfiles/activerecord-5.1/Gemfile.sqlite3
+- gemfiles/activerecord-5.2/Gemfile.base
+- gemfiles/activerecord-5.2/Gemfile.mysql2
+- gemfiles/activerecord-5.2/Gemfile.postgresql
+- gemfiles/activerecord-5.2/Gemfile.sqlite3
 - lib/schema_plus/indexes.rb
 - lib/schema_plus/indexes/active_record/base.rb
 - lib/schema_plus/indexes/active_record/connection_adapters/abstract_adapter.rb
@@ -213,7 +217,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.7.8
 signing_key: 
 specification_version: 4
 summary: Adds shorthands and conveniences to ActiveRecord's handling of indexes