mongoid 6.4.4 → 7.5.1

data/spec/mongoid/criteria/modifiable_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Criteria::Modifiable do
@@ -10,7 +12,7 @@ describe Mongoid::Criteria::Modifiable do
 
     context "when provided valid attributes" do
       let(:band) do
-        criteria.create(genres: [ "electro" ])
+        criteria.create!(genres: [ "electro" ])
       end
 
       it "returns the created document" do
@@ -31,7 +33,7 @@ describe Mongoid::Criteria::Modifiable do
       context "when provided valid attributes & using block" do
 
         let(:band) do
-          criteria.create do |c|
+          criteria.create! do |c|
             c.genres = [ "electro" ]
           end
         end
@@ -49,6 +51,29 @@ describe Mongoid::Criteria::Modifiable do
         end
       end
     end
+
+    context 'when the object is polymorphic' do
+
+      let(:movie) do
+        Movie.new
+      end
+
+      let(:criteria) do
+        Rating.where(ratable: movie)
+      end
+
+      let(:document) do
+        criteria.create!
+      end
+
+      it 'sets the polymorphic id' do
+        expect(document.ratable_id).to eq(movie.id)
+      end
+
+      it 'sets the type field' do
+        expect(document.ratable_type).to eq('Movie')
+      end
+    end
   end
 
   describe "#create!" do
@@ -65,6 +90,29 @@ describe Mongoid::Criteria::Modifiable do
         }.to raise_error(Mongoid::Errors::Validations)
       end
     end
+
+    context 'when the object is polymorphic' do
+
+      let(:movie) do
+        Movie.new
+      end
+
+      let(:criteria) do
+        Rating.where(ratable: movie)
+      end
+
+      let(:document) do
+        criteria.create!
+      end
+
+      it 'sets the polymorphic id' do
+        expect(document.ratable_id).to eq(movie.id)
+      end
+
+      it 'sets the type field' do
+        expect(document.ratable_type).to eq('Movie')
+      end
+    end
   end
 
   [ :delete, :delete_all, :destroy, :destroy_all ].each do |method|
@@ -84,9 +132,9 @@ describe Mongoid::Criteria::Modifiable do
       end
 
       before do
-        Person.create(title: "Madam")
+        Person.create!(title: "Madam")
         2.times do |n|
-          Person.create(
+          Person.create!(
             title: "Sir",
             name: name,
             addresses: [ address_one, address_two ]
@@ -171,7 +219,7 @@ describe Mongoid::Criteria::Modifiable do
       context "when providing an attribute" do
 
         let!(:person) do
-          Person.create(title: "Senior")
+          Person.create!(title: "Senior")
         end
 
         it "returns the document" do
@@ -184,11 +232,11 @@ describe Mongoid::Criteria::Modifiable do
         context "with an owner with a BSON identity type" do
 
           let!(:person) do
-            Person.create
+            Person.create!
           end
 
           let!(:game) do
-            Game.create(person: person)
+            Game.create!(person: person)
           end
 
           context "when providing the object directly" do
@@ -217,11 +265,11 @@ describe Mongoid::Criteria::Modifiable do
         context "with an owner with an Integer identity type" do
 
           let!(:jar) do
-            Jar.create
+            Jar.create!
           end
 
           let!(:cookie) do
-            Cookie.create(jar: jar)
+            Cookie.create!(jar: jar)
           end
 
           let(:from_db) do
@@ -240,11 +288,11 @@ describe Mongoid::Criteria::Modifiable do
       context "when providing a document" do
 
         let!(:person) do
-          Person.create
+          Person.create!
         end
 
         let!(:game) do
-          Game.create
+          Game.create!
         end
 
         let(:from_db) do
@@ -295,6 +343,25 @@ describe Mongoid::Criteria::Modifiable do
           expect(person.pets).to be true
         end
       end
+
+      context 'when the object is polymorphic' do
+
+        let(:movie) do
+          Movie.new
+        end
+
+        let(:document) do
+          Rating.find_or_create_by(ratable: movie)
+        end
+
+        it 'sets the polymorphic id' do
+          expect(document.ratable_id).to eq(movie.id)
+        end
+
+        it 'sets the type field' do
+          expect(document.ratable_type).to eq('Movie')
+        end
+      end
     end
   end
 
@@ -305,7 +372,7 @@ describe Mongoid::Criteria::Modifiable do
       context "when providing an attribute" do
 
         let!(:person) do
-          Person.create(title: "Senior")
+          Person.create!(title: "Senior")
         end
 
         it "returns the document" do
@@ -318,11 +385,11 @@ describe Mongoid::Criteria::Modifiable do
         context "with an owner with a BSON identity type" do
 
           let!(:person) do
-            Person.create
+            Person.create!
           end
 
           let!(:game) do
-            Game.create(person: person)
+            Game.create!(person: person)
           end
 
           context "when providing the object directly" do
@@ -351,11 +418,11 @@ describe Mongoid::Criteria::Modifiable do
         context "with an owner with an Integer identity type" do
 
           let!(:jar) do
-            Jar.create
+            Jar.create!
           end
 
           let!(:cookie) do
-            Cookie.create(jar: jar)
+            Cookie.create!(jar: jar)
           end
 
           let(:from_db) do
@@ -374,11 +441,11 @@ describe Mongoid::Criteria::Modifiable do
       context "when providing a document" do
 
         let!(:person) do
-          Person.create
+          Person.create!
         end
 
         let!(:game) do
-          Game.create
+          Game.create!
         end
 
         let(:from_db) do
@@ -447,6 +514,25 @@ describe Mongoid::Criteria::Modifiable do
         end
       end
     end
+
+    context 'when the object is polymorphic' do
+
+      let(:movie) do
+        Movie.new
+      end
+
+      let(:document) do
+        Rating.find_or_create_by!(ratable: movie)
+      end
+
+      it 'sets the polymorphic id' do
+        expect(document.ratable_id).to eq(movie.id)
+      end
+
+      it 'sets the type field' do
+        expect(document.ratable_type).to eq('Movie')
+      end
+    end
   end
 
   describe ".find_or_initialize_by" do
@@ -454,7 +540,7 @@ describe Mongoid::Criteria::Modifiable do
     context "when the document is found" do
 
       let!(:person) do
-        Person.create(title: "Senior")
+        Person.create!(title: "Senior")
       end
 
       it "returns the document" do
@@ -500,12 +586,31 @@ describe Mongoid::Criteria::Modifiable do
         end
       end
     end
+
+    context 'when the object is polymorphic' do
+
+      let(:movie) do
+        Movie.new
+      end
+
+      let(:document) do
+        Rating.find_or_initialize_by(ratable: movie)
+      end
+
+      it 'sets the polymorphic id' do
+        expect(document.ratable_id).to eq(movie.id)
+      end
+
+      it 'sets the type field' do
+        expect(document.ratable_type).to eq('Movie')
+      end
+    end
   end
 
   describe "first_or_create" do
 
     let!(:band) do
-      Band.create(name: "Depeche Mode")
+      Band.create!(name: "Depeche Mode")
     end
 
     context "when the document is found" do
@@ -599,7 +704,7 @@ describe Mongoid::Criteria::Modifiable do
           end
 
           it 'uses the values from the attributes' do
-            expect(document.map).to eq({ foo: :bar })
+            expect(document.map).to eq('foo' => :bar )
           end
         end
 
@@ -618,7 +723,7 @@ describe Mongoid::Criteria::Modifiable do
 
           let(:band) do
             record = Record.new(producers: ['testing'])
-            band = Band.create(records: [record])
+            band = Band.create!(records: [record])
           end
 
           let(:document) do
@@ -646,10 +751,10 @@ describe Mongoid::Criteria::Modifiable do
         context 'when the criteria has a deeply-nested selector with query operators' do
 
           let(:criteria) do
-            band = Band.create
+            band = Band.create!
             Mongoid::Criteria.new(Record) do |criteria|
               criteria.embedded = true
-              criteria.metadata = Band.reflect_on_association(:records)
+              criteria.association = Band.reflect_on_association(:records)
               criteria.parent_document = band
               criteria.selector = { "records" => { "producers"=>{"$in"=>["nonexistent"] } } }
             end
@@ -687,7 +792,7 @@ describe Mongoid::Criteria::Modifiable do
       context "when the criteria is on an embedded relation" do
 
         let!(:band) do
-          Band.create(name: "Placebo")
+          Band.create!(name: "Placebo")
         end
 
         let(:document) do
@@ -703,6 +808,29 @@ describe Mongoid::Criteria::Modifiable do
         end
       end
 
+      context 'when the object is polymorphic' do
+
+        let(:movie) do
+          Movie.new
+        end
+
+        let(:criteria) do
+          Rating.where(ratable: movie)
+        end
+
+        let(:document) do
+          criteria.first_or_create
+        end
+
+        it 'sets the polymorphic id' do
+          expect(document.ratable_id).to eq(movie.id)
+        end
+
+        it 'sets the type field' do
+          expect(document.ratable_type).to eq('Movie')
+        end
+      end
+
       context "when a block is provided" do
 
         let(:document) do
@@ -919,12 +1047,35 @@ describe Mongoid::Criteria::Modifiable do
         end
       end
     end
+
+    context 'when the object is polymorphic' do
+
+      let(:movie) do
+        Movie.new
+      end
+
+      let(:criteria) do
+        Rating.where(ratable: movie)
+      end
+
+      let(:document) do
+        criteria.first_or_create!
+      end
+
+      it 'sets the polymorphic id' do
+        expect(document.ratable_id).to eq(movie.id)
+      end
+
+      it 'sets the type field' do
+        expect(document.ratable_type).to eq('Movie')
+      end
+    end
   end
 
   describe "first_or_initialize" do
 
     let!(:band) do
-      Band.create(name: "Depeche Mode")
+      Band.create!(name: "Depeche Mode")
     end
 
     context "when the document is found" do
@@ -938,6 +1089,29 @@ describe Mongoid::Criteria::Modifiable do
       end
     end
 
+    context 'when the object is polymorphic' do
+
+      let(:movie) do
+        Movie.new
+      end
+
+      let(:criteria) do
+        Rating.where(ratable: movie)
+      end
+
+      let(:document) do
+        criteria.first_or_initialize
+      end
+
+      it 'sets the polymorphic id' do
+        expect(document.ratable_id).to eq(movie.id)
+      end
+
+      it 'sets the type field' do
+        expect(document.ratable_type).to eq('Movie')
+      end
+    end
+
     context "when the document is not found" do
 
       context "when attributes are provided" do
@@ -1075,11 +1249,11 @@ describe Mongoid::Criteria::Modifiable do
     end
 
     let!(:address_one) do
-      person.addresses.create(street: "Oranienstr")
+      person.addresses.create!(street: "Oranienstr")
     end
 
     let!(:address_two) do
-      person.addresses.create(street: "Wienerstr")
+      person.addresses.create!(street: "Wienerstr")
     end
 
     context "when updating the root document" do
@@ -1220,11 +1394,11 @@ describe Mongoid::Criteria::Modifiable do
         end
 
         let!(:preference_one) do
-          person.preferences.create(name: "First")
+          person.preferences.create!(name: "First")
         end
 
         let!(:preference_two) do
-          person.preferences.create(name: "Second")
+          person.preferences.create!(name: "Second")
         end
 
         context "when updating the relation directly" do
@@ -1263,15 +1437,15 @@ describe Mongoid::Criteria::Modifiable do
   describe "#update_all" do
 
     let!(:person) do
-      Person.create(title: "Sir")
+      Person.create!(title: "Sir")
     end
 
     let!(:address_one) do
-      person.addresses.create(street: "Oranienstr")
+      person.addresses.create!(street: "Oranienstr")
     end
 
     let!(:address_two) do
-      person.addresses.create(street: "Wienerstr")
+      person.addresses.create!(street: "Wienerstr")
     end
 
     context "when updating the root document" do
@@ -1370,11 +1544,11 @@ describe Mongoid::Criteria::Modifiable do
       context "when the relation is a references many" do
 
         let!(:post_one) do
-          person.posts.create(title: "First")
+          person.posts.create!(title: "First")
         end
 
         let!(:post_two) do
-          person.posts.create(title: "Second")
+          person.posts.create!(title: "Second")
         end
 
         context "when updating the relation directly" do
@@ -1423,11 +1597,11 @@ describe Mongoid::Criteria::Modifiable do
         end
 
         let!(:preference_one) do
-          person.preferences.create(name: "First")
+          person.preferences.create!(name: "First")
         end
 
         let!(:preference_two) do
-          person.preferences.create(name: "Second")
+          person.preferences.create!(name: "Second")
         end
 
         context "when updating the relation directly" do
@@ -1601,7 +1775,7 @@ describe Mongoid::Criteria::Modifiable do
           end
         end
 
-        context 'when a write method is chained' do
+        context 'when a write method is chained producing conflicting criteria' do
 
           let(:new_person) do
             criteria.create_with(attrs).find_or_create_by(query)
@@ -1625,9 +1799,10 @@ describe Mongoid::Criteria::Modifiable do
               criteria.create_with(attrs).find_or_create_by(query)
             end
 
-            it 'finds the matching document' do
+            it 'creates a new document' do
               person
-              expect(found_person.id).to eq(person.id)
+              expect(found_person.id).not_to eq(person.id)
+              expect(found_person.id > person.id).to be true
             end
           end
         end

data/spec/mongoid/criteria/options_spec.rb

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

data/spec/mongoid/criteria/queryable/aggregable_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Criteria::Queryable::Aggregable do
@@ -12,7 +14,7 @@ describe Mongoid::Criteria::Queryable::Aggregable do
       expect(aggregation).to_not equal(query)
     end
 
-    it "sets the aggrating flag" do
+    it "sets the aggregating flag" do
       expect(aggregation).to be_aggregating
     end
   end

data/spec/mongoid/criteria/queryable/expandable_spec.rb

@@ -0,0 +1,61 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+describe Mongoid::Criteria::Queryable::Expandable do
+
+  let(:query) do
+    Mongoid::Query.new
+  end
+
+  describe '#expand_condition_to_array_values' do
+    shared_examples_for 'expands' do
+
+      it 'expands' do
+        query.send(:expand_condition_to_array_values, criterion).should == expected
+      end
+
+      context 'when input is frozen' do
+        before do
+          criterion.freeze
+        end
+
+        it 'expands' do
+          query.send(:expand_condition_to_array_values, criterion).should == expected
+        end
+      end
+
+      it 'does not modify input' do
+        criterion_copy = criterion.dup.freeze
+
+        query.send(:expand_condition_to_array_values, criterion).should == expected
+
+        expect(criterion).to eq(criterion_copy)
+      end
+    end
+
+    context 'literal value' do
+      let(:criterion) do
+        {foo: 4}
+      end
+
+      let(:expected) do
+        {foo: [4]}
+      end
+
+      it_behaves_like 'expands'
+    end
+
+    context 'Range value' do
+      let(:criterion) do
+        {foo: 1..4}
+      end
+
+      let(:expected) do
+        {foo: [1, 2, 3, 4]}
+      end
+
+      it_behaves_like 'expands'
+    end
+  end
+end

data/spec/mongoid/criteria/queryable/extensions/array_spec.rb

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

data/spec/mongoid/criteria/queryable/extensions/big_decimal_spec.rb

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

data/spec/mongoid/criteria/queryable/extensions/bignum_spec.rb

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

data/spec/mongoid/criteria/queryable/extensions/boolean_spec.rb

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
-describe Boolean do
+describe Mongoid::Boolean do
 
   describe ".evolve" do
 

data/spec/mongoid/criteria/queryable/extensions/date_spec.rb

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