active_record_extended 1.3.1 → 2.1.0

data/spec/sql_inspections/unionize_sql_spec.rb

@@ -74,7 +74,7 @@ RSpec.describe "Union SQL Queries" do
 
       it "should alias the union from clause to 'happy_users'" do
         expect(described_method).to match_regex(/FROM \(+.+\) UNION \(.+\)+ happy_users$/)
-        expect(described_method).to match_regex(/^SELECT happy_users\.id FROM.+happy_users$/)
+        expect(described_method).to match_regex(/^SELECT (happy_users\.id|"happy_users"\."id") FROM.+happy_users$/)
       end
     end
 
@@ -83,7 +83,7 @@ RSpec.describe "Union SQL Queries" do
 
       it "should retain the actual class calling table name as the union alias" do
         expect(described_method).to match_regex(/FROM \(+.+\) UNION \(.+\)+ users$/)
-        expect(described_method).to match_regex(/^SELECT \"users\"\.\"id\" FROM.+users$/)
+        expect(described_method).to match_regex(/^SELECT "users"\."id" FROM.+users$/)
       end
     end
   end

data/spec/sql_inspections/window_sql_spec.rb

@@ -82,5 +82,17 @@ RSpec.describe "Active Record WINDOW Query inspection" do
         end
       end
     end
+
+    context "when not providing a partition by value" do
+      it "should construct a window function" do
+        query =
+          Tag
+          .define_window(:no_args).partition_by(order_by: { tag_number: :desc })
+          .select_window(:row_number, over: :no_args, as: :my_row)
+          .to_sql
+
+        expect(query).to eq("SELECT (ROW_NUMBER() OVER no_args) AS \"my_row\" FROM \"tags\" WINDOW no_args AS (ORDER BY tag_number DESC)")
+      end
+    end
   end
 end

data/spec/sql_inspections/with_cte_sql_spec.rb

@@ -21,6 +21,20 @@ RSpec.describe "Active Record WITH CTE tables" do
     expect(query).to match_regex(with_personal_query)
   end
 
+  it "will pipe Children CTE's into the Parent relation" do
+    personal_id_one_query = User.where(personal_id: 1)
+    personal_id_two_query = User.where(personal_id: 2)
+
+    sub_query       = personal_id_two_query.with(personal_id_one: personal_id_one_query)
+    query           = User.all.with(personal_id_two: sub_query)
+    expected_order  = User.with(
+      personal_id_one: personal_id_one_query,
+      personal_id_two: personal_id_two_query
+    )
+
+    expect(query.to_sql).to eq(expected_order.to_sql)
+  end
+
   context "when multiple CTE's" do
     let(:chained_with) do
       User.with(personal_id_one: User.where(personal_id: 1))
@@ -36,6 +50,7 @@ RSpec.describe "Active Record WITH CTE tables" do
           .joins("JOIN personal_id_two ON personal_id_two.id = users.id")
           .to_sql
     end
+
     it "Should only contain a single WITH statement" do
       expect(with_arguments.scan(/WITH/).count).to eq(1)
       expect(with_arguments.scan(/AS/).count).to eq(2)
@@ -60,7 +75,21 @@ RSpec.describe "Active Record WITH CTE tables" do
     end
 
     it "generates an expression with recursive" do
-      expect(with_recursive).to match_regex(with_recursive_personal_query)
+      query = User.with
+                  .recursive(personal_id_one: User.where(personal_id: 1))
+                  .joins("JOIN personal_id_one ON personal_id_one.id = users.id")
+                  .to_sql
+
+      expect(query).to match_regex(with_recursive_personal_query)
+    end
+
+    it "will maintain the CTE table when merging" do
+      sub_query = User.with.recursive(personal_id_one: User.where(personal_id: 1))
+      query     = User.merge(sub_query)
+                      .joins("JOIN personal_id_one ON personal_id_one.id = users.id")
+                      .to_sql
+
+      expect(query).to match_regex(with_recursive_personal_query)
     end
   end
 end

data/spec/support/database_cleaner.rb

@@ -2,7 +2,7 @@
 
 require "database_cleaner"
 
-DatabaseCleaner.strategy = :truncation
+DatabaseCleaner.strategy = :transaction
 
 RSpec.configure do |config|
   config.before do

data/spec/support/models.rb

@@ -5,6 +5,8 @@ class ApplicationRecord < ActiveRecord::Base
 end
 
 class User < ApplicationRecord
+  has_many :groups_users, class_name: "GroupsUser"
+  has_many :groups, through: :groups_users, dependent: :destroy
   has_many :hm_tags, class_name: "Tag"
   has_one :profile_l, class_name: "ProfileL"
   has_one :profile_r, class_name: "ProfileR"
@@ -66,3 +68,13 @@ class VersionControl < ApplicationRecord
   # t.jsonb :source, default: {}, null: false
   #
 end
+
+class Group < ApplicationRecord
+  has_many :groups_users, class_name: "GroupsUser"
+  has_many :users, through: :groups_users, dependent: :destroy
+end
+
+class GroupsUser < ApplicationRecord
+  belongs_to :user
+  belongs_to :group
+end

metadata

@@ -1,16 +1,16 @@
 --- !ruby/object:Gem::Specification
 name: active_record_extended
 version: !ruby/object:Gem::Version
-  version: 1.3.1
+  version: 2.1.0
 platform: ruby
 authors:
 - George Protacio-Karaszi
 - Dan McClain
 - Olivier El Mekki
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-10-02 00:00:00.000000000 Z
+date: 2022-01-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -18,20 +18,20 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '5.1'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: 7.1.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '5.1'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: 7.1.0
 - !ruby/object:Gem::Dependency
   name: ar_outer_joins
   requirement: !ruby/object:Gem::Requirement
@@ -52,14 +52,14 @@ dependencies:
     requirements:
     - - "<"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "<"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -69,7 +69,7 @@ dependencies:
         version: '1.16'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '2.1'
+        version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -79,7 +79,7 @@ dependencies:
         version: '1.16'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '2.1'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: database_cleaner
   requirement: !ruby/object:Gem::Requirement
@@ -98,14 +98,14 @@ dependencies:
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '10.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '10.0'
 - !ruby/object:Gem::Dependency
@@ -153,9 +153,8 @@ files:
 - lib/active_record_extended/arel/aggregate_function_name.rb
 - lib/active_record_extended/arel/nodes.rb
 - lib/active_record_extended/arel/predications.rb
+- lib/active_record_extended/arel/sql_literal.rb
 - lib/active_record_extended/arel/visitors/postgresql_decorator.rb
-- lib/active_record_extended/patch/5_0/predicate_builder_decorator.rb
-- lib/active_record_extended/patch/5_0/regex_match.rb
 - lib/active_record_extended/patch/5_1/where_clause.rb
 - lib/active_record_extended/patch/5_2/where_clause.rb
 - lib/active_record_extended/predicate_builder/array_handler_decorator.rb
@@ -199,7 +198,7 @@ homepage: https://github.com/georgekaraszi/ActiveRecordExtended
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -207,16 +206,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.4'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.7
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: Adds extended functionality to Activerecord Postgres implementation
 test_files:

data/lib/active_record_extended/patch/5_0/predicate_builder_decorator.rb

@@ -1,87 +0,0 @@
-# frozen_string_literal: true
-
-# Stripped from Rails 5.1.x
-# This patch is used so that when querying with hash elements that do not belong to an association,
-# but instead a data attribute. It returns the corrected attribute binds back to the query builder.
-#
-# Without joins
-# Before:
-#   User.where.contains(data: { nickname: "george" })
-#     #=> "SELECT \"people\".* FROM \"people\" WHERE (\"data\".\"nickname\" @> 'george')"
-#
-# After:
-#  User.where.contains(data: { nickname: "george" })
-#   #=> "SELECT \"people\".* FROM \"people\" WHERE (\"people\".\"data\" @> '\"nickname\"=>\"george\"')"
-#
-# With Joins
-# Before:
-#   Tag.joins(:user).where.contains(people: { data: { nickname: "george" } })
-#   #=> NoMethodError: undefined method `type' for nil:NilClass
-#
-# After:
-#  Tag.joins(:user).where.contains(people: { data: { nickname: "george" } })
-#  #=> "SELECT \"tags\".* FROM \"tags\" INNER JOIN \"people\" ON \"people\".\"id\" = \"tags\".\"person_id\"
-#         WHERE (\"people\".\"data\" @> '\"nickname\"=>\"george\"')"
-#
-module ActiveRecord
-  class TableMetadata
-    def has_column?(column_name) # rubocop:disable Naming/PredicateName
-      klass&.columns_hash&.key?(column_name.to_s)
-    end
-  end
-
-  class PredicateBuilder
-    def create_binds_for_hash(attributes) # rubocop:disable Metrics/PerceivedComplexity, Metrics/AbcSize
-      result = attributes.dup
-      binds = []
-
-      attributes.each do |column_name, value| # rubocop:disable Metrics/BlockLength
-        if value.is_a?(Hash) && !table.has_column?(column_name)
-          attrs, bvs = associated_predicate_builder(column_name).create_binds_for_hash(value)
-          result[column_name] = attrs
-          binds += bvs
-          next
-        elsif value.is_a?(Relation)
-          binds += value.bound_attributes
-        elsif value.is_a?(Range) && !table.type(column_name).respond_to?(:subtype)
-          first = value.begin
-          last = value.end
-          unless first.respond_to?(:infinite?) && first.infinite?
-            binds << build_bind_param(column_name, first)
-            first = Arel::Nodes::BindParam.new
-          end
-          unless last.respond_to?(:infinite?) && last.infinite?
-            binds << build_bind_param(column_name, last)
-            last = Arel::Nodes::BindParam.new
-          end
-
-          result[column_name] = RangeHandler::RangeWithBinds.new(first, last, value.exclude_end?)
-        elsif can_be_bound?(column_name, value)
-          result[column_name] = Arel::Nodes::BindParam.new
-          binds << build_bind_param(column_name, value)
-        end
-
-        # Find the foreign key when using queries such as:
-        # Post.where(author: author)
-        #
-        # For polymorphic relationships, find the foreign key and type:
-        # PriceEstimate.where(estimate_of: treasure)
-        if table.associated_with?(column_name)
-          result[column_name] = AssociationQueryHandler.value_for(table, column_name, value)
-        end
-      end
-
-      [result, binds]
-    end
-
-    def can_be_bound?(column_name, value)
-      return if table.associated_with?(column_name)
-      case value
-      when Array, Range
-        table.type(column_name).respond_to?(:subtype)
-      else
-        !value.nil? && handler_for(value).is_a?(BasicObjectHandler)
-      end
-    end
-  end
-end

data/lib/active_record_extended/patch/5_0/regex_match.rb

@@ -1,10 +0,0 @@
-# frozen_string_literal: true
-
-class Regexp
-  # Stripped from ActiveSupport v5.1
-  unless //.respond_to?(:match?)
-    def match?(string, pos = 0)
-      !(!match(string, pos))
-    end
-  end
-end