mongoid 9.0.11 → 9.1.0

data/spec/mongoid/threaded_spec.rb

@@ -1,40 +1,34 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "spec_helper"
+require 'spec_helper'
 
 describe Mongoid::Threaded do
-
   let(:object) do
     double
   end
 
-  describe "#begin" do
-
+  describe '#begin' do
     before do
-      described_class.begin_execution("load")
+      described_class.begin_execution('load')
     end
 
     after do
-      described_class.stack("load").clear
+      described_class.stack('load').clear
     end
 
-    it "adds a boolean to the load stack" do
-      expect(described_class.stack("load")).to eq([ true ])
+    it 'adds a boolean to the load stack' do
+      expect(described_class.stack('load')).to eq([ true ])
     end
   end
 
-  describe "#executing?" do
-
-    context "when loading is not set" do
-
-      it "returns false" do
-        expect(described_class).to_not be_executing(:load)
+  describe '#executing?' do
+    context 'when loading is not set' do
+      it 'returns false' do
+        expect(described_class).not_to be_executing(:load)
       end
     end
 
-    context "when the stack has elements" do
-
+    context 'when the stack has elements' do
       before do
         described_class.stack('load').push(true)
       end
@@ -43,74 +37,68 @@ describe Mongoid::Threaded do
         described_class.stack('load').clear
       end
 
-      it "returns true" do
+      it 'returns true' do
         expect(described_class).to be_executing(:load)
       end
     end
 
-    context "when the stack has no elements" do
-
+    context 'when the stack has no elements' do
       before do
         described_class.stack('load').clear
       end
 
-      it "returns false" do
-        expect(described_class).to_not be_executing(:load)
+      it 'returns false' do
+        expect(described_class).not_to be_executing(:load)
       end
     end
   end
 
-  describe "#stack" do
-
-    context "when no stack has been initialized" do
-
+  describe '#stack' do
+    context 'when no stack has been initialized' do
       let(:loading) do
-        described_class.stack("load")
+        described_class.stack('load')
       end
 
-      it "returns an empty stack" do
+      it 'returns an empty stack' do
         expect(loading).to be_empty
       end
     end
 
-    context "when a stack has been initialized" do
-
+    context 'when a stack has been initialized' do
       before do
         described_class.stack('load').push(true)
       end
 
       let(:loading) do
-        described_class.stack("load")
+        described_class.stack('load')
       end
 
       after do
         described_class.stack('load').clear
       end
 
-      it "returns the stack" do
+      it 'returns the stack' do
         expect(loading).to eq([ true ])
       end
     end
   end
 
-  describe "#exit" do
-
+  describe '#exit' do
     before do
-      described_class.begin_execution("load")
-      described_class.exit_execution("load")
+      described_class.begin_execution('load')
+      described_class.exit_execution('load')
     end
 
     after do
-      described_class.stack("load").clear
+      described_class.stack('load').clear
     end
 
-    it "removes a boolean from the stack" do
-      expect(described_class.stack("load")).to be_empty
+    it 'removes a boolean from the stack' do
+      expect(described_class.stack('load')).to be_empty
     end
   end
 
-  describe "#begin_validate" do
-
+  describe '#begin_validate' do
     let(:person) do
       Person.new
     end
@@ -123,13 +111,12 @@ describe Mongoid::Threaded do
       described_class.exit_validate(person)
     end
 
-    it "marks the document as being validated" do
+    it 'marks the document as being validated' do
       expect(described_class.validations_for(Person)).to eq([ person.id ])
     end
   end
 
-  describe "#exit_validate" do
-
+  describe '#exit_validate' do
     let(:person) do
       Person.new
     end
@@ -139,19 +126,17 @@ describe Mongoid::Threaded do
       described_class.exit_validate(person)
     end
 
-    it "unmarks the document as being validated" do
+    it 'unmarks the document as being validated' do
       expect(described_class.validations_for(Person)).to be_empty
     end
   end
 
-  describe "#validated?" do
-
+  describe '#validated?' do
     let(:person) do
       Person.new
     end
 
-    context "when the document is validated" do
-
+    context 'when the document is validated' do
       before do
         described_class.begin_validate(person)
       end
@@ -160,21 +145,19 @@ describe Mongoid::Threaded do
         described_class.exit_validate(person)
       end
 
-      it "returns true" do
+      it 'returns true' do
         expect(described_class.validated?(person)).to be true
       end
     end
 
-    context "when the document is not validated" do
-
-      it "returns false" do
+    context 'when the document is not validated' do
+      it 'returns false' do
         expect(described_class.validated?(person)).to be false
       end
     end
   end
 
-  describe "#begin_autosave" do
-
+  describe '#begin_autosave' do
     let(:person) do
       Person.new
     end
@@ -187,13 +170,12 @@ describe Mongoid::Threaded do
       described_class.exit_autosave(person)
     end
 
-    it "marks the document as being autosaved" do
+    it 'marks the document as being autosaved' do
       expect(described_class.autosaves_for(Person)).to eq([ person.id ])
     end
   end
 
-  describe "#exit_autosave" do
-
+  describe '#exit_autosave' do
     let(:person) do
       Person.new
     end
@@ -203,19 +185,17 @@ describe Mongoid::Threaded do
       described_class.exit_autosave(person)
     end
 
-    it "unmarks the document as being autosaved" do
+    it 'unmarks the document as being autosaved' do
       expect(described_class.autosaves_for(Person)).to be_empty
     end
   end
 
-  describe "#autosaved?" do
-
+  describe '#autosaved?' do
     let(:person) do
       Person.new
     end
 
-    context "when the document is autosaved" do
-
+    context 'when the document is autosaved' do
       before do
         described_class.begin_autosave(person)
       end
@@ -224,21 +204,19 @@ describe Mongoid::Threaded do
         described_class.exit_autosave(person)
       end
 
-      it "returns true" do
+      it 'returns true' do
         expect(described_class.autosaved?(person)).to be true
       end
     end
 
-    context "when the document is not autosaved" do
-
-      it "returns false" do
+    context 'when the document is not autosaved' do
+      it 'returns false' do
         expect(described_class.autosaved?(person)).to be false
       end
     end
   end
 
-  describe "#begin_without_default_scope" do
-
+  describe '#begin_without_default_scope' do
     let(:klass) do
       Appointment
     end
@@ -247,15 +225,14 @@ describe Mongoid::Threaded do
       described_class.exit_without_default_scope(klass)
     end
 
-    it "adds the given class to the without_default_scope stack" do
+    it 'adds the given class to the without_default_scope stack' do
       described_class.begin_without_default_scope(klass)
 
       expect(described_class.stack(:without_default_scope)).to include(klass)
     end
   end
 
-  describe "#exit_without_default_scope" do
-
+  describe '#exit_without_default_scope' do
     let(:klass) do
       Appointment
     end
@@ -264,21 +241,19 @@ describe Mongoid::Threaded do
       described_class.begin_without_default_scope(klass)
     end
 
-    it "removes the given class from the without_default_scope stack" do
+    it 'removes the given class from the without_default_scope stack' do
       described_class.exit_without_default_scope(klass)
 
       expect(described_class.stack(:without_default_scope)).not_to include(klass)
     end
   end
 
-  describe "#without_default_scope?" do
-
+  describe '#without_default_scope?' do
     let(:klass) do
       Appointment
     end
 
-    context "when klass has begun without_default_scope" do
-
+    context 'when klass has begun without_default_scope' do
       before do
         described_class.begin_without_default_scope(klass)
       end
@@ -287,30 +262,29 @@ describe Mongoid::Threaded do
         described_class.exit_without_default_scope(klass)
       end
 
-      it "returns true" do
+      it 'returns true' do
         expect(described_class.without_default_scope?(klass)).to be(true)
       end
     end
 
-    context "when klass has exited without_default_scope" do
-
+    context 'when klass has exited without_default_scope' do
       before do
         described_class.begin_without_default_scope(klass)
         described_class.exit_without_default_scope(klass)
       end
 
-      it "returns false" do
+      it 'returns false' do
         expect(described_class.without_default_scope?(klass)).to be(false)
       end
     end
   end
 
-  describe "#get_session" do
+  describe '#get_session' do
     let(:session) do
       double('session')
     end
 
-    context "without client specified" do
+    context 'without client specified' do
       before do
         described_class.set_session(session)
       end

data/spec/mongoid/timestamps/created/short_spec.rb

@@ -1,12 +1,9 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "spec_helper"
+require 'spec_helper'
 
 describe Mongoid::Timestamps::Created::Short do
-
-  describe ".included" do
-
+  describe '.included' do
     let(:quiz) do
       ShortQuiz.new
     end
@@ -20,34 +17,33 @@ describe Mongoid::Timestamps::Created::Short do
       quiz.run_callbacks(:save)
     end
 
-    it "adds c_at to the document" do
-      expect(fields["c_at"]).to_not be_nil
+    it 'adds c_at to the document' do
+      expect(fields['c_at']).not_to be_nil
     end
 
-    it "does not add u_at to the document" do
-      expect(fields["u_at"]).to be_nil
+    it 'does not add u_at to the document' do
+      expect(fields['u_at']).to be_nil
     end
 
-    it "does not add the created_at to the document" do
-      expect(fields["created_at"]).to be_nil
+    it 'does not add the created_at to the document' do
+      expect(fields['created_at']).to be_nil
     end
 
-    it "forces the c_at timestamps to UTC" do
+    it 'forces the c_at timestamps to UTC' do
       expect(quiz.created_at).to be_within(10).of(Time.now.utc)
     end
   end
 
-  context "when the document is created" do
-
+  context 'when the document is created' do
     let(:quiz) do
       ShortQuiz.create!
     end
 
-    it "runs the created callbacks" do
+    it 'runs the created callbacks' do
       expect(quiz.created_at).to be_within(10).of(Time.now.utc)
     end
 
-    it "allows access via the raw field" do
+    it 'allows access via the raw field' do
       expect(quiz.c_at).to eq(quiz.created_at)
     end
   end

data/spec/mongoid/timestamps/created_spec.rb

@@ -1,12 +1,9 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "spec_helper"
+require 'spec_helper'
 
 describe Mongoid::Timestamps::Created do
-
-  describe ".included" do
-
+  describe '.included' do
     let(:quiz) do
       Quiz.new
     end
@@ -20,39 +17,38 @@ describe Mongoid::Timestamps::Created do
       quiz.run_callbacks(:save)
     end
 
-    it "adds created_at to the document" do
-      expect(fields["created_at"]).to_not be_nil
+    it 'adds created_at to the document' do
+      expect(fields['created_at']).not_to be_nil
     end
 
-    it "does not add updated_at to the document" do
-      expect(fields["updated_at"]).to be_nil
+    it 'does not add updated_at to the document' do
+      expect(fields['updated_at']).to be_nil
     end
 
-    it "forces the created_at timestamps to UTC" do
+    it 'forces the created_at timestamps to UTC' do
       expect(quiz.created_at).to be_within(10).of(Time.now.utc)
     end
   end
 
-  context "when the document is created" do
-
+  context 'when the document is created' do
     let(:quiz) do
       Quiz.create!
     end
 
-    it "runs the created callbacks" do
-      expect(quiz.created_at).to_not be_nil
+    it 'runs the created callbacks' do
+      expect(quiz.created_at).not_to be_nil
       expect(quiz.created_at).to be_within(10).of(Time.now.utc)
     end
   end
 
-  context "when the document is destroyed" do
+  context 'when the document is destroyed' do
     let(:book) do
       Book.create!
     end
 
     before do
       Cover.before_save do
-        destroy if title == "delete me"
+        destroy if title == 'delete me'
       end
     end
 
@@ -60,11 +56,11 @@ describe Mongoid::Timestamps::Created do
       Cover.reset_callbacks(:save)
     end
 
-    it "does not set the created_at timestamp" do
-      book.covers << Cover.new(title: "delete me")
-      expect {
+    it 'does not set the created_at timestamp' do
+      book.covers << Cover.new(title: 'delete me')
+      expect do
         book.save
-      }.not_to raise_error
+      end.not_to raise_error
     end
   end
 end

data/spec/mongoid/timestamps/timeless_spec.rb

@@ -1,12 +1,9 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "spec_helper"
+require 'spec_helper'
 
 describe Mongoid::Timestamps::Timeless do
-
-  describe "#timeless" do
-
+  describe '#timeless' do
     before(:all) do
       class Chicken
         include Mongoid::Document
@@ -30,8 +27,7 @@ describe Mongoid::Timestamps::Timeless do
       Object.send(:remove_const, :Egg)
     end
 
-    context "when timeless is used on one instance and then not used on another instance" do
-
+    context 'when timeless is used on one instance and then not used on another instance' do
       let!(:first_instance) do
         egg = Egg.create!
         egg.timeless.save!
@@ -43,47 +39,42 @@ describe Mongoid::Timestamps::Timeless do
       end
 
       it "second instance's created_at is not nil" do
-        expect(second_instance.created_at).to_not be_nil
+        expect(second_instance.created_at).not_to be_nil
       end
     end
 
-    context "when others persist in the scope of the chain" do
-
-      context "when the root executes normally" do
-
+    context 'when others persist in the scope of the chain' do
+      context 'when the root executes normally' do
         let!(:chicken) do
           Chicken.create!
         end
 
-        it "creates the parent with a timestamp" do
-          expect(chicken.created_at).to_not be_nil
+        it 'creates the parent with a timestamp' do
+          expect(chicken.created_at).not_to be_nil
         end
 
-        it "creates the child with no timestamp" do
+        it 'creates the child with no timestamp' do
           expect(Egg.last.created_at).to be_nil
         end
       end
 
-      context "when the root executes timeless" do
-
+      context 'when the root executes timeless' do
         let!(:chicken) do
           Chicken.timeless.create!
         end
 
-        it "creates the parent with a timestamp" do
+        it 'creates the parent with a timestamp' do
           expect(chicken.created_at).to be_nil
         end
 
-        it "creates the child with no timestamp" do
+        it 'creates the child with no timestamp' do
           expect(Egg.last.created_at).to be_nil
         end
       end
     end
 
-    context "when used as a proxy method" do
-
-      context "when used on the document instance" do
-
+    context 'when used as a proxy method' do
+      context 'when used on the document instance' do
         let(:document) do
           Dokument.new
         end
@@ -92,56 +83,53 @@ describe Mongoid::Timestamps::Timeless do
           document.timeless.save!
         end
 
-        it "does not set the created timestamp" do
+        it 'does not set the created timestamp' do
           expect(document.created_at).to be_nil
         end
 
-        it "does not set the updated timestamp" do
+        it 'does not set the updated timestamp' do
           expect(document.updated_at).to be_nil
         end
 
-        it "clears out the timeless option after save" do
-          expect(document).to_not be_timeless
+        it 'clears out the timeless option after save' do
+          expect(document).not_to be_timeless
         end
 
-        context "when subsequently persisting" do
-
+        context 'when subsequently persisting' do
           before do
-            document.update_attribute(:title, "Sir")
+            document.update_attribute(:title, 'Sir')
           end
 
-          it "sets the updated timestamp" do
-            expect(document.updated_at).to_not be_nil
+          it 'sets the updated timestamp' do
+            expect(document.updated_at).not_to be_nil
           end
         end
       end
 
-      context "when used on the class" do
-
+      context 'when used on the class' do
         let!(:document) do
           Dokument.timeless.create!
         end
 
-        it "does not set the created timestamp" do
+        it 'does not set the created timestamp' do
           expect(document.created_at).to be_nil
         end
 
-        it "does not set the updated timestamp" do
+        it 'does not set the updated timestamp' do
           expect(document.updated_at).to be_nil
         end
 
-        it "clears out the timeless option after save" do
-          expect(document).to_not be_timeless
+        it 'clears out the timeless option after save' do
+          expect(document).not_to be_timeless
         end
 
-        context "when subsequently persisting" do
-
+        context 'when subsequently persisting' do
           before do
-            document.update_attribute(:title, "Sir")
+            document.update_attribute(:title, 'Sir')
           end
 
-          it "sets the updated timestamp" do
-            expect(document.updated_at).to_not be_nil
+          it 'sets the updated timestamp' do
+            expect(document.updated_at).not_to be_nil
           end
         end
       end