mongo 2.6.4 → 2.7.0.rc0

data/spec/mongo/monitoring/event/server_heartbeat_succeeded_spec.rb

@@ -0,0 +1,31 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::ServerHeartbeatSucceeded do
+
+  let(:address) do
+    Mongo::Address.new('127.0.0.1:27017')
+  end
+
+  let(:monitoring) { double('monitoring') }
+
+  let(:cluster) do
+    double('cluster').tap do |cluster|
+      allow(cluster).to receive(:addresses).and_return([address])
+      allow(cluster).to receive(:servers_list).and_return([])
+    end
+  end
+
+  let(:topology) do
+    Mongo::Cluster::Topology::Unknown.new({}, monitoring, cluster)
+  end
+
+  let(:event) do
+    described_class.new(address, 1)
+  end
+
+  describe '#summary' do
+    it 'renders correctly' do
+      expect(event.summary).to eq('#<ServerHeartbeatSucceeded address=127.0.0.1:27017>')
+    end
+  end
+end

data/spec/mongo/monitoring/event/server_opening_spec.rb

@@ -0,0 +1,35 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::ServerOpening do
+
+  let(:address) do
+    Mongo::Address.new('127.0.0.1:27017')
+  end
+
+  let(:monitoring) { double('monitoring') }
+
+  let(:cluster) do
+    double('cluster').tap do |cluster|
+      allow(cluster).to receive(:addresses).and_return([address])
+      allow(cluster).to receive(:servers_list).and_return([])
+    end
+  end
+
+  let(:topology) do
+    Mongo::Cluster::Topology::Unknown.new({}, monitoring, cluster)
+  end
+
+  let(:event) do
+    described_class.new(address, topology)
+  end
+
+  describe '#summary' do
+    skip_if_linting
+
+    it 'renders correctly' do
+      expect(topology).to receive(:server_descriptions).and_return({
+        '127.0.0.1:27017' => Mongo::Server::Description.new(Mongo::Address.new('127.0.0.1:27017'))})
+      expect(event.summary).to eq('#<ServerOpening address=127.0.0.1:27017 topology=Unknown[127.0.0.1:27017]>')
+    end
+  end
+end

data/spec/mongo/monitoring/event/topology_changed_spec.rb

@@ -0,0 +1,41 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::TopologyChanged do
+
+  let(:address) do
+    Mongo::Address.new('127.0.0.1:27017')
+  end
+
+  let(:monitoring) { double('monitoring') }
+
+  let(:cluster) do
+    double('cluster').tap do |cluster|
+      allow(cluster).to receive(:addresses).and_return([address])
+      allow(cluster).to receive(:servers_list).and_return([])
+    end
+  end
+
+  let(:prev_topology) do
+    Mongo::Cluster::Topology::Unknown.new({}, monitoring, cluster)
+  end
+
+  let(:new_topology) do
+    Mongo::Cluster::Topology::Unknown.new({}, monitoring, cluster)
+  end
+
+  let(:event) do
+    described_class.new(prev_topology, new_topology)
+  end
+
+  describe '#summary' do
+    skip_if_linting
+
+    it 'renders correctly' do
+      expect(prev_topology).to receive(:server_descriptions).and_return({
+        '127.0.0.1:27017' => Mongo::Server::Description.new(Mongo::Address.new('127.0.0.1:27017'))})
+      expect(new_topology).to receive(:server_descriptions).and_return({
+        '127.0.0.1:99999' => Mongo::Server::Description.new(Mongo::Address.new('127.0.0.1:99999'))})
+      expect(event.summary).to eq('#<TopologyChanged prev=Unknown[127.0.0.1:27017] new=Unknown[127.0.0.1:99999]>')
+    end
+  end
+end

data/spec/mongo/monitoring/event/topology_closed_spec.rb

@@ -0,0 +1,35 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::TopologyClosed do
+
+  let(:address) do
+    Mongo::Address.new('127.0.0.1:27017')
+  end
+
+  let(:monitoring) { double('monitoring') }
+
+  let(:cluster) do
+    double('cluster').tap do |cluster|
+      allow(cluster).to receive(:addresses).and_return([address])
+      allow(cluster).to receive(:servers_list).and_return([])
+    end
+  end
+
+  let(:topology) do
+    Mongo::Cluster::Topology::Unknown.new({}, monitoring, cluster)
+  end
+
+  let(:event) do
+    described_class.new(topology)
+  end
+
+  describe '#summary' do
+    skip_if_linting
+
+    it 'renders correctly' do
+      expect(topology).to receive(:server_descriptions).and_return({
+        '127.0.0.1:27017' => Mongo::Server::Description.new(Mongo::Address.new('127.0.0.1:27017'))})
+      expect(event.summary).to eq('#<TopologyClosed topology=Unknown[127.0.0.1:27017]>')
+    end
+  end
+end

data/spec/mongo/monitoring/event/topology_opening_spec.rb

@@ -0,0 +1,35 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::TopologyOpening do
+
+  let(:address) do
+    Mongo::Address.new('127.0.0.1:27017')
+  end
+
+  let(:monitoring) { double('monitoring') }
+
+  let(:cluster) do
+    double('cluster').tap do |cluster|
+      allow(cluster).to receive(:addresses).and_return([address])
+      allow(cluster).to receive(:servers_list).and_return([])
+    end
+  end
+
+  let(:topology) do
+    Mongo::Cluster::Topology::Unknown.new({}, monitoring, cluster)
+  end
+
+  let(:event) do
+    described_class.new(topology)
+  end
+
+  describe '#summary' do
+    skip_if_linting
+
+    it 'renders correctly' do
+      expect(topology).to receive(:server_descriptions).and_return({
+        '127.0.0.1:27017' => Mongo::Server::Description.new(Mongo::Address.new('127.0.0.1:27017'))})
+      expect(event.summary).to eq('#<TopologyOpening topology=Unknown[127.0.0.1:27017]>')
+    end
+  end
+end

data/spec/mongo/monitoring_spec.rb

@@ -45,7 +45,7 @@ describe Mongo::Monitoring do
       end
 
       it 'includes the global subscribers' do
-        expect(monitoring.subscribers.size).to eq(6)
+        expect(monitoring.subscribers.size).to eq(7)
       end
     end
 
@@ -58,7 +58,7 @@ describe Mongo::Monitoring do
         end
 
         it 'includes the global subscribers' do
-          expect(monitoring.subscribers.size).to eq(6)
+          expect(monitoring.subscribers.size).to eq(7)
         end
       end
 

data/spec/mongo/operation/aggregate_spec.rb

@@ -14,7 +14,7 @@ describe Mongo::Operation::Aggregate do
   let(:spec) do
     { :selector => selector,
       :options => options,
-      :db_name => TEST_DB
+      :db_name => SpecConfig.instance.test_db
     }
   end
   let(:op) { described_class.new(spec) }
@@ -41,7 +41,7 @@ describe Mongo::Operation::Aggregate do
       let(:other_spec) do
         { :selector => other_selector,
           :options => options,
-          :db_name => TEST_DB,
+          :db_name => SpecConfig.instance.test_db,
         }
       end
       let(:other) { described_class.new(other_spec) }

data/spec/mongo/operation/collections_info_spec.rb

@@ -4,7 +4,7 @@ describe Mongo::Operation::CollectionsInfo do
 
   let(:spec) do
     { selector: { listCollections: 1 },
-      db_name: TEST_DB
+      db_name: SpecConfig.instance.test_db
     }
   end
 
@@ -32,7 +32,7 @@ describe Mongo::Operation::CollectionsInfo do
 
     let(:info) do
       docs = op.execute(authorized_primary).documents
-      docs.collect { |info| info['name'].sub("#{TEST_DB}.", '') }
+      docs.collect { |info| info['name'].sub("#{SpecConfig.instance.test_db}.", '') }
     end
 
     it 'returns the list of collection info' do

data/spec/mongo/operation/command_spec.rb

@@ -7,7 +7,7 @@ describe Mongo::Operation::Command do
   let(:spec) do
     { :selector => selector,
       :options     => options,
-      :db_name  => TEST_DB
+      :db_name  => SpecConfig.instance.test_db
     }
   end
   let(:op) { described_class.new(spec) }

data/spec/mongo/operation/create_index_spec.rb

@@ -2,6 +2,11 @@ require 'spec_helper'
 
 describe Mongo::Operation::CreateIndex do
 
+  before do
+    authorized_collection.drop
+    authorized_collection.insert_one(test: 1)
+  end
+
   describe '#execute' do
 
     context 'when the index is created' do
@@ -11,17 +16,13 @@ describe Mongo::Operation::CreateIndex do
       end
 
       let(:operation) do
-        described_class.new(indexes: [ spec ], db_name: TEST_DB, coll_name: TEST_COLL)
+        described_class.new(indexes: [ spec ], db_name: SpecConfig.instance.test_db, coll_name: TEST_COLL)
       end
 
       let(:response) do
         operation.execute(authorized_primary)
       end
 
-      after do
-        authorized_collection.indexes.drop_one('random_1')
-      end
-
       it 'returns ok' do
         expect(response).to be_successful
       end
@@ -34,21 +35,17 @@ describe Mongo::Operation::CreateIndex do
       end
 
       let(:operation) do
-        described_class.new(indexes: [ spec ], db_name: TEST_DB, coll_name: TEST_COLL)
+        described_class.new(indexes: [ spec ], db_name: SpecConfig.instance.test_db, coll_name: TEST_COLL)
       end
 
       let(:second_operation) do
-        described_class.new(indexes: [ spec.merge(unique: false) ], db_name: TEST_DB, coll_name: TEST_COLL)
+        described_class.new(indexes: [ spec.merge(unique: false) ], db_name: SpecConfig.instance.test_db, coll_name: TEST_COLL)
       end
 
       before do
         operation.execute(authorized_primary)
       end
 
-      after do
-        authorized_collection.indexes.drop_one('random_1')
-      end
-
       it 'raises an exception' do
         expect {
           second_operation.execute(authorized_primary)

data/spec/mongo/operation/create_user_spec.rb

@@ -13,11 +13,14 @@ describe Mongo::Operation::CreateUser do
     end
 
     let(:operation) do
-      described_class.new(user: user, db_name: TEST_DB)
+      described_class.new(user: user, db_name: SpecConfig.instance.test_db)
     end
 
-    after do
-      root_authorized_client.database.users.remove('durran')
+    before do
+      users = root_authorized_client.database.users
+      if users.info('durran').any?
+        users.remove('durran')
+      end
     end
 
     context 'when user creation was successful' do

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

@@ -8,9 +8,9 @@ describe Mongo::Operation::Delete do
 
   let(:spec) do
     { :deletes       => documents,
-      :db_name       => TEST_DB,
+      :db_name       => SpecConfig.instance.test_db,
       :coll_name     => TEST_COLL,
-      :write_concern => Mongo::WriteConcern.get(WRITE_CONCERN),
+      :write_concern => Mongo::WriteConcern.get(SpecConfig.instance.write_concern),
       :ordered       => true
     }
   end
@@ -46,9 +46,9 @@ describe Mongo::Operation::Delete do
 
         let(:other_spec) do
           { :deletes       => other_docs,
-            :db_name       => TEST_DB,
+            :db_name       => SpecConfig.instance.test_db,
             :coll_name     => TEST_COLL,
-            :write_concern => Mongo::WriteConcern.get(WRITE_CONCERN),
+            :write_concern => Mongo::WriteConcern.get(SpecConfig.instance.write_concern),
             :ordered       => true
           }
         end
@@ -64,6 +64,15 @@ describe Mongo::Operation::Delete do
   describe '#bulk_execute' do
 
     before do
+      begin
+        authorized_collection.delete_many
+      rescue Mongo::Error::OperationFailure
+      end
+      begin
+        authorized_collection.indexes.drop_all
+      rescue Mongo::Error::OperationFailure
+      end
+
       authorized_collection.insert_many([
         { name: 'test', field: 'test' },
         { name: 'testing', field: 'test' }
@@ -79,7 +88,7 @@ describe Mongo::Operation::Delete do
       let(:op) do
         described_class.new({
           deletes: documents,
-          db_name: TEST_DB,
+          db_name: SpecConfig.instance.test_db,
           coll_name: TEST_COLL,
           write_concern: Mongo::WriteConcern.get(w: 1)
         })
@@ -103,9 +112,9 @@ describe Mongo::Operation::Delete do
       let(:op) do
         described_class.new({
           deletes: documents,
-          db_name: TEST_DB,
+          db_name: SpecConfig.instance.test_db,
           coll_name: TEST_COLL,
-          write_concern: Mongo::WriteConcern.get(WRITE_CONCERN)
+          write_concern: Mongo::WriteConcern.get(SpecConfig.instance.write_concern)
         })
       end
 
@@ -132,9 +141,9 @@ describe Mongo::Operation::Delete do
 
       let(:spec) do
         { :deletes       => documents,
-          :db_name       => TEST_DB,
+          :db_name       => SpecConfig.instance.test_db,
           :coll_name     => TEST_COLL,
-          :write_concern => Mongo::WriteConcern.get(WRITE_CONCERN),
+          :write_concern => Mongo::WriteConcern.get(SpecConfig.instance.write_concern),
           :ordered       => true
         }
       end
@@ -148,7 +157,7 @@ describe Mongo::Operation::Delete do
         context 'when write concern is acknowledged' do
 
           let(:write_concern) do
-            Mongo::WriteConcern.get(WRITE_CONCERN)
+            Mongo::WriteConcern.get(SpecConfig.instance.write_concern)
           end
 
           it 'aborts after first error' do
@@ -181,9 +190,9 @@ describe Mongo::Operation::Delete do
 
       let(:spec) do
         { :deletes       => documents,
-          :db_name       => TEST_DB,
+          :db_name       => SpecConfig.instance.test_db,
           :coll_name     => TEST_COLL,
-          :write_concern => Mongo::WriteConcern.get(WRITE_CONCERN),
+          :write_concern => Mongo::WriteConcern.get(SpecConfig.instance.write_concern),
           :ordered       => false
         }
       end

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

@@ -3,7 +3,7 @@ require 'spec_helper'
 describe Mongo::Operation::Delete::Command do
 
   let(:write_concern) do
-    Mongo::WriteConcern.get(WRITE_CONCERN)
+    Mongo::WriteConcern.get(SpecConfig.instance.write_concern)
   end
 
   let(:session) { nil }
@@ -89,6 +89,7 @@ describe Mongo::Operation::Delete::Command do
   describe '#message' do
 
     context 'when the server does not support OP_MSG' do
+      max_server_version '3.4'
 
       let(:expected_selector) do
         {
@@ -99,7 +100,7 @@ describe Mongo::Operation::Delete::Command do
         }
       end
 
-      it 'creates the correct query wire protocol message', unless: op_msg_enabled? do
+      it 'creates the correct query wire protocol message' do
         expect(Mongo::Protocol::Query).to receive(:new).with(authorized_collection.database.name,
                                                              '$cmd',
                                                              expected_selector,

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

@@ -3,7 +3,7 @@ require 'spec_helper'
 describe Mongo::Operation::Delete::OpMsg do
 
   let(:write_concern) do
-    Mongo::WriteConcern.get(WRITE_CONCERN)
+    Mongo::WriteConcern.get(SpecConfig.instance.write_concern)
   end
 
   let(:session) { nil }
@@ -95,7 +95,7 @@ describe Mongo::Operation::Delete::OpMsg do
             delete: TEST_COLL,
             ordered: true,
             writeConcern: write_concern.options,
-            '$db' => TEST_DB,
+            '$db' => SpecConfig.instance.test_db,
             lsid: session.session_id
         }
       end
@@ -113,7 +113,8 @@ describe Mongo::Operation::Delete::OpMsg do
         authorized_client.start_session
       end
 
-      context 'when the topology is replica set or sharded', if: (replica_set? || sharded?) && op_msg_enabled? do
+      context 'when the topology is replica set or sharded', if: (replica_set? || sharded?) do
+        min_server_version '3.6'
 
         let(:expected_global_args) do
           global_args.merge(Mongo::Operation::CLUSTER_TIME => authorized_client.cluster.cluster_time)
@@ -121,12 +122,13 @@ describe Mongo::Operation::Delete::OpMsg do
 
         it 'creates the correct OP_MSG message' do
           authorized_client.command(ping:1)
-          expect(Mongo::Protocol::Msg).to receive(:new).with([:none], {}, expected_global_args, expected_payload_1)
+          expect(Mongo::Protocol::Msg).to receive(:new).with([], {}, expected_global_args, expected_payload_1)
           op.send(:message, authorized_primary)
         end
       end
 
-      context 'when the topology is standalone', if: standalone? && op_msg_enabled? do
+      context 'when the topology is standalone', if: standalone? do
+        min_server_version '3.6'
 
         let(:expected_global_args) do
           global_args
@@ -134,25 +136,29 @@ describe Mongo::Operation::Delete::OpMsg do
 
         it 'creates the correct OP_MSG message' do
           authorized_client.command(ping:1)
-          expect(Mongo::Protocol::Msg).to receive(:new).with([:none], {}, expected_global_args, expected_payload_1)
+          expect(Mongo::Protocol::Msg).to receive(:new).with([], {}, expected_global_args, expected_payload_1)
           op.send(:message, authorized_primary)
         end
 
         context 'when an implicit session is created and the topology is then updated and the server does not support sessions' do
 
           let(:expected_global_args) do
-            global_args.delete(:lsid)
-            global_args
+            global_args.dup.tap do |args|
+              args.delete(:lsid)
+            end
           end
 
           before do
             session.instance_variable_set(:@options, { implicit: true })
+            # Topology is standalone, hence there is exactly one server
+            authorized_client.cluster.servers.first.monitor.stop!(true)
             allow(authorized_primary.features).to receive(:sessions_enabled?).and_return(false)
           end
 
           it 'creates the correct OP_MSG message' do
             authorized_client.command(ping:1)
-            expect(Mongo::Protocol::Msg).to receive(:new).with([:none], {}, expected_global_args, expected_payload_1)
+            expect(expected_global_args[:session]).to be nil
+            expect(Mongo::Protocol::Msg).to receive(:new).with([], {}, expected_global_args, expected_payload_1)
             op.send(:message, authorized_primary)
           end
         end
@@ -171,11 +177,15 @@ describe Mongo::Operation::Delete::OpMsg do
             authorized_client.send(:get_session)
           end
 
-          context 'when the topology is replica set or sharded', if: test_sessions? do
+          context 'when the topology is replica set or sharded' do
+            min_server_version '3.6'
+            require_topology :replica_set, :sharded
 
             let(:expected_global_args) do
-              global_args.delete(:lsid)
-              global_args.merge!(Mongo::Operation::CLUSTER_TIME => authorized_client.cluster.cluster_time)
+              global_args.dup.tap do |args|
+                args.delete(:lsid)
+                args.merge!(Mongo::Operation::CLUSTER_TIME => authorized_client.cluster.cluster_time)
+              end
             end
 
             it 'does not send a session id in the command' do
@@ -188,8 +198,9 @@ describe Mongo::Operation::Delete::OpMsg do
           context 'when the topology is standalone', if: standalone? && sessions_enabled? do
 
             let(:expected_global_args) do
-              global_args.delete(:lsid)
-              global_args
+              global_args.dup.tap do |args|
+                args.delete(:lsid)
+              end
             end
 
             it 'creates the correct OP_MSG message' do
@@ -200,15 +211,19 @@ describe Mongo::Operation::Delete::OpMsg do
           end
         end
 
-        context 'when the session is explicit', if: test_sessions? do
+        context 'when the session is explicit' do
+          min_server_version '3.6'
+          require_topology :replica_set, :sharded
 
           let(:session) do
             authorized_client.start_session
           end
 
           let(:expected_global_args) do
-            global_args.delete(:lsid)
-            global_args.merge!(Mongo::Operation::CLUSTER_TIME => authorized_client.cluster.cluster_time)
+            global_args.dup.tap do |args|
+              args.delete(:lsid)
+              args.merge!(Mongo::Operation::CLUSTER_TIME => authorized_client.cluster.cluster_time)
+            end
           end
 
           it 'does not send a session id in the command' do