mongo 2.8.0 → 2.9.0.rc0

data/spec/mongo/error/parser_spec.rb

@@ -22,7 +22,7 @@ describe Mongo::Error::Parser do
     context 'when the document contains an errmsg' do
 
       let(:document) do
-        { 'errmsg' => 'no such command: notacommand', 'code'=>59 }
+        { 'errmsg' => 'no such command: notacommand', 'code' => 59 }
       end
 
       it 'returns the message' do
@@ -101,6 +101,26 @@ describe Mongo::Error::Parser do
       described_class.new(document)
     end
 
+    context 'when document contains code and ok: 1' do
+      let(:document) do
+        { 'ok' => 1, 'errmsg' => 'not master', 'code' => 10107, 'codeName' => 'NotMaster' }
+      end
+
+      it 'returns nil' do
+        expect(parser.code).to be nil
+      end
+    end
+
+    context 'when document contains code and ok: 1.0' do
+      let(:document) do
+        { 'ok' => 1.0, 'errmsg' => 'not master', 'code' => 10107, 'codeName' => 'NotMaster' }
+      end
+
+      it 'returns nil' do
+        expect(parser.code).to be nil
+      end
+    end
+
     context 'when document contains code' do
       let(:document) do
         { 'ok' => 0, 'errmsg' => 'not master', 'code' => 10107, 'codeName' => 'NotMaster' }
@@ -109,6 +129,16 @@ describe Mongo::Error::Parser do
       it 'returns the code' do
         expect(parser.code).to eq(10107)
       end
+
+      context 'with legacy option' do
+        let(:parser) do
+          described_class.new(document, nil, legacy: true)
+        end
+
+        it 'returns nil' do
+          expect(parser.code).to be nil
+        end
+      end
     end
 
     context 'when document does not contain code' do
@@ -149,6 +179,26 @@ describe Mongo::Error::Parser do
       described_class.new(document)
     end
 
+    context 'when document contains code name and ok: 1' do
+      let(:document) do
+        { 'ok' => 1, 'errmsg' => 'not master', 'code' => 10107, 'codeName' => 'NotMaster' }
+      end
+
+      it 'returns nil' do
+        expect(parser.code_name).to be nil
+      end
+    end
+
+    context 'when document contains code name and ok: 1.0' do
+      let(:document) do
+        { 'ok' => 1.0, 'errmsg' => 'not master', 'code' => 10107, 'codeName' => 'NotMaster' }
+      end
+
+      it 'returns nil' do
+        expect(parser.code_name).to be nil
+      end
+    end
+
     context 'when document contains code name' do
       let(:document) do
         { 'ok' => 0, 'errmsg' => 'not master', 'code' => 10107, 'codeName' => 'NotMaster' }
@@ -157,6 +207,16 @@ describe Mongo::Error::Parser do
       it 'returns the code name' do
         expect(parser.code_name).to eq('NotMaster')
       end
+
+      context 'with legacy option' do
+        let(:parser) do
+          described_class.new(document, nil, legacy: true)
+        end
+
+        it 'returns nil' do
+          expect(parser.code_name).to be nil
+        end
+      end
     end
 
     context 'when document does not contain code name' do

data/spec/mongo/grid/stream/read_spec.rb

@@ -238,10 +238,10 @@ describe Mongo::Grid::FSBucket::Stream::Read do
         stream.close
       end
 
-      it 'raises an exception' do
+      it 'does not raise an exception' do
         expect {
           stream.close
-        }.to raise_error(Mongo::Error::ClosedStream)
+        }.not_to raise_error
       end
     end
   end

data/spec/mongo/monitoring/event/cmap/connection_check_out_failed_spec.rb

@@ -0,0 +1,23 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::Cmap::ConnectionCheckOutFailed do
+
+  describe '#summary' do
+
+    let(:address) do
+      Mongo::Address.new('127.0.0.1:27017')
+    end
+
+    let(:reason) do
+      described_class::TIMEOUT
+    end
+
+    let(:event) do
+      described_class.new(address, reason)
+    end
+
+    it 'renders correctly' do
+      expect(event.summary).to eq('#<ConnectionCheckOutFailed address=127.0.0.1:27017 reason=timeout>')
+    end
+  end
+end

data/spec/mongo/monitoring/event/cmap/connection_check_out_started_spec.rb

@@ -0,0 +1,19 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::Cmap::ConnectionCheckOutStarted do
+
+  describe '#summary' do
+
+    let(:address) do
+      Mongo::Address.new('127.0.0.1:27017')
+    end
+
+    let(:event) do
+      described_class.new(address)
+    end
+
+    it 'renders correctly' do
+      expect(event.summary).to eq('#<ConnectionCheckOutStarted address=127.0.0.1:27017>')
+    end
+  end
+end

data/spec/mongo/monitoring/event/cmap/connection_checked_in_spec.rb

@@ -0,0 +1,23 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::Cmap::ConnectionCheckedIn do
+
+  describe '#summary' do
+
+    let(:address) do
+      Mongo::Address.new('127.0.0.1:27017')
+    end
+
+    let(:id) do
+      1
+    end
+
+    let(:event) do
+      described_class.new(address, id)
+    end
+
+    it 'renders correctly' do
+      expect(event.summary).to eq('#<ConnectionCheckedIn address=127.0.0.1:27017 connection_id=1>')
+    end
+  end
+end

data/spec/mongo/monitoring/event/cmap/connection_checked_out_spec.rb

@@ -0,0 +1,23 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::Cmap::ConnectionCheckedOut do
+
+  describe '#summary' do
+
+    let(:address) do
+      Mongo::Address.new('127.0.0.1:27017')
+    end
+
+    let(:id) do
+      1
+    end
+
+    let(:event) do
+      described_class.new(address, id)
+    end
+
+    it 'renders correctly' do
+      expect(event.summary).to eq('#<ConnectionCheckedOut address=127.0.0.1:27017 connection_id=1>')
+    end
+  end
+end

data/spec/mongo/monitoring/event/cmap/connection_closed_spec.rb

@@ -0,0 +1,27 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::Cmap::ConnectionClosed do
+
+  describe '#summary' do
+
+    let(:address) do
+      Mongo::Address.new('127.0.0.1:27017')
+    end
+
+    let(:reason) do
+      described_class::STALE
+    end
+
+    let(:id) do
+      1
+    end
+
+    let(:event) do
+      described_class.new(address, id, reason)
+    end
+
+    it 'renders correctly' do
+      expect(event.summary).to eq('#<ConnectionClosed address=127.0.0.1:27017 connection_id=1 reason=stale>')
+    end
+  end
+end

data/spec/mongo/monitoring/event/cmap/connection_created_spec.rb

@@ -0,0 +1,24 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::Cmap::ConnectionCreated do
+
+  describe '#summary' do
+
+    let(:address) do
+      Mongo::Address.new('127.0.0.1:27017')
+    end
+
+
+    let(:id) do
+      1
+    end
+
+    let(:event) do
+      described_class.new(address, id)
+    end
+
+    it 'renders correctly' do
+      expect(event.summary).to eq('#<ConnectionCreated address=127.0.0.1:27017 connection_id=1>')
+    end
+  end
+end

data/spec/mongo/monitoring/event/cmap/connection_ready_spec.rb

@@ -0,0 +1,24 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::Cmap::ConnectionReady do
+
+  describe '#summary' do
+
+    let(:address) do
+      Mongo::Address.new('127.0.0.1:27017')
+    end
+
+
+    let(:id) do
+      1
+    end
+
+    let(:event) do
+      described_class.new(address, id)
+    end
+
+    it 'renders correctly' do
+      expect(event.summary).to eq('#<ConnectionReady address=127.0.0.1:27017 connection_id=1>')
+    end
+  end
+end

data/spec/mongo/monitoring/event/cmap/pool_cleared_spec.rb

@@ -0,0 +1,19 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::Cmap::PoolCleared do
+
+  describe '#summary' do
+
+    let(:address) do
+      Mongo::Address.new('127.0.0.1:27017')
+    end
+
+    let(:event) do
+      described_class.new(address)
+    end
+
+    it 'renders correctly' do
+      expect(event.summary).to eq('#<PoolCleared address=127.0.0.1:27017>')
+    end
+  end
+end

data/spec/mongo/monitoring/event/cmap/pool_closed_spec.rb

@@ -0,0 +1,19 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::Cmap::PoolClosed do
+
+  describe '#summary' do
+
+    let(:address) do
+      Mongo::Address.new('127.0.0.1:27017')
+    end
+
+    let(:event) do
+      described_class.new(address)
+    end
+
+    it 'renders correctly' do
+      expect(event.summary).to eq('#<PoolClosed address=127.0.0.1:27017>')
+    end
+  end
+end

data/spec/mongo/monitoring/event/cmap/pool_created_spec.rb

@@ -0,0 +1,26 @@
+require 'lite_spec_helper'
+
+describe Mongo::Monitoring::Event::Cmap::PoolCreated do
+
+  describe '#summary' do
+
+    let(:address) do
+      Mongo::Address.new('127.0.0.1:27017')
+    end
+
+    let(:options) do
+      {
+         wait_queue_timeout: 3,
+         min_pool_size: 5,
+      }
+    end
+
+    let(:event) do
+      described_class.new(address, options)
+    end
+
+    it 'renders correctly' do
+      expect(event.summary).to eq('#<PoolCreated address=127.0.0.1:27017 options={:wait_queue_timeout=>3, :min_pool_size=>5}>')
+    end
+  end
+end

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

@@ -10,7 +10,6 @@ describe Mongo::Operation::Delete do
     { :deletes       => documents,
       :db_name       => SpecConfig.instance.test_db,
       :coll_name     => TEST_COLL,
-      :write_concern => Mongo::WriteConcern.get(SpecConfig.instance.write_concern),
       :ordered       => true
     }
   end
@@ -48,7 +47,6 @@ describe Mongo::Operation::Delete do
           { :deletes       => other_docs,
             :db_name       => SpecConfig.instance.test_db,
             :coll_name     => TEST_COLL,
-            :write_concern => Mongo::WriteConcern.get(SpecConfig.instance.write_concern),
             :ordered       => true
           }
         end
@@ -114,7 +112,6 @@ describe Mongo::Operation::Delete do
           deletes: documents,
           db_name: SpecConfig.instance.test_db,
           coll_name: TEST_COLL,
-          write_concern: Mongo::WriteConcern.get(SpecConfig.instance.write_concern)
         })
       end
 
@@ -143,7 +140,6 @@ describe Mongo::Operation::Delete do
         { :deletes       => documents,
           :db_name       => SpecConfig.instance.test_db,
           :coll_name     => TEST_COLL,
-          :write_concern => Mongo::WriteConcern.get(SpecConfig.instance.write_concern),
           :ordered       => true
         }
       end
@@ -157,7 +153,7 @@ describe Mongo::Operation::Delete do
         context 'when write concern is acknowledged' do
 
           let(:write_concern) do
-            Mongo::WriteConcern.get(SpecConfig.instance.write_concern)
+            Mongo::WriteConcern.get(w: :majority)
           end
 
           it 'aborts after first error' do
@@ -192,7 +188,6 @@ describe Mongo::Operation::Delete do
         { :deletes       => documents,
           :db_name       => SpecConfig.instance.test_db,
           :coll_name     => TEST_COLL,
-          :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(SpecConfig.instance.write_concern)
+    Mongo::WriteConcern.get(w: :majority)
   end
 
   let(:session) { nil }

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(SpecConfig.instance.write_concern)
+    Mongo::WriteConcern.get(w: :majority)
   end
 
   let(:session) { nil }

data/spec/mongo/operation/delete_spec.rb

@@ -23,7 +23,7 @@ describe Mongo::Operation::Delete do
     { :deletes        => [ document ],
       :db_name       => SpecConfig.instance.test_db,
       :coll_name     => TEST_COLL,
-      :write_concern => Mongo::WriteConcern.get(SpecConfig.instance.write_concern),
+      :write_concern => Mongo::WriteConcern.get(w: :majority),
       :ordered       => true
     }
   end
@@ -59,7 +59,7 @@ describe Mongo::Operation::Delete do
           { :deletes        => [ other_doc ],
             :db_name       => SpecConfig.instance.test_db,
             :coll_name     => TEST_COLL,
-            :write_concern => Mongo::WriteConcern.get(SpecConfig.instance.write_concern),
+            :write_concern => Mongo::WriteConcern.get(w: :majority),
             :ordered       => true
           }
         end
@@ -92,7 +92,7 @@ describe Mongo::Operation::Delete do
           deletes: [ document ],
           db_name: SpecConfig.instance.test_db,
           coll_name: TEST_COLL,
-          write_concern: Mongo::WriteConcern.get(SpecConfig.instance.write_concern)
+          write_concern: Mongo::WriteConcern.get(w: :majority)
         })
       end
 
@@ -136,7 +136,7 @@ describe Mongo::Operation::Delete do
           deletes: [ document ],
           db_name: SpecConfig.instance.test_db,
           coll_name: TEST_COLL,
-          write_concern: Mongo::WriteConcern.get(SpecConfig.instance.write_concern)
+          write_concern: Mongo::WriteConcern.get(w: :majority)
         })
       end