mongoid_listable 0.1.0 → 0.1.3

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    M2JkMzNmNGMyODlkMTEwMzRkZjhkNjcwOGE2ZmEyZDU5YmI3NmE2Zg==
+    OGY4YzBjMjJiMzExZjI2M2M2YTFmNTBkMjM5MmY0ZTMyMjNjNzYwZQ==
   data.tar.gz: !binary |-
-    MmJjNDY1OGIyOThmNmNkMTVhN2NiNGQ3YzVkYjkwNTdhMDU2YzkzZQ==
+    YmVlYmFjMDk1YmE2NjE4ZjkxZDgwZTYwMzhmOWU1ZTI2ZTcyMGZjMw==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    YzBhYmRkYzMwN2FkNDlhZjNkY2Q0MzI1MzAwNmE4ZjFkZTMxMzNjN2VkNmU3
-    MzNkZmUxZTEzZmE3YjA2ZjVhNjgzMmIzMDIwODE2MWI2Yzk3OGMyYmJkYjkz
-    NWFjN2RjYTIzN2I3MmZmYTZmNWI3ZDQyOWEyZDc0YzBhNDUzZmU=
+    OTliYmRiODE5MTlmYzA3OTM0MjE2MWMyYzg1OWZiMGIzYmY1ZTIyYzdlZmVh
+    MDBhMTcyMjdjMDIxZjdhOTc0OTU0YWNhMGM2YjAwNDlkNjdiOTNkM2M0OTIw
+    YTBmOTRmNzRiOWQwNTE4MDYwNTI4ZmUzNzlhOTQxMjVhOWU2N2Q=
   data.tar.gz: !binary |-
-    YzI5MGM3MTZjYjg2MWY3ZjQzNTExNGYzMWJmNzA3ZjFkNWZlZTlmMzdlYjQ4
-    MWVhZWVkMTQ3NGU3ZjFiODI0NTU0NDQ0MWZjNzhkNGM3MGIzMTFiYjliMTRi
-    M2Q4NDk5MWRlODA5MWIyOTdmOWZhZjFiZjYyODg1NDhkNWFlMTg=
+    YWVhZGRiNGFhZGQ3ZjA4MmE0NWQ3ZDdiNjIxYmI5NDk5OTk3OWFiYzdhNTNl
+    MGY1NDg0ODhkYzNkNTU5MTg4ZTYxNjBjMDc2MWNkNGNlNTc2ZGJmMzdhYzAx
+    OWQzYTRlYWY1MDhiYWEwMmI2YTRiNTZhOGU2NmJhZWRiYWU5OGI=

data/.rspec

@@ -0,0 +1,2 @@
+--color
+--format documentation

data/README.md

@@ -4,11 +4,28 @@
 [![Coverage Status](https://coveralls.io/repos/richardcalahan/mongoid_listable/badge.png?branch=master)](https://coveralls.io/r/richardcalahan/mongoid_listable?branch=master)
 [![Gem Version](https://badge.fury.io/rb/mongoid_listable.png)](http://badge.fury.io/rb/mongoid_listable)
 
-Mongoid Listable will eventually be a full replacement library for Mongoid List or Mongoid Orderable. Both 
-libraries fail to accomplish the simple task this library handles: separate position scopes for each
-defined `has_many` / `belongs_to` relation.
+Mongoid Listable manages lists for isolated collections or for more complex `has_many` / `belongs_to` relationships.
+There are two main macros:   
+
+`listed` for isolated lists not belonging to any parent object.   
+`lists`  for `has_many` / `belongs_to` relationships
+
+
+## Basic Usage - Isolated
+
+    class Photo
+      include Mongoid::Document
+      include Mongoid::Listable
+      
+      listed
+    end
+    
+The `listed` macro will assign a `position` field and a `list` scope to the Photo class. All Photo instances 
+added or removed, (or whose position fields are updated) will trigger automatic reording of all sibling instances.
+    
+    
+## Basic Usage - Has Many
 
-## Basic Usage
 
     class User
       include Mongoid::Document
@@ -26,16 +43,31 @@ defined `has_many` / `belongs_to` relation.
       ...
     end
     
-In this example photos that are assigned to a user will maintain position based on the index
-of the id in the array argument.
+In this example photos that are added to or removed from a user, or have their position attribute updated
+will maintain logical order based on the method used:
+
+    # setter
+    User.first.photos = [ photo_a, photo_c, photo_b ]
+    User.first.photos.last == photo_b #=> true
+    
+    # ids setter
+    User.first.photo_ids = [ '527fe97c67df6f07e1000003', '527fe97c67df6f07e1000004', '527fe97c67df6f07e1000003' ]
+    User.first.photos[1].id == '527fe97c67df6f07e1000004' #=> true
+    
+    # push
+    photo = Photo.create
+    User.first.photos << photo
+    User.first.photos.last == photo #=> true
+    
+    
 
-Each photo that belongs to the user will automatically obtain a field called `user_position`. The position field
-is derived from the foreign key of the relation, replacing "\_id" with "_position". 
+Each photo that belongs to the user will automatically obtain a field called `user_position`. The field
+is derived from the foreign key of the relation, replacing "\_id" with "_position".
 
-The 1-n relation of a user to their photos will automatically be ordered by `user_position` unless otherwise specified
+The `has_many` relation of a user to their photos will automatically be ordered by `user_position` unless otherwise specified
 via the standard `order` option to the `has_many` macro. 
     
-## Complex Relations
+## Advanced Usage - Has Many
 
     # Handles multiple has_many relations on same model!
     
@@ -77,13 +109,10 @@ via the standard `order` option to the `has_many` macro.
 In this example, there are two `has_many` relations defined between a user and photos. Each photo belonging to a user will 
 obtain two position fields: `featured_by_user_position` and `kodaked_by_user_position`.
 
-You can optionally override the name of the position column:
-
-    lists :photos, column: :users_photos_order
 
 ## Todo
 
-There's a lot more to add to the library! At this point, it conveniently handles has many relationships.
+Need to test embedded documents.
 
 ## Installation
 

data/lib/mongoid/listable.rb

@@ -49,24 +49,17 @@ module Mongoid
       send(name).uniq(&:id).count
     end
 
-    private
-
-    # Applies a position change on column. Which objects are repositioned
-    # depends on the direction of the change.
+    # Retrieves an object's list siblings
     #
-    # @param [ Symbol ] name The name of position column
+    # @return [ Array ]
     #
     # @since 0.1.0
-    def apply_change_on column
-      from, to = change_on column
-      if to > from
-        reposition siblings.between(column => from..to), column, from
-      elsif to < from       
-        reposition siblings.between(column => to..from), column, to + 1
-      end
-      set column, to
+    def siblings field=:position
+      self.class.exists(field => true).ne id: id
     end
 
+    private
+
     # Resets column on objects starting at 'start'
     #
     # @param [ Array ] objects The objects to interate
@@ -80,42 +73,5 @@ module Mongoid
       end
     end
 
-    # Returns the old and new values for column
-    #
-    # @param [ Symbol ] column The column to retrieve the change
-    # @return [Array] [from, to]
-    #
-    # @since 0.1.0
-    def change_on column
-      from, to = send "#{column}_change"
-      to = safe_to to
-      [from, to]
-    end
-
-    # Ensures the 'to' value is within acceptable bounds
-    #
-    # @param [ Integer ] to The supplied position value
-    # @return [ Integer ] The acceptable position value
-    #
-    # @since 0.1.0
-    def safe_to to
-      if to > self.class.count
-        self.class.count
-      elsif to < 1
-        1
-      else
-        to
-      end
-    end
-
-    # Retrieves an object's list siblings
-    #
-    # @return [ Array ]
-    #
-    # @since 0.1.0
-    def siblings
-      self.class.list.ne id: id
-    end
-
   end
 end

data/lib/mongoid/listable/accessors.rb

@@ -16,11 +16,12 @@ module Mongoid
         # @return [ Object ] self
         #
         # @since 0.0.6
-        def _ids_setter name, meta
+        def ids_set name, meta
           ids_method = "#{name.to_s.singularize}_ids="
           redefine_method ids_method do |ids|            
             send meta.setter, meta.klass.find(ids).sort_by_attr(:id, ids)
           end
+
           self
         end
 
@@ -33,17 +34,16 @@ module Mongoid
         # @return [ Object ] self
         #
         # @since 0.0.6
-        def _setter name, meta
+        def set name, meta
           before_method "#{name}=" do |objects|
             objects ||= []
-            objects.each_with_index do |object, index|
-              object.set field_name(meta), index + 1
-            end
+            reposition objects, field_name(meta), 1
 
             (send(name).to_a - objects).each do |object|
               object.unset field_name(meta)
             end
           end
+
           self
         end
 

data/lib/mongoid/listable/callbacks.rb

@@ -6,104 +6,160 @@ module Mongoid
 
       module ClassMethods
 
-        # Defines a mongoid relation after_add callback.
-        # Sets the position attribute to current relations length + 1
+        # Defines a mongoid before_create callback.
+        # Sets the position field to current object count + 1
         #
-        # @param [ Symbol ]   name The name of the has_many relation
-        # @param [ MetaData ] meta The MetaData class
+        # @param [ Hash ] The configuration hash
         #
         # @return [ Object ] self
         #
-        # @since 0.0.6
-        def added name, meta
-          callback = "#{name.to_s.singularize}_added"
-          define_method callback do |object|
-            if object[field_name(meta)].nil?
-              object.set field_name(meta), has_many_count(name) + 1
+        # @since 0.1.0
+        def created name
+          callback = "#{name}_#{__method__}"
+          define_method callback do
+            position = send name
+            if position.present?
+              siblings = siblings(name).gte name => position
+              reposition siblings, name, position + 1
+            else
+              set name, siblings(name).count + 1
             end
           end
-          meta[:before_add] = callback
+
+          before_create callback
           self
         end
 
-        # Defines a mongoid relation before_remove callback.
-        # Resets the position index on all objects that came after
+        # Defines a mongoid before_update callback.        
+        # If the position column has changed, apply the change.
+        # Hoe the change is applied varies depending on the redrection
+        # of the update.
         #
-        # @param [ Symbol ]   name The name of the has_many relation
-        # @param [ MetaData ] meta The MetaData class
+        # @param [ Hash ] The configuration hash
         #
         # @return [ Object ] self
         #
-        # @since 0.0.6
-        def removed name, meta       
-          field_name = field_name meta
-          callback   = "#{name.to_s.singularize}_removed"
-
-          define_method callback do |object|
-            position = object.send field_name
-            send(name).where(field_name.gt => position)
-              .each_with_index do |object, index|
-              object.set field_name, position + index
-            end
-            object.unset field_name
+        # @since 0.1.0
+        def updated name
+          callback = "#{name}_#{__method__}"
+          define_method callback do       
+            apply_change_on name if send("#{name}_changed?")
           end
 
-          meta[:before_remove] = callback
+          before_update callback
           self
         end
 
-        # Defines a mongoid before_create callback.
-        # Sets the position field to current object count + 1
+        # Defines a mongoid before_destroy callback.        
+        # Resets all sibling object's higher in the list
         #
         # @param [ Hash ] The configuration hash
         #
         # @return [ Object ] self
         #
         # @since 0.1.0
-        def created configuration          
-          define_method __method__ do             
-            set configuration[:column], self.class.count + 1
+        def destroyed name
+          callback = "#{name}_#{__method__}"
+          define_method callback do
+            position = send name
+            siblings = siblings(name).gt(name => position)
+            reposition siblings, name, position
           end
-          before_create __method__
+
+          before_destroy callback
           self
         end
 
-        # Defines a mongoid before_update callback.        
-        # If the position column has changed, apply the change.
-        # Hoe the change is applied varies depending on the redrection
-        # of the update.
+        # Defines a mongoid has_many relation after_add callback.
+        # Sets the position attribute to current relations length + 1
         #
-        # @param [ Hash ] The configuration hash
+        # @param [ Symbol ]   name The name of the has_many relation
+        # @param [ MetaData ] meta The MetaData class
         #
         # @return [ Object ] self
         #
-        # @since 0.1.0
-        def updated configuration
-          define_method __method__ do             
-            column = configuration[:column]            
-            apply_change_on column if send("#{column}_changed?")
+        # @since 0.0.6
+        def added name, meta
+          callback = "#{name.to_s.singularize}_added"
+          define_method callback do |object|
+            if object[field_name(meta)].nil?
+              object.set field_name(meta), has_many_count(name) + 1
+            end
           end
-          before_update __method__
+          meta[:before_add] = callback
           self
         end
 
-        # Defines a mongoid before_destroy callback.        
-        # Resets all sibling object's higher in the list
+        # Defines a mongoid has_many relation before_remove callback.
+        # Resets the position index on all objects that came after
         #
-        # @param [ Hash ] The configuration hash
+        # @param [ Symbol ]   name The name of the has_many relation
+        # @param [ MetaData ] meta The MetaData class
         #
         # @return [ Object ] self
         #
-        # @since 0.1.0
-        def destroyed configuration
-          define_method __method__ do             
-            column = configuration[:column]
-            reposition siblings.gt(column => position), column, position
+        # @since 0.0.6
+        def removed name, meta       
+          field_name = field_name meta
+          callback   = "#{name.to_s.singularize}_removed"
+          define_method callback do |object|
+            position = object.send field_name
+            reposition object.siblings(field_name).gt(field_name => position), 
+            field_name, position
+            object.unset field_name
           end
-          before_destroy __method__
+
+          meta[:before_remove] = callback
           self
         end
 
+      end # ClassMethods
+
+      private
+
+      # Applies a position change on column. Which objects are repositioned
+      # depends on the direction of the change.
+      #
+      # @param [ Symbol ] name The name of position column
+      #
+      # @since 0.1.0
+      def apply_change_on name
+        from, to = change_on name
+        siblings = siblings name
+        if to > from
+          reposition siblings.between(name => from..to), name, from
+        elsif to < from       
+          reposition siblings.between(name => to..from), name, to + 1
+        end
+        set name, to
+      end
+
+      # Returns the old and new values for column
+      #
+      # @param [ Symbol ] column The column to retrieve the change
+      # @return [Array] [from, to]
+      #
+      # @since 0.1.0
+      def change_on column
+        from, to = send "#{column}_change"
+        to = safe_to to
+        [from, to]
+      end
+
+      # Ensures the 'to' value is within acceptable bounds
+      #
+      # @param [ Integer ] to The supplied position value
+      # @return [ Integer ] The acceptable position value
+      #
+      # @since 0.1.0
+      def safe_to to
+        if to > self.class.count
+          self.class.count
+        elsif to < 1
+          1
+        else
+          to
+        end
       end
 
     end    

data/lib/mongoid/listable/macros.rb

@@ -16,16 +16,16 @@ module Mongoid
         #
         # @since 0.1.0
         def listed options={}
-          configuration = { column: :position }
+          configuration = { field_name: :position }
           configuration.merge! options if options.is_a?(Hash)
 
-          field configuration[:column], type: Integer
+          field_name = configuration[:field_name]
 
-          created(configuration)
-            .updated(configuration)
-            .destroyed(configuration)
+          field field_name, type: Integer
 
-          scope :list, order_by(configuration[:column] => :asc)
+          created(field_name).updated(field_name).destroyed(field_name)
+
+          scope :list, order_by(field_name => :asc)
           self
         end
 
@@ -39,11 +39,13 @@ module Mongoid
         # @since 0.0.1
         def lists name, options={}
           meta = reflect_on_association name
-          meta.klass.send :field, field_name(meta), type: Integer
-          _ids_setter(name, meta)
-            ._setter(name, meta)
-            .added(name, meta)
-            .removed(name, meta)
+          field_name = field_name(meta)
+          meta.klass.send :field, field_name, type: Integer
+          ids_set(name, meta).set(name, meta)
+            .added(name, meta).removed(name, meta)
+
+          meta.klass.updated(field_name).destroyed(field_name)
+
           meta[:order] ||= "#{field_name(meta)} asc"
         end
 

data/lib/mongoid/listable/version.rb

@@ -1,5 +1,5 @@
 module Mongoid
   module Listable
-    VERSION = '0.1.0'
+    VERSION = '0.1.3'
   end
 end

data/spec/mongoid_listable_spec.rb

@@ -1,126 +1,162 @@
 require 'spec_helper'
 
-describe Mongoid::Listable do 
+describe Mongoid::Listable do   
 
-  before :all do 
-    User.create!    
-    10.times { Photo.create }
-  end  
-
-  it 'created a user' do
-    expect(User.first).to be_true
+  def ensure_order objects, field
+    objects.each_with_index do |object, index|
+      expect(object[field]).to eq(index + 1)
+    end
   end
 
-  it 'created 10 photos' do    
-    expect(Photo.count).to eq(10)
-  end
+  describe 'listed' do 
 
-  it 'added the position field for a photo' do 
-    expect(Photo.instance_methods.include?(:position)).to be_true
-  end
+    before :all do
+      Photo.destroy_all
+      10.times { Photo.create }
+    end
 
-  it 'orders photos' do 
-    Photo.list.each_with_index do |photo, index|
-      expect(photo.position).to eq(index + 1)
+    after :each do 
+      ensure_order Photo.list, :position
     end
-  end
 
-  it 'updates photo order on position change higher' do
-    photo_a_id = Photo.list[3].id
-    photo_b_id = Photo.list[7].id
+    it 'adds an object to the beginning of a list on create' do 
+      photo = Photo.create position: 1
+      expect(Photo.list.first.position).to eq(1)
+      expect(photo.id).to eq(Photo.list.first.id)
+    end
 
-    expect(Photo.find(photo_a_id).position).to eq(4)
-    expect(Photo.find(photo_b_id).position).to eq(8)
+    it 'adds an object to the middle of a list on create' do 
+      photo = Photo.create position: 5
+      expect(Photo.list[4].position).to eq(5)
+      expect(photo.id).to eq(Photo.list[4].id)
+    end
 
-    Photo.find(photo_a_id).update_attribute :position, 9
+    it 'adds an object to the end of a list on create' do 
+      photo = Photo.create
+      expect(Photo.list.last.position).to eq(13)
+      expect(photo.id).to eq(Photo.list.last.id)
+    end
 
-    expect(Photo.find(photo_a_id).position).to eq(9)
-    expect(Photo.find(photo_b_id).position).to eq(7)
-  end
+    it 'updates the position of an object higher' do 
+      photo_id = Photo.list[1].id
+      Photo.list[1].update_attribute :position, 4
+      expect(photo_id).to eq(Photo.list[3].id)
+      expect(Photo.list[3].position).to eq(4)
+    end
 
-  it 'updates photo order on position change lower' do
-    photo_a_id = Photo.list[5].id
-    photo_b_id = Photo.list[9].id
+    it 'updates the position of an object lower' do 
+      photo_id = Photo.list[9].id
+      Photo.list[9].update_attribute :position, 2
+      expect(photo_id).to eq(Photo.list[1].id)
+      expect(Photo.list[1].position).to eq(2)      
+    end
 
-    expect(Photo.find(photo_a_id).position).to eq(6)
-    expect(Photo.find(photo_b_id).position).to eq(10)
+    it 'updates the position of an object the same' do 
+      photo_id = Photo.list[4].id
+      Photo.list[4].update_attribute :position, 5
+      expect(photo_id).to eq(Photo.list[4].id)
+      expect(Photo.list[4].position).to eq(5)      
+    end
 
-    Photo.find(photo_a_id).update_attribute :position, 2
+    it 'removes an object from the beginning of a list on destroy' do 
+      Photo.list.first.destroy
+      expect(Photo.list.first.position).to eq(1)
+    end
 
-    expect(Photo.find(photo_a_id).position).to eq(2)
-    expect(Photo.find(photo_b_id).position).to eq(10)
-  end
+    it 'removes an object from the middle of a list on destroy' do 
+      Photo.list[6].destroy
+      expect(Photo.list.first.position).to eq(1)
+      expect(Photo.list[6].position).to eq(7)
+      expect(Photo.list.last.position).to eq(11)
+    end
 
-  it 'updates photo order on photo destroy' do
-    photo_a_id = Photo.list[5].id
-    photo_b_id = Photo.list[9].id
+    it 'removes an object from the end of a list on destroy' do 
+      Photo.list.last.destroy
+      expect(Photo.list.last.position).to eq(10)
+    end
+  end
 
-    expect(Photo.find(photo_b_id).position).to eq(10)
+  describe 'lists' do 
 
-    Photo.find(photo_a_id).destroy
+    before :all do
+      User.destroy_all
+      Photo.destroy_all
 
-    expect(Photo.find(photo_b_id).position).to eq(9)
-  end
+      User.create
+      10.times { Photo.create }
+    end
 
-  it 'creates a new photo' do
-    Photo.create
-    expect(Photo.all.count).to eq(10)
-    expect(Photo.last.position).to eq(10)
-  end
+    after :each do 
+      ensure_order User.first.photos, :user_position
+    end
 
-  it 'adds photos to a user with the default setter' do 
-    User.first.photos = Photo.all
-    expect(User.first.photos.count).to eq(10)
-  end
+    it 'sets object list of an owner with the default setter' do
+      photos = Photo.all
+      User.first.photos = photos
+      photos.each_with_index do |photo, index|
+        expect(photo.id).to eq(User.first.photos[index].id)
+      end
+      expect(User.first.photos.count).to eq(Photo.count)
+    end
 
-  it 'orders photos for a user' do 
-    User.first.photos.each_with_index do |photo, index|
-      expect(photo.user_position).to eq(index + 1)
+    it 'sets object list of an owner with the default ids setter' do 
+      ids = Photo.all[2..7].collect(&:id)
+      User.first.photo_ids = ids
+      ids.each_with_index do |id, index|
+        expect(id).to eq(User.first.photos[index].id)
+      end
+      expect(User.first.photos.count).to eq(6)
     end
-  end
 
-  it 'removes photos from a user' do 
-    User.first.photos = nil
-    expect(User.first.photos.count).to eq(0)
-  end
+    it 'pushes objects to the list of an owner' do 
+      User.first.photos << Photo.all
+      expect(User.first.photos.count).to eq(10)
+    end
 
-  it 'adds 5 photos to a user with the default ids setter' do
-    User.first.photo_ids = Photo.all[0..4].map &:id
-    expect(User.first.photos.count).to eq(5)
-  end
+    it 'updates the position of an object higher' do 
+      photo_id = User.first.photos[4].id
+      Photo.find(photo_id).update_attribute :user_position, 6
+      expect(photo_id).to eq(User.first.photos[5].id)
+      expect(User.first.photos[5].user_position).to eq(6)
+    end
 
-  it 'adds all photos to a user with the default ids setter' do
-    User.first.photo_ids = Photo.all.map &:id
-    expect(User.first.photos.count).to eq(10)
-  end
+    it 'updates the position of an object lower' do 
+      photo_id = User.first.photos[4].id
+      Photo.find(photo_id).update_attribute :user_position, 1
+      expect(photo_id).to eq(User.first.photos[0].id)
+      expect(User.first.photos[0].user_position).to eq(1)
+    end
 
-  it 'orders photos for a user' do 
-    User.first.photos.each_with_index do |photo, index|
-      expect(photo.user_position).to eq(index + 1)
+    it 'updates the position of an object the same' do 
+      photo_id = User.first.photos[4].id
+      Photo.find(photo_id).update_attribute :user_position, 5
+      expect(photo_id).to eq(User.first.photos[4].id)
+      expect(User.first.photos[4].user_position).to eq(5)
     end
-  end
 
-  it 'adds a new photo to a user' do 
-    User.first.photos << Photo.new
-    expect(User.first.photos.count).to eq(11)
-    expect(User.first.photos.last.user_position).to eq(11)
-  end
+    it 'updates the position of an object out of bounds high' do 
+      photo_id = User.first.photos[4].id
+      Photo.find(photo_id).update_attribute :user_position, 1000
+      expect(photo_id).to eq(User.first.photos.last.id)
+      expect(User.first.photos.last.user_position).to eq(User.first.photos.count)
+    end
 
-  it 'adds a created photo to a user' do 
-    User.first.photos << Photo.create
-    expect(User.first.photos.count).to eq(12)
-    expect(User.first.photos.last.user_position).to eq(12)
-  end
+    it 'updates the position of an object out of bounds low' do 
+      photo_id = User.first.photos[4].id
+      Photo.find(photo_id).update_attribute :user_position, -2
+      expect(photo_id).to eq(User.first.photos.first.id)
+      expect(User.first.photos.first.user_position).to eq(1)
+    end
 
-  it 'deletes a photo from a user' do 
-    User.first.photos.delete(Photo.all[5])
-    expect(User.first.photos.count).to eq(11)
-    expect(User.first.photos.last.user_position).to eq(11)
-  end
+    it 'removes objects from the list of an owner with the default unsetter' do 
+      User.first.photos.delete(Photo.first)
+      expect(User.first.photos.count).to eq(9)
+    end
 
-  it 'orders photos for a user' do 
-    User.first.photos.each_with_index do |photo, index|
-      expect(photo.user_position).to eq(index + 1)
+    it 'removes objects from the list of an owner by destroy' do 
+      Photo.all[3].destroy
+      expect(User.first.photos.count).to eq(8)
+      Photo.first.destroy
     end
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mongoid_listable
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.3
 platform: ruby
 authors:
 - richardcalahan
@@ -61,6 +61,7 @@ extra_rdoc_files: []
 files:
 - .coveralls.yml
 - .gitignore
+- .rspec
 - .travis.yml
 - Gemfile
 - LICENSE.txt