mongoid 9.0.11 → 9.1.0

data/spec/mongoid/atomic/paths_spec.rb

@@ -1,16 +1,14 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "spec_helper"
+require 'spec_helper'
 
 describe Mongoid::Atomic::Paths do
-
   let(:person) do
     Person.new
   end
 
   let(:address) do
-    Address.new(street: "testing")
+    Address.new(street: 'testing')
   end
 
   let(:location) do
@@ -21,100 +19,86 @@ describe Mongoid::Atomic::Paths do
     Name.new
   end
 
-  describe "#atomic_delete_modifier" do
-
+  describe '#atomic_delete_modifier' do
     before do
       person.addresses << address
       person.name = name
     end
 
-    context "when document is an embeds_one" do
-
-      it "returns $unset" do
-        expect(name.atomic_delete_modifier).to eq("$unset")
+    context 'when document is an embeds_one' do
+      it 'returns $unset' do
+        expect(name.atomic_delete_modifier).to eq('$unset')
       end
     end
 
-    context "when document is an embeds_many" do
-
-      it "returns $pull" do
-        expect(address.atomic_delete_modifier).to eq("$pull")
+    context 'when document is an embeds_many' do
+      it 'returns $pull' do
+        expect(address.atomic_delete_modifier).to eq('$pull')
       end
     end
   end
 
-  describe "#atomic_insert_modifier" do
-
+  describe '#atomic_insert_modifier' do
     before do
       person.addresses << address
       person.name = name
     end
 
-    context "when document is an embeds_one" do
-
-      it "returns $set" do
-        expect(name.atomic_insert_modifier).to eq("$set")
+    context 'when document is an embeds_one' do
+      it 'returns $set' do
+        expect(name.atomic_insert_modifier).to eq('$set')
       end
     end
 
-    context "when document is an embeds_many" do
-
-      it "returns $push" do
-        expect(address.atomic_insert_modifier).to eq("$push")
+    context 'when document is an embeds_many' do
+      it 'returns $push' do
+        expect(address.atomic_insert_modifier).to eq('$push')
       end
     end
   end
 
-  describe "#atomic_path" do
-
-    context "when the document is a parent" do
-
-      it "returns an empty string" do
+  describe '#atomic_path' do
+    context 'when the document is a parent' do
+      it 'returns an empty string' do
         expect(person.atomic_path).to be_empty
       end
     end
 
-    context "when the document is embedded" do
-
+    context 'when the document is embedded' do
       before do
         person.addresses << address
       end
 
-      it "returns the inverse_of value of the association" do
-        expect(address.atomic_path).to eq("addresses")
+      it 'returns the inverse_of value of the association' do
+        expect(address.atomic_path).to eq('addresses')
       end
     end
 
-    context "when document embedded multiple levels" do
-
+    context 'when document embedded multiple levels' do
       before do
         address.locations << location
         person.addresses << address
       end
 
-      it "returns the JSON notation to the document" do
-        expect(location.atomic_path).to eq("addresses.locations")
+      it 'returns the JSON notation to the document' do
+        expect(location.atomic_path).to eq('addresses.locations')
       end
     end
   end
 
-  describe "#atomic_selector" do
-
-    context "when the document is a parent" do
-
-      it "returns an id.atomic_selector" do
-        expect(person.atomic_selector).to eq({ "_id" => person.id })
+  describe '#atomic_selector' do
+    context 'when the document is a parent' do
+      it 'returns an id.atomic_selector' do
+        expect(person.atomic_selector).to eq({ '_id' => person.id })
       end
     end
 
-    context "when the document is embedded" do
-
+    context 'when the document is embedded' do
       before do
         person.addresses << address
       end
 
       context 'when the parent is persisted' do
-
         let(:person) do
           Person.create!
         end
@@ -123,9 +107,9 @@ describe Mongoid::Atomic::Paths do
           person.should be_persisted
         end
 
-        it "returns the association with id.atomic_selector" do
+        it 'returns the association with id.atomic_selector' do
           expect(address.atomic_selector).to eq(
-            { "_id" => person.id, "addresses._id" => address.id }
+            { '_id' => person.id, 'addresses._id' => address.id }
           )
         end
       end
@@ -135,23 +119,21 @@ describe Mongoid::Atomic::Paths do
           person.should be_new_record
         end
 
-        it "returns the association with id.atomic_selector" do
+        it 'returns the association with id.atomic_selector' do
           expect(address.atomic_selector).to eq(
-            { "_id" => person.id, "addresses._id" => address.id }
+            { '_id' => person.id, 'addresses._id' => address.id }
           )
         end
       end
     end
 
-    context "when document embedded multiple levels" do
-
+    context 'when document embedded multiple levels' do
       before do
         address.locations << location
         person.addresses << address
       end
 
       context 'when the parent is persisted' do
-
         let(:person) do
           Person.create!
         end
@@ -160,12 +142,12 @@ describe Mongoid::Atomic::Paths do
           person.should be_persisted
         end
 
-        it "returns the JSON notation to the document with ids" do
+        it 'returns the JSON notation to the document with ids' do
           expect(location.atomic_selector).to eq(
             {
-              "_id" => person.id,
-              "addresses._id" => address.id,
-              "addresses.0.locations._id" => location.id
+              '_id' => person.id,
+              'addresses._id' => address.id,
+              'addresses.0.locations._id' => location.id
             }
           )
         end
@@ -176,14 +158,14 @@ describe Mongoid::Atomic::Paths do
           person.should be_new_record
         end
 
-        it "returns the JSON notation to the document with ids" do
+        it 'returns the JSON notation to the document with ids' do
           expect(location.atomic_selector).to eq(
             {
-              "_id" => person.id,
-              "addresses._id" => address.id,
+              '_id' => person.id,
+              'addresses._id' => address.id,
               # This condition is technically acceptable for finds
               # but probably won't work for modifications of 'locations'.
-              "addresses.locations._id" => location.id
+              'addresses.locations._id' => location.id
             }
           )
         end
@@ -191,42 +173,36 @@ describe Mongoid::Atomic::Paths do
     end
   end
 
-  describe "#atomic_position" do
-
-    context "when the document is a parent" do
-
-      it "returns an empty string" do
+  describe '#atomic_position' do
+    context 'when the document is a parent' do
+      it 'returns an empty string' do
         expect(person.atomic_position).to be_empty
       end
     end
 
-    context "when the document is embedded" do
-
+    context 'when the document is embedded' do
       before do
         person.addresses << address
       end
 
-      context "when the document is new" do
-
-        it "returns the.atomic_path without index" do
-          expect(address.atomic_position).to eq("addresses")
+      context 'when the document is new' do
+        it 'returns the.atomic_path without index' do
+          expect(address.atomic_position).to eq('addresses')
         end
       end
 
-      context "when the document is not new" do
-
+      context 'when the document is not new' do
         before do
           address.instance_variable_set(:@new_record, false)
         end
 
-        it "returns the.atomic_path plus index" do
-          expect(address.atomic_position).to eq("addresses.0")
+        it 'returns the.atomic_path plus index' do
+          expect(address.atomic_position).to eq('addresses.0')
         end
       end
     end
 
-    context "when document embedded multiple levels" do
-
+    context 'when document embedded multiple levels' do
       let(:other) do
         Location.new
       end
@@ -237,68 +213,60 @@ describe Mongoid::Atomic::Paths do
         person.addresses << address
       end
 
-      context "when the document is new" do
-
-        it "returns the.atomic_path with parent indexes" do
-          expect(location.atomic_position).to eq("addresses.0.locations")
+      context 'when the document is new' do
+        it 'returns the.atomic_path with parent indexes' do
+          expect(location.atomic_position).to eq('addresses.0.locations')
         end
       end
 
-      context "when the document is not new" do
-
+      context 'when the document is not new' do
         before do
           location.instance_variable_set(:@new_record, false)
         end
 
-        it "returns the.atomic_path plus index" do
-          expect(location.atomic_position).to eq("addresses.0.locations.1")
+        it 'returns the.atomic_path plus index' do
+          expect(location.atomic_position).to eq('addresses.0.locations.1')
         end
       end
     end
   end
 
-  describe "#atomic_path" do
-
-    context "when the document is a parent" do
-
-      it "returns an empty string" do
+  describe '#atomic_path' do
+    context 'when the document is a parent' do
+      it 'returns an empty string' do
         expect(person.atomic_path).to be_empty
       end
     end
 
-    context "when the document is embedded" do
-
+    context 'when the document is embedded' do
       before do
         person.addresses << address
       end
 
-      context "when the document is not new" do
-
+      context 'when the document is not new' do
         before do
           address.instance_variable_set(:@new_record, false)
         end
 
-        it "returns the.atomic_path without the index" do
-          expect(address.atomic_path).to eq("addresses")
+        it 'returns the.atomic_path without the index' do
+          expect(address.atomic_path).to eq('addresses')
         end
 
-        context "and there are 10 or more documents" do
-
+        context 'and there are 10 or more documents' do
           before do
             10.times do
               person.addresses << address
             end
           end
 
-          it "returns the.atomic_path without the index" do
-            expect(address.atomic_path).to eq("addresses")
+          it 'returns the.atomic_path without the index' do
+            expect(address.atomic_path).to eq('addresses')
           end
         end
       end
     end
 
-    context "when document embedded multiple levels" do
-
+    context 'when document embedded multiple levels' do
       let(:other) do
         Location.new
       end
@@ -309,41 +277,37 @@ describe Mongoid::Atomic::Paths do
         person.addresses << address
       end
 
-      context "when the document is not new" do
-
+      context 'when the document is not new' do
         before do
           location.instance_variable_set(:@new_record, false)
         end
 
-        it "returns the.atomic_path plus index" do
-          expect(location.atomic_path).to eq("addresses.0.locations")
+        it 'returns the.atomic_path plus index' do
+          expect(location.atomic_path).to eq('addresses.0.locations')
         end
       end
     end
 
-    context "when the same class is embedded in multiple associations" do
-
+    context 'when the same class is embedded in multiple associations' do
       let(:customer) do
         Customer.new
       end
 
-      context "assignment after saving" do
-
-        it "correctly sets the association for the embedded class" do
-
+      context 'assignment after saving' do
+        it 'correctly sets the association for the embedded class' do
           customer.home_address = CustomerAddress.new
           customer.work_address = CustomerAddress.new
 
-          expect(customer.home_address.atomic_path).to eq("home_address")
-          expect(customer.work_address.atomic_path).to eq("work_address")
+          expect(customer.home_address.atomic_path).to eq('home_address')
+          expect(customer.work_address.atomic_path).to eq('work_address')
 
           customer.save!
 
           customer.home_address = CustomerAddress.new
           customer.work_address = CustomerAddress.new
 
-          expect(customer.home_address.atomic_path).to eq("home_address")
-          expect(customer.work_address.atomic_path).to eq("work_address")
+          expect(customer.home_address.atomic_path).to eq('home_address')
+          expect(customer.work_address.atomic_path).to eq('work_address')
         end
       end
     end