mongoid 7.0.0 → 7.4.0

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
@@ -112,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
@@ -131,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
@@ -207,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
@@ -226,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
@@ -309,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
@@ -330,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
@@ -342,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
@@ -372,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
@@ -406,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
@@ -425,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
@@ -443,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
@@ -504,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
@@ -539,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
@@ -550,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
@@ -569,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
@@ -585,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
@@ -605,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
@@ -678,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
@@ -746,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
@@ -795,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
 
@@ -809,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
@@ -879,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
@@ -909,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
@@ -930,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
@@ -945,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
@@ -1102,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
@@ -1116,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
@@ -1136,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
@@ -1258,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
@@ -1277,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
@@ -1293,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
@@ -1365,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
@@ -1386,17 +1388,262 @@ 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!
+    end
+
+    context "when documents are persisted" do
+      before do
+        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 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
 
   describe "#create" do
@@ -1404,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
@@ -1431,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
@@ -1479,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
@@ -1491,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
@@ -1507,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
@@ -1666,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
@@ -1715,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
@@ -1774,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
@@ -1828,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
@@ -1879,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
@@ -1975,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
@@ -2112,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
@@ -2160,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
@@ -2183,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
@@ -2235,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
@@ -2357,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" ]
@@ -2369,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" ]
@@ -2413,13 +2698,26 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     end
 
     context "when providing a criteria class method" do
+      context "without keyword arguments" do
 
-      let(:addresses) do
-        person.addresses.california
+        let(:addresses) do
+          person.addresses.california
+        end
+
+        it "applies the criteria to the documents" do
+          expect(addresses).to eq([ address_one ])
+        end
       end
 
-      it "applies the criteria to the documents" do
-        expect(addresses).to eq([ address_one ])
+      context "with keyword arguments" do
+
+        let(:addresses) do
+          person.addresses.city_and_state(city: "Sacramento", state: "CA")
+        end
+
+        it "applies the criteria to the documents" do
+          expect(addresses).to eq([])
+        end
       end
     end
 
@@ -2504,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
@@ -2537,11 +2835,30 @@ 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
+
+        let!(:popped) do
+          person.addresses.pop(0)
+        end
+
+        it "returns an empty array" do
+          expect(popped).to eq([])
+        end
+
+        it "does not remove the document from the relation" do
+          expect(person.addresses).to eq([ address_one, address_two ])
+        end
+
+        it "does not persist the pop" do
+          expect(person.reload.addresses).to eq([ address_one, address_two ])
+        end
       end
 
       context "when the number is not larger than the relation" do
@@ -2601,6 +2918,125 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     end
   end
 
+  describe "#shift" do
+
+    let(:person) do
+      Person.create!
+    end
+
+    context "when no argument is provided" do
+
+      let!(:address_one) do
+        person.addresses.create!(street: "sonnenallee")
+      end
+
+      let!(:address_two) do
+        person.addresses.create!(street: "hermannstr")
+      end
+
+      let!(:shifted) do
+        person.addresses.shift
+      end
+
+      it "returns the shifted document" do
+        expect(shifted).to eq(address_one)
+      end
+
+      it "removes the document from the relation" do
+        expect(person.addresses).to eq([ address_two ])
+      end
+
+      it "persists the shift" do
+        expect(person.reload.addresses).to eq([ address_two ])
+      end
+    end
+
+    context "when an integer is provided" do
+
+      let!(:address_one) do
+        person.addresses.create!(street: "sonnenallee")
+      end
+
+      let!(:address_two) do
+        person.addresses.create!(street: "hermannstr")
+      end
+
+      context "when the number is zero" do
+
+        let!(:shifted) do
+          person.addresses.shift(0)
+        end
+
+        it "returns an empty array" do
+          expect(shifted).to eq([])
+        end
+
+        it "does not remove the document from the relation" do
+          expect(person.addresses).to eq([ address_one, address_two ])
+        end
+
+        it "does not persist the shift" do
+          expect(person.reload.addresses).to eq([ address_one, address_two ])
+        end
+      end
+
+      context "when the number is not larger than the relation" do
+
+        let!(:shifted) do
+          person.addresses.shift(2)
+        end
+
+        it "returns the shifted documents" do
+          expect(shifted).to eq([ address_one, address_two ])
+        end
+
+        it "removes the document from the relation" do
+          expect(person.addresses).to be_empty
+        end
+
+        it "persists the shift" do
+          expect(person.reload.addresses).to be_empty
+        end
+      end
+
+      context "when the number is larger than the relation" do
+
+        let!(:shifted) do
+          person.addresses.shift(4)
+        end
+
+        it "returns the shifted documents" do
+          expect(shifted).to eq([ address_one, address_two ])
+        end
+
+        it "removes the document from the relation" do
+          expect(person.addresses).to be_empty
+        end
+
+        it "persists the shift" do
+          expect(person.reload.addresses).to be_empty
+        end
+      end
+    end
+
+    context "when the relation is empty" do
+
+      context "when providing no number" do
+
+        it "returns nil" do
+          expect(person.addresses.shift).to be_nil
+        end
+      end
+
+      context "when providing a number" do
+
+        it "returns nil" do
+          expect(person.addresses.shift(2)).to be_nil
+        end
+      end
+    end
+  end
+
   describe "#scoped" do
 
     let(:person) do
@@ -2670,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
 
@@ -2712,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
@@ -2723,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
@@ -2762,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
@@ -2779,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
@@ -2800,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
@@ -2933,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
@@ -2998,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
@@ -3152,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
@@ -3171,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
@@ -3194,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
@@ -3225,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
 
@@ -3237,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
@@ -3260,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
@@ -3303,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
@@ -3373,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
@@ -3422,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
@@ -3458,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
@@ -3476,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
@@ -3556,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
@@ -3587,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
@@ -3626,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" }]}
           ]
@@ -3640,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" }]}
             ]
@@ -3660,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
@@ -3679,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
@@ -3687,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 ]
       )
@@ -3788,7 +4224,7 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     end
   end
 
-  context "#delete or #clear with before_remove callback" do
+  context "#delete, or #clear, or #pop, or #shift with before_remove callback" do
 
     let(:artist) do
       Artist.new
@@ -3834,35 +4270,81 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
         end
       end
 
-      context "when errors are raised" do
+      describe "#pop" do
 
         before do
-          expect(artist).to receive(:before_remove_song).and_raise
+          artist.songs.pop
         end
 
-        describe "#delete" do
+        it "executes the callback" do
+          artist.songs.pop
+          expect(artist.before_remove_embedded_called).to be true
+        end
+      end
 
-          it "does not remove the document from the relation" do
-            begin; artist.songs.delete(song); rescue; end
-            expect(artist.songs).to eq([ song ])
-          end
+      describe "#shift" do
+
+        before do
+          artist.songs.shift
         end
 
-        describe "#clear" do
+        it "executes the callback" do
+          artist.songs.shift
+          expect(artist.before_remove_embedded_called).to be true
+        end
+      end
+    end
 
-          before do
-            begin; artist.songs.clear; rescue; end
-          end
+    context "when errors are raised" do
 
-          it "removes the documents from the relation" do
-            expect(artist.songs).to eq([ song ])
-          end
+      before do
+        expect(artist).to receive(:before_remove_song).and_raise
+      end
+
+      describe "#delete" do
+
+        it "does not remove the document from the relation" do
+          begin; artist.songs.delete(song); rescue; end
+          expect(artist.songs).to eq([ song ])
+        end
+      end
+
+      describe "#clear" do
+
+        before do
+          begin; artist.songs.clear; rescue; end
+        end
+
+        it "removes the documents from the relation" do
+          expect(artist.songs).to eq([ song ])
+        end
+      end
+
+      describe "#pop" do
+
+        before do
+          begin; artist.songs.pop; rescue; end
+        end
+
+        it "should remove from collection" do
+          expect(artist.songs).to eq([ song ])
+        end
+      end
+
+      describe "#shift" do
+
+        before do
+          begin; artist.songs.shift; rescue; end
+        end
+
+        it "should remove from collection" do
+          expect(artist.songs).to eq([ song ])
         end
       end
     end
   end
 
-  context "#delete or #clear with after_remove callback" do
+  context "#delete, or #clear, or #pop, or #shift with after_remove callback" do
 
     let(:artist) do
       Artist.new
@@ -3899,6 +4381,30 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
           expect(artist.after_remove_embedded_called).to be true
         end
       end
+
+      describe "#pop" do
+
+        before do
+          artist.labels.pop
+        end
+
+        it "executes the callback" do
+          artist.labels.pop
+          expect(artist.after_remove_embedded_called).to be true
+        end
+      end
+
+      describe "#shift" do
+
+        before do
+          artist.labels.shift
+        end
+
+        it "executes the callback" do
+          artist.labels.shift
+          expect(artist.after_remove_embedded_called).to be true
+        end
+      end
     end
 
     context "when errors are raised" do
@@ -3928,6 +4434,28 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
           expect(artist.labels).to be_empty
         end
       end
+
+      describe "#pop" do
+
+        before do
+          begin; artist.labels.pop; rescue; end
+        end
+
+        it "should remove from collection" do
+          expect(artist.labels).to be_empty
+        end
+      end
+
+      describe "#shift" do
+
+        before do
+          begin; artist.labels.shift; rescue; end
+        end
+
+        it "should remove from collection" do
+          expect(artist.labels).to be_empty
+        end
+      end
     end
   end
 
@@ -3944,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
@@ -3956,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
@@ -4006,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
@@ -4083,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