mongoid 6.4.8 → 7.5.4

data/spec/mongoid/validatable/uniqueness_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Validatable::UniquenessValidator do
@@ -20,7 +22,7 @@ describe Mongoid::Validatable::UniquenessValidator do
 
           before do
             Dictionary.with(collection: "dicts") do |klass|
-              klass.create(name: "websters")
+              klass.create!(name: "websters")
             end
           end
 
@@ -78,7 +80,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute id is unique" do
 
             let(:dictionary) do
-              Dictionary.create
+              Dictionary.create!
             end
 
             let(:word) do
@@ -170,7 +172,7 @@ describe Mongoid::Validatable::UniquenessValidator do
                 context "when a document with no value exists in the database" do
 
                   before do
-                    Dictionary.create
+                    Dictionary.create!
                   end
 
                   let(:dictionary) do
@@ -188,7 +190,7 @@ describe Mongoid::Validatable::UniquenessValidator do
                 context "when single localization" do
 
                   before do
-                    Dictionary.create(description: "english")
+                    Dictionary.create!(description: "english")
                   end
 
                   let(:dictionary) do
@@ -224,7 +226,7 @@ describe Mongoid::Validatable::UniquenessValidator do
                   context "when single localization" do
 
                     before do
-                      Dictionary.create(description: "english")
+                      Dictionary.create!(description: "english")
                     end
 
                     let(:dictionary) do
@@ -280,7 +282,7 @@ describe Mongoid::Validatable::UniquenessValidator do
                 context "when there are no special characters" do
 
                   before do
-                    Dictionary.create(description: "english")
+                    Dictionary.create!(description: "english")
                   end
 
                   let(:dictionary) do
@@ -295,7 +297,7 @@ describe Mongoid::Validatable::UniquenessValidator do
                 context "when special characters exist" do
 
                   before do
-                    Dictionary.create(description: "english")
+                    Dictionary.create!(description: "english")
                   end
 
                   let(:dictionary) do
@@ -315,7 +317,7 @@ describe Mongoid::Validatable::UniquenessValidator do
                   context "when signle localization" do
 
                     before do
-                      Dictionary.create(description: "english")
+                      Dictionary.create!(description: "english")
                     end
 
                     let(:dictionary) do
@@ -357,7 +359,7 @@ describe Mongoid::Validatable::UniquenessValidator do
                 context "when the document is the match in the database" do
 
                   let!(:dictionary) do
-                    Dictionary.create(description: "english")
+                    Dictionary.create!(description: "english")
                   end
 
                   it "returns true" do
@@ -417,7 +419,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is unique" do
 
             let!(:oxford) do
-              Dictionary.create(name: "Oxford")
+              Dictionary.create!(name: "Oxford")
             end
 
             let(:dictionary) do
@@ -445,7 +447,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is not the match" do
 
               before do
-                Dictionary.create(name: "Oxford")
+                Dictionary.create!(name: "Oxford")
               end
 
               let!(:dictionary) do
@@ -467,7 +469,7 @@ describe Mongoid::Validatable::UniquenessValidator do
               context "when the field has changed" do
 
                 let!(:dictionary) do
-                  Dictionary.create(name: "Oxford")
+                  Dictionary.create!(name: "Oxford")
                 end
 
                 it "returns true" do
@@ -519,7 +521,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the attribute is not unique" do
 
               before do
-                Dictionary.create(name: "Oxford")
+                Dictionary.create!(name: "Oxford")
               end
 
               let(:dictionary) do
@@ -546,7 +548,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is unique" do
 
             before do
-              Dictionary.create(name: "Oxford", language: "English")
+              Dictionary.create!(name: "Oxford", language: "English")
             end
 
             let(:dictionary) do
@@ -561,7 +563,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is unique in the scope" do
 
             before do
-              Dictionary.create(name: "Oxford", language: "English")
+              Dictionary.create!(name: "Oxford", language: "English")
             end
 
             let(:dictionary) do
@@ -576,7 +578,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is not unique with no scope" do
 
             before do
-              Dictionary.create(name: "Oxford", language: "English")
+              Dictionary.create!(name: "Oxford", language: "English")
             end
 
             let(:dictionary) do
@@ -591,7 +593,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is not unique in another scope" do
 
             before do
-              Dictionary.create(name: "Oxford", language: "English")
+              Dictionary.create!(name: "Oxford", language: "English")
             end
 
             let(:dictionary) do
@@ -608,7 +610,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is not the match" do
 
               before do
-                Dictionary.create(name: "Oxford", language: "English")
+                Dictionary.create!(name: "Oxford", language: "English")
               end
 
               let(:dictionary) do
@@ -628,7 +630,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is the match in the database" do
 
               let!(:dictionary) do
-                Dictionary.create(name: "Oxford", language: "English")
+                Dictionary.create!(name: "Oxford", language: "English")
               end
 
               it "returns true" do
@@ -651,7 +653,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is unique" do
 
             before do
-              Dictionary.create(name: "Oxford", publisher: "Amazon")
+              Dictionary.create!(name: "Oxford", publisher: "Amazon")
             end
 
             let(:dictionary) do
@@ -666,7 +668,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is unique in the scope" do
 
             before do
-              Dictionary.create(name: "Oxford", publisher: "Amazon")
+              Dictionary.create!(name: "Oxford", publisher: "Amazon")
             end
 
             let(:dictionary) do
@@ -706,7 +708,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is not unique with no scope" do
 
             before do
-              Dictionary.create(name: "Oxford", publisher: "Amazon")
+              Dictionary.create!(name: "Oxford", publisher: "Amazon")
             end
 
             let(:dictionary) do
@@ -721,7 +723,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is not unique in another scope" do
 
             before do
-              Dictionary.create(name: "Oxford", publisher: "Amazon")
+              Dictionary.create!(name: "Oxford", publisher: "Amazon")
             end
 
             let(:dictionary) do
@@ -738,7 +740,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is not the match" do
 
               before do
-                Dictionary.create(name: "Oxford", publisher: "Amazon")
+                Dictionary.create!(name: "Oxford", publisher: "Amazon")
               end
 
               let(:dictionary) do
@@ -758,7 +760,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is the match in the database" do
 
               let!(:dictionary) do
-                Dictionary.create(name: "Oxford", publisher: "Amazon")
+                Dictionary.create!(name: "Oxford", publisher: "Amazon")
               end
 
               it "returns true" do
@@ -769,7 +771,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when one of the scopes is a time" do
 
               before do
-                Dictionary.create(
+                Dictionary.create!(
                   name: "Oxford",
                   publisher: "Amazon",
                   published: 10.days.ago.to_time
@@ -796,20 +798,47 @@ describe Mongoid::Validatable::UniquenessValidator do
           end
         end
 
-        context "when a range scope is provided" do
+        context "when a range condition is provided" do
 
           before do
-            Dictionary.validates_uniqueness_of(:name, :scope => Dictionary.where(:year.gte => 1900, :year.lt => 2000))
-            Dictionary.create(name: "French-English", year: 1950)
-            Dictionary.create(name: "French-English", year: 1960)
+            Dictionary.validates_uniqueness_of(:name,
+              conditions: -> { Dictionary.where(:year.gte => 1900, :year.lt => 2000) })
           end
 
           after do
             Dictionary.reset_callbacks(:validate)
           end
 
-          it "successfully prevents uniqueness violation" do
-            expect(Dictionary.all.size).to eq(1)
+          context 'when multiple documents would match the condition' do
+            it "prevents creation of new document" do
+              Dictionary.create!(name: "French-English", year: 1950)
+
+              expect do
+                Dictionary.create!(name: "French-English", year: 1960)
+              end.to raise_error(Mongoid::Errors::Validations, /Name is already taken/)
+
+              expect(Dictionary.all.size).to eq(1)
+            end
+          end
+
+          context 'when only new document would match the condition' do
+            it 'creates the new document' do
+              Dictionary.create!(name: "French-English", year: 950)
+              expect do
+                Dictionary.create!(name: "French-English", year: 1950)
+              end.not_to raise_error
+            end
+          end
+
+          context 'when only existing document matches the condition' do
+            it 'creates the new document' do
+              pending 'https://jira.mongodb.org/browse/MONGOID-4815'
+
+              Dictionary.create!(name: "French-English", year: 1950)
+              expect do
+                Dictionary.create!(name: "French-English", year: 950)
+              end.not_to raise_error
+            end
           end
         end
 
@@ -826,7 +855,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is unique" do
 
             before do
-              Dictionary.create(name: "Oxford", publisher: "Amazon")
+              Dictionary.create!(name: "Oxford", publisher: "Amazon")
             end
 
             let(:dictionary) do
@@ -841,7 +870,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is unique in the scope" do
 
             before do
-              Dictionary.create(
+              Dictionary.create!(
                 name: "Oxford",
                 publisher: "Amazon",
                 year: 2011
@@ -864,7 +893,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is not unique with no scope" do
 
             before do
-              Dictionary.create(name: "Oxford", publisher: "Amazon")
+              Dictionary.create!(name: "Oxford", publisher: "Amazon")
             end
 
             let(:dictionary) do
@@ -879,7 +908,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is not unique in another scope" do
 
             before do
-              Dictionary.create(
+              Dictionary.create!(
                 name: "Oxford",
                 publisher: "Amazon",
                 year: 1995
@@ -904,7 +933,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is not the match" do
 
               before do
-                Dictionary.create(
+                Dictionary.create!(
                   name: "Oxford",
                   publisher: "Amazon",
                   year: 1960
@@ -932,7 +961,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is the match in the database" do
 
               let!(:dictionary) do
-                Dictionary.create(
+                Dictionary.create!(
                   name: "Oxford",
                   publisher: "Amazon",
                   year: 1960
@@ -959,7 +988,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is unique" do
 
             before do
-              Dictionary.create(name: "Oxford")
+              Dictionary.create!(name: "Oxford")
             end
 
             let(:dictionary) do
@@ -976,7 +1005,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is not the match" do
 
               before do
-                Dictionary.create(name: "Oxford")
+                Dictionary.create!(name: "Oxford")
               end
 
               let(:dictionary) do
@@ -996,7 +1025,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is the match in the database" do
 
               let!(:dictionary) do
-                Dictionary.create(name: "Oxford")
+                Dictionary.create!(name: "Oxford")
               end
 
               it "returns true" do
@@ -1021,7 +1050,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when there are no special characters" do
 
               before do
-                Dictionary.create(name: "Oxford")
+                Dictionary.create!(name: "Oxford")
               end
 
               let(:dictionary) do
@@ -1036,7 +1065,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when special characters exist" do
 
               before do
-                Dictionary.create(name: "Oxford")
+                Dictionary.create!(name: "Oxford")
               end
 
               let(:dictionary) do
@@ -1054,7 +1083,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is not the match" do
 
               before do
-                Dictionary.create(name: "Oxford")
+                Dictionary.create!(name: "Oxford")
               end
 
               let(:dictionary) do
@@ -1074,7 +1103,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is the match in the database" do
 
               let!(:dictionary) do
-                Dictionary.create(name: "Oxford")
+                Dictionary.create!(name: "Oxford")
               end
 
               it "returns true" do
@@ -1104,7 +1133,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is nil" do
 
             before do
-              Dictionary.create
+              Dictionary.create!
             end
 
             let(:dictionary) do
@@ -1130,7 +1159,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is blank" do
 
             before do
-              Dictionary.create(name: "")
+              Dictionary.create!(name: "")
             end
 
             let(:dictionary) do
@@ -1159,7 +1188,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is unique" do
 
             before do
-              Login.create(username: "Oxford")
+              Login.create!(username: "Oxford")
             end
 
             let(:login) do
@@ -1176,7 +1205,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is not the match" do
 
               before do
-                Login.create(username: "Oxford")
+                Login.create!(username: "Oxford")
               end
 
               let(:login) do
@@ -1196,7 +1225,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is the match in the database" do
 
               let!(:login) do
-                Login.create(username: "Oxford")
+                Login.create!(username: "Oxford")
               end
 
               it "returns true" do
@@ -1219,7 +1248,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is unique" do
 
             before do
-              Login.create(username: "Oxford", application_id: 1)
+              Login.create!(username: "Oxford", application_id: 1)
             end
 
             let(:login) do
@@ -1234,7 +1263,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is unique in the scope" do
 
             before do
-              Login.create(username: "Oxford", application_id: 1)
+              Login.create!(username: "Oxford", application_id: 1)
             end
 
             let(:login) do
@@ -1249,7 +1278,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is not unique with no scope" do
 
             before do
-              Login.create(username: "Oxford", application_id: 1)
+              Login.create!(username: "Oxford", application_id: 1)
             end
 
             let(:login) do
@@ -1264,7 +1293,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is not unique in another scope" do
 
             before do
-              Login.create(username: "Oxford", application_id: 1)
+              Login.create!(username: "Oxford", application_id: 1)
             end
 
             let(:login) do
@@ -1281,7 +1310,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is not the match" do
 
               before do
-                Login.create(username: "Oxford", application_id: 1)
+                Login.create!(username: "Oxford", application_id: 1)
               end
 
               let(:login) do
@@ -1301,7 +1330,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is the match in the database" do
 
               let!(:login) do
-                Login.create(username: "Oxford", application_id: 1)
+                Login.create!(username: "Oxford", application_id: 1)
               end
 
               it "returns true" do
@@ -1324,7 +1353,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is unique" do
 
             before do
-              Login.create(username: "Oxford")
+              Login.create!(username: "Oxford")
             end
 
             let(:login) do
@@ -1341,7 +1370,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is not the match" do
 
               before do
-                Login.create(username: "Oxford")
+                Login.create!(username: "Oxford")
               end
 
               let(:login) do
@@ -1361,7 +1390,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is the match in the database" do
 
               let!(:login) do
-                Login.create(username: "Oxford")
+                Login.create!(username: "Oxford")
               end
 
               it "returns true" do
@@ -1386,7 +1415,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when there are no special characters" do
 
               before do
-                Login.create(username: "Oxford")
+                Login.create!(username: "Oxford")
               end
 
               let(:login) do
@@ -1401,7 +1430,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when special characters exist" do
 
               before do
-                Login.create(username: "Oxford")
+                Login.create!(username: "Oxford")
               end
 
               let(:login) do
@@ -1419,7 +1448,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is not the match" do
 
               before do
-                Login.create(username: "Oxford")
+                Login.create!(username: "Oxford")
               end
 
               let(:login) do
@@ -1439,7 +1468,7 @@ describe Mongoid::Validatable::UniquenessValidator do
             context "when the document is the match in the database" do
 
               let!(:login) do
-                Login.create(username: "Oxford")
+                Login.create!(username: "Oxford")
               end
 
               it "returns true" do
@@ -1462,7 +1491,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is nil" do
 
             before do
-              Login.create
+              Login.create!
             end
 
             let(:login) do
@@ -1488,7 +1517,7 @@ describe Mongoid::Validatable::UniquenessValidator do
           context "when the attribute is blank" do
 
             before do
-              Login.create(username: "")
+              Login.create!(username: "")
             end
 
             let(:login) do
@@ -1515,7 +1544,7 @@ describe Mongoid::Validatable::UniquenessValidator do
         context "when the attribute is unique" do
 
           before do
-            Bar.create(lat_lng: LatLng.new(52.30, 13.25))
+            Bar.create!(lat_lng: LatLng.new(52.30, 13.25))
           end
 
           let(:unique_bar) do
@@ -1531,7 +1560,7 @@ describe Mongoid::Validatable::UniquenessValidator do
         context "when the attribute is not unique" do
 
           before do
-            Bar.create(lat_lng: LatLng.new(52.30, 13.25))
+            Bar.create!(lat_lng: LatLng.new(52.30, 13.25))
           end
 
           let(:non_unique_bar) do
@@ -1558,7 +1587,7 @@ describe Mongoid::Validatable::UniquenessValidator do
         context "when the attribute is unique" do
 
           before do
-            Band.create(name: 'Foo', active: false)
+            Band.create!(name: 'Foo', active: false)
           end
 
           let(:unique_band) do
@@ -1574,7 +1603,7 @@ describe Mongoid::Validatable::UniquenessValidator do
         context "when the attribute is not unique" do
 
           before do
-            Band.create(name: 'Foo')
+            Band.create!(name: 'Foo')
           end
 
           let(:non_unique_band) do
@@ -1592,17 +1621,17 @@ describe Mongoid::Validatable::UniquenessValidator do
   context "when the document is embedded" do
 
     let(:word) do
-      Word.create(name: "Schadenfreude")
+      Word.create!(name: "Schadenfreude")
     end
 
     context "when in an embeds_many" do
 
       let!(:def_one) do
-        word.definitions.create(description: "1")
+        word.definitions.create!(description: "1")
       end
 
       let!(:def_two) do
-        word.definitions.create(description: "2")
+        word.definitions.create!(description: "2")
       end
 
       context "when a document is being destroyed" do
@@ -2429,7 +2458,7 @@ describe Mongoid::Validatable::UniquenessValidator do
     end
 
     let!(:person) do
-      Person.create(ssn: "132-11-1111", username: "aaasdaffff")
+      Person.create!(ssn: "132-11-1111", username: "aaasdaffff")
     end
 
     after do

data/spec/mongoid/validatable_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Validatable do
@@ -34,7 +36,7 @@ describe Mongoid::Validatable do
       end
 
       let(:documents) do
-        Mongoid::Relations::Targets::Enumerable.new([ address ])
+        Mongoid::Association::Referenced::HasMany::Enumerable.new([ address ])
       end
 
       before do
@@ -100,7 +102,7 @@ describe Mongoid::Validatable do
 
         before do
           account.name = "Testing"
-          account.save
+          account.save!
           account.name = "Testing a really long name."
         end
 
@@ -118,7 +120,7 @@ describe Mongoid::Validatable do
 
       before do
         pizza.build_topping(name: "cheese")
-        pizza.save
+        pizza.save!
       end
 
       let(:from_db) do

data/spec/mongoid_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid do
@@ -49,8 +51,14 @@ describe Mongoid do
     end
 
     it "disconnects from all active clients" do
+      method_name = if Gem::Version.new(Mongo::VERSION) >= Gem::Version.new('2.19.0')
+                      :close
+                    else
+                      :disconnect!
+                    end
+
       clients.each do |client|
-        expect(client.cluster).to receive(:disconnect!).and_call_original
+        expect(client.cluster).to receive(method_name).and_call_original
       end
       Mongoid.disconnect_clients
     end