mongo 2.11.2 → 2.13.0.beta1

data/spec/mongo/error/unsupported_option_spec.rb

@@ -0,0 +1,54 @@
+require 'spec_helper'
+
+describe Mongo::Error::UnsupportedOption do
+  describe '.hint_error' do
+    context 'with no options' do
+      let(:error) { described_class.hint_error }
+
+      it 'creates an error with a default message' do
+        expect(error.message).to eq(
+          "The MongoDB server handling this request does not support the hint " \
+          "option on this command. The hint option is supported on update commands " \
+          "on MongoDB server versions 4.2 and later and on findAndModify and delete " \
+          "commands on MongoDB server versions 4.4 and later"
+        )
+      end
+
+      context 'with unacknowledged_write: true' do
+        let(:error) { described_class.hint_error(unacknowledged_write: true) }
+
+        it 'creates an error with a default unacknowledged writes message' do
+          expect(error.message).to eq(
+            "The hint option cannot be specified on an unacknowledged " \
+            "write operation. Remove the hint option or perform this " \
+            "operation with a write concern of at least { w: 1 }"
+          )
+        end
+      end
+    end
+  end
+
+  describe '.allow_disk_use_error' do
+    let(:error) { described_class.allow_disk_use_error }
+
+    it 'creates an error with a default message' do
+      expect(error.message).to eq(
+        "The MongoDB server handling this request does not support the allow_disk_use " \
+        "option on this command. The allow_disk_use option is supported on find commands " \
+        "on MongoDB server versions 4.4 and later"
+      )
+    end
+  end
+
+  describe '.commit_quorum_error' do
+    let(:error) { described_class.commit_quorum_error }
+
+    it 'creates an error with a default message' do
+      expect(error.message).to eq(
+        "The MongoDB server handling this request does not support the commit_quorum " \
+        "option on this command. The commit_quorum option is supported on createIndexes commands " \
+        "on MongoDB server versions 4.4 and later"
+      )
+    end
+  end
+end

data/spec/mongo/grid/fs_bucket_spec.rb

@@ -193,6 +193,24 @@ describe Mongo::Grid::FSBucket do
         fs.find({filename: 'test.txt'}, options)
       end
 
+      context 'when provided allow_disk_use' do
+        context 'when allow_disk_use is true' do
+          let(:options) { { allow_disk_use: true } }
+
+          it 'sets allow_disk_use on the view' do
+            expect(view.options[:allow_disk_use]).to be true
+          end
+        end
+
+        context 'when allow_disk_use is false' do
+          let(:options) { { allow_disk_use: false } }
+
+          it 'sets allow_disk_use on the view' do
+            expect(view.options[:allow_disk_use]).to be false
+          end
+        end
+      end
+
       context 'when provided batch_size' do
 
         let(:options) do

data/spec/mongo/grid/stream/write_spec.rb

@@ -289,6 +289,38 @@ describe Mongo::Grid::FSBucket::Stream::Write do
         end
       end
 
+      context 'when the files collection is empty but indexes already exist with double values' do
+        before do
+          fs.files_collection.indexes.create_one(
+            { filename: 1.0, uploadDate: 1.0 },
+            name: 'filename_1_uploadDate_1'
+          )
+
+          fs.chunks_collection.indexes.create_one(
+            { files_id: 1.0, n: 1.0 },
+            name: 'files_id_1_n_1',
+            unique: true
+          )
+        end
+
+        it 'does not raise an exception' do
+          expect do
+            stream.write(file)
+          end.not_to raise_error
+        end
+
+        it 'does not create new indexes' do
+          stream.write(file)
+
+          files_indexes = fs.files_collection.indexes.map { |index| index['key'] }
+          chunks_indexes = fs.chunks_collection.indexes.map { |index| index['key'] }
+
+          # Ruby parses the index keys with integer values
+          expect(files_indexes).to eq([{ '_id' => 1 }, { 'filename' => 1, 'uploadDate' => 1 }])
+          expect(chunks_indexes).to eq([{ '_id' => 1 }, { 'files_id' => 1, 'n' => 1 }])
+        end
+      end
+
       context 'when the files collection is not empty' do
 
         before do

data/spec/mongo/index/view_spec.rb

@@ -258,6 +258,73 @@ describe Mongo::Index::View do
           end
         end
 
+        context 'when commit quorum options are specified' do
+          require_topology :replica_set, :sharded
+          context 'on server versions >= 4.4' do
+            min_server_fcv '4.3'
+
+            let(:subscriber) { EventSubscriber.new }
+
+            let(:client) do
+              authorized_client.tap do |client|
+                client.subscribe(Mongo::Monitoring::COMMAND, subscriber)
+              end
+            end
+
+            let(:authorized_collection) { client['view-subscribed'] }
+
+            context 'when commit_quorum value is supported' do
+              let!(:result) do
+                view.create_many(
+                  { key: { random: 1 }, unique: true },
+                  { key: { testing: -1 }, unique: true },
+                  { commit_quorum: 'majority' }
+                )
+              end
+
+              let(:events) do
+                subscriber.command_started_events('createIndexes')
+              end
+
+              it 'returns ok' do
+                expect(result).to be_successful
+              end
+
+              it 'passes the commit_quorum option to the server' do
+                expect(events.length).to eq(1)
+                command = events.first.command
+                expect(command['commitQuorum']).to eq('majority')
+              end
+            end
+
+            context 'when commit_quorum value is not supported' do
+              it 'raises an exception' do
+                expect do
+                  view.create_many(
+                    { key: { random: 1 }, unique: true },
+                    { key: { testing: -1 }, unique: true },
+                    { commit_quorum: 'unsupported-value' }
+                  )
+                end.to raise_error(Mongo::Error::OperationFailure, /Commit quorum cannot be satisfied with the current replica set configuration/)
+              end
+            end
+          end
+
+          context 'on server versions < 4.4' do
+            max_server_fcv '4.2'
+
+            it 'raises an exception' do
+              expect do
+                view.create_many(
+                  { key: { random: 1 }, unique: true },
+                  { key: { testing: -1 }, unique: true },
+                  { commit_quorum: 'majority' }
+                )
+              end.to raise_error(Mongo::Error::UnsupportedOption, /The MongoDB server handling this request does not support the commit_quorum option/)
+            end
+          end
+        end
+
         context 'when collation is specified' do
           min_server_fcv '3.4'
 
@@ -505,6 +572,25 @@ describe Mongo::Index::View do
         end
       end
     end
+
+    context 'when using bucket option' do
+
+      let(:spec) do
+        { 'any' => 1 }
+      end
+
+      let(:result) do
+        view.create_many([key: spec, bucket_size: 1])
+      end
+
+      it 'warns of deprecation' do
+        RSpec::Mocks.with_temporary_scope do
+          view.client.should receive(:log_warn).and_call_original
+
+          result
+        end
+      end
+    end
   end
 
   describe '#create_one' do
@@ -592,6 +678,25 @@ describe Mongo::Index::View do
           expect(result).to be_successful
         end
       end
+
+      context 'when using bucket option' do
+
+        let(:spec) do
+          { 'any' => 1 }
+        end
+
+        let(:result) do
+          view.create_one(spec, bucket_size: 1)
+        end
+
+        it 'warns of deprecation' do
+          RSpec::Mocks.with_temporary_scope do
+            view.client.should receive(:log_warn).and_call_original
+
+            result
+          end
+        end
+      end
     end
 
     context 'when index creation fails' do
@@ -667,6 +772,67 @@ describe Mongo::Index::View do
         expect(indexes[:partialFilterExpression]).to eq(expression)
       end
     end
+
+    context 'when providing commit_quorum option' do
+      require_topology :replica_set, :sharded
+      context 'on server versions >= 4.4' do
+        min_server_fcv '4.3'
+
+        let(:subscriber) { EventSubscriber.new }
+
+        let(:client) do
+          authorized_client.tap do |client|
+            client.subscribe(Mongo::Monitoring::COMMAND, subscriber)
+          end
+        end
+
+        let(:authorized_collection) { client['view-subscribed'] }
+
+        let(:indexes) do
+          authorized_collection.indexes.get('x_1')
+        end
+
+        context 'when commit_quorum value is supported' do
+          let!(:result) { view.create_one({ 'x' => 1 }, commit_quorum: 'majority') }
+
+          it 'returns ok' do
+            expect(result).to be_successful
+          end
+
+          it 'creates an index' do
+            expect(indexes).to_not be_nil
+          end
+
+          let(:events) do
+            subscriber.command_started_events('createIndexes')
+          end
+
+          it 'passes the commit_quorum option to the server' do
+            expect(events.length).to eq(1)
+            command = events.first.command
+            expect(command['commitQuorum']).to eq('majority')
+          end
+        end
+
+        context 'when commit_quorum value is not supported' do
+          it 'raises an exception' do
+            expect do
+              view.create_one({ 'x' => 1 }, commit_quorum: 'unsupported-value')
+            end.to raise_error(Mongo::Error::OperationFailure, /Commit quorum cannot be satisfied with the current replica set configuration/)
+          end
+        end
+      end
+
+      context 'on server versions < 4.4' do
+        max_server_fcv '4.2'
+
+        it 'raises an exception' do
+          expect do
+            view.create_one({ 'x' => 1 }, commit_quorum: 'majority')
+          end.to raise_error(Mongo::Error::UnsupportedOption, /The MongoDB server handling this request does not support the commit_quorum option/)
+        end
+      end
+    end
   end
 
   describe '#get' do

data/spec/mongo/operation/aggregate/result_spec.rb

@@ -2,8 +2,13 @@ require 'spec_helper'
 
 describe Mongo::Operation::Aggregate::Result do
 
+  let(:description) do
+    Mongo::Server::Description.new(double('description address'),
+      'minWireVersion' => 0, 'maxWireVersion' => 2)
+  end
+
   let(:result) do
-    described_class.new(reply)
+    described_class.new(reply, description)
   end
 
   let(:cursor_id) { 0 }

data/spec/mongo/operation/aggregate_spec.rb

@@ -64,7 +64,7 @@ describe Mongo::Operation::Aggregate do
 
       it 'raises an exception' do
         expect {
-          op.execute(authorized_primary)
+          op.execute(authorized_primary, client: nil)
         }.to raise_error(Mongo::Error::OperationFailure)
       end
     end

data/spec/mongo/operation/collections_info_spec.rb

@@ -31,7 +31,7 @@ describe Mongo::Operation::CollectionsInfo do
     end
 
     let(:info) do
-      docs = op.execute(authorized_primary).documents
+      docs = op.execute(authorized_primary, client: nil).documents
       docs.collect { |info| info['name'].sub("#{SpecConfig.instance.test_db}.", '') }
     end
 

data/spec/mongo/operation/command_spec.rb

@@ -43,7 +43,7 @@ describe Mongo::Operation::Command do
     context 'when the command succeeds' do
 
       let(:response) do
-        op.execute(authorized_primary)
+        op.execute(authorized_primary, client: nil)
       end
 
       it 'returns the reponse' do
@@ -59,7 +59,7 @@ describe Mongo::Operation::Command do
 
       it 'raises an exception' do
         expect {
-          op.execute(authorized_primary)
+          op.execute(authorized_primary, client: nil)
         }.to raise_error(Mongo::Error::OperationFailure)
       end
     end
@@ -72,7 +72,7 @@ describe Mongo::Operation::Command do
 
       it 'raises an error' do
         expect {
-          op.execute(authorized_primary)
+          op.execute(authorized_primary, client: nil)
         }.to raise_error(Mongo::Error::MaxBSONSize)
       end
     end

data/spec/mongo/operation/create_index_spec.rb

@@ -20,7 +20,7 @@ describe Mongo::Operation::CreateIndex do
       end
 
       let(:response) do
-        operation.execute(authorized_primary)
+        operation.execute(authorized_primary, client: nil)
       end
 
       it 'returns ok' do
@@ -43,12 +43,12 @@ describe Mongo::Operation::CreateIndex do
       end
 
       before do
-        operation.execute(authorized_primary)
+        operation.execute(authorized_primary, client: nil)
       end
 
       it 'raises an exception' do
         expect {
-          second_operation.execute(authorized_primary)
+          second_operation.execute(authorized_primary, client: nil)
         }.to raise_error(Mongo::Error::OperationFailure)
       end
     end

data/spec/mongo/operation/create_user_spec.rb

@@ -26,7 +26,7 @@ describe Mongo::Operation::CreateUser do
     context 'when user creation was successful' do
 
       let!(:response) do
-        operation.execute(root_authorized_primary)
+        operation.execute(root_authorized_primary, client: nil)
       end
 
       it 'saves the user in the database' do
@@ -38,8 +38,8 @@ describe Mongo::Operation::CreateUser do
 
       it 'raises an exception' do
         expect {
-          operation.execute(root_authorized_primary)
-          operation.execute(root_authorized_primary)
+          operation.execute(root_authorized_primary, client: nil)
+          operation.execute(root_authorized_primary, client: nil)
         }.to raise_error(Mongo::Error::OperationFailure)
       end
     end

data/spec/mongo/operation/delete/bulk_spec.rb

@@ -99,7 +99,9 @@ describe Mongo::Operation::Delete do
         end
 
         it 'deletes the document from the database' do
-          op.bulk_execute(authorized_primary)
+          authorized_primary.with_connection do |connection|
+            op.bulk_execute(connection, client: nil)
+          end
           expect(authorized_collection.find.count).to eq(1)
         end
       end
@@ -122,7 +124,9 @@ describe Mongo::Operation::Delete do
         end
 
         it 'deletes the documents from the database' do
-          op.bulk_execute(authorized_primary)
+          authorized_primary.with_connection do |connection|
+            op.bulk_execute(connection, client: nil)
+          end
           expect(authorized_collection.find.count).to eq(0)
         end
       end
@@ -157,7 +161,9 @@ describe Mongo::Operation::Delete do
           end
 
           it 'aborts after first error' do
-            failing_delete.bulk_execute(authorized_primary)
+            authorized_primary.with_connection do |connection|
+              failing_delete.bulk_execute(connection, client: nil)
+            end
             expect(authorized_collection.find.count).to eq(2)
           end
         end
@@ -169,7 +175,9 @@ describe Mongo::Operation::Delete do
           end
 
           it 'aborts after first error' do
-            failing_delete.bulk_execute(authorized_primary)
+            authorized_primary.with_connection do |connection|
+              failing_delete.bulk_execute(connection, client: nil)
+            end
             expect(authorized_collection.find.count).to eq(2)
           end
         end
@@ -205,7 +213,9 @@ describe Mongo::Operation::Delete do
           end
 
           it 'does not abort after first error' do
-            failing_delete.bulk_execute(authorized_primary)
+            authorized_primary.with_connection do |connection|
+              failing_delete.bulk_execute(connection, client: nil)
+            end
             expect(authorized_collection.find.count).to eq(1)
           end
         end
@@ -217,7 +227,9 @@ describe Mongo::Operation::Delete do
           end
 
           it 'does not abort after first error' do
-            failing_delete.bulk_execute(authorized_primary)
+            authorized_primary.with_connection do |connection|
+              failing_delete.bulk_execute(connection, client: nil)
+            end
             expect(authorized_collection.find.count).to eq(1)
           end
         end