mongo 2.13.2 → 2.15.0

data/spec/mongo/server/description/features_spec.rb

@@ -1,3 +1,6 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
 require 'spec_helper'
 
 describe Mongo::Server::Description::Features do

data/spec/mongo/server/description_query_methods_spec.rb

@@ -1,3 +1,6 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
 require 'spec_helper'
 
 # For conciseness these tests are arranged by description types
@@ -93,7 +96,7 @@ describe Mongo::Server::Description do
   end
 
   context 'primary' do
-    let(:desc_options) { {'ismaster' => true,
+    let(:desc_options) { {'isWritablePrimary' => true,
       'minWireVersion' => 2, 'maxWireVersion' => 8,
       'setName' => 'foo', 'ok' => ok} }
 

data/spec/mongo/server/description_spec.rb

@@ -1,815 +1,846 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
 require 'spec_helper'
 
 describe Mongo::Server::Description do
 
-  let(:replica) do
-    {
-      'setName' => 'mongodb_set',
-      'ismaster' => true,
-      'secondary' => false,
-      'hosts' => [
-        '127.0.0.1:27018',
-        '127.0.0.1:27019'
-      ],
-      'arbiters' => [
-        '127.0.0.1:27120'
-      ],
-      'primary' => authorized_primary.address.to_s,
-      'tags' => { 'rack' => 'a' },
-      'me' => '127.0.0.1:27019',
-      'maxBsonObjectSize' => 16777216,
-      'maxMessageSizeBytes' => 48000000,
-      'maxWriteBatchSize' => 1000,
-      'maxWireVersion' => 2,
-      'minWireVersion' => 1,
-      'localTime' => Time.now,
-      'lastWrite' => { 'lastWriteDate' => Time.now },
-      'logicalSessionTimeoutMinutes' => 7,
-      'operationTime' => 1,
-      '$clusterTime' => 1,
-      'ok' => 1
-    }
-  end
-
-  let(:address) do
-    Mongo::Address.new(authorized_primary.address.to_s)
-  end
-
-  let(:monitoring) do
-    Mongo::Monitoring.new(monitoring: false)
-  end
-
-  declare_topology_double
-
-  let(:cluster) do
-    double('cluster').tap do |cl|
-      allow(cl).to receive(:topology).and_return(topology)
-      allow(cl).to receive(:app_metadata).and_return(app_metadata)
-      allow(cl).to receive(:options).and_return({})
-    end
-  end
-
-  describe '#initialize' do
-    context 'when Time.now is mocked' do
-      it 'does not freeze mocked time' do
-        obj = Time.now
-        expect(Time).to receive(:now).at_least(:once).and_return(obj)
-        expect(obj.frozen?).to be false
-
-        description = described_class.new(address, replica)
-        expect(description.last_update_time).to eq(obj)
-        expect(obj.frozen?).to be false
+  %w[ismaster isWritablePrimary].each do |primary_param|
+    context "#{primary_param} as primary parameter" do
+
+      let(:replica) do
+        {
+          'setName' => 'mongodb_set',
+          primary_param => true,
+          'secondary' => false,
+          'hosts' => [
+            '127.0.0.1:27018',
+            '127.0.0.1:27019'
+          ],
+          'arbiters' => [
+            '127.0.0.1:27120'
+          ],
+          'primary' => authorized_primary.address.to_s,
+          'tags' => { 'rack' => 'a' },
+          'me' => '127.0.0.1:27019',
+          'maxBsonObjectSize' => 16777216,
+          'maxMessageSizeBytes' => 48000000,
+          'maxWriteBatchSize' => 1000,
+          'maxWireVersion' => 2,
+          'minWireVersion' => 1,
+          'localTime' => Time.now,
+          'lastWrite' => { 'lastWriteDate' => Time.now },
+          'logicalSessionTimeoutMinutes' => 7,
+          'operationTime' => 1,
+          '$clusterTime' => 1,
+          'connectionId' => 11,
+          'ok' => 1
+        }
+      end
+
+      let(:address) do
+        Mongo::Address.new(authorized_primary.address.to_s)
+      end
+
+      let(:monitoring) do
+        Mongo::Monitoring.new(monitoring: false)
+      end
+
+      declare_topology_double
+
+      let(:cluster) do
+        double('cluster').tap do |cl|
+          allow(cl).to receive(:topology).and_return(topology)
+          allow(cl).to receive(:app_metadata).and_return(app_metadata)
+          allow(cl).to receive(:options).and_return({})
+        end
       end
-    end
-  end
 
-  describe '#arbiters' do
+      describe '#arbiters' do
 
-    context 'when the replica set has arbiters' do
+        context 'when the replica set has arbiters' do
 
-      let(:description) do
-        described_class.new(address, replica)
-      end
+          let(:description) do
+            described_class.new(address, replica)
+          end
 
-      it 'returns the arbiters' do
-        expect(description.arbiters).to eq([ '127.0.0.1:27120' ])
-      end
-    end
+          it 'returns the arbiters' do
+            expect(description.arbiters).to eq([ '127.0.0.1:27120' ])
+          end
+        end
 
-    context 'when the replica set has no arbiters' do
+        context 'when the replica set has no arbiters' do
 
-      let(:description) do
-        described_class.new(address, {})
-      end
+          let(:description) do
+            described_class.new(address, {})
+          end
 
-      it 'returns an empty array' do
-        expect(description.arbiters).to be_empty
-      end
-    end
+          it 'returns an empty array' do
+            expect(description.arbiters).to be_empty
+          end
+        end
 
-    context 'when the addresses are not lowercase' do
+        context 'when the addresses are not lowercase' do
 
-      let(:config) do
-        replica.merge(
-                   {
-                       'arbiters' => [
-                           'SERVER:27017'
-                       ],
-                   }
-        )
-      end
+          let(:config) do
+            replica.merge(
+              {
+                'arbiters' => [
+                  'SERVER:27017'
+                ],
+              }
+            )
+          end
 
-      let(:description) do
-        described_class.new(address, config)
-      end
+          let(:description) do
+            described_class.new(address, config)
+          end
 
-      it 'normalizes the addresses to lowercase' do
-        expect(description.arbiters).to eq(['server:27017'])
+          it 'normalizes the addresses to lowercase' do
+            expect(description.arbiters).to eq(['server:27017'])
+          end
+        end
       end
-    end
-  end
 
-  describe '#hosts' do
+      describe '#hosts' do
 
-    let(:description) do
-      described_class.new(address, replica)
-    end
+        let(:description) do
+          described_class.new(address, replica)
+        end
 
-    it 'returns all the hosts in the replica set' do
-      expect(description.hosts).to eq([ '127.0.0.1:27018', '127.0.0.1:27019' ])
-    end
+        it 'returns all the hosts in the replica set' do
+          expect(description.hosts).to eq([ '127.0.0.1:27018', '127.0.0.1:27019' ])
+        end
 
-    context 'when the addresses are not lowercase' do
+        context 'when the addresses are not lowercase' do
 
-      let(:config) do
-        replica.merge(
-            {
+          let(:config) do
+            replica.merge(
+              {
                 'hosts' => [
-                    'SERVER:27017'
+                  'SERVER:27017'
                 ],
-            }
-        )
-      end
+              }
+            )
+          end
 
-      let(:description) do
-        described_class.new(address, config)
-      end
+          let(:description) do
+            described_class.new(address, config)
+          end
 
-      it 'normalizes the addresses to lowercase' do
-        expect(description.hosts).to eq(['server:27017'])
+          it 'normalizes the addresses to lowercase' do
+            expect(description.hosts).to eq(['server:27017'])
+          end
+        end
       end
-    end
-  end
 
-  describe '#max_bson_object_size' do
+      describe '#max_bson_object_size' do
 
-    let(:description) do
-      described_class.new(address, replica)
-    end
+        let(:description) do
+          described_class.new(address, replica)
+        end
 
-    it 'returns the value' do
-      expect(description.max_bson_object_size).to eq(16777216)
-    end
-  end
+        it 'returns the value' do
+          expect(description.max_bson_object_size).to eq(16777216)
+        end
+      end
 
-  describe '#max_message_size' do
+      describe '#max_message_size' do
 
-    let(:description) do
-      described_class.new(address, replica)
-    end
+        let(:description) do
+          described_class.new(address, replica)
+        end
 
-    it 'returns the value' do
-      expect(description.max_message_size).to eq(48000000)
-    end
-  end
+        it 'returns the value' do
+          expect(description.max_message_size).to eq(48000000)
+        end
+      end
 
-  describe '#max_write_batch_size' do
+      describe '#max_write_batch_size' do
 
-    let(:description) do
-      described_class.new(address, replica)
-    end
+        let(:description) do
+          described_class.new(address, replica)
+        end
 
-    it 'returns the value' do
-      expect(description.max_write_batch_size).to eq(1000)
-    end
-  end
+        it 'returns the value' do
+          expect(description.max_write_batch_size).to eq(1000)
+        end
+      end
 
-  describe '#max_wire_version' do
+      describe '#max_wire_version' do
 
-    context 'when the max wire version is provided' do
+        context 'when the max wire version is provided' do
 
-      let(:description) do
-        described_class.new(address, replica)
-      end
+          let(:description) do
+            described_class.new(address, replica)
+          end
 
-      it 'returns the value' do
-        expect(description.max_wire_version).to eq(2)
-      end
-    end
+          it 'returns the value' do
+            expect(description.max_wire_version).to eq(2)
+          end
+        end
 
-    context 'when the max wire version is not provided' do
+        context 'when the max wire version is not provided' do
 
-      let(:description) do
-        described_class.new(address, {})
-      end
+          let(:description) do
+            described_class.new(address, {})
+          end
 
-      it 'returns the default' do
-        expect(description.max_wire_version).to eq(0)
+          it 'returns the default' do
+            expect(description.max_wire_version).to eq(0)
+          end
+        end
       end
-    end
-  end
 
-  describe '#min_wire_version' do
+      describe '#min_wire_version' do
 
-    context 'when the min wire version is provided' do
+        context 'when the min wire version is provided' do
 
-      let(:description) do
-        described_class.new(address, replica)
-      end
+          let(:description) do
+            described_class.new(address, replica)
+          end
 
-      it 'returns the value' do
-        expect(description.min_wire_version).to eq(1)
-      end
-    end
+          it 'returns the value' do
+            expect(description.min_wire_version).to eq(1)
+          end
+        end
 
-    context 'when the min wire version is not provided' do
+        context 'when the min wire version is not provided' do
 
-      let(:description) do
-        described_class.new(address, {})
-      end
+          let(:description) do
+            described_class.new(address, {})
+          end
 
-      it 'returns the default' do
-        expect(description.min_wire_version).to eq(0)
+          it 'returns the default' do
+            expect(description.min_wire_version).to eq(0)
+          end
+        end
       end
-    end
-  end
 
-  describe '#tags' do
+      describe '#tags' do
 
-    context 'when the server has tags' do
+        context 'when the server has tags' do
 
-      let(:description) do
-        described_class.new(address, replica)
-      end
+          let(:description) do
+            described_class.new(address, replica)
+          end
 
-      it 'returns the tags' do
-        expect(description.tags).to eq(replica['tags'])
-      end
-    end
+          it 'returns the tags' do
+            expect(description.tags).to eq(replica['tags'])
+          end
+        end
 
-    context 'when the server does not have tags' do
+        context 'when the server does not have tags' do
 
-      let(:config) do
-        { 'ismaster' => true }
-      end
+          let(:config) do
+            { primary_param => true }
+          end
 
-      let(:description) do
-        described_class.new(address, config)
-      end
+          let(:description) do
+            described_class.new(address, config)
+          end
 
-      it 'returns an empty hash' do
-        expect(description.tags).to eq({})
+          it 'returns an empty hash' do
+            expect(description.tags).to eq({})
+          end
+        end
       end
-    end
-  end
 
-  describe '#passives' do
+      describe '#passives' do
 
-    context 'when passive servers exists' do
+        context 'when passive servers exists' do
 
-      let(:description) do
-        described_class.new(address, { 'passives' => [ '127.0.0.1:27025' ] })
-      end
+          let(:description) do
+            described_class.new(address, { 'passives' => [ '127.0.0.1:27025' ] })
+          end
 
-      it 'returns a list of the passives' do
-        expect(description.passives).to eq([ '127.0.0.1:27025' ])
-      end
-    end
+          it 'returns a list of the passives' do
+            expect(description.passives).to eq([ '127.0.0.1:27025' ])
+          end
+        end
 
-    context 'when no passive servers exist' do
+        context 'when no passive servers exist' do
 
-      let(:description) do
-        described_class.new(address, replica)
-      end
+          let(:description) do
+            described_class.new(address, replica)
+          end
 
-      it 'returns an empty array' do
-        expect(description.passives).to be_empty
-      end
-    end
+          it 'returns an empty array' do
+            expect(description.passives).to be_empty
+          end
+        end
 
-    context 'when the addresses are not lowercase' do
+        context 'when the addresses are not lowercase' do
 
-      let(:config) do
-        replica.merge(
-            {
+          let(:config) do
+            replica.merge(
+              {
                 'passives' => [
-                    'SERVER:27017'
+                  'SERVER:27017'
                 ],
-            }
-        )
-      end
+              }
+            )
+          end
 
-      let(:description) do
-        described_class.new(address, config)
-      end
+          let(:description) do
+            described_class.new(address, config)
+          end
 
-      it 'normalizes the addresses to lowercase' do
-        expect(description.passives).to eq(['server:27017'])
-      end
+          it 'normalizes the addresses to lowercase' do
+            expect(description.passives).to eq(['server:27017'])
+          end
 
-      it 'normalizes the addresses to lowercase' do
+          it 'normalizes the addresses to lowercase' do
 
+          end
+        end
       end
-    end
-  end
 
-  describe '#primary?' do
+      describe '#primary?' do
 
-    context 'when the server is a primary' do
+        context 'when the server is a primary' do
 
-      context 'when the hostname contains no capital letters' do
+          context 'when the hostname contains no capital letters' do
 
-        let(:description) do
-          described_class.new(address, replica)
-        end
+            let(:description) do
+              described_class.new(address, replica)
+            end
+
+            it 'returns true' do
+              expect(description).to be_primary
+            end
+          end
+
+          context 'when the hostname contains capital letters' do
 
-        it 'returns true' do
-          expect(description).to be_primary
+            let(:description) do
+              described_class.new('localhost:27017',
+                                  { primary_param => true, 'ok' => 1,
+                                    'minWireVersion' => 2, 'maxWireVersion' => 3,
+                                    'primary' => 'LOCALHOST:27017',
+                                    'setName' => 'itsASet!'})
+            end
+
+            it 'returns true' do
+              expect(description).to be_primary
+            end
+          end
         end
       end
 
-      context 'when the hostname contains capital letters' do
+      describe '#average_round_trip_time' do
 
         let(:description) do
-          described_class.new('localhost:27017',
-                              { 'ismaster' => true, 'ok' => 1,
-                                'minWireVersion' => 2, 'maxWireVersion' => 3,
-                                'primary' => 'LOCALHOST:27017',
-                                'setName' => 'itsASet!'})
+          described_class.new(address, { 'secondary' => false }, 4.5)
         end
 
-        it 'returns true' do
-          expect(description).to be_primary
+        it 'defaults to nil' do
+          expect(described_class.new(address).average_round_trip_time).to be nil
+        end
+
+        it 'can be set via the constructor' do
+          expect(description.average_round_trip_time).to eq(4.5)
         end
       end
-    end
-  end
 
-  describe '#average_round_trip_time' do
+      describe '#replica_set_name' do
 
-    let(:description) do
-      described_class.new(address, { 'secondary' => false }, 4.5)
-    end
+        context 'when the server is in a replica set' do
 
-    it 'defaults to nil' do
-      expect(described_class.new(address).average_round_trip_time).to be nil
-    end
+          let(:description) do
+            described_class.new(address, replica)
+          end
 
-    it 'can be set via the constructor' do
-      expect(description.average_round_trip_time).to eq(4.5)
-    end
-  end
+          it 'returns the replica set name' do
+            expect(description.replica_set_name).to eq('mongodb_set')
+          end
+        end
 
-  describe '#replica_set_name' do
+        context 'when the server is not in a replica set' do
 
-    context 'when the server is in a replica set' do
+          let(:description) do
+            described_class.new(address, {})
+          end
 
-      let(:description) do
-        described_class.new(address, replica)
+          it 'returns nil' do
+            expect(description.replica_set_name).to be_nil
+          end
+        end
       end
 
-      it 'returns the replica set name' do
-        expect(description.replica_set_name).to eq('mongodb_set')
-      end
-    end
+      describe '#servers' do
 
-    context 'when the server is not in a replica set' do
+        let(:config) do
+          replica.merge({ 'passives' => [ '127.0.0.1:27025' ]})
+        end
 
-      let(:description) do
-        described_class.new(address, {})
-      end
+        let(:description) do
+          described_class.new(address, config)
+        end
 
-      it 'returns nil' do
-        expect(description.replica_set_name).to be_nil
+        it 'returns the hosts + arbiters + passives' do
+          expect(description.servers).to eq(
+            [ '127.0.0.1:27018', '127.0.0.1:27019', '127.0.0.1:27120', '127.0.0.1:27025' ]
+          )
+        end
       end
-    end
-  end
 
-  describe '#servers' do
+      describe '#server_type' do
 
-    let(:config) do
-      replica.merge({ 'passives' => [ '127.0.0.1:27025' ]})
-    end
+        context 'when the server is an arbiter' do
 
-    let(:description) do
-      described_class.new(address, config)
-    end
+          let(:description) do
+            described_class.new(address, { 'arbiterOnly' => true,
+                                           'minWireVersion' => 2, 'maxWireVersion' => 3,
+                                           'setName' => 'test', 'ok' => 1 })
+          end
 
-    it 'returns the hosts + arbiters + passives' do
-      expect(description.servers).to eq(
-        [ '127.0.0.1:27018', '127.0.0.1:27019', '127.0.0.1:27120', '127.0.0.1:27025' ]
-      )
-    end
-  end
+          it 'returns :arbiter' do
+            expect(description.server_type).to eq(:arbiter)
+          end
+        end
 
-  describe '#server_type' do
+        context 'when the server is a ghost' do
 
-    context 'when the server is an arbiter' do
+          let(:description) do
+            described_class.new(address, { 'isreplicaset' => true,
+                                           'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
+          end
 
-      let(:description) do
-        described_class.new(address, { 'arbiterOnly' => true,
-          'minWireVersion' => 2, 'maxWireVersion' => 3,
-          'setName' => 'test', 'ok' => 1 })
-      end
+          it 'returns :ghost' do
+            expect(description.server_type).to eq(:ghost)
+          end
+        end
 
-      it 'returns :arbiter' do
-        expect(description.server_type).to eq(:arbiter)
-      end
-    end
+        context 'when the server is a mongos' do
+
+          let(:config) do
+            { 'msg' => 'isdbgrid', primary_param => true,
+              'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 }
+          end
+
+          let(:description) do
+            described_class.new(address, config)
+          end
+
+          it 'returns :sharded' do
+            expect(description.server_type).to eq(:sharded)
+          end
+
+          context 'when client and server addresses are different' do
+            let(:config) do
+              { 'msg' => 'isdbgrid', primary_param => true,
+                'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1,
+                'me' => '127.0.0.1',
+              }
+            end
+
+            let(:address) do
+              Mongo::Address.new('localhost')
+            end
+
+            it 'returns :sharded' do
+              expect(description.server_type).to eq(:sharded)
+            end
+          end
+        end
 
-    context 'when the server is a ghost' do
+        context 'when the server is a primary' do
 
-      let(:description) do
-        described_class.new(address, { 'isreplicaset' => true,
-          'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
-      end
+          let(:description) do
+            described_class.new(address, replica)
+          end
 
-      it 'returns :ghost' do
-        expect(description.server_type).to eq(:ghost)
-      end
-    end
+          it 'returns :primary' do
+            expect(description.server_type).to eq(:primary)
+          end
+        end
 
-    context 'when the server is a mongos' do
+        context 'when the server is a secondary' do
 
-      let(:config) do
-        { 'msg' => 'isdbgrid', 'ismaster' => true,
-          'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 }
-      end
+          let(:description) do
+            described_class.new(address, { 'secondary' => true,
+                                           'minWireVersion' => 2, 'maxWireVersion' => 3,
+                                           'setName' => 'test', 'ok' => 1 })
+          end
 
-      let(:description) do
-        described_class.new(address, config)
-      end
+          it 'returns :secondary' do
+            expect(description.server_type).to eq(:secondary)
+          end
+        end
 
-      it 'returns :sharded' do
-        expect(description.server_type).to eq(:sharded)
-      end
+        context 'when the server is standalone' do
 
-      context 'when client and server addresses are different' do
-        let(:config) do
-          { 'msg' => 'isdbgrid', 'ismaster' => true,
-            'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1,
-            'me' => '127.0.0.1',
-          }
-        end
+          let(:description) do
+            described_class.new(address, { primary_param => true,
+                                           'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
+          end
 
-        let(:address) do
-          Mongo::Address.new('localhost')
+          it 'returns :standalone' do
+            expect(description.server_type).to eq(:standalone)
+          end
         end
 
-        it 'returns :sharded' do
-          expect(description.server_type).to eq(:sharded)
-        end
-      end
-    end
+        context 'when the server is hidden' do
 
-    context 'when the server is a primary' do
+          let(:description) do
+            described_class.new(address, { primary_param => false,
+                                           'minWireVersion' => 2, 'maxWireVersion' => 3, 'setName' => 'test',
+                                           'hidden' => true, 'ok' => 1 })
+          end
 
-      let(:description) do
-        described_class.new(address, replica)
-      end
+          it 'returns :other' do
+            expect(description.server_type).to eq(:other)
+          end
+        end
 
-      it 'returns :primary' do
-        expect(description.server_type).to eq(:primary)
-      end
-    end
+        context 'when the server is other' do
 
-    context 'when the server is a secondary' do
+          let(:description) do
+            described_class.new(address, { primary_param => false,
+                                           'minWireVersion' => 2, 'maxWireVersion' => 3, 'setName' => 'test',
+                                           'ok' => 1 })
+          end
 
-      let(:description) do
-        described_class.new(address, { 'secondary' => true,
-          'minWireVersion' => 2, 'maxWireVersion' => 3,
-          'setName' => 'test', 'ok' => 1 })
-      end
+          it 'returns :other' do
+            expect(description.server_type).to eq(:other)
+          end
+        end
 
-      it 'returns :secondary' do
-        expect(description.server_type).to eq(:secondary)
-      end
-    end
+        context 'when the description has no configuration' do
 
-    context 'when the server is standalone' do
+          let(:description) do
+            described_class.new(address)
+          end
 
-      let(:description) do
-        described_class.new(address, { 'ismaster' => true,
-          'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
+          it 'returns :unknown' do
+            expect(description.server_type).to eq(:unknown)
+          end
+        end
       end
 
-      it 'returns :standalone' do
-        expect(description.server_type).to eq(:standalone)
-      end
-    end
+      describe '#is_server?' do
 
-    context 'when the server is hidden' do
+        let(:listeners) do
+          Mongo::Event::Listeners.new
+        end
 
-      let(:description) do
-        described_class.new(address, { 'ismaster' => false,
-          'minWireVersion' => 2, 'maxWireVersion' => 3, 'setName' => 'test',
-          'hidden' => true, 'ok' => 1 })
-      end
+        let(:server) do
+          Mongo::Server.new(address, cluster, monitoring, listeners,
+                            monitoring_io: false)
+        end
 
-      it 'returns :other' do
-        expect(description.server_type).to eq(:other)
-      end
-    end
+        let(:description) do
+          described_class.new(address, {})
+        end
 
-    context 'when the server is other' do
+        context 'when the server address matches the description address' do
 
-      let(:description) do
-        described_class.new(address, { 'ismaster' => false,
-          'minWireVersion' => 2, 'maxWireVersion' => 3, 'setName' => 'test',
-          'ok' => 1 })
-      end
+          it 'returns true' do
+            expect(description.is_server?(server)).to be(true)
+          end
+        end
 
-      it 'returns :other' do
-        expect(description.server_type).to eq(:other)
-      end
-    end
+        context 'when the server address does not match the description address' do
 
-    context 'when the description has no configuration' do
+          let(:other_address) do
+            Mongo::Address.new('127.0.0.1:27020')
+          end
 
-      let(:description) do
-        described_class.new(address)
-      end
+          let(:server) do
+            Mongo::Server.new(other_address, cluster, monitoring, listeners,
+                              monitoring_io: false)
+          end
 
-      it 'returns :unknown' do
-        expect(description.server_type).to eq(:unknown)
+          it 'returns false' do
+            expect(description.is_server?(server)).to be(false)
+          end
+        end
       end
-    end
-  end
 
-  describe '#is_server?' do
+      describe '#me_mismatch?' do
 
-    let(:listeners) do
-      Mongo::Event::Listeners.new
-    end
+        let(:description) do
+          described_class.new(address, config)
+        end
 
-    let(:server) do
-      Mongo::Server.new(address, cluster, monitoring, listeners,
-        monitoring_io: false)
-    end
+        context 'when the server address matches the me field' do
 
-    let(:description) do
-      described_class.new(address, {})
-    end
+          let(:config) do
+            replica.merge('me' => address.to_s)
+          end
+
+          it 'returns false' do
+            expect(description.me_mismatch?).to be(false)
+          end
+        end
 
-    context 'when the server address matches the description address' do
+        context 'when the server address does not match the me field' do
 
-      it 'returns true' do
-        expect(description.is_server?(server)).to be(true)
-      end
-    end
+          let(:config) do
+            replica.merge('me' => 'localhost:27020')
+          end
 
-    context 'when the server address does not match the description address' do
+          it 'returns true' do
+            expect(description.me_mismatch?).to be(true)
+          end
+        end
 
-      let(:other_address) do
-        Mongo::Address.new('127.0.0.1:27020')
-      end
+        context 'when there is no me field' do
 
-      let(:server) do
-        Mongo::Server.new(other_address, cluster, monitoring, listeners,
-          monitoring_io: false)
-      end
+          let(:config) do
+            replica.tap do |r|
+              r.delete('me')
+            end
+          end
 
-      it 'returns false' do
-        expect(description.is_server?(server)).to be(false)
+          it 'returns false' do
+            expect(description.me_mismatch?).to be(false)
+          end
+        end
       end
-    end
-  end
 
-  describe '#me_mismatch?' do
+      describe '#lists_server?' do
 
-    let(:description) do
-      described_class.new(address, config)
-    end
+        let(:description) do
+          described_class.new(address, replica)
+        end
 
-    context 'when the server address matches the me field' do
+        let(:server_address) do
+          Mongo::Address.new('127.0.0.1:27018')
+        end
 
-      let(:config) do
-        replica.merge('me' => address.to_s)
-      end
+        let(:listeners) do
+          Mongo::Event::Listeners.new
+        end
 
-      it 'returns false' do
-        expect(description.me_mismatch?).to be(false)
-      end
-    end
+        let(:server) do
+          Mongo::Server.new(server_address, cluster, monitoring, listeners,
+                            monitoring_io: false)
+        end
 
-    context 'when the server address does not match the me field' do
+        context 'when the server is included in the description hosts list' do
 
-      let(:config) do
-        replica.merge('me' => 'localhost:27020')
-      end
+          it 'returns true' do
+            expect(description.lists_server?(server)).to be(true)
+          end
+        end
 
-      it 'returns true' do
-        expect(description.me_mismatch?).to be(true)
-      end
-    end
+        context 'when the server is not included in the description hosts list' do
 
-    context 'when there is no me field' do
+          let(:server_address) do
+            Mongo::Address.new('127.0.0.1:27017')
+          end
 
-      let(:config) do
-        replica.tap do |r|
-          r.delete('me')
+          it 'returns false' do
+            expect(description.lists_server?(server)).to be(false)
+          end
         end
       end
 
-      it 'returns false' do
-        expect(description.me_mismatch?).to be(false)
-      end
-    end
-  end
-
-  describe '#lists_server?' do
+      describe '#replica_set_member?' do
 
-    let(:description) do
-      described_class.new(address, replica)
-    end
+        context 'when the description is from a mongos' do
 
-    let(:server_address) do
-      Mongo::Address.new('127.0.0.1:27018')
-    end
+          let(:config) do
+            { 'msg' => 'isdbgrid', primary_param => true }
+          end
 
-    let(:listeners) do
-      Mongo::Event::Listeners.new
-    end
+          let(:description) do
+            described_class.new(address, config)
+          end
 
-    let(:server) do
-      Mongo::Server.new(server_address, cluster, monitoring, listeners,
-        monitoring_io: false)
-    end
-
-    context 'when the server is included in the description hosts list' do
-
-      it 'returns true' do
-        expect(description.lists_server?(server)).to be(true)
-      end
-    end
+          it 'returns false' do
+            expect(description.replica_set_member?).to be(false)
+          end
+        end
 
-    context 'when the server is not included in the description hosts list' do
+        context 'when the description is from a standalone' do
 
-      let(:server_address) do
-        Mongo::Address.new('127.0.0.1:27017')
-      end
+          let(:description) do
+            described_class.new(address, { primary_param => true,
+                                           'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
+          end
 
-      it 'returns false' do
-        expect(description.lists_server?(server)).to be(false)
-      end
-    end
-  end
+          it 'returns false' do
+            expect(description.replica_set_member?).to be(false)
+          end
+        end
 
-  describe '#replica_set_member?' do
+        context 'when the description is from a replica set member' do
 
-    context 'when the description is from a mongos' do
+          let(:description) do
+            described_class.new(address, replica)
+          end
 
-      let(:config) do
-        { 'msg' => 'isdbgrid', 'ismaster' => true }
+          it 'returns true' do
+            expect(description.replica_set_member?).to be(true)
+          end
+        end
       end
 
-      let(:description) do
-        described_class.new(address, config)
-      end
+      describe '#logical_session_timeout_minutes' do
 
-      it 'returns false' do
-        expect(description.replica_set_member?).to be(false)
-      end
-    end
+        context 'when a logical session timeout value is in the config' do
 
-    context 'when the description is from a standalone' do
+          let(:description) do
+            described_class.new(address, replica)
+          end
 
-      let(:description) do
-        described_class.new(address, { 'ismaster' => true,
-          'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
-      end
+          it 'returns the logical session timeout value' do
+            expect(description.logical_session_timeout).to eq(7)
+          end
+        end
 
-      it 'returns false' do
-        expect(description.replica_set_member?).to be(false)
-      end
-    end
+        context 'when a logical session timeout value is not in the config' do
 
-    context 'when the description is from a replica set member' do
+          let(:description) do
+            described_class.new(address, { primary_param => true,
+                                           'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
+          end
 
-      let(:description) do
-        described_class.new(address, replica)
+          it 'returns nil' do
+            expect(description.logical_session_timeout).to be(nil)
+          end
+        end
       end
 
-      it 'returns true' do
-        expect(description.replica_set_member?).to be(true)
-      end
-    end
-  end
+      describe '#==' do
 
-  describe '#logical_session_timeout_minutes' do
-
-    context 'when a logical session timeout value is in the config' do
+        let(:description) do
+          described_class.new(address, replica)
+        end
 
-      let(:description) do
-        described_class.new(address, replica)
-      end
+        let(:other) do
+          described_class.new(address, replica.merge(
+            'localTime' => 1,
+            'lastWrite' => { 'lastWriteDate' => 1 },
+            'operationTime' => 2,
+            '$clusterTime' => 2
+          ))
+        end
 
-      it 'returns the logical session timeout value' do
-        expect(description.logical_session_timeout).to eq(7)
-      end
-    end
+        it 'excludes certain fields' do
+          expect(description == other).to be(true)
+        end
 
-    context 'when a logical session timeout value is not in the config' do
+        context 'when the classes do not match' do
 
-      let(:description) do
-        described_class.new(address, { 'ismaster' => true,
-          'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
-      end
+          let(:description) do
+            described_class.new(address, replica)
+          end
 
-      it 'returns nil' do
-        expect(description.logical_session_timeout).to be(nil)
-      end
-    end
-  end
+          it 'returns false' do
+            expect(description == Array.new).to be(false)
+          end
+        end
 
-  describe '#==' do
+        context 'when the configs match' do
 
-    let(:description) do
-      described_class.new(address, replica)
-    end
+          let(:description) do
+            described_class.new(address, replica)
+          end
 
-    let(:other) do
-      described_class.new(address, replica.merge(
-        'localTime' => 1,
-        'lastWrite' => { 'lastWriteDate' => 1 },
-        'operationTime' => 2,
-        '$clusterTime' => 2
-      ))
-    end
+          let(:other) do
+            described_class.new(address, replica)
+          end
 
-    it 'excludes certain fields' do
-      expect(description == other).to be(true)
-    end
+          it 'returns true' do
+            expect(description == other).to be(true)
+          end
+        end
 
-    context 'when the classes do not match' do
+        context 'when the configs match, but have different connectionId values' do
 
-      let(:description) do
-        described_class.new(address, replica)
-      end
+          let(:description) do
+            described_class.new(address, replica)
+          end
 
-      it 'returns false' do
-        expect(description == Array.new).to be(false)
-      end
-    end
+          let(:other) do
+            described_class.new(address, replica.merge(
+              'connectionId' => 12
+            ))
+          end
 
-    context 'when the configs match' do
+          it 'returns true' do
+            expect(description == other).to be(true)
+          end
+        end
 
-      let(:description) do
-        described_class.new(address, replica)
-      end
+        context 'when the configs do not match' do
 
-      let(:other) do
-        described_class.new(address, replica)
-      end
+          let(:description) do
+            described_class.new(address, replica)
+          end
 
-      it 'returns true' do
-        expect(description == other).to be(true)
-      end
-    end
+          let(:other) do
+            described_class.new(address, { primary_param => true,
+                                           'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
+          end
 
-    context 'when the configs do not match' do
+          it 'returns false' do
+            expect(description == other).to be(false)
+          end
+        end
 
-      let(:description) do
-        described_class.new(address, replica)
+        context 'when one config is a subset of the other' do
+          let(:one) do
+            described_class.new(address, { primary_param => true,
+                                           'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
+          end
+
+          let(:two) do
+            described_class.new(address, { primary_param => true,
+                                           'minWireVersion' => 2, 'maxWireVersion' => 3,
+                                           'ok' => 1, 'setName' => 'mongodb_set' })
+          end
+
+          it 'returns false when first config is the receiver' do
+            expect(one == two).to be false
+          end
+
+          it 'returns false when second config is the receiver' do
+            expect(two == one).to be false
+          end
+        end
       end
 
-      let(:other) do
-        described_class.new(address, { 'ismaster' => true,
-          'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
-      end
+      describe '#last_update_time' do
+        context 'stub description' do
+          let(:description) { described_class.new(address) }
 
-      it 'returns false' do
-        expect(description == other).to be(false)
-      end
-    end
+          it 'is present' do
+            expect(description.last_update_time).to be_a(Time)
+          end
+        end
 
-    context 'when one config is a subset of the other' do
-      let(:one) do
-        described_class.new(address, { 'ismaster' => true,
-          'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
-      end
+        context 'filled out description' do
+          let(:description) { described_class.new(address, replica) }
 
-      let(:two) do
-        described_class.new(address, { 'ismaster' => true,
-          'minWireVersion' => 2, 'maxWireVersion' => 3,
-          'ok' => 1, 'setName' => 'mongodb_set' })
+          it 'is present' do
+            expect(description.last_update_time).to be_a(Time)
+          end
+        end
       end
 
-      it 'returns false when first config is the receiver' do
-        expect(one == two).to be false
-      end
+      describe '#last_update_monotime' do
+        context 'stub description' do
+          let(:description) { described_class.new(address) }
 
-      it 'returns false when second config is the receiver' do
-        expect(two == one).to be false
-      end
-    end
-  end
+          it 'is present' do
+            expect(description.last_update_monotime).to be_a(Float)
+          end
+        end
 
-  describe '#last_update_time' do
-    context 'stub description' do
-      let(:description) { described_class.new(address) }
+        context 'filled out description' do
+          let(:description) { described_class.new(address, replica) }
 
-      it 'is present' do
-        expect(description.last_update_time).to be_a(Time)
+          it 'is present' do
+            expect(description.last_update_monotime).to be_a(Float)
+          end
+        end
       end
     end
 
-    context 'filled out description' do
-      let(:description) { described_class.new(address, replica) }
-
-      it 'is present' do
-        expect(description.last_update_time).to be_a(Time)
-      end
-    end
   end
 end