mongoid 6.4.8 → 7.0.0

data/spec/app/models/vertex.rb

@@ -0,0 +1,6 @@
+class Vertex
+  include Mongoid::Document
+
+  has_and_belongs_to_many :parents, inverse_of: :children, class_name: 'Vertex'
+  has_and_belongs_to_many :children, inverse_of: :parents, class_name: 'Vertex'
+end

data/spec/app/models/wiki_page.rb

@@ -9,6 +9,6 @@ class WikiPage
 
   embeds_many :edits, validate: false
   has_many :comments, dependent: :destroy, validate: false
-  has_many :child_pages, class_name: "WikiPage", dependent: :delete, inverse_of: :parent_pages
+  has_many :child_pages, class_name: "WikiPage", dependent: :delete_all, inverse_of: :parent_pages
   belongs_to :parent_pages, class_name: "WikiPage", inverse_of: :child_pages
 end

data/spec/config/mongoid.yml

@@ -32,6 +32,7 @@ test:
     log_level: :warn
     belongs_to_required_by_default: false
     app_name: 'testing'
+    background_indexing: false
 test_with_max_staleness:
   clients:
     default:

data/spec/mongoid/{relations → association}/accessors_spec.rb

@@ -1,6 +1,6 @@
 require "spec_helper"
 
-describe Mongoid::Relations::Accessors do
+describe Mongoid::Association::Accessors do
 
   describe "\#{getter}?" do
 

data/spec/mongoid/{relations → association}/auto_save_spec.rb

@@ -1,11 +1,12 @@
 require "spec_helper"
 
-describe Mongoid::Relations::AutoSave do
+describe Mongoid::Association::Referenced::AutoSave do
 
   describe ".auto_save" do
 
     before(:all) do
-      Person.autosave(Person.relations["account"].merge!(autosave: true))
+      Person.has_many :drugs, validate: false, autosave: true
+      Person.has_one :account, validate: false, autosave: true
     end
 
     after(:all) do
@@ -42,13 +43,8 @@ describe Mongoid::Relations::AutoSave do
 
       context "when the relation has already had the autosave callback added" do
 
-        let(:metadata) do
-          Person.relations["drugs"].merge!(autosave: true)
-        end
-
         before do
-          Person.autosave(metadata)
-          Person.autosave(metadata)
+          Person.has_many :drugs, validate: false, autosave: true
         end
 
         let(:drug) do
@@ -71,6 +67,11 @@ describe Mongoid::Relations::AutoSave do
         context "when saving a new parent document" do
 
           context 'when persistence options are not set on the parent' do
+
+            before do
+              Person.has_many :drugs, validate: false, autosave: true
+            end
+
             before do
               person.drugs << drug
               person.save
@@ -254,10 +255,6 @@ describe Mongoid::Relations::AutoSave do
 
       context "when it has two relations with autosaves" do
 
-        before do
-          Person.autosave(Person.relations["drugs"].merge!(autosave: true))
-        end
-
         let!(:person) do
           Person.create(drugs: [percocet], account: account)
         end
@@ -309,6 +306,57 @@ describe Mongoid::Relations::AutoSave do
           end
         end
       end
+
+      context 'when the autosave should be cascaded' do
+
+        before do
+          class King
+            include Mongoid::Document
+            has_one :peasant, autosave: true
+          end
+
+          class Peasant
+            include Mongoid::Document
+            belongs_to :king
+            has_one :harvest, autosave: true
+          end
+
+          class Harvest
+            include Mongoid::Document
+            field :season, type: String
+            belongs_to :peasant
+          end
+        end
+
+        after do
+          Object.send(:remove_const, :King)
+          Object.send(:remove_const, :Peasant)
+          Object.send(:remove_const, :Harvest)
+        end
+
+        let(:king) do
+          King.create
+        end
+
+        let(:peasant) do
+          Peasant.create
+        end
+
+        let(:harvest) do
+          Harvest.create(season: 'Summer')
+        end
+
+        before do
+          peasant.harvest = harvest
+          king.peasant = peasant
+          harvest.season = 'Fall'
+          king.save
+        end
+
+        it 'cascades the save' do
+          expect(harvest.reload.season).to eq('Fall')
+        end
+      end
     end
   end
 end

data/spec/mongoid/{relations → association}/builders_spec.rb

@@ -1,6 +1,6 @@
 require "spec_helper"
 
-describe Mongoid::Relations::Builders do
+describe Mongoid::Association::Builders do
 
   describe "#build_#\{name}" do
 

data/spec/mongoid/association/constrainable_spec.rb

@@ -0,0 +1,115 @@
+require "spec_helper"
+
+describe Mongoid::Association::Constrainable do
+
+  describe "#convert_to_foreign_key" do
+
+    context "when the id's class stores object ids" do
+
+      before(:all) do
+        Person.field(
+          :_id,
+          type: BSON::ObjectId,
+          pre_processed: true,
+          default: ->{ BSON::ObjectId.new },
+          overwrite: true
+        )
+      end
+
+      let(:constrainable) do
+        Post.belongs_to :person
+      end
+
+      context "when provided an object id" do
+
+        let(:object) do
+          BSON::ObjectId.new
+        end
+
+        it "returns the object id" do
+          expect(constrainable.convert_to_foreign_key(object)).to eq(object)
+        end
+      end
+
+      context "when provided a string" do
+
+        let(:object) do
+          BSON::ObjectId.new
+        end
+
+        it "returns an object id from the string" do
+          expect(constrainable.convert_to_foreign_key(object.to_s)).to eq(object)
+        end
+      end
+    end
+
+    context "when the id's class does not store object ids" do
+
+      let(:constrainable) do
+        Alert.belongs_to :account
+      end
+
+      it "returns the object" do
+        expect(constrainable.convert_to_foreign_key("testing")).to eq("testing")
+      end
+    end
+
+    context 'when the association is polymorphic' do
+
+      let(:constrainable) do
+        Post.relations['posteable']
+      end
+
+      let(:result) do
+        constrainable.convert_to_foreign_key(object)
+      end
+
+      context 'when a BSON::ObjectId is passed' do
+
+        let(:object) do
+          BSON::ObjectId.new
+        end
+
+        it 'returns the object id' do
+          expect(result).to eq(object)
+        end
+      end
+
+      context 'when a string is passed' do
+
+        context 'when the string represents an ObjectId' do
+
+          let(:object) do
+            BSON::ObjectId.new.to_s
+          end
+
+          it 'returns the object id' do
+            expect(result).to eq(BSON::ObjectId.from_string(object))
+          end
+        end
+
+        context 'when the string does not represent an ObjectId' do
+
+          let(:object) do
+            'some-other-string'
+          end
+
+          it 'returns the object' do
+            expect(result).to eq(object)
+          end
+        end
+      end
+
+      context 'when a model object is passed' do
+
+        let(:object) do
+          Post.new
+        end
+
+        it 'returns the id' do
+          expect(result).to eq(object.id)
+        end
+      end
+    end
+  end
+end

data/spec/mongoid/{relations → association}/counter_cache_spec.rb

@@ -1,6 +1,6 @@
 require "spec_helper"
 
-describe Mongoid::Relations::CounterCache do
+describe Mongoid::Association::Referenced::CounterCache do
 
   describe "#reset_counters" do