mongoid 7.0.0 → 7.4.0

data/spec/mongoid/equality_spec.rb

@@ -1,4 +1,5 @@
-# -*- coding: utf-8 -*-
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Equality do
@@ -84,92 +85,194 @@ describe Mongoid::Equality do
 
   describe ".===" do
 
-    context "when comparable is an instance of this document" do
+    context "when legacy_triple_equals is set" do
+      config_override :legacy_triple_equals, true
+
+      context "when comparable is an instance of this document" do
 
-      it "returns true" do
-        expect(klass === person).to be true
+        it "returns true" do
+          expect(klass === person).to be true
+        end
       end
-    end
 
-    context "when comparable is a relation of this document" do
+      context "when comparable is a relation of this document" do
+
+        let(:relation) do
+          Post.new(person: person).person
+        end
 
-      let(:relation) do
-        Post.new(person: person).person
+        it "returns true" do
+          expect(klass === relation).to be true
+        end
       end
 
-      it "returns true" do
-        expect(klass === relation).to be true
+      context "when comparable is the same class" do
+
+        it "returns false" do
+          expect(klass === Person).to be false
+        end
       end
-    end
 
-    context "when comparable is the same class" do
+      context "when the comparable is a subclass" do
 
-      it "returns true" do
-        expect(klass === Person).to be true
+        it "returns false" do
+          expect(Person === Doctor).to be false
+        end
       end
-    end
 
-    context "when the comparable is a subclass" do
+      context "when the comparable is an instance of a subclass" do
 
-      it "returns false" do
-        expect(Person === Doctor).to be false
+        it "returns true" do
+          expect(Person === Doctor.new).to be true
+        end
       end
     end
 
-    context "when the comparable is an instance of a subclass" do
+    context "when legacy_triple_equals is not set" do
+      config_override :legacy_triple_equals, false
+
+      context "when comparable is an instance of this document" do
 
-      it "returns true" do
-        expect(Person === Doctor.new).to be true
+        it "returns true" do
+          expect(klass === person).to be true
+        end
       end
-    end
-  end
 
-  describe "#===" do
+      context "when comparable is a relation of this document" do
+
+        let(:relation) do
+          Post.new(person: person).person
+        end
+
+        it "returns true" do
+          expect(klass === relation).to be true
+        end
+      end
+
+      context "when comparable is the same class" do
 
-    context "when comparable is the same type" do
+        it "returns true" do
+          expect(klass === Person).to be true
+        end
+      end
 
-      context "when the instance is different" do
+      context "when the comparable is a subclass" do
 
         it "returns false" do
-          expect(person === Person.new).to be false
+          expect(Person === Doctor).to be false
         end
       end
 
-      context "when the instance is the same" do
+      context "when the comparable is an instance of a subclass" do
 
         it "returns true" do
-          expect(person === person).to be true
+          expect(Person === Doctor.new).to be true
         end
       end
     end
+  end
 
-    context "when the comparable is a subclass" do
+  describe "#===" do
 
-      it "returns false" do
-        expect(person === Doctor.new).to be false
+    context "when legacy_triple_equals is set" do
+      config_override :legacy_triple_equals, true
+
+      context "when comparable is the same type" do
+
+        context "when the instance is different" do
+
+          it "returns false" do
+            expect(person === Person.new).to be false
+          end
+        end
+
+        context "when the instance is the same" do
+
+          it "returns true" do
+            expect(person === person).to be true
+          end
+        end
+      end
+
+      context "when the comparable is a subclass" do
+
+        it "returns false" do
+          expect(person === Doctor.new).to be false
+        end
+      end
+
+      context "when comparing to a class" do
+
+        context "when the class is the same" do
+
+          it "returns false" do
+            expect(person === Person).to be false
+          end
+        end
+
+        context "when the class is a subclass" do
+
+          it "returns false" do
+            expect(person === Doctor).to be false
+          end
+        end
+
+        context "when the class is a superclass" do
+
+          it "returns false" do
+            expect(Doctor.new === Person).to be false
+          end
+        end
       end
     end
 
-    context "when comparing to a class" do
+    context "when legacy_triple_equals is not set" do
+      config_override :legacy_triple_equals, false
 
-      context "when the class is the same" do
+      context "when comparable is the same type" do
 
-        it "returns true" do
-          expect(person === Person).to be true
+        context "when the instance is different" do
+
+          it "returns false" do
+            expect(person === Person.new).to be false
+          end
+        end
+
+        context "when the instance is the same" do
+
+          it "returns true" do
+            expect(person === person).to be true
+          end
         end
       end
 
-      context "when the class is a subclass" do
+      context "when the comparable is a subclass" do
 
         it "returns false" do
-          expect(person === Doctor).to be false
+          expect(person === Doctor.new).to be false
         end
       end
 
-      context "when the class is a superclass" do
+      context "when comparing to a class" do
+        context "when the class is the same" do
 
-        it "returns true" do
-          expect(Doctor.new === Person).to be true
+          it "returns true" do
+            expect(person === Person).to be true
+          end
+        end
+
+        context "when the class is a subclass" do
+
+          it "returns false" do
+            expect(person === Doctor).to be false
+          end
+        end
+
+        context "when the class is a superclass" do
+
+          it "returns true" do
+            expect(Doctor.new === Person).to be true
+          end
         end
       end
     end

data/spec/mongoid/errors/ambiguous_relationship_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Mongoid::Errors::AmbiguousRelationship do
@@ -10,19 +12,19 @@ describe Mongoid::Errors::AmbiguousRelationship do
 
     it "contains the problem in the message" do
       expect(error.message).to include(
-        "Ambiguous relations :drugs, :evil_drugs defined on Person."
+        "Ambiguous associations :drugs, :evil_drugs defined on Person."
       )
     end
 
     it "contains the summary in the message" do
       expect(error.message).to include(
-        "When Mongoid attempts to set an inverse document of a relation in memory"
+        "When Mongoid attempts to set an inverse document of an association in memory"
       )
     end
 
     it "contains the resolution in the message" do
       expect(error.message).to include(
-        "On the :person relation on Drug you must add an :inverse_of option"
+        "On the :person association on Drug you must add an :inverse_of option"
       )
     end
   end

data/spec/mongoid/errors/callback_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::Callback do

data/spec/mongoid/errors/delete_restriction_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::DeleteRestriction do
@@ -10,13 +12,13 @@ describe Mongoid::Errors::DeleteRestriction do
 
     it "contains the problem in the message" do
       expect(error.message).to include(
-        "Cannot delete Person because of dependent 'drugs'."
+        "Cannot destroy Person because of dependent 'drugs'."
       )
     end
 
     it "contains the summary in the message" do
       expect(error.message).to include(
-        "When defining 'drugs' with a :dependent => :restrict,"
+        "When defining 'drugs' with a :dependent => :restrict_with_error,"
       )
     end
 

data/spec/mongoid/errors/document_not_destroyed_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::DocumentNotDestroyed do

data/spec/mongoid/errors/document_not_found_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::DocumentNotFound do

data/spec/mongoid/errors/eager_load_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::EagerLoad do

data/spec/mongoid/errors/invalid_collection_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::InvalidCollection do

data/spec/mongoid/errors/invalid_config_file_spec.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+describe Mongoid::Errors::InvalidConfigFile do
+
+  describe "#message" do
+
+    let(:error) do
+      described_class.new('/my/path')
+    end
+
+    it "contains the problem in the message" do
+      expect(error.message).to include(
+        "Invalid configuration file: /my/path."
+      )
+    end
+
+    it "contains the summary in the message" do
+      expect(error.message).to include(
+        "Your mongoid.yml configuration file does not contain the"
+      )
+    end
+
+    it "contains the resolution in the message" do
+      expect(error.message).to include(
+        "Ensure your configuration file contains the correct contents."
+      )
+    end
+  end
+end

data/spec/mongoid/errors/invalid_config_option_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::InvalidConfigOption do

data/spec/mongoid/errors/invalid_field_option_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::InvalidFieldOption do

data/spec/mongoid/errors/invalid_field_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::InvalidField do

data/spec/mongoid/errors/invalid_find_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::InvalidFind do

data/spec/mongoid/errors/invalid_includes_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::InvalidIncludes do

data/spec/mongoid/errors/invalid_index_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::InvalidIndex do

data/spec/mongoid/errors/invalid_options_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::InvalidOptions do
@@ -10,13 +12,13 @@ describe Mongoid::Errors::InvalidOptions do
 
     it "contains the problem in the message" do
       expect(error.message).to include(
-        "Invalid option :invalid provided to relation :name."
+        "Invalid option :invalid provided to association :name."
       )
     end
 
     it "contains the summary in the message" do
       expect(error.message).to include(
-        "Mongoid checks the options that are passed to the relation macros"
+        "Mongoid checks the options that are passed to the association macros"
       )
     end
 

data/spec/mongoid/errors/invalid_path_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Mongoid::Errors::InvalidPath do

data/spec/mongoid/errors/invalid_relation_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::InvalidRelation do
@@ -18,13 +20,13 @@ describe Mongoid::Errors::InvalidRelation do
 
     it "contains the problem in the message" do
       expect(error.message).to include(
-                                   "Defining a relation named 'crazy_method' is not allowed."
+                                   "Defining an association named 'crazy_method' is not allowed."
                                )
     end
 
     it "contains the summary in the message" do
       expect(error.message).to include(
-                                   "Defining this relation would override the method 'crazy_method'"
+                                   "Defining this association would override the method 'crazy_method'"
                                )
     end
 

data/spec/mongoid/errors/invalid_scope_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::InvalidScope do
@@ -22,7 +24,7 @@ describe Mongoid::Errors::InvalidScope do
 
     it "contains the resolution in the message" do
       expect(error.message).to include(
-        "Change the scope to be a proc wrapped critera."
+        "Change the scope to be a proc wrapped criteria."
       )
     end
   end

data/spec/mongoid/errors/invalid_set_polymorphic_relation_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::InvalidSetPolymorphicRelation do

data/spec/mongoid/errors/invalid_storage_options_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::InvalidStorageOptions do

data/spec/mongoid/errors/invalid_time_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::InvalidTime do

data/spec/mongoid/errors/inverse_not_found_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::InverseNotFound do
@@ -16,7 +18,7 @@ describe Mongoid::Errors::InverseNotFound do
 
     it "contains the summary in the message" do
       expect(error.message).to include(
-        "hen adding a document to a relation, Mongoid attempts to link"
+        "hen adding a document to an association, Mongoid attempts to link"
       )
     end
 

data/spec/mongoid/errors/mixed_client_configuration_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::MixedClientConfiguration do

data/spec/mongoid/errors/mixed_relations_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::MixedRelations do

data/spec/mongoid/errors/mongoid_error_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Mongoid::Errors::MongoidError do
@@ -7,14 +9,26 @@ describe Mongoid::Errors::MongoidError do
   let(:options) { {} }
 
   before do
-    {"message_title" => "message", "summary_title" => "summary", "resolution_title" => "resolution"}.each do |key, name|
-      expect(::I18n).to receive(:translate).with("mongoid.errors.messages.#{key}", {}).and_return(name)
-    end
-
-    ["message", "summary", "resolution"].each do |name|
-      expect(::I18n).to receive(:translate).
-        with("mongoid.errors.messages.#{key}.#{name}", {}).
-      and_return(name)
+    if RUBY_VERSION.start_with?('3.', '2.7')
+      {"message_title" => "message", "summary_title" => "summary", "resolution_title" => "resolution"}.each do |key, name|
+        expect(::I18n).to receive(:translate).with("mongoid.errors.messages.#{key}", **{}).and_return(name)
+      end
+
+      ["message", "summary", "resolution"].each do |name|
+        expect(::I18n).to receive(:translate).
+          with("mongoid.errors.messages.#{key}.#{name}", **{}).
+          and_return(name)
+      end
+    else
+      {"message_title" => "message", "summary_title" => "summary", "resolution_title" => "resolution"}.each do |key, name|
+        expect(::I18n).to receive(:translate).with("mongoid.errors.messages.#{key}", {}).and_return(name)
+      end
+
+      ["message", "summary", "resolution"].each do |name|
+        expect(::I18n).to receive(:translate).
+          with("mongoid.errors.messages.#{key}.#{name}", {}).
+          and_return(name)
+      end
     end
 
     error.compose_message(key, options)

data/spec/mongoid/errors/nested_attributes_metadata_not_found_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::NestedAttributesMetadataNotFound do
@@ -10,19 +12,19 @@ describe Mongoid::Errors::NestedAttributesMetadataNotFound do
 
     it "contains the problem in the message" do
       expect(error.message).to include(
-        "Could not find metadata for relation 'posts' on model: Person."
+        "Could not find metadata for association 'posts' on model: Person."
       )
     end
 
     it "contains the summary in the message" do
       expect(error.message).to include(
-        "When defining nested attributes for a relation, Mongoid needs"
+        "When defining nested attributes for an association, Mongoid needs"
       )
     end
 
     it "contains the resolution in the message" do
       expect(error.message).to include(
-        "Make sure that there is a relation defined named 'posts' on Person"
+        "Make sure that there is an association defined named 'posts' on Person"
       )
     end
   end

data/spec/mongoid/errors/no_client_config_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::NoClientConfig do
@@ -5,12 +7,12 @@ describe Mongoid::Errors::NoClientConfig do
   describe "#message" do
 
     let(:error) do
-      described_class.new(:secondary)
+      described_class.new(:analytics)
     end
 
     it "contains the problem in the message" do
       expect(error.message).to include(
-        "No configuration could be found for a client named 'secondary'."
+        "No configuration could be found for a client named 'analytics'."
       )
     end
 

data/spec/mongoid/errors/no_client_database_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Errors::NoClientDatabase do
@@ -5,12 +7,12 @@ describe Mongoid::Errors::NoClientDatabase do
   describe "#message" do
 
     let(:error) do
-      described_class.new(:secondary, { hosts: [ "127.0.0.1:27017" ] })
+      described_class.new(:analytics, { hosts: [ "127.0.0.1:27017" ] })
     end
 
     it "contains the problem in the message" do
       expect(error.message).to include(
-        "No database provided for client configuration: :secondary."
+        "No database provided for client configuration: :analytics."
       )
     end
 
@@ -22,7 +24,7 @@ describe Mongoid::Errors::NoClientDatabase do
 
     it "contains the resolution in the message" do
       expect(error.message).to include(
-        "If configuring via a mongoid.yml, ensure that within your :secondary"
+        "If configuring via a mongoid.yml, ensure that within your :analytics"
       )
     end
   end