mongo 2.1.0.beta → 2.2.0

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

@@ -18,6 +18,40 @@ describe Mongo::Collection::View::Readable do
     authorized_collection.delete_many
   end
 
+  shared_examples_for 'a read concern aware operation' do
+
+    context 'when a read concern is provided', if: find_command_enabled? do
+
+      let(:new_view) do
+        Mongo::Collection::View.new(new_collection, selector, options)
+      end
+
+      context 'when the read concern is valid' do
+
+        let(:new_collection) do
+          authorized_collection.with(read_concern: { level: 'local' })
+        end
+
+        it 'sends the read concern' do
+          expect { result }.to_not raise_error
+        end
+      end
+
+      context 'when the read concern is not valid' do
+
+        let(:new_collection) do
+          authorized_collection.with(read_concern: { level: 'na' })
+        end
+
+        it 'raises an exception' do
+          expect {
+            result
+          }.to raise_error(Mongo::Error::OperationFailure)
+        end
+      end
+    end
+  end
+
   describe '#allow_partial_results' do
 
     let(:new_view) do
@@ -25,7 +59,7 @@ describe Mongo::Collection::View::Readable do
     end
 
     it 'sets the flag' do
-      expect(new_view.send(:flags)).to include(:partial)
+      expect(new_view.options[:allow_partial_results]).to be true
     end
 
     it 'returns a new View' do
@@ -60,6 +94,15 @@ describe Mongo::Collection::View::Readable do
       view.aggregate(pipeline)
     end
 
+    context 'when incorporating read concern' do
+
+      let(:result) do
+        new_view.aggregate(pipeline, options).to_a
+      end
+
+      it_behaves_like 'a read concern aware operation'
+    end
+
     context 'when not iterating the aggregation' do
 
       it 'returns the aggregation object' do
@@ -127,6 +170,15 @@ describe Mongo::Collection::View::Readable do
       view.map_reduce(map, reduce)
     end
 
+    context 'when incorporating read concern' do
+
+      let(:result) do
+        new_view.map_reduce(map, reduce, options).to_a
+      end
+
+      it_behaves_like 'a read concern aware operation'
+    end
+
     context 'when not iterating the map/reduce' do
 
       it 'returns the map/reduce object' do
@@ -218,6 +270,19 @@ describe Mongo::Collection::View::Readable do
       authorized_collection.delete_many
     end
 
+    let(:result) do
+      view.count(options)
+    end
+
+    context 'when incorporating read concern' do
+
+      let(:result) do
+        new_view.count(options)
+      end
+
+      it_behaves_like 'a read concern aware operation'
+    end
+
     context 'when a selector is provided' do
 
       let(:selector) do
@@ -227,6 +292,10 @@ describe Mongo::Collection::View::Readable do
       it 'returns the count of matching documents' do
         expect(view.count).to eq(1)
       end
+
+      it 'returns an integer' do
+        expect(view.count).to be_a(Integer)
+      end
     end
 
     context 'when no selector is provided' do
@@ -253,6 +322,15 @@ describe Mongo::Collection::View::Readable do
 
   describe '#distinct' do
 
+    context 'when incorporating read concern' do
+
+      let(:result) do
+        new_view.distinct(:field, options)
+      end
+
+      it_behaves_like 'a read concern aware operation'
+    end
+
     context 'when a selector is provided' do
 
       let(:selector) do
@@ -318,7 +396,7 @@ describe Mongo::Collection::View::Readable do
         end
 
         it 'returns the distinct values' do
-          expect(distinct).to eq([ 'test1', 'test2', 'test3' ])
+          expect(distinct.sort).to eq([ 'test1', 'test2', 'test3' ])
         end
       end
 
@@ -329,7 +407,7 @@ describe Mongo::Collection::View::Readable do
         end
 
         it 'returns the distinct values' do
-          expect(distinct).to eq([ 'test1', 'test2', 'test3' ])
+          expect(distinct.sort).to eq([ 'test1', 'test2', 'test3' ])
         end
       end
 
@@ -360,7 +438,7 @@ describe Mongo::Collection::View::Readable do
       end
 
       it 'returns the distinct values' do
-        expect(distinct).to eq([ 'test1', 'test2', 'test3' ])
+        expect(distinct.sort).to eq([ 'test1', 'test2', 'test3' ])
       end
     end
 
@@ -465,53 +543,40 @@ describe Mongo::Collection::View::Readable do
     end
   end
 
-  describe '#no_cursor_timeout' do
+  describe '#max_value' do
 
     let(:new_view) do
-      view.no_cursor_timeout
+      view.max_value(_id: 1)
     end
 
-    it 'sets the flag' do
-      expect(new_view.send(:flags)).to include(:no_cursor_timeout)
-    end
-
-    it 'returns a new View' do
-      expect(new_view).not_to be(view)
+    it 'sets the value in the options' do
+      expect(new_view.max_value).to eq('_id' => 1)
     end
   end
 
-  describe '#parallel_scan', unless: sharded? do
+  describe '#min_value' do
 
-    let(:documents) do
-      (1..200).map do |i|
-        { name: "testing-scan-#{i}" }
-      end
+    let(:new_view) do
+      view.min_value(_id: 1)
     end
 
-    before do
-      authorized_collection.insert_many(documents)
+    it 'sets the value in the options' do
+      expect(new_view.min_value).to eq('_id' => 1)
     end
+  end
 
-    let(:cursors) do
-      view.parallel_scan(2)
-    end
+  describe '#no_cursor_timeout' do
 
-    it 'returns an array of cursors', if: write_command_enabled? do
-      cursors.each do |cursor|
-        expect(cursor.class).to be(Mongo::Cursor)
-      end
+    let(:new_view) do
+      view.no_cursor_timeout
     end
 
-    it 'returns the correct number of documents', if: write_command_enabled? do
-      expect(
-        cursors.reduce(0) { |total, cursor| total + cursor.to_a.size }
-      ).to eq(200)
+    it 'sets the flag' do
+      expect(new_view.options[:no_cursor_timeout]).to be true
     end
 
-    it 'raises an error', unless: write_command_enabled? do
-      expect {
-        cursors
-      }.to raise_error(Mongo::Error::OperationFailure)
+    it 'returns a new View' do
+      expect(new_view).not_to be(view)
     end
   end
 
@@ -527,6 +592,10 @@ describe Mongo::Collection::View::Readable do
         { 'y' => 1 }
       end
 
+      before do
+        authorized_collection.insert_one(y: 'value', a: 'other_value')
+      end
+
       it 'sets the projection' do
         new_view = view.projection(new_projection)
         expect(new_view.projection).to eq(new_projection)
@@ -535,6 +604,10 @@ describe Mongo::Collection::View::Readable do
       it 'returns a new View' do
         expect(view.projection(new_projection)).not_to be(view)
       end
+
+      it 'returns only that field on the collection' do
+        expect(view.projection(new_projection).first.keys).to match_array(['_id', 'y'])
+      end
     end
 
     context 'when projection is not specified' do
@@ -645,13 +718,13 @@ describe Mongo::Collection::View::Readable do
   describe '#modifiers' do
 
     let(:options) do
-      { :modifiers => { :$orderby => Mongo::Index::ASCENDING } }
+      { :modifiers => { '$orderby' => 1 } }
     end
 
     context 'when a modifiers document is specified' do
 
       let(:new_modifiers) do
-        { :modifiers => { :$orderby => Mongo::Index::DESCENDING } }
+        { '$orderby' => -1 }
       end
 
       it 'sets the new_modifiers document' do
@@ -779,81 +852,5 @@ describe Mongo::Collection::View::Readable do
         expect(view.sort).to eq(options[:sort])
       end
     end
-
-    context 'when an option is a cursor flag' do
-
-      let(:query_spec_options) do
-        view.send(:query_spec)[:options]
-      end
-
-      context 'when allow_partial_results is set as an option' do
-
-        let(:options) do
-          { :allow_partial_results => true }
-        end
-
-        it 'sets the cursor flag' do
-          expect(query_spec_options[:flags]).to eq([:partial])
-        end
-
-        context 'when allow_partial_results is also called as a method' do
-
-          before do
-            view.allow_partial_results
-          end
-
-          it 'sets only one cursor flag' do
-            expect(query_spec_options[:flags]).to eq([:partial])
-          end
-        end
-      end
-
-      context 'when oplog_replay is set as an option' do
-
-        let(:options) do
-          { :oplog_replay => true }
-        end
-
-        it 'sets the cursor flag' do
-          expect(query_spec_options[:flags]).to eq([:oplog_replay])
-        end
-      end
-
-      context 'when no_cursor_timeout is set as an option' do
-
-        let(:options) do
-          { :no_cursor_timeout => true }
-        end
-
-        it 'sets the cursor flag' do
-          expect(query_spec_options[:flags]).to eq([:no_cursor_timeout])
-        end
-      end
-
-      context 'when cursor_type is set as an option' do
-
-        context 'when :tailable is the cursor type' do
-
-          let(:options) do
-            { :cursor_type => :tailable }
-          end
-
-          it 'sets the cursor flag' do
-            expect(query_spec_options[:flags]).to eq([:tailable_cursor])
-          end
-        end
-
-        context 'when :tailable_await is the cursor type' do
-
-          let(:options) do
-            { :cursor_type => :tailable_await }
-          end
-
-          it 'sets the cursor flags' do
-            expect(query_spec_options[:flags]).to eq([:await_data, :tailable_cursor])
-          end
-        end
-      end
-    end
   end
 end