mongo 2.13.0 → 2.15.0.alpha

data/spec/mongo/crypt/auto_decryption_context_spec.rb

@@ -76,7 +76,7 @@ describe Mongo::Crypt::AutoDecryptionContext do
       end
 
       let(:logger) do
-        ::Logger.new($stdout).tap do |logger|
+        ::Logger.new(STDOUT).tap do |logger|
           logger.level = ::Logger::DEBUG
         end
       end

data/spec/mongo/crypt/auto_encryption_context_spec.rb

@@ -93,7 +93,7 @@ describe Mongo::Crypt::AutoEncryptionContext do
       end
 
       let(:logger) do
-        ::Logger.new($stdout).tap do |logger|
+        ::Logger.new(STDOUT).tap do |logger|
           logger.level = ::Logger::DEBUG
         end
       end

data/spec/mongo/crypt/binary_spec.rb

@@ -1,9 +1,4 @@
-require 'mongo'
-require 'support/lite_constraints'
-
-RSpec.configure do |config|
-  config.extend(LiteConstraints)
-end
+require 'lite_spec_helper'
 
 describe Mongo::Crypt::Binary do
   require_libmongocrypt

data/spec/mongo/crypt/binding/binary_spec.rb

@@ -1,9 +1,4 @@
-require 'mongo'
-require 'support/lite_constraints'
-
-RSpec.configure do |config|
-  config.extend(LiteConstraints)
-end
+require 'lite_spec_helper'
 
 describe 'Mongo::Crypt::Binding' do
   describe 'binary_t bindings' do

data/spec/mongo/crypt/binding/context_spec.rb

@@ -1,10 +1,5 @@
-require 'mongo'
-require 'support/lite_constraints'
-require 'mongo/crypt/helpers/mongo_crypt_spec_helper'
-
-RSpec.configure do |config|
-  config.extend(LiteConstraints)
-end
+require 'lite_spec_helper'
+require_relative '../helpers/mongo_crypt_spec_helper'
 
 shared_context 'initialized for data key creation' do
   let(:master_key) { "ru\xfe\x00" * 24 }

data/spec/mongo/crypt/binding/helpers_spec.rb

@@ -1,9 +1,4 @@
-require 'mongo'
-require 'support/lite_constraints'
-
-RSpec.configure do |config|
-  config.extend(LiteConstraints)
-end
+require 'lite_spec_helper'
 
 describe 'Mongo::Crypt::Binding' do
   describe 'helper methods' do

data/spec/mongo/crypt/binding/mongocrypt_spec.rb

@@ -1,10 +1,5 @@
-require 'mongo'
-require 'support/lite_constraints'
-require 'mongo/crypt/helpers/mongo_crypt_spec_helper'
-
-RSpec.configure do |config|
-  config.extend(LiteConstraints)
-end
+require 'lite_spec_helper'
+require_relative '../helpers/mongo_crypt_spec_helper'
 
 describe 'Mongo::Crypt::Binding' do
   describe 'mongocrypt_t binding' do

data/spec/mongo/crypt/binding/status_spec.rb

@@ -1,9 +1,4 @@
-require 'mongo'
-require 'support/lite_constraints'
-
-RSpec.configure do |config|
-  config.extend(LiteConstraints)
-end
+require 'lite_spec_helper'
 
 describe 'Mongo::Crypt::Binding' do
   describe 'mongocrypt_status_t binding' do

data/spec/mongo/crypt/binding/version_spec.rb

@@ -1,9 +1,4 @@
-require 'mongo'
-require 'support/lite_constraints'
-
-RSpec.configure do |config|
-  config.extend(LiteConstraints)
-end
+require 'lite_spec_helper'
 
 describe 'Mongo::Crypt::Binding' do
   require_libmongocrypt

data/spec/mongo/crypt/data_key_context_spec.rb

@@ -188,7 +188,7 @@ describe Mongo::Crypt::DataKeyContext do
       end
 
       context 'with valid endpoint' do
-        let(:options) { { master_key: { region: 'us-east-2', key: 'arn', endpoint: 'endpoint/to/kms' } } }
+        let(:options) { { master_key: { region: 'us-east-2', key: 'arn', endpoint: 'kms.us-east-2.amazonaws.com:443' } } }
 
         it 'does not raise an exception' do
           expect do

data/spec/mongo/crypt/explicit_decryption_context_spec.rb

@@ -57,7 +57,7 @@ describe Mongo::Crypt::ExplicitDecryptionContext do
       end
 
       let(:logger) do
-        ::Logger.new($stdout).tap do |logger|
+        ::Logger.new(STDOUT).tap do |logger|
           logger.level = ::Logger::DEBUG
         end
       end

data/spec/mongo/crypt/explicit_encryption_context_spec.rb

@@ -153,7 +153,7 @@ describe Mongo::Crypt::ExplicitEncryptionContext do
 
       let(:key_alt_name) { nil }
       let(:logger) do
-        ::Logger.new($stdout).tap do |logger|
+        ::Logger.new(STDOUT).tap do |logger|
           logger.level = ::Logger::DEBUG
         end
       end

data/spec/mongo/crypt/status_spec.rb

@@ -1,9 +1,4 @@
-require 'mongo'
-require 'support/lite_constraints'
-
-RSpec.configure do |config|
-  config.extend(LiteConstraints)
-end
+require 'lite_spec_helper'
 
 describe Mongo::Crypt::Status do
   require_libmongocrypt

data/spec/mongo/database_spec.rb

@@ -2,6 +2,19 @@ require 'spec_helper'
 
 describe Mongo::Database do
 
+  shared_context 'more than 100 collections' do
+    let(:client) do
+      root_authorized_client.use('many-collections')
+    end
+
+    before do
+      120.times do |i|
+        client["coll-#{i}"].drop
+        client["coll-#{i}"].create
+      end
+    end
+  end
+
   describe '#==' do
 
     let(:database) do
@@ -82,6 +95,26 @@ describe Mongo::Database do
         expect(collection.server_selector).to eq(Mongo::ServerSelector.get(mode: :secondary))
         expect(collection.read_preference).to eq(BSON::Document.new(mode: :secondary))
       end
+
+      context ':server_api option' do
+
+        let(:client) do
+          new_local_client_nmio(['localhost'], server_api: {version: '1'})
+        end
+
+        it 'is not transfered to the collection' do
+          client.options[:server_api].should == {'version' => '1'}
+          collection.options[:server_api].should be nil
+        end
+      end
+    end
+
+    context 'when providing :server_api option' do
+      it 'is rejected' do
+        lambda do
+          database['foo', server_api: {version: '1'}]
+        end.should raise_error(ArgumentError, 'The :server_api option cannot be specified for collection objects. It can only be specified on Client level')
+      end
     end
   end
 
@@ -228,6 +261,20 @@ describe Mongo::Database do
         end
       end
     end
+
+    context 'when there are more than 100 collections' do
+      include_context 'more than 100 collections'
+
+      let(:collection_names) do
+        client.database.collection_names.sort
+      end
+
+      it 'lists all collections' do
+        collection_names.length.should == 120
+        collection_names.should include('coll-0')
+        collection_names.should include('coll-119')
+      end
+    end
   end
 
   describe '#list_collections' do
@@ -292,21 +339,40 @@ describe Mongo::Database do
         described_class.new(root_authorized_client, 'admin')
       end
 
-      it 'does not include system collections' do
-        expect(result.none? { |name| name =~ /(^|\.)system\./ }).to be true
+      shared_examples 'does not include system collections' do
+        it 'does not include system collections' do
+          expect(result.none? { |name| name =~ /(^|\.)system\./ }).to be true
+        end
+      end
+
+      context 'server 4.7+' do
+        min_server_fcv '4.7'
+        # https://jira.mongodb.org/browse/SERVER-35804
+        require_topology :single, :replica_set
+
+        include_examples 'does not include system collections'
+
+        it 'returns results' do
+          expect(result).to include('acol')
+        end
       end
 
-      context 'server 3.0+' do
+      context 'server 3.0-4.5' do
         min_server_fcv '3.0'
+        max_server_version '4.5'
+
+        include_examples 'does not include system collections'
 
         it 'returns results' do
           expect(result).to include('acol')
         end
       end
 
-      context 'server 2.6-' do
+      context 'server 2.6' do
         max_server_version '2.6'
 
+        include_examples 'does not include system collections'
+
         it 'returns results' do
           expect(result).to include('admin.acol')
         end
@@ -391,6 +457,25 @@ describe Mongo::Database do
         end
       end
     end
+
+    context 'when there are more than 100 collections' do
+      include_context 'more than 100 collections'
+
+      let(:collections) do
+        client.database.list_collections
+      end
+
+      let(:collection_names) do
+        # 2.6 server prefixes collection names with database name
+        collections.map { |info| info['name'].sub(/^many-collections\./, '') }.sort
+      end
+
+      it 'lists all collections' do
+        collections.length.should == 120
+        collection_names.should include('coll-0')
+        collection_names.should include('coll-119')
+      end
+    end
   end
 
   describe '#collections' do
@@ -541,6 +626,24 @@ describe Mongo::Database do
         end
       end
     end
+
+    context 'when there are more than 100 collections' do
+      include_context 'more than 100 collections'
+
+      let(:collections) do
+        client.database.collections
+      end
+
+      let(:collection_names) do
+        collections.map(&:name).sort
+      end
+
+      it 'lists all collections' do
+        collections.length.should == 120
+        collection_names.should include('coll-0')
+        collection_names.should include('coll-119')
+      end
+    end
   end
 
   describe '#command' do
@@ -772,6 +875,29 @@ describe Mongo::Database do
         end
       end
     end
+
+    context 'when client server api is not set' do
+      require_no_required_api_version
+      min_server_fcv '4.7'
+
+      it 'passes server api parameters' do
+        lambda do
+          database.command(ping: 1, apiVersion: 'does-not-exist')
+        end.should raise_error(
+          an_instance_of(Mongo::Error::OperationFailure).and having_attributes(code: 322))
+      end
+    end
+
+    context 'when client server api is set' do
+      require_required_api_version
+      min_server_fcv '4.7'
+
+      it 'reports server api conflict' do
+        lambda do
+          database.command(ping: 1, apiVersion: 'does-not-exist')
+        end.should raise_error(Mongo::Error::ServerApiConflict)
+      end
+    end
   end
 
   describe '#drop' do
@@ -825,6 +951,50 @@ describe Mongo::Database do
         end
       end
 
+      context 'when write concern is passed in as an option' do
+        min_server_fcv '3.4'
+        require_topology :single
+
+        let(:client_options) do
+          {
+            write_concern: {w: 0},
+            database: :test
+          }
+        end
+
+        let(:session) do
+          client.start_session
+        end
+
+        let(:subscriber) { EventSubscriber.new }
+
+        let(:client) do
+          root_authorized_client.tap do |client|
+            client.subscribe(Mongo::Monitoring::COMMAND, subscriber)
+          end.with(client_options)
+        end
+
+        let(:events) do
+          subscriber.command_started_events('dropDatabase')
+        end
+
+        let(:database_test_wc) do
+          client.database
+        end
+
+        let!(:command) do
+          Utils.get_command_event(client, 'dropDatabase') do |client|
+            database_test_wc.drop({ write_concern: {w: 'majority'} })
+          end.command
+        end
+
+        it 'applies the write concern passed in as an option' do
+          expect(events.length).to eq(1)
+          expect(command).to_not be_nil
+          expect(command[:writeConcern][:w]).to eq('majority')
+        end
+      end
+
       context 'when the server does not support write concern on the dropDatabase command' do
         max_server_version '3.2'
 

data/spec/mongo/error/bulk_write_error_spec.rb

@@ -31,19 +31,19 @@ describe Mongo::Error::BulkWriteError do
 
   describe '#message' do
     it 'is correct' do
-      expect(error.message).to eq("message1 (1), message2 (2) (note1, note2)")
+      expect(error.message).to eq("Multiple errors: [1]: message1; [2]: message2 (note1, note2)")
     end
   end
 
   describe '#to_s' do
     it 'is correct' do
-      expect(error.to_s).to eq("message1 (1), message2 (2) (note1, note2)")
+      expect(error.to_s).to eq("Multiple errors: [1]: message1; [2]: message2 (note1, note2)")
     end
   end
 
   describe '#inspect' do
     it 'is correct' do
-      expect(error.inspect).to eq("#<Mongo::Error::BulkWriteError: message1 (1), message2 (2) (note1, note2)>")
+      expect(error.inspect).to eq("#<Mongo::Error::BulkWriteError: Multiple errors: [1]: message1; [2]: message2 (note1, note2)>")
     end
   end
 end

data/spec/mongo/error/no_server_available_spec.rb

@@ -16,7 +16,7 @@ describe Mongo::Error::NoServerAvailable do
     end
 
     it 'is correct' do
-      expect(error.message).to eq('No primary server is available in cluster: #<Cluster topology=Unknown[127.0.0.1:27017] servers=[#<Server address=127.0.0.1:27017 UNKNOWN>]> with timeout=30, LT=0.015')
+      expect(error.message).to eq('No primary server is available in cluster: #<Cluster topology=Unknown[127.0.0.1:27017] servers=[#<Server address=127.0.0.1:27017 UNKNOWN NO-MONITORING>]> with timeout=30, LT=0.015')
     end
 
     context 'when cluster is nil' do

data/spec/mongo/error/parser_spec.rb

@@ -25,10 +25,22 @@ describe Mongo::Error::Parser do
       end
 
       it 'returns the message' do
-        expect(parser.message).to eq('no such command: notacommand (59)')
+        expect(parser.message).to eq('[59]: no such command: notacommand')
       end
     end
 
+    context 'when the document contains an errmsg and code name' do
+
+      let(:document) do
+        { 'errmsg' => 'no such command: notacommand', 'code' => 59, 'codeName' => 'foo' }
+      end
+
+      it 'returns the message' do
+        expect(parser.message).to eq('[59:foo]: no such command: notacommand')
+      end
+    end
+
+=begin
     context 'when the document contains writeErrors' do
 
       context 'when only a single error exists' do
@@ -38,7 +50,7 @@ describe Mongo::Error::Parser do
         end
 
         it 'returns the message' do
-          expect(parser.message).to eq('Unknown modifier: $st (9)')
+          expect(parser.message).to eq('[9]: Unknown modifier: $st')
         end
       end
 
@@ -55,11 +67,30 @@ describe Mongo::Error::Parser do
 
         it 'returns the messages concatenated' do
           expect(parser.message).to eq(
-            'Unknown modifier: $st (9), Unknown modifier: $bl (9)'
+            'Multiple errors: 9: Unknown modifier: $st; 9: Unknown modifier: $bl'
+          )
+        end
+      end
+
+      context 'when multiple errors with code names exist' do
+
+        let(:document) do
+          {
+            'writeErrors' => [
+              { 'code' => 9, 'codeName' => 'foo', 'errmsg' => 'Unknown modifier: $st' },
+              { 'code' => 9, 'codeName' => 'foo', 'errmsg' => 'Unknown modifier: $bl' },
+            ]
+          }
+        end
+
+        it 'returns the messages concatenated' do
+          expect(parser.message).to eq(
+            'Multiple errors: [9:foo]: Unknown modifier: $st; [9:foo]: Unknown modifier: $bl'
           )
         end
       end
     end
+=end
 
     context 'when the document contains $err' do
 
@@ -68,7 +99,7 @@ describe Mongo::Error::Parser do
       end
 
       it 'returns the message' do
-        expect(parser.message).to eq('not authorized for query (13)')
+        expect(parser.message).to eq('[13]: not authorized for query')
       end
     end
 
@@ -79,7 +110,7 @@ describe Mongo::Error::Parser do
       end
 
       it 'returns the message' do
-        expect(parser.message).to eq('not authorized for query (13)')
+        expect(parser.message).to eq('[13]: not authorized for query')
       end
     end
 
@@ -90,7 +121,7 @@ describe Mongo::Error::Parser do
       end
 
       it 'returns the message' do
-        expect(parser.message).to eq('Not enough data-bearing nodes (100)')
+        expect(parser.message).to eq('[100]: Not enough data-bearing nodes')
       end
     end
   end