mongoid 3.1.6 → 4.0.2

data/spec/mongoid/relations/auto_save_spec.rb

@@ -5,7 +5,6 @@ describe Mongoid::Relations::AutoSave do
   describe ".auto_save" do
 
     before(:all) do
-      Person.autosaved_relations.delete_one(:account)
       Person.autosave(Person.relations["account"].merge!(autosave: true))
     end
 
@@ -34,7 +33,7 @@ describe Mongoid::Relations::AutoSave do
         end
 
         it "does not save the relation" do
-          game.should_not be_persisted
+          expect(game).to_not be_persisted
         end
       end
     end
@@ -48,7 +47,6 @@ describe Mongoid::Relations::AutoSave do
         end
 
         before do
-          Person.autosaved_relations.delete_one(:drugs)
           Person.autosave(metadata)
           Person.autosave(metadata)
         end
@@ -58,7 +56,7 @@ describe Mongoid::Relations::AutoSave do
         end
 
         it "does not add the autosave callback twice" do
-          drug.should_receive(:save).once
+          expect(drug).to receive(:save).once
           person.drugs.push(drug)
           person.save
         end
@@ -78,7 +76,7 @@ describe Mongoid::Relations::AutoSave do
           end
 
           it "saves the relation" do
-            drug.should be_persisted
+            expect(drug).to be_persisted
           end
         end
 
@@ -91,7 +89,7 @@ describe Mongoid::Relations::AutoSave do
           end
 
           it "saves the relation" do
-            drug.should be_persisted
+            expect(drug).to be_persisted
           end
         end
 
@@ -108,7 +106,7 @@ describe Mongoid::Relations::AutoSave do
 
           it 'does not load the association' do
             from_db.save
-            from_db.ivar(:drugs).should be_false
+            expect(from_db.ivar(:drugs)).to be false
           end
         end
       end
@@ -127,7 +125,11 @@ describe Mongoid::Relations::AutoSave do
           end
 
           it "saves the relation" do
-            account.should be_persisted
+            expect(account).to be_persisted
+          end
+
+          it "persists on the database" do
+            expect(account.reload).to_not be_nil
           end
         end
 
@@ -140,7 +142,26 @@ describe Mongoid::Relations::AutoSave do
           end
 
           it "saves the relation" do
-            account.should be_persisted
+            expect(account).to be_persisted
+          end
+
+          it "persists on the database" do
+            expect(account.reload).to_not be_nil
+          end
+        end
+
+        context "when updating the child" do
+
+          before do
+            person.account = account
+            person.save
+          end
+
+          it "sends one insert" do
+            account.name = "account"
+            expect_query(1) do
+              person.with(write: {w:0}).save
+            end
           end
         end
 
@@ -157,7 +178,7 @@ describe Mongoid::Relations::AutoSave do
 
           it 'does not load the association' do
             from_db.save
-            from_db.ivar(:account).should be_false
+            expect(from_db.ivar(:account)).to be false
           end
         end
       end
@@ -180,7 +201,7 @@ describe Mongoid::Relations::AutoSave do
           end
 
           it "saves the relation" do
-            movie.should be_persisted
+            expect(movie).to be_persisted
           end
         end
 
@@ -193,7 +214,7 @@ describe Mongoid::Relations::AutoSave do
           end
 
           it "saves the relation" do
-            movie.should be_persisted
+            expect(movie).to be_persisted
           end
         end
       end
@@ -201,7 +222,6 @@ describe Mongoid::Relations::AutoSave do
       context "when it has two ralations with autosaves" do
 
         before do
-          Person.autosaved_relations.delete_one(:drugs)
           Person.autosave(Person.relations["drugs"].merge!(autosave: true))
         end
 
@@ -233,11 +253,11 @@ describe Mongoid::Relations::AutoSave do
           end
 
           it 'loads the updated association' do
-            from_db.ivar(:drugs).should be_true
+            expect(from_db.ivar(:drugs)).to eq([placebo])
           end
 
           it 'doest not load the other association' do
-            from_db.ivar(:account).should be_false
+            expect(from_db.ivar(:account)).to be false
           end
         end
 
@@ -248,11 +268,11 @@ describe Mongoid::Relations::AutoSave do
           end
 
           it 'doest not load drugs association' do
-            from_db.ivar(:drugs).should be_false
+            expect(from_db.ivar(:drugs)).to be false
           end
 
           it 'doest not load account association' do
-            from_db.ivar(:account).should be_false
+            expect(from_db.ivar(:account)).to be false
           end
         end
       end

data/spec/mongoid/relations/bindings/embedded/in_spec.rb

@@ -41,11 +41,11 @@ describe Mongoid::Relations::Bindings::Embedded::In do
         end
 
         it "parentizes the documents" do
-          name._parent.should eq(person)
+          expect(name._parent).to eq(person)
         end
 
         it "sets the inverse relation" do
-          person.name.should eq(name)
+          expect(person.name).to eq(name)
         end
       end
 
@@ -56,7 +56,7 @@ describe Mongoid::Relations::Bindings::Embedded::In do
         end
 
         it "does nothing" do
-          name.should_receive(:namable=).never
+          expect(name).to receive(:namable=).never
           binding.bind_one
         end
       end
@@ -77,22 +77,22 @@ describe Mongoid::Relations::Bindings::Embedded::In do
           end
 
           it "parentizes the documents" do
-            address._parent.should eq(person)
+            expect(address._parent).to eq(person)
           end
 
           it "sets the inverse relation" do
-            person.addresses.should include(address)
+            expect(person.addresses).to include(address)
           end
         end
 
         context "when the base has metadata" do
 
           before do
-            address.metadata = person_metadata
+            address.__metadata = person_metadata
           end
 
           it "does not overwrite the existing metadata" do
-            address.should_receive(:metadata=).never
+            expect(address).to receive(:__metadata=).never
             binding.bind_one
           end
         end
@@ -105,7 +105,7 @@ describe Mongoid::Relations::Bindings::Embedded::In do
         end
 
         it "does nothing" do
-          address.should_receive(:addressable=).never
+          expect(address).to receive(:addressable=).never
           binding.bind_one
         end
       end
@@ -128,14 +128,14 @@ describe Mongoid::Relations::Bindings::Embedded::In do
         end
 
         it "removes the inverse relation" do
-          person.name.should be_nil
+          expect(person.name).to be_nil
         end
       end
 
       context "when the document is not unbindable" do
 
         it "does nothing" do
-          name.should_receive(:namable=).never
+          expect(name).to receive(:namable=).never
           binding.unbind_one
         end
       end
@@ -155,14 +155,14 @@ describe Mongoid::Relations::Bindings::Embedded::In do
         end
 
         it "removes the inverse relation" do
-          person.addresses.should be_empty
+          expect(person.addresses).to be_empty
         end
       end
 
       context "when the document is not unbindable" do
 
         it "does nothing" do
-          address.should_receive(:addressable=).never
+          expect(address).to receive(:addressable=).never
           binding.unbind_one
         end
       end

data/spec/mongoid/relations/bindings/embedded/many_spec.rb

@@ -35,18 +35,18 @@ describe Mongoid::Relations::Bindings::Embedded::Many do
       end
 
       it "parentizes the document" do
-        address_two._parent.should eq(person)
+        expect(address_two._parent).to eq(person)
       end
 
       it "sets the inverse relation" do
-        address_two.addressable.should eq(person)
+        expect(address_two.addressable).to eq(person)
       end
     end
 
     context "when the document is not bindable" do
 
       it "does nothing" do
-        person.addresses.should_receive(:<<).never
+        expect(person.addresses).to receive(:<<).never
         binding.bind_one(address)
       end
     end

data/spec/mongoid/relations/bindings/embedded/one_spec.rb

@@ -27,11 +27,11 @@ describe Mongoid::Relations::Bindings::Embedded::One do
       end
 
       it "parentizes the documents" do
-        target._parent.should eq(person)
+        expect(target._parent).to eq(person)
       end
 
       it "sets the inverse relation" do
-        target.namable.should eq(person)
+        expect(target.namable).to eq(person)
       end
     end
 
@@ -42,7 +42,7 @@ describe Mongoid::Relations::Bindings::Embedded::One do
       end
 
       it "does nothing" do
-        person.should_receive(:name=).never
+        expect(person).to receive(:name=).never
         binding.bind_one
       end
     end
@@ -62,14 +62,14 @@ describe Mongoid::Relations::Bindings::Embedded::One do
       end
 
       it "removes the inverse relation" do
-        target.namable.should be_nil
+        expect(target.namable).to be_nil
       end
     end
 
     context "when the document is not unbindable" do
 
       it "does nothing" do
-        person.should_receive(:name=).never
+        expect(person).to receive(:name=).never
         binding.unbind_one
       end
     end

data/spec/mongoid/relations/bindings/referenced/in_spec.rb

@@ -33,17 +33,17 @@ describe Mongoid::Relations::Bindings::Referenced::In do
       context "when the document is bindable with default pk" do
 
         before do
-          person.should_receive(:save).never
-          game.should_receive(:save).never
+          expect(person).to receive(:save).never
+          expect(game).to receive(:save).never
           binding.bind_one
         end
 
         it "sets the inverse relation" do
-          person.game.should eq(game)
+          expect(person.game).to eq(game)
         end
 
         it "sets the foreign key" do
-          game.person_id.should eq(person.id)
+          expect(game.person_id).to eq(person.id)
         end
       end
 
@@ -52,8 +52,8 @@ describe Mongoid::Relations::Bindings::Referenced::In do
         before do
           Game.belongs_to :person, index: true, validate: true, primary_key: :username
 
-          person.should_receive(:save).never
-          game.should_receive(:save).never
+          expect(person).to receive(:save).never
+          expect(game).to receive(:save).never
           binding.bind_one
         end
 
@@ -62,7 +62,7 @@ describe Mongoid::Relations::Bindings::Referenced::In do
         end
 
         it "sets the inverse relation" do
-          person.game.should eq(game)
+          expect(person.game).to eq(game)
         end
 
         let(:person) do
@@ -70,7 +70,7 @@ describe Mongoid::Relations::Bindings::Referenced::In do
         end
 
         it "sets the fk with username field" do
-          game.person_id.should eq(person.username)
+          expect(game.person_id).to eq(person.username)
         end
       end
 
@@ -81,7 +81,7 @@ describe Mongoid::Relations::Bindings::Referenced::In do
         end
 
         it "does nothing" do
-          game.should_receive(:person=).never
+          expect(game).to receive(:person=).never
           binding.bind_one
         end
       end
@@ -96,17 +96,17 @@ describe Mongoid::Relations::Bindings::Referenced::In do
       context "when the document is bindable" do
 
         before do
-          person.should_receive(:save).never
-          post.should_receive(:save).never
+          expect(person).to receive(:save).never
+          expect(post).to receive(:save).never
           binding.bind_one
         end
 
         it "sets the inverse relation" do
-          person.posts.should include(post)
+          expect(person.posts).to include(post)
         end
 
         it "sets the foreign key" do
-          post.person_id.should eq(person.id)
+          expect(post.person_id).to eq(person.id)
         end
       end
 
@@ -117,7 +117,7 @@ describe Mongoid::Relations::Bindings::Referenced::In do
         end
 
         it "does nothing" do
-          post.should_receive(:person=).never
+          expect(post).to receive(:person=).never
           binding.bind_one
         end
       end
@@ -136,24 +136,24 @@ describe Mongoid::Relations::Bindings::Referenced::In do
 
         before do
           binding.bind_one
-          person.should_receive(:delete).never
-          game.should_receive(:delete).never
+          expect(person).to receive(:delete).never
+          expect(game).to receive(:delete).never
           binding.unbind_one
         end
 
         it "removes the inverse relation" do
-          person.game.should be_nil
+          expect(person.game).to be_nil
         end
 
         it "removed the foreign key" do
-          game.person_id.should be_nil
+          expect(game.person_id).to be_nil
         end
       end
 
       context "when the document is not unbindable" do
 
         it "does nothing" do
-          game.should_receive(:person=).never
+          expect(game).to receive(:person=).never
           binding.unbind_one
         end
       end
@@ -169,24 +169,24 @@ describe Mongoid::Relations::Bindings::Referenced::In do
 
         before do
           binding.bind_one
-          person.should_receive(:delete).never
-          post.should_receive(:delete).never
+          expect(person).to receive(:delete).never
+          expect(post).to receive(:delete).never
           binding.unbind_one
         end
 
         it "removes the inverse relation" do
-          person.posts.should be_empty
+          expect(person.posts).to be_empty
         end
 
         it "removes the foreign key" do
-          post.person_id.should be_nil
+          expect(post.person_id).to be_nil
         end
       end
 
       context "when the document is not unbindable" do
 
         it "does nothing" do
-          post.should_receive(:person=).never
+          expect(post).to receive(:person=).never
           binding.unbind_one
         end
       end
@@ -210,7 +210,7 @@ describe Mongoid::Relations::Bindings::Referenced::In do
       end
 
       it "does not set the foreign key" do
-        game.person_id.should be_nil
+        expect(game.person_id).to be_nil
       end
     end
   end
@@ -234,7 +234,7 @@ describe Mongoid::Relations::Bindings::Referenced::In do
       end
 
       it "does not unset the foreign key" do
-        game.person_id.should eq(person.id)
+        expect(game.person_id).to eq(person.id)
       end
     end
   end