mongoid 7.0.2 → 8.1.1

data/spec/mongoid/association/embedded/embeds_one_models.rb

@@ -0,0 +1,73 @@
+# frozen_string_literal: true
+
+class EomParent
+  include Mongoid::Document
+  include Mongoid::Timestamps
+
+  embeds_one :child, class_name: 'EomChild'
+
+  field :name, type: String
+end
+
+class EomChild
+  include Mongoid::Document
+
+  embedded_in :parent, class_name: 'EomParent'
+
+  field :a, type: Integer, default: 0
+  field :b, type: Integer, default: 0
+end
+
+# Models with associations with :class_name as a :: prefixed string
+
+class EomCcParent
+  include Mongoid::Document
+
+  embeds_one :child, class_name: '::EomCcChild'
+end
+
+class EomCcChild
+  include Mongoid::Document
+
+  embedded_in :parent, class_name: '::EomCcParent'
+end
+
+# Models referencing other models which should not be loaded unless the
+# respective association is referenced
+
+autoload :EomDnlMissingChild, 'mongoid/association/embedded/embeds_one_dnl_models'
+
+class EomDnlParent
+  include Mongoid::Document
+
+  embeds_one :child, class_name: 'EomDnlChild'
+  embeds_one :missing_child, class_name: 'EomDnlMissingChild'
+end
+
+autoload :EomDnlMissingParent, 'mongoid/association/embedded/embeds_one_dnl_models'
+
+class EomDnlChild
+  include Mongoid::Document
+
+  embedded_in :parent, class_name: 'EomDnlParent'
+  embedded_in :missing_parent, class_name: 'EomDnlMissingParent'
+end
+
+class EomAddress
+  include Mongoid::Document
+
+  field :city, type: String
+
+  embedded_in :addressable, polymorphic: true
+end
+
+# app/models/company.rb
+class EomCompany
+  include Mongoid::Document
+
+  embeds_one :address, class_name: 'EomAddress', as: :addressable
+  accepts_nested_attributes_for :address
+
+  embeds_one :delivery_address, class_name: 'EomAddress', as: :addressable
+  accepts_nested_attributes_for :delivery_address
+end

data/spec/mongoid/association/embedded/embeds_one_query_spec.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+require_relative './embeds_one_models'
+
+describe Mongoid::Association::Embedded::EmbedsOne do
+
+  context 'when projecting with #only' do
+    before do
+      parent = EomParent.new(name: 'foo')
+      parent.child = EomChild.new(a: 1, b: 2)
+      parent.save!
+    end
+
+    let(:parent) do
+      EomParent.where(name: 'foo').only(:name, 'child._id', 'child.a').first
+    end
+
+    it 'populates specified fields only' do
+      expect(parent.child.a).to eq(1)
+      # has a default value specified in the model
+      expect do
+        parent.child.b
+      end.to raise_error(ActiveModel::MissingAttributeError)
+      expect(parent.child.attributes.keys).to eq(['_id', 'a'])
+    end
+  end
+end

data/spec/mongoid/association/embedded/embeds_one_spec.rb

@@ -1,4 +1,7 @@
+# frozen_string_literal: true
+
 require "spec_helper"
+require_relative './embeds_one_models'
 
 describe Mongoid::Association::Embedded::EmbedsOne do
 
@@ -444,6 +447,30 @@ describe Mongoid::Association::Embedded::EmbedsOne do
           it 'returns the :as attribute' do
             expect(association.inverses).to eq([ :containable ])
           end
+
+          context 'when class_name is given and is a :: prefixed string' do
+
+            let(:association) do
+              EomCcParent.relations['child']
+            end
+
+            it 'returns the inverse in an array' do
+              inverses = association.inverses
+              expect(inverses).to eq([:parent])
+            end
+
+            context 'when other associations referencing unloaded classes exist' do
+              let(:association) do
+                EomDnlParent.relations['child']
+              end
+
+              it 'does not load other classes' do
+                inverses = association.inverses
+                expect(inverses).to eq([:parent])
+                expect(Object.const_defined?(:EoDnlMarker)).to be false
+              end
+            end
+          end
         end
       end
     end
@@ -604,6 +631,16 @@ describe Mongoid::Association::Embedded::EmbedsOne do
       it 'returns the class name option' do
         expect(association.relation_class_name).to eq('OtherEmbeddedObject')
       end
+
+      context ':class_name is a :: prefixed string' do
+        let(:association) do
+          EomCcParent.relations['child']
+        end
+
+        it 'returns the :: prefixed string' do
+          expect(association.relation_class_name).to eq('::EomCcChild')
+        end
+      end
     end
 
     context 'when the class_name option is not specified' do
@@ -612,6 +649,17 @@ describe Mongoid::Association::Embedded::EmbedsOne do
         expect(association.relation_class_name).to eq('EmbeddedObject')
       end
     end
+
+    context 'when another association in the model is referencing a third model class' do
+      let(:association) do
+        EomDnlParent.relations['child']
+      end
+
+      it 'does not attempt to load the third class' do
+        expect(association.relation_class_name).to eq('EomDnlChild')
+        expect(Object.const_defined?(:EoDnlMarker)).to be false
+      end
+    end
   end
 
   describe '#klass' do
@@ -905,4 +953,32 @@ describe Mongoid::Association::Embedded::EmbedsOne do
       expect(association.create_relation(owner, target)).to be_a(EmbeddedObject)
     end
   end
-end
+
+  context "when multiple embeds_one associations reference the same class" do
+    let(:acme) { EomCompany.create(address: { city: 'Gotham' }, delivery_address: { city: 'Parcelville' }) }
+
+    context "when the first assignment is modified" do
+      before do
+        acme.update(address: EomAddress.new(city: 'Bigville'))
+        acme.reload
+      end
+
+      it "updates the correct association" do
+        expect(acme.address.city).to eq("Bigville")
+        expect(acme.delivery_address.city).to eq("Parcelville")
+      end
+    end
+
+    context "when the second assignment is modified" do
+      before do
+        acme.update(delivery_address: EomAddress.new(city: 'Bigville'))
+        acme.reload
+      end
+
+      it "updates the correct association" do
+        expect(acme.address.city).to eq("Gotham")
+        expect(acme.delivery_address.city).to eq("Bigville")
+      end
+    end
+  end
+end

data/spec/mongoid/association/macros_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Association::Macros do
@@ -404,7 +406,7 @@ describe Mongoid::Association::Macros do
               let(:default_require) { true }
 
               it 'does not require the association' do
-                expect(relation.save).to be(true)
+                expect(relation.save!).to be(true)
               end
             end
 
@@ -413,7 +415,7 @@ describe Mongoid::Association::Macros do
               let(:default_require) { false }
 
               it 'does not require the association' do
-                expect(relation.save).to be(true)
+                expect(relation.save!).to be(true)
               end
             end
           end
@@ -496,7 +498,7 @@ describe Mongoid::Association::Macros do
                 let(:default_require) { true }
 
                 it 'does not require the association' do
-                  expect(relation.save).to be(true)
+                  expect(relation.save!).to be(true)
                 end
               end
 
@@ -505,7 +507,7 @@ describe Mongoid::Association::Macros do
                 let(:default_require) { true }
 
                 it 'does not require the association' do
-                  expect(relation.save).to be(true)
+                  expect(relation.save!).to be(true)
                 end
               end
             end
@@ -524,7 +526,7 @@ describe Mongoid::Association::Macros do
                 let(:default_require) { true }
 
                 it 'does not require the association' do
-                  expect(relation.save).to be(true)
+                  expect(relation.save!).to be(true)
                 end
               end
 
@@ -533,7 +535,7 @@ describe Mongoid::Association::Macros do
                 let(:default_require) { false }
 
                 it 'does not require the association' do
-                  expect(relation.save).to be(true)
+                  expect(relation.save!).to be(true)
                 end
               end
             end
@@ -556,7 +558,7 @@ describe Mongoid::Association::Macros do
               let(:default_require) { true }
 
               it 'does not require the association' do
-                expect(relation.save).to be(true)
+                expect(relation.save!).to be(true)
               end
             end
 
@@ -565,7 +567,7 @@ describe Mongoid::Association::Macros do
               let(:default_require) { false }
 
               it 'does not require the association' do
-                expect(relation.save).to be(true)
+                expect(relation.save!).to be(true)
               end
             end
           end
@@ -614,7 +616,7 @@ describe Mongoid::Association::Macros do
         let(:default_require) { false }
 
         it 'does not require the association' do
-          expect(relation.save).to be(true)
+          expect(relation.save!).to be(true)
         end
       end
     end

data/spec/mongoid/association/nested/many_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Association::Nested::Many do

data/spec/mongoid/association/nested/one_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Association::Nested::One do
@@ -134,20 +136,24 @@ describe Mongoid::Association::Nested::One do
         { allow_destroy: true }
       end
 
-      let(:builder) do
-        described_class.new(association, {
-          id: name.id,
-          last_name: "Lang",
-          _destroy: true
-        }, options)
-      end
+      %i(id _id).each do |id_field|
+        context "#{id_field}" do
+          let(:builder) do
+            described_class.new(association, {
+              id_field => name.id,
+              last_name: "Lang",
+              _destroy: true
+            }, options)
+          end
 
-      before do
-        builder.build(person)
-      end
+          before do
+            builder.build(person)
+          end
 
-      it "deletes the relation" do
-        expect(person.name).to be_nil
+          it "deletes the relation" do
+            expect(person.name).to be_nil
+          end
+        end
       end
     end
   end

data/spec/mongoid/association/options_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Association::Options do

data/spec/mongoid/association/polymorphic_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe "Polymorphic Associations" do

data/spec/mongoid/association/referenced/belongs_to/binding_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Association::Referenced::BelongsTo::Binding do
@@ -81,7 +83,8 @@ describe Mongoid::Association::Referenced::BelongsTo::Binding do
         end
 
         it "does nothing" do
-          expect(game).to receive(:person=).never
+          expect(game).to receive(:person=).with(person).never
+          expect(game).to receive(:person=).with(nil).once
           binding.bind_one
         end
       end

data/spec/mongoid/association/referenced/belongs_to/buildable_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Association::Referenced::BelongsTo::Buildable do
@@ -12,8 +14,12 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
       association.build(base, object)
     end
 
+    let(:options) do
+      {}
+    end
+
     let(:association) do
-      Post.relations['person']
+      Mongoid::Association::Referenced::BelongsTo.new(Post, :person, options)
     end
 
     context "when provided an id" do
@@ -21,7 +27,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
       context "when the object is an object id" do
 
         let!(:person) do
-          Person.create(_id: object)
+          Person.create!(_id: object, username: 'Bob')
         end
 
         let(:object) do
@@ -29,7 +35,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
         end
 
         before do
-          expect(Person).to receive(:where).with(association.primary_key => object).and_call_original
+          expect_any_instance_of(Mongoid::Criteria).to receive(:where).with(association.primary_key => object).and_call_original
         end
 
         it "sets the document" do
@@ -37,6 +43,41 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
         end
       end
 
+      context "when scope is specified" do
+
+        let!(:person) do
+          Person.create(_id: object, username: 'Bob')
+        end
+
+        let(:object) do
+          BSON::ObjectId.new
+        end
+
+        let(:options) do
+          {
+            scope: -> { where(username: 'Bob') }
+          }
+        end
+
+        context "when document satisfies scope" do
+
+          it "sets the document" do
+            expect(document).to eq(person)
+          end
+        end
+
+        context "when document does not satisfy scope" do
+
+          let!(:person) do
+            Person.create(_id: object, username: 'Bruce')
+          end
+
+          it "returns nil" do
+            expect(document).to eq(nil)
+          end
+        end
+      end
+
       context 'when the id is nil' do
 
         let(:object) do
@@ -55,7 +96,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
       context "when the id does not correspond to a document in the database" do
 
         let!(:person) do
-          Person.create
+          Person.create!
         end
 
         let(:object) do
@@ -63,7 +104,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
         end
 
         before do
-          expect(Person).to receive(:where).with(association.primary_key => object).and_call_original
+          expect_any_instance_of(Mongoid::Criteria).to receive(:where).with(association.primary_key => object).and_call_original
         end
 
         it 'returns nil' do
@@ -74,7 +115,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
       context "when the object is an integer" do
 
         let!(:person) do
-          Person.create(_id: object)
+          Person.create!(_id: object)
         end
 
         let(:object) do
@@ -82,7 +123,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
         end
 
         before do
-          expect(Person).to receive(:where).with(association.primary_key => object).and_call_original
+          expect_any_instance_of(Mongoid::Criteria).to receive(:where).with(association.primary_key => object).and_call_original
         end
 
         it "sets the document" do
@@ -94,7 +135,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
     context "when provided a object" do
 
       let!(:person) do
-        Person.create
+        Person.create!
       end
 
       let(:object) do
@@ -102,7 +143,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
       end
 
       before do
-        expect(Person).not_to receive(:where)
+        expect_any_instance_of(Mongoid::Criteria).not_to receive(:where)
       end
 
       it "returns the object" do
@@ -113,7 +154,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
     context "when the document is persisted" do
 
       let!(:person) do
-        Person.create
+        Person.create!
       end
 
       let(:game) do
@@ -128,16 +169,16 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
     context 'setting an associated document to nil' do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let(:game) do
-        Game.create(person: person)
+        Game.create!(person: person)
       end
 
       before do
         game.person = nil
-        game.save
+        game.save!
       end
 
       it 'sets the person_id to nil' do
@@ -153,11 +194,11 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
   describe '#substitute' do
 
     let(:person) do
-      Person.create
+      Person.create!
     end
 
     let(:game) do
-      Game.create(person: person)
+      Game.create!(person: person)
     end
 
     context 'setting an associated document to nil' do
@@ -178,7 +219,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
     context 'setting an associated document to other doc' do
 
       let(:other_person) do
-        Person.create
+        Person.create!
       end
 
       before do
@@ -194,4 +235,58 @@ describe Mongoid::Association::Referenced::BelongsTo::Buildable do
       end
     end
   end
+
+  context 'when the object is already associated with another object' do
+
+    context "when inverse is has_many" do
+
+      let(:drug1) do
+        Drug.create!
+      end
+
+      let(:drug2) do
+        Drug.create!
+      end
+
+      let(:person) do
+        Person.create!
+      end
+
+      before do
+        drug1.person = person
+        drug2.person = person
+      end
+
+      it 'does not clear the object of its previous association' do
+        expect(drug1.person).to eq(person)
+        expect(drug2.person).to eq(person)
+        expect(person.drugs).to eq([drug1, drug2])
+      end
+    end
+
+    context "when inverse is has_one" do
+
+      let(:account1) do
+        Account.create!(name: "1")
+      end
+
+      let(:account2) do
+        Account.create!(name: "2")
+      end
+
+      let(:person) do
+        Person.create!
+      end
+
+      before do
+        account1.person = person
+        account2.person = person
+      end
+
+      it 'clears the object of its previous association' do
+        expect(account1.person).to be_nil
+        expect(account2.person).to eq(person)
+      end
+    end
+  end
 end