mongo 2.2.7 → 2.3.0

data/spec/mongo/operation/write/create_index_spec.rb

@@ -15,7 +15,7 @@ describe Mongo::Operation::Write::CreateIndex do
       end
 
       let(:response) do
-        operation.execute(authorized_primary.context)
+        operation.execute(authorized_primary)
       end
 
       after do
@@ -42,7 +42,7 @@ describe Mongo::Operation::Write::CreateIndex do
       end
 
       before do
-        operation.execute(authorized_primary.context)
+        operation.execute(authorized_primary)
       end
 
       after do
@@ -51,12 +51,12 @@ describe Mongo::Operation::Write::CreateIndex do
 
       it 'raises an exception', if: write_command_enabled? do
         expect {
-          second_operation.execute(authorized_primary.context)
+          second_operation.execute(authorized_primary)
         }.to raise_error(Mongo::Error::OperationFailure)
       end
 
       it 'does not raise an exception', unless: write_command_enabled? do
-        expect(second_operation.execute(authorized_primary.context)).to be_successful
+        expect(second_operation.execute(authorized_primary)).to be_successful
       end
     end
   end

data/spec/mongo/operation/write/create_user_spec.rb

@@ -23,7 +23,7 @@ describe Mongo::Operation::Write::CreateUser do
     context 'when user creation was successful' do
 
       let!(:response) do
-        operation.execute(root_authorized_primary.context)
+        operation.execute(root_authorized_primary)
       end
 
       it 'saves the user in the database' do
@@ -35,8 +35,8 @@ describe Mongo::Operation::Write::CreateUser do
 
       it 'raises an exception' do
         expect {
-          operation.execute(root_authorized_primary.context)
-          operation.execute(root_authorized_primary.context)
+          operation.execute(root_authorized_primary)
+          operation.execute(root_authorized_primary)
         }.to raise_error(Mongo::Error::OperationFailure)
       end
     end

data/spec/mongo/operation/write/delete_spec.rb

@@ -1,24 +1,23 @@
 require 'spec_helper'
 
 describe Mongo::Operation::Write::Delete do
-  include_context 'operation'
 
-  let(:document) { { :q => { :foo => 1 }, :limit => 1 } }
+  let(:document) do
+    {
+      :q => { :foo => 1 },
+      :limit => 1
+    }
+  end
+
   let(:spec) do
     { :delete        => document,
-      :db_name       => db_name,
-      :coll_name     => coll_name,
-      :write_concern => write_concern,
+      :db_name       => TEST_DB,
+      :coll_name     => TEST_COLL,
+      :write_concern => Mongo::WriteConcern.get(WRITE_CONCERN),
       :ordered       => true
     }
   end
 
-  let(:delete_write_cmd) do
-    double('delete_write_cmd').tap do |d|
-      allow(d).to receive(:execute) { [] }
-    end
-  end
-
   let(:op) { described_class.new(spec) }
 
   describe '#initialize' do
@@ -45,11 +44,12 @@ describe Mongo::Operation::Write::Delete do
 
       context 'when two ops have different specs' do
         let(:other_doc) { { :q => { :bar => 1 }, :limit => 1 } }
+
         let(:other_spec) do
           { :delete        => other_doc,
-            :db_name       => db_name,
-            :coll_name     => coll_name,
-            :write_concern => write_concern,
+            :db_name       => TEST_DB,
+            :coll_name     => TEST_COLL,
+            :write_concern => Mongo::WriteConcern.get(WRITE_CONCERN),
             :ordered       => true
           }
         end
@@ -82,7 +82,7 @@ describe Mongo::Operation::Write::Delete do
           delete: document,
           db_name: TEST_DB,
           coll_name: TEST_COLL,
-          write_concern: Mongo::WriteConcern.get(:w => 1)
+          write_concern: Mongo::WriteConcern.get(WRITE_CONCERN)
         })
       end
 
@@ -93,7 +93,7 @@ describe Mongo::Operation::Write::Delete do
         end
 
         let(:result) do
-          delete.execute(authorized_primary.context)
+          delete.execute(authorized_primary)
         end
 
         it 'deletes the documents from the database' do
@@ -113,7 +113,7 @@ describe Mongo::Operation::Write::Delete do
 
         it 'raises an exception' do
           expect {
-            delete.execute(authorized_primary.context)
+            delete.execute(authorized_primary)
           }.to raise_error(Mongo::Error::OperationFailure)
         end
       end
@@ -126,7 +126,7 @@ describe Mongo::Operation::Write::Delete do
           delete: document,
           db_name: TEST_DB,
           coll_name: TEST_COLL,
-          write_concern: Mongo::WriteConcern.get(:w => 1)
+          write_concern: Mongo::WriteConcern.get(WRITE_CONCERN)
         })
       end
 
@@ -137,7 +137,7 @@ describe Mongo::Operation::Write::Delete do
         end
 
         let(:result) do
-          delete.execute(authorized_primary.context)
+          delete.execute(authorized_primary)
         end
 
         it 'deletes the documents from the database' do
@@ -156,13 +156,13 @@ describe Mongo::Operation::Write::Delete do
         end
 
         let(:result) do
-          delete.execute(authorized_primary.context)
+          delete.execute(authorized_primary)
         end
 
         it 'does not delete any documents' do
 
           expect {
-            op.execute(authorized_primary.context)
+            op.execute(authorized_primary)
           }.to raise_error(Mongo::Error::OperationFailure)
 
           expect(authorized_collection.find.count).to eq(2)
@@ -177,10 +177,39 @@ describe Mongo::Operation::Write::Delete do
 
         it 'raises an error' do
           expect {
-            op.execute(authorized_primary.context)
+            op.execute(authorized_primary)
           }.to raise_error(Mongo::Error::MaxBSONSize)
         end
       end
     end
+
+    context 'when write concern { w: 0 } is used' do
+
+      let(:delete) do
+        described_class.new({
+                              delete: document,
+                              db_name: TEST_DB,
+                              coll_name: TEST_COLL,
+                              write_concern: Mongo::WriteConcern.get(:w => 0)
+                            })
+      end
+
+
+      let(:document) do
+        { 'q' => { field: 'test' }, 'limit' => 1 }
+      end
+
+      let(:result) do
+        delete.execute(authorized_primary)
+      end
+
+      before do
+        expect(delete).to receive(:execute_message).and_call_original
+      end
+
+      it 'uses op codes instead of write commands' do
+        expect(result.written_count).to eq(0)
+      end
+    end
   end
 end

data/spec/mongo/operation/write/drop_index_spec.rb

@@ -23,7 +23,7 @@ describe Mongo::Operation::Write::DropIndex do
       end
 
       let(:response) do
-        operation.execute(authorized_primary.context)
+        operation.execute(authorized_primary)
       end
 
       it 'removes the index' do
@@ -43,7 +43,7 @@ describe Mongo::Operation::Write::DropIndex do
 
       it 'raises an exception' do
         expect {
-          operation.execute(authorized_primary.context)
+          operation.execute(authorized_primary)
         }.to raise_error(Mongo::Error::OperationFailure)
       end
     end

data/spec/mongo/operation/write/insert_spec.rb

@@ -83,7 +83,7 @@ describe Mongo::Operation::Write::Insert do
       end
 
       let(:inserted_ids) do
-        insert.execute(authorized_primary.context).inserted_ids
+        insert.execute(authorized_primary).inserted_ids
       end
 
       let(:collection_ids) do
@@ -112,7 +112,7 @@ describe Mongo::Operation::Write::Insert do
       context 'when the insert succeeds' do
 
         let!(:response) do
-          insert.execute(authorized_primary.context)
+          insert.execute(authorized_primary)
         end
 
         it 'reports the correct written count', if: write_command_enabled? do
@@ -148,8 +148,8 @@ describe Mongo::Operation::Write::Insert do
 
         it 'raises an error' do
           expect {
-            failing_insert.execute(authorized_primary.context)
-            failing_insert.execute(authorized_primary.context)
+            failing_insert.execute(authorized_primary)
+            failing_insert.execute(authorized_primary)
           }.to raise_error(Mongo::Error::OperationFailure)
         end
       end
@@ -167,7 +167,7 @@ describe Mongo::Operation::Write::Insert do
         end
 
         let!(:response) do
-          insert.execute(authorized_primary.context)
+          insert.execute(authorized_primary)
         end
 
         it 'reports the correct written count', if: write_command_enabled? do
@@ -203,8 +203,8 @@ describe Mongo::Operation::Write::Insert do
 
         it 'raises an error' do
           expect {
-            failing_insert.execute(authorized_primary.context)
-            failing_insert.execute(authorized_primary.context)
+            failing_insert.execute(authorized_primary)
+            failing_insert.execute(authorized_primary)
           }.to raise_error(Mongo::Error::OperationFailure)
         end
       end
@@ -229,8 +229,8 @@ describe Mongo::Operation::Write::Insert do
 
         it 'raises an error' do
           expect {
-            failing_insert.execute(authorized_primary.context)
-            failing_insert.execute(authorized_primary.context)
+            failing_insert.execute(authorized_primary)
+            failing_insert.execute(authorized_primary)
           }.to raise_error(Mongo::Error::OperationFailure)
         end
 
@@ -244,17 +244,48 @@ describe Mongo::Operation::Write::Insert do
 
         it 'raises an error' do
           expect {
-            insert.execute(authorized_primary.context)
+            insert.execute(authorized_primary)
           }.to raise_error(Mongo::Error::MaxBSONSize)
         end
 
         it 'does not insert the document' do
           expect {
-            insert.execute(authorized_primary.context)
+            insert.execute(authorized_primary)
           }.to raise_error(Mongo::Error::MaxBSONSize)
           expect(authorized_collection.find.count).to eq(0)
         end
       end
     end
+
+    context 'when write concern { w: 0 } is used' do
+
+      let(:spec) do
+        { :documents     => documents,
+          :db_name       => TEST_DB,
+          :coll_name     => TEST_COLL,
+          :write_concern => Mongo::WriteConcern.get(:w => 0)
+        }
+      end
+
+      let(:documents) do
+        [{ '_id' => 1 }]
+      end
+
+      let(:op) do
+        described_class.new(spec)
+      end
+
+      before do
+        expect(op).to receive(:execute_message).and_call_original
+      end
+
+      let(:response) do
+        op.execute(authorized_primary)
+      end
+
+      it 'uses op codes instead of write commands' do
+        expect(response.written_count).to eq(0)
+      end
+    end
   end
 end

data/spec/mongo/operation/write/remove_user_spec.rb

@@ -18,7 +18,7 @@ describe Mongo::Operation::Write::RemoveUser do
     context 'when user removal was successful' do
 
       let!(:response) do
-        operation.execute(root_authorized_primary.context)
+        operation.execute(root_authorized_primary)
       end
 
       it 'removes the user from the database' do
@@ -29,17 +29,17 @@ describe Mongo::Operation::Write::RemoveUser do
     context 'when removal was not successful' do
 
       before do
-        operation.execute(root_authorized_primary.context)
+        operation.execute(root_authorized_primary)
       end
 
       it 'raises an exception', if: write_command_enabled? do
         expect {
-          operation.execute(root_authorized_primary.context)
+          operation.execute(root_authorized_primary)
         }.to raise_error(Mongo::Error::OperationFailure)
       end
 
       it 'does not raise an exception', unless: write_command_enabled? do
-        expect(operation.execute(root_authorized_primary.context).written_count).to eq(0)
+        expect(operation.execute(root_authorized_primary).written_count).to eq(0)
       end
     end
   end

data/spec/mongo/operation/write/update_spec.rb

@@ -99,7 +99,7 @@ describe Mongo::Operation::Write::Update do
         end
 
         let(:result) do
-          update.execute(authorized_primary.context)
+          update.execute(authorized_primary)
         end
 
         it 'updates the document' do
@@ -127,7 +127,7 @@ describe Mongo::Operation::Write::Update do
 
         it 'raises an exception' do
           expect {
-            update.execute(authorized_primary.context)
+            update.execute(authorized_primary)
           }.to raise_error(Mongo::Error::OperationFailure)
         end
       end
@@ -151,7 +151,7 @@ describe Mongo::Operation::Write::Update do
         end
 
         let(:result) do
-          update.execute(authorized_primary.context)
+          update.execute(authorized_primary)
         end
 
         it 'updates the documents' do
@@ -179,7 +179,7 @@ describe Mongo::Operation::Write::Update do
 
         it 'raises an exception' do
           expect {
-            update.execute(authorized_primary.context)
+            update.execute(authorized_primary)
           }.to raise_error(Mongo::Error::OperationFailure)
         end
       end
@@ -192,7 +192,7 @@ describe Mongo::Operation::Write::Update do
 
         it 'raises an error' do
           expect {
-            update.execute(authorized_primary.context)
+            update.execute(authorized_primary)
           }.to raise_error(Mongo::Error::MaxBSONSize)
         end
       end
@@ -204,7 +204,7 @@ describe Mongo::Operation::Write::Update do
         end
 
         let(:result) do
-          update.execute(authorized_primary.context)
+          update.execute(authorized_primary)
         end
 
         it 'inserts the document' do
@@ -224,5 +224,33 @@ describe Mongo::Operation::Write::Update do
         end
       end
     end
+
+    context 'when write concern { w: 0 } is used' do
+
+      let(:update) do
+        described_class.new({
+                                update: document,
+                                db_name: TEST_DB,
+                                coll_name: TEST_COLL,
+                                write_concern: Mongo::WriteConcern.get(:w => 0)
+                            })
+      end
+
+      let(:document) do
+        { 'q' => { name: 'test' }, 'u' => { '$set' => { field: 'blah' }}, limit: 1 }
+      end
+
+      let(:result) do
+        update.execute(authorized_primary)
+      end
+
+      before do
+        expect(update).to receive(:execute_message).and_call_original
+      end
+
+      it 'uses op codes instead of write commands' do
+        expect(result.written_count).to eq(0)
+      end
+    end
   end
 end

data/spec/mongo/operation/write/update_user_spec.rb

@@ -35,7 +35,7 @@ describe Mongo::Operation::Write::UpdateUser do
     context 'when user update was successful' do
 
       let!(:response) do
-        operation.execute(root_authorized_primary.context)
+        operation.execute(root_authorized_primary)
       end
 
       it 'updates the user in the database' do