mongo 2.6.4 → 2.7.0.rc0

data/spec/mongo/index/view_spec.rb

@@ -10,16 +10,23 @@ describe Mongo::Index::View do
     {}
   end
 
+  before do
+    begin
+      authorized_collection.delete_many
+    rescue Mongo::Error::OperationFailure
+    end
+    begin
+      authorized_collection.indexes.drop_all
+    rescue Mongo::Error::OperationFailure
+    end
+  end
+
   describe '#drop_one' do
 
     let(:spec) do
       { another: -1 }
     end
 
-    after do
-      begin; view.drop_one('another_-1'); rescue; end
-    end
-
     before do
       view.create_one(spec, unique: true)
     end
@@ -180,10 +187,6 @@ describe Mongo::Index::View do
           view_with_write_concern.drop_all
         end
 
-        after do
-          view.drop_all
-        end
-
         context 'when the server accepts writeConcern for the dropIndexes operation', if: collation_enabled? do
 
           it 'applies the write concern' do
@@ -218,11 +221,6 @@ describe Mongo::Index::View do
             )
           end
 
-          after do
-            view.drop_one('random_1')
-            view.drop_one('testing_-1')
-          end
-
           it 'returns ok' do
             expect(result).to be_successful
           end
@@ -265,10 +263,6 @@ describe Mongo::Index::View do
             )
           end
 
-          after do
-            begin; view.drop_one('random_1'); rescue; end
-          end
-
           let(:index_info) do
             view.get('random_1')
           end
@@ -316,11 +310,6 @@ describe Mongo::Index::View do
 
         context 'when the collection has a write concern' do
 
-          after do
-            begin; view.drop_one('random_1'); rescue; end
-            begin; view.drop_one('testing_-1'); rescue; end
-          end
-
           let(:collection) do
             authorized_collection.with(write: INVALID_WRITE_CONCERN)
           end
@@ -365,11 +354,6 @@ describe Mongo::Index::View do
                              ])
           end
 
-          after do
-            view.drop_one('random_1')
-            view.drop_one('testing_-1')
-          end
-
           it 'returns ok' do
             expect(result).to be_successful
           end
@@ -414,10 +398,6 @@ describe Mongo::Index::View do
             view.get('random_1')
           end
 
-          after do
-            begin; view.drop_one('random_1'); rescue; end
-          end
-
           context 'when the server supports collations', if: collation_enabled? do
 
             it 'returns ok' do
@@ -461,11 +441,6 @@ describe Mongo::Index::View do
 
         context 'when the collection has a write concern' do
 
-          after do
-            begin; view.drop_one('random_1'); rescue; end
-            begin; view.drop_one('testing_-1'); rescue; end
-          end
-
           let(:collection) do
             authorized_collection.with(write: INVALID_WRITE_CONCERN)
           end
@@ -509,10 +484,6 @@ describe Mongo::Index::View do
           view.create_one(spec, unique: true)
         end
 
-        after do
-          view.drop_one('name_1')
-        end
-
         it 'raises an exception' do
           expect {
             view.create_many([{ key: { name: 1 }, unique: false }])
@@ -562,10 +533,6 @@ describe Mongo::Index::View do
 
       context 'when the collection has a write concern' do
 
-        after do
-          begin; view.drop_one('random_1'); rescue; end
-        end
-
         let(:collection) do
           authorized_collection.with(write: INVALID_WRITE_CONCERN)
         end
@@ -597,10 +564,6 @@ describe Mongo::Index::View do
 
       context 'when the index is created on an subdocument field' do
 
-        after do
-          begin; view.drop_one('random_1'); rescue; end
-        end
-
         let(:spec) do
           { 'sub_document.random' => 1 }
         end
@@ -609,10 +572,6 @@ describe Mongo::Index::View do
           view.create_one(spec, unique: true)
         end
 
-        after do
-          begin; view.drop_one('sub_document.random_1'); rescue; end
-        end
-
         it 'returns ok' do
           expect(result).to be_successful
         end
@@ -629,10 +588,6 @@ describe Mongo::Index::View do
         view.create_one(spec, unique: true)
       end
 
-      after do
-        view.drop_one('name_1')
-      end
-
       it 'raises an exception' do
         expect {
           view.create_one(spec, unique: false)
@@ -650,10 +605,6 @@ describe Mongo::Index::View do
         view.create_one(spec, unique: true, name: 'random_name')
       end
 
-      after do
-        view.drop_one('random_name')
-      end
-
       it 'returns ok' do
         expect(result).to be_successful
       end
@@ -688,10 +639,6 @@ describe Mongo::Index::View do
         authorized_collection.indexes.get('x_1')
       end
 
-      after do
-        view.drop_one('x_1')
-      end
-
       it 'returns ok' do
         expect(result).to be_successful
       end
@@ -716,10 +663,6 @@ describe Mongo::Index::View do
       view.create_one(spec, unique: true, name: 'random_name')
     end
 
-    after do
-      begin; view.drop_one('random_name'); rescue; end
-    end
-
     context 'when providing a name' do
 
       let(:index) do
@@ -779,10 +722,6 @@ describe Mongo::Index::View do
         view.create_one(spec, unique: true)
       end
 
-      after do
-        view.drop_one('name_1')
-      end
-
       let(:indexes) do
         view.each
       end

data/spec/mongo/lint_spec.rb

@@ -2,6 +2,12 @@ require 'lite_spec_helper'
 
 describe Mongo::Lint do
 
+  before(:all) do
+    # Since we are installing an expectation on ENV, close any open clients
+    # which may have background threads reading ENV
+    ClientRegistry.instance.close_all_clients
+  end
+
   before do
     expect(ENV).to receive(:[]).with('MONGO_RUBY_DRIVER_LINT').at_least(:once).and_return('1')
   end
@@ -149,4 +155,74 @@ describe Mongo::Lint do
       end
     end
   end
+
+  describe '.validate_read_concern_option' do
+    it 'accepts nil' do
+      expect do
+        described_class.validate_read_concern_option(nil)
+      end.to_not raise_error
+    end
+
+    it 'accepts empty hash' do
+      expect do
+        described_class.validate_read_concern_option({})
+      end.to_not raise_error
+    end
+
+    it "rejects an object which is not a hash" do
+      expect do
+        described_class.validate_read_concern_option(:local)
+      end.to raise_error(Mongo::Error::LintError)
+    end
+
+    [:local, :majority, :snapshot].each do |level|
+      it "accepts :#{level}" do
+        expect do
+          described_class.validate_read_concern_option({level: level})
+        end.to_not raise_error
+      end
+
+      it "rejects #{level} as string" do
+        expect do
+          described_class.validate_read_concern_option({level: level.to_s})
+        end.to raise_error(Mongo::Error::LintError)
+      end
+    end
+
+    it "rejects a bogus level" do
+      expect do
+        described_class.validate_read_concern_option({level: :bogus})
+      end.to raise_error(Mongo::Error::LintError)
+    end
+
+    it "rejects level given as a string key" do
+      expect do
+        described_class.validate_read_concern_option({'level' => :snapshot})
+      end.to raise_error(Mongo::Error::LintError)
+    end
+
+    it "rejects a bogus key as symbol" do
+      expect do
+        described_class.validate_read_concern_option({foo: 'bar'})
+      end.to raise_error(Mongo::Error::LintError)
+    end
+
+    it "rejects a bogus key as string" do
+      expect do
+        described_class.validate_read_concern_option({'foo' => 'bar'})
+      end.to raise_error(Mongo::Error::LintError)
+    end
+
+    %w(afterClusterTime after_cluster_time).each do |key|
+      [:to_s, :to_sym].each do |conv|
+        key = key.send(conv)
+
+        it "rejects #{key.inspect}" do
+          expect do
+            described_class.validate_read_concern_option({key => 123})
+          end.to raise_error(Mongo::Error::LintError)
+        end
+      end
+    end
+  end
 end

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

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'lite_spec_helper'
 
 describe Mongo::Monitoring::Event::CommandFailed do
 

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

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'lite_spec_helper'
 
 describe Mongo::Monitoring::Event::CommandStarted do
 

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

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'lite_spec_helper'
 
 describe Mongo::Monitoring::Event::CommandSucceeded do
 

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

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'lite_spec_helper'
 
 describe Mongo::Monitoring::Event::Secure do
 

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

@@ -0,0 +1,35 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::ServerClosed 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('#<ServerClosed address=127.0.0.1:27017 topology=Unknown[127.0.0.1:27017]>')
+    end
+  end
+end

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

@@ -0,0 +1,38 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::ServerDescriptionChanged 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(:previous_desc) { Mongo::Server::Description.new(address) }
+  let(:updated_desc) { Mongo::Server::Description.new(address) }
+
+  let(:event) do
+    described_class.new(address, topology, previous_desc, updated_desc)
+  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("#<ServerDescriptionChanged address=127.0.0.1:27017 topology=Unknown[127.0.0.1:27017] prev=#{previous_desc.inspect} new=#{updated_desc.inspect}>")
+    end
+  end
+end

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

@@ -0,0 +1,31 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::ServerHeartbeatFailed 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, Mongo::Error::SocketError.new('foo'))
+  end
+
+  describe '#summary' do
+    it 'renders correctly' do
+      expect(event.summary).to eq('#<ServerHeartbeatFailed address=127.0.0.1:27017 error=#<Mongo::Error::SocketError: foo>>')
+    end
+  end
+end

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

@@ -0,0 +1,31 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::ServerHeartbeatStarted 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)
+  end
+
+  describe '#summary' do
+    it 'renders correctly' do
+      expect(event.summary).to eq('#<ServerHeartbeatStarted address=127.0.0.1:27017>')
+    end
+  end
+end