pg_ltree 1.1.8 → 1.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: af73780dad2653e9068d80d56ad53a1cac86f88968cbe9f697bc832e42aedcf6
-  data.tar.gz: a4bea89ea088fe18ce1a5d5ce9606bceecb1833a36b9ec4ce7b252bcde5e7f83
+  metadata.gz: ba0cf379067eb0d7c24ff36828795d78b6cbbdc5b6738c45427868164940516d
+  data.tar.gz: c63501e14473075c2916567fdee20d974c09fe77d93c0553b9d16c185af436a1
 SHA512:
-  metadata.gz: '086bcf6439fdfd185d738ab1c120c41192567f8d3e9c1ee456c0ef82bf28d0e6c921d9c9a397ff5cacd7735dfc21605c078ade8927266e6685ef7fefbc1d0b58'
-  data.tar.gz: 9317b961bd9ae4f6ff781c3aa72bf2b5ee0c5136bb6b2a28482853a8c0262278f5a00daf6d8b4dfd71302d4e2b52975abc8e4f2b0629fffdb03037d95ca51ced
+  metadata.gz: 8c8545986852ec4d61a2b9623a2142d4f79e2a3613d75fb7efff6511e70699531831527a625b89693b4b9d21c8bc7e4937f432bb14315ea82c01cc4a6eb34079
+  data.tar.gz: 1be5c7b2a06dc1228ba07ab50284f5d6af6147a27bbec92fb6768b92d7f718399dacf6150532ea2c1939a5660cd62e7f650463d7f534301df5cde8b29d7e9fb6

data/Rakefile

@@ -20,32 +20,3 @@ Rake::TestTask.new(:test) do |t|
   t.verbose = false
 end
 
-desc 'Default: run unit tests.'
-task default: 'test:all'
-
-namespace :test do
-  AVAILABLE_CASES = %w(
-    activerecord_40_pg_017 activerecord_40_pg_018
-    activerecord_41_pg_017 activerecord_41_pg_018
-    activerecord_42_pg_017 activerecord_42_pg_018
-    activerecord_50_pg_018 activerecord_51_pg_020
-    activerecord_51_pg_021 activerecord_52_pg_100
-    activerecord_60_pg_021 activerecord_60_pg_100
-    activerecord_60_pg_110
-  ).freeze
-
-  AVAILABLE_CASES.each do |version|
-    desc "Test pg_ltree against #{version}"
-    task version do
-      sh "bundle install --gemfile=gemfiles/#{version}.gemfile --quiet"
-      sh "BUNDLE_GEMFILE='gemfiles/#{version}.gemfile' bundle exec rake -t test"
-    end
-  end
-
-  desc 'Run all tests for pg_ltree'
-  task :all do
-    AVAILABLE_CASES.each do |version|
-      sh "rake test:#{version}"
-    end
-  end
-end

data/lib/pg_ltree/ltree.rb

@@ -3,18 +3,16 @@ module PgLtree
   #
   # @see [ActiveRecord::Base]
   # @see http://www.postgresql.org/docs/current/static/ltree.html
-  #
-  # @author a.ponomarenko
   module Ltree
     # Initialzie ltree for active model
     #
     # @param column [String] ltree column name
-    def ltree(column = :path, options: { cascade: true })
+    def ltree(column = :path, cascade: true)
       cattr_accessor :ltree_path_column
 
       self.ltree_path_column = column
 
-      if options[:cascade]
+      if cascade
         after_update :cascade_update
         after_destroy :cascade_destroy
       end
@@ -245,7 +243,7 @@ module PgLtree
       #
       # @return [ActiveRecord::Relation]
       def cascade_destroy
-        ltree_scope.where("#{ltree_scope.table_name}.#{ltree_path_column} <@ ?", ltree_path_in_database).delete_all
+        ltree_scope.where("#{ltree_scope.table_name}.#{ltree_path_column} <@ ?", ltree_path_in_database).destroy_all
       end
     end
   end

data/lib/pg_ltree/scoped_for.rb

@@ -1,8 +1,6 @@
 module PgLtree
   # Narrowing the Scope for ActiveRecord Model
   # @note When model have composite uniq key (for example: state + path), you should use this module for narrowing the scope
-  #
-  # @author a.ponomarenko
   module ScopedFor
     # Define base instance scope for model by columns
     #
@@ -21,7 +19,7 @@ module PgLtree
       #
       # @return current class
       def ltree_scope
-        self.class.where *(ltree_scoped_for.map { |column| { column => public_send(column) } })
+        self.class.where(*(ltree_scoped_for.map { |column| { column => public_send(column) } }))
       end
     end
   end

data/lib/pg_ltree/version.rb

@@ -1,7 +1,4 @@
 # Organize ActiveRecord model into a tree structure with PostgreSQL LTree
-#
-# @author a.ponomarenko
 module PgLtree
-  # Gem Version
-  VERSION = '1.1.8'.freeze
+  VERSION = '1.1.9'.freeze
 end

data/test/database.yml.sample

@@ -1,5 +1,5 @@
 adapter:  postgresql
 host:     localhost
 username: postgres
-password:
+password: postgres
 database: pg_ltree_test

data/test/pg_ltree/ltree_test.rb

@@ -17,6 +17,9 @@ class PgLtree::LtreeTest < BaseTest
       Top.Collections.Pictures.Astronomy.Stars
       Top.Collections.Pictures.Astronomy.Galaxies
       Top.Collections.Pictures.Astronomy.Astronauts
+      Top.Collections.Videos
+      Top.Collections.Videos.Vacation
+      Top.Collections.Videos.NewYear
     ).each do |path|
       TreeNode.create! path: path
     end
@@ -50,6 +53,8 @@ class PgLtree::LtreeTest < BaseTest
       Top.Collections.Pictures.Astronomy.Stars
       Top.Collections.Pictures.Astronomy.Galaxies
       Top.Collections.Pictures.Astronomy.Astronauts
+      Top.Collections.Videos.Vacation
+      Top.Collections.Videos.NewYear
     )
   end
 
@@ -60,6 +65,8 @@ class PgLtree::LtreeTest < BaseTest
       Top.Hobbies.Amateurs_Astronomy
       Top.Collections.Pictures.Astronomy.Galaxies
       Top.Collections.Pictures.Astronomy.Astronauts
+      Top.Collections.Videos.Vacation
+      Top.Collections.Videos.NewYear
     )
   end
 
@@ -191,8 +198,52 @@ class PgLtree::LtreeTest < BaseTest
   end
 
   test '.cascade_destroy' do
-    TreeNode.find_by(path: 'Top.Collections').destroy
+    assert_equal TreeNode.where("path <@ 'Top.Collections'").pluck(:path), %w(
+      Top.Collections
+      Top.Collections.Pictures
+      Top.Collections.Pictures.Astronomy
+      Top.Collections.Pictures.Astronomy.Stars
+      Top.Collections.Pictures.Astronomy.Galaxies
+      Top.Collections.Pictures.Astronomy.Astronauts
+      Top.Collections.Videos
+      Top.Collections.Videos.Vacation
+      Top.Collections.Videos.NewYear
+    )
 
-    assert_equal TreeNode.where("path ~ 'Top.Collections'").pluck(:path), %w()
+    TreeNode.find_by(path: 'Top.Collections.Pictures').destroy
+
+    assert_equal TreeNode.where("path <@ 'Top.Collections'").pluck(:path), %w(
+      Top.Collections
+      Top.Collections.Videos
+      Top.Collections.Videos.Vacation
+      Top.Collections.Videos.NewYear
+    )
+  end
+
+  test '.destroy' do
+    assert_equal TreeWithoutCascadeNode.where("path <@ 'Top.Collections'").pluck(:path), %w(
+      Top.Collections
+      Top.Collections.Pictures
+      Top.Collections.Pictures.Astronomy
+      Top.Collections.Pictures.Astronomy.Stars
+      Top.Collections.Pictures.Astronomy.Galaxies
+      Top.Collections.Pictures.Astronomy.Astronauts
+      Top.Collections.Videos
+      Top.Collections.Videos.Vacation
+      Top.Collections.Videos.NewYear
+    )
+
+    TreeWithoutCascadeNode.find_by(path: 'Top.Collections.Pictures').destroy
+
+    assert_equal TreeWithoutCascadeNode.where("path <@ 'Top.Collections'").pluck(:path), %w(
+      Top.Collections
+      Top.Collections.Pictures.Astronomy
+      Top.Collections.Pictures.Astronomy.Stars
+      Top.Collections.Pictures.Astronomy.Galaxies
+      Top.Collections.Pictures.Astronomy.Astronauts
+      Top.Collections.Videos
+      Top.Collections.Videos.Vacation
+      Top.Collections.Videos.NewYear
+    )
   end
 end

data/test/pg_ltree/scoped_for_test.rb

@@ -17,6 +17,9 @@ class PgLtree::ScopedForTest < BaseTest
       Top.Collections.Pictures.Astronomy.Stars
       Top.Collections.Pictures.Astronomy.Galaxies
       Top.Collections.Pictures.Astronomy.Astronauts
+      Top.Collections.Videos
+      Top.Collections.Videos.Vacation
+      Top.Collections.Videos.NewYear
     ).each do |path|
       %i( active deactive ).each do |status|
         NotUniqTreeNode.create! new_path: path, status: status
@@ -47,6 +50,8 @@ class PgLtree::ScopedForTest < BaseTest
       Top.Collections.Pictures.Astronomy.Stars
       Top.Collections.Pictures.Astronomy.Galaxies
       Top.Collections.Pictures.Astronomy.Astronauts
+      Top.Collections.Videos.Vacation
+      Top.Collections.Videos.NewYear
     )
   end
 
@@ -161,32 +166,25 @@ class PgLtree::ScopedForTest < BaseTest
   end
 
   test '.cascade_destroy' do
-    assert_equal NotUniqTreeNode.where("new_path <@ 'Top.Collections'").pluck(:new_path), %w(
+    assert_equal NotUniqTreeNode.where("new_path <@ 'Top.Collections'").where(status: :active).pluck(:new_path), %w(
       Top.Collections
-      Top.Collections
-      Top.Collections.Pictures
       Top.Collections.Pictures
       Top.Collections.Pictures.Astronomy
-      Top.Collections.Pictures.Astronomy
       Top.Collections.Pictures.Astronomy.Stars
-      Top.Collections.Pictures.Astronomy.Stars
-      Top.Collections.Pictures.Astronomy.Galaxies
       Top.Collections.Pictures.Astronomy.Galaxies
       Top.Collections.Pictures.Astronomy.Astronauts
-      Top.Collections.Pictures.Astronomy.Astronauts
+      Top.Collections.Videos
+      Top.Collections.Videos.Vacation
+      Top.Collections.Videos.NewYear
     )
 
-    NotUniqTreeNode.find_by(new_path: 'Top.Collections', status: :active).destroy
+    NotUniqTreeNode.find_by(new_path: 'Top.Collections.Pictures', status: :active).destroy
 
-    assert_equal NotUniqTreeNode.where("new_path <@ 'Top.Collections'").where(status: :active).pluck(:new_path), %w()
-
-    assert_equal NotUniqTreeNode.where("new_path <@ 'Top.Collections'").where(status: :deactive).pluck(:new_path), %w(
+    assert_equal NotUniqTreeNode.where("new_path <@ 'Top.Collections'").where(status: :active).pluck(:new_path), %w(
       Top.Collections
-      Top.Collections.Pictures
-      Top.Collections.Pictures.Astronomy
-      Top.Collections.Pictures.Astronomy.Stars
-      Top.Collections.Pictures.Astronomy.Galaxies
-      Top.Collections.Pictures.Astronomy.Astronauts
+      Top.Collections.Videos
+      Top.Collections.Videos.Vacation
+      Top.Collections.Videos.NewYear
     )
   end
 end

data/test/test_helper.rb

@@ -7,6 +7,8 @@ rescue Bundler::BundlerError => e
   exit e.status_code
 end
 
+$VERBOSE=nil
+
 require 'pg'
 require 'pg_ltree'
 require 'minitest/autorun'
@@ -76,3 +78,8 @@ end
 class TreeNode < ActiveRecord::Base
   ltree
 end
+
+class TreeWithoutCascadeNode < ActiveRecord::Base
+  self.table_name = 'tree_nodes'
+  ltree :path, cascade: false
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pg_ltree
 version: !ruby/object:Gem::Version
-  version: 1.1.8
+  version: 1.1.9
 platform: ruby
 authors:
 - Andrei Panamarenka
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-12-13 00:00:00.000000000 Z
+date: 2022-08-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,20 +16,20 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.0.0
-    - - "<="
+        version: '5.2'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 7.0.0.rc1
+        version: '8.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.0.0
-    - - "<="
+        version: '5.2'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 7.0.0.rc1
+        version: '8.0'
 - !ruby/object:Gem::Dependency
   name: pg
   requirement: !ruby/object:Gem::Requirement
@@ -120,6 +120,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: appraisal
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Organise ActiveRecord model into a tree structure with PostgreSQL LTree
 email:
 - andrei.panamarenka@gmail.com
@@ -144,7 +158,7 @@ homepage: https://github.com/sjke/pg_ltree
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -159,8 +173,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.0.pre2
-signing_key: 
+rubygems_version: 3.1.6
+signing_key:
 specification_version: 4
 summary: Organise ActiveRecord model into a tree structure with PostgreSQL LTree
 test_files: