acts_as_list 1.0.4 → 1.2.4

data/test/shared_no_addition.rb

@@ -1,38 +0,0 @@
-# frozen_string_literal: true
-
-module Shared
-  module NoAddition
-    def setup
-      (1..4).each { |counter| NoAdditionMixin.create! pos: counter, parent_id: 5 }
-    end
-
-    def test_insert
-      new = NoAdditionMixin.create(parent_id: 20)
-      assert_nil new.pos
-      assert !new.in_list?
-
-      new = NoAdditionMixin.create(parent_id: 20)
-      assert_nil new.pos
-    end
-
-    def test_update_does_not_add_to_list
-      new = NoAdditionMixin.create(parent_id: 20)
-      new.update_attribute(:updated_at, Time.now) # force some change
-      new.reload
-
-      assert !new.in_list?
-    end
-
-    def test_update_scope_does_not_add_to_list
-      new = NoAdditionMixin.create
-
-      new.update_attribute(:parent_id, 20)
-      new.reload
-      assert !new.in_list?
-
-      new.update_attribute(:parent_id, 5)
-      new.reload
-      assert !new.in_list?
-    end
-  end
-end

data/test/shared_quoting.rb

@@ -1,23 +0,0 @@
-# frozen_string_literal: true
-
-module Shared
-  module Quoting
-
-    def setup
-      3.times { |counter| QuotedList.create! order: counter }
-    end
-
-    def test_create
-      assert_equal QuotedList.in_list.size, 3
-    end
-
-    # This test execute raw queries involving table name
-    def test_moving
-      item = QuotedList.first
-      item.higher_items
-      item.lower_items
-      item.send :bottom_item # Part of private api
-    end
-
-  end
-end

data/test/shared_top_addition.rb

@@ -1,110 +0,0 @@
-# frozen_string_literal: true
-
-module Shared
-  module TopAddition
-    def setup
-      (1..4).each { |counter| TopAdditionMixin.create! pos: counter, parent_id: 5 }
-    end
-
-    def test_setup_state
-      # If we explicitly define a position (as above) then that position is what gets applied
-      assert_equal [1, 2, 3, 4], TopAdditionMixin.where(parent_id: 5).order('pos').map(&:id)
-    end
-
-    def test_reordering
-      TopAdditionMixin.where(id: 2).first.move_lower
-      assert_equal [1, 3, 2, 4], TopAdditionMixin.where(parent_id: 5).order('pos').map(&:id)
-
-      TopAdditionMixin.where(id: 2).first.move_higher
-      assert_equal [1, 2, 3, 4], TopAdditionMixin.where(parent_id: 5).order('pos').map(&:id)
-
-      TopAdditionMixin.where(id: 1).first.move_to_bottom
-      assert_equal [2, 3, 4, 1], TopAdditionMixin.where(parent_id: 5).order('pos').map(&:id)
-
-      TopAdditionMixin.where(id: 1).first.move_to_top
-      assert_equal [1, 2, 3, 4], TopAdditionMixin.where(parent_id: 5).order('pos').map(&:id)
-
-      TopAdditionMixin.where(id: 2).first.move_to_bottom
-      assert_equal [1, 3, 4, 2], TopAdditionMixin.where(parent_id: 5).order('pos').map(&:id)
-
-      TopAdditionMixin.where(id: 4).first.move_to_top
-      assert_equal [4, 1, 3, 2], TopAdditionMixin.where(parent_id: 5).order('pos').map(&:id)
-    end
-
-    def test_injection
-      item = TopAdditionMixin.new(parent_id: 1)
-      assert_equal({ parent_id: 1 }, item.scope_condition)
-      assert_equal "pos", item.position_column
-    end
-
-    def test_insert
-      new = TopAdditionMixin.create(parent_id: 20)
-      assert_equal 1, new.pos
-      assert new.first?
-      assert new.last?
-
-      new = TopAdditionMixin.create(parent_id: 20)
-      assert_equal 1, new.pos
-      assert new.first?
-      assert !new.last?
-
-      new = TopAdditionMixin.acts_as_list_no_update { TopAdditionMixin.create(parent_id: 20) }
-      assert_equal_or_nil $default_position, new.pos
-      assert_equal $default_position.is_a?(Integer), new.first?
-      assert !new.last?
-
-      new = TopAdditionMixin.create(parent_id: 20)
-      assert_equal 1, new.pos
-      assert_equal $default_position.nil?, new.first?
-      assert !new.last?
-
-      new = TopAdditionMixin.create(parent_id: 0)
-      assert_equal 1, new.pos
-      assert new.first?
-      assert new.last?
-    end
-
-    def test_insert_at
-      new = TopAdditionMixin.create(parent_id: 20)
-      assert_equal 1, new.pos
-
-      new = TopAdditionMixin.create(parent_id: 20)
-      assert_equal 1, new.pos
-
-      new = TopAdditionMixin.create(parent_id: 20)
-      assert_equal 1, new.pos
-
-      new = TopAdditionMixin.acts_as_list_no_update { TopAdditionMixin.create(parent_id: 20) }
-      assert_equal_or_nil $default_position, new.pos
-
-      new4 = TopAdditionMixin.create(parent_id: 20)
-      assert_equal 1, new4.pos
-
-      new4.insert_at(3)
-      assert_equal 3, new4.pos
-    end
-
-    def test_supplied_position
-      new = TopAdditionMixin.create(parent_id: 20, pos: 3)
-      assert_equal 3, new.pos
-    end
-
-    def test_delete_middle
-      TopAdditionMixin.where(id: 2).first.destroy
-
-      assert_equal [1, 3, 4], TopAdditionMixin.where(parent_id: 5).order('pos').map(&:id)
-
-      assert_equal 1, TopAdditionMixin.where(id: 1).first.pos
-      assert_equal 2, TopAdditionMixin.where(id: 3).first.pos
-      assert_equal 3, TopAdditionMixin.where(id: 4).first.pos
-
-      TopAdditionMixin.acts_as_list_no_update { TopAdditionMixin.where(id: 3).first.destroy }
-
-      assert_equal [1, 4], TopAdditionMixin.where(parent_id: 5).order('pos').map(&:id)
-
-      assert_equal 1, TopAdditionMixin.where(id: 1).first.pos
-      assert_equal 3, TopAdditionMixin.where(id: 4).first.pos
-    end
-
-  end
-end

data/test/shared_zero_based.rb

@@ -1,104 +0,0 @@
-# frozen_string_literal: true
-
-module Shared
-  module ZeroBased
-    def setup
-       (1..4).each { |counter| ZeroBasedMixin.create! pos: counter, parent_id: 5 }
-    end
-
-    def test_insert
-      new = ZeroBasedMixin.create(parent_id: 20)
-      assert_equal 0, new.pos
-      assert new.first?
-      assert new.last?
-
-      new = ZeroBasedMixin.create(parent_id: 20)
-      assert_equal 1, new.pos
-      assert !new.first?
-      assert new.last?
-
-      new = ZeroBasedMixin.acts_as_list_no_update { ZeroBasedMixin.create(parent_id: 20) }
-      assert_equal_or_nil $default_position, new.pos
-      assert !new.first?
-      assert !new.last?
-
-      new = ZeroBasedMixin.create(parent_id: 20)
-      assert_equal 2, new.pos
-      assert !new.first?
-      assert new.last?
-
-      new = ZeroBasedMixin.create(parent_id: 0)
-      assert_equal 0, new.pos
-      assert new.first?
-      assert new.last?
-
-      new = ZeroBasedMixin.create(parent_id: 1, pos: -500)
-      assert_equal 0, new.pos
-      assert new.first?
-      assert new.last?
-    end
-
-    def test_reordering
-      assert_equal [1, 2, 3, 4], ZeroBasedMixin.where(parent_id: 5).order('pos').map(&:id)
-
-      ListMixin.where(id: 2).first.move_lower
-      assert_equal [1, 3, 2, 4], ZeroBasedMixin.where(parent_id: 5).order('pos').map(&:id)
-
-      ListMixin.where(id: 2).first.move_higher
-      assert_equal [1, 2, 3, 4], ZeroBasedMixin.where(parent_id: 5).order('pos').map(&:id)
-
-      ListMixin.where(id: 1).first.move_to_bottom
-      assert_equal [2, 3, 4, 1], ZeroBasedMixin.where(parent_id: 5).order('pos').map(&:id)
-
-      ListMixin.where(id: 1).first.move_to_top
-      assert_equal [1, 2, 3, 4], ZeroBasedMixin.where(parent_id: 5).order('pos').map(&:id)
-
-      ListMixin.where(id: 2).first.move_to_bottom
-      assert_equal [1, 3, 4, 2], ZeroBasedMixin.where(parent_id: 5).order('pos').map(&:id)
-
-      ListMixin.where(id: 4).first.move_to_top
-      assert_equal [4, 1, 3, 2], ZeroBasedMixin.where(parent_id: 5).order('pos').map(&:id)
-    end
-
-    def test_insert_at
-      new = ZeroBasedMixin.create(parent_id: 20)
-      assert_equal 0, new.pos
-
-      new = ZeroBasedMixin.create(parent_id: 20)
-      assert_equal 1, new.pos
-
-      new = ZeroBasedMixin.create(parent_id: 20)
-      assert_equal 2, new.pos
-
-      new_noup = ZeroBasedMixin.acts_as_list_no_update { ZeroBasedMixin.create(parent_id: 20) }
-      assert_equal_or_nil $default_position, new_noup.pos
-
-      new4 = ZeroBasedMixin.create(parent_id: 20)
-      assert_equal 3, new4.pos
-
-      new4.insert_at(2)
-      assert_equal 2, new4.pos
-
-      new.reload
-      assert_equal 3, new.pos
-
-      new.insert_at(2)
-      assert_equal 2, new.pos
-
-      new4.reload
-      assert_equal 3, new4.pos
-
-      new5 = ListMixin.create(parent_id: 20)
-      assert_equal 4, new5.pos
-
-      new5.insert_at(1)
-      assert_equal 1, new5.pos
-
-      new4.reload
-      assert_equal 4, new4.pos
-
-      new_noup.reload
-      assert_equal_or_nil $default_position, new_noup.pos
-    end
-  end
-end

data/test/test_default_scope_with_select.rb

@@ -1,33 +0,0 @@
-require 'helper'
-
-class Animal < ActiveRecord::Base
-  acts_as_list
-  default_scope -> { select(:name) }
-end
-
-class DefaultScopeWithSelectTest < Minitest::Test
-  def setup
-    ActiveRecord::Base.connection.create_table :animals do |t|
-      t.column :position, :integer
-      t.column :name, :string
-    end
-
-    ActiveRecord::Base.connection.schema_cache.clear!
-    Animal.reset_column_information
-    super
-  end
-
-  def teardown
-    teardown_db
-    super
-  end
-
-  def test_default_scope_with_select
-    animal1 = Animal.create name: 'Fox'
-    animal2 = Animal.create name: 'Panda'
-    animal3 = Animal.create name: 'Wildebeast'
-    assert_equal 1, animal1.position
-    assert_equal 2, animal2.position
-    assert_equal 3, animal3.position
-  end
-end

data/test/test_joined_list.rb

@@ -1,61 +0,0 @@
-# frozen_string_literal: true
-
-require 'helper'
-
-class Section < ActiveRecord::Base
-  has_many :items
-  acts_as_list
-
-  scope :visible, -> { where(visible: true) }
-end
-
-class Item < ActiveRecord::Base
-  belongs_to :section
-  acts_as_list scope: :section
-
-  scope :visible, -> { where(visible: true).joins(:section).merge(Section.visible) }
-end
-
-class JoinedTestCase < Minitest::Test
-  def setup
-    ActiveRecord::Base.connection.create_table :sections do |t|
-      t.column :position, :integer
-      t.column :visible, :boolean, default: true
-    end
-
-    ActiveRecord::Base.connection.create_table :items do |t|
-      t.column :position, :integer
-      t.column :section_id, :integer
-      t.column :visible, :boolean, default: true
-    end
-
-    ActiveRecord::Base.connection.schema_cache.clear!
-    [Section, Item].each(&:reset_column_information)
-    super
-  end
-
-  def teardown
-    teardown_db
-    super
-  end
-end
-
-# joining the relation returned by `#higher_items` or `#lower_items` to another table
-# previously could result in ambiguous column names in the query
-class TestHigherLowerItems < JoinedTestCase
-  def test_higher_items
-    section = Section.create
-    item1 = Item.create section: section
-    item2 = Item.create section: section
-    item3 = Item.create section: section
-    assert_equal item3.higher_items.visible, [item2, item1]
-  end
-
-  def test_lower_items
-    section = Section.create
-    item1 = Item.create section: section
-    item2 = Item.create section: section
-    item3 = Item.create section: section
-    assert_equal item1.lower_items.visible, [item2, item3]
-  end
-end