mongoid 9.0.11 → 9.1.0

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

@@ -1,11 +1,9 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "spec_helper"
-require_relative './embeds_one_models'
+require 'spec_helper'
+require_relative 'embeds_one_models'
 
 describe Mongoid::Association::Embedded::EmbedsOne do
-
   before do
     class Container; include Mongoid::Document; end
     class EmbeddedObject; include Mongoid::Document; end
@@ -29,14 +27,13 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   let(:options) do
-    { }
+    {}
   end
 
   describe '#relation_complements' do
-
     let(:expected_complements) do
       [
-        Mongoid::Association::Embedded::EmbeddedIn,
+        Mongoid::Association::Embedded::EmbeddedIn
       ]
     end
 
@@ -46,7 +43,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#setup!' do
-
     it 'sets up a getter for the relation' do
       expect(Mongoid::Association::Accessors).to receive(:define_getter!).with(association)
       association.setup!
@@ -73,10 +69,9 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when the :validate option is true' do
-
       let(:options) do
         {
-            validate: true
+          validate: true
         }
       end
 
@@ -93,10 +88,9 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when the :validate option is false' do
-
       let(:options) do
         {
-            validate: false
+          validate: false
         }
       end
 
@@ -107,7 +101,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when the :validate option is not provided' do
-
       let(:association) do
         # Note that it is necessary to create the association directly, otherwise the
         # setup! method will be called by the :embeds_one macro
@@ -122,9 +115,7 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#type' do
-
     context 'when polymorphic' do
-
       let(:options) do
         { as: :containable }
       end
@@ -135,7 +126,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when not polymorphic' do
-
       it 'returns nil' do
         expect(association.type).to be_nil
       end
@@ -143,9 +133,7 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#inverse_type' do
-
     context 'when polymorphic' do
-
       let(:options) do
         { as: :containable }
       end
@@ -156,7 +144,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when not polymorphic' do
-
       it 'returns nil' do
         expect(association.inverse_type).to be_nil
       end
@@ -164,9 +151,7 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#inverse_type_setter' do
-
     context 'when polymorphic' do
-
       let(:options) do
         { as: :containable }
       end
@@ -177,7 +162,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when not polymorphic' do
-
       it 'returns nil' do
         expect(association.inverse_type).to be_nil
       end
@@ -185,58 +169,49 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#embedded?' do
-
     it 'returns true' do
       expect(association.embedded?).to be(true)
     end
   end
 
   describe '#primary_key' do
-
     it 'returns nil' do
       expect(association.primary_key).to be_nil
     end
   end
 
   describe '#indexed?' do
-
     it 'returns false' do
       expect(association.indexed?).to be(false)
     end
   end
 
   describe '#relation' do
-
     it 'returns Mongoid::Association::Embedded::EmbedsOne::Proxy' do
       expect(association.relation).to be(Mongoid::Association::Embedded::EmbedsOne::Proxy)
     end
   end
 
   describe '#validation_default' do
-
     it 'returns true' do
       expect(association.validation_default).to be(true)
     end
   end
 
   describe '#name' do
-
     it 'returns the name of the relation' do
       expect(association.name).to be(name)
     end
   end
 
   describe '#options' do
-
     it 'returns the options' do
       expect(association.options).to be(options)
     end
   end
 
   describe '#cyclic?' do
-
     context 'when :cyclic is specified in the options' do
-
       let(:options) do
         { cyclic: true }
       end
@@ -247,7 +222,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when :cyclic is not specified in the options' do
-
       it 'returns false' do
         expect(association.cyclic?).to be(false)
       end
@@ -255,13 +229,10 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#merge!' do
-
   end
 
   describe '#store_as' do
-
     context 'when :store_as is specified in the options' do
-
       let(:options) do
         {
           store_as: :baby_kangaroo
@@ -274,7 +245,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when :store_as is not specified in the options' do
-
       it 'returns name as a String' do
         expect(association.store_as).to eq(name.to_s)
       end
@@ -282,23 +252,19 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#touchable?' do
-
     it 'return false' do
       expect(association.send(:touchable?)).to be(false)
     end
   end
 
   describe '#order' do
-
     it 'returns nil' do
       expect(association.order).to be_nil
     end
   end
 
   describe '#as' do
-
     context 'when :as is specified in the options' do
-
       let(:options) do
         {
           as: :containable
@@ -311,7 +277,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when :as is not specified in the options' do
-
       it 'returns nil' do
         expect(association.as).to be_nil
       end
@@ -319,9 +284,7 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#polymorphic?' do
-
     context 'when :as is specified in the options' do
-
       let(:options) do
         {
           as: :containable
@@ -331,11 +294,9 @@ describe Mongoid::Association::Embedded::EmbedsOne do
       it 'returns true' do
         expect(association.polymorphic?).to be(true)
       end
-
     end
 
     context 'when :as is not specified in the options' do
-
       it 'returns false' do
         expect(association.polymorphic?).to be(false)
       end
@@ -343,20 +304,17 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#type_setter' do
-
     context 'when polymorphic' do
-
       let(:options) do
         { as: :containable }
       end
 
       it 'returns the type followed by = as a String' do
-        expect(association.type_setter).to eq("containable_type=")
+        expect(association.type_setter).to eq('containable_type=')
       end
     end
 
     context 'when not polymorphic' do
-
       it 'returns nil' do
         expect(association.type).to be_nil
       end
@@ -364,30 +322,25 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#dependent' do
-
     it 'returns nil' do
       expect(association.dependent).to be_nil
     end
   end
 
   describe '#inverse_type' do
-
     it 'returns nil' do
       expect(association.inverse_type).to be_nil
     end
   end
 
   describe '#bindable?' do
-
     it 'returns false' do
       expect(association.bindable?(Person.new)).to be(false)
     end
   end
 
   describe '#inverses' do
-
     context 'when polymorphic' do
-
       before do
         EmbeddedObject.embedded_in :containable, polymorphic: true
       end
@@ -399,20 +352,17 @@ describe Mongoid::Association::Embedded::EmbedsOne do
       end
 
       context 'when another object is passed to the method' do
-
         let(:instance_of_other_class) do
           EmbeddedObject.new
         end
 
         context 'when the relation class has only one relation whose class matches the owning class' do
-
           it 'returns the :as attribute of this association' do
-            expect(association.inverses(instance_of_other_class)).to match_array([ :containable ])
+            expect(association.inverses(instance_of_other_class)).to contain_exactly(:containable)
           end
         end
 
         context 'when :inverse_of is specified' do
-
           before do
             options.merge!(inverse_of: :inverse_name)
           end
@@ -423,17 +373,14 @@ describe Mongoid::Association::Embedded::EmbedsOne do
         end
 
         context 'when inverse_of is not specified' do
-
           it 'returns the :as attribute of this association' do
-            expect(association.inverses(instance_of_other_class)).to match_array([ :containable ])
+            expect(association.inverses(instance_of_other_class)).to contain_exactly(:containable)
           end
         end
       end
 
       context 'when another object is not passed to the method' do
-
         context 'when inverse_of is specified' do
-
           before do
             options.merge!(inverse_of: :inverse_name)
           end
@@ -444,20 +391,18 @@ describe Mongoid::Association::Embedded::EmbedsOne do
         end
 
         context 'when inverse_of is not specified' 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])
+              expect(inverses).to eq([ :parent ])
             end
 
             context 'when other associations referencing unloaded classes exist' do
@@ -467,7 +412,7 @@ describe Mongoid::Association::Embedded::EmbedsOne do
 
               it 'does not load other classes' do
                 inverses = association.inverses
-                expect(inverses).to eq([:parent])
+                expect(inverses).to eq([ :parent ])
                 expect(Object.const_defined?(:EoDnlMarker)).to be false
               end
             end
@@ -477,13 +422,11 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when not polymorphic' do
-
       before do
         EmbeddedObject.embedded_in :container
       end
 
       context 'when inverse_of is specified' do
-
         before do
           options.merge!(inverse_of: :inverse_name)
         end
@@ -494,50 +437,42 @@ describe Mongoid::Association::Embedded::EmbedsOne do
       end
 
       context 'when inverse_of is not specified' do
-
         it 'uses the inverse class to find the inverse name' do
           expect(association.inverses).to eq([ :container ])
         end
       end
 
       context 'when :cyclic is specified' do
-
         it 'returns the cyclic inverse name' do
-
         end
       end
     end
   end
 
   describe '##inverse' do
-
     context 'when polymorphic' do
-
       before do
         EmbeddedObject.embedded_in :containable, polymorphic: true
       end
 
       let(:options) do
         {
-            as: :containable
+          as: :containable
         }
       end
 
       context 'when another object is passed to the method' do
-
         let(:instance_of_other_class) do
           EmbeddedObject.new
         end
 
         context 'when the relation class has only one relation whose class matches the owning class' do
-
           it 'returns the :as attribute of this association' do
             expect(association.inverse(instance_of_other_class)).to eq(:containable)
           end
         end
 
         context 'when :inverse_of is specified' do
-
           before do
             options.merge!(inverse_of: :inverse_name)
           end
@@ -548,7 +483,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
         end
 
         context 'when inverse_of is not specified' do
-
           it 'returns the :as attribute of this association' do
             expect(association.inverse(instance_of_other_class)).to eq(:containable)
           end
@@ -556,9 +490,7 @@ describe Mongoid::Association::Embedded::EmbedsOne do
       end
 
       context 'when another object is not passed to the method' do
-
         context 'when inverse_of is specified' do
-
           before do
             options.merge!(inverse_of: :inverse_name)
           end
@@ -569,7 +501,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
         end
 
         context 'when inverse_of is not specified' do
-
           it 'returns the :as attribute' do
             expect(association.inverse).to eq(:containable)
           end
@@ -578,13 +509,11 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when not polymorphic' do
-
       before do
         EmbeddedObject.embedded_in :container
       end
 
       context 'when inverse_of is specified' do
-
         before do
           options.merge!(inverse_of: :inverse_name)
         end
@@ -595,36 +524,29 @@ describe Mongoid::Association::Embedded::EmbedsOne do
       end
 
       context 'when inverse_of is not specified' do
-
         it 'uses the inverse class to find the inverse name' do
           expect(association.inverse).to eq(:container)
         end
       end
 
       context 'when :cyclic is specified' do
-
         it 'returns the cyclic inverse name' do
-
         end
       end
     end
   end
 
   describe '#inverse_association' do
-
   end
 
   describe '#autosave' do
-
     it 'returns false' do
       expect(association.autosave).to be(false)
     end
   end
 
   describe '#relation_class_name' do
-
     context 'when the :class_name option is specified' do
-
       let(:options) do
         { class_name: 'OtherEmbeddedObject' }
       end
@@ -645,7 +567,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when the class_name option is not specified' do
-
       it 'uses the name of the relation to deduce the class name' do
         expect(association.relation_class_name).to eq('EmbeddedObject')
       end
@@ -664,9 +585,7 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#klass' do
-
     context 'when the :class_name option is specified' do
-
       let!(:_class) do
         class OtherEmbeddedObject; end
         OtherEmbeddedObject
@@ -682,7 +601,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when the class_name option is not specified' do
-
       it 'uses the name of the relation to deduce the class name' do
         expect(association.klass).to eq(EmbeddedObject)
       end
@@ -690,23 +608,19 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#inverse_class_name' do
-
     it 'returns the name of the owner class' do
       expect(association.inverse_class_name).to eq(Container.name)
     end
   end
 
   describe '#inverse_class' do
-
     it 'returns the owner class' do
       expect(association.inverse_class).to be(Container)
     end
   end
 
   describe '#inverse_of' do
-
     context 'when :inverse_of is specified in the options' do
-
       let(:options) do
         { inverse_of: :objects_list }
       end
@@ -717,7 +631,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when :inverse_of is not specified in the options' do
-
       it 'returns nil' do
         expect(association.inverse_of).to be_nil
       end
@@ -725,25 +638,20 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#key' do
-
     it 'returns the name of the relation as a string' do
       expect(association.key).to eq(name.to_s)
     end
   end
 
   describe '#setter' do
-
     it 'returns a string of the name followed by =' do
       expect(association.setter).to eq("#{name}=")
     end
   end
 
   describe '#validate?' do
-
     context 'when :validate is specified in the options' do
-
       context 'when validate is true' do
-
         let(:options) do
           { validate: true }
         end
@@ -754,7 +662,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
       end
 
       context 'when validate is false' do
-
         let(:options) do
           { validate: false }
         end
@@ -766,7 +673,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when :validate is not specified in the options' do
-
       it 'returns the validation_default' do
         expect(association.send(:validate?)).to eq(association.validation_default)
       end
@@ -774,11 +680,8 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#autobuilding?' do
-
     context 'when :autobuild is specified in the options' do
-
       context 'when autobuild is true' do
-
         let(:options) do
           { autobuild: true }
         end
@@ -789,7 +692,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
       end
 
       context 'when autobuild is false' do
-
         let(:options) do
           { autobuild: false }
         end
@@ -801,7 +703,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when :validate is not specified in the options' do
-
       it 'returns false' do
         expect(association.autobuilding?).to be(false)
       end
@@ -809,23 +710,19 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#forced_nil_inverse?' do
-
     it 'returns false' do
       expect(association.forced_nil_inverse?).to be(false)
     end
   end
 
   describe '#stores_foreign_key?' do
-
     it 'returns false' do
       expect(association.stores_foreign_key?).to be(false)
     end
   end
 
   describe '#inverse_setter' do
-
     context 'when an inverse can be determined' do
-
       before do
         EmbeddedObject.embedded_in :container
       end
@@ -836,7 +733,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when an inverse cannot be determined' do
-
       it 'returns nil' do
         expect(association.inverse_setter).to be_nil
       end
@@ -844,11 +740,10 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#extension' do
-
     context 'when a block is passed' do
-
       let(:association) do
-        embeds_one_class.embeds_one name, options do; end
+        embeds_one_class.embeds_one name, options do
+        end
       end
 
       it 'defines an extension module' do
@@ -857,12 +752,12 @@ describe Mongoid::Association::Embedded::EmbedsOne do
 
       it 'returns the extension' do
         expect(association.extension).to eq(
-          "#{embeds_one_class.name}::#{embeds_one_class.name}#{name.to_s.camelize}RelationExtension".constantize)
+          "#{embeds_one_class.name}::#{embeds_one_class.name}#{name.to_s.camelize}RelationExtension".constantize
+        )
       end
     end
 
     context 'when an :extension is not specified in the options' do
-
       it 'returns false' do
         expect(association.extension).to be_nil
       end
@@ -870,34 +765,28 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#criteria' do
-
     it 'does not respond to the method' do
-      expect {
+      expect do
         association.criteria
-      }.to raise_exception(NoMethodError)
+      end.to raise_exception(NoMethodError)
     end
   end
 
   describe '#destructive?' do
-
     it 'returns false' do
       expect(association.destructive?).to be(false)
     end
   end
 
   describe '#nested_builder' do
-
     it 'returns an instance of Association::Nested::One' do
       expect(association.nested_builder({}, {})).to be_a(Mongoid::Association::Nested::One)
     end
   end
 
   describe '#cascading_callbacks?' do
-
     context 'when cascade_callbacks is specified in the options' do
-
       context 'when :cascade_callbacks is true' do
-
         let(:options) do
           { cascade_callbacks: true }
         end
@@ -908,7 +797,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
       end
 
       context 'when :cascade_callbacks is false' do
-
         let(:options) do
           { cascade_callbacks: false }
         end
@@ -920,7 +808,6 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
 
     context 'when cascade_callbacks is not specified in the options' do
-
       it 'returns false' do
         expect(association.cascading_callbacks?).to be(false)
       end
@@ -928,14 +815,12 @@ describe Mongoid::Association::Embedded::EmbedsOne do
   end
 
   describe '#path' do
-
     it 'returns an instance of Mongoid::Atomic::Paths::Root' do
-      expect(association.path(double( :_parent => true))).to be_a(Mongoid::Atomic::Paths::Embedded::One)
+      expect(association.path(double(_parent: true))).to be_a(Mongoid::Atomic::Paths::Embedded::One)
     end
   end
 
   describe '#create_relation' do
-
     let(:owner) do
       Container.new
     end
@@ -955,30 +840,30 @@ describe Mongoid::Association::Embedded::EmbedsOne do
     end
   end
 
-  context "when multiple embeds_one associations reference the same class" do
+  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
+    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")
+      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
+    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")
+      it 'updates the correct association' do
+        expect(acme.address.city).to eq('Gotham')
+        expect(acme.delivery_address.city).to eq('Bigville')
       end
     end
   end