acts_as_dag 1.2.6 → 2.0.0

data/spec/deprecated_spec.rb

@@ -0,0 +1,128 @@
+require 'spec_helper'
+
+describe 'acts_as_dag' do
+  shared_examples_for "DAG Model" do
+    describe "reorganization" do
+      let(:totem) { klass.create(:name => "totem") }
+      let(:totem_pole) { klass.create(:name => "totem pole") }
+      let(:big_totem_pole) { klass.create(:name => "big totem pole") }
+      let(:big_model_totem_pole) { klass.create(:name => "big model totem pole") }
+      let(:big_red_model_totem_pole) { klass.create(:name => "big red model totem pole") }
+
+      before do
+        # Rspec alone doesn't support transactional tests, so clear it all out between tests
+        klass.destroy_all
+        # Ensure the models exist
+        totem; totem_pole; big_totem_pole; big_model_totem_pole; big_red_model_totem_pole
+      end
+
+      it "should be able to determine whether one category is an ancestor of the other by inspecting the name" do
+        ActsAsDAG::Deprecated::HelperMethods.should_descend_from?(totem_pole, big_totem_pole).should be_truthy
+        ActsAsDAG::Deprecated::HelperMethods.should_descend_from?(big_totem_pole, totem_pole).should be_falsy
+      end
+
+      it "should be able to determine the number of matching words in two categories names" do
+        ActsAsDAG::Deprecated::HelperMethods.matching_word_count(totem_pole, big_totem_pole).should == 2
+      end
+
+      it "should arrange the categories correctly when not passed any arguments" do
+        klass.reorganize
+
+        totem.children.should == [totem_pole]
+        totem_pole.children.should == [big_totem_pole]
+        big_totem_pole.children.should == [big_model_totem_pole]
+        big_model_totem_pole.children.should == [big_red_model_totem_pole]
+      end
+
+      it "should arrange the categories correctly when passed a set of nodes to reorganize" do
+        klass.reorganize [totem, totem_pole, big_totem_pole, big_model_totem_pole, big_red_model_totem_pole]
+
+        totem.reload.children.should == [totem_pole]
+        totem_pole.reload.children.should == [big_totem_pole]
+        big_totem_pole.reload.children.should == [big_model_totem_pole]
+        big_model_totem_pole.reload.children.should == [big_red_model_totem_pole]
+      end
+
+      it "should arrange the categories correctly when inserting a category into an existing chain" do
+        totem.add_child(big_totem_pole)
+
+        klass.reorganize
+
+        totem.children.should == [totem_pole]
+        totem_pole.children.should == [big_totem_pole]
+        big_totem_pole.children.should == [big_model_totem_pole]
+        big_model_totem_pole.reload.children.should == [big_red_model_totem_pole]
+      end
+
+      it "should still work when there are categories that are permutations of each other" do
+        big_totem_pole_model = klass.create(:name => "big totem pole model")
+
+        klass.reorganize [totem, totem_pole, big_totem_pole, big_model_totem_pole, big_red_model_totem_pole, big_totem_pole_model]
+
+        totem.children.should == [totem_pole]
+        totem_pole.children.should == [big_totem_pole]
+        (big_totem_pole.children - [big_model_totem_pole, big_totem_pole_model]).should == []
+        big_model_totem_pole.reload.children.should == [big_red_model_totem_pole]
+        big_totem_pole_model.reload.children.should == [big_red_model_totem_pole]
+      end
+
+      describe "when there is a single long inheritance chain" do
+        before(:each) do
+          totem.add_child(totem_pole)
+          totem_pole.add_child(big_totem_pole)
+          big_totem_pole.add_child(big_model_totem_pole)
+          big_model_totem_pole.add_child(big_red_model_totem_pole)
+        end
+
+        describe "and we are reorganizing the middle of the chain" do
+          # Totem
+          #   |
+          # Totem Pole
+          #  *|*       \
+          #  *|*      Big Totem Pole
+          #  *|*       /
+          # Big Model Totem Pole
+          #   |
+          # Big Red Model Totem Pole
+          #
+          before(:each) do
+            totem_pole.add_child(big_model_totem_pole)
+          end
+
+          it "should return multiple instances of descendants before breaking the old link" do
+            totem.descendants.sort_by(&:id).should == [totem_pole, big_totem_pole, big_model_totem_pole, big_model_totem_pole, big_red_model_totem_pole, big_red_model_totem_pole].sort_by(&:id)
+          end
+
+          it "should return the correct inheritance chain after breaking the old link" do
+            totem_pole.remove_child(big_model_totem_pole)
+
+            totem_pole.children.sort_by(&:id).should == [big_totem_pole].sort_by(&:id)
+            totem.descendants.sort_by(&:id).should == [totem_pole, big_totem_pole, big_model_totem_pole, big_red_model_totem_pole].sort_by(&:id)
+          end
+
+          it "should return the correct inheritance chain after breaking the old link when there is are two ancestor root nodes" do
+            pole = klass.create(:name => "pole")
+            totem_pole.add_parent(pole)
+            totem_pole.remove_child(big_model_totem_pole)
+
+            pole.descendants.sort_by(&:id).should == [totem_pole, big_totem_pole, big_model_totem_pole, big_red_model_totem_pole].sort_by(&:id)
+            totem_pole.children.sort_by(&:id).should == [big_totem_pole].sort_by(&:id)
+            totem.descendants.sort_by(&:id).should == [totem_pole, big_totem_pole, big_model_totem_pole, big_red_model_totem_pole].sort_by(&:id)
+          end
+        end
+      end
+    end
+  end
+
+  describe "models with separate link tables" do
+    let(:klass) { SeparateLinkModel }
+
+    it_should_behave_like "DAG Model"
+  end
+
+  describe "models with unified link tables" do
+    let(:klass) { UnifiedLinkModel }
+
+    it_should_behave_like "DAG Model"
+  end
+end

data/spec/spec_helper.rb

@@ -3,8 +3,10 @@ require 'active_record'
 require 'logger'
 require 'acts_as_dag'
 
+puts ActiveRecord.version
+
 ActiveRecord::Base.logger = Logger.new(STDOUT)
-ActiveRecord::Base.logger.level = Logger::INFO
+ActiveRecord::Base.logger.level = Logger::WARN
 ActiveRecord::Base.establish_connection(:adapter => "sqlite3", :database => ":memory:")
 
 ActiveRecord::Schema.define(:version => 0) do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: acts_as_dag
 version: !ruby/object:Gem::Version
-  version: 1.2.6
+  version: 2.0.0
 platform: ruby
 authors:
 - Nicholas Jakobsen
@@ -15,16 +15,30 @@ dependencies:
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">"
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '4.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">"
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '4.0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.2'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.2'
 description: 
 email: technical@rrnpilot.org
 executables: []
@@ -32,10 +46,12 @@ extensions: []
 extra_rdoc_files: []
 files:
 - LICENSE
-- README.rdoc
+- README.md
 - lib/acts_as_dag.rb
 - lib/acts_as_dag/acts_as_dag.rb
+- lib/acts_as_dag/deprecated.rb
 - spec/acts_as_dag_spec.rb
+- spec/deprecated_spec.rb
 - spec/spec_helper.rb
 homepage: http://github.com/rrn/acts_as_dag
 licenses: []
@@ -56,7 +72,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.13
+rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: Adds directed acyclic graph functionality to ActiveRecord.

data/README.rdoc

@@ -1,28 +0,0 @@
-== ActsAsDAG
-
-Adds Directed Acyclic Graph functionality to ActiveRecord
-
-Used by RRN data mapper to setup link and descendant database tables
-
-## Installation
-
-### Migration
-
-```ruby
-class CreateActsAsDagTables < ActiveRecord::Migration
-  def change
-    create_table "acts_as_dag_descendants", :force => true do |t|
-      t.string :category_type
-      t.references :ancestor
-      t.references :descendant
-      t.integer :distance
-    end
-
-    create_table "acts_as_dag_links", :force => true do |t|
-      t.string :category_type
-      t.references :parent
-      t.references :child
-    end
-  end
-end
-```