Merge pull request #754 from estolfo/RUBY-1093-neg-limit

RUBY-1093 Negative limits become their absolute value in find commands

Author: estolfo

lib/mongo/collection/view/builder/find_command.rb

@@ -100,7 +100,7 @@ def find_command
 
           def convert_negative_limit(command)
             if command[:limit] && command[:limit] < 0
-              command.delete('limit')
+              command['limit'] = command['limit'].abs
               command[:singleBatch] = true
             end
             command

spec/mongo/collection/view/builder/find_command_spec.rb

@@ -159,8 +159,8 @@
         expect(selector['singleBatch']).to be true
       end
 
-      it 'removes the limit from the command' do
-        expect(selector['limit']).to be_nil
+      it 'converts the limit to a positive value' do
+        expect(selector['limit']).to be(options[:limit].abs)
       end
     end
 

spec/mongo/cursor_spec.rb

@@ -136,14 +136,10 @@
               Mongo::Collection::View.new(authorized_collection, {}, :limit => -2)
             end
 
-            it 'returns the positive number of documents', unless: find_command_enabled? do
+            it 'returns the positive number of documents' do
               expect(cursor.to_a.count).to eq(2)
             end
 
-            it 'returns all documents', if: find_command_enabled? do
-              expect(cursor.to_a.count).to eq(10)
-            end
-
             it 'iterates the documents' do
               cursor.each do |doc|
                 expect(doc).to have_key('field')