mongo_mapper_acts_as_list 0.3.1 → 0.3.2

data/Gemfile

@@ -0,0 +1,7 @@
+source 'http://rubygems.org'
+
+gem 'bson_ext'
+gem 'mongo_mapper'
+gem 'rails', '3.1.1'
+gem 'ruby-debug19'
+gem 'shoulda'

data/Rakefile

@@ -0,0 +1,32 @@
+require 'rubygems'
+require 'bundler/setup'
+require 'rake'
+require 'rake/testtask'
+require File.expand_path('../lib/mongo_mapper/plugins/version', __FILE__)
+
+Rake::TestTask.new do |t|
+  t.libs = %w(test)
+  t.pattern = 'test/*_test.rb'
+end
+  
+task :default => :test
+
+
+
+desc 'Builds the gem'
+task :build do
+  sh "gem build mongo_mapper_acts_as_list.gemspec"
+end
+
+desc 'Builds and installs the gem'
+task :install => :build do
+  sh "gem install mongo_mapper_acts_as_list-#{MongoMapper::Plugins::ActsAsList::Version}"
+end
+
+desc 'Tags version, pushes to remote, and pushes gem'
+task :release => :build do
+  sh "git tag v#{MongoMapper::Plugins::ActsAsList::Version}"
+  sh "git push origin master"
+  sh "git push origin v#{MongoMapper::Plugins::ActsAsList::Version}"
+  sh "gem push mongo_mapper_acts_as_list-#{MongoMapper::Plugins::ActsAsList::Version}.gem"
+end

data/init.rb

@@ -0,0 +1 @@
+require 'mongo_mapper_acts_as_list'

data/lib/mongo_mapper/plugins/acts_as_list.rb

@@ -63,6 +63,8 @@ module MongoMapper
             end
 
             #{scope_condition_method}
+            
+            key '#{configuration[:column]}', Integer
 
             before_destroy :decrement_positions_on_lower_items
             before_create  :add_to_list_bottom

data/lib/mongo_mapper/plugins/version.rb

@@ -8,7 +8,7 @@ module MongoMapper
       
       
       
-      Version = '0.3.1'
+      Version = '0.3.2'
       
       
       

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mongo_mapper_acts_as_list
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.3.2
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2011-10-23 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mongo_mapper
-  requirement: &70119513438580 !ruby/object:Gem::Requirement
+  requirement: &70330524729220 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70119513438580
+  version_requirements: *70330524729220
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70119513438100 !ruby/object:Gem::Requirement
+  requirement: &70330524728780 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,12 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70119513438100
-description: 
-email:
-- tomas.celizna@gmail.com
+  version_requirements: *70330524728780
+description: This is a port of classic Rails' acts_as_list to Mongo Mapper. This Mongo
+  Mapper plugin provides the capabilities for sorting and reordering a number of objects
+  in a list. If you do not specify custom position column in the options, a key named
+  position will be added and used by default.
+email: tomas.celizna@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -43,9 +45,10 @@ files:
 - lib/mongo_mapper/plugins/acts_as_list.rb
 - lib/mongo_mapper/plugins/version.rb
 - lib/mongo_mapper_acts_as_list.rb
-- test/mongo_mapper_acts_as_list_test.rb
-- test/test_helper.rb
+- Gemfile
+- Rakefile
 - README.rdoc
+- init.rb
 homepage: http://github.com/tomasc/mongo_mapper_acts_as_list
 licenses: []
 post_install_message: 
@@ -60,7 +63,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 3420355153011742463
+      hash: -342600062179569977
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -72,5 +75,5 @@ rubyforge_project:
 rubygems_version: 1.8.7
 signing_key: 
 specification_version: 3
-summary: Port of classic Rails ActsAsList for MongoMapper
+summary: Port of classic Rails ActsAsList for MongoMapper.
 test_files: []

data/test/mongo_mapper_acts_as_list_test.rb

@@ -1,362 +0,0 @@
-require 'test_helper'
-
-
-
-# ---------------------------------------------------------------------
-# CLASS SETUP
-
-
-
-class ListMixin
-  
-  include MongoMapper::Document
-  include MongoMapper::Plugins::ActsAsList
-
-  key :pos, Integer
-  key :parent_id, Integer
-  key :original_id, Integer
-
-  acts_as_list :column => :pos, :scope => :parent_id
-  
-end
-
-class ListMixinSub1 < ListMixin
-end
-
-class ListMixinSub2 < ListMixin
-end
-
-class ListMixinWithArrayScope
-  
-  include MongoMapper::Document
-  include MongoMapper::Plugins::ActsAsList
-
-  key :pos, Integer
-  key :parent_id, Integer
-
-  acts_as_list :column => :pos, :scope => [:parent_id, :original_id]
-  
-end
-
-
-
-# ---------------------------------------------------------------------
-
-
-
-class ScopeTest < ActiveSupport::TestCase
-
-  def setup
-    @lm1 = ListMixin.create! :pos => 1, :parent_id => 5, :original_id => 1
-    @lm2 = ListMixinWithArrayScope.create! :pos => 1, :parent_id => 5, :original_id => 1
-  end
-
-  def test_symbol_scope
-    assert_equal @lm1.scope_condition, { :parent_id => 5 }
-    assert_equal @lm2.scope_condition, { :parent_id => 5, :original_id => 1 }
-  end
-
-end
-
-
-
-# ---------------------------------------------------------------------
-
-
-
-class ActiveSupport::TestCase
-	
-	private
-	
-	def mixins_with_parent_id(parent_id)
-		ListMixin.where(:parent_id => parent_id).sort(:pos).all.map(&:original_id)
-	end
-	
-	def mixin_with_original_id(original_id)
-		ListMixin.where(:original_id => original_id).first
-	end
-end
-
-
-
-# ---------------------------------------------------------------------
-
-
-
-class ListTest < ActiveSupport::TestCase
-
-    def setup
-      (1..4).each{ |counter| ListMixin.create! :pos => counter, :parent_id => 5, :original_id => counter }
-    end
-
-    def test_reordering
-      assert_equal [1, 2, 3, 4], mixins_with_parent_id(5)
-
-      mixin_with_original_id(2).move_lower
-      assert_equal [1, 3, 2, 4], mixins_with_parent_id(5)
-
-      mixin_with_original_id(2).move_higher
-      assert_equal [1, 2, 3, 4], mixins_with_parent_id(5)
-
-      mixin_with_original_id(1).move_to_bottom
-      assert_equal [2, 3, 4, 1], mixins_with_parent_id(5)
-
-      mixin_with_original_id(1).move_to_top
-      assert_equal [1, 2, 3, 4], mixins_with_parent_id(5)
-
-      mixin_with_original_id(2).move_to_bottom
-      assert_equal [1, 3, 4, 2], mixins_with_parent_id(5)
-
-      mixin_with_original_id(4).move_to_top
-      assert_equal [4, 1, 3, 2], mixins_with_parent_id(5)
-    end
-
-    def test_move_to_bottom_with_next_to_last_item
-      assert_equal [1, 2, 3, 4], mixins_with_parent_id(5)
-      mixin_with_original_id(3).move_to_bottom
-      assert_equal [1, 2, 4, 3], mixins_with_parent_id(5)
-    end
-
-    def test_next_prev
-      assert_equal mixin_with_original_id(2), mixin_with_original_id(1).lower_item
-      assert_nil mixin_with_original_id(1).higher_item
-      assert_equal mixin_with_original_id(3), mixin_with_original_id(4).higher_item
-      assert_nil mixin_with_original_id(4).lower_item
-    end
-
-    def test_injection
-      item = ListMixin.new(:parent_id => 1)
-      assert_equal item.scope_condition, {:parent_id => 1}
-      assert_equal "pos", item.position_column
-    end
-
-    def test_insert
-      new = ListMixin.create(:parent_id => 20)
-      assert_equal 1, new.pos
-      assert new.first?
-      assert new.last?
-
-      new = ListMixin.create(:parent_id => 20)
-      assert_equal 2, new.pos
-      assert !new.first?
-      assert new.last?
-
-      new = ListMixin.create(:parent_id => 20)
-      assert_equal 3, new.pos
-      assert !new.first?
-      assert new.last?
-
-      new = ListMixin.create(:parent_id => 0)
-      assert_equal 1, new.pos
-      assert new.first?
-      assert new.last?
-    end
-
-    def test_insert_at
-      new = ListMixin.create(:parent_id => 20)
-      assert_equal 1, new.pos
-
-      new = ListMixin.create(:parent_id => 20)
-      assert_equal 2, new.pos
-
-      new = ListMixin.create(:parent_id => 20)
-      assert_equal 3, new.pos
-
-      new4 = ListMixin.create(:parent_id => 20)
-      assert_equal 4, new4.pos
-
-      new4.insert_at(3)
-      assert_equal 3, new4.pos
-
-      new.reload
-      assert_equal 4, new.pos
-
-      new.insert_at(2)
-      assert_equal 2, new.pos
-
-      new4.reload
-      assert_equal 4, new4.pos
-
-      new5 = ListMixin.create(:parent_id => 20)
-      assert_equal 5, new5.pos
-
-      new5.insert_at(1)
-      assert_equal 1, new5.pos
-
-      new4.reload
-      assert_equal 5, new4.pos
-    end
-
-    def test_delete_middle
-      assert_equal [1, 2, 3, 4], mixins_with_parent_id(5)
-
-      mixin_with_original_id(2).destroy
-
-      assert_equal [1, 3, 4], mixins_with_parent_id(5)
-
-      assert_equal 1, mixin_with_original_id(1).pos
-      assert_equal 2, mixin_with_original_id(3).pos
-      assert_equal 3, mixin_with_original_id(4).pos
-
-      mixin_with_original_id(1).destroy
-
-      assert_equal [3, 4], mixins_with_parent_id(5)
-
-      assert_equal 1, mixin_with_original_id(3).pos
-      assert_equal 2, mixin_with_original_id(4).pos
-    end
-
-    def test_nil_scope
-      new1, new2, new3 = ListMixin.create, ListMixin.create, ListMixin.create
-      new2.move_higher
-      assert_equal [new2, new1, new3], ListMixin.where(:parent_id => nil).sort(:pos).all
-    end
-
-    def test_remove_from_list_should_then_fail_in_list? 
-      assert_equal true, mixin_with_original_id(1).in_list?
-      mixin_with_original_id(1).remove_from_list
-      assert_equal false, mixin_with_original_id(1).in_list?
-    end 
-
-    def test_remove_from_list_should_set_position_to_nil 
-      assert_equal [1, 2, 3, 4], mixins_with_parent_id(5)
-
-      mixin_with_original_id(2).remove_from_list 
-
-      assert_equal [2, 1, 3, 4], mixins_with_parent_id(5)
-
-      assert_equal 1,   mixin_with_original_id(1).pos
-      assert_equal nil, mixin_with_original_id(2).pos
-      assert_equal 2,   mixin_with_original_id(3).pos
-      assert_equal 3,   mixin_with_original_id(4).pos
-    end 
-
-    def test_remove_before_destroy_does_not_shift_lower_items_twice 
-      assert_equal [1, 2, 3, 4], mixins_with_parent_id(5)
-
-      mixin_with_original_id(2).remove_from_list 
-      mixin_with_original_id(2).destroy 
-
-      assert_equal [1, 3, 4], mixins_with_parent_id(5)
-
-      assert_equal 1, mixin_with_original_id(1).pos
-      assert_equal 2, mixin_with_original_id(3).pos
-      assert_equal 3, mixin_with_original_id(4).pos
-    end 
-  
-end
-
-
-
-# ---------------------------------------------------------------------
-
-
-
-class ListSubTest < ActiveSupport::TestCase
-
-  def setup
-    (1..4).each{ |i| ((i % 2 == 1) ? ListMixinSub1 : ListMixinSub2).create! :pos => i, :parent_id => 5000, :original_id => i }
-  end
-
-  def test_reordering
-    assert_equal [1, 2, 3, 4], mixins_with_parent_id(5000)
-
-    mixin_with_original_id(2).move_lower
-    assert_equal [1, 3, 2, 4], mixins_with_parent_id(5000)
-      
-    mixin_with_original_id(2).move_higher
-    assert_equal [1, 2, 3, 4], mixins_with_parent_id(5000)
-      
-    mixin_with_original_id(1).move_to_bottom
-    assert_equal [2, 3, 4, 1], mixins_with_parent_id(5000)
-
-    mixin_with_original_id(1).move_to_top
-    assert_equal [1, 2, 3, 4], mixins_with_parent_id(5000)
-      
-    mixin_with_original_id(2).move_to_bottom
-    assert_equal [1, 3, 4, 2], mixins_with_parent_id(5000)
-      
-    mixin_with_original_id(4).move_to_top
-    assert_equal [4, 1, 3, 2], mixins_with_parent_id(5000)
-  end
-
-  def test_move_to_bottom_with_next_to_last_item
-    assert_equal [1, 2, 3, 4], mixins_with_parent_id(5000)
-    mixin_with_original_id(3).move_to_bottom
-    assert_equal [1, 2, 4, 3], mixins_with_parent_id(5000)
-  end
-
-  def test_next_prev
-    assert_equal mixin_with_original_id(2), mixin_with_original_id(1).lower_item
-    assert_nil mixin_with_original_id(1).higher_item
-    assert_equal mixin_with_original_id(3), mixin_with_original_id(4).higher_item
-    assert_nil mixin_with_original_id(4).lower_item
-  end
-
-  def test_injection
-    item = ListMixin.new(:parent_id => 1)
-    assert_equal item.scope_condition, { :parent_id => 1 }
-    assert_equal "pos", item.position_column
-  end
-
-  def test_insert_at
-    new = ListMixin.create(:parent_id => 20)
-    assert_equal 1, new.pos
- 
-    new = ListMixinSub1.create(:parent_id => 20)
-    assert_equal 2, new.pos
- 
-    new = ListMixinSub2.create(:parent_id => 20)
-    assert_equal 3, new.pos
- 
-    new4 = ListMixin.create(:parent_id => 20)
-    assert_equal 4, new4.pos
- 
-    new4.insert_at(3)
-    assert_equal 3, new4.pos
- 
-    new.reload
-    assert_equal 4, new.pos
- 
-    new.insert_at(2)
-    assert_equal 2, new.pos
- 
-    new4.reload
-    assert_equal 4, new4.pos
- 
-    new5 = ListMixinSub1.create(:parent_id => 20)
-    assert_equal 5, new5.pos
- 
-    new5.insert_at(1)
-    assert_equal 1, new5.pos
- 
-    new4.reload
-    assert_equal 5, new4.pos
-  end
-
-  def test_delete_middle
-    assert_equal [1, 2, 3, 4], mixins_with_parent_id(5000)
- 
-    mixin_with_original_id(2).destroy
- 
-    assert_equal [1, 3, 4], mixins_with_parent_id(5000)
- 
-    assert_equal 1, mixin_with_original_id(1).pos
-    assert_equal 2, mixin_with_original_id(3).pos
-    assert_equal 3, mixin_with_original_id(4).pos
- 
-    mixin_with_original_id(1).destroy
- 
-    assert_equal [3, 4], mixins_with_parent_id(5000)
- 
-    assert_equal 1, mixin_with_original_id(3).pos
-    assert_equal 2, mixin_with_original_id(4).pos
-  end
-
-end
-
-
-
-# ---------------------------------------------------------------------
-
-

data/test/test_helper.rb

@@ -1,32 +0,0 @@
-require 'rubygems'
-$:.unshift File.expand_path(File.dirname(__FILE__) + '/../lib')
-
-require 'mongo_mapper'
-require 'mongo_mapper/plugins/acts_as_list'
-require 'test/unit'
-require 'ruby-debug'
-
-
-
-class ActiveSupport::TestCase
-  
-  # Drop all collections after each test case.
-  def teardown
-    MongoMapper.database.collections.each { |coll| coll.remove }
-  end
-
-  # Make sure that each test case has a teardown
-  # method to clear the db after each test.
-  def inherited(base)
-    base.define_method teardown do 
-      super
-    end
-  end
-
-end
-
-
-
-MongoMapper.connection = Mongo::Connection.new('127.0.0.1', 27017)
-MongoMapper.database = "mongo_mapper_acts_as_list_test"
-MongoMapper.database.collections.each { |c| c.drop_indexes }