mongoid 9.0.11 → 9.1.0

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

@@ -1,12 +1,9 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "spec_helper"
+require 'spec_helper'
 
 describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
-
-  describe "#===" do
-
+  describe '#===' do
     let(:base) do
       Person.new
     end
@@ -16,27 +13,23 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
     end
 
     let(:association) do
-      Person.relations["name"]
+      Person.relations['name']
     end
 
     let(:relation) do
       described_class.new(base, target, association)
     end
 
-    context "when the proxied document is same class" do
-
-      it "returns true" do
-        expect((relation === Name.new)).to be true
+    context 'when the proxied document is same class' do
+      it 'returns true' do
+        expect(relation === Name.new).to be true
       end
     end
   end
 
-  describe "#=" do
-
-    context "when the relation is not cyclic" do
-
-      context "when the parent is a new record" do
-
+  describe '#=' do
+    context 'when the relation is not cyclic' do
+      context 'when the parent is a new record' do
         let(:person) do
           Person.new
         end
@@ -49,24 +42,23 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           person.name = name
         end
 
-        it "sets the target of the relation" do
+        it 'sets the target of the relation' do
           expect(person.name).to eq(name)
         end
 
-        it "sets the base on the inverse relation" do
+        it 'sets the base on the inverse relation' do
           expect(name.namable).to eq(person)
         end
 
-        it "sets the same instance on the inverse relation" do
+        it 'sets the same instance on the inverse relation' do
           expect(name.namable).to eql(person)
         end
 
-        it "does not save the target" do
-          expect(name).to_not be_persisted
+        it 'does not save the target' do
+          expect(name).not_to be_persisted
         end
 
-        context "with overwritten getter" do
-
+        context 'with overwritten getter' do
           before do
             person.name = nil
             def person.name_with_default
@@ -78,15 +70,14 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
             end
           end
 
-          it "sets the target without an invinite recursion" do
+          it 'sets the target without an infinite recursion' do
             person.name = name
             expect(person.name).to be_present
           end
         end
       end
 
-      context "when the parent is not a new record" do
-
+      context 'when the parent is not a new record' do
         let(:person) do
           Person.create!
         end
@@ -95,8 +86,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           Name.new
         end
 
-        context "when setting with a hash" do
-
+        context 'when setting with a hash' do
           before do
             person.name = {}
           end
@@ -105,74 +95,70 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
             person.name
           end
 
-          it "sets the target of the relation" do
+          it 'sets the target of the relation' do
             expect(person.name).to eq(child_name)
           end
 
-          it "sets the base on the inverse relation" do
+          it 'sets the base on the inverse relation' do
             expect(child_name.namable).to eq(person)
           end
 
-          it "sets the same instance on the inverse relation" do
+          it 'sets the same instance on the inverse relation' do
             expect(child_name.namable).to eql(person)
           end
 
-          it "saves the target" do
+          it 'saves the target' do
             expect(child_name).to be_persisted
           end
 
-          context "when replacing a relation with a hash" do
-
+          context 'when replacing a relation with a hash' do
             before do
               person.name = {}
             end
 
-            it "sets the relation with the proper object" do
+            it 'sets the relation with the proper object' do
               expect(person.name).to be_a(Name)
             end
           end
         end
 
-        context "when setting to the same document" do
-
+        context 'when setting to the same document' do
           before do
             person.name = name
             person.name = person.name
           end
 
-          it "does not change the relation" do
+          it 'does not change the relation' do
             expect(person.name).to eq(name)
           end
 
-          it "does not persist any change" do
+          it 'does not persist any change' do
             expect(person.reload.name).to eq(name)
           end
         end
 
-        context "when setting directly" do
-
+        context 'when setting directly' do
           before do
             person.name = name
           end
 
-          it "sets the target of the relation" do
+          it 'sets the target of the relation' do
             expect(person.name).to eq(name)
           end
 
-          it "sets the base on the inverse relation" do
+          it 'sets the base on the inverse relation' do
             expect(name.namable).to eq(person)
           end
 
-          it "sets the same instance on the inverse relation" do
+          it 'sets the same instance on the inverse relation' do
             expect(name.namable).to eql(person)
           end
 
-          it "saves the target" do
+          it 'saves the target' do
             expect(name).to be_persisted
           end
 
-          context "when replacing an existing document" do
-
+          context 'when replacing an existing document' do
             let(:pet_owner) do
               PetOwner.create!
             end
@@ -190,22 +176,21 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
               pet_owner.pet = pet_two
             end
 
-            it "runs the destroy callbacks on the old document" do
+            it 'runs the destroy callbacks on the old document' do
               expect(pet_one.destroy_flag).to be true
             end
 
-            it "keeps the name of the destroyed" do
-              expect(pet_one.name).to eq("kika")
+            it 'keeps the name of the destroyed' do
+              expect(pet_one.name).to eq('kika')
             end
 
-            it "saves the new name" do
-              expect(pet_owner.pet.name).to eq("tiksy")
+            it 'saves the new name' do
+              expect(pet_owner.pet.name).to eq('tiksy')
             end
           end
         end
 
         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)
           end
@@ -219,7 +204,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           end
 
           before do
-            expect(Mongo::Logger.logger).to receive(:debug?).exactly(2).times
+            expect(Mongo::Logger.logger).to receive(:debug?).twice
           end
 
           it 'does not execute an unnecessary unset for the relation' do
@@ -227,27 +212,24 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           end
         end
 
-        context "when setting via the parent attributes" do
-
+        context 'when setting via the parent attributes' do
           before do
             person.attributes = { name: name }
           end
 
-          it "sets the target of the relation" do
+          it 'sets the target of the relation' do
             expect(person.name).to eq(name)
           end
 
-          it "does not save the target" do
-            expect(name).to_not be_persisted
+          it 'does not save the target' do
+            expect(name).not_to be_persisted
           end
         end
       end
     end
 
-    context "when the relation is cyclic" do
-
-      context "when the parent is a new record" do
-
+    context 'when the relation is cyclic' do
+      context 'when the parent is a new record' do
         let(:parent_shelf) do
           Shelf.new
         end
@@ -260,25 +242,24 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           parent_shelf.child_shelf = child_shelf
         end
 
-        it "sets the target of the relation" do
+        it 'sets the target of the relation' do
           expect(parent_shelf.child_shelf).to eq(child_shelf)
         end
 
-        it "sets the base on the inverse relation" do
+        it 'sets the base on the inverse relation' do
           expect(child_shelf.parent_shelf).to eq(parent_shelf)
         end
 
-        it "sets the same instance on the inverse relation" do
+        it 'sets the same instance on the inverse relation' do
           expect(child_shelf.parent_shelf).to eql(parent_shelf)
         end
 
-        it "does not save the target" do
-          expect(child_shelf).to_not be_persisted
+        it 'does not save the target' do
+          expect(child_shelf).not_to be_persisted
         end
       end
 
-      context "when the parent is not a new record" do
-
+      context 'when the parent is not a new record' do
         let(:parent_shelf) do
           Shelf.create!
         end
@@ -291,26 +272,25 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           parent_shelf.child_shelf = child_shelf
         end
 
-        it "sets the target of the relation" do
+        it 'sets the target of the relation' do
           expect(parent_shelf.child_shelf).to eq(child_shelf)
         end
 
-        it "sets the base on the inverse relation" do
+        it 'sets the base on the inverse relation' do
           expect(child_shelf.parent_shelf).to eq(parent_shelf)
         end
 
-        it "sets the same instance on the inverse relation" do
+        it 'sets the same instance on the inverse relation' do
           expect(child_shelf.parent_shelf).to eql(parent_shelf)
         end
 
-        it "saves the target" do
+        it 'saves the target' do
           expect(child_shelf).to be_persisted
         end
       end
     end
 
-    context "when setting a new document multiple times in a row" do
-
+    context 'when setting a new document multiple times in a row' do
       let(:parent) do
         Parent.create!
       end
@@ -321,18 +301,15 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
         parent.first_child = Child.new
       end
 
-      it "saves the child document" do
+      it 'saves the child document' do
         expect(parent.first_child).to be_a(Child)
       end
     end
   end
 
-  describe "#= nil" do
-
-    context "when the relation is not cyclic" do
-
-      context "when the parent is a new record" do
-
+  describe '#= nil' do
+    context 'when the relation is not cyclic' do
+      context 'when the parent is a new record' do
         let(:person) do
           Person.new
         end
@@ -346,17 +323,16 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           person.name = nil
         end
 
-        it "sets the relation to nil" do
+        it 'sets the relation to nil' do
           expect(person.name).to be_nil
         end
 
-        it "removes the inverse relation" do
+        it 'removes the inverse relation' do
           expect(name.namable).to be_nil
         end
       end
 
-      context "when the inverse is already nil" do
-
+      context 'when the inverse is already nil' do
         let(:person) do
           Person.new
         end
@@ -365,13 +341,12 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           person.name = nil
         end
 
-        it "sets the relation to nil" do
+        it 'sets the relation to nil' do
           expect(person.name).to be_nil
         end
       end
 
-      context "when the parent is persisted" do
-
+      context 'when the parent is persisted' do
         let(:person) do
           Person.create!
         end
@@ -380,48 +355,44 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           Name.new
         end
 
-        context "when setting directly" do
-
+        context 'when setting directly' do
           before do
             person.name = name
             person.name = nil
           end
 
-          it "sets the relation to nil" do
+          it 'sets the relation to nil' do
             expect(person.name).to be_nil
           end
 
-          it "removed the inverse relation" do
+          it 'removed the inverse relation' do
             expect(name.namable).to be_nil
           end
 
-          it "deletes the child document" do
+          it 'deletes the child document' do
             expect(name).to be_destroyed
           end
         end
 
-        context "when setting via parent attributes" do
-
+        context 'when setting via parent attributes' do
           before do
             person.name = name
             person.attributes = { name: nil }
           end
 
-          it "sets the relation to nil" do
+          it 'sets the relation to nil' do
             expect(person.name).to be_nil
           end
 
-          it "does not delete the child document" do
-            expect(name).to_not be_destroyed
+          it 'does not delete the child document' do
+            expect(name).not_to be_destroyed
           end
         end
       end
     end
 
-    context "when the relation is cyclic" do
-
-      context "when the parent is a new record" do
-
+    context 'when the relation is cyclic' do
+      context 'when the parent is a new record' do
         let(:parent_shelf) do
           Shelf.new
         end
@@ -435,17 +406,16 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           parent_shelf.child_shelf = nil
         end
 
-        it "sets the relation to nil" do
+        it 'sets the relation to nil' do
           expect(parent_shelf.child_shelf).to be_nil
         end
 
-        it "removes the inverse relation" do
+        it 'removes the inverse relation' do
           expect(child_shelf.parent_shelf).to be_nil
         end
       end
 
-      context "when the inverse is already nil" do
-
+      context 'when the inverse is already nil' do
         let(:parent_shelf) do
           Shelf.new
         end
@@ -454,13 +424,12 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           parent_shelf.child_shelf = nil
         end
 
-        it "sets the relation to nil" do
+        it 'sets the relation to nil' do
           expect(parent_shelf.child_shelf).to be_nil
         end
       end
 
-      context "when the documents are not new records" do
-
+      context 'when the documents are not new records' do
         let(:parent_shelf) do
           Shelf.create!
         end
@@ -474,15 +443,15 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           parent_shelf.child_shelf = nil
         end
 
-        it "sets the relation to nil" do
+        it 'sets the relation to nil' do
           expect(parent_shelf.child_shelf).to be_nil
         end
 
-        it "removed the inverse relation" do
+        it 'removed the inverse relation' do
           expect(child_shelf.parent_shelf).to be_nil
         end
 
-        it "deletes the child document" do
+        it 'deletes the child document' do
           expect(child_shelf).to be_destroyed
         end
       end
@@ -490,15 +459,10 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
   end
 
   describe "#build_#\{name}" do
-
-    context "when the relation is not cyclic" do
-
-      context "when the parent is a new record" do
-
-        context "when not providing any attributes" do
-
-          context "when building once" do
-
+    context 'when the relation is not cyclic' do
+      context 'when the parent is a new record' do
+        context 'when not providing any attributes' do
+          context 'when building once' do
             let(:person) do
               Person.new
             end
@@ -507,25 +471,24 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
               person.build_name
             end
 
-            it "sets the target of the relation" do
+            it 'sets the target of the relation' do
               expect(person.name).to eq(name)
             end
 
-            it "sets the base on the inverse relation" do
+            it 'sets the base on the inverse relation' do
               expect(name.namable).to eq(person)
             end
 
-            it "sets no attributes" do
+            it 'sets no attributes' do
               expect(name.first_name).to be_nil
             end
 
-            it "does not save the target" do
-              expect(name).to_not be_persisted
+            it 'does not save the target' do
+              expect(name).not_to be_persisted
             end
           end
 
-          context "when building twice" do
-
+          context 'when building twice' do
             let(:person) do
               Person.new
             end
@@ -535,26 +498,25 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
               person.build_name
             end
 
-            it "sets the target of the relation" do
+            it 'sets the target of the relation' do
               expect(person.name).to eq(name)
             end
 
-            it "sets the base on the inverse relation" do
+            it 'sets the base on the inverse relation' do
               expect(name.namable).to eq(person)
             end
 
-            it "sets no attributes" do
+            it 'sets no attributes' do
               expect(name.first_name).to be_nil
             end
 
-            it "does not save the target" do
-              expect(name).to_not be_persisted
+            it 'does not save the target' do
+              expect(name).not_to be_persisted
             end
           end
         end
 
-        context "when passing nil as the attributes" do
-
+        context 'when passing nil as the attributes' do
           let(:person) do
             Person.new
           end
@@ -563,71 +525,67 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
             person.build_name(nil)
           end
 
-          it "sets the target of the relation" do
+          it 'sets the target of the relation' do
             expect(person.name).to eq(name)
           end
 
-          it "sets the base on the inverse relation" do
+          it 'sets the base on the inverse relation' do
             expect(name.namable).to eq(person)
           end
 
-          it "sets no attributes" do
+          it 'sets no attributes' do
             expect(name.first_name).to be_nil
           end
 
-          it "does not save the target" do
-            expect(name).to_not be_persisted
+          it 'does not save the target' do
+            expect(name).not_to be_persisted
           end
         end
 
-        context "when providing attributes" do
-
+        context 'when providing attributes' do
           let(:person) do
             Person.new
           end
 
           let!(:name) do
-            person.build_name(first_name: "James")
+            person.build_name(first_name: 'James')
           end
 
-          it "sets the target of the relation" do
+          it 'sets the target of the relation' do
             expect(person.name).to eq(name)
           end
 
-          it "sets the base on the inverse relation" do
+          it 'sets the base on the inverse relation' do
             expect(name.namable).to eq(person)
           end
 
-          it "sets the attributes" do
-            expect(name.first_name).to eq("James")
+          it 'sets the attributes' do
+            expect(name.first_name).to eq('James')
           end
 
-          it "does not save the target" do
-            expect(name).to_not be_persisted
+          it 'does not save the target' do
+            expect(name).not_to be_persisted
           end
         end
       end
 
-      context "when the parent is not a new record" do
-
+      context 'when the parent is not a new record' do
         let(:person) do
           Person.create!
         end
 
         let!(:name) do
-          person.build_name(first_name: "James")
+          person.build_name(first_name: 'James')
         end
 
-        it "does not save the target" do
-          expect(name).to_not be_persisted
+        it 'does not save the target' do
+          expect(name).not_to be_persisted
         end
       end
     end
 
-    context "when the relation is cyclic" do
-
-      context "when the parent is a new record" do
-
+    context 'when the relation is cyclic' do
+      context 'when the parent is a new record' do
         let(:parent_shelf) do
           Shelf.new
         end
@@ -636,25 +594,24 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           parent_shelf.build_child_shelf(level: 1)
         end
 
-        it "sets the target of the relation" do
+        it 'sets the target of the relation' do
           expect(parent_shelf.child_shelf).to eq(child_shelf)
         end
 
-        it "sets the base on the inverse relation" do
+        it 'sets the base on the inverse relation' do
           expect(child_shelf.parent_shelf).to eq(parent_shelf)
         end
 
-        it "sets the attributes" do
+        it 'sets the attributes' do
           expect(child_shelf.level).to eq(1)
         end
 
-        it "does not save the target" do
-          expect(child_shelf).to_not be_persisted
+        it 'does not save the target' do
+          expect(child_shelf).not_to be_persisted
         end
       end
 
-      context "when the parent is not a new record" do
-
+      context 'when the parent is not a new record' do
         let(:parent_shelf) do
           Shelf.create!
         end
@@ -663,19 +620,16 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           parent_shelf.build_child_shelf(level: 2)
         end
 
-        it "does not save the target" do
-          expect(child_shelf).to_not be_persisted
+        it 'does not save the target' do
+          expect(child_shelf).not_to be_persisted
         end
       end
     end
   end
 
   describe "#create_#\{name}" do
-
-    context "when the parent is a new record" do
-
-      context "when not providing any attributes" do
-
+    context 'when the parent is a new record' do
+      context 'when not providing any attributes' do
         let(:person) do
           Person.new
         end
@@ -684,25 +638,24 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           person.create_name
         end
 
-        it "sets the target of the relation" do
+        it 'sets the target of the relation' do
           expect(person.name).to eq(name)
         end
 
-        it "sets the base on the inverse relation" do
+        it 'sets the base on the inverse relation' do
           expect(name.namable).to eq(person)
         end
 
-        it "sets no attributes" do
+        it 'sets no attributes' do
           expect(name.first_name).to be_nil
         end
 
-        it "saves the target" do
+        it 'saves the target' do
           expect(name).to be_persisted
         end
       end
 
-      context "when passing nil as the attributes" do
-
+      context 'when passing nil as the attributes' do
         let(:person) do
           Person.new
         end
@@ -711,73 +664,68 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           person.create_name(nil)
         end
 
-        it "sets the target of the relation" do
+        it 'sets the target of the relation' do
           expect(person.name).to eq(name)
         end
 
-        it "sets the base on the inverse relation" do
+        it 'sets the base on the inverse relation' do
           expect(name.namable).to eq(person)
         end
 
-        it "sets no attributes" do
+        it 'sets no attributes' do
           expect(name.first_name).to be_nil
         end
 
-        it "saves the target" do
+        it 'saves the target' do
           expect(name).to be_persisted
         end
       end
 
-      context "when providing attributes" do
-
+      context 'when providing attributes' do
         let(:person) do
           Person.new
         end
 
         let!(:name) do
-          person.create_name(first_name: "James")
+          person.create_name(first_name: 'James')
         end
 
-        it "sets the target of the relation" do
+        it 'sets the target of the relation' do
           expect(person.name).to eq(name)
         end
 
-        it "sets the base on the inverse relation" do
+        it 'sets the base on the inverse relation' do
           expect(name.namable).to eq(person)
         end
 
-        it "sets the attributes" do
-          expect(name.first_name).to eq("James")
+        it 'sets the attributes' do
+          expect(name.first_name).to eq('James')
         end
 
-        it "saves the target" do
+        it 'saves the target' do
           expect(name).to be_persisted
         end
       end
 
-      context "when the parent is not a new record" do
-
+      context 'when the parent is not a new record' do
         let(:person) do
           Person.create!
         end
 
         let!(:name) do
-          person.create_name(first_name: "James")
+          person.create_name(first_name: 'James')
         end
 
-        it "does not save the target" do
+        it 'does not save the target' do
           expect(name).to be_persisted
         end
       end
     end
   end
 
-  describe "when the relationship is polymorphic" do
-
-    context "when updating an aliased embedded document" do
-
-      context "when the embedded document inherits its relationship" do
-
+  describe 'when the relationship is polymorphic' do
+    context 'when updating an aliased embedded document' do
+      context 'when the embedded document inherits its relationship' do
         let(:courier_job) do
           CourierJob.create!
         end
@@ -793,7 +741,7 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
         after do
           # Note that this is necessary, otherwise all further Address objects will be
           # saved with a "_type" attributes. See Traversable#inherited for why.
-          Address.send(:remove_defaults, "_type")
+          Address.send(:remove_defaults, '_type')
         end
 
         before do
@@ -802,28 +750,26 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           courier_job.reload
         end
 
-        it "the child is embedded correctly" do
+        it 'the child is embedded correctly' do
           expect(courier_job.drop_address).to eq(new_child)
         end
       end
     end
   end
 
-  describe ".embedded?" do
-
-    it "returns true" do
+  describe '.embedded?' do
+    it 'returns true' do
       expect(described_class).to be_embedded
     end
   end
 
-  describe "#respond_to?" do
-
+  describe '#respond_to?' do
     let(:person) do
       Person.new
     end
 
     let!(:name) do
-      person.build_name(first_name: "Tony")
+      person.build_name(first_name: 'Tony')
     end
 
     let(:document) do
@@ -831,57 +777,52 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
     end
 
     Mongoid::Document.public_instance_methods(true).each do |method|
-
       context "when checking #{method}" do
-
-        it "returns true" do
+        it 'returns true' do
           expect(document.respond_to?(method)).to be true
         end
       end
     end
 
-    it "responds to persisted?" do
+    it 'responds to persisted?' do
       expect(document).to respond_to(:persisted?)
     end
   end
 
-  context "when the embedded document has an array field" do
-
+  context 'when the embedded document has an array field' do
     let!(:person) do
       Person.create!
     end
 
     let!(:name) do
       person.create_name(
-          first_name: "Syd",
-          last_name: "Vicious",
-          aliases: nil
+        first_name: 'Syd',
+        last_name: 'Vicious',
+        aliases: nil
       )
     end
 
-    context "when saving the array on a persisted document" do
-
+    context 'when saving the array on a persisted document' do
       let(:from_db) do
         Person.find(person.id).name
       end
 
       before do
-        from_db.aliases = [ "Syd", "Sydney" ]
+        from_db.aliases = %w[Syd Sydney]
         from_db.save!
       end
 
-      it "sets the values of the array" do
-        expect(from_db.aliases).to eq([ "Syd", "Sydney" ])
+      it 'sets the values of the array' do
+        expect(from_db.aliases).to eq(%w[Syd Sydney])
       end
 
-      it "persists the array" do
-        expect(Person.find(person.id).name.aliases).to eq([ "Syd", "Sydney" ])
+      it 'persists the array' do
+        expect(Person.find(person.id).name.aliases).to eq(%w[Syd Sydney])
       end
     end
   end
 
-  context "when embedding a many under a one" do
-
+  context 'when embedding a many under a one' do
     let!(:person) do
       Person.create!
     end
@@ -890,14 +831,12 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
       person.create_name
     end
 
-    context "when the documents are reloaded from the database" do
-
+    context 'when the documents are reloaded from the database' do
       let(:from_db) do
         Person.first
       end
 
-      context "when adding a new many" do
-
+      context 'when adding a new many' do
         let(:name) do
           from_db.name
         end
@@ -906,48 +845,44 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           name.translations.new
         end
 
-        context "when saving the root" do
-
+        context 'when saving the root' do
           before do
             from_db.save!
           end
 
-          it "persists the new document on the first save" do
-            expect(from_db.reload.name.translations).to_not be_empty
+          it 'persists the new document on the first save' do
+            expect(from_db.reload.name.translations).not_to be_empty
           end
         end
       end
     end
   end
 
-  context "when embedding a one under a many" do
-
+  context 'when embedding a one under a many' do
     let!(:person) do
       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
-
+    context 'when a parent was removed outside of mongoid' do
       before do
         person.collection.find(_id: person.id).update_one(
-            "$pull" => { "addresses" => { _id: address_one.id }}
+          '$pull' => { 'addresses' => { _id: address_one.id } }
         )
       end
 
-      it "reloads the correct number" do
+      it 'reloads the correct number' do
         expect(person.reload.addresses.count).to eq(1)
       end
 
-      context "when adding a child" do
-
+      context 'when adding a child' do
         let(:code) do
           Code.new
         end
@@ -956,27 +891,25 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
           address_two.code = code
         end
 
-        it "reloads the correct number" do
+        it 'reloads the correct number' do
           expect(person.reload.addresses.count).to eq(1)
         end
       end
     end
   end
 
-  context "when embedded documents are stored without ids" do
-
+  context 'when embedded documents are stored without ids' do
     let!(:band) do
-      Band.create!(name: "Moderat")
+      Band.create!(name: 'Moderat')
     end
 
     before do
-      band.collection.
-          find(_id: band.id).
-          update_one("$set" => { label: { _id: BSON::ObjectId.new, name: "Mute" }})
+      band.collection
+          .find(_id: band.id)
+          .update_one('$set' => { label: { _id: BSON::ObjectId.new, name: 'Mute' } })
     end
 
-    context "when loading the documents" do
-
+    context 'when loading the documents' do
       before do
         band.reload
       end
@@ -985,33 +918,31 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
         band.label
       end
 
-      it "creates proper documents from the db" do
-        expect(label.name).to eq("Mute")
+      it 'creates proper documents from the db' do
+        expect(label.name).to eq('Mute')
       end
 
-      it "assigns ids to the documents" do
-        expect(label.id).to_not be_nil
+      it 'assigns ids to the documents' do
+        expect(label.id).not_to be_nil
       end
 
-      context "when subsequently updating the documents" do
-
+      context 'when subsequently updating the documents' do
         before do
-          label.update_attribute(:name, "Interscope")
+          label.update_attribute(:name, 'Interscope')
         end
 
-        it "updates the document" do
-          expect(label.name).to eq("Interscope")
+        it 'updates the document' do
+          expect(label.name).to eq('Interscope')
         end
 
-        it "persists the change" do
-          expect(label.reload.name).to eq("Interscope")
+        it 'persists the change' do
+          expect(label.reload.name).to eq('Interscope')
         end
       end
     end
   end
 
-  context "when parent validation of child is set to false" do
-
+  context 'when parent validation of child is set to false' do
     let(:building) do
       building = Building.create!
       building.building_address = BuildingAddress.new
@@ -1019,21 +950,21 @@ describe Mongoid::Association::Embedded::EmbedsOne::Proxy do
       building.reload
     end
 
-    it "parent successfully embeds an invalid child" do
+    it 'parent successfully embeds an invalid child' do
       expect(building.building_address).to be_a(BuildingAddress)
     end
   end
 
-  context "when assigning a hash" do
+  context 'when assigning a hash' do
     let(:building) { Building.create! }
 
     before do
-      building.building_address = { city: "NYC" }
+      building.building_address = { city: 'NYC' }
     end
 
-    it "creates the objects correctly" do
+    it 'creates the objects correctly' do
       expect(building.building_address).to be_a(BuildingAddress)
-      expect(building.building_address.city).to eq("NYC")
+      expect(building.building_address.city).to eq('NYC')
     end
   end
 end