mongo 2.1.0.beta → 2.2.0

data/spec/mongo/collection/view/builder/modifiers_spec.rb

@@ -0,0 +1,210 @@
+require 'spec_helper'
+
+describe Mongo::Collection::View::Builder::Modifiers do
+
+  describe '.map_driver_options' do
+
+    shared_examples_for 'transformable driver options' do
+
+      it 'maps hint' do
+        expect(transformed[:hint]).to eq("_id" => 1)
+      end
+
+      it 'maps comment' do
+        expect(transformed[:comment]).to eq('testing')
+      end
+
+      it 'maps max scan' do
+        expect(transformed[:max_scan]).to eq(200)
+      end
+
+      it 'maps max time ms' do
+        expect(transformed[:max_time_ms]).to eq(500)
+      end
+
+      it 'maps max' do
+        expect(transformed[:max_value]).to eq("name" => 'joe')
+      end
+
+      it 'maps min' do
+        expect(transformed[:min_value]).to eq("name" => 'albert')
+      end
+
+      it 'maps return key' do
+        expect(transformed[:return_key]).to be true
+      end
+
+      it 'maps show record id' do
+        expect(transformed[:show_disk_loc]).to be true
+      end
+
+      it 'maps snapshot' do
+        expect(transformed[:snapshot]).to be true
+      end
+
+      it 'maps explain' do
+        expect(transformed[:explain]).to be true
+      end
+
+      it 'returns a BSON document' do
+        expect(transformed).to be_a(BSON::Document)
+      end
+    end
+
+    context 'when the keys are strings' do
+
+      let(:modifiers) do
+        {
+          '$orderby' => { name: 1 },
+          '$hint' => { _id: 1 },
+          '$comment' => 'testing',
+          '$snapshot' => true,
+          '$maxScan' => 200,
+          '$max' => { name: 'joe' },
+          '$min' => { name: 'albert' },
+          '$maxTimeMS' => 500,
+          '$returnKey' => true,
+          '$showDiskLoc' => true,
+          '$explain' => true
+        }
+      end
+
+      let(:transformed) do
+        described_class.map_driver_options(modifiers)
+      end
+
+      it_behaves_like 'transformable driver options'
+    end
+
+    context 'when the keys are symbols' do
+
+      let(:modifiers) do
+        {
+          :$orderby => { name: 1 },
+          :$hint => { _id: 1 },
+          :$comment => 'testing',
+          :$snapshot => true,
+          :$maxScan => 200,
+          :$max => { name: 'joe' },
+          :$min => { name: 'albert' },
+          :$maxTimeMS => 500,
+          :$returnKey => true,
+          :$showDiskLoc => true,
+          :$explain => true
+        }
+      end
+
+      let(:transformed) do
+        described_class.map_driver_options(modifiers)
+      end
+
+      it_behaves_like 'transformable driver options'
+    end
+  end
+
+  describe '.map_server_modifiers' do
+
+    shared_examples_for 'transformable server modifiers' do
+
+      it 'maps hint' do
+        expect(transformed[:$hint]).to eq("_id" => 1)
+      end
+
+      it 'maps comment' do
+        expect(transformed[:$comment]).to eq('testing')
+      end
+
+      it 'maps max scan' do
+        expect(transformed[:$maxScan]).to eq(200)
+      end
+
+      it 'maps max time ms' do
+        expect(transformed[:$maxTimeMS]).to eq(500)
+      end
+
+      it 'maps max' do
+        expect(transformed[:$max]).to eq("name" => 'joe')
+      end
+
+      it 'maps min' do
+        expect(transformed[:$min]).to eq("name" => 'albert')
+      end
+
+      it 'maps return key' do
+        expect(transformed[:$returnKey]).to be true
+      end
+
+      it 'maps show record id' do
+        expect(transformed[:$showDiskLoc]).to be true
+      end
+
+      it 'maps snapshot' do
+        expect(transformed[:$snapshot]).to be true
+      end
+
+      it 'maps explain' do
+        expect(transformed[:$explain]).to be true
+      end
+
+      it 'returns a BSON document' do
+        expect(transformed).to be_a(BSON::Document)
+      end
+
+      it 'does not include non modifiers' do
+        expect(transformed[:limit]).to be_nil
+      end
+    end
+
+    context 'when the keys are strings' do
+
+      let(:options) do
+        {
+          'sort' => { name: 1 },
+          'hint' => { _id: 1 },
+          'comment' => 'testing',
+          'snapshot' => true,
+          'max_scan' => 200,
+          'max_value' => { name: 'joe' },
+          'min_value' => { name: 'albert' },
+          'max_time_ms' => 500,
+          'return_key' => true,
+          'show_disk_loc' => true,
+          'explain' => true,
+          'limit' => 10
+        }
+      end
+
+      let(:transformed) do
+        described_class.map_server_modifiers(options)
+      end
+
+      it_behaves_like 'transformable server modifiers'
+    end
+
+    context 'when the keys are symbols' do
+
+      let(:options) do
+        {
+          :sort => { name: 1 },
+          :hint => { _id: 1 },
+          :comment => 'testing',
+          :snapshot => true,
+          :max_scan => 200,
+          :max_value => { name: 'joe' },
+          :min_value => { name: 'albert' },
+          :max_time_ms => 500,
+          :return_key => true,
+          :show_disk_loc => true,
+          :explain => true,
+          :limit => 10
+        }
+      end
+
+      let(:transformed) do
+        described_class.map_server_modifiers(options)
+      end
+
+      it_behaves_like 'transformable server modifiers'
+    end
+  end
+end

data/spec/mongo/collection/view/builder/op_query_spec.rb

@@ -0,0 +1,154 @@
+require 'spec_helper'
+
+describe Mongo::Collection::View::Builder::OpQuery do
+
+  describe '#specification' do
+
+    let(:filter) do
+      { 'name' => 'test' }
+    end
+
+    let(:builder) do
+      described_class.new(view)
+    end
+
+    let(:specification) do
+      builder.specification
+    end
+
+    let(:view) do
+      Mongo::Collection::View.new(authorized_collection, filter, options)
+    end
+
+    context 'when there are modifiers in the options' do
+
+      let(:options) do
+        {
+          sort: { _id: 1 },
+          projection: { name: 1 },
+          hint: { name: 1 },
+          skip: 10,
+          limit: 20,
+          batch_size: 5,
+          single_batch: false,
+          comment: "testing",
+          max_scan: 200,
+          max_time_ms: 40,
+          max_value: { name: 'joe' },
+          min_value: { name: 'albert' },
+          return_key: true,
+          show_disk_loc: true,
+          snapshot: true,
+          tailable: true,
+          oplog_replay: true,
+          no_cursor_timeout: true,
+          tailable_await: true,
+          allow_partial_results: true,
+          read_concern: { level: 'local' }
+        }
+      end
+
+      let(:selector) do
+        specification[:selector]
+      end
+
+      let(:opts) do
+        specification[:options]
+      end
+
+      let(:flags) do
+        opts[:flags]
+      end
+
+      it 'maps the collection name' do
+        expect(specification[:coll_name]).to eq(authorized_collection.name)
+      end
+
+      it 'maps the filter' do
+        expect(selector['$query']).to eq(filter)
+      end
+
+      it 'maps sort' do
+        expect(selector['$orderby']).to eq('_id' => 1)
+      end
+
+      it 'maps projection' do
+        expect(opts['project']).to eq('name' => 1)
+      end
+
+      it 'maps hint' do
+        expect(selector['$hint']).to eq('name' => 1)
+      end
+
+      it 'maps skip' do
+        expect(opts['skip']).to eq(10)
+      end
+
+      it 'maps limit' do
+        expect(opts['limit']).to eq(20)
+      end
+
+      it 'maps batch size' do
+        expect(opts['batch_size']).to eq(5)
+      end
+
+      it 'maps comment' do
+        expect(selector['$comment']).to eq('testing')
+      end
+
+      it 'maps max scan' do
+        expect(selector['$maxScan']).to eq(200)
+      end
+
+      it 'maps max time ms' do
+        expect(selector['$maxTimeMS']).to eq(40)
+      end
+
+      it 'maps max' do
+        expect(selector['$max']).to eq('name' => 'joe')
+      end
+
+      it 'maps min' do
+        expect(selector['$min']).to eq('name' => 'albert')
+      end
+
+      it 'does not map read concern' do
+        expect(selector['$readConcern']).to be_nil
+        expect(selector['readConcern']).to be_nil
+        expect(opts['readConcern']).to be_nil
+      end
+
+      it 'maps return key' do
+        expect(selector['$returnKey']).to be true
+      end
+
+      it 'maps show record id' do
+        expect(selector['$showDiskLoc']).to be true
+      end
+
+      it 'maps snapshot' do
+        expect(selector['$snapshot']).to be true
+      end
+
+      it 'maps tailable' do
+        expect(flags).to include(:tailable_cursor)
+      end
+
+      it 'maps oplog replay' do
+        expect(flags).to include(:oplog_replay)
+      end
+
+      it 'maps no cursor timeout' do
+        expect(flags).to include(:no_cursor_timeout)
+      end
+
+      it 'maps await data' do
+        expect(flags).to include(:await_data)
+      end
+
+      it 'maps allow partial results' do
+        expect(flags).to include(:partial)
+      end
+    end
+  end
+end

data/spec/mongo/collection/view/explainable_spec.rb

@@ -25,8 +25,7 @@ describe Mongo::Collection::View::Explainable do
     end
 
     it 'executes an explain' do
-      expect(explain[:cursor] == 'BasicCursor' ||
-             explain[:queryPlanner]).to be_truthy
+      expect(explain[:cursor] == 'BasicCursor' || explain[:queryPlanner]).to be_truthy
     end
   end
 end

data/spec/mongo/collection/view/immutable_spec.rb

@@ -0,0 +1,54 @@
+require 'spec_helper'
+
+describe Mongo::Collection::View::Immutable do
+
+  let(:selector) do
+    {}
+  end
+
+  let(:options) do
+    {}
+  end
+
+  let(:view) do
+    Mongo::Collection::View.new(authorized_collection, selector, options)
+  end
+
+  after do
+    authorized_collection.delete_many
+  end
+
+  describe '#configure' do
+
+    context 'when the options have modifiers' do
+
+      let(:options) do
+        { :max_time_ms => 500 }
+      end
+
+      let(:new_view) do
+        view.projection(_id: 1)
+      end
+
+      it 'returns a new view' do
+        expect(view).not_to be(new_view)
+      end
+
+      it 'creates a new options hash' do
+        expect(view.options).not_to be(new_view.options)
+      end
+
+      it 'keeps the modifier fields already in the options hash' do
+        expect(new_view.modifiers[:$maxTimeMS]).to eq(500)
+      end
+
+      it 'sets the option' do
+        expect(new_view.projection).to eq('_id' => 1)
+      end
+
+      it 'creates a new modifiers document' do
+        expect(view.modifiers).not_to be(new_view.modifiers)
+      end
+    end
+  end
+end

data/spec/mongo/collection/view/map_reduce_spec.rb

@@ -66,6 +66,39 @@ describe Mongo::Collection::View::MapReduce do
       end
     end
 
+    context 'when out is in the options' do
+
+      after do
+        authorized_client['output_collection'].delete_many
+      end
+
+      context 'when out is a string' do
+
+        let(:options) do
+          { :out => 'output_collection' }
+        end
+
+        it 'iterates over the documents in the result' do
+          map_reduce.each do |document|
+            expect(document[:value]).to_not be_nil
+          end
+        end
+      end
+
+      context 'when out is a document' do
+
+        let(:options) do
+          { :out => {  replace: 'output_collection' } }
+        end
+
+        it 'iterates over the documents in the result' do
+          map_reduce.each do |document|
+            expect(document[:value]).to_not be_nil
+          end
+        end
+      end
+    end
+
     context 'when out is inline' do
 
       let(:new_map_reduce) do
@@ -82,7 +115,7 @@ describe Mongo::Collection::View::MapReduce do
     context 'when out is a collection' do
 
       after do
-        authorized_client['output_collection'].find.delete_many
+        authorized_client['output_collection'].delete_many
       end
 
       context 'when the option is to replace' do
@@ -135,6 +168,17 @@ describe Mongo::Collection::View::MapReduce do
           expect(new_map_reduce.count).to eq(2)
         end
       end
+
+      context 'when the option is a collection name' do
+
+        let(:new_map_reduce) do
+          map_reduce.out('output_collection')
+        end
+
+        it 'fetches the results from the collection' do
+          expect(new_map_reduce.count).to eq(2)
+        end
+      end
     end
 
     context 'when the view has a selector' do
@@ -142,7 +186,7 @@ describe Mongo::Collection::View::MapReduce do
       context 'when the selector is basic' do
 
         let(:selector) do
-          { name: 'Berlin' }
+          { 'name' => 'Berlin' }
         end
 
         it 'applies the selector to the map/reduce' do
@@ -159,7 +203,7 @@ describe Mongo::Collection::View::MapReduce do
       context 'when the selector is advanced' do
 
         let(:selector) do
-          { :$query => { name: 'Berlin' }}
+          { :$query => { 'name' => 'Berlin' }}
         end
 
         it 'applies the selector to the map/reduce' do
@@ -229,7 +273,7 @@ describe Mongo::Collection::View::MapReduce do
   describe '#out' do
 
     let(:location) do
-      { replace: 'testing' }
+      { 'replace' => 'testing' }
     end
 
     let(:new_map_reduce) do
@@ -241,13 +285,64 @@ describe Mongo::Collection::View::MapReduce do
     end
 
     it 'includes the out value in the operation spec' do
-      expect(new_map_reduce.send(:map_reduce_spec)[:selector][:out]).to be(location)
+      expect(new_map_reduce.send(:map_reduce_spec)[:selector][:out]).to eq(location)
     end
 
     context 'when out is not defined' do
 
       it 'defaults to inline' do
-        expect(map_reduce.send(:map_reduce_spec)[:selector][:out]).to eq(inline: 1)
+        expect(map_reduce.send(:map_reduce_spec)[:selector][:out]).to eq('inline' => 1)
+      end
+    end
+
+    context 'when out is specified in the options' do
+
+      let(:location) do
+        { 'replace' => 'testing' }
+      end
+
+      let(:options) do
+        { :out => location }
+      end
+
+      it 'sets the out value' do
+        expect(map_reduce.out).to eq(location)
+      end
+
+      it 'includes the out value in the operation spec' do
+        expect(map_reduce.send(:map_reduce_spec)[:selector][:out]).to eq(location)
+      end
+    end
+
+    context 'when out is not inline' do
+
+      let(:location) do
+        { 'replace' => 'testing' }
+      end
+
+      let(:options) do
+        { :out => location }
+      end
+
+      it 'does not allow the operation on a secondary' do
+        expect(map_reduce.send(:secondary_ok?)).to be false
+      end
+
+      context 'when the context is not a valid server for writing' do
+
+        it 'reroutes the operation to a primary' do
+          allow(map_reduce).to receive(:valid_server?).and_return(false)
+          expect(Mongo::Logger.logger).to receive(:warn?).and_call_original
+          map_reduce.to_a
+        end
+      end
+
+      context 'when the context is a valid server for writing' do
+
+        it 'does not reroute the operation to a primary' do
+          expect(Mongo::Logger.logger).not_to receive(:warn?)
+          map_reduce.to_a
+        end
       end
     end
   end
@@ -255,7 +350,7 @@ describe Mongo::Collection::View::MapReduce do
   describe '#scope' do
 
     let(:object) do
-      { value: 'testing' }
+      { 'value' => 'testing' }
     end
 
     let(:new_map_reduce) do
@@ -267,7 +362,7 @@ describe Mongo::Collection::View::MapReduce do
     end
 
     it 'includes the scope object in the operation spec' do
-      expect(new_map_reduce.send(:map_reduce_spec)[:selector][:scope]).to be(object)
+      expect(new_map_reduce.send(:map_reduce_spec)[:selector][:scope]).to eq(object)
     end
   end
 
@@ -316,7 +411,7 @@ describe Mongo::Collection::View::MapReduce do
     end
 
     it 'includes the sort object in the operation spec' do
-      expect(map_reduce.send(:map_reduce_spec)[:selector][:sort]).to be(sort)
+      expect(map_reduce.send(:map_reduce_spec)[:selector][:sort][:name]).to eq(sort[:name])
     end
   end