mongoid 7.0.5 → 8.1.5

data/spec/mongoid/association/referenced/has_many/enumerable_spec.rb

@@ -1,17 +1,19 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
-describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
+describe Mongoid::Association::Referenced::HasMany::Enumerable do
 
   describe "#==" do
 
     context "when comparing with an enumerable" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let!(:post) do
-        Post.create(person_id: person.id)
+        Post.create!(person_id: person.id)
       end
 
       context "when only a criteria target exists" do
@@ -85,7 +87,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
           context "when the loaded has no docs and added is persisted" do
 
             before do
-              post.save
+              post.save!
               enumerable._added[post.id] = post
             end
 
@@ -111,72 +113,103 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
 
   describe "#===" do
 
-    let(:enumerable) do
-      described_class.new([])
-    end
+    let(:data) { [] }
 
-    context "when compared to an array class" do
+    shared_examples 'standard library-compatible enumerable' do
+      context "when compared to an array class" do
 
-      it "returns true" do
-        expect(enumerable === Array).to be true
+        it "returns false" do
+          expect(enumerable === Array).to be false
+        end
       end
-    end
 
-    context "when compared to an enumerable class" do
+      context "when compared to an enumerable class" do
 
-      it "returns true" do
-        expect(enumerable === described_class).to be true
+        it "returns false" do
+          expect(enumerable === described_class).to be false
+        end
       end
-    end
 
-    context "when compared to a different class" do
+      context "when compared to a different class" do
 
-      it "returns false" do
-        expect(enumerable === Mongoid::Document).to be false
+        it "returns false" do
+          expect(enumerable === Mongoid::Document).to be false
+        end
       end
-    end
 
-    context "when compared to an array instance" do
+      context "when compared to an array instance" do
 
-      context "when the entries are equal" do
+        context "when the entries are equal" do
 
-        let(:other) do
-          described_class.new([])
-        end
+          let(:data) do
+            [Post.new(id: 2)]
+          end
 
-        it "returns true" do
-          expect(enumerable === other).to be true
+          let(:other) do
+            described_class.new([Post.new(id: 2)])
+          end
+
+          it "returns true" do
+            expect(enumerable === other).to be true
+          end
         end
-      end
 
-      context "when the entries are not equal" do
+        context "when the entries are both empty" do
 
-        let(:other) do
-          described_class.new([ Band.new ])
+          let(:other) do
+            described_class.new([])
+          end
+
+          it "returns true" do
+            expect(enumerable === other).to be true
+          end
         end
 
-        it "returns false" do
-          expect(enumerable === other).to be false
+        context "when the entries are not equal" do
+
+          let(:other) do
+            described_class.new([ Band.new ])
+          end
+
+          it "returns false" do
+            expect(enumerable === other).to be false
+          end
         end
       end
     end
+
+    context 'enumerable' do
+      let(:enumerable) do
+        described_class.new(data)
+      end
+
+      include_examples 'standard library-compatible enumerable'
+    end
+
+    context 'Array instance' do
+      let(:enumerable) do
+        data
+      end
+
+      include_examples 'standard library-compatible enumerable'
+    end
   end
 
   describe "#<<" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post) do
-      Post.create(person_id: person.id)
+      Post.create!(person_id: person.id)
     end
 
     let!(:enumerable) do
       described_class.new([])
     end
 
-    context "when the relation is empty" do
+    context "when the association is empty" do
 
       let!(:added) do
         enumerable << post
@@ -198,18 +231,18 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
     end
   end
 
-  describe "#any?" do
+  describe "#empty?" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post_one) do
-      Post.create(person_id: person.id)
+      Post.create!(person_id: person.id)
     end
 
     let!(:post_two) do
-      Post.create(person_id: person.id)
+      Post.create!(person_id: person.id)
     end
 
     context "when only a criteria target exists" do
@@ -222,28 +255,139 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
         described_class.new(criteria)
       end
 
-      let!(:any) do
-        enumerable.any?
+      let(:empty) do
+        enumerable.empty?
       end
 
-      it "returns true" do
-        expect(any).to be true
+      it "returns false" do
+        expect(empty).to be false
       end
 
-      it "retains the correct length" do
-        expect(enumerable.length).to eq(2)
+      context 'when #empty? is called' do
+
+        before { empty }
+
+        it "retains the correct length" do
+          expect(enumerable.length).to eq(2)
+        end
+
+        it "retains the correct length when calling to_a" do
+          expect(enumerable.to_a.length).to eq(2)
+        end
+
+        context "when iterating over the association a second time" do
+
+          before do
+            enumerable.each { |post| post }
+          end
+
+          it "retains the correct length" do
+            expect(enumerable.length).to eq(2)
+          end
+
+          it "retains the correct length when calling to_a" do
+            expect(enumerable.to_a.length).to eq(2)
+          end
+        end
       end
+    end
 
-      it "retains the correct length when calling to_a" do
-        expect(enumerable.to_a.length).to eq(2)
+    context "when the documents have been loaded" do
+      let(:criteria) do
+        Post.where(person_id: person.id)
       end
 
-      context "when iterating over the relation a second time" do
+      let!(:enumerable) do
+        described_class.new(criteria)
+      end
+
+      before do
+        enumerable.load_all!
+      end
+
+      it "is _loaded" do
+        expect(enumerable._loaded?).to be true
+      end
+
+      it "it does not call #exists? on the unloaded scope" do
+        expect(enumerable._unloaded).to_not receive(:exists?)
+        expect(enumerable.empty?).to be false
+      end
+    end
+
+    context "when the documents are not loaded" do
+
+      let(:criteria) do
+        Post.where(person_id: person.id)
+      end
+
+      let!(:enumerable) do
+        described_class.new(criteria)
+      end
+
+      it "is not _loaded" do
+        expect(enumerable._loaded?).to be false
+      end
+
+      it "it calls #exists? on the unloaded scope" do
+        expect(enumerable._unloaded).to receive(:exists?)
+        expect(enumerable.empty?).to be true
+      end
+
+      context "when documents are added" do
 
         before do
-          enumerable.each { |post| post }
+          enumerable << post_one
         end
 
+        it "is not _loaded" do
+          expect(enumerable._loaded?).to be false
+        end
+
+        it "it does not call #exists? on the unloaded scope" do
+          expect(enumerable._unloaded).to_not receive(:exists?)
+          expect(enumerable.empty?).to be false
+        end
+      end
+    end
+  end
+
+  describe "#any?" do
+
+    let(:person) do
+      Person.create!
+    end
+
+    let!(:post_one) do
+      Post.create!(person_id: person.id)
+    end
+
+    let!(:post_two) do
+      Post.create!(person_id: person.id)
+    end
+
+    context "when only a criteria target exists" do
+
+      let(:criteria) do
+        Post.where(person_id: person.id)
+      end
+
+      let!(:enumerable) do
+        described_class.new(criteria)
+      end
+
+      let(:any) do
+        enumerable.any?
+      end
+
+      it "returns true" do
+        expect(any).to be true
+      end
+
+      context 'when #any? is called' do
+
+        before { any }
+
         it "retains the correct length" do
           expect(enumerable.length).to eq(2)
         end
@@ -251,6 +395,148 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
         it "retains the correct length when calling to_a" do
           expect(enumerable.to_a.length).to eq(2)
         end
+
+        context "when iterating over the association a second time" do
+
+          before do
+            enumerable.each { |post| post }
+          end
+
+          it "retains the correct length" do
+            expect(enumerable.length).to eq(2)
+          end
+
+          it "retains the correct length when calling to_a" do
+            expect(enumerable.to_a.length).to eq(2)
+          end
+        end
+      end
+    end
+
+    context "when the documents have been loaded" do
+      let(:criteria) do
+        Post.where(person_id: person.id)
+      end
+
+      let!(:enumerable) do
+        described_class.new(criteria)
+      end
+
+      before do
+        enumerable.load_all!
+      end
+
+      it "is _loaded" do
+        expect(enumerable._loaded?).to be true
+      end
+
+      it "it does not call #exists? on the unloaded scope" do
+        expect(enumerable._unloaded).to_not receive(:exists?)
+        expect(enumerable.any?).to be true
+      end
+
+      context "when a block is given" do
+        it "returns true when the predicate is true" do
+          expect(
+            enumerable.any? { |doc| true }
+          ).to be true
+        end
+
+        it "returns false when the predicate is false" do
+          expect(
+            enumerable.any? { |doc| false }
+          ).to be false
+        end
+      end
+
+      context "when an argument is given" do
+
+        it "returns true when the argument is true" do
+          expect(enumerable.any?(Post)).to be true
+        end
+
+        it "returns false when the argument is false" do
+          expect(enumerable.any?(Sandwich)).to be false
+        end
+      end
+
+      context "when both an argument and a block are given" do
+
+        it "gives precedence to the pattern" do
+          expect(
+            enumerable.any?(Post) { |doc| false }
+          ).to be true
+        end
+      end
+    end
+
+    context "when the documents are not loaded" do
+
+      let(:criteria) do
+        Post.where(person_id: person.id)
+      end
+
+      let!(:enumerable) do
+        described_class.new(criteria)
+      end
+
+      it "is not _loaded" do
+        expect(enumerable._loaded?).to be false
+      end
+
+      it "it calls #exists? on the unloaded scope" do
+        expect(enumerable._unloaded).to receive(:exists?)
+        expect(enumerable.any?).to be false
+      end
+
+      context "when documents are added" do
+
+        before do
+          enumerable << post_one
+        end
+
+        it "is not _loaded" do
+          expect(enumerable._loaded?).to be false
+        end
+
+        it "it does not call #exists? on the unloaded scope" do
+          expect(enumerable._unloaded).to_not receive(:exists?)
+          expect(enumerable.any?).to be true
+        end
+      end
+
+      context "when a block is given" do
+        it "returns true when the predicate is true" do
+          expect(
+            enumerable.any? { |doc| true }
+          ).to be true
+        end
+
+        it "returns false when the predicate is false" do
+          expect(
+            enumerable.any? { |doc| false }
+          ).to be false
+        end
+      end
+
+      context "when an argument is given" do
+
+        it "returns true when the argument is true" do
+          expect(enumerable.any?(Post)).to be true
+        end
+
+        it "returns false when the argument is false" do
+          expect(enumerable.any?(Sandwich)).to be false
+        end
+      end
+
+      context "when both an argument and a block are given" do
+
+        it "gives precedence to the pattern" do
+          expect(
+            enumerable.any?(Post) { |doc| false }
+          ).to be true
+        end
       end
     end
   end
@@ -258,15 +544,15 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#clear" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post) do
-      Post.create(person_id: person.id)
+      Post.create!(person_id: person.id)
     end
 
     let!(:post_two) do
-      Post.create(person_id: person.id)
+      Post.create!(person_id: person.id)
     end
 
     let(:criteria) do
@@ -304,15 +590,15 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#clone" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post) do
-      Post.create(title: "one", person_id: person.id)
+      Post.create!(title: "one", person_id: person.id)
     end
 
     let!(:post_two) do
-      Post.create(title: "two", person_id: person.id)
+      Post.create!(title: "two", person_id: person.id)
     end
 
     let(:criteria) do
@@ -344,13 +630,13 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#delete" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     context "when the document is loaded" do
 
       let!(:post) do
-        Post.create(person_id: person.id)
+        Post.create!(person_id: person.id)
       end
 
       let!(:enumerable) do
@@ -404,7 +690,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
     context "when the document is unloaded" do
 
       let!(:post) do
-        Post.create(person_id: person.id)
+        Post.create!(person_id: person.id)
       end
 
       let(:criteria) do
@@ -431,7 +717,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
     context "when the document is not found" do
 
       let!(:post) do
-        Post.create(person_id: person.id)
+        Post.create!(person_id: person.id)
       end
 
       let(:criteria) do
@@ -457,13 +743,13 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#delete_if" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     context "when the document is loaded" do
 
       let!(:post) do
-        Post.create(person_id: person.id)
+        Post.create!(person_id: person.id)
       end
 
       let!(:enumerable) do
@@ -517,7 +803,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
     context "when the document is unloaded" do
 
       let!(:post) do
-        Post.create(person_id: person.id)
+        Post.create!(person_id: person.id)
       end
 
       let(:criteria) do
@@ -544,7 +830,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
     context "when the block doesn't match" do
 
       let!(:post) do
-        Post.create(person_id: person.id)
+        Post.create!(person_id: person.id)
       end
 
       let(:criteria) do
@@ -568,11 +854,11 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#detect" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post) do
-      Post.create(person: person, title: "test")
+      Post.create!(person: person, title: "test")
     end
 
     let(:criteria) do
@@ -598,11 +884,11 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#each" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post) do
-      Post.create(person_id: person.id)
+      Post.create!(person_id: person.id)
     end
 
     context "when only a criteria target exists" do
@@ -629,18 +915,18 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
         expect(enumerable).to be__loaded
       end
 
-      context 'when the base relation is accessed from each document' do
+      context 'when the base association is accessed from each document' do
 
         let(:persons) do
           described_class.new(criteria).collect(&:person)
         end
 
         before do
-          Post.create(person_id: person.id)
-          Post.create(person_id: person.id)
+          Post.create!(person_id: person.id)
+          Post.create!(person_id: person.id)
         end
 
-        it 'sets the base relation from the criteria' do
+        it 'sets the base association from the criteria' do
           expect(persons.uniq.size).to eq(1)
         end
       end
@@ -748,7 +1034,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#entries" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let(:criteria) do
@@ -762,7 +1048,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
     context "when the added contains a persisted document" do
 
       let!(:post) do
-        Post.create(person_id: person.id)
+        Post.create!(person_id: person.id)
       end
 
       before do
@@ -782,7 +1068,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#first" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     context "when the enumerable is not loaded" do
@@ -800,7 +1086,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
         context "when added is empty" do
 
           let!(:post) do
-            Post.create(person_id: person.id)
+            Post.create!(person_id: person.id)
           end
 
           let(:first) do
@@ -824,7 +1110,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
         context "when added is not empty" do
 
           let!(:post) do
-            Post.create(person_id: person.id)
+            Post.create!(person_id: person.id)
           end
 
           let(:post_two) do
@@ -896,7 +1182,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
       context "when loaded is not empty" do
 
         let!(:post) do
-          Post.create(person_id: person.id)
+          Post.create!(person_id: person.id)
         end
 
         let(:enumerable) do
@@ -915,7 +1201,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
       context "when loaded is empty" do
 
         let!(:post) do
-          Post.create(person_id: person.id)
+          Post.create!(person_id: person.id)
         end
 
         let(:enumerable) do
@@ -951,10 +1237,10 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
       end
     end
 
-    context 'when the id_sort option is none' do
+    context 'when including a limit' do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let(:criteria) do
@@ -966,22 +1252,22 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
       end
 
       let!(:first_post) do
-        person.posts.create(title: "One")
+        person.posts.create!(title: "One")
       end
 
       let!(:second_post) do
-        person.posts.create(title: "Two")
+        person.posts.create!(title: "Two")
       end
 
-      it 'does not use the sort on id' do
-        expect(enumerable.first(id_sort: :none)).to eq(first_post)
+      it 'returns the matching document' do
+        expect(enumerable.first(1)).to eq([first_post])
       end
     end
 
-    context 'when the id_sort option is not provided' do
+    context 'when no parameters are provided' do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let(:criteria) do
@@ -993,11 +1279,11 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
       end
 
       let!(:first_post) do
-        person.posts.create(title: "One")
+        person.posts.create!(title: "One")
       end
 
       let!(:second_post) do
-        person.posts.create(title: "Two")
+        person.posts.create!(title: "Two")
       end
 
       it 'uses the sort on id' do
@@ -1009,15 +1295,15 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#include?" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post_one) do
-      Post.create(person_id: person.id)
+      Post.create!(person_id: person.id)
     end
 
     let!(:post_two) do
-      Post.create(person_id: person.id)
+      Post.create!(person_id: person.id)
     end
 
     context "when no criteria exists" do
@@ -1079,7 +1365,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
         described_class.new(criteria)
       end
 
-      let!(:included) do
+      let(:included) do
         enumerable.include?(post_two)
       end
 
@@ -1095,7 +1381,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
         expect(enumerable.to_a.length).to eq(2)
       end
 
-      context "when iterating over the relation a second time" do
+      context "when iterating over the association a second time" do
 
         before do
           enumerable.each { |post| post }
@@ -1263,7 +1549,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#last" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     context "when the enumerable is not loaded" do
@@ -1279,7 +1565,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
       context "when unloaded is not empty" do
 
         let!(:post) do
-          Post.create(person_id: person.id)
+          Post.create!(person_id: person.id)
         end
 
         let(:last) do
@@ -1341,11 +1627,11 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
       context "when added is not empty" do
 
         let!(:post_one) do
-          person.posts.create
+          person.posts.create!
         end
 
         let!(:post_two) do
-          person.posts.create
+          person.posts.create!
         end
 
         let(:last) do
@@ -1366,7 +1652,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
       context "when loaded is not empty" do
 
         let!(:post) do
-          Post.create(person_id: person.id)
+          Post.create!(person_id: person.id)
         end
 
         let(:enumerable) do
@@ -1385,7 +1671,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
       context "when loaded is empty" do
 
         let!(:post) do
-          Post.create(person_id: person.id)
+          Post.create!(person_id: person.id)
         end
 
         let(:enumerable) do
@@ -1421,10 +1707,10 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
       end
     end
 
-    context 'when the id_sort option is none' do
+    context 'when including a limit' do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let(:criteria) do
@@ -1436,22 +1722,22 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
       end
 
       let!(:first_post) do
-        person.posts.create(title: "One")
+        person.posts.create!(title: "One")
       end
 
       let!(:second_post) do
-        person.posts.create(title: "Two")
+        person.posts.create!(title: "Two")
       end
 
-      it 'does not use the sort on id' do
-        expect(enumerable.last(id_sort: :none)).to eq(first_post)
+      it 'returns the matching document' do
+        expect(enumerable.last(1)).to eq([second_post])
       end
     end
 
-    context 'when the id_sort option is not provided' do
+    context 'when no parameters are provided' do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let(:criteria) do
@@ -1463,11 +1749,11 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
       end
 
       let!(:first_post) do
-        person.posts.create(title: "One")
+        person.posts.create!(title: "One")
       end
 
       let!(:second_post) do
-        person.posts.create(title: "Two")
+        person.posts.create!(title: "Two")
       end
 
       it 'uses the sort on id' do
@@ -1500,11 +1786,11 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#load_all!" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post) do
-      Post.create(person_id: person.id)
+      Post.create!(person_id: person.id)
     end
 
     let(:criteria) do
@@ -1535,15 +1821,15 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#reset" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let(:post) do
-      Post.create(person_id: person.id)
+      Post.create!(person_id: person.id)
     end
 
     let(:post_two) do
-      Post.create(person_id: person.id)
+      Post.create!(person_id: person.id)
     end
 
     let(:enumerable) do
@@ -1591,11 +1877,11 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#size" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post) do
-      Post.create(person_id: person.id)
+      Post.create!(person_id: person.id)
     end
 
     context "when the base is new" do
@@ -1607,7 +1893,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
       context "when the added contains a persisted document" do
 
         let!(:post) do
-          Post.create(person_id: person.id)
+          Post.create!(person_id: person.id)
         end
 
         context "when the enumerable is not loaded" do
@@ -1678,7 +1964,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
       context "when the added contains persisted documents" do
 
         let(:post_two) do
-          Post.create(person_id: person.id)
+          Post.create!(person_id: person.id)
         end
 
         before do
@@ -1699,11 +1985,11 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#to_json" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post) do
-      Post.create(title: "test", person_id: person.id)
+      Post.create!(title: "test", person_id: person.id)
     end
 
     let(:criteria) do
@@ -1730,11 +2016,11 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#to_json(parameters)" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post) do
-      Post.create(title: "test", person_id: person.id)
+      Post.create!(title: "test", person_id: person.id)
     end
 
     let(:criteria) do
@@ -1753,11 +2039,11 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#as_json" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post) do
-      Post.create(title: "test", person_id: person.id)
+      Post.create!(title: "test", person_id: person.id)
     end
 
     let(:criteria) do
@@ -1785,11 +2071,11 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#as_json(parameters)" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post) do
-      Post.create(title: "test", person_id: person.id)
+      Post.create!(title: "test", person_id: person.id)
     end
 
     let(:criteria) do
@@ -1812,11 +2098,11 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe "#uniq" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post) do
-      Post.create(person_id: person.id)
+      Post.create!(person_id: person.id)
     end
 
     let(:criteria) do
@@ -1848,7 +2134,7 @@ describe Mongoid::Association::Referenced::HasMany::Targets::Enumerable do
   describe 'setting the same parent object on enumerated children objects' do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     context 'when a single child is fetched' do