mongoid 7.1.1 → 8.1.0

data/spec/mongoid/interceptable_spec.rb

@@ -1,11 +1,18 @@
 # frozen_string_literal: true
-# encoding: utf-8
 
 require "spec_helper"
 require_relative './interceptable_spec_models'
 
 describe Mongoid::Interceptable do
 
+  before do
+    # The find and initialize callbacks I added were causing failures
+    # because they were causing updates when we were asserting no updates
+    # happened.
+    Label.reset_callbacks(:initialize)
+    Label.reset_callbacks(:find)
+  end
+
   class TestClass
     include Mongoid::Interceptable
 
@@ -78,7 +85,7 @@ describe Mongoid::Interceptable do
   describe ".after_find" do
 
     let!(:player) do
-      Player.create
+      Player.create!
     end
 
     context "when the callback is on a root document" do
@@ -131,7 +138,7 @@ describe Mongoid::Interceptable do
     context "when the callback is on an embedded document" do
 
       let!(:implant) do
-        player.implants.create
+        player.implants.create!
       end
 
       context "when when the document is instantiated" do
@@ -201,7 +208,7 @@ describe Mongoid::Interceptable do
           ]
         })
         book.id = '123'
-        book.save
+        book.save!
         book
       end
 
@@ -213,7 +220,7 @@ describe Mongoid::Interceptable do
         book.pages.each do | page |
           page.notes.destroy_all
           page.notes.new(message: new_message)
-          page.save
+          page.save!
         end
       end
 
@@ -276,7 +283,7 @@ describe Mongoid::Interceptable do
 
       before do
         expect(artist).to receive(:before_create_stub).once.and_return(true)
-        artist.save
+        artist.save!
       end
 
       it "gets saved" do
@@ -321,7 +328,7 @@ describe Mongoid::Interceptable do
         end
 
         it "the save returns true" do
-          expect(artist.save).to be true
+          expect(artist.save!).to be true
         end
       end
 
@@ -345,7 +352,7 @@ describe Mongoid::Interceptable do
     context "when updating" do
 
       let(:artist) do
-        Artist.create(name: "Depeche Mode").tap do |artist|
+        Artist.create!(name: "Depeche Mode").tap do |artist|
           artist.name = "The Mountain Goats"
         end
       end
@@ -361,7 +368,7 @@ describe Mongoid::Interceptable do
         end
 
         it "the save returns true" do
-          expect(artist.save).to be true
+          expect(artist.save!).to be true
         end
       end
 
@@ -386,7 +393,7 @@ describe Mongoid::Interceptable do
   describe ".before_destroy" do
 
     let(:artist) do
-      Artist.create(name: "Depeche Mode")
+      Artist.create!(name: "Depeche Mode")
     end
 
     before do
@@ -431,7 +438,7 @@ describe Mongoid::Interceptable do
       end
 
       let!(:record) do
-        moderat.records.create(name: "Moderat")
+        moderat.records.create!(name: "Moderat")
       end
 
       before do
@@ -545,7 +552,7 @@ describe Mongoid::Interceptable do
 
           it "does not cascade to the child" do
             Band.accepts_nested_attributes_for :records, allow_destroy: true
-            expect(band.update_attributes(attributes)).to be true
+            expect(band.update_attributes!(attributes)).to be true
           end
         end
       end
@@ -573,10 +580,22 @@ describe Mongoid::Interceptable do
         end
 
         context "when saving the root" do
+          context 'with prevent_multiple_calls_of_embedded_callbacks enabled' do
+            config_override :prevent_multiple_calls_of_embedded_callbacks, true
+
+            it "executes the callbacks only once for each document" do
+              expect(note).to receive(:update_saved).once
+              band.save!
+            end
+          end
 
-          it "only executes the callbacks once for each embed" do
-            expect(note).to receive(:update_saved).twice
-            band.save
+          context 'with prevent_multiple_calls_of_embedded_callbacks disabled' do
+            config_override :prevent_multiple_calls_of_embedded_callbacks, false
+
+            it "executes the callbacks once for each ember" do
+              expect(note).to receive(:update_saved).twice
+              band.save!
+            end
           end
         end
       end
@@ -585,7 +604,7 @@ describe Mongoid::Interceptable do
     context "when cascading after initialize" do
 
       let!(:person) do
-        Person.create
+        Person.create!
       end
 
       before do
@@ -624,7 +643,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -635,7 +654,7 @@ describe Mongoid::Interceptable do
           context "when the parent is persisted" do
 
             let(:band) do
-              Band.create(name: "Moderat")
+              Band.create!(name: "Moderat")
             end
 
             let!(:label) do
@@ -643,7 +662,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -655,7 +674,7 @@ describe Mongoid::Interceptable do
         context "when the child is persisted" do
 
           let(:band) do
-            Band.create(name: "Moderat")
+            Band.create!(name: "Moderat")
           end
 
           let!(:label) do
@@ -664,7 +683,7 @@ describe Mongoid::Interceptable do
 
           before do
             label.after_create_called = false
-            band.save
+            band.save!
           end
 
           it "does not execute the callback" do
@@ -688,7 +707,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -699,7 +718,7 @@ describe Mongoid::Interceptable do
           context "when the parent is persisted" do
 
             let(:band) do
-              Band.create(name: "Moderat")
+              Band.create!(name: "Moderat")
             end
 
             let!(:label) do
@@ -707,7 +726,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -719,7 +738,7 @@ describe Mongoid::Interceptable do
         context "when the child is persisted" do
 
           let(:band) do
-            Band.create(name: "Moderat")
+            Band.create!(name: "Moderat")
           end
 
           let!(:label) do
@@ -727,7 +746,7 @@ describe Mongoid::Interceptable do
           end
 
           before do
-            band.save
+            band.save!
           end
 
           it "executes the callback" do
@@ -751,7 +770,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "does not execute the callback" do
@@ -762,7 +781,7 @@ describe Mongoid::Interceptable do
           context "when the parent is persisted" do
 
             let(:band) do
-              Band.create(name: "Moderat")
+              Band.create!(name: "Moderat")
             end
 
             let!(:label) do
@@ -770,7 +789,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "does not execute the callback" do
@@ -782,7 +801,7 @@ describe Mongoid::Interceptable do
         context "when the child is persisted" do
 
           let(:band) do
-            Band.create(name: "Moderat")
+            Band.create!(name: "Moderat")
           end
 
           context "when the child is dirty" do
@@ -793,7 +812,7 @@ describe Mongoid::Interceptable do
 
             before do
               label.name = "Nothing"
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -808,7 +827,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "does not execute the callback" do
@@ -833,7 +852,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -844,7 +863,7 @@ describe Mongoid::Interceptable do
           context "when the parent is persisted" do
 
             let(:band) do
-              Band.create(name: "Moderat")
+              Band.create!(name: "Moderat")
             end
 
             let!(:label) do
@@ -852,7 +871,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -864,7 +883,7 @@ describe Mongoid::Interceptable do
         context "when the child is persisted" do
 
           let(:band) do
-            Band.create(name: "Moderat")
+            Band.create!(name: "Moderat")
           end
 
           let!(:label) do
@@ -872,7 +891,7 @@ describe Mongoid::Interceptable do
           end
 
           before do
-            band.save
+            band.save!
           end
 
           it "executes the callback" do
@@ -896,7 +915,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -907,7 +926,7 @@ describe Mongoid::Interceptable do
           context "when the parent is persisted" do
 
             let(:band) do
-              Band.create(name: "Moderat")
+              Band.create!(name: "Moderat")
             end
 
             let!(:record) do
@@ -915,7 +934,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -927,16 +946,16 @@ describe Mongoid::Interceptable do
         context "when the child is persisted" do
 
           let(:band) do
-            Band.create(name: "Moderat")
+            Band.create!(name: "Moderat")
           end
 
           let!(:record) do
-            band.records.create(name: "Moderat")
+            band.records.create!(name: "Moderat")
           end
 
           before do
             record.before_create_called = false
-            band.save
+            band.save!
           end
 
           it "does not execute the callback" do
@@ -960,7 +979,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -975,7 +994,7 @@ describe Mongoid::Interceptable do
           context "when the parent is persisted" do
 
             let(:band) do
-              Band.create(name: "Moderat")
+              Band.create!(name: "Moderat")
             end
 
             let!(:record) do
@@ -983,7 +1002,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -999,15 +1018,15 @@ describe Mongoid::Interceptable do
         context "when the child is persisted" do
 
           let(:band) do
-            Band.create(name: "Moderat")
+            Band.create!(name: "Moderat")
           end
 
           let!(:record) do
-            band.records.create(name: "Moderat")
+            band.records.create!(name: "Moderat")
           end
 
           before do
-            band.save
+            band.save!
           end
 
           it "executes the callback" do
@@ -1022,7 +1041,7 @@ describe Mongoid::Interceptable do
         context "when the child is created" do
 
           let!(:band) do
-            Band.create
+            Band.create!
           end
 
           let!(:label) do
@@ -1050,7 +1069,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "does not execute the callback" do
@@ -1061,7 +1080,7 @@ describe Mongoid::Interceptable do
           context "when the parent is persisted" do
 
             let(:band) do
-              Band.create(name: "Moderat")
+              Band.create!(name: "Moderat")
             end
 
             let!(:record) do
@@ -1069,7 +1088,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "does not execute the callback" do
@@ -1081,18 +1100,18 @@ describe Mongoid::Interceptable do
         context "when the child is persisted" do
 
           let(:band) do
-            Band.create(name: "Moderat")
+            Band.create!(name: "Moderat")
           end
 
           let!(:record) do
-            band.records.create(name: "Moderat")
+            band.records.create!(name: "Moderat")
           end
 
           context "when the child is dirty" do
 
             before do
               record.name = "Nothing"
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -1107,7 +1126,7 @@ describe Mongoid::Interceptable do
           context "when the child is not dirty" do
 
             before do
-              band.save
+              band.save!
             end
 
             it "does not execute the callback" do
@@ -1132,7 +1151,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -1147,7 +1166,7 @@ describe Mongoid::Interceptable do
           context "when the parent is persisted" do
 
             let(:band) do
-              Band.create(name: "Moderat")
+              Band.create!(name: "Moderat")
             end
 
             let!(:record) do
@@ -1155,7 +1174,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -1170,7 +1189,7 @@ describe Mongoid::Interceptable do
           context 'when the parent is updated' do
 
             let(:band) do
-              Band.create(name: "Moderat")
+              Band.create!(name: "Moderat")
             end
 
             before do
@@ -1190,7 +1209,7 @@ describe Mongoid::Interceptable do
         context 'when the parent is updated' do
 
           let(:band) do
-            Band.create(name: "Moderat")
+            Band.create!(name: "Moderat")
           end
 
           before do
@@ -1209,15 +1228,15 @@ describe Mongoid::Interceptable do
         context "when the child is persisted" do
 
           let(:band) do
-            Band.create(name: "Moderat")
+            Band.create!(name: "Moderat")
           end
 
           let!(:record) do
-            band.records.create(name: "Moderat")
+            band.records.create!(name: "Moderat")
           end
 
           before do
-            band.save
+            band.save!
           end
 
           it "executes the callback" do
@@ -1252,7 +1271,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -1263,7 +1282,7 @@ describe Mongoid::Interceptable do
           context "when the root is persisted" do
 
             let(:band) do
-              Band.create(name: "Moderat")
+              Band.create!(name: "Moderat")
             end
 
             let!(:record) do
@@ -1275,7 +1294,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -1287,20 +1306,20 @@ describe Mongoid::Interceptable do
         context "when the child is persisted" do
 
           let(:band) do
-            Band.create(name: "Moderat")
+            Band.create!(name: "Moderat")
           end
 
           let!(:record) do
-            band.records.create(name: "Moderat")
+            band.records.create!(name: "Moderat")
           end
 
           let!(:track) do
-            record.tracks.create(name: "Berlin")
+            record.tracks.create!(name: "Berlin")
           end
 
           before do
             track.before_create_called = false
-            band.save
+            band.save!
           end
 
           it "does not execute the callback" do
@@ -1328,7 +1347,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             let(:reloaded) do
@@ -1347,7 +1366,7 @@ describe Mongoid::Interceptable do
           context "when the parent is persisted" do
 
             let(:band) do
-              Band.create(name: "Moderat")
+              Band.create!(name: "Moderat")
             end
 
             let!(:record) do
@@ -1359,7 +1378,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             let(:reloaded) do
@@ -1379,19 +1398,19 @@ describe Mongoid::Interceptable do
         context "when the child is persisted" do
 
           let(:band) do
-            Band.create(name: "Moderat")
+            Band.create!(name: "Moderat")
           end
 
           let!(:record) do
-            band.records.create(name: "Moderat")
+            band.records.create!(name: "Moderat")
           end
 
           let!(:track) do
-            record.tracks.create(name: "Berlin")
+            record.tracks.create!(name: "Berlin")
           end
 
           before do
-            band.save
+            band.save!
           end
 
           let(:reloaded) do
@@ -1427,7 +1446,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "does not execute the callback" do
@@ -1438,7 +1457,7 @@ describe Mongoid::Interceptable do
           context "when the parent is persisted" do
 
             let(:band) do
-              Band.create(name: "Moderat")
+              Band.create!(name: "Moderat")
             end
 
             let!(:record) do
@@ -1450,7 +1469,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "does not execute the callback" do
@@ -1462,22 +1481,22 @@ describe Mongoid::Interceptable do
         context "when the child is persisted" do
 
           let(:band) do
-            Band.create(name: "Moderat")
+            Band.create!(name: "Moderat")
           end
 
           let!(:record) do
-            band.records.create(name: "Moderat")
+            band.records.create!(name: "Moderat")
           end
 
           let!(:track) do
-            record.tracks.create(name: "Berlin")
+            record.tracks.create!(name: "Berlin")
           end
 
           context "when the child is dirty" do
 
             before do
               track.name = "Rusty Nails"
-              band.save
+              band.save!
             end
 
             let(:reloaded) do
@@ -1496,7 +1515,7 @@ describe Mongoid::Interceptable do
           context "when the child is not dirty" do
 
             before do
-              band.save
+              band.save!
             end
 
             it "does not execute the callback" do
@@ -1544,7 +1563,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -1555,7 +1574,7 @@ describe Mongoid::Interceptable do
           context "when the parent is persisted" do
 
             let(:band) do
-              Band.create(name: "Moderat")
+              Band.create!(name: "Moderat")
             end
 
             let!(:record) do
@@ -1567,7 +1586,7 @@ describe Mongoid::Interceptable do
             end
 
             before do
-              band.save
+              band.save!
             end
 
             it "executes the callback" do
@@ -1579,19 +1598,19 @@ describe Mongoid::Interceptable do
         context "when the child is persisted" do
 
           let(:band) do
-            Band.create(name: "Moderat")
+            Band.create!(name: "Moderat")
           end
 
           let!(:record) do
-            band.records.create(name: "Moderat")
+            band.records.create!(name: "Moderat")
           end
 
           let!(:track) do
-            record.tracks.create(name: "Berlin")
+            record.tracks.create!(name: "Berlin")
           end
 
           before do
-            band.save
+            band.save!
           end
 
           it "executes the callback" do
@@ -1618,11 +1637,11 @@ describe Mongoid::Interceptable do
     end
 
     before do
-      parent.save
+      parent.save!
     end
 
     it "does not duplicate the child documents" do
-      parent.children.create(position: 1)
+      parent.children.create!(position: 1)
       expect(ParentDoc.find(parent.id).children.size).to eq(1)
     end
   end
@@ -1673,7 +1692,7 @@ describe Mongoid::Interceptable do
     context "when updating a document" do
 
       let(:person) do
-        Person.create.tap do |person|
+        Person.create!.tap do |person|
           person.attributes = {
             mode: :prevent_save,
             title: "Associate",
@@ -1707,8 +1726,8 @@ describe Mongoid::Interceptable do
   context "when loading a model multiple times" do
 
     before do
-      load "spec/app/models/callback_test.rb"
-      load "spec/app/models/callback_test.rb"
+      load File.join(MODELS, "callback_test.rb")
+      load File.join(MODELS, "callback_test.rb")
     end
 
     let(:callback) do
@@ -1739,8 +1758,12 @@ describe Mongoid::Interceptable do
         [InterceptableSpec::CbChild, :after_validation],
         [InterceptableSpec::CbParent, :after_validation],
         [InterceptableSpec::CbParent, :before_save],
+        [InterceptableSpec::CbParent, :around_save_open],
         [InterceptableSpec::CbParent, :before_create],
+        [InterceptableSpec::CbParent, :around_create_open],
+        [InterceptableSpec::CbParent, :around_create_close],
         [InterceptableSpec::CbParent, :after_create],
+        [InterceptableSpec::CbParent, :around_save_close],
         [InterceptableSpec::CbParent, :after_save],
       ]
     end
@@ -1762,26 +1785,601 @@ describe Mongoid::Interceptable do
 
     let(:expected) do
       [
+        [InterceptableSpec::CbCascadedChild, :before_validation],
+        [InterceptableSpec::CbCascadedChild, :after_validation],
         [InterceptableSpec::CbParent, :before_validation],
         [InterceptableSpec::CbCascadedChild, :before_validation],
         [InterceptableSpec::CbCascadedChild, :after_validation],
+
         [InterceptableSpec::CbParent, :after_validation],
         [InterceptableSpec::CbParent, :before_save],
-        [InterceptableSpec::CbCascadedChild, :before_save],
+        [InterceptableSpec::CbParent, :around_save_open],
         [InterceptableSpec::CbParent, :before_create],
+        [InterceptableSpec::CbParent, :around_create_open],
+
+        [InterceptableSpec::CbCascadedChild, :before_save],
+        [InterceptableSpec::CbCascadedChild, :around_save_open],
         [InterceptableSpec::CbCascadedChild, :before_create],
-        [InterceptableSpec::CbParent, :after_create],
+        [InterceptableSpec::CbCascadedChild, :around_create_open],
+
+        [InterceptableSpec::CbCascadedChild, :around_create_close],
         [InterceptableSpec::CbCascadedChild, :after_create],
-        [InterceptableSpec::CbParent, :after_save],
+        [InterceptableSpec::CbCascadedChild, :around_save_close],
         [InterceptableSpec::CbCascadedChild, :after_save],
+
+        [InterceptableSpec::CbParent, :around_create_close],
+        [InterceptableSpec::CbParent, :after_create],
+        [InterceptableSpec::CbParent, :around_save_close],
+        [InterceptableSpec::CbParent, :after_save]
       ]
     end
 
     it 'calls callbacks in the right order' do
-      pending 'MONGOID-3795'
-
       parent.save!
       expect(registry.calls).to eq expected
     end
   end
+
+  context "with associations" do
+    context "has_one" do
+      let(:registry) { InterceptableSpec::CallbackRegistry.new }
+
+      let(:parent) do
+        InterceptableSpec::CbHasOneParent.new(registry).tap do |parent|
+          parent.child = InterceptableSpec::CbHasOneChild.new(registry)
+        end
+      end
+
+      let(:expected) do
+        [
+          [InterceptableSpec::CbHasOneParent, :before_validation],
+          [InterceptableSpec::CbHasOneChild, :before_validation],
+          [InterceptableSpec::CbHasOneChild, :after_validation],
+          [InterceptableSpec::CbHasOneParent, :after_validation],
+          [InterceptableSpec::CbHasOneParent, :before_save],
+
+          [InterceptableSpec::CbHasOneParent, :around_save_open],
+          [InterceptableSpec::CbHasOneParent, :before_create],
+          [InterceptableSpec::CbHasOneParent, :around_create_open],
+
+          [InterceptableSpec::CbHasOneParent, :insert_into_database],
+
+          [InterceptableSpec::CbHasOneChild, :before_validation],
+          [InterceptableSpec::CbHasOneChild, :after_validation],
+          [InterceptableSpec::CbHasOneChild, :before_save],
+          [InterceptableSpec::CbHasOneChild, :around_save_open],
+          [InterceptableSpec::CbHasOneChild, :before_create],
+          [InterceptableSpec::CbHasOneChild, :around_create_open],
+
+          [InterceptableSpec::CbHasOneChild, :around_create_close],
+          [InterceptableSpec::CbHasOneChild, :after_create],
+          [InterceptableSpec::CbHasOneChild, :around_save_close],
+          [InterceptableSpec::CbHasOneChild, :after_save],
+
+          [InterceptableSpec::CbHasOneParent, :around_create_close],
+          [InterceptableSpec::CbHasOneParent, :after_create],
+          [InterceptableSpec::CbHasOneParent, :around_save_close],
+          [InterceptableSpec::CbHasOneParent, :after_save],
+        ]
+      end
+
+      it 'calls callbacks in the right order' do
+        parent.save!
+        expect(registry.calls).to eq expected
+      end
+    end
+
+    context "embeds_one" do
+      let(:registry) { InterceptableSpec::CallbackRegistry.new }
+
+      let(:parent) do
+        InterceptableSpec::CbEmbedsOneParent.new(registry).tap do |parent|
+          parent.child = InterceptableSpec::CbEmbedsOneChild.new(registry)
+        end
+      end
+
+      context "create" do
+        let(:expected) do
+          [
+            [InterceptableSpec::CbEmbedsOneChild, :before_validation],
+            [InterceptableSpec::CbEmbedsOneChild, :after_validation],
+            [InterceptableSpec::CbEmbedsOneParent, :before_validation],
+            [InterceptableSpec::CbEmbedsOneChild, :before_validation],
+            [InterceptableSpec::CbEmbedsOneChild, :after_validation],
+            [InterceptableSpec::CbEmbedsOneParent, :after_validation],
+
+            [InterceptableSpec::CbEmbedsOneParent, :before_save],
+            [InterceptableSpec::CbEmbedsOneParent, :around_save_open],
+            [InterceptableSpec::CbEmbedsOneParent, :before_create],
+            [InterceptableSpec::CbEmbedsOneParent, :around_create_open],
+
+            [InterceptableSpec::CbEmbedsOneChild, :before_save],
+            [InterceptableSpec::CbEmbedsOneChild, :around_save_open],
+            [InterceptableSpec::CbEmbedsOneChild, :before_create],
+            [InterceptableSpec::CbEmbedsOneChild, :around_create_open],
+
+            [InterceptableSpec::CbEmbedsOneParent, :insert_into_database],
+
+            [InterceptableSpec::CbEmbedsOneChild, :around_create_close],
+            [InterceptableSpec::CbEmbedsOneChild, :after_create],
+            [InterceptableSpec::CbEmbedsOneChild, :around_save_close],
+            [InterceptableSpec::CbEmbedsOneChild, :after_save],
+
+            [InterceptableSpec::CbEmbedsOneParent, :around_create_close],
+            [InterceptableSpec::CbEmbedsOneParent, :after_create],
+            [InterceptableSpec::CbEmbedsOneParent, :around_save_close],
+            [InterceptableSpec::CbEmbedsOneParent, :after_save]
+          ]
+        end
+
+        it 'calls callbacks in the right order' do
+          parent.save!
+          expect(registry.calls).to eq expected
+        end
+      end
+
+      context "update" do
+        let(:expected) do
+          [
+            [InterceptableSpec::CbEmbedsOneChild, :before_validation],
+            [InterceptableSpec::CbEmbedsOneChild, :after_validation],
+            [InterceptableSpec::CbEmbedsOneParent, :before_validation],
+            [InterceptableSpec::CbEmbedsOneChild, :before_validation],
+            [InterceptableSpec::CbEmbedsOneChild, :after_validation],
+            [InterceptableSpec::CbEmbedsOneParent, :after_validation],
+
+            [InterceptableSpec::CbEmbedsOneParent, :before_save],
+            [InterceptableSpec::CbEmbedsOneParent, :around_save_open],
+            [InterceptableSpec::CbEmbedsOneParent, :before_update],
+            [InterceptableSpec::CbEmbedsOneParent, :around_update_open],
+
+            [InterceptableSpec::CbEmbedsOneChild, :before_save],
+            [InterceptableSpec::CbEmbedsOneChild, :around_save_open],
+            [InterceptableSpec::CbEmbedsOneChild, :before_update],
+            [InterceptableSpec::CbEmbedsOneChild, :around_update_open],
+
+            [InterceptableSpec::CbEmbedsOneChild, :around_update_close],
+            [InterceptableSpec::CbEmbedsOneChild, :after_update],
+            [InterceptableSpec::CbEmbedsOneChild, :around_save_close],
+            [InterceptableSpec::CbEmbedsOneChild, :after_save],
+
+            [InterceptableSpec::CbEmbedsOneParent, :around_update_close],
+            [InterceptableSpec::CbEmbedsOneParent, :after_update],
+            [InterceptableSpec::CbEmbedsOneParent, :around_save_close],
+            [InterceptableSpec::CbEmbedsOneParent, :after_save]
+          ]
+        end
+
+        it 'calls callbacks in the right order' do
+          parent.callback_registry = nil
+          parent.child.callback_registry = nil
+          parent.save!
+
+          parent.callback_registry = registry
+          parent.child.callback_registry = registry
+          parent.name = "name"
+          parent.child.age = 10
+
+          parent.save!
+          expect(registry.calls).to eq expected
+        end
+      end
+    end
+
+    context "has_many" do
+      let(:registry) { InterceptableSpec::CallbackRegistry.new }
+
+      let(:parent) do
+        InterceptableSpec::CbHasManyParent.new(registry).tap do |parent|
+          parent.children = [
+            InterceptableSpec::CbHasManyChild.new(registry),
+            InterceptableSpec::CbHasManyChild.new(registry)
+          ]
+        end
+      end
+
+      let(:expected) do
+        [
+          [InterceptableSpec::CbHasManyParent, :before_validation],
+          [InterceptableSpec::CbHasManyChild, :before_validation],
+          [InterceptableSpec::CbHasManyChild, :after_validation],
+          [InterceptableSpec::CbHasManyChild, :before_validation],
+          [InterceptableSpec::CbHasManyChild, :after_validation],
+          [InterceptableSpec::CbHasManyParent, :after_validation],
+
+          [InterceptableSpec::CbHasManyParent, :before_save],
+          [InterceptableSpec::CbHasManyParent, :around_save_open],
+          [InterceptableSpec::CbHasManyParent, :before_create],
+          [InterceptableSpec::CbHasManyParent, :around_create_open],
+
+          [InterceptableSpec::CbHasManyParent, :insert_into_database],
+
+          [InterceptableSpec::CbHasManyChild, :before_validation],
+          [InterceptableSpec::CbHasManyChild, :after_validation],
+          [InterceptableSpec::CbHasManyChild, :before_save],
+          [InterceptableSpec::CbHasManyChild, :around_save_open],
+          [InterceptableSpec::CbHasManyChild, :before_create],
+          [InterceptableSpec::CbHasManyChild, :around_create_open],
+          [InterceptableSpec::CbHasManyChild, :around_create_close],
+          [InterceptableSpec::CbHasManyChild, :after_create],
+          [InterceptableSpec::CbHasManyChild, :around_save_close],
+          [InterceptableSpec::CbHasManyChild, :after_save],
+
+          [InterceptableSpec::CbHasManyChild, :before_validation],
+          [InterceptableSpec::CbHasManyChild, :after_validation],
+          [InterceptableSpec::CbHasManyChild, :before_save],
+          [InterceptableSpec::CbHasManyChild, :around_save_open],
+          [InterceptableSpec::CbHasManyChild, :before_create],
+          [InterceptableSpec::CbHasManyChild, :around_create_open],
+          [InterceptableSpec::CbHasManyChild, :around_create_close],
+          [InterceptableSpec::CbHasManyChild, :after_create],
+          [InterceptableSpec::CbHasManyChild, :around_save_close],
+          [InterceptableSpec::CbHasManyChild, :after_save],
+
+          [InterceptableSpec::CbHasManyParent, :around_create_close],
+          [InterceptableSpec::CbHasManyParent, :after_create],
+          [InterceptableSpec::CbHasManyParent, :around_save_close],
+          [InterceptableSpec::CbHasManyParent, :after_save]
+        ]
+      end
+
+      it 'calls callbacks in the right order' do
+        parent.save!
+        expect(registry.calls).to eq expected
+      end
+    end
+
+    context "embeds_many" do
+      let(:registry) { InterceptableSpec::CallbackRegistry.new }
+
+      let(:parent) do
+        InterceptableSpec::CbEmbedsManyParent.new(registry).tap do |parent|
+          parent.children = [
+            InterceptableSpec::CbEmbedsManyChild.new(registry),
+            InterceptableSpec::CbEmbedsManyChild.new(registry),
+          ]
+        end
+      end
+
+      let(:expected) do
+        [
+          [InterceptableSpec::CbEmbedsManyChild, :before_validation],
+          [InterceptableSpec::CbEmbedsManyChild, :after_validation],
+          [InterceptableSpec::CbEmbedsManyChild, :before_validation],
+          [InterceptableSpec::CbEmbedsManyChild, :after_validation],
+          [InterceptableSpec::CbEmbedsManyParent, :before_validation],
+          [InterceptableSpec::CbEmbedsManyChild, :before_validation],
+          [InterceptableSpec::CbEmbedsManyChild, :after_validation],
+          [InterceptableSpec::CbEmbedsManyChild, :before_validation],
+          [InterceptableSpec::CbEmbedsManyChild, :after_validation],
+          [InterceptableSpec::CbEmbedsManyParent, :after_validation],
+
+          [InterceptableSpec::CbEmbedsManyParent, :before_save],
+          [InterceptableSpec::CbEmbedsManyParent, :around_save_open],
+          [InterceptableSpec::CbEmbedsManyParent, :before_create],
+          [InterceptableSpec::CbEmbedsManyParent, :around_create_open],
+
+          [InterceptableSpec::CbEmbedsManyChild, :before_save],
+          [InterceptableSpec::CbEmbedsManyChild, :around_save_open],
+          [InterceptableSpec::CbEmbedsManyChild, :before_save],
+
+          [InterceptableSpec::CbEmbedsManyChild, :around_save_open],
+          [InterceptableSpec::CbEmbedsManyChild, :before_create],
+          [InterceptableSpec::CbEmbedsManyChild, :around_create_open],
+
+          [InterceptableSpec::CbEmbedsManyChild, :before_create],
+          [InterceptableSpec::CbEmbedsManyChild, :around_create_open],
+
+          [InterceptableSpec::CbEmbedsManyParent, :insert_into_database],
+
+          [InterceptableSpec::CbEmbedsManyChild, :around_create_close],
+          [InterceptableSpec::CbEmbedsManyChild, :after_create],
+
+          [InterceptableSpec::CbEmbedsManyChild, :around_create_close],
+          [InterceptableSpec::CbEmbedsManyChild, :after_create],
+
+          [InterceptableSpec::CbEmbedsManyChild, :around_save_close],
+          [InterceptableSpec::CbEmbedsManyChild, :after_save],
+
+          [InterceptableSpec::CbEmbedsManyChild, :around_save_close],
+          [InterceptableSpec::CbEmbedsManyChild, :after_save],
+
+          [InterceptableSpec::CbEmbedsManyParent, :around_create_close],
+          [InterceptableSpec::CbEmbedsManyParent, :after_create],
+          [InterceptableSpec::CbEmbedsManyParent, :around_save_close],
+          [InterceptableSpec::CbEmbedsManyParent, :after_save]
+        ]
+      end
+
+      it 'calls callbacks in the right order' do
+        parent.save!
+        expect(registry.calls).to eq expected
+      end
+    end
+  end
+
+  context "when accessing parent document from callbacks" do
+    shared_examples 'accesses the correct parent' do
+      it "accesses the correct parent in after_find" do
+        expect(from_db.after_find_player).to eq(player._id)
+      end
+
+      it "accesses the correct parent in after_initialize" do
+        expect(from_db.after_initialize_player).to eq(player._id)
+      end
+
+      it "accesses the correct parent in default" do
+        expect(from_db.after_default_player).to eq(player._id)
+      end
+
+      it "accesses the correct parent in unpersisted after_initialize" do
+        expect(unpersisted.after_initialize_player).to eq(player._id)
+      end
+    end
+
+    context "when using create methods" do
+
+      context "when the child is an embeds_many association" do
+        let!(:player) do
+          Player.create!.tap do |player|
+            player.implants.create!
+          end
+        end
+
+        let(:unpersisted) { player.implants.first }
+
+        before do
+          # The default is originally set when creating this document, and it is
+          # subsequently persisted to the database. Therefore when we retrieve
+          # this document from the database, this field is already set, and
+          # the default Proc is not called. This unset is needed to allow the
+          # default Proc to be called when the document is retrieved from the
+          # database.
+          Player.find(player.id).implants.first.unset(:after_default_player)
+        end
+
+        let(:from_db) do
+          Player.find(player.id).implants.first
+        end
+
+        include_examples 'accesses the correct parent'
+      end
+
+      context "when the child is an embeds_one association" do
+        let!(:player) do
+          Player.create!.tap do |player|
+            player.create_augmentation
+          end
+        end
+
+        let(:unpersisted) { player.augmentation }
+
+        before do
+          Player.find(player.id).augmentation.unset(:after_default_player)
+        end
+
+        let(:from_db) do
+          Player.find(player.id).augmentation
+        end
+
+        include_examples 'accesses the correct parent'
+      end
+
+      context "when the child is a has_many association" do
+        let!(:player) do
+          Player.create!.tap do |player|
+            player.weapons.create!
+          end
+        end
+
+        let(:unpersisted) { player.weapons.first }
+
+        before do
+          Player.find(player.id).weapons.first.unset(:after_default_player)
+        end
+
+        let(:from_db) do
+          Player.find(player.id).weapons.first
+        end
+
+        include_examples 'accesses the correct parent'
+      end
+
+      context "when the child is a has_one association" do
+        let!(:player) do
+          Player.create!.tap do |player|
+            player.create_powerup
+            player.save!
+          end
+        end
+
+        let(:unpersisted) { player.powerup }
+
+        before do
+          Player.find(player.id).powerup.unset(:after_default_player)
+        end
+
+        let(:from_db) do
+          Player.find(player.id).powerup
+        end
+
+        include_examples 'accesses the correct parent'
+      end
+
+      context "when the child is a has_and_belongs_to_many association" do
+        let!(:player) do
+          Player.create!.tap do |player|
+            player.shields.create!
+          end
+        end
+
+        let(:unpersisted) { player.shields.first }
+
+        before do
+          Player.find(player.id).shields.unset(:after_default_player)
+        end
+
+        let(:from_db) do
+          Player.find(player.id).shields.first
+        end
+
+        include_examples 'accesses the correct parent'
+      end
+    end
+
+    context "when using build methods" do
+
+      context "when the child is an embeds_many association" do
+        let!(:player) do
+          Player.create!.tap do |player|
+            player.implants.build
+            player.implants.first.save!
+          end
+        end
+
+        let(:unpersisted) { player.implants.first }
+
+        before do
+          Player.find(player.id).implants.first.unset(:after_default_player)
+        end
+
+        let(:from_db) do
+          Player.find(player.id).implants.first
+        end
+
+        include_examples 'accesses the correct parent'
+      end
+
+      context "when the child is an embeds_one association" do
+        let!(:player) do
+          Player.create!.tap do |player|
+            player.build_augmentation
+            player.save!
+          end
+        end
+
+        let(:unpersisted) { player.augmentation }
+
+        before do
+          Player.find(player.id).augmentation.unset(:after_default_player)
+        end
+
+        let(:from_db) do
+          Player.find(player.id).augmentation
+        end
+
+        include_examples 'accesses the correct parent'
+      end
+
+      context "when the child is a has_many association" do
+        let!(:player) do
+          Player.create!.tap do |player|
+            player.weapons.build
+            player.weapons.first.save!
+          end
+        end
+
+        let(:unpersisted) { player.weapons.first }
+
+        before do
+          Player.find(player.id).weapons.first.unset(:after_default_player)
+        end
+
+        let(:from_db) do
+          Player.find(player.id).weapons.first
+        end
+
+        include_examples 'accesses the correct parent'
+      end
+
+      context "when the child is a has_one association" do
+        let!(:player) do
+          Player.create!.tap do |player|
+            player.build_powerup
+            player.powerup.save!
+          end
+        end
+
+        let(:unpersisted) { player.powerup }
+
+        before do
+          Player.find(player.id).powerup.unset(:after_default_player)
+        end
+
+        let(:from_db) do
+          Player.find(player.id).powerup
+        end
+
+        include_examples 'accesses the correct parent'
+      end
+
+      context "when the child is a has_and_belongs_to_many association" do
+        let!(:player) do
+          Player.create!.tap do |player|
+            player.shields.build
+            player.shields.first.save!
+          end
+        end
+
+        let(:unpersisted) { player.shields.first }
+
+        before do
+          Player.find(player.id).shields.unset(:after_default_player)
+        end
+
+        let(:from_db) do
+          Player.find(player.id).shields.first
+        end
+
+        include_examples 'accesses the correct parent'
+      end
+    end
+  end
+
+  context "when accessing associations in defaults" do
+    context "when not using autobuilding" do
+      let(:band) { InterceptableBand.create(name: "Molejo") }
+      let(:song) { band.songs.create(name: "Cilada") }
+
+      it "assigns the default correctly" do
+        expect(song.band_name).to eq("Molejo")
+      end
+    end
+
+    context "when using autobuilding" do
+      before do
+        InterceptablePlane.create!.tap do |plane|
+          plane.wings.create!
+        end
+      end
+
+      let(:plane) { InterceptablePlane.first }
+      let(:wing) { InterceptableWing.first }
+      let(:engine) { wing.engine }
+
+      it "sets the defaults correctly" do
+        expect(wing._id).to eq("hello-wing")
+        expect(wing.p_id).to eq(plane._id.to_s)
+        expect(wing.e_id).to eq(engine._id.to_s)
+        expect(engine._id).to eq("hello-engine-#{wing.id}")
+      end
+    end
+  end
+
+  # This case is rather niche. The _ids method used to use the `.only` method
+  # to get only the _ids for an association, which was causing a
+  # MissingAttributeError to be raised when accessing another association. This
+  # was fixed by using `.pluck` over `.only`. Look at MONGOID-5306 for a more
+  # detailed explanation.
+  context "when accessing _ids in validate and access an association in after_initialize" do
+    it "doesn't raise a MissingAttributeError" do
+      company = InterceptableCompany.create!
+      shop = InterceptableShop.create!(company: company)
+      user = InterceptableUser.new
+      user.company = company
+      expect do
+        user.save!
+      end.to_not raise_error(ActiveModel::MissingAttributeError)
+    end
+  end
 end