mongoid 9.0.11 → 9.1.1

data/spec/mongoid/config_spec.rb

@@ -1,32 +1,27 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "spec_helper"
-require "support/feature_sandbox"
+require 'spec_helper'
+require 'support/feature_sandbox'
 
 describe Mongoid::Config do
-
   after do
     Mongoid.configure do |config|
       config.load_configuration(CONFIG)
     end
   end
 
-  describe "#configured?" do
-
+  describe '#configured?' do
     after do
       described_class.connect_to(database_id, read: :primary)
     end
 
-    context "when a default client config exists" do
-
-      context "when a default database is configured" do
-
+    context 'when a default client config exists' do
+      context 'when a default database is configured' do
         let(:config) do
           {
             default: {
               database: database_id,
-              hosts: [ "127.0.0.1:27017" ]
+              hosts: [ '127.0.0.1:27017' ]
             }
           }
         end
@@ -35,47 +30,43 @@ describe Mongoid::Config do
           described_class.send(:clients=, config)
         end
 
-        it "returns true" do
+        it 'returns true' do
           expect(described_class).to be_configured
         end
       end
     end
 
-    context "when no default client config exists" do
-
+    context 'when no default client config exists' do
       before do
         described_class.clients.clear
       end
 
-      it "returns false" do
-        expect(described_class).to_not be_configured
+      it 'returns false' do
+        expect(described_class).not_to be_configured
       end
     end
   end
 
-  describe "#destructive_fields" do
-
+  describe '#destructive_fields' do
     Mongoid::Composable.prohibited_methods.each do |method|
-
       it "contains #{method}" do
         expect(described_class.destructive_fields).to include(method)
       end
     end
   end
 
-  context "when the log level is not set in the configuration" do
-
+  context 'when the log level is not set in the configuration' do
     before do
       Mongoid.configure do |config|
         config.load_configuration(CONFIG)
       end
     end
 
-    it "sets the Mongoid logger level to the default" do
+    it 'sets the Mongoid logger level to the default' do
       expect(Mongoid.logger.level).to eq(Logger::INFO)
     end
 
-    it "sets the Mongo driver logger level to the default" do
+    it 'sets the Mongo driver logger level to the default' do
       expect(Mongo::Logger.logger.level).to eq(Logger::INFO)
     end
   end
@@ -95,7 +86,7 @@ describe Mongoid::Config do
     context 'is set in the config' do
       it 'sets the value' do
         Mongoid::Config.reset
-        configuration = CONFIG.merge(options: {background_indexing: true})
+        configuration = CONFIG.merge(options: { background_indexing: true })
 
         Mongoid.configure { |config| config.load_configuration(configuration) }
 
@@ -105,7 +96,6 @@ describe Mongoid::Config do
   end
 
   context 'when the belongs_to_required_by_default option is not set in the config' do
-
     before do
       Mongoid::Config.reset
       Mongoid.configure do |config|
@@ -119,7 +109,6 @@ describe Mongoid::Config do
   end
 
   context 'when the belongs_to_required_by_default option is set in the config' do
-
     before do
       Mongoid.configure do |config|
         config.load_configuration(conf)
@@ -127,7 +116,6 @@ describe Mongoid::Config do
     end
 
     context 'when the value is set to true' do
-
       let(:conf) do
         CONFIG.merge(options: { belongs_to_required_by_default: true })
       end
@@ -138,7 +126,6 @@ describe Mongoid::Config do
     end
 
     context 'when the value is set to false' do
-
       let(:conf) do
         CONFIG.merge(options: { belongs_to_required_by_default: false })
       end
@@ -157,7 +144,6 @@ describe Mongoid::Config do
   end
 
   context 'when the app_name is set in the config' do
-
     let(:conf) do
       CONFIG.merge(options: { app_name: 'admin-reporting' })
     end
@@ -174,7 +160,6 @@ describe Mongoid::Config do
   end
 
   context 'when the app_name is not set in the config' do
-
     before do
       Mongoid::Config.reset
       Mongoid.configure do |config|
@@ -195,22 +180,22 @@ describe Mongoid::Config do
 
         Mongoid.configure { |config| config.load_configuration(configuration) }
 
-        expect(Mongoid::Config.discriminator_key).to eq("_type")
+        expect(Mongoid::Config.discriminator_key).to eq('_type')
       end
     end
 
     context 'is set in the config' do
       it 'sets the value' do
         Mongoid::Config.reset
-        configuration = CONFIG.merge(options: {discriminator_key: "test"})
+        configuration = CONFIG.merge(options: { discriminator_key: 'test' })
 
         Mongoid.configure { |config| config.load_configuration(configuration) }
 
-        expect(Mongoid::Config.discriminator_key).to eq("test")
+        expect(Mongoid::Config.discriminator_key).to eq('test')
       end
 
       it 'is set globally' do
-        expect(Mongoid.discriminator_key).to eq("test")
+        expect(Mongoid.discriminator_key).to eq('test')
       end
     end
   end
@@ -225,33 +210,30 @@ describe Mongoid::Config do
       end
     end
 
-    context "when it is not set in the config" do
-
+    context 'when it is not set in the config' do
       let(:conf) { CONFIG }
 
-      it "it is set to its default" do
+      it 'is set to its default' do
         expect(Mongoid.send(option)).to eq(:immediate)
       end
     end
 
     context 'when the value is :immediate' do
-
       let(:conf) do
         CONFIG.merge(options: { option => :immediate })
       end
 
-      it "is set to false" do
+      it 'is set to false' do
         expect(Mongoid.send(option)).to be(:immediate)
       end
     end
 
     context 'when the value is :global_thread_pool' do
-
       let(:conf) do
         CONFIG.merge(options: { option => :global_thread_pool })
       end
 
-      it "is set to false" do
+      it 'is set to false' do
         expect(Mongoid.send(option)).to be(:global_thread_pool)
       end
     end
@@ -267,29 +249,26 @@ describe Mongoid::Config do
       end
     end
 
-    context "when it is not set in the config" do
-
+    context 'when it is not set in the config' do
       let(:conf) { CONFIG }
 
-      it "it is set to its default" do
+      it 'is set to its default' do
         expect(Mongoid.send(option)).to eq(nil)
       end
     end
 
     context 'when the value is set to a number' do
-
       let(:conf) do
         CONFIG.merge(options: { option => 5 })
       end
 
-      it "is set to the number" do
+      it 'is set to the number' do
         expect(Mongoid.send(option)).to be(5)
       end
     end
   end
 
-  shared_examples "a config option" do
-
+  shared_examples 'a config option' do
     before do
       Mongoid::Config.reset
       Mongoid.configure do |config|
@@ -298,32 +277,29 @@ describe Mongoid::Config do
     end
 
     context 'when the value is false' do
-
       let(:conf) do
         CONFIG.merge(options: { option => false })
       end
 
-      it "is set to false" do
+      it 'is set to false' do
         expect(Mongoid.send(option)).to be(false)
       end
     end
 
     context 'when the value is true' do
-
       let(:conf) do
         CONFIG.merge(options: { option => true })
       end
 
-      it "is set to true" do
+      it 'is set to true' do
         expect(Mongoid.send(option)).to be(true)
       end
     end
 
-    context "when it is not set in the config" do
-
+    context 'when it is not set in the config' do
       let(:conf) { CONFIG }
 
-      it "it is set to its default" do
+      it 'is set to its default' do
         expect(Mongoid.send(option)).to eq(default)
       end
     end
@@ -333,7 +309,7 @@ describe Mongoid::Config do
     let(:option) { :map_big_decimal_to_decimal128 }
     let(:default) { true }
 
-    it_behaves_like "a config option"
+    it_behaves_like 'a config option'
   end
 
   context 'when setting the allow_bson5_decimal128 option in the config' do
@@ -342,33 +318,31 @@ describe Mongoid::Config do
     let(:option) { :allow_bson5_decimal128 }
     let(:default) { false }
 
-    it_behaves_like "a config option"
+    it_behaves_like 'a config option'
   end
 
   context 'when setting the legacy_readonly option in the config' do
     let(:option) { :legacy_readonly }
     let(:default) { false }
 
-    it_behaves_like "a config option"
+    it_behaves_like 'a config option'
   end
 
   context 'when setting the legacy_persistence_context_behavior option in the config' do
     let(:option) { :legacy_persistence_context_behavior }
     let(:default) { false }
 
-    it_behaves_like "a config option"
+    it_behaves_like 'a config option'
   end
 
-  describe "#load!" do
-
+  describe '#load!' do
     let(:file) do
-      File.join(File.dirname(__FILE__), "..", "config", "mongoid.yml")
+      File.join(File.dirname(__FILE__), '..', 'config', 'mongoid.yml')
     end
 
-    context "when existing clients exist in the configuration" do
-
+    context 'when existing clients exist in the configuration' do
       let(:client) do
-        Mongo::Client.new([ "127.0.0.1:27017" ])
+        Mongo::Client.new([ '127.0.0.1:27017' ])
       end
 
       before do
@@ -380,37 +354,35 @@ describe Mongoid::Config do
         client.close
       end
 
-      it "clears the previous clients" do
+      it 'clears the previous clients' do
         expect(Mongoid::Clients.clients[:test]).to be_nil
       end
     end
 
-    context "when the log level is set in the configuration" do
-
+    context 'when the log level is set in the configuration' do
       before do
         described_class.load!(file, :test)
       end
 
-      it "sets the Mongoid logger level" do
+      it 'sets the Mongoid logger level' do
         expect(Mongoid.logger.level).to eq(Logger::WARN)
       end
 
-      it "sets the Mongo driver logger level" do
+      it 'sets the Mongo driver logger level' do
         expect(Mongo::Logger.logger.level).to eq(Logger::WARN)
       end
 
-      context "when in a Rails environment" do
-
+      context 'when in a Rails environment' do
         around do |example|
           FeatureSandbox.quarantine do
-            require "support/rails_mock"
+            require 'support/rails_mock'
             Mongoid.logger = Rails.logger
             described_class.load!(file, :test)
             example.run
           end
         end
 
-        it "keeps the Mongoid logger level the same as the Rails logger" do
+        it 'keeps the Mongoid logger level the same as the Rails logger' do
           expect(Mongoid.logger.level).to eq(Rails.logger.level)
           expect(Mongoid.logger.level).not_to eq(Mongoid::Config.log_level)
         end
@@ -421,8 +393,7 @@ describe Mongoid::Config do
       end
     end
 
-    context "when provided an environment" do
-
+    context 'when provided an environment' do
       before do
         described_class.load!(file, :test)
       end
@@ -431,37 +402,36 @@ describe Mongoid::Config do
         described_class.reset
       end
 
-      it "sets the include root in json option" do
+      it 'sets the include root in json option' do
         expect(described_class.include_root_in_json).to be false
       end
 
-      it "sets the include type with serialization option" do
+      it 'sets the include type with serialization option' do
         expect(described_class.include_type_for_serialization).to be false
       end
 
-      it "sets the scope overwrite option" do
+      it 'sets the scope overwrite option' do
         expect(described_class.scope_overwrite_exception).to be false
       end
 
-      it "sets the preload models option" do
+      it 'sets the preload models option' do
         expect(described_class.preload_models).to be false
       end
 
-      it "sets the raise not found error option" do
+      it 'sets the raise not found error option' do
         expect(described_class.raise_not_found_error).to be true
       end
 
-      it "sets the use utc option" do
+      it 'sets the use utc option' do
         expect(described_class.use_utc).to be false
       end
 
-      it "sets the join_contexts default option" do
+      it 'sets the join_contexts default option' do
         expect(described_class.join_contexts).to be false
       end
     end
 
-    context "when provided an environment with driver options" do
-
+    context 'when provided an environment with driver options' do
       before do
         described_class.load!(file, :test)
       end
@@ -470,17 +440,16 @@ describe Mongoid::Config do
         described_class.reset
       end
 
-      it "sets the Mongo.broken_view_options option" do
+      it 'sets the Mongo.broken_view_options option' do
         expect(Mongo.broken_view_options).to eq(false)
       end
 
-      it "does not override the unset Mongo.validate_update_replace option" do
+      it 'does not override the unset Mongo.validate_update_replace option' do
         expect(Mongo.validate_update_replace).to eq(false)
       end
     end
 
-    context "when provided an environment with a nil driver option" do
-
+    context 'when provided an environment with a nil driver option' do
       before do
         described_class.load!(file, :test_nil)
       end
@@ -489,61 +458,57 @@ describe Mongoid::Config do
         described_class.reset
       end
 
-      it "sets the Mongo.broken_view_options option to nil" do
+      it 'sets the Mongo.broken_view_options option to nil' do
         expect(Mongo.broken_view_options).to be_nil
       end
     end
 
-    context "when the rack environment is set" do
-
+    context 'when the rack environment is set' do
       before do
-        ENV["RACK_ENV"] = "test"
+        ENV['RACK_ENV'] = 'test'
       end
 
       after do
-        ENV["RACK_ENV"] = nil
+        ENV['RACK_ENV'] = nil
         described_class.reset
       end
 
-      context "when mongoid options are provided" do
-
+      context 'when mongoid options are provided' do
         before do
           described_class.load!(file)
         end
 
-        it "sets the include root in json option" do
+        it 'sets the include root in json option' do
           expect(described_class.include_root_in_json).to be false
         end
 
-        it "sets the include type with serialization option" do
+        it 'sets the include type with serialization option' do
           expect(described_class.include_type_for_serialization).to be false
         end
 
-        it "sets the scope overwrite option" do
+        it 'sets the scope overwrite option' do
           expect(described_class.scope_overwrite_exception).to be false
         end
 
-        it "sets the preload models option" do
+        it 'sets the preload models option' do
           expect(described_class.preload_models).to be false
         end
 
-        it "sets the raise not found error option" do
+        it 'sets the raise not found error option' do
           expect(described_class.raise_not_found_error).to be true
         end
 
-        it "sets the use utc option" do
+        it 'sets the use utc option' do
           expect(described_class.use_utc).to be false
         end
 
-        it "sets the join_contexts default option" do
+        it 'sets the join_contexts default option' do
           expect(described_class.join_contexts).to be false
         end
       end
 
-      context "when client configurations are provided" do
-
-        context "when a default is provided" do
-
+      context 'when client configurations are provided' do
+        context 'when a default is provided' do
           before do
             described_class.load!(file, :test_with_max_staleness)
           end
@@ -552,21 +517,20 @@ describe Mongoid::Config do
             described_class.clients[:default]
           end
 
-          it "sets the default hosts" do
+          it 'sets the default hosts' do
             expect(default[:hosts]).to eq(SpecConfig.instance.addresses)
             # and make sure the value is not empty
             expect(default[:hosts].first).to include(':')
           end
 
-          context "when the default has options" do
-
+          context 'when the default has options' do
             let(:options) do
-              default["options"]
+              default['options']
             end
 
-            it "sets the read option" do
-              expect(options["read"]).to eq({ "mode" => :primary_preferred,
-                                              "max_staleness" => 100 })
+            it 'sets the read option' do
+              expect(options['read']).to eq({ 'mode' => :primary_preferred,
+                                              'max_staleness' => 100 })
             end
           end
         end
@@ -575,68 +539,66 @@ describe Mongoid::Config do
 
     context 'when schema map is provided with uuid' do
       let(:file) do
-        File.join(File.dirname(__FILE__), "..", "config", "mongoid_with_schema_map_uuid.yml")
+        File.join(File.dirname(__FILE__), '..', 'config', 'mongoid_with_schema_map_uuid.yml')
       end
+      let(:client) { Mongoid.default_client }
 
       before do
         described_class.load!(file, :test)
       end
 
-      let(:client) { Mongoid.default_client }
-
       it 'passes uuid to driver' do
         Mongo::Client.should receive(:new).with(SpecConfig.instance.addresses,
-          {
-            auto_encryption_options: {
-              'key_vault_namespace' => 'admin.datakeys',
-              'kms_providers' => {'local' => {'key' => 'z7iYiYKLuYymEWtk4kfny1ESBwwFdA58qMqff96A8ghiOcIK75lJGPUIocku8LOFjQuEgeIP4xlln3s7r93FV9J5sAE7zg8U'}},
-              'schema_map' => {'blog_development.comments' => {
-                'bsonType' => 'object',
-                'properties' => {
-                  'message' => {'encrypt' => {
-                    'algorithm' => 'AEAD_AES_256_CBC_HMAC_SHA_512-Deterministic',
-                    'bsonType' => 'string',
-                    'keyId' => [BSON::Binary.new("G\xF0 5\xCC@HX\xA2%b\x97\xA9a\xA8\xE7", :uuid)],
-                  }},
-                },
-              }}},
-            database: 'mongoid_test',
-            platform: "mongoid-#{Mongoid::VERSION}",
-            wrapping_libraries: [
-              {'name' => 'Mongoid', 'version' => Mongoid::VERSION},
-            ]
-          }
-        )
+                                                {
+                                                  auto_encryption_options: {
+                                                    'key_vault_namespace' => 'admin.datakeys',
+                                                    'kms_providers' => { 'local' => { 'key' => 'z7iYiYKLuYymEWtk4kfny1ESBwwFdA58qMqff96A8ghiOcIK75lJGPUIocku8LOFjQuEgeIP4xlln3s7r93FV9J5sAE7zg8U' } },
+                                                    'schema_map' => { 'blog_development.comments' => {
+                                                      'bsonType' => 'object',
+                                                      'properties' => {
+                                                        'message' => { 'encrypt' => {
+                                                          'algorithm' => 'AEAD_AES_256_CBC_HMAC_SHA_512-Deterministic',
+                                                          'bsonType' => 'string',
+                                                          'keyId' => [ BSON::Binary.new(
+                                                            "G\xF0 5\xCC@HX\xA2%b\x97\xA9a\xA8\xE7", :uuid
+                                                          ) ]
+                                                        } }
+                                                      }
+                                                    } }
+                                                  },
+                                                  database: 'mongoid_test',
+                                                  platform: "mongoid-#{Mongoid::VERSION}",
+                                                  wrapping_libraries: [
+                                                    { 'name' => 'Mongoid', 'version' => Mongoid::VERSION }
+                                                  ]
+                                                })
 
         client
       end
     end
   end
 
-  describe "#options=" do
-
-    context "when there are no options" do
-
+  describe '#options=' do
+    context 'when there are no options' do
       before do
         described_class.options = nil
       end
 
-      it "does not try to assign options" do
+      it 'does not try to assign options' do
         expect(described_class.preload_models).to be false
       end
     end
 
-    context "when provided a non-existent option" do
-
-      it "raises an error" do
-        expect {
+    context 'when provided a non-existent option' do
+      it 'raises an error' do
+        expect do
           described_class.options = { bad_option: true }
-        }.to raise_error(Mongoid::Errors::InvalidConfigOption)
+        end.to raise_error(Mongoid::Errors::InvalidConfigOption)
       end
     end
 
     context 'when invalid global_executor_concurrency option provided' do
-      it "raises an error" do
+      it 'raises an error' do
         expect do
           described_class.options = {
             async_query_executor: :immediate,
@@ -647,66 +609,58 @@ describe Mongoid::Config do
     end
   end
 
-  describe "#clients=" do
-
-    context "when no clients configuration exists" do
-
-      it "raises an error" do
-        expect {
+  describe '#clients=' do
+    context 'when no clients configuration exists' do
+      it 'raises an error' do
+        expect do
           described_class.send(:clients=, nil)
-        }.to raise_error(Mongoid::Errors::NoClientsConfig)
+        end.to raise_error(Mongoid::Errors::NoClientsConfig)
       end
     end
 
-    context "when no default client exists" do
-
-      it "raises an error" do
-        expect {
+    context 'when no default client exists' do
+      it 'raises an error' do
+        expect do
           described_class.send(:clients=, {})
-        }.to raise_error(Mongoid::Errors::NoDefaultClient)
+        end.to raise_error(Mongoid::Errors::NoDefaultClient)
       end
     end
 
-    context "when a default client exists" do
-
-      context "when no hosts are provided" do
-
+    context 'when a default client exists' do
+      context 'when no hosts are provided' do
         let(:clients) do
-          { "default" => { database: database_id }}
+          { 'default' => { database: database_id } }
         end
 
-        it "raises an error" do
-          expect {
+        it 'raises an error' do
+          expect do
             described_class.send(:clients=, clients)
-          }.to raise_error(Mongoid::Errors::NoClientHosts)
+          end.to raise_error(Mongoid::Errors::NoClientHosts)
         end
       end
 
-      context "when no database is provided" do
-
+      context 'when no database is provided' do
         let(:clients) do
-          { "default" => { hosts: [ "127.0.0.1:27017" ] }}
+          { 'default' => { hosts: [ '127.0.0.1:27017' ] } }
         end
 
-        it "raises an error" do
-          expect {
+        it 'raises an error' do
+          expect do
             described_class.send(:clients=, clients)
-          }.to raise_error(Mongoid::Errors::NoClientDatabase)
+          end.to raise_error(Mongoid::Errors::NoClientDatabase)
         end
       end
 
-      context "when a uri and standard options are provided" do
-
+      context 'when a uri and standard options are provided' do
         let(:clients) do
-          { "default" =>
-            { hosts: [ "127.0.0.1:27017" ], uri: "mongodb://127.0.0.1:27017" }
-          }
+          { 'default' =>
+            { hosts: [ '127.0.0.1:27017' ], uri: 'mongodb://127.0.0.1:27017' } }
         end
 
-        it "raises an error" do
-          expect {
+        it 'raises an error' do
+          expect do
             described_class.send(:clients=, clients)
-          }.to raise_error(Mongoid::Errors::MixedClientConfiguration)
+          end.to raise_error(Mongoid::Errors::MixedClientConfiguration)
         end
       end
     end
@@ -741,8 +695,7 @@ describe Mongoid::Config do
     end
   end
 
-  describe "#purge!" do
-
+  describe '#purge!' do
     it 'deletes models' do
       House.create!(name: '1', model: 'Big')
       expect(House.count).to eq(1)
@@ -761,8 +714,7 @@ describe Mongoid::Config do
     end
   end
 
-  describe "#truncate!" do
-
+  describe '#truncate!' do
     it 'deletes models' do
       House.create!(name: '1', model: 'Big')
       expect(House.count).to eq(1)
@@ -782,13 +734,13 @@ describe Mongoid::Config do
 
     it 'does not drop indexes' do
       User.create_indexes
-      expect(User.collection.indexes.map {|i| i['name'] }).to eq %w[_id_ name_1]
+      expect(User.collection.indexes.map { |i| i['name'] }).to eq %w[_id_ name_1]
       Mongoid.truncate!
-      expect(User.collection.indexes.map {|i| i['name'] }).to eq %w[_id_ name_1]
+      expect(User.collection.indexes.map { |i| i['name'] }).to eq %w[_id_ name_1]
     end
   end
 
-  describe "#override_database" do
+  describe '#override_database' do
     let(:database) do
       "test_override_#{Time.now.to_i}"
     end
@@ -814,7 +766,7 @@ describe Mongoid::Config do
       expect(Band.count).to eq(0)
     end
 
-    context '#truncate and #purge' do
+    describe '#truncate and #purge' do
       before do
         House.create!(name: '1', model: 'Big')
         expect(House.count).to eq(1)
@@ -826,7 +778,7 @@ describe Mongoid::Config do
         expect(House.count).to eq(1)
       end
 
-      context '#purge' do
+      describe '#purge' do
         it 'respects persistence context overrides' do
           House.create!(name: '2', model: 'Tiny')
           expect(House.count).to eq(1)
@@ -835,7 +787,7 @@ describe Mongoid::Config do
         end
       end
 
-      context '#truncate' do
+      describe '#truncate' do
         it '#truncate! respects persistence context overrides' do
           House.create!(name: '2', model: 'Tiny')
           expect(House.count).to eq(1)
@@ -848,9 +800,7 @@ describe Mongoid::Config do
 
   describe 'deprecations' do
     {}.each do |option, default|
-
       context ":#{option} option" do
-
         before do
           Mongoid::Warnings.class_eval do
             instance_variable_set(:"@#{option}_deprecated", false)