mongoid 7.0.0 → 7.4.0

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

@@ -0,0 +1,69 @@
+# frozen_string_literal: true
+
+class EmmCongress
+  include Mongoid::Document
+  include Mongoid::Timestamps
+
+  embeds_many :legislators, class_name: 'EmmLegislator'
+
+  field :name, type: String
+end
+
+class EmmLegislator
+  include Mongoid::Document
+
+  embedded_in :congress, class_name: 'EmmCongress'
+
+  field :a, type: Integer, default: 0
+  field :b, type: Integer, default: 0
+end
+
+# Models with associations with :class_name as a :: prefixed string
+
+class EmmCcCongress
+  include Mongoid::Document
+
+  embeds_many :legislators, class_name: '::EmmCcLegislator'
+
+  field :name, type: String
+end
+
+class EmmCcLegislator
+  include Mongoid::Document
+
+  embedded_in :congress, class_name: '::EmmCcCongress'
+
+  field :a, type: Integer, default: 0
+  field :b, type: Integer, default: 0
+end
+
+class EmmManufactory
+  include Mongoid::Document
+
+  embeds_many :products, order: :id.desc, class_name: 'EmmProduct'
+end
+
+
+class EmmProduct
+  include Mongoid::Document
+
+  embedded_in :manufactory, class_name: 'EmmManufactory'
+
+  field :name, type: String
+end
+
+class EmmInner
+  include Mongoid::Document
+
+  embeds_many :friends, :class_name => self.name, :cyclic => true
+  embedded_in :parent, :class_name => self.name, :cyclic => true
+
+  field :level, :type => Integer
+end
+
+class EmmOuter
+  include Mongoid::Document
+  embeds_many :inners, class_name: 'EmmInner'
+
+  field :level, :type => Integer
+end

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

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+require_relative './embeds_many_models'
+
+describe Mongoid::Association::Embedded::EmbedsMany do
+
+  context 'when projecting with #only' do
+    before do
+      congress = EmmCongress.new(name: 'foo')
+      congress.legislators << EmmLegislator.new(a: 1, b: 2)
+      congress.save!
+    end
+
+    let(:congress) do
+      EmmCongress.where(name: 'foo').only(:name, 'legislators._id', 'legislators.a').first
+    end
+
+    let(:legislator) { congress.legislators.first }
+
+    it 'populates specified fields only' do
+      expect(legislator.a).to eq(1)
+      # has a default value specified in the model
+      expect do
+        legislator.b
+      end.to raise_error(ActiveModel::MissingAttributeError)
+      expect(legislator.attributes.keys).to eq(['_id', 'a'])
+    end
+
+    context 'when using only with $' do
+      before do
+        Patient.destroy_all
+        Patient.create!(
+          title: 'Steve',
+          addresses: [
+            Address.new(number: '123'),
+            Address.new(number: '456'),
+          ],
+        )
+      end
+
+      let(:patient) do
+        Patient.where('addresses.number' => {'$gt' => 100}).only('addresses.$').first
+      end
+
+      it 'loads embedded association' do
+        expect(patient.addresses.first.number).to eq(123)
+      end
+    end
+  end
+end

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Association::Embedded::EmbedsMany do
@@ -646,6 +648,16 @@ describe Mongoid::Association::Embedded::EmbedsMany do
       it 'returns the class name option' do
         expect(association.klass).to eq(_class)
       end
+
+      context 'when the class name is prefixed with ::' do
+        let(:options) do
+          { class_name: '::OtherEmbeddedObject' }
+        end
+
+        it 'returns the class name option' do
+          expect(association.klass).to eq(_class)
+        end
+      end
     end
 
     context 'when the class_name option is not specified' do
@@ -849,4 +861,4 @@ describe Mongoid::Association::Embedded::EmbedsMany do
       expect(association.path(double( :_parent => true))).to be_a(Mongoid::Atomic::Paths::Embedded::Many)
     end
   end
-end
+end

data/spec/mongoid/association/embedded/embeds_one/binding_spec.rb

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

data/spec/mongoid/association/embedded/embeds_one/buildable_spec.rb

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

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
@@ -85,7 +87,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
       context "when the parent is not a new record" do
 
         let(:person) do
-          Person.create
+          Person.create!
         end
 
         let(:name) do
@@ -171,7 +173,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           context "when replacing an existing document" do
 
             let(:pet_owner) do
-              PetOwner.create
+              PetOwner.create!
             end
 
             let(:pet_one) do
@@ -204,7 +206,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
         context 'when the original document does not need to be unset because it will be replaced by the $set' do
 
           let!(:pet_owner) do
-            PetOwner.create(pet: pet_one)
+            PetOwner.create!(pet: pet_one)
           end
 
           let(:pet_one) do
@@ -277,7 +279,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
       context "when the parent is not a new record" do
 
         let(:parent_shelf) do
-          Shelf.create
+          Shelf.create!
         end
 
         let(:child_shelf) do
@@ -309,7 +311,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
     context "when setting a new document multiple times in a row" do
 
       let(:parent) do
-        Parent.create
+        Parent.create!
       end
 
       before do
@@ -370,7 +372,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
       context "when the parent is persisted" do
 
         let(:person) do
-          Person.create
+          Person.create!
         end
 
         let(:name) do
@@ -459,7 +461,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
       context "when the documents are not new records" do
 
         let(:parent_shelf) do
-          Shelf.create
+          Shelf.create!
         end
 
         let(:child_shelf) do
@@ -608,7 +610,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
       context "when the parent is not a new record" do
 
         let(:person) do
-          Person.create
+          Person.create!
         end
 
         let!(:name) do
@@ -653,7 +655,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
       context "when the parent is not a new record" do
 
         let(:parent_shelf) do
-          Shelf.create
+          Shelf.create!
         end
 
         let!(:child_shelf) do
@@ -755,7 +757,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
       context "when the parent is not a new record" do
 
         let(:person) do
-          Person.create
+          Person.create!
         end
 
         let!(:name) do
@@ -776,7 +778,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
       context "when the embedded document inherits its relationship" do
 
         let(:courier_job) do
-          CourierJob.create
+          CourierJob.create!
         end
 
         let(:old_child) do
@@ -845,7 +847,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
   context "when the embedded document has an array field" do
 
     let!(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:name) do
@@ -864,7 +866,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
 
       before do
         from_db.aliases = [ "Syd", "Sydney" ]
-        from_db.save
+        from_db.save!
       end
 
       it "sets the values of the array" do
@@ -880,7 +882,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
   context "when embedding a many under a one" do
 
     let!(:person) do
-      Person.create
+      Person.create!
     end
 
     before do
@@ -906,7 +908,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
         context "when saving the root" do
 
           before do
-            from_db.save
+            from_db.save!
           end
 
           it "persists the new document on the first save" do
@@ -920,15 +922,15 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
   context "when embedding a one under a many" 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: "kreuzberg")
+      person.addresses.create!(street: "kreuzberg")
     end
 
     context "when a parent was removed outside of mongoid" do
@@ -963,7 +965,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::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
@@ -1010,9 +1012,9 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
   context "when parent validation of child is set to false" do
 
     let(:building) do
-      building = Building.create
+      building = Building.create!
       building.building_address = BuildingAddress.new
-      building.save
+      building.save!
       building.reload
     end
 

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

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+# This file is added as an autoload target, but should never be actually
+# processed.
+
+# Marker class - if this class exists, this file was attempted to be required.
+class EoDnlMarker
+end

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

@@ -0,0 +1,54 @@
+# 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

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,4 @@ describe Mongoid::Association::Embedded::EmbedsOne do
       expect(association.create_relation(owner, target)).to be_a(EmbeddedObject)
     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