updateable_views_inheritance 1.4.8 → 1.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a7b78d1ecf36dad960f1b6ffccb8a640cbb9e1f06a8a8c1719f25f022e040aff
-  data.tar.gz: 2a3f41c5bb02ca8b2907c1a563cac2c4ed06429f712fed5f3b5f6ffcd988f34c
+  metadata.gz: a4134e2e8c505f5ca5c6d53762497a18b168b5cb43027a121f337908d1595d5b
+  data.tar.gz: 7dc39d94d346f6efef8862c2482afe72d97f745b822b59b313041dcb21fb9ebc
 SHA512:
-  metadata.gz: 33bbf8b1caf17c212f5f8f2712476f212cea3fd09c5d8ada27c6ee235bd1e8e65792afff98606daeb2c462756f21331f76597bc2f27dae17ecbcc90cfaa41ecc
-  data.tar.gz: b26957b4721cffcae777c4e9e7e7280f2a5ab2d1a008c00f551bb6c0db39fe7c6f19616570efde7d8cc8e18802d36814ce1fca80c6add8401571bc73a6e880bb
+  metadata.gz: e50eeaa1e8e686fe94bd47f02f8109335b7f38efa94e65c108dc5e0c5dcc7509e476ee14ce3de31160673d6bcce0f540dcbf0142f774b1db7f017ed106251b15
+  data.tar.gz: e2e3e32f9b7e6f77c113c1d80701f6a1345a4352df0779d5cfc3f450bfe11fe04c6b65955ba07de440c4733df71ba84073294f274e6c14e4b80fda1024391208

data/.github/workflows/build.yml

@@ -41,15 +41,14 @@ jobs:
       - name: Set up Ruby
       # To automatically get bug fixes and new Ruby versions for ruby/setup-ruby,
       # change this to (see https://github.com/ruby/setup-ruby#versioning):
-      # uses: ruby/setup-ruby@v1
-        uses: ruby/setup-ruby@55283cc23133118229fd3f97f9336ee23a179fcf # v1.146.0
+        uses: ruby/setup-ruby@v1
         with:
           ruby-version: ${{ matrix.ruby-version }}
-          bundler: 1.17.3 # temporary set until rails 5 upgrade
+          bundler: 2.4.22 # temporary set until rails 5 upgrade
           bundler-cache: true # runs 'bundle install' and caches installed gems automatically
 
       - name: Run tests
-        run: bundle exec rake 2>/dev/null # disable warnings until ruby 3 upgrade
+        run: bundle exec rake
 
       # From https://stackoverflow.com/questions/74199483/sonarcloud-ci-cant-find-source-files-for-ruby-simplecov-coverage
       - name: Fix code coverage paths for DeepSource

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+## 1.5.1 (27 February 2025)
+
+Upgrade to Rails 5.1
+
+## 1.5.0 (31 January 2025)
+
+Upgrade to Rails 5
+
 ## 1.4.8 (17 December 2024)
 Bugfixes:
   - Fix pk_and_sequence_for to be interoperable with other AR methods

data/lib/updateable_views_inheritance/active_record_extensions.rb

@@ -0,0 +1,22 @@
+
+module UpdateableViewsInheritance# :nodoc:
+  module ActiveRecordExtensions #:nodoc:
+    module ClassMethods #:nodoc:
+      attr_accessor :disable_inheritance_instantiation
+
+      def instantiate(attributes, column_types = {})
+        object = super(attributes, column_types = {})
+        if object.class.name == self.name || self.disable_inheritance_instantiation
+          object
+        else
+          object.class.find(attributes.with_indifferent_access[:id])
+        end
+      end
+    end
+  end
+end
+
+# Here we override AR class methods. If you need to override
+# instance methods, prepend() them to the class, not to the singleton class
+ActiveRecord::Base.singleton_class.send :prepend, UpdateableViewsInheritance::ActiveRecordExtensions::ClassMethods
+

data/lib/updateable_views_inheritance/postgresql_adapter.rb

@@ -150,16 +150,6 @@ module ActiveRecord #:nodoc:
           execute "DROP VIEW #{quote_table_name(name)}"
         end
 
-        # Return the list of all views in the schema search path.
-        def views(name=nil)
-          schemas = schema_search_path.split(/,\s*/).map { |p| quote(p) }.join(',')
-          query(<<~SQL, name).map { |row| row[0] }
-            SELECT viewname
-              FROM pg_views
-             WHERE schemaname IN (#{schemas})
-          SQL
-        end
-
         # Checks whether relation +name+ is a view.
         def is_view?(name)
           result = query(<<~SQL, name).map { |row| row[0] }
@@ -262,11 +252,6 @@ module ActiveRecord #:nodoc:
           false
         end
 
-        def table_exists_with_updateable_views_inheritance_support?(name)
-          is_view?(name) ? true : table_exists_without_updateable_views_inheritance_support?(name)
-        end
-        alias_method_chain :table_exists?, :updateable_views_inheritance_support
-
         module Tutuf #:nodoc:
           class ClassTableReflection
             class << self
@@ -357,33 +342,37 @@ module ActiveRecord #:nodoc:
           SQL
 
           # update
-          execute(<<~SQL)
+          update_rule = <<~SQL
             CREATE OR REPLACE RULE #{quote_column_name("#{child_view}_update")} AS
             ON UPDATE TO #{quote_table_name(child_view)} DO INSTEAD (
-              #{ if parent_columns.empty?
-                   ''
-                 else
-                   "UPDATE #{parent_table}
-                       SET #{ parent_columns.map { |col| "#{quote_column_name(col)} = NEW.#{quote_column_name(col)}" }.join(', ')}
-                       WHERE #{parent_pk} = OLD.#{parent_pk};"
-                 end }
-              #{ if child_columns.empty?
-                   ''
-                 else
-                   "UPDATE #{child_table}
-                       SET #{ child_columns.map { |col| "#{quote_column_name(col)} = NEW.#{quote_column_name(col)}" }.join(', ')}
-                       WHERE #{child_pk} = OLD.#{parent_pk}"
-                 end }
-            )
           SQL
+          unless parent_columns.empty?
+            update_rule += <<~SQL
+              UPDATE #{parent_table}
+              SET #{parent_columns.map { |col| "#{quote_column_name(col)} = NEW.#{quote_column_name(col)}" }.join(', ')}
+              WHERE #{parent_pk} = OLD.#{parent_pk};
+            SQL
+          end
+          unless child_columns.empty?
+            update_rule += <<~SQL
+              UPDATE #{child_table}
+              SET #{ child_columns.map { |col| "#{quote_column_name(col)} = NEW.#{quote_column_name(col)}" }.join(', ')}
+              WHERE #{child_pk} = OLD.#{parent_pk}
+            SQL
+          end
+          update_rule += ")"
+          execute(update_rule)
         end
 
         def insert_returning_clause(parent_pk, child_pk, child_view)
-          columns_cast_to_null = columns(child_view)
-                                 .reject { |c| c.name == parent_pk }
-                                 .map { |c| "CAST (NULL AS #{c.sql_type})" }
-                                 .join(", ")
-          "RETURNING #{child_pk}, #{columns_cast_to_null}"
+          columns_cast = columns(child_view).map do |c|
+            if c.name == parent_pk
+              "#{child_pk}::#{c.sql_type}"
+            else
+              "NULL::#{c.sql_type}"
+            end
+          end.join(", ")
+          "RETURNING #{columns_cast}"
         end
 
         def create_system_table_records(parent_relation, child_aggregate_view, child_relation)
@@ -403,7 +392,7 @@ module ActiveRecord #:nodoc:
         end
 
         def parent_table(relation)
-          if table_exists?('updateable_views_inheritance')
+          if data_source_exists?('updateable_views_inheritance')
            res = query(<<-SQL, 'Parent relation')[0]
               SELECT parent_relation
                 FROM updateable_views_inheritance

data/lib/updateable_views_inheritance/version.rb

@@ -1,3 +1,3 @@
 module UpdateableViewsInheritance
-  VERSION = "1.4.8"
+  VERSION = "1.5.1"
 end

data/lib/updateable_views_inheritance.rb

@@ -1,5 +1,5 @@
 require 'active_record'
 require 'active_record/connection_adapters/postgresql_adapter'
 require "updateable_views_inheritance/version"
-require "updateable_views_inheritance/active_record"
+require "updateable_views_inheritance/active_record_extensions"
 require 'updateable_views_inheritance/postgresql_adapter'

data/test/deep_hierarchy_test.rb

@@ -3,6 +3,8 @@ require File.join(File.dirname(__FILE__), 'test_helper')
 class DeepHierarchyTest < ActiveSupport::TestCase
   def setup
     ActiveRecord::Migrator.up(File.dirname(__FILE__) + '/fixtures/migrations/', 8)
+
+    ActiveRecord::FixtureSet.reset_cache
     # order of fixtures is important for the test - last loaded should not be with max(id)
     %w(boats electric_trains rack_trains steam_trains cars maglev_trains bicycles).each do |f|
       ActiveRecord::FixtureSet.create_fixtures(File.dirname(__FILE__) + '/fixtures/', f)
@@ -10,10 +12,6 @@ class DeepHierarchyTest < ActiveSupport::TestCase
     @connection = ActiveRecord::Base.connection
   end
 
-  def teardown
-    ActiveRecord::FixtureSet.reset_cache
-  end
-
   def test_deeper_hierarchy
     assert_equal [["boats"], ["railed_vehicles", ["trains", ["steam_trains"], ["rack_trains"], ["electric_trains", ["maglev_trains"]]]], ["wheeled_vehicles", ["bicycles"], ["cars"]]].sort,
                   @connection.send(:get_view_hierarchy_for, :vehicles).sort
@@ -43,17 +41,17 @@ class DeepHierarchyTest < ActiveSupport::TestCase
 
   def test_single_table_inheritance_deeper_hierarchy_contents
     mag = MaglevTrain.first
-    assert_equal [mag.id.to_s, mag.name, mag.number_of_rails.to_s, mag.max_speed.to_s, mag.magnetic_field.to_s, (sprintf("%.2f",mag.electricity_consumption))], (@connection.query("SELECT id, name, number_of_rails, max_speed, magnetic_field, electricity_consumption FROM all_vehicles WHERE id=#{mag.id}").first)
+    assert_equal [mag.id, mag.name, mag.number_of_rails, mag.max_speed, mag.magnetic_field, (sprintf("%.2f",mag.electricity_consumption))], (@connection.query("SELECT id, name, number_of_rails, max_speed, magnetic_field, electricity_consumption FROM all_vehicles WHERE id=#{mag.id}").first)
   end
 
-  class OrderColumnsInAggregateView < ActiveRecord::Migration
-    def self.up
+  class OrderColumnsInAggregateView < ActiveRecord::Migration[4.2]
+    def up
       rebuild_single_table_inheritance_view(:all_vehicles,:vehicles, %w(max_speed number_of_wheels id))
     end
   end
 
   def test_single_table_inheritance_view_order_view_columns
-    OrderColumnsInAggregateView.up
+    OrderColumnsInAggregateView.new.up
     assert_equal %w(max_speed number_of_wheels id),
                  (@connection.query("SELECT attname
                                        FROM pg_class, pg_attribute

data/test/dummy/app/models/locomotive.rb

@@ -1,3 +1,2 @@
 class Locomotive < ActiveRecord::Base
-  abstract_class = true
 end

data/test/dummy/app/models/vehicle.rb

@@ -1,4 +1,3 @@
 class Vehicle < ActiveRecord::Base
-  abstract_class = true
   self.inheritance_column = :vehicle_type
-end
+end

data/test/dummy/config/environments/test.rb

@@ -8,8 +8,8 @@ Dummy::Application.configure do
   config.cache_classes = true
 
   # Configure static asset server for tests with Cache-Control for performance
-  config.serve_static_files = true
-  config.static_cache_control = "public, max-age=3600"
+  config.public_file_server.enabled = true
+  config.public_file_server.headers = { 'Cache-Control' => 'public, max-age=3600' }
 
   # Show full error reports and disable caching
   config.consider_all_requests_local       = true

data/test/fixtures/migrations/1_add_updateable_views_inheritance.rb

@@ -1,5 +1,5 @@
-class AddUpdateableViewsInheritance < ActiveRecord::Migration
-  def self.up
+class AddUpdateableViewsInheritance < ActiveRecord::Migration[4.2]
+  def up
     create_table(:updateable_views_inheritance, :id => false) do |t|
       t.column :parent_relation, :string
       t.column :child_aggregate_view, :string

data/test/fixtures/migrations/2_create_with_default_table.rb

@@ -1,5 +1,5 @@
-class CreateWithDefaultTable < ActiveRecord::Migration
-  def self.up
+class CreateWithDefaultTable < ActiveRecord::Migration[4.2]
+  def up
     create_table :locomotives do |t|
       t.column :name, :string
       t.column :max_speed, :integer

data/test/fixtures/migrations/3_create_with_explicit_table.rb

@@ -1,11 +1,11 @@
-class CreateWithExplicitTable < ActiveRecord::Migration
-  def self.up
+class CreateWithExplicitTable < ActiveRecord::Migration[4.2]
+  def up
     create_child(:electric_locomotives, :table => :raw_electric_locomotives, :parent => :locomotives)  do |t|
       t.decimal :electricity_consumption, :precision => 6, :scale => 2
     end
   end
-  
+
   def self.down
     drop_child  :electric_locomotives
   end
-end
+end

data/test/fixtures/migrations/4_create_deeper_hierarchy.rb

@@ -1,10 +1,10 @@
-class CreateDeeperHierarchy < ActiveRecord::Migration
-  def self.up
+class CreateDeeperHierarchy < ActiveRecord::Migration[4.2]
+  def up
     create_child(:maglev_locomotives, :parent => :electric_locomotives)  do |t|
       t.column :magnetic_field, :integer
     end
   end
-  
+
   def self.down
     drop_child :maglev_locomotives
   end

data/test/fixtures/migrations/5_default_column_values.rb

@@ -1,13 +1,13 @@
-class DefaultColumnValues < ActiveRecord::Migration
-  def self.up
+class DefaultColumnValues < ActiveRecord::Migration[4.2]
+  def up
     create_child(:rack_locomotives, :parent => :locomotives) do |t|
       t.column :bidirectional, :boolean, :default => false
       t.column :narrow_gauge, :boolean, :default => true
       t.column :rail_system, :string, :default => 'Abt'
     end
   end
-  
+
   def self.down
     drop_child  :rack_locomotives
   end
-end
+end

data/test/fixtures/migrations/6_single_table_inheritance_view.rb

@@ -1,5 +1,5 @@
-class SingleTableInheritanceView < ActiveRecord::Migration
-  def self.up
+class SingleTableInheritanceView < ActiveRecord::Migration[4.2]
+  def up
     rebuild_parent_and_children_views(:locomotives)
     create_single_table_inheritance_view(:all_locomotives,:locomotives)
   end
@@ -7,4 +7,4 @@ class SingleTableInheritanceView < ActiveRecord::Migration
   def self.down
     drop_view(:all_locomotives)
   end
-end
+end

data/test/fixtures/migrations/7_second_deep_hierarchy.rb

@@ -1,4 +1,4 @@
-class SecondDeepHierarchy < ActiveRecord::Migration
+class SecondDeepHierarchy < ActiveRecord::Migration[4.2]
   # Create tables and views for the following inheritance hierarchy:
   #
   #                               vehicles
@@ -13,37 +13,37 @@ class SecondDeepHierarchy < ActiveRecord::Migration
   #             steam_trains    rack_trains     electric_trains
   #                                                   |
   #                                               maglev_trains
-  def self.up
+  def up
     create_table :vehicles do |t|
       t.column :name, :string
       t.column :vehicle_type, :string
     end
-    
+
     create_child(:wheeled_vehicles, :parent => :vehicles) do |t|
       t.column :number_of_wheels, :integer
     end
-    
+
     create_child(:bicycles, :parent => :wheeled_vehicles) do |t|
       t.column :number_of_gears, :integer
     end
-    
+
     create_child(:cars, :parent => :wheeled_vehicles) do |t|
       t.column :number_of_doors, :integer
     end
-    
+
     create_child(:railed_vehicles, :parent => :vehicles) do |t|
       t.column :number_of_rails, :integer
     end
-    
+
     create_child(:trains, :parent => :railed_vehicles) do |t|
       t.column :max_speed, :integer
     end
-  
+
     create_child(:steam_trains, :parent => :trains) do |t|
       t.column :water_consumption, :decimal, :precision => 6, :scale => 2
       t.column :coal_consumption,  :decimal, :precision => 6, :scale => 2
     end
-    
+
     create_child(:rack_trains, :parent => :trains) do |t|
       t.column :bidirectional, :boolean, :default => false
       t.column :narrow_gauge, :boolean, :default => true
@@ -53,7 +53,7 @@ class SecondDeepHierarchy < ActiveRecord::Migration
     create_child(:electric_trains, :parent => :trains) do |t|
       t.column :electricity_consumption, :decimal, :precision => 6, :scale => 2
     end
-    
+
     create_child(:maglev_trains, :parent => :electric_trains) do |t|
       t.column :magnetic_field, :integer
     end
@@ -62,9 +62,9 @@ class SecondDeepHierarchy < ActiveRecord::Migration
       t.column :mast_number, :integer
     end
   end
-  
+
   def self.down
     %w(boats maglev_trains electric_trains steam_trains rack_trains trains railed_vehicles cars bicycles wheeled_vehicles).each{|child| drop_child(child)}
     drop_table :vehicles
   end
-end
+end

data/test/fixtures/migrations/8_second_single_table_inheritance_view.rb

@@ -1,5 +1,5 @@
-class SecondSingleTableInheritanceView < ActiveRecord::Migration
-  def self.up
+class SecondSingleTableInheritanceView < ActiveRecord::Migration[4.2]
+  def up
     rebuild_parent_and_children_views(:vehicles)
     create_single_table_inheritance_view(:all_vehicles,:vehicles)
   end
@@ -7,4 +7,4 @@ class SecondSingleTableInheritanceView < ActiveRecord::Migration
   def self.down
     drop_view(:all_vehicles)
   end
-end
+end

data/test/instantiation_test.rb

@@ -3,11 +3,12 @@ require_relative 'test_helper'
 class InstantiationTest < ActiveSupport::TestCase
   def setup
     ActiveRecord::Migrator.up(File.dirname(__FILE__) + '/fixtures/migrations/', 7)
+
+    ActiveRecord::FixtureSet.reset_cache
     # order of fixtures is important for the test - last loaded should not be with max(id)
     %w[steam_locomotives electric_locomotives maglev_locomotives bicycles].each do |f|
       ActiveRecord::FixtureSet.create_fixtures(File.dirname(__FILE__) + '/fixtures/', f)
     end
-    @connection = ActiveRecord::Base.connection
 
     Locomotive.disable_inheritance_instantiation = true
     ElectricLocomotive.disable_inheritance_instantiation = false
@@ -15,7 +16,6 @@ class InstantiationTest < ActiveSupport::TestCase
 
   def teardown
     Locomotive.disable_inheritance_instantiation = false
-    ActiveRecord::FixtureSet.reset_cache
   end
 
   # FIXME: flaky test_setting_disable_inheritance_instantiation_does_not_load_child_columns

data/test/migration_test.rb

@@ -22,7 +22,8 @@ class UpdateableViewsInheritanceMigrationTest < ActiveSupport::TestCase
     ActiveRecord::Migrator.up(File.dirname(__FILE__) + '/fixtures/migrations/', 3)
     ActiveRecord::Migrator.down(File.dirname(__FILE__) + '/fixtures/migrations/', 2)
     assert_equal %w(steam_locomotives), @connection.views.sort
-    assert_equal %w(locomotives
+    assert_equal %w(ar_internal_metadata
+                    locomotives
                     schema_migrations
                     steam_locomotives_data
                     updateable_views_inheritance), @connection.tables.sort

data/test/pg_insert_returning_with_rules_test.rb

@@ -0,0 +1,60 @@
+require_relative 'test_helper'
+
+class InsertReturningOnViewWithRulesAndDefaultValue < ActiveSupport::TestCase
+  def setup
+    @conn = ActiveRecord::Base.connection.raw_connection
+    @conn.exec("SET client_min_messages TO 'ERROR'")
+
+    @conn.exec(<<-SQL.squish)
+      CREATE TABLE parent (
+        id SERIAL PRIMARY KEY,
+        name TEXT
+      )
+    SQL
+
+    @conn.exec(<<-SQL.squish)
+      CREATE TABLE child (
+        parent_id INTEGER PRIMARY KEY REFERENCES parent(id),
+        surname TEXT
+      )
+    SQL
+
+    @conn.exec(<<-SQL.squish)
+      CREATE VIEW v AS (
+        SELECT id, name, surname
+        FROM parent JOIN child ON parent.id=child.parent_id
+      )
+    SQL
+
+    @conn.exec(<<-SQL.squish)
+     ALTER VIEW v ALTER id SET DEFAULT nextval('parent_id_seq'::regclass)
+    SQL
+
+    @conn.exec(<<-SQL.squish)
+      CREATE RULE v_on_insert AS ON INSERT TO v DO INSTEAD (
+        INSERT INTO parent (id, name)
+          VALUES( DEFAULT, NEW.name );
+        INSERT INTO child  (parent_id, surname)
+          VALUES( currval('parent_id_seq'), NEW.surname ) RETURNING parent_id, NULL::text, NULL::text;
+      )
+    SQL
+
+    @sql = "INSERT INTO v (name, surname) VALUES ('parent', 'child') RETURNING id"
+  end
+
+  def teardown
+    @conn.exec("DROP VIEW IF EXISTS v")
+    @conn.exec("DROP TABLE IF EXISTS parent CASCADE")
+    @conn.exec("DROP TABLE IF EXISTS child CASCADE")
+  end
+
+  def test_async_exec_with_empty_binds
+    res = @conn.async_exec(@sql, [])
+    assert_equal [[1]], res.values
+  end
+
+  def test_async_exec_with_no_binds
+    res = @conn.async_exec(@sql)
+    assert_equal [[1]], res.values
+  end
+end

data/test/schema_test.rb

@@ -12,8 +12,8 @@ class SchemaTest < ActiveSupport::TestCase
     assert_equal 'public.locomotives_id_seq', seq.to_s
   end
 
-  class CreateChildInSchemaWithPublicParent < ActiveRecord::Migration
-    def self.up
+  class CreateChildInSchemaWithPublicParent < ActiveRecord::Migration[4.2]
+    def up
       execute "CREATE SCHEMA interrail"
       create_child('interrail.steam_locomotives', parent: 'locomotives') do |t|
         t.decimal :interrail_water_consumption, precision: 6, scale: 2
@@ -23,14 +23,14 @@ class SchemaTest < ActiveSupport::TestCase
   end
 
   def test_pk_and_sequence_for_child_and_parent_in_different_schemas
-    CreateChildInSchemaWithPublicParent.up
+    CreateChildInSchemaWithPublicParent.new.up
     pk, seq = @connection.pk_and_sequence_for('interrail.steam_locomotives')
     assert_equal 'id', pk
     assert_equal 'public.locomotives_id_seq', seq.to_s
   end
 
-  class CreateChildInSchemaWithParentInSchema < ActiveRecord::Migration
-    def self.up
+  class CreateChildInSchemaWithParentInSchema < ActiveRecord::Migration[4.2]
+    def up
       execute "CREATE SCHEMA interrail"
       create_table 'interrail.locomotives' do |t|
         t.column :interrail_name, :string
@@ -45,7 +45,7 @@ class SchemaTest < ActiveSupport::TestCase
   end
 
   def test_pk_and_sequence_for_child_and_parent_in_same_nonpublic_schema
-    CreateChildInSchemaWithParentInSchema.up
+    CreateChildInSchemaWithParentInSchema.new.up
     pk, seq = @connection.pk_and_sequence_for('interrail.steam_locomotives')
     assert_equal 'id', pk
     assert_equal 'interrail.locomotives_id_seq', seq.to_s
@@ -64,8 +64,8 @@ class SchemaTest < ActiveSupport::TestCase
                  @connection.views.sort
   end
 
-  class ParentTableWithOnlyOneColumn < ActiveRecord::Migration
-    def self.up
+  class ParentTableWithOnlyOneColumn < ActiveRecord::Migration[4.2]
+    def up
       create_table(:parent_pk_only){}
       create_table :child_data do |t|
         t.column :name, :string
@@ -80,15 +80,15 @@ class SchemaTest < ActiveSupport::TestCase
   end
 
   def test_parent_table_with_only_one_column
-    ParentTableWithOnlyOneColumn.up
+    ParentTableWithOnlyOneColumn.new.up
     assert @connection.views.include?('child')
     assert_equal %w(id name), @connection.columns(:child).map{|c| c.name}.sort
   ensure
     ParentTableWithOnlyOneColumn.down
   end
 
-  class ChildTableWithOnlyOneColumn < ActiveRecord::Migration
-    def self.up
+  class ChildTableWithOnlyOneColumn < ActiveRecord::Migration[4.2]
+    def up
       create_table :parent do |t|
         t.column :name, :string
       end
@@ -98,7 +98,7 @@ class SchemaTest < ActiveSupport::TestCase
   end
 
   def test_child_table_with_only_one_column
-    ChildTableWithOnlyOneColumn.up
+    ChildTableWithOnlyOneColumn.new.up
     assert @connection.views.include?('child_pk_only'), "Could not create child view when child table has only one column"
     assert_equal %w(id name), @connection.columns(:child_pk_only).map{|c| c.name}.sort
   end
@@ -132,8 +132,8 @@ class SchemaTest < ActiveSupport::TestCase
     assert SteamLocomotive.columns.find { |c| c.name == 'water_consumption' }.null
   end
 
-  class ChangeDefaultValueOfColumn < ActiveRecord::Migration
-    def self.up
+  class ChangeDefaultValueOfColumn < ActiveRecord::Migration[4.2]
+    def up
       remove_parent_and_children_views(:rack_locomotives)
       change_column_default(:rack_locomotives_data, :rail_system, 'Marsh')
       rebuild_parent_and_children_views(:rack_locomotives)
@@ -141,24 +141,24 @@ class SchemaTest < ActiveSupport::TestCase
   end
 
   def test_change_default_value_of_column
-    ChangeDefaultValueOfColumn.up
+    ChangeDefaultValueOfColumn.new.up
     RackLocomotive.reset_column_information
     assert_equal 'Marsh', RackLocomotive.new.rail_system
   end
 
-  class RemoveChildrenViews < ActiveRecord::Migration
-    def self.up
+  class RemoveChildrenViews < ActiveRecord::Migration[4.2]
+    def up
       remove_parent_and_children_views(:locomotives)
     end
   end
 
   def test_remove_parent_and_children_views
-    RemoveChildrenViews.up
+    RemoveChildrenViews.new.up
     assert @connection.views.empty?
   end
 
-  class RemoveColumnInParentTable < ActiveRecord::Migration
-    def self.up
+  class RemoveColumnInParentTable < ActiveRecord::Migration[4.2]
+    def up
       remove_parent_and_children_views(:locomotives)
       remove_column(:locomotives, :max_speed)
       rebuild_parent_and_children_views(:locomotives)
@@ -166,15 +166,15 @@ class SchemaTest < ActiveSupport::TestCase
   end
 
   def test_remove_column_parent_table
-    RemoveColumnInParentTable.up
+    RemoveColumnInParentTable.new.up
     assert_equal %w(coal_consumption id name type water_consumption),
                  @connection.columns(:steam_locomotives).map{ |c| c.name }.sort
     assert_equal %w(electricity_consumption id magnetic_field name type),
                  @connection.columns(:maglev_locomotives).map{ |c| c.name }.sort
   end
 
-  class RenameColumnInParentTable < ActiveRecord::Migration
-    def self.up
+  class RenameColumnInParentTable < ActiveRecord::Migration[4.2]
+    def up
       ActiveRecord::FixtureSet.create_fixtures(File.dirname(__FILE__) + '/fixtures/', :electric_locomotives)
       ActiveRecord::FixtureSet.create_fixtures(File.dirname(__FILE__) + '/fixtures/', :maglev_locomotives)
       ActiveRecord::FixtureSet.create_fixtures(File.dirname(__FILE__) + '/fixtures/', :steam_locomotives)
@@ -186,7 +186,7 @@ class SchemaTest < ActiveSupport::TestCase
   end
 
   def test_rename_column_parent_table
-    RenameColumnInParentTable.up
+    RenameColumnInParentTable.new.up
     assert_equal %w(coal_consumption id maximal_speed name type water_consumption),
                  @connection.columns(:steam_locomotives).map{ |c| c.name }.sort
     assert_equal %w(electricity_consumption id magnetic_field maximal_speed name type),
@@ -194,15 +194,15 @@ class SchemaTest < ActiveSupport::TestCase
 
   end
 
-  class AddColumnToParentTable < ActiveRecord::Migration
-    def self.up
+  class AddColumnToParentTable < ActiveRecord::Migration[4.2]
+    def up
       add_column(:raw_electric_locomotives, :number_of_engines, :integer)
       rebuild_parent_and_children_views(:electric_locomotives)
     end
   end
 
   def test_add_column_to_parent_table
-    AddColumnToParentTable.up
+    AddColumnToParentTable.new.up
     assert_equal %w(electricity_consumption id max_speed name number_of_engines type),
                  @connection.columns(:electric_locomotives).map{ |c| c.name }.sort
     assert_equal %w(electricity_consumption id magnetic_field max_speed name number_of_engines type),
@@ -210,8 +210,8 @@ class SchemaTest < ActiveSupport::TestCase
 
   end
 
-  class ChangeChildRelationView < ActiveRecord::Migration
-    def self.up
+  class ChangeChildRelationView < ActiveRecord::Migration[4.2]
+    def up
       remove_parent_and_children_views(:electric_locomotives)
       rename_column(:raw_electric_locomotives, :electricity_consumption, :electric_consumption)
       rebuild_parent_and_children_views(:electric_locomotives)
@@ -219,13 +219,13 @@ class SchemaTest < ActiveSupport::TestCase
   end
 
   def test_change_child_relation_view
-    ChangeChildRelationView.up
+    ChangeChildRelationView.new.up
     assert_equal %w(electric_consumption id max_speed name type),
                  @connection.columns(:electric_locomotives).map{ |c| c.name }.sort
   end
 
-  class ChangeColumnInChildTable < ActiveRecord::Migration
-    def self.up
+  class ChangeColumnInChildTable < ActiveRecord::Migration[4.2]
+    def up
       drop_view(:steam_locomotives)
       rename_column(:steam_locomotives_data, :coal_consumption, :fuel_consumption)
       create_child_view(:locomotives, :steam_locomotives)
@@ -233,17 +233,13 @@ class SchemaTest < ActiveSupport::TestCase
   end
 
   def test_change_column_in_child_table
-    ChangeColumnInChildTable.up
+    ChangeColumnInChildTable.new.up
     assert_equal %w(fuel_consumption id max_speed name type water_consumption),
                  @connection.columns(:steam_locomotives).map(&:name).sort
   end
 
-  def test_table_exists
-    #TODO: test table_exists? monkey patch
-  end
-
-  class CreateChildInSchema < ActiveRecord::Migration
-    def self.up
+  class CreateChildInSchema < ActiveRecord::Migration[4.2]
+    def up
       execute "CREATE SCHEMA interrail"
       create_table 'interrail.locomotives' do |t|
         t.column :interrail_name, :string
@@ -258,7 +254,7 @@ class SchemaTest < ActiveSupport::TestCase
   end
 
   def test_create_child_in_schema
-    CreateChildInSchema.up
+    CreateChildInSchema.new.up
     assert_equal %w[id
                     interrail_coal_consumption
                     interrail_max_speed
@@ -268,8 +264,8 @@ class SchemaTest < ActiveSupport::TestCase
                  @connection.columns('interrail.steam_locomotives').map(&:name).sort
   end
 
-  class ChangeTablesInTwoInheritanceChains < ActiveRecord::Migration
-    def self.up
+  class ChangeTablesInTwoInheritanceChains < ActiveRecord::Migration[4.2]
+    def up
       add_column(:maglev_locomotives_data, :levitation_height, :integer)
       add_column(:bicycles_data, :wheel_size, :integer)
       rebuild_all_parent_and_children_views
@@ -279,7 +275,7 @@ class SchemaTest < ActiveSupport::TestCase
   def test_rebuild_all_parent_and_children_views
     ActiveRecord::Migrator.up(File.dirname(__FILE__) + '/fixtures/migrations/', 7)
     @connection.execute "DROP VIEW all_locomotives" #FIXME: single table inheritance view should be rebuilt as well
-    ChangeTablesInTwoInheritanceChains.up
+    ChangeTablesInTwoInheritanceChains.new.up
 
     assert @connection.columns(:maglev_locomotives).map{ |c| c.name }.include?('levitation_height'),
            "Newly added column not present in view after rebuild for 1. hierarchy"
@@ -287,8 +283,8 @@ class SchemaTest < ActiveSupport::TestCase
            "Newly added column not present in view after rebuild for 2. hierarchy"
   end
 
-  class UseExistingTable < ActiveRecord::Migration
-    def self.up
+  class UseExistingTable < ActiveRecord::Migration[4.2]
+    def up
       create_table :tbl_diesel_locomotives do |t|
         t.belongs_to :locomotives
         t.integer :num_cylinders
@@ -301,12 +297,12 @@ class SchemaTest < ActiveSupport::TestCase
   end
 
   def test_skip_creating_child_table
-    UseExistingTable.up
+    UseExistingTable.new.up
     assert @connection.columns(:diesel_locomotives).map(&:name).include?("num_cylinders")
   end
 
-  class ReservedSQLWords < ActiveRecord::Migration
-    def self.up
+  class ReservedSQLWords < ActiveRecord::Migration[4.2]
+    def up
       create_child(:table, parent: :locomotives) do |t|
         t.integer :column
       end
@@ -317,14 +313,14 @@ class SchemaTest < ActiveSupport::TestCase
   end
 
   def test_reserved_words_in_tables_and_columns
-    ReservedSQLWords.up
+    ReservedSQLWords.new.up
     assert @connection.columns(:table).map(&:name).include?("column")
   ensure
     ReservedSQLWords.down
   end
 
-  class ChildTableIsActuallyView < ActiveRecord::Migration
-    def self.up
+  class ChildTableIsActuallyView < ActiveRecord::Migration[4.2]
+    def up
       execute <<-SQL.squish
         CREATE VIEW punk_locomotives_data AS (
           SELECT steam_locomotives.id,
@@ -346,7 +342,7 @@ class SchemaTest < ActiveSupport::TestCase
   end
 
   def test_child_table_is_view
-    ChildTableIsActuallyView.up
+    ChildTableIsActuallyView.new.up
     assert_equal @connection.columns(:punk_locomotives).map(&:name).sort,
                  %w(coal electro id max_speed name type)
   end

data/test/single_table_inheritance_test.rb

@@ -30,8 +30,8 @@ class SingleTableInheritanceAggregateViewTest < ActiveSupport::TestCase
     assert_equal electric_locomotive.name, @connection.query("SELECT name FROm all_locomotives WHERE id=#{electric_locomotive.id}").first.first
   end
 
-  class AddColumnToParentTable < ActiveRecord::Migration
-    def self.up
+  class AddColumnToParentTable < ActiveRecord::Migration[4.2]
+    def up
       add_column(:raw_electric_locomotives, :number_of_engines, :integer)
       drop_view(:all_locomotives)
       rebuild_parent_and_children_views(:electric_locomotives)
@@ -40,13 +40,13 @@ class SingleTableInheritanceAggregateViewTest < ActiveSupport::TestCase
   end
 
   def test_single_table_inheritance_view_add_column_to_parent_table
-    AddColumnToParentTable.up
+    AddColumnToParentTable.new.up
     assert_equal %w(coal_consumption id max_speed name type water_consumption electricity_consumption bidirectional narrow_gauge magnetic_field rail_system number_of_engines).sort,
                  @connection.columns(:all_locomotives).map{ |c| c.name }.sort
   end
 
-  class RemoveColumnInParentTable < ActiveRecord::Migration
-    def self.up
+  class RemoveColumnInParentTable < ActiveRecord::Migration[4.2]
+    def up
       drop_view(:all_locomotives)
       remove_parent_and_children_views(:locomotives)
       remove_column(:locomotives, :max_speed)
@@ -56,13 +56,13 @@ class SingleTableInheritanceAggregateViewTest < ActiveSupport::TestCase
   end
 
   def test_single_table_inheritance_view_remove_column_parent_table
-    RemoveColumnInParentTable.up
+    RemoveColumnInParentTable.new.up
     assert_equal %w(coal_consumption id name type water_consumption electricity_consumption bidirectional narrow_gauge magnetic_field rail_system).sort,
                  @connection.columns(:all_locomotives).map{ |c| c.name }.sort
   end
 
-  class RenameColumnInParentTable < ActiveRecord::Migration
-    def self.up
+  class RenameColumnInParentTable < ActiveRecord::Migration[4.2]
+    def up
       drop_view(:all_locomotives)
       remove_parent_and_children_views(:locomotives)
       rename_column(:locomotives, :max_speed, :maximal_speed)
@@ -72,13 +72,13 @@ class SingleTableInheritanceAggregateViewTest < ActiveSupport::TestCase
   end
 
   def test_single_table_inheritance_view_rename_column_parent_table
-    RenameColumnInParentTable.up
+    RenameColumnInParentTable.new.up
     assert_equal %w(coal_consumption id maximal_speed name type water_consumption electricity_consumption bidirectional narrow_gauge magnetic_field rail_system).sort,
                  @connection.columns(:all_locomotives).map{ |c| c.name }.sort
   end
 
-  class ChangeChildRelationView < ActiveRecord::Migration
-    def self.up
+  class ChangeChildRelationView < ActiveRecord::Migration[4.2]
+    def up
       drop_view(:all_locomotives)
       remove_parent_and_children_views(:electric_locomotives)
       rename_column(:raw_electric_locomotives, :electricity_consumption, :electric_consumption)
@@ -88,13 +88,13 @@ class SingleTableInheritanceAggregateViewTest < ActiveSupport::TestCase
   end
 
   def test_single_table_inheritance_view_change_child_relation_view
-    ChangeChildRelationView.up
+    ChangeChildRelationView.new.up
     assert_equal %w(coal_consumption id max_speed name type water_consumption electric_consumption bidirectional narrow_gauge magnetic_field rail_system).sort,
                  @connection.columns(:all_locomotives).map{ |c| c.name }.sort
   end
 
-  class ConflictColumns < ActiveRecord::Migration
-    def self.up
+  class ConflictColumns < ActiveRecord::Migration[4.2]
+    def up
       drop_view(:all_locomotives)
       add_column(:raw_electric_locomotives, :number_of_engines, :integer)
       add_column(:steam_locomotives_data, :number_of_engines, :string)
@@ -105,14 +105,14 @@ class SingleTableInheritanceAggregateViewTest < ActiveSupport::TestCase
   end
 
   def test_single_table_inheritance_view_conflict_columns
-    ConflictColumns.up
+    ConflictColumns.new.up
     assert_equal %w(coal_consumption id max_speed name type water_consumption electricity_consumption bidirectional narrow_gauge magnetic_field rail_system number_of_engines).sort,
                  @connection.columns(:all_locomotives).map{ |c| c.name }.sort
     assert_equal 'text', @connection.columns(:all_locomotives).detect{|c| c.name == "number_of_engines"}.sql_type
   end
 
-  class ConflictColumnsWithValues < ActiveRecord::Migration
-    def self.up
+  class ConflictColumnsWithValues < ActiveRecord::Migration[4.2]
+    def up
       add_column(:raw_electric_locomotives, :number_of_engines, :integer)
       add_column(:steam_locomotives_data, :number_of_engines, :string)
       execute("UPDATE raw_electric_locomotives SET number_of_engines = 2")
@@ -126,7 +126,7 @@ class SingleTableInheritanceAggregateViewTest < ActiveSupport::TestCase
 
   # FIXME: flaky test_single_table_inheritance_view_conflict_columns_with_values
   # def test_single_table_inheritance_view_conflict_columns_with_values
-  #   ConflictColumnsWithValues.up
+  #   ConflictColumnsWithValues.new.up
   #   ::SteamLocomotive.reset_column_information
   #   ::ElectricLocomotive.reset_column_information
   #   assert_equal %w(coal_consumption id max_speed name type water_consumption electricity_consumption bidirectional narrow_gauge magnetic_field rail_system number_of_engines).sort,

data/updateable_views_inheritance.gemspec

@@ -18,14 +18,14 @@ Gem::Specification.new do |s|
   s.test_files    = s.files.grep(%r{^(test|spec|features)/})
   s.require_paths = ["lib"]
 
-  s.add_dependency "activerecord", "~> 4.2.8"
-  s.add_dependency "pg", "~> 0.21"
+  s.add_dependency "activerecord", "~> 5.1.7"
+  s.add_dependency "pg"
 
   s.add_development_dependency 'bigdecimal', '1.3.5'
   s.add_development_dependency "bundler"
   s.add_development_dependency "minitest"
   s.add_development_dependency "minitest-reporters"
-  s.add_development_dependency "rails", '= 4.2.11.1'
+  s.add_development_dependency "rails", '~> 5.1.7'
   s.add_development_dependency "rake"
   s.add_development_dependency "simplecov"
   s.add_development_dependency "simplecov_json_formatter"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: updateable_views_inheritance
 version: !ruby/object:Gem::Version
-  version: 1.4.8
+  version: 1.5.1
 platform: ruby
 authors:
 - Sava Chankov
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-17 00:00:00.000000000 Z
+date: 2025-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -17,28 +17,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 4.2.8
+        version: 5.1.7
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 4.2.8
+        version: 5.1.7
 - !ruby/object:Gem::Dependency
   name: pg
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.21'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.21'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: bigdecimal
   requirement: !ruby/object:Gem::Requirement
@@ -99,16 +99,16 @@ dependencies:
   name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 4.2.11.1
+        version: 5.1.7
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 4.2.11.1
+        version: 5.1.7
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -187,7 +187,7 @@ files:
 - lib/generators/updateable_views_inheritance/install_generator.rb
 - lib/generators/updateable_views_inheritance/templates/create_updateable_views_inheritance.rb
 - lib/updateable_views_inheritance.rb
-- lib/updateable_views_inheritance/active_record.rb
+- lib/updateable_views_inheritance/active_record_extensions.rb
 - lib/updateable_views_inheritance/postgresql_adapter.rb
 - lib/updateable_views_inheritance/version.rb
 - tasks/updateable_views_inheritance_tasks.rake
@@ -261,7 +261,7 @@ files:
 - test/install_generator_test.rb
 - test/instantiation_test.rb
 - test/migration_test.rb
-- test/pg_insert_returning_with_rules_spec.rb
+- test/pg_insert_returning_with_rules_test.rb
 - test/schema_test.rb
 - test/single_table_inheritance_test.rb
 - test/test_helper.rb
@@ -360,7 +360,7 @@ test_files:
 - test/install_generator_test.rb
 - test/instantiation_test.rb
 - test/migration_test.rb
-- test/pg_insert_returning_with_rules_spec.rb
+- test/pg_insert_returning_with_rules_test.rb
 - test/schema_test.rb
 - test/single_table_inheritance_test.rb
 - test/test_helper.rb

data/lib/updateable_views_inheritance/active_record.rb

@@ -1,18 +0,0 @@
-module ActiveRecord #:nodoc:
-  class Base #:nodoc:
-    class << self
-      attr_accessor :disable_inheritance_instantiation
-
-      private
-      def instantiate_with_updateable_views_inheritance_support(attributes, column_types = {})
-        object = instantiate_without_updateable_views_inheritance_support(attributes, column_types = {})
-        if object.class.name == self.name || self.disable_inheritance_instantiation
-          object
-        else
-          object.class.find(attributes.with_indifferent_access[:id])
-        end
-      end
-      alias_method_chain :instantiate, :updateable_views_inheritance_support
-    end
-  end
-end

data/test/pg_insert_returning_with_rules_spec.rb

@@ -1,58 +0,0 @@
-gem "pg", "0.18.2"
-
-require "pg"
-require "rspec"
-
-describe "Insert returning on view with rules and default value" do
-  before(:each) do
-    @conn = PG.connect(dbname: 'updateable_views_inheritance_test')
-    @conn.exec(%q{ SET client_min_messages TO 'ERROR' })
-
-    @conn.exec(%q{ CREATE TABLE parent ( id SERIAL PRIMARY KEY,
-                                         name TEXT) })
-    @conn.exec(%q{ CREATE TABLE child ( parent_id INTEGER PRIMARY KEY REFERENCES parent(id),
-                                        surname TEXT) })
-
-    @conn.exec(%q{ CREATE VIEW v AS (SELECT id, name, surname FROM parent JOIN child ON parent.id=child.parent_id) })
-    @conn.exec(%q{ ALTER VIEW v ALTER id SET DEFAULT nextval('parent_id_seq'::regclass) })
-    #
-    # The old way that didn't return anything when binds are empty
-    #
-    # @conn.exec(%q{ CREATE RULE v_on_insert AS ON INSERT TO v DO INSTEAD
-    #               (
-    #                 SELECT setval('parent_id_seq', NEW.id);
-    #                 INSERT INTO parent (id, name)
-    #                   VALUES( currval('parent_id_seq'), NEW.name ) RETURNING id, name, NULL::text;
-    #                 INSERT INTO child  (parent_id, surname)
-    #                   VALUES( currval('parent_id_seq'), NEW.surname );
-    #
-    #               )
-    #             })
-    @conn.exec(%q{ CREATE RULE v_on_insert AS ON INSERT TO v DO INSTEAD
-                  (
-                    INSERT INTO parent (id, name)
-                      VALUES( DEFAULT, NEW.name );
-                    INSERT INTO child  (parent_id, surname)
-                      VALUES( currval('parent_id_seq'), NEW.surname ) RETURNING parent_id, NULL::text, NULL::te;
-                  )
-                })
-
-    @sql = %q{ INSERT INTO v (name, surname) VALUES ('parent', 'child') RETURNING id}
-  end
-
-  after(:each) do
-    @conn.exec(%q{ DROP VIEW IF EXISTS v })
-    @conn.exec(%q{ DROP TABLE IF EXISTS parent CASCADE})
-    @conn.exec(%q{ DROP TABLE IF EXISTS child CASCADE})
-  end
-
-  it 'async exec with empty binds' do
-    res = @conn.async_exec(@sql, [])
-    expect(res.values).to eq([["1"]])
-  end
-
-  it 'async exec with no binds' do
-    res = @conn.async_exec(@sql)
-    expect(res.values).to eq([["1"]])
-  end
-end