acts_as_ordered_tree 1.3.1 → 2.0.0.beta3

data/spec/node/movements/move_to_child_with_index_spec.rb

@@ -0,0 +1,124 @@
+# coding: utf-8
+
+require 'spec_helper'
+
+describe ActsAsOrderedTree::Node::Movements, '#move_to_child_with_index', :transactional do
+  shared_examples '#move_to_child_with_index' do |factory|
+    describe "#move_to_child_with_index #{factory}" do
+      tree :factory => factory do
+        root {
+          node_1
+          node_2
+          node_3 {
+            node_4
+          }
+        }
+      end
+
+      context 'moving node to same parent and same position' do
+        before { node_2.move_to_child_with_index(root, 1) }
+
+        expect_tree_to_match {
+          root {
+            node_1
+            node_2
+            node_3 {
+              node_4
+            }
+          }
+        }
+      end
+
+      context 'moving node to same parent with another position' do
+        before { node_1.move_to_child_with_index(root, 1) }
+
+        expect_tree_to_match {
+          root {
+            node_2
+            node_1
+            node_3 {
+              node_4
+            }
+          }
+        }
+      end
+
+      context 'moving node to same parent to lowest position' do
+        before { node_1.move_to_child_with_index(root, -1) }
+
+        expect_tree_to_match {
+          root {
+            node_2
+            node_3 {
+              node_4
+            }
+            node_1
+          }
+        }
+      end
+
+      context 'moving node to position with negative index' do
+        before { node_4.move_to_child_with_index(root, -2) }
+
+        expect_tree_to_match {
+          root {
+            node_1
+            node_4
+            node_2
+            node_3
+          }
+        }
+      end
+
+      context 'moving node to root with index starting from end' do
+        before { node_4.move_to_child_with_index(nil, -1) }
+
+        expect_tree_to_match {
+          node_4
+          root {
+            node_1
+            node_2
+            node_3
+          }
+        }
+      end
+
+      context 'moving to node to very position with large negative index' do
+        before { node_4.move_to_child_with_index(root, -100) }
+
+        expect_tree_to_match {
+          root {
+            node_4
+            node_1
+            node_2
+            node_3
+          }
+        }
+      end
+
+      context 'moving to node to very large position' do
+        before { node_4.move_to_child_with_index(root, 100) }
+
+        expect_tree_to_match {
+          root {
+            node_1
+            node_2
+            node_3
+            node_4
+          }
+        }
+      end
+
+      context 'when attribute, not related to tree, changed' do
+        before { @old_name = node_2.name }
+        before { node_2.name = 'new name' }
+
+        it { expect{node_2.move_to_child_with_index(root, 1)}.to change(node_2, :name).to(@old_name) }
+      end
+    end
+  end
+
+  include_examples '#move_to_child_with_index', :default
+  include_examples '#move_to_child_with_index', :default_with_counter_cache
+  include_examples '#move_to_child_with_index', :scoped
+end

data/spec/node/movements/move_to_child_with_position_spec.rb

@@ -0,0 +1,85 @@
+# coding: utf-8
+
+require 'spec_helper'
+
+describe ActsAsOrderedTree::Node::Movements, '#move_to_child_with_position', :transactional do
+  shared_examples '#move_to_child_with_position' do |factory|
+    describe "#move_to_child_with_position #{factory}" do
+      tree :factory => factory do
+        root {
+          node_1
+          node_2
+          node_3 {
+            node_4
+          }
+        }
+      end
+
+      context 'moving node to same parent and same position' do
+        before { node_2.move_to_child_with_position(root, 2) }
+
+        expect_tree_to_match {
+          root {
+            node_1
+            node_2
+            node_3 {
+              node_4
+            }
+          }
+        }
+      end
+
+      context 'moving node to same parent with another position' do
+        before { node_1.move_to_child_with_position(root, 2) }
+
+        expect_tree_to_match {
+          root {
+            node_2
+            node_1
+            node_3 {
+              node_4
+            }
+          }
+        }
+      end
+
+      context 'moving node to same parent to lowest position' do
+        before { node_1.move_to_child_with_position(root, 3) }
+
+        expect_tree_to_match {
+          root {
+            node_2
+            node_3 {
+              node_4
+            }
+            node_1
+          }
+        }
+      end
+
+      context 'moving to node to very large position' do
+        before { node_4.move_to_child_with_position(root, 100) }
+
+        expect_tree_to_match {
+          root {
+            node_1
+            node_2
+            node_3
+            node_4
+          }
+        }
+      end
+
+      context 'when attribute, not related to tree, changed' do
+        before { @old_name = node_2.name }
+        before { node_2.name = 'new name' }
+
+        it { expect{node_2.move_to_child_with_position(root, 2)}.to change(node_2, :name).to(@old_name) }
+      end
+    end
+  end
+
+  include_examples '#move_to_child_with_position', :default
+  include_examples '#move_to_child_with_position', :default_with_counter_cache
+  include_examples '#move_to_child_with_position', :scoped
+end

data/spec/node/movements/move_to_left_of_spec.rb

@@ -0,0 +1,120 @@
+# coding: utf-8
+
+require 'spec_helper'
+
+describe ActsAsOrderedTree::Node::Movements, '#move_to_left_of', :transactional do
+  shared_examples '#move_to_left_of' do |factory|
+    describe "#move_to_left_of #{factory}" do
+      tree :factory => factory do
+        root {
+          node_1
+          node_2
+          node_3 {
+            node_4
+          }
+        }
+      end
+
+      context 'moving node to next position' do
+        before { node_1.move_to_left_of(node_2) }
+
+        expect_tree_to_match {
+          root {
+            node_1
+            node_2
+            node_3 {
+              node_4
+            }
+          }
+        }
+      end
+
+      context 'moving node to same parent higher' do
+        before { node_3.move_to_left_of(node_1) }
+
+        expect_tree_to_match {
+          root {
+            node_3 {
+              node_4
+            }
+            node_1
+            node_2
+          }
+        }
+      end
+
+      context 'moving node to same parent lower' do
+        before { node_1.move_to_left_of(node_3) }
+
+        expect_tree_to_match {
+          root {
+            node_2
+            node_1
+            node_3 {
+              node_4
+            }
+          }
+        }
+      end
+
+      context 'moving inner node to left of root' do
+        before { node_3.move_to_left_of(root) }
+
+        expect_tree_to_match {
+          node_3 {
+            node_4
+          }
+          root {
+            node_1
+            node_2
+          }
+        }
+      end
+
+      context 'moving inner node to left of another inner node (shallower)' do
+        before { node_4.move_to_left_of(node_1) }
+
+        expect_tree_to_match {
+          root {
+            node_4
+            node_1
+            node_2
+            node_3
+          }
+        }
+      end
+
+      context 'moving inner node to left of another inner node (deeper)' do
+        before { node_1.move_to_left_of(node_4) }
+
+        expect_tree_to_match {
+          root {
+            node_2
+            node_3 {
+              node_1
+              node_4
+            }
+          }
+        }
+      end
+
+      context 'attempt to perform impossible movement' do
+        it { expect{ root.move_to_left_of(node_1) }.not_to change(current_tree, :all) }
+        it { expect{ node_3.move_to_left_of(node_4) }.not_to change(current_tree, :all) }
+        it { expect{ node_1.move_to_left_of(node_1) }.not_to change(current_tree, :all) }
+        it { expect{ node_3.move_to_left_of(node_3) }.not_to change(current_tree, :all) }
+      end
+
+      context 'when attribute, not related to tree, changed' do
+        before { @old_name = node_2.name }
+        before { node_2.name = 'new name' }
+
+        it { expect{node_2.move_to_left_of(node_4)}.to change(node_2, :name).to(@old_name) }
+      end
+    end
+  end
+
+  include_examples '#move_to_left_of', :default
+  include_examples '#move_to_left_of', :default_with_counter_cache
+  include_examples '#move_to_left_of', :scoped, :scope_type => 's'
+end

data/spec/node/movements/move_to_right_of_spec.rb

@@ -0,0 +1,120 @@
+# coding: utf-8
+
+require 'spec_helper'
+
+describe ActsAsOrderedTree::Node::Movements, '#move_to_right_of', :transactional do
+  shared_examples '#move_to_right_of' do |factory|
+    describe "#move_to_right_of #{factory}" do
+      tree :factory => factory do
+        root {
+          node_1
+          node_2
+          node_3 {
+            node_4
+          }
+        }
+      end
+
+      context 'moving node to same parent higher' do
+        before { node_3.move_to_right_of(node_1) }
+
+        expect_tree_to_match {
+          root {
+            node_1
+            node_3 {
+              node_4
+            }
+            node_2
+          }
+        }
+      end
+
+      context 'moving node to next position' do
+        before { node_1.move_to_right_of(node_2) }
+
+        expect_tree_to_match {
+          root {
+            node_2
+            node_1
+            node_3 {
+              node_4
+            }
+          }
+        }
+      end
+
+      context 'moving node to same parent lower' do
+        before { node_1.move_to_right_of(node_3) }
+
+        expect_tree_to_match {
+          root {
+            node_2
+            node_3 {
+              node_4
+            }
+            node_1
+          }
+        }
+      end
+
+      context 'moving inner node to right of root node' do
+        before { node_3.move_to_right_of(root) }
+
+        expect_tree_to_match {
+          root {
+            node_1
+            node_2
+          }
+          node_3 {
+            node_4
+          }
+        }
+      end
+
+      context 'moving inner node to right of another inner node (shallower)' do
+        before { node_4.move_to_right_of(node_1) }
+
+        expect_tree_to_match {
+          root {
+            node_1
+            node_4
+            node_2
+            node_3
+          }
+        }
+      end
+
+      context 'moving inner node to right of another inner node (deeper)' do
+        before { node_1.move_to_right_of(node_4) }
+
+        expect_tree_to_match {
+          root {
+            node_2
+            node_3 {
+              node_4
+              node_1
+            }
+          }
+        }
+      end
+
+      context 'Attempt to perform impossible movement' do
+        it { expect{ root.move_to_right_of(node_1) }.not_to change(current_tree, :all) }
+        it { expect{ node_3.move_to_right_of(node_4) }.not_to change(current_tree, :all) }
+        it { expect{ node_1.move_to_right_of(node_1) }.not_to change(current_tree, :all) }
+        it { expect{ node_3.move_to_right_of(node_3) }.not_to change(current_tree, :all) }
+      end
+
+      context 'when attribute, not related to tree changed' do
+        before { @old_name = node_2.name }
+        before { node_2.name = 'new name' }
+
+        it { expect{node_2.move_to_right_of(node_4)}.to change(node_2, :name).to(@old_name) }
+      end
+    end
+  end
+
+  include_examples '#move_to_right_of', :default
+  include_examples '#move_to_right_of', :default_with_counter_cache
+  include_examples '#move_to_right_of', :scoped, :scope_type => 's'
+end