mongo 2.4.1 → 2.14.0

data/spec/mongo/server/description_spec.rb

@@ -14,59 +14,52 @@ describe Mongo::Server::Description do
       'arbiters' => [
         '127.0.0.1:27120'
       ],
-      'primary' => '127.0.0.1:27019',
+      'primary' => authorized_primary.address.to_s,
       'tags' => { 'rack' => 'a' },
       'me' => '127.0.0.1:27019',
       'maxBsonObjectSize' => 16777216,
       'maxMessageSizeBytes' => 48000000,
       'maxWriteBatchSize' => 1000,
       'maxWireVersion' => 2,
-      'minWireVersion' => 0,
+      '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('127.0.0.1:27017')
+    Mongo::Address.new(authorized_primary.address.to_s)
   end
 
   let(:monitoring) do
     Mongo::Monitoring.new(monitoring: false)
   end
 
-  let(:topology) do
-    double('topology')
-  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 '#arbiter?' do
-
-    context 'when the server is an arbiter' do
-
-      let(:description) do
-        described_class.new(address, { 'arbiterOnly' => true, 'setName' => 'test' })
-      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
 
-      it 'returns true' do
-        expect(description).to be_arbiter
-      end
-    end
-
-    context 'when the server is not an arbiter' do
-
-      let(:description) do
-        described_class.new(address, replica)
-      end
-
-      it 'returns false' do
-        expect(description).to_not be_arbiter
+        description = described_class.new(address, replica)
+        expect(description.last_update_time).to eq(obj)
+        expect(obj.frozen?).to be false
       end
     end
   end
@@ -117,60 +110,6 @@ describe Mongo::Server::Description do
     end
   end
 
-  describe '#ghost?' do
-
-    context 'when the server is a ghost' do
-
-      let(:config) do
-        { 'isreplicaset' => true }
-      end
-
-      let(:description) do
-        described_class.new(address, config)
-      end
-
-      it 'returns true' do
-        expect(description).to be_ghost
-      end
-    end
-
-    context 'when the server is not a ghost' do
-
-      let(:description) do
-        described_class.new(address, replica)
-      end
-
-      it 'returns false' do
-        expect(description).to_not be_ghost
-      end
-    end
-  end
-
-  describe '#hidden?' do
-
-    context 'when the server is hidden' do
-
-      let(:description) do
-        described_class.new(address, { 'hidden' => true })
-      end
-
-      it 'returns true' do
-        expect(description).to be_hidden
-      end
-    end
-
-    context 'when the server is not hidden' do
-
-      let(:description) do
-        described_class.new(address, replica)
-      end
-
-      it 'returns false' do
-        expect(description).to_not be_hidden
-      end
-    end
-  end
-
   describe '#hosts' do
 
     let(:description) do
@@ -270,7 +209,7 @@ describe Mongo::Server::Description do
       end
 
       it 'returns the value' do
-        expect(description.min_wire_version).to eq(0)
+        expect(description.min_wire_version).to eq(1)
       end
     end
 
@@ -315,60 +254,6 @@ describe Mongo::Server::Description do
     end
   end
 
-  describe '#mongos?' do
-
-    context 'when the server is a mongos' do
-
-      let(:config) do
-        { 'msg' => 'isdbgrid', 'ismaster' => true }
-      end
-
-      let(:description) do
-        described_class.new(address, config)
-      end
-
-      it 'returns true' do
-        expect(description).to be_mongos
-      end
-    end
-
-    context 'when the server is not a mongos' do
-
-      let(:description) do
-        described_class.new(address, replica)
-      end
-
-      it 'returns false' do
-        expect(description).to_not be_mongos
-      end
-    end
-  end
-
-  describe '#passive?' do
-
-    context 'when the server is passive' do
-
-      let(:description) do
-        described_class.new(address, { 'passive' => true })
-      end
-
-      it 'returns true' do
-        expect(description).to be_passive
-      end
-    end
-
-    context 'when the server is not passive' do
-
-      let(:description) do
-        described_class.new(address, replica)
-      end
-
-      it 'returns false' do
-        expect(description).to_not be_passive
-      end
-    end
-  end
-
   describe '#passives' do
 
     context 'when passive servers exists' do
@@ -421,25 +306,32 @@ describe Mongo::Server::Description do
 
   describe '#primary?' do
 
-    context 'when the server is not a primary' do
+    context 'when the server is a primary' do
+
+      context 'when the hostname contains no capital letters' do
 
-      let(:description) do
-        described_class.new(address, { 'ismaster' => false })
-      end
+        let(:description) do
+          described_class.new(address, replica)
+        end
 
-      it 'returns true' do
-        expect(description).to_not be_primary
+        it 'returns true' do
+          expect(description).to be_primary
+        end
       end
-    end
 
-    context 'when the server is a primary' do
+      context 'when the hostname contains capital letters' do
 
-      let(:description) do
-        described_class.new(address, replica)
-      end
+        let(:description) do
+          described_class.new('localhost:27017',
+                              { 'ismaster' => true, 'ok' => 1,
+                                'minWireVersion' => 2, 'maxWireVersion' => 3,
+                                'primary' => 'LOCALHOST:27017',
+                                'setName' => 'itsASet!'})
+        end
 
-      it 'returns false' do
-        expect(description).to be_primary
+        it 'returns true' do
+          expect(description).to be_primary
+        end
       end
     end
   end
@@ -450,8 +342,8 @@ describe Mongo::Server::Description do
       described_class.new(address, { 'secondary' => false }, 4.5)
     end
 
-    it 'defaults to 0' do
-      expect(described_class.new(address).average_round_trip_time).to eq(0)
+    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
@@ -484,31 +376,6 @@ describe Mongo::Server::Description do
     end
   end
 
-  describe '#secondary?' do
-
-    context 'when the server is not a secondary' do
-
-      let(:description) do
-        described_class.new(address, { 'secondary' => false })
-      end
-
-      it 'returns true' do
-        expect(description).to_not be_secondary
-      end
-    end
-
-    context 'when the server is a secondary' do
-
-      let(:description) do
-        described_class.new(address, { 'secondary' => true, 'setName' => 'test' })
-      end
-
-      it 'returns false' do
-        expect(description).to be_secondary
-      end
-    end
-  end
-
   describe '#servers' do
 
     let(:config) do
@@ -526,37 +393,14 @@ describe Mongo::Server::Description do
     end
   end
 
-  describe '#standalone?' do
-
-    context 'when the server is standalone' do
-
-      let(:description) do
-        described_class.new(address, { 'ismaster' => true, 'ok' => 1 })
-      end
-
-      it 'returns true' do
-        expect(description).to be_standalone
-      end
-    end
-
-    context 'when the server is part of a replica set' do
-
-      let(:description) do
-        described_class.new(address, replica)
-      end
-
-      it 'returns false' do
-        expect(description).to_not be_standalone
-      end
-    end
-  end
-
   describe '#server_type' do
 
     context 'when the server is an arbiter' do
 
       let(:description) do
-        described_class.new(address, { 'arbiterOnly' => true, 'setName' => 'test' })
+        described_class.new(address, { 'arbiterOnly' => true,
+          'minWireVersion' => 2, 'maxWireVersion' => 3,
+          'setName' => 'test', 'ok' => 1 })
       end
 
       it 'returns :arbiter' do
@@ -567,7 +411,8 @@ describe Mongo::Server::Description do
     context 'when the server is a ghost' do
 
       let(:description) do
-        described_class.new(address, { 'isreplicaset' => true })
+        described_class.new(address, { 'isreplicaset' => true,
+          'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
       end
 
       it 'returns :ghost' do
@@ -578,7 +423,8 @@ describe Mongo::Server::Description do
     context 'when the server is a mongos' do
 
       let(:config) do
-        { 'msg' => 'isdbgrid', 'ismaster' => true }
+        { 'msg' => 'isdbgrid', 'ismaster' => true,
+          'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 }
       end
 
       let(:description) do
@@ -588,6 +434,23 @@ describe Mongo::Server::Description do
       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', 'ismaster' => 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 primary' do
@@ -604,7 +467,9 @@ describe Mongo::Server::Description do
     context 'when the server is a secondary' do
 
       let(:description) do
-        described_class.new(address, { 'secondary' => true, 'setName' => 'test' })
+        described_class.new(address, { 'secondary' => true,
+          'minWireVersion' => 2, 'maxWireVersion' => 3,
+          'setName' => 'test', 'ok' => 1 })
       end
 
       it 'returns :secondary' do
@@ -615,7 +480,8 @@ describe Mongo::Server::Description do
     context 'when the server is standalone' do
 
       let(:description) do
-        described_class.new(address, { 'ismaster' => true, 'ok' => 1 })
+        described_class.new(address, { 'ismaster' => true,
+          'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
       end
 
       it 'returns :standalone' do
@@ -623,54 +489,40 @@ describe Mongo::Server::Description do
       end
     end
 
-    context 'when the description has no configuration' do
-
-      let(:description) do
-        described_class.new(address)
-      end
-
-      it 'returns :unknown' do
-        expect(description.server_type).to eq(:unknown)
-      end
-    end
-  end
-
-  describe '#unknown?' do
-
-    context 'when the description has no configuration' do
+    context 'when the server is hidden' do
 
       let(:description) do
-        described_class.new(address)
+        described_class.new(address, { 'ismaster' => false,
+          'minWireVersion' => 2, 'maxWireVersion' => 3, 'setName' => 'test',
+          'hidden' => true, 'ok' => 1 })
       end
 
-      it 'returns true' do
-        expect(description).to be_unknown
+      it 'returns :other' do
+        expect(description.server_type).to eq(:other)
       end
     end
 
-    context 'when the command was not ok' do
+    context 'when the server is other' do
 
       let(:description) do
-        described_class.new(address, { 'ok' => 0 })
+        described_class.new(address, { 'ismaster' => false,
+          'minWireVersion' => 2, 'maxWireVersion' => 3, 'setName' => 'test',
+          'ok' => 1 })
       end
 
-      it 'returns true' do
-        expect(description).to be_unknown
+      it 'returns :other' do
+        expect(description.server_type).to eq(:other)
       end
     end
 
-    context 'when the description has a configuration' do
-
-      let(:config) do
-        { 'hosts' => [ '127.0.0.1:27019', '127.0.0.1:27020' ], 'ok' => 1 }
-      end
+    context 'when the description has no configuration' do
 
       let(:description) do
-        described_class.new(address, config)
+        described_class.new(address)
       end
 
-      it 'returns false' do
-        expect(description).to_not be_unknown
+      it 'returns :unknown' do
+        expect(description.server_type).to eq(:unknown)
       end
     end
   end
@@ -682,7 +534,8 @@ describe Mongo::Server::Description do
     end
 
     let(:server) do
-      Mongo::Server.new(address, cluster, monitoring, listeners)
+      Mongo::Server.new(address, cluster, monitoring, listeners,
+        monitoring_io: false)
     end
 
     let(:description) do
@@ -703,7 +556,8 @@ describe Mongo::Server::Description do
       end
 
       let(:server) do
-        Mongo::Server.new(other_address, cluster, monitoring, listeners)
+        Mongo::Server.new(other_address, cluster, monitoring, listeners,
+          monitoring_io: false)
       end
 
       it 'returns false' do
@@ -769,7 +623,8 @@ describe Mongo::Server::Description do
     end
 
     let(:server) do
-      Mongo::Server.new(server_address, cluster, monitoring, listeners)
+      Mongo::Server.new(server_address, cluster, monitoring, listeners,
+        monitoring_io: false)
     end
 
     context 'when the server is included in the description hosts list' do
@@ -811,7 +666,8 @@ describe Mongo::Server::Description do
     context 'when the description is from a standalone' do
 
       let(:description) do
-        described_class.new(address, { 'ismaster' => true, 'ok' => 1 })
+        described_class.new(address, { 'ismaster' => true,
+          'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
       end
 
       it 'returns false' do
@@ -831,6 +687,32 @@ describe Mongo::Server::Description do
     end
   end
 
+  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
+
+      it 'returns the logical session timeout value' do
+        expect(description.logical_session_timeout).to eq(7)
+      end
+    end
+
+    context 'when a logical session timeout value is not in the config' do
+
+      let(:description) do
+        described_class.new(address, { 'ismaster' => true,
+          'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
+      end
+
+      it 'returns nil' do
+        expect(description.logical_session_timeout).to be(nil)
+      end
+    end
+  end
+
   describe '#==' do
 
     let(:description) do
@@ -838,7 +720,12 @@ describe Mongo::Server::Description do
     end
 
     let(:other) do
-      described_class.new(address, replica.merge('localTime' => 1))
+      described_class.new(address, replica.merge(
+        'localTime' => 1,
+        'lastWrite' => { 'lastWriteDate' => 1 },
+        'operationTime' => 2,
+        '$clusterTime' => 2
+      ))
     end
 
     it 'excludes certain fields' do
@@ -871,6 +758,23 @@ describe Mongo::Server::Description do
       end
     end
 
+    context 'when the configs match, but have different connectionId values' do
+
+      let(:description) do
+        described_class.new(address, replica)
+      end
+
+      let(:other) do
+        described_class.new(address, replica.merge(
+          'connectionId' => 12
+        ))
+      end
+
+      it 'returns true' do
+        expect(description == other).to be(true)
+      end
+    end
+
     context 'when the configs do not match' do
 
       let(:description) do
@@ -878,12 +782,52 @@ describe Mongo::Server::Description do
       end
 
       let(:other) do
-        described_class.new(address, { 'ismaster' => true, 'ok' => 1 })
+        described_class.new(address, { 'ismaster' => true,
+          'minWireVersion' => 2, 'maxWireVersion' => 3, 'ok' => 1 })
       end
 
       it 'returns false' do
         expect(description == other).to be(false)
       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
+
+      let(:two) do
+        described_class.new(address, { 'ismaster' => 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
+
+  describe '#last_update_time' do
+    context 'stub description' do
+      let(:description) { described_class.new(address) }
+
+      it 'is present' do
+        expect(description.last_update_time).to be_a(Time)
+      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

data/spec/mongo/server/monitor/app_metadata_spec.rb

@@ -0,0 +1,23 @@
+require 'lite_spec_helper'
+require_relative '../app_metadata_shared'
+
+describe Mongo::Server::Monitor::AppMetadata do
+
+  describe '#document' do
+    let(:document) do
+      app_metadata.send(:document)
+    end
+
+    context 'when user is given and auth_mech is not given' do
+      let(:app_metadata) do
+        described_class.new(user: 'foo')
+      end
+
+      it 'does not include saslSupportedMechs' do
+        expect(document).not_to have_key(:saslSupportedMechs)
+      end
+    end
+
+    it_behaves_like 'app metadata document'
+  end
+end