mongoid 7.3.5 → 7.4.0

data/spec/mongoid/association/embedded/embeds_many/proxy_spec.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
-# encoding: utf-8
 
 require "spec_helper"
-require_relative '../embeds_many_models.rb'
 
 describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
 
@@ -116,7 +114,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       context "when the parent is not a new record" do
 
         let(:person) do
-          Person.create
+          Person.create!
         end
 
         let(:address) do
@@ -135,7 +133,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       context "when appending more than one document at once" do
 
         let(:person) do
-          Person.create
+          Person.create!
         end
 
         let(:address_one) do
@@ -211,7 +209,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
         context "when the parent is not a new record" do
 
           let(:parent_role) do
-            Role.create(name: "CEO")
+            Role.create!(name: "CEO")
           end
 
           let(:child_role) do
@@ -230,7 +228,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
 
       context "when the child has one sided many to many relation" do
         let(:person) do
-          Person.create
+          Person.create!
         end
 
         let(:message) do
@@ -313,7 +311,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when the parent is not a new record" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let(:address) do
@@ -334,11 +332,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       context "when setting via an overridden method from the parent" do
 
         let!(:person) do
-          Person.create
+          Person.create!
         end
 
         let!(:address) do
-          person.addresses.create(street: "Alt Treptow")
+          person.addresses.create!(street: "Alt Treptow")
         end
 
         let!(:new_address) do
@@ -346,7 +344,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
         end
 
         before do
-          person.update_attributes(set_addresses: [ new_address ])
+          person.update_attributes!(set_addresses: [ new_address ])
         end
 
         it "overwrites the existing addresses" do
@@ -376,7 +374,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
 
           before do
             person.addresses = [ address_two ]
-            person.save
+            person.save!
           end
 
           it "sets the new documents" do
@@ -410,7 +408,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
 
           before do
             doctor.addresses = [ address_one ]
-            doctor.save
+            doctor.save!
           end
 
           it "sets the documents" do
@@ -429,7 +427,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
 
             before do
               doctor.addresses = [ address_two ]
-              doctor.save
+              doctor.save!
             end
 
             it "sets the new documents" do
@@ -447,7 +445,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when replacing an existing relation" do
 
       let(:person) do
-        Person.create(addresses: addresses)
+        Person.create!(addresses: addresses)
       end
 
       let(:addresses) do
@@ -508,7 +506,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       end
 
       let(:tracking_id) do
-        MyCompany::Model::TrackingId.create
+        MyCompany::Model::TrackingId.create!
       end
 
       let(:history) do
@@ -543,7 +541,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
           inflect.singular("address_numbers", "address_number")
         end
         slave.address_numbers << AddressNumber.new(country_code: 1)
-        slave.save
+        slave.save!
       end
 
       it "requires an inflection to determine the class" do
@@ -554,15 +552,15 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when setting the entire tree via a hash" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let!(:address_one) do
-        person.addresses.create(street: "Tauentzienstr")
+        person.addresses.create!(street: "Tauentzienstr")
       end
 
       let!(:address_two) do
-        person.addresses.create(street: "Kudamm")
+        person.addresses.create!(street: "Kudamm")
       end
 
       let(:attributes) do
@@ -573,7 +571,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
 
         before do
           attributes["addresses"][0]["city"] = "Berlin"
-          person.update_attributes(attributes)
+          person.update_attributes!(attributes)
         end
 
         it "sets the new attributes" do
@@ -589,15 +587,15 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when setting an embedded sub-document tree via a hash" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let!(:address_one) do
-        person.addresses.create(street: "Tauentzienstr")
+        person.addresses.create!(street: "Tauentzienstr")
       end
 
       let!(:location_one) do
-        person.addresses.first.locations.create(name: "Work")
+        person.addresses.first.locations.create!(name: "Work")
       end
 
       let(:attributes) do
@@ -609,7 +607,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
         before do
           attributes["city"] = "Berlin"
           attributes["locations"][0]["name"] = "Home"
-          person.addresses.first.update_attributes(attributes)
+          person.addresses.first.update_attributes!(attributes)
         end
 
         it "sets the new attributes on the address" do
@@ -682,7 +680,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       context "when the parent is not a new record" do
 
         let(:parent_role) do
-          Role.create(name: "CTO")
+          Role.create!(name: "CTO")
         end
 
         let(:child_role) do
@@ -750,7 +748,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       context "when the parent is persisted" do
 
         let(:person) do
-          Person.create
+          Person.create!
         end
 
         let(:address) do
@@ -799,7 +797,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
           context "when saving the parent" do
 
             before do
-              person.save
+              person.save!
               person.reload
             end
 
@@ -813,7 +811,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       context "when setting on a reload" do
 
         let(:person) do
-          Person.create
+          Person.create!
         end
 
         let(:address) do
@@ -883,7 +881,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       context "when the documents are not new records" do
 
         let(:parent_role) do
-          Role.create
+          Role.create!
         end
 
         let(:child_role) do
@@ -913,13 +911,13 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   describe "#as_document" do
 
     let!(:person) do
-      Person.create
+      Person.create!
     end
 
     context 'when a string is used to access an attribute' do
 
       let!(:address) do
-        person.addresses.create(street: "one")
+        person.addresses.create!(street: "one")
       end
 
       let(:document) do
@@ -934,7 +932,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context 'when a symbol is used to access an attribute' do
 
       let!(:address) do
-        person.addresses.create(street: "one")
+        person.addresses.create!(street: "one")
       end
 
       let(:document) do
@@ -949,7 +947,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when the relation has no default scope" do
 
       let!(:address) do
-        person.addresses.create(street: "one")
+        person.addresses.create!(street: "one")
       end
 
       let(:document) do
@@ -1106,7 +1104,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       context "when providing nested attributes" do
 
         let(:person) do
-          Person.create
+          Person.create!
         end
 
         let(:address) do
@@ -1120,7 +1118,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
         context "when followed by a save" do
 
           before do
-            address.save
+            address.save!
           end
 
           let(:location) do
@@ -1140,13 +1138,13 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when the parent has been persisted" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       context "when the children are persisted" do
 
         let!(:address) do
-          person.addresses.create(street: "High St")
+          person.addresses.create!(street: "High St")
         end
 
         let!(:relation) do
@@ -1262,7 +1260,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when the parent is not a new record" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let(:address) do
@@ -1281,7 +1279,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when concatenating an empty array" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       before do
@@ -1297,7 +1295,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when appending more than one document at once" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let(:address_one) do
@@ -1369,7 +1367,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       context "when the parent is not a new record" do
 
         let(:parent_role) do
-          Role.create(name: "CEO")
+          Role.create!(name: "CEO")
         end
 
         let(:child_role) do
@@ -1390,49 +1388,261 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   describe "#count" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     before do
-      person.addresses.create(street: "Upper")
+      person.addresses.create!(street: "Upper")
       person.addresses.build(street: "Bond")
+      person.addresses.build(street: "Lower")
     end
 
     it "returns the number of persisted documents" do
       expect(person.addresses.count).to eq(1)
     end
+
+    context 'block form' do
+      it "iterates across all documents" do
+        expect(person.addresses.count {|a| a.persisted? }).to eq(1)
+        expect(person.addresses.count {|a| !a.persisted? }).to eq(2)
+        expect(person.addresses.count {|a| a.street.include?('on') }).to eq(1)
+        expect(person.addresses.count {|a| a.street.ends_with?('er') }).to eq(2)
+      end
+    end
+
+    context 'argument form' do
+      it "behaves correctly when given a model instance" do
+        expect(person.addresses.count(person.addresses.first)).to eq(1)
+      end
+
+      it "behaves correctly when given a non-model instance" do
+        expect(person.addresses.count(1)).to eq(0)
+      end
+    end
   end
 
   describe "#any?" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     context "when documents are persisted" do
       before do
-        person.addresses.create(street: "Upper")
+        person.addresses.create!(street: "Upper")
       end
 
       it "returns true" do
         expect(person.addresses.any?).to be true
       end
+
+      it "block form iterates across all documents" do
+        expect(person.addresses.any? {|a| a.street == "Upper" }).to be true
+        expect(person.addresses.any? {|a| a.street == "Bond" }).to be false
+      end
+
+      context 'argument form' do
+        it "behaves correctly when given a model instance" do
+          expect(person.addresses.any?(person.addresses.first)).to be true
+        end
+
+        it "behaves correctly when given a non-model instance" do
+          expect(person.addresses.any?(1)).to be false
+        end
+      end
     end
-    
+
     context "when documents are not persisted" do
       before do
         person.addresses.build(street: "Bond")
       end
-      
+
       it "returns true" do
         expect(person.addresses.any?).to be true
       end
+
+      it "block form iterates across all documents" do
+        expect(person.addresses.any? {|a| a.street == "Upper" }).to be false
+        expect(person.addresses.any? {|a| a.street == "Bond" }).to be true
+      end
+
+      context 'argument form' do
+        it "behaves correctly when given a model instance" do
+          expect(person.addresses.any?(person.addresses.first)).to be true
+        end
+
+        it "behaves correctly when given a non-model instance" do
+          expect(person.addresses.any?(1)).to be false
+        end
+      end
     end
 
-    context "when documents are not created" do
+    context "when documents are not present" do
       it "returns false" do
         expect(person.addresses.any?).to be false
       end
+
+      it "block form iterates across all documents" do
+        expect(person.addresses.any?(&:a)).to be false
+      end
+
+      it "argument form is supported" do
+        expect(person.addresses.any?(1)).to be false
+      end
+    end
+  end
+
+  describe "#all?" do
+
+    let(:person) do
+      Person.create!
+    end
+
+    context "when documents are persisted" do
+      before do
+        person.addresses.create!(street: "Upper")
+      end
+
+      it "returns true" do
+        expect(person.addresses.all?).to be true
+      end
+
+      it "block form iterates across all documents" do
+        expect(person.addresses.all? {|a| a.street == "Upper" }).to be true
+        expect(person.addresses.all? {|a| a.street == "Bond" }).to be false
+      end
+
+      context 'argument form' do
+        it "behaves correctly when given a model instance" do
+          expect(person.addresses.all?(person.addresses.first)).to be true
+        end
+
+        it "behaves correctly when given a non-model instance" do
+          expect(person.addresses.all?(1)).to be false
+        end
+      end
+    end
+
+    context "when documents are not persisted" do
+      before do
+        person.addresses.build(street: "Bond")
+      end
+
+      it "returns true" do
+        expect(person.addresses.all?).to be true
+      end
+
+      it "block form iterates across all documents" do
+        expect(person.addresses.all? {|a| a.street == "Upper" }).to be false
+        expect(person.addresses.all? {|a| a.street == "Bond" }).to be true
+      end
+
+      context 'argument form' do
+        it "behaves correctly when given a model instance" do
+          expect(person.addresses.all?(person.addresses.first)).to be true
+        end
+
+        it "behaves correctly when given a non-model instance" do
+          expect(person.addresses.all?(1)).to be false
+        end
+      end
+    end
+
+    context "when documents are not present" do
+      it "returns false" do
+        expect(person.addresses.all?).to be true
+      end
+
+      it "block form iterates across all documents" do
+        expect(person.addresses.all?(&:foo)).to be true
+      end
+
+      context 'argument form' do
+        it "behaves correctly when given nil" do
+          expect(person.addresses.all?(nil)).to be true
+        end
+
+        it "behaves correctly when given a non-model instance" do
+          expect(person.addresses.all?(1)).to be true
+        end
+      end
+    end
+  end
+
+  describe "#none?" do
+
+    let(:person) do
+      Person.create!
+    end
+
+    context "when documents are persisted" do
+      before do
+        person.addresses.create!(street: "Upper")
+      end
+
+      it "returns true" do
+        expect(person.addresses.none?).to be false
+      end
+
+      it "block form iterates across all documents" do
+        expect(person.addresses.none? {|a| a.street == "Upper" }).to be false
+        expect(person.addresses.none? {|a| a.street == "Bond" }).to be true
+      end
+
+      context 'argument form' do
+        it "behaves correctly when given a model instance" do
+          expect(person.addresses.none?(person.addresses.first)).to be false
+        end
+
+        it "behaves correctly when given a non-model instance" do
+          expect(person.addresses.none?(1)).to be true
+        end
+      end
+    end
+
+    context "when documents are not persisted" do
+      before do
+        person.addresses.build(street: "Bond")
+      end
+
+      it "returns true" do
+        expect(person.addresses.none?).to be false
+      end
+
+      it "block form iterates across all documents" do
+        expect(person.addresses.none? {|a| a.street == "Upper" }).to be true
+        expect(person.addresses.none? {|a| a.street == "Bond" }).to be false
+      end
+
+      context 'argument form' do
+        it "behaves correctly when given a model instance" do
+          expect(person.addresses.none?(person.addresses.first)).to be false
+        end
+
+        it "behaves correctly when given a non-model instance" do
+          expect(person.addresses.none?(1)).to be true
+        end
+      end
+    end
+
+    context "when documents are not present" do
+      it "returns false" do
+        expect(person.addresses.none?).to be true
+      end
+
+      it "block form iterates across all documents" do
+        expect(person.addresses.none?(&:a)).to be true
+      end
+
+      context 'argument form' do
+        it "behaves correctly when given nil" do
+          expect(person.addresses.none?(nil)).to be true
+        end
+
+        it "behaves correctly when given a non-model instance" do
+          expect(person.addresses.none?(1)).to be true
+        end
+      end
     end
   end
 
@@ -1441,11 +1651,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when providing multiple attributes" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let!(:addresses) do
-        person.addresses.create([{ street: "Bond" }, { street: "Upper" }])
+        person.addresses.create!([{ street: "Bond" }, { street: "Upper" }])
       end
 
       it "creates multiple documents" do
@@ -1468,11 +1678,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when the relation is not cyclic" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let!(:address) do
-        person.addresses.create(street: "Bond") do |address|
+        person.addresses.create!(street: "Bond") do |address|
           address.state = "CA"
         end
       end
@@ -1516,7 +1726,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       context "when embedding a multi word named document" do
 
         let!(:component) do
-          person.address_components.create(street: "Test")
+          person.address_components.create!(street: "Test")
         end
 
         it "saves the embedded document" do
@@ -1528,11 +1738,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when the relation is cyclic" do
 
       let!(:entry) do
-        Entry.create(title: "hi")
+        Entry.create!(title: "hi")
       end
 
       let!(:child_entry) do
-        entry.child_entries.create(title: "hello")
+        entry.child_entries.create!(title: "hello")
       end
 
       it "creates a new child" do
@@ -1544,7 +1754,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   describe "#create!" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     context "when providing multiple attributes" do
@@ -1703,7 +1913,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   describe "#delete_if" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     context "when the documents are new" do
@@ -1752,11 +1962,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when the documents persisted" do
 
       let!(:address_one) do
-        person.addresses.create(street: "Bond")
+        person.addresses.create!(street: "Bond")
       end
 
       let!(:address_two) do
-        person.addresses.create(street: "Upper")
+        person.addresses.create!(street: "Upper")
       end
 
       context "when a block is provided" do
@@ -1811,7 +2021,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     describe "##{method}" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       context "when the documents are new" do
@@ -1865,11 +2075,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       context "when the documents persisted" do
 
         let!(:address_one) do
-          person.addresses.create(street: "Bond")
+          person.addresses.create!(street: "Bond")
         end
 
         let!(:address_two) do
-          person.addresses.create(street: "Upper")
+          person.addresses.create!(street: "Upper")
         end
 
         context "when conditions are provided" do
@@ -1916,7 +2126,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
         context "when removing and resaving" do
 
           let(:owner) do
-            PetOwner.create(title: "AKC")
+            PetOwner.create!(title: "AKC")
           end
 
           before do
@@ -2012,13 +2222,13 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   describe "#exists?" do
 
     let!(:person) do
-      Person.create
+      Person.create!
     end
 
     context "when documents exist in the database" do
 
       before do
-        person.addresses.create(street: "Bond St")
+        person.addresses.create!(street: "Bond St")
       end
 
       it "returns true" do
@@ -2149,12 +2359,50 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
         end
       end
     end
+
+    context "with block" do
+      let!(:author) do
+        Person.create!(title: 'Person')
+      end
+
+      let!(:video_one) do
+        author.videos.create!(title: 'video one')
+      end
+
+      let!(:video_two) do
+        author.videos.create!(title: 'video two')
+      end
+
+      it "finds one" do
+        expect(
+          author.videos.find do |video|
+            video.title == 'video one'
+          end
+        ).to eq(video_one)
+      end
+
+      it "returns first match of multiple" do
+        expect(
+          author.videos.find do |video|
+            ['video one', 'video two'].include?(video.title)
+          end
+        ).to be_a(Video)
+      end
+
+      it "returns nil when not found" do
+        expect(
+          author.videos.find do |video|
+            video.title == 'non exiting one'
+          end
+        ).to be_nil
+      end
+    end
   end
 
   describe "#find_or_create_by" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:address) do
@@ -2197,11 +2445,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     # context "when the child belongs to another document" do
     #
     #   let(:product) do
-    #     Product.create
+    #     Product.create!
     #   end
     #
     #   let(:purchase) do
-    #     Purchase.create
+    #     Purchase.create!
     #   end
     #
     #   let(:line_item) do
@@ -2220,7 +2468,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   describe "#find_or_create_by!" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:address) do
@@ -2272,11 +2520,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     # context "when the child belongs to another document" do
     #
     #   let(:product) do
-    #     Product.create
+    #     Product.create!
     #   end
     #
     #   let(:purchase) do
-    #     Purchase.create
+    #     Purchase.create!
     #   end
     #
     #   let(:line_item) do
@@ -2394,11 +2642,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   describe "#method_missing" do
 
     let!(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:address_one) do
-      person.addresses.create(
+      person.addresses.create!(
           street: "Market",
           state: "CA",
           services: [ "1", "2" ]
@@ -2406,7 +2654,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     end
 
     let!(:address_two) do
-      person.addresses.create(
+      person.addresses.create!(
           street: "Madison",
           state: "NY",
           services: [ "1", "2" ]
@@ -2554,17 +2802,17 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   describe "#pop" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     context "when no argument is provided" do
 
       let!(:address_one) do
-        person.addresses.create(street: "sonnenallee")
+        person.addresses.create!(street: "sonnenallee")
       end
 
       let!(:address_two) do
-        person.addresses.create(street: "hermannstr")
+        person.addresses.create!(street: "hermannstr")
       end
 
       let!(:popped) do
@@ -2587,11 +2835,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when an integer is provided" do
 
       let!(:address_one) do
-        person.addresses.create(street: "sonnenallee")
+        person.addresses.create!(street: "sonnenallee")
       end
 
       let!(:address_two) do
-        person.addresses.create(street: "hermannstr")
+        person.addresses.create!(street: "hermannstr")
       end
 
       context "when the number is zero" do
@@ -2673,17 +2921,17 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   describe "#shift" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     context "when no argument is provided" do
 
       let!(:address_one) do
-        person.addresses.create(street: "sonnenallee")
+        person.addresses.create!(street: "sonnenallee")
       end
 
       let!(:address_two) do
-        person.addresses.create(street: "hermannstr")
+        person.addresses.create!(street: "hermannstr")
       end
 
       let!(:shifted) do
@@ -2706,11 +2954,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when an integer is provided" do
 
       let!(:address_one) do
-        person.addresses.create(street: "sonnenallee")
+        person.addresses.create!(street: "sonnenallee")
       end
 
       let!(:address_two) do
-        person.addresses.create(street: "hermannstr")
+        person.addresses.create!(street: "hermannstr")
       end
 
       context "when the number is zero" do
@@ -2858,11 +3106,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     describe "##{method}" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       before do
-        person.addresses.create(street: "Upper")
+        person.addresses.create!(street: "Upper")
         person.addresses.build(street: "Bond")
       end
 
@@ -2900,7 +3148,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when there are no documents present" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       it "updates nothing" do
@@ -2911,11 +3159,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when documents are present" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let!(:address) do
-        person.addresses.create(street: "Hobrecht", number: 27)
+        person.addresses.create!(street: "Hobrecht", number: 27)
       end
 
       context "when updating with a where clause" do
@@ -2950,16 +3198,16 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when updating the bottom level" do
 
       let!(:person) do
-        Person.create
+        Person.create!
       end
 
       let!(:address) do
-        person.addresses.create(street: "Joachimstr")
+        person.addresses.create!(street: "Joachimstr")
       end
 
       let!(:location) do
-        address.locations.create(name: "vacation", number: 0)
-        address.locations.create(name: "work", number: 3)
+        address.locations.create!(name: "vacation", number: 0)
+        address.locations.create!(name: "work", number: 3)
       end
 
       context "when updating with replacement of embedded array" do
@@ -2967,7 +3215,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
         context "when updating with a hash" do
 
           before do
-            address.update_attributes(locations: [{ name: "home" }])
+            address.update_attributes!(locations: [{ name: "home" }])
           end
 
           it "updates the attributes" do
@@ -2988,7 +3236,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
 
         before do
           location.number = 7
-          location.save
+          location.save!
         end
 
         let(:updated_location_number) do
@@ -3121,7 +3369,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when creating a persisted tree" do
 
       let(:quiz) do
-        Quiz.create
+        Quiz.create!
       end
 
       let(:page) do
@@ -3186,7 +3434,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       context "when saving the root" do
 
         before do
-          person.save
+          person.save!
         end
 
         it "persists the first level document" do
@@ -3340,11 +3588,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   context "when accessing the parent in a destroy callback" do
 
     let!(:league) do
-      League.create
+      League.create!
     end
 
     let!(:division) do
-      league.divisions.create
+      league.divisions.create!
     end
 
     before do
@@ -3359,15 +3607,15 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   context "when updating the parent with all attributes" do
 
     let!(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:address) do
-      person.addresses.create
+      person.addresses.create!
     end
 
     before do
-      person.update_attributes(person.attributes)
+      person.update_attributes!(person.attributes)
     end
 
     it "does not duplicate the embedded documents" do
@@ -3382,13 +3630,13 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   context "when embedding children named versions" do
 
     let(:acolyte) do
-      Acolyte.create(name: "test")
+      Acolyte.create!(name: "test")
     end
 
     context "when creating a child" do
 
       let(:version) do
-        acolyte.versions.create(number: 1)
+        acolyte.versions.create!(number: 1)
       end
 
       it "allows the operation" do
@@ -3413,7 +3661,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     let!(:account) do
       Account.new(name: "Testing").tap do |acct|
         acct.memberships.build
-        acct.save
+        acct.save!
       end
     end
 
@@ -3425,10 +3673,10 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
 
       before do
         account.name = ""
-        account.save
       end
 
       it "does not lose the parent reference" do
+        expect(account.save).to eq false
         expect(from_db.memberships.first.account).to eq(account)
       end
     end
@@ -3448,15 +3696,15 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   context "when moving an embedded document from one parent to another" do
 
     let!(:person_one) do
-      Person.create
+      Person.create!
     end
 
     let!(:person_two) do
-      Person.create
+      Person.create!
     end
 
     let!(:address) do
-      person_one.addresses.create(street: "Kudamm")
+      person_one.addresses.create!(street: "Kudamm")
     end
 
     before do
@@ -3491,7 +3739,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   context "when the relation has a default scope" do
 
     let!(:person) do
-      Person.create
+      Person.create!
     end
 
     context "when the default scope is a sort" do
@@ -3561,7 +3809,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   context "when indexing the documents" do
 
     let!(:person) do
-      Person.create
+      Person.create!
     end
 
     context "when the documents have a limiting default scope" do
@@ -3610,18 +3858,18 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   context "when the embedded document has an array field" do
 
     let!(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:video) do
-      person.videos.create
+      person.videos.create!
     end
 
     context "when saving the array on a persisted document" do
 
       before do
         video.genres = [ "horror", "scifi" ]
-        video.save
+        video.save!
       end
 
       it "sets the value" do
@@ -3646,7 +3894,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
 
           before do
             loaded_video.genres = [ "comedy" ]
-            loaded_video.save
+            loaded_video.save!
           end
 
           it "sets the new value" do
@@ -3664,15 +3912,15 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   context "when destroying an embedded document" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:address_one) do
-      person.addresses.create(street: "hobrecht")
+      person.addresses.create!(street: "hobrecht")
     end
 
     let!(:address_two) do
-      person.addresses.create(street: "maybachufer")
+      person.addresses.create!(street: "maybachufer")
     end
 
     before do
@@ -3744,7 +3992,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   context "when the association has an order defined" do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let(:message_one) do
@@ -3775,11 +4023,11 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
 
       before do
         person.aliases = [ { name: "A", priority: 3 }, { name: "B", priority: 4 }]
-        person.save
+        person.save!
       end
 
       let!(:person2) do
-        Person.create( aliases: [ { name: "C", priority: 1 }, { name: "D", priority: 2 }])
+        Person.create!( aliases: [ { name: "C", priority: 1 }, { name: "D", priority: 2 }])
       end
 
       it "allows ordering on a key of an embedded document" do
@@ -3814,7 +4062,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   context "when updating multiple levels in one update" do
 
     let!(:person) do
-      Person.create(
+      Person.create!(
           addresses: [
               { locations: [{ name: "home" }]}
           ]
@@ -3828,7 +4076,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       end
 
       before do
-        from_db.update_attributes(
+        from_db.update_attributes!(
             addresses: [
                 { locations: [{ name: "work" }]}
             ]
@@ -3848,15 +4096,15 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   context "when the embedded relation sorts on a boolean" do
 
     let(:circuit) do
-      Circuit.create
+      Circuit.create!
     end
 
     let!(:bus_one) do
-      circuit.buses.create(saturday: true)
+      circuit.buses.create!(saturday: true)
     end
 
     let!(:bus_two) do
-      circuit.buses.create(saturday: false)
+      circuit.buses.create!(saturday: false)
     end
 
     it "orders properly with the boolean" do
@@ -3867,7 +4115,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   context "when batch replacing multiple relations in a single update" do
 
     let(:document) do
-      Person.create
+      Person.create!
     end
 
     let(:person) do
@@ -3875,23 +4123,23 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     end
 
     let!(:symptom_one) do
-      person.symptoms.create
+      person.symptoms.create!
     end
 
     let!(:symptom_two) do
-      person.symptoms.create
+      person.symptoms.create!
     end
 
     let!(:appointment_one) do
-      person.appointments.create
+      person.appointments.create!
     end
 
     let!(:appointment_two) do
-      person.appointments.create
+      person.appointments.create!
     end
 
     before do
-      person.update_attributes(
+      person.update_attributes!(
           appointments: [ appointment_one.as_document, appointment_two.as_document ],
           symptoms: [ symptom_one.as_document, symptom_two.as_document ]
       )
@@ -4224,7 +4472,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when the parent has an after create callback" do
 
       before do
-        server.save
+        server.save!
       end
 
       it "does not push the embedded documents twice" do
@@ -4236,7 +4484,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
   context "when embedded documents are stored without ids" do
 
     let!(:band) do
-      Band.create(name: "Moderat")
+      Band.create!(name: "Moderat")
     end
 
     before do
@@ -4286,7 +4534,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       company = Company.new
       4.times { |i| company.staffs << Staff.new(age: 50 + i)}
       2.times { |i| company.staffs << Staff.new(age: 40)}
-      company.save
+      company.save!
       company.staffs.delete_if {|x| x.age >= 50}
       expect(company.staffs.count).to eq(2)
     end
@@ -4363,7 +4611,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     context "when the parent is persisted" do
 
       let(:zone) do
-        DNS::Zone.create
+        DNS::Zone.create!
       end
 
       let(:soa_1) do
@@ -4401,24 +4649,4 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
       end
     end
   end
-
-  context "when using assign_attributes with an already populated array" do
-    let(:post) { EmmPost.create! }
-
-    before do
-      post.assign_attributes(company_tags: [{id: BSON::ObjectId.new, title: 'a'}],
-        user_tags: [{id: BSON::ObjectId.new, title: 'b'}])
-      post.save!
-      post.reload
-      post.assign_attributes(company_tags: [{id: BSON::ObjectId.new, title: 'c'}],
-        user_tags: [])
-      post.save!
-      post.reload
-    end
-
-    it "has the correct embedded documents" do
-      expect(post.company_tags.length).to eq(1)
-      expect(post.company_tags.first.title).to eq("c")
-    end
-  end
 end