mongoid 7.0.2 → 8.1.1

data/spec/mongoid/association/auto_save_spec.rb

@@ -1,4 +1,8 @@
+# frozen_string_literal: true
+
 require "spec_helper"
+require_relative './referenced/has_many_models'
+require_relative './referenced/has_one_models'
 
 describe Mongoid::Association::Referenced::AutoSave do
 
@@ -19,22 +23,59 @@ describe Mongoid::Association::Referenced::AutoSave do
 
     context "when the option is not provided" do
 
-      let(:game) do
-        Game.new(name: "Tekken")
-      end
+      context 'has_many' do
+
+        let(:parent) do
+          HmmSchool.new
+        end
+
+        let(:child) do
+          HmmStudent.new(name: "Panda")
+        end
+
+        before do
+          parent.students = [child]
+        end
+
+        context "when saving the parent document" do
+
+          before do
+            parent.associations[:students].options[:autosave].should be_falsy
+
+            parent.save!
+          end
 
-      before do
-        person.game = game
+          it "does not save the child" do
+            expect(child).to_not be_persisted
+          end
+        end
       end
 
-      context "when saving the parent document" do
+      context 'has_one' do
+
+        let(:parent) do
+          HomCollege.new
+        end
+
+        let(:child) do
+          HomAccreditation.new
+        end
 
         before do
-          person.save
+          parent.accreditation = child
         end
 
-        it "does not save the relation" do
-          expect(game).to_not be_persisted
+        context "when saving the parent document" do
+
+          before do
+            parent.associations[:accreditation].options[:autosave].should be_falsy
+
+            parent.save!
+          end
+
+          it "does not save the child" do
+            expect(child).to_not be_persisted
+          end
         end
       end
     end
@@ -54,7 +95,7 @@ describe Mongoid::Association::Referenced::AutoSave do
         it "does not add the autosave callback twice" do
           expect(drug).to receive(:save).once
           person.drugs.push(drug)
-          person.save
+          person.save!
         end
       end
 
@@ -74,7 +115,7 @@ describe Mongoid::Association::Referenced::AutoSave do
 
             before do
               person.drugs << drug
-              person.save
+              person.save!
             end
 
             it "saves the relation" do
@@ -100,7 +141,7 @@ describe Mongoid::Association::Referenced::AutoSave do
             before do
               person.with(database: other_database) do |per|
                 per.drugs << drug
-                per.save
+                per.save!
               end
             end
 
@@ -115,9 +156,9 @@ describe Mongoid::Association::Referenced::AutoSave do
         context "when saving an existing parent document" do
 
           before do
-            person.save
+            person.save!
             person.drugs << drug
-            person.save
+            person.save!
           end
 
           it "saves the relation" do
@@ -133,11 +174,11 @@ describe Mongoid::Association::Referenced::AutoSave do
 
           before do
             person.drugs << drug
-            person.save
+            person.save!
           end
 
           it 'does not load the association' do
-            from_db.save
+            from_db.save!
             expect(from_db.ivar(:drugs)).to be false
           end
         end
@@ -153,7 +194,7 @@ describe Mongoid::Association::Referenced::AutoSave do
 
           before do
             person.account = account
-            person.save
+            person.save!
           end
 
           it "saves the relation" do
@@ -168,9 +209,9 @@ describe Mongoid::Association::Referenced::AutoSave do
         context "when saving an existing parent document" do
 
           before do
-            person.save
+            person.save!
             person.account = account
-            person.save
+            person.save!
           end
 
           it "saves the relation" do
@@ -186,14 +227,14 @@ describe Mongoid::Association::Referenced::AutoSave do
 
           before do
             person.account = account
-            person.save
+            person.save!
           end
 
           it "sends one insert" do
             account.name = "account"
             expect_query(1) do
               person.with(write: {w:0}) do |_person|
-                _person.save
+                _person.save!
               end
             end
           end
@@ -207,11 +248,11 @@ describe Mongoid::Association::Referenced::AutoSave do
 
           before do
             person.account = account
-            person.save
+            person.save!
           end
 
           it 'does not load the association' do
-            from_db.save
+            from_db.save!
             expect(from_db.ivar(:account)).to be false
           end
         end
@@ -231,7 +272,7 @@ describe Mongoid::Association::Referenced::AutoSave do
 
           before do
             ghost.movie = movie
-            ghost.save
+            ghost.save!
           end
 
           it "saves the relation" do
@@ -242,9 +283,9 @@ describe Mongoid::Association::Referenced::AutoSave do
         context "when saving an existing parent document" do
 
           before do
-            ghost.save
+            ghost.save!
             ghost.movie = movie
-            ghost.save
+            ghost.save!
           end
 
           it "saves the relation" do
@@ -256,7 +297,7 @@ describe Mongoid::Association::Referenced::AutoSave do
       context "when it has two relations with autosaves" do
 
         let!(:person) do
-          Person.create(drugs: [percocet], account: account)
+          Person.create!(drugs: [percocet], account: account)
         end
 
         let(:from_db) do
@@ -279,7 +320,7 @@ describe Mongoid::Association::Referenced::AutoSave do
 
           before do
             from_db.drugs = [placebo]
-            from_db.save
+            from_db.save!
           end
 
           it 'loads the updated association' do
@@ -294,7 +335,7 @@ describe Mongoid::Association::Referenced::AutoSave do
         context "when updating none document" do
 
           before do
-            from_db.save
+            from_db.save!
           end
 
           it 'doest not load drugs association' do
@@ -335,22 +376,22 @@ describe Mongoid::Association::Referenced::AutoSave do
         end
 
         let(:king) do
-          King.create
+          King.create!
         end
 
         let(:peasant) do
-          Peasant.create
+          Peasant.create!
         end
 
         let(:harvest) do
-          Harvest.create(season: 'Summer')
+          Harvest.create!(season: 'Summer')
         end
 
         before do
           peasant.harvest = harvest
           king.peasant = peasant
           harvest.season = 'Fall'
-          king.save
+          king.save!
         end
 
         it 'cascades the save' do

data/spec/mongoid/association/builders_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Association::Builders do
@@ -174,7 +176,7 @@ describe Mongoid::Association::Builders do
     context "when the parent is persisted" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       context "when the relation is a has one" do

data/spec/mongoid/association/constrainable_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Association::Constrainable do

data/spec/mongoid/association/counter_cache_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Association::Referenced::CounterCache do
@@ -7,7 +9,7 @@ describe Mongoid::Association::Referenced::CounterCache do
     context "when counter is reset" do
 
       let(:person) do
-        Person.create do |person|
+        Person.create! do |person|
           person[:drugs_count] = 3
         end
       end
@@ -28,7 +30,7 @@ describe Mongoid::Association::Referenced::CounterCache do
     context "when reset with invalid name" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       it "expect to raise an error" do
@@ -41,11 +43,11 @@ describe Mongoid::Association::Referenced::CounterCache do
     context "when counter gets messy" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let!(:post) do
-        person.posts.create(title: "my first post")
+        person.posts.create!(title: "my first post")
       end
 
       before do
@@ -66,11 +68,11 @@ describe Mongoid::Association::Referenced::CounterCache do
     context "when the counter is on a subclass" do
 
       let(:subscription) do
-        Subscription.create
+        Subscription.create!
       end
 
       let!(:pack) do
-        subscription.packs.create
+        subscription.packs.create!
       end
 
       before do
@@ -94,8 +96,8 @@ describe Mongoid::Association::Referenced::CounterCache do
 
       before do
         subscription.with(collection: 'other') do |sub|
-          sub.save
-          sub.packs.create
+          sub.save!
+          sub.packs.create!
         end
       end
 
@@ -112,7 +114,7 @@ describe Mongoid::Association::Referenced::CounterCache do
     context "when counter is reset" do
 
       let(:person) do
-        Person.create do |person|
+        Person.create! do |person|
           person[:drugs_count] = 3
         end
       end
@@ -131,14 +133,14 @@ describe Mongoid::Association::Referenced::CounterCache do
       it "expect to raise an error" do
         expect {
           Person.reset_counters "1", :drugs
-        }.to raise_error(Mongoid::Errors::DocumentNotFound)
+        }.to raise_error(Mongoid::Errors::DocumentNotFound, /Document\(s\) not found for class Person with id\(s\)/)
       end
     end
 
     context "when reset with invalid name" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       it "expect to raise an error" do
@@ -151,11 +153,11 @@ describe Mongoid::Association::Referenced::CounterCache do
     context "when counter gets messy" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let!(:post) do
-        person.posts.create(title: "my first post")
+        person.posts.create!(title: "my first post")
       end
 
       before do
@@ -171,11 +173,11 @@ describe Mongoid::Association::Referenced::CounterCache do
     context "when the counter is on a subclass" do
 
       let(:subscription) do
-        Subscription.create
+        Subscription.create!
       end
 
       let!(:pack) do
-        subscription.packs.create
+        subscription.packs.create!
       end
 
       before do
@@ -193,7 +195,7 @@ describe Mongoid::Association::Referenced::CounterCache do
     context "when was 3 " do
 
       let(:person) do
-        Person.create do |person|
+        Person.create! do |person|
           person[:drugs_count] = 3
         end
       end
@@ -222,7 +224,7 @@ describe Mongoid::Association::Referenced::CounterCache do
     end
     context "when update with 2 and use a string argument" do
 
-      let(:person) { Person.create }
+      let(:person) { Person.create! }
 
       before do
         Person.update_counters person.id, "drugs_count" => 2
@@ -235,7 +237,7 @@ describe Mongoid::Association::Referenced::CounterCache do
 
     context "when update more multiple counters" do
 
-      let(:person) { Person.create }
+      let(:person) { Person.create! }
 
       before do
         Person.update_counters(person.id, :drugs_count => 2, :second_counter => 5)
@@ -253,7 +255,7 @@ describe Mongoid::Association::Referenced::CounterCache do
 
   describe "#increment_counter" do
 
-    let(:person) { Person.create }
+    let(:person) { Person.create! }
 
     context "when increment 3 times" do
 
@@ -285,8 +287,8 @@ describe Mongoid::Association::Referenced::CounterCache do
 
       before do
         person.with(collection: 'other') do |per|
-          per.save
-          per.drugs.create
+          per.save!
+          per.drugs.create!
         end
       end
 
@@ -301,7 +303,7 @@ describe Mongoid::Association::Referenced::CounterCache do
   describe "#decrement_counter" do
 
     let(:person) do
-      Person.create do |p|
+      Person.create! do |p|
         p[:drugs_count] = 3
       end
     end
@@ -335,8 +337,8 @@ describe Mongoid::Association::Referenced::CounterCache do
 
       before do
         person.with(collection: 'other') do |per|
-          per.save
-          drug = per.drugs.create
+          per.save!
+          drug = per.drugs.create!
           drug.destroy
         end
       end
@@ -356,11 +358,11 @@ describe Mongoid::Association::Referenced::CounterCache do
       context 'when #destroy is called on the object' do
 
         let(:person) do
-          Person.create
+          Person.create!
         end
 
         let!(:drug) do
-          person.drugs.create
+          person.drugs.create!
         end
 
         before do
@@ -375,7 +377,7 @@ describe Mongoid::Association::Referenced::CounterCache do
       context 'when #create is called on the object' do
 
         let(:person) do
-          Person.create { |p| p.drugs += [Drug.create, Drug.create] }
+          Person.create! { |p| p.drugs += [Drug.create!, Drug.create!] }
         end
 
         it "updates the counter cache" do
@@ -386,7 +388,7 @@ describe Mongoid::Association::Referenced::CounterCache do
       context 'when #update is called on the object' do
 
         let(:person1) do
-          Person.create { |p| p.drugs += [Drug.create, Drug.create] }
+          Person.create! { |p| p.drugs += [Drug.create!, Drug.create!] }
         end
 
         let(:drug) do
@@ -394,7 +396,7 @@ describe Mongoid::Association::Referenced::CounterCache do
         end
 
         let(:person2) do
-          Person.create
+          Person.create!
         end
 
         before do
@@ -415,9 +417,9 @@ describe Mongoid::Association::Referenced::CounterCache do
 
         context 'when foreign_key differs from model name' do
 
-          let(:genre) { PostGenre.create }
+          let(:genre) { PostGenre.create! }
 
-          let(:post) { Post.create }
+          let(:post) { Post.create! }
 
           it 'updates correct counter cache' do
             post.update post_genre: genre
@@ -430,11 +432,11 @@ describe Mongoid::Association::Referenced::CounterCache do
     context "when parent is frozen" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let!(:drug) do
-        person.drugs.create
+        person.drugs.create!
       end
 
       before do