mongoid 9.0.11 → 9.1.0

data/spec/mongoid/association/referenced/has_many_spec.rb

@@ -1,11 +1,9 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "spec_helper"
-require_relative './has_many_models'
+require 'spec_helper'
+require_relative 'has_many_models'
 
 describe Mongoid::Association::Referenced::HasMany do
-
   before do
     class OwnerObject; include Mongoid::Document; end
     class BelongingObject; include Mongoid::Document; end
@@ -29,14 +27,13 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   let(:options) do
-    { }
+    {}
   end
 
   describe '#relation_complements' do
-
     let(:expected_complements) do
       [
-          Mongoid::Association::Referenced::BelongsTo,
+        Mongoid::Association::Referenced::BelongsTo
       ]
     end
 
@@ -46,7 +43,6 @@ describe Mongoid::Association::Referenced::HasMany 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,12 +69,10 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'autosave' do
-
       context 'when the :autosave option is true' do
-
         let(:options) do
           {
-              autosave: true
+            autosave: true
           }
         end
 
@@ -95,10 +89,9 @@ describe Mongoid::Association::Referenced::HasMany do
       end
 
       context 'when the :autosave option is false' do
-
         let(:options) do
           {
-              autosave: false
+            autosave: false
           }
         end
 
@@ -109,7 +102,6 @@ describe Mongoid::Association::Referenced::HasMany do
       end
 
       context 'when the :autosave 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 :has_many macro
@@ -124,10 +116,9 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when the :validate option is true' do
-
       let(:options) do
         {
-            validate: true
+          validate: true
         }
       end
 
@@ -144,10 +135,9 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when the :validate option is false' do
-
       let(:options) do
         {
-            validate: false
+          validate: false
         }
       end
 
@@ -158,7 +148,6 @@ describe Mongoid::Association::Referenced::HasMany 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 :has_many macro
@@ -172,13 +161,10 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'polymorphic' do
-
       context 'when the as option is provided' do
-
-
         let(:options) do
           {
-              as: :containable
+            as: :containable
           }
         end
 
@@ -192,7 +178,6 @@ describe Mongoid::Association::Referenced::HasMany do
       end
 
       context 'when the as option is not provided' do
-
         it 'does not set the polymorphic attribute on the owner class' do
           expect(has_many_class.polymorphic).to be(false)
         end
@@ -200,14 +185,11 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'dependent' do
-
       context 'when the dependent option is provided' do
-
         context 'when the dependent option is :delete_all' do
-
           let(:options) do
             {
-                dependent: :delete_all
+              dependent: :delete_all
             }
           end
 
@@ -224,10 +206,9 @@ describe Mongoid::Association::Referenced::HasMany do
         end
 
         context 'when the dependent option is :destroy' do
-
           let(:options) do
             {
-                dependent: :destroy
+              dependent: :destroy
             }
           end
 
@@ -244,10 +225,9 @@ describe Mongoid::Association::Referenced::HasMany do
         end
 
         context 'when the dependent option is :nullify' do
-
           let(:options) do
             {
-                dependent: :nullify
+              dependent: :nullify
             }
           end
 
@@ -264,10 +244,9 @@ describe Mongoid::Association::Referenced::HasMany do
         end
 
         context 'when the dependent option is :restrict_with_exception' do
-
           let(:options) do
             {
-                dependent: :restrict_with_exception
+              dependent: :restrict_with_exception
             }
           end
 
@@ -284,10 +263,9 @@ describe Mongoid::Association::Referenced::HasMany do
         end
 
         context 'when the dependent option is :restrict_with_error' do
-
           let(:options) do
             {
-                dependent: :restrict_with_error
+              dependent: :restrict_with_error
             }
           end
 
@@ -305,7 +283,6 @@ describe Mongoid::Association::Referenced::HasMany do
       end
 
       context 'when the dependent option is not provided' do
-
         it 'does not set up the dependency' do
           expect(Mongoid::Association::Depending).not_to receive(:define_dependency!)
           association.setup!
@@ -315,9 +292,7 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#type' do
-
     context 'when polymorphic' do
-
       let(:options) do
         { as: :containable }
       end
@@ -328,7 +303,6 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when not polymorphic' do
-
       it 'returns nil' do
         expect(association.type).to be_nil
       end
@@ -336,9 +310,7 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#inverse_type' do
-
     context 'when polymorphic' do
-
       let(:options) do
         { as: :containable }
       end
@@ -349,7 +321,6 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when not polymorphic' do
-
       it 'returns nil' do
         expect(association.inverse_type).to be_nil
       end
@@ -357,9 +328,7 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#inverse_type_setter' do
-
     context 'when polymorphic' do
-
       let(:options) do
         { as: :containable }
       end
@@ -370,7 +339,6 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when not polymorphic' do
-
       it 'returns nil' do
         expect(association.inverse_type).to be_nil
       end
@@ -378,9 +346,7 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#foreign_key' do
-
     context 'when options has foreign_key specified' do
-
       let(:options) do
         { foreign_key: :other_object_id }
       end
@@ -391,7 +357,6 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when options does not have foreign_key specified' do
-
       it 'returns the default foreign key, the name of the inverse followed by "_id"' do
         expect(association.foreign_key).to eq("#{association.inverse}_id")
       end
@@ -399,19 +364,16 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#embedded?' do
-
     it 'returns false' do
       expect(association.embedded?).to be(false)
     end
   end
 
   describe '#primary_key' do
-
     context 'when the :primary_key option is specified' do
-
       let(:options) do
         {
-            primary_key: 'guid'
+          primary_key: 'guid'
         }
       end
 
@@ -421,7 +383,6 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when the :primary_key option is not specified' do
-
       it 'returns the primary_key default' do
         expect(association.primary_key).to eq(Mongoid::Association::Relatable::PRIMARY_KEY_DEFAULT)
       end
@@ -429,35 +390,30 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#relation' do
-
     it 'returns Mongoid::Association::Referenced::HasMany::Proxy' do
       expect(association.relation).to be(Mongoid::Association::Referenced::HasMany::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 '#merge!' do
-
   end
 
   describe '#store_as' do
@@ -467,19 +423,16 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#touchable?' do
-
     it 'return false' do
       expect(association.send(:touchable?)).to be(false)
     end
   end
 
   describe '#order' do
-
     context 'when order is specified in the options' do
-
       let(:options) do
         {
-            order: :rating.desc
+          order: :rating.desc
         }
       end
 
@@ -489,7 +442,6 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when order is not specified in the options' do
-
       it 'returns nil' do
         expect(association.order).to be_nil
       end
@@ -497,9 +449,7 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#scope' do
-
     context 'when scope is specified in the options' do
-
       let(:options) do
         { scope: -> { unscoped.where(foo: :bar) } }
       end
@@ -510,7 +460,6 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when scope is not specified in the options' do
-
       it 'returns nil' do
         expect(association.scope).to be_nil
       end
@@ -518,12 +467,10 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#as' do
-
     context 'when :as is specified in the options' do
-
       let(:options) do
         {
-            as: :containable
+          as: :containable
         }
       end
 
@@ -533,7 +480,6 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when :as is not specified in the options' do
-
       it 'returns nil' do
         expect(association.as).to be_nil
       end
@@ -541,23 +487,19 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#polymorphic?' do
-
     context 'when :as is specified in the options' do
-
       let(:options) do
         {
-            as: :containable
+          as: :containable
         }
       end
 
       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
@@ -565,20 +507,17 @@ describe Mongoid::Association::Referenced::HasMany 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
@@ -586,14 +525,11 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#dependent' do
-
     context 'when the dependent option is provided' do
-
       context 'when the dependent option is :delete_all' do
-
         let(:options) do
           {
-              dependent: :delete_all
+            dependent: :delete_all
           }
         end
 
@@ -603,10 +539,9 @@ describe Mongoid::Association::Referenced::HasMany do
       end
 
       context 'when the dependent option is :destroy' do
-
         let(:options) do
           {
-              dependent: :destroy
+            dependent: :destroy
           }
         end
 
@@ -616,10 +551,9 @@ describe Mongoid::Association::Referenced::HasMany do
       end
 
       context 'when the dependent option is :nullify' do
-
         let(:options) do
           {
-              dependent: :nullify
+            dependent: :nullify
           }
         end
 
@@ -629,10 +563,9 @@ describe Mongoid::Association::Referenced::HasMany do
       end
 
       context 'when the dependent option is :restrict_with_exception' do
-
         let(:options) do
           {
-              dependent: :restrict_with_exception
+            dependent: :restrict_with_exception
           }
         end
 
@@ -642,10 +575,9 @@ describe Mongoid::Association::Referenced::HasMany do
       end
 
       context 'when the dependent option is :restrict_with_error' do
-
         let(:options) do
           {
-              dependent: :restrict_with_error
+            dependent: :restrict_with_error
           }
         end
 
@@ -656,7 +588,6 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when the dependent option is not provided' do
-
       it 'returns nil' do
         expect(association.dependent).to be_nil
       end
@@ -664,48 +595,41 @@ describe Mongoid::Association::Referenced::HasMany do
   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
         BelongingObject.belongs_to :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
           BelongingObject.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
@@ -716,17 +640,14 @@ describe Mongoid::Association::Referenced::HasMany 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
@@ -737,7 +658,6 @@ describe Mongoid::Association::Referenced::HasMany do
         end
 
         context 'when inverse_of is not specified' do
-
           it 'returns the :as attribute' do
             expect(association.inverses).to eq([ :containable ])
           end
@@ -746,13 +666,11 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when not polymorphic' do
-
       before do
         BelongingObject.belongs_to :owner_object
       end
 
       context 'when inverse_of is specified' do
-
         before do
           options.merge!(inverse_of: :inverse_name)
         end
@@ -763,50 +681,42 @@ describe Mongoid::Association::Referenced::HasMany 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([ :owner_object ])
         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
         BelongingObject.belongs_to :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
           BelongingObject.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
@@ -817,7 +727,6 @@ describe Mongoid::Association::Referenced::HasMany 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
@@ -825,9 +734,7 @@ describe Mongoid::Association::Referenced::HasMany 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
@@ -838,7 +745,6 @@ describe Mongoid::Association::Referenced::HasMany do
         end
 
         context 'when inverse_of is not specified' do
-
           it 'returns the :as attribute' do
             expect(association.inverse).to eq(:containable)
           end
@@ -847,13 +753,11 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when not polymorphic' do
-
       before do
         BelongingObject.belongs_to :owner_object
       end
 
       context 'when inverse_of is specified' do
-
         before do
           options.merge!(inverse_of: :inverse_name)
         end
@@ -864,34 +768,27 @@ describe Mongoid::Association::Referenced::HasMany 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(:owner_object)
         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
-
     context 'when the autosave option is specified' do
-
       context 'when the autosave option is true' do
-
         let(:options) do
           {
-              autosave: true
+            autosave: true
           }
         end
 
@@ -901,10 +798,9 @@ describe Mongoid::Association::Referenced::HasMany do
       end
 
       context 'when the autosave option is false' do
-
         let(:options) do
           {
-              autosave: false
+            autosave: false
           }
         end
 
@@ -915,7 +811,6 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when the autosave option is not specified' do
-
       it 'returns false' do
         expect(association.autosave).to be(false)
       end
@@ -923,9 +818,7 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#relation_class_name' do
-
     context 'when the :class_name option is specified' do
-
       let(:options) do
         { class_name: 'OtherBelongingObject' }
       end
@@ -936,7 +829,6 @@ describe Mongoid::Association::Referenced::HasMany 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('BelongingObject')
       end
@@ -944,9 +836,7 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#relation_class' do
-
     context 'when the :class_name option is specified' do
-
       let!(:_class) do
         class OtherBelongingObject; end
         OtherBelongingObject
@@ -962,7 +852,6 @@ describe Mongoid::Association::Referenced::HasMany 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).to eq(BelongingObject)
       end
@@ -970,23 +859,19 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#inverse_class_name' do
-
     it 'returns the name of the owner class' do
       expect(association.inverse_class_name).to eq(OwnerObject.name)
     end
   end
 
   describe '#inverse_class' do
-
     it 'returns the owner class' do
       expect(association.inverse_class).to be(OwnerObject)
     end
   end
 
   describe '#inverse_of' do
-
     context 'when :inverse_of is specified in the options' do
-
       let(:options) do
         { inverse_of: :a_belonging_object }
       end
@@ -997,7 +882,6 @@ describe Mongoid::Association::Referenced::HasMany 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
@@ -1005,25 +889,20 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#key' do
-
     it 'returns the primary key' do
       expect(association.key).to eq(association.primary_key)
     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
@@ -1034,7 +913,6 @@ describe Mongoid::Association::Referenced::HasMany do
       end
 
       context 'when validate is false' do
-
         let(:options) do
           { validate: false }
         end
@@ -1046,7 +924,6 @@ describe Mongoid::Association::Referenced::HasMany 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
@@ -1054,23 +931,19 @@ describe Mongoid::Association::Referenced::HasMany 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
         BelongingObject.belongs_to :owner_object
       end
@@ -1081,7 +954,6 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when an inverse cannot be determined' do
-
       it 'returns nil' do
         expect(association.inverse_setter).to be_nil
       end
@@ -1089,11 +961,10 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#extension' do
-
     context 'when a block is passed' do
-
       let(:association) do
-        has_many_class.embeds_one name, options do; end
+        has_many_class.embeds_one name, options do
+        end
       end
 
       it 'defines an extension module' do
@@ -1102,12 +973,12 @@ describe Mongoid::Association::Referenced::HasMany do
 
       it 'returns the extension' do
         expect(association.extension).to eq(
-          "#{has_many_class.name}::#{has_many_class.name}#{name.to_s.camelize}RelationExtension".constantize)
+          "#{has_many_class.name}::#{has_many_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
@@ -1115,28 +986,23 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#foreign_key_setter' do
-
     it 'returns the foreign key followed by "="' do
       expect(association.foreign_key_setter).to eq("#{association.foreign_key}=")
     end
   end
 
   describe '#criteria' do
-
     it 'returns a criteria object' do
       expect(association.criteria(OwnerObject.new)).to be_a(Mongoid::Criteria)
     end
   end
 
   describe '#destructive?' do
-
     context 'when the dependent option is provided' do
-
       context 'when the dependent option is :delete_all' do
-
         let(:options) do
           {
-              dependent: :delete_all
+            dependent: :delete_all
           }
         end
 
@@ -1146,10 +1012,9 @@ describe Mongoid::Association::Referenced::HasMany do
       end
 
       context 'when the dependent option is :destroy' do
-
         let(:options) do
           {
-              dependent: :destroy
+            dependent: :destroy
           }
         end
 
@@ -1159,10 +1024,9 @@ describe Mongoid::Association::Referenced::HasMany do
       end
 
       context 'when the dependent option is :nullify' do
-
         let(:options) do
           {
-              dependent: :nullify
+            dependent: :nullify
           }
         end
 
@@ -1172,10 +1036,9 @@ describe Mongoid::Association::Referenced::HasMany do
       end
 
       context 'when the dependent option is :restrict_with_exception' do
-
         let(:options) do
           {
-              dependent: :restrict_with_exception
+            dependent: :restrict_with_exception
           }
         end
 
@@ -1185,10 +1048,9 @@ describe Mongoid::Association::Referenced::HasMany do
       end
 
       context 'when the dependent option is :restrict_with_error' do
-
         let(:options) do
           {
-              dependent: :restrict_with_error
+            dependent: :restrict_with_error
           }
         end
 
@@ -1199,7 +1061,6 @@ describe Mongoid::Association::Referenced::HasMany do
     end
 
     context 'when the dependent option is not provided' do
-
       it 'returns false' do
         expect(association.destructive?).to be(false)
       end
@@ -1207,28 +1068,24 @@ describe Mongoid::Association::Referenced::HasMany do
   end
 
   describe '#nested_builder' do
-
     it 'returns an instance of Association::Nested::Many' do
       expect(association.nested_builder({}, {})).to be_a(Mongoid::Association::Nested::Many)
     end
   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::Root)
+      expect(association.path(double(_parent: true))).to be_a(Mongoid::Atomic::Paths::Root)
     end
   end
 
   describe '#foreign_key_check' do
-
     it 'returns the nil' do
       expect(association.foreign_key_check).to be_nil
     end
   end
 
   describe '#create_relation' do
-
     let(:owner) do
       OwnerObject.new
     end
@@ -1248,8 +1105,10 @@ describe Mongoid::Association::Referenced::HasMany do
     end
   end
 
-  context "when adding an object to the association" do
+  context 'when adding an object to the association' do
     let!(:start_time) { Timecop.freeze(Time.at(Time.now.to_i)) }
+    let!(:school) { HmmSchool.create! }
+    let!(:student) { HmmStudent.create! }
 
     let(:update_time) do
       Timecop.freeze(Time.at(Time.now.to_i) + 2)
@@ -1259,15 +1118,12 @@ describe Mongoid::Association::Referenced::HasMany do
       Timecop.return
     end
 
-    let!(:school) { HmmSchool.create! }
-    let!(:student) { HmmStudent.create! }
-
     before do
       update_time
       student.update(school: school)
     end
 
-    it "updates the updated_at" do
+    it 'updates the updated_at' do
       expect(student.updated_at).to eq(update_time)
     end
   end