mongo 2.0.5 → 2.0.6

data/spec/mongo/operation/write/bulk/bulk_delete_spec.rb

@@ -226,6 +226,7 @@ describe Mongo::Operation::Write::BulkDelete do
       
           it 'does not abort after first error' do
             failing_delete.execute(authorized_primary.context)
+            sleep(1)
             expect(authorized_collection.find.count).to eq(1)
           end
         end

data/spec/mongo/server/connection_pool_spec.rb

@@ -9,10 +9,10 @@ describe Mongo::Server::ConnectionPool do
   describe '#checkin' do
 
     let(:server) do
-      Mongo::Server.new(address, Mongo::Event::Listeners.new, ssl: SSL)
+      Mongo::Server.new(address, double('cluster'), Mongo::Event::Listeners.new, ssl: SSL)
     end
 
-    let(:pool) do
+    let!(:pool) do
       described_class.get(server)
     end
 
@@ -39,10 +39,10 @@ describe Mongo::Server::ConnectionPool do
   describe '#checkout' do
 
     let(:server) do
-      Mongo::Server.new(address, Mongo::Event::Listeners.new, ssl: SSL)
+      Mongo::Server.new(address, double('cluster'), Mongo::Event::Listeners.new, ssl: SSL)
     end
 
-    let(:pool) do
+    let!(:pool) do
       described_class.get(server)
     end
 
@@ -87,10 +87,10 @@ describe Mongo::Server::ConnectionPool do
   describe '.get' do
 
     let(:server) do
-      Mongo::Server.new(address, Mongo::Event::Listeners.new, ssl: SSL)
+      Mongo::Server.new(address, double('cluster'), Mongo::Event::Listeners.new, ssl: SSL)
     end
 
-    let(:pool) do
+    let!(:pool) do
       described_class.get(server)
     end
 
@@ -102,10 +102,10 @@ describe Mongo::Server::ConnectionPool do
   describe '#inspect' do
 
     let(:server) do
-      Mongo::Server.new(address, Mongo::Event::Listeners.new, ssl: SSL)
+      Mongo::Server.new(address, double('cluster'), Mongo::Event::Listeners.new, ssl: SSL)
     end
 
-    let(:pool) do
+    let!(:pool) do
       described_class.get(server)
     end
 
@@ -121,10 +121,10 @@ describe Mongo::Server::ConnectionPool do
   describe '#with_connection' do
 
     let(:server) do
-      Mongo::Server.new(address, Mongo::Event::Listeners.new, ssl: SSL)
+      Mongo::Server.new(address, double('cluster'), Mongo::Event::Listeners.new, ssl: SSL)
     end
 
-    let(:pool) do
+    let!(:pool) do
       described_class.get(server)
     end
 

data/spec/mongo/server/connection_spec.rb

@@ -7,7 +7,7 @@ describe Mongo::Server::Connection do
   end
 
   let(:server) do
-    Mongo::Server.new(address, Mongo::Event::Listeners.new, ssl: SSL)
+    Mongo::Server.new(address, double('cluster'), Mongo::Event::Listeners.new, ssl: SSL)
   end
 
   describe '#connect!' do

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

@@ -15,16 +15,36 @@ describe Mongo::Server::Description::Features do
       end
     end
 
-    context 'when the server wire version range is higher' do
+    context 'when the server wire version range min is higher' do
 
       it 'raises an exception' do
         expect {
-          described_class.new(0..4)
+          described_class.new(described_class::DRIVER_WIRE_VERSIONS.max+1..described_class::DRIVER_WIRE_VERSIONS.max+2)
         }.to raise_error(Mongo::Error::UnsupportedFeatures)
       end
     end
 
-    context 'when the server wire version range is lower' do
+    context 'when the server wire version range max is higher' do
+
+      let(:features) do
+        described_class.new(0..4)
+      end
+
+      it 'sets the server wire version range' do
+        expect(features.server_wire_versions).to eq(0..4)
+      end
+    end
+
+    context 'when the server wire version range max is lower' do
+
+      it 'raises an exception' do
+        expect {
+          described_class.new(described_class::DRIVER_WIRE_VERSIONS.min-2..described_class::DRIVER_WIRE_VERSIONS.min-1)
+        }.to raise_error(Mongo::Error::UnsupportedFeatures)
+      end
+    end
+
+    context 'when the server wire version range max is lower' do
 
       let(:features) do
         described_class.new(0..2)

data/spec/mongo/server/description/inspector/description_changed_spec.rb

@@ -0,0 +1,78 @@
+require 'spec_helper'
+
+describe Mongo::Server::Description::Inspector::DescriptionChanged do
+
+  let(:listeners) do
+    Mongo::Event::Listeners.new
+  end
+
+  let(:inspection) do
+    described_class.new(listeners)
+  end
+
+  let(:address) do
+    Mongo::Address.new('127.0.0.1:27017')
+  end
+
+  describe '.run' do
+
+    let(:config) do
+      {
+          'ismaster' => true,
+          'secondary' => false,
+          'hosts' => [ '127.0.0.1:27018', '127.0.0.1:27019' ],
+          'setName' => 'test'
+      }
+    end
+
+    let(:description) do
+      Mongo::Server::Description.new(address, config, listeners)
+    end
+
+    let(:updated) do
+      Mongo::Server::Description.new(address, new_config, listeners)
+    end
+
+    let(:listener) do
+      double('listener')
+    end
+
+    before do
+      listeners.add_listener(Mongo::Event::DESCRIPTION_CHANGED, listener)
+    end
+
+    context 'when there is no change' do
+
+      let(:new_config) do
+        {
+            'ismaster' => true,
+            'secondary' => false,
+            'hosts' => [ '127.0.0.1:27018', '127.0.0.1:27019' ],
+            'setName' => 'test'
+        }
+      end
+
+      it 'does not fire a description changed event' do
+        expect(listener).to_not receive(:handle)
+        inspection.run(description, updated)
+      end
+    end
+
+    context 'when there is a change' do
+
+      let(:new_config) do
+        {
+            'ismaster' => true,
+            'secondary' => false,
+            'hosts' => [ '127.0.0.1:27018', '127.0.0.1:27020' ],
+            'setName' => 'test'
+        }
+      end
+
+      it 'fires a description changed event' do
+        expect(listener).to receive(:handle)
+        inspection.run(description, updated)
+      end
+    end
+  end
+end

data/spec/mongo/server/description_spec.rb

@@ -22,6 +22,7 @@ describe Mongo::Server::Description do
       'maxWriteBatchSize' => 1000,
       'maxWireVersion' => 2,
       'minWireVersion' => 0,
+      'localTime' => Time.now,
       'ok' => 1
     }
   end
@@ -78,6 +79,27 @@ describe Mongo::Server::Description do
         expect(description.arbiters).to be_empty
       end
     end
+
+    context 'when the addresses are not lowercase' do
+
+      let(:config) do
+        replica.merge(
+                   {
+                       'arbiters' => [
+                           'SERVER:27017'
+                       ],
+                   }
+        )
+      end
+
+      let(:description) do
+        described_class.new(address, config)
+      end
+
+      it 'normalizes the addresses to lowercase' do
+        expect(description.arbiters).to eq(['server:27017'])
+      end
+    end
   end
 
   describe '#ghost?' do
@@ -143,6 +165,27 @@ describe Mongo::Server::Description do
     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
+
+      let(:config) do
+        replica.merge(
+            {
+                'hosts' => [
+                    'SERVER:27017'
+                ],
+            }
+        )
+      end
+
+      let(:description) do
+        described_class.new(address, config)
+      end
+
+      it 'normalizes the addresses to lowercase' do
+        expect(description.hosts).to eq(['server:27017'])
+      end
+    end
   end
 
   describe '#max_bson_object_size' do
@@ -334,6 +377,31 @@ describe Mongo::Server::Description do
         expect(description.passives).to be_empty
       end
     end
+
+    context 'when the addresses are not lowercase' do
+
+      let(:config) do
+        replica.merge(
+            {
+                'passives' => [
+                    'SERVER:27017'
+                ],
+            }
+        )
+      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
+
+      end
+    end
   end
 
   describe '#primary?' do
@@ -507,4 +575,174 @@ describe Mongo::Server::Description do
       end
     end
   end
+
+  describe '#is_server?' do
+
+    let(:listeners) do
+      Mongo::Event::Listeners.new
+    end
+
+    let(:server) do
+      Mongo::Server.new(address, double('cluster'), listeners)
+    end
+
+    let(:description) do
+      described_class.new(address, {})
+    end
+
+    context 'when the server address matches the description address' do
+
+      it 'returns true' do
+        expect(description.is_server?(server)).to be(true)
+      end
+    end
+
+    context 'when the server address does not match the description address' do
+
+      let(:other_address) do
+        Mongo::Address.new('127.0.0.1:27020')
+      end
+
+      let(:server) do
+        Mongo::Server.new(other_address, double('cluster'), listeners)
+      end
+
+      it 'returns false' do
+        expect(description.is_server?(server)).to be(false)
+      end
+    end
+  end
+
+  describe '#lists_server?' do
+
+    let(:description) do
+      described_class.new(address, replica)
+    end
+
+    let(:server_address) do
+      Mongo::Address.new('127.0.0.1:27018')
+    end
+
+    let(:listeners) do
+      Mongo::Event::Listeners.new
+    end
+
+    let(:server) do
+      Mongo::Server.new(server_address, double('cluster'), listeners)
+    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
+
+    context 'when the server is not included in the description hosts list' do
+
+      let(:server_address) do
+        Mongo::Address.new('127.0.0.1:27017')
+      end
+
+      it 'returns false' do
+        expect(description.lists_server?(server)).to be(false)
+      end
+    end
+  end
+
+  describe '#replica_set_member?' do
+
+    context 'when the description is from a mongos' do
+
+      let(:config) do
+        { 'msg' => 'isdbgrid', 'ismaster' => true }
+      end
+
+      let(:description) do
+        described_class.new(address, config)
+      end
+
+      it 'returns false' do
+        expect(description.replica_set_member?).to be(false)
+      end
+    end
+
+    context 'when the description is from a standalone' do
+
+      let(:description) do
+        described_class.new(address, { 'ismaster' => true, 'ok' => 1 })
+      end
+
+      it 'returns false' do
+        expect(description.replica_set_member?).to be(false)
+      end
+    end
+
+    context 'when the description is from a replica set member' do
+
+      let(:description) do
+        described_class.new(address, replica)
+      end
+
+      it 'returns true' do
+        expect(description.replica_set_member?).to be(true)
+      end
+    end
+  end
+
+  describe '#==' do
+
+    let(:description) do
+      described_class.new(address, replica)
+    end
+
+    let(:other) do
+      described_class.new(address, replica.merge('localTime' => 1))
+    end
+
+    it 'excludes certain fields' do
+      expect(description == other).to be(true)
+    end
+
+    context 'when the classes do not match' do
+
+      let(:description) do
+        described_class.new(address, replica)
+      end
+
+      it 'returns false' do
+        expect(description == Array.new).to be(false)
+      end
+    end
+
+    context 'when the configs match' do
+
+      let(:description) do
+        described_class.new(address, replica)
+      end
+
+      let(:other) do
+        described_class.new(address, replica)
+      end
+
+      it 'returns true' do
+        expect(description == other).to be(true)
+      end
+    end
+
+    context 'when the configs do not match' do
+
+      let(:description) do
+        described_class.new(address, replica)
+      end
+
+      let(:other) do
+        described_class.new(address, { 'ismaster' => true, 'ok' => 1 })
+      end
+
+      it 'returns false' do
+        expect(description == other).to be(false)
+      end
+    end
+  end
 end