mongoid 5.4.1 → 6.0.0.beta

data/spec/mongoid/positional_spec.rb

@@ -17,8 +17,8 @@ describe Mongoid::Positional do
           "children.0.field" => "value",
           "children.0.children.1.children.3.field" => "value"
         },
-        "$push" => {
-          "children.0.children.1.children.3.fields" => {'$each' => [ "value", "value" ]}
+        "$pushAll" => {
+          "children.0.children.1.children.3.fields" => [ "value", "value" ]
         }
       }
     end
@@ -113,8 +113,8 @@ describe Mongoid::Positional do
               "children.$.field" => "value",
               "children.0.children.1.children.3.field" => "value"
             },
-            "$push" => {
-              "children.0.children.1.children.3.fields" => { '$each' => [ "value", "value" ] }
+            "$pushAll" => {
+              "children.0.children.1.children.3.fields" => [ "value", "value" ]
             }
           }
         end
@@ -141,8 +141,8 @@ describe Mongoid::Positional do
               "children.0.field" => "value",
               "children.0.children.1.children.3.field" => "value"
             },
-            "$push" => {
-              "children.0.children.1.children.3.fields" => { '$each' => [ "value", "value" ] }
+            "$pushAll" => {
+              "children.0.children.1.children.3.fields" => [ "value", "value" ]
             }
           }
         end
@@ -170,8 +170,8 @@ describe Mongoid::Positional do
             "children.$.field" => "value",
             "children.0.children.1.children.3.field" => "value"
           },
-          "$push" => {
-            "children.0.children.1.children.3.fields" => { '$each' => [ "value", "value" ] }
+          "$pushAll" => {
+            "children.0.children.1.children.3.fields" => [ "value", "value" ]
           }
         }
       end
@@ -203,8 +203,8 @@ describe Mongoid::Positional do
             "children.$.field" => "value",
             "children.0.children.1.children.3.field" => "value"
           },
-          "$push" => {
-            "children.0.children.1.children.3.fields" => { '$each' => [ "value", "value" ] }
+          "$pushAll" => {
+            "children.0.children.1.children.3.fields" => [ "value", "value" ]
           }
         }
       end

data/spec/mongoid/query_cache_spec.rb

@@ -7,6 +7,95 @@ describe Mongoid::QueryCache do
     Mongoid::QueryCache.cache { spec.run }
   end
 
+  context 'when iterating over objects sharing the same base' do
+
+    let(:server) do
+      relations.first.mongo_client.cluster.next_primary
+    end
+
+    before do
+      person = Person.create
+      3.times do
+        person.send(relation).create
+      end
+      person.save
+    end
+
+    let!(:relations) do
+      Person.first.send(relation).to_a
+    end
+
+    context 'when the association is has-many' do
+
+      let(:relation) do
+        :posts
+      end
+
+      context 'when query cache is disabled' do
+
+        before do
+          Mongoid::QueryCache.enabled = false
+        end
+
+        it 'queries for each access to the base' do
+          expect(server).to receive(:context).exactly(relations.size).times.and_call_original
+          relations.each do |object|
+            object.person
+          end
+        end
+      end
+
+      context 'when query cache is enabled' do
+
+        before do
+          Mongoid::QueryCache.enabled = true
+        end
+
+        it 'queries only once for the base' do
+          expect(server).to receive(:context).exactly(1).times.and_call_original
+          relations.each do |object|
+            object.person
+          end
+        end
+      end
+    end
+
+    context 'when the association is embeds-many' do
+
+      let(:relation) do
+        :symptoms
+      end
+
+      context 'when query cache is disabled' do
+
+        before do
+          Mongoid::QueryCache.enabled = false
+        end
+
+        it 'does not query for access to the base' do
+          expect(server).to receive(:context).exactly(0).times.and_call_original
+          relations.each do |object|
+            object.person
+          end
+        end
+      end
+
+      context 'when query cache is enabled' do
+
+        before do
+          Mongoid::QueryCache.enabled = true
+        end
+
+        it 'does not query for access to the base' do
+          expect(server).to receive(:context).exactly(0).times.and_call_original
+          relations.each do |object|
+            object.person
+          end
+        end
+      end
+    end
+  end
+
   context "when querying for a single document" do
 
     [ :first, :one, :last ].each do |method|
@@ -75,40 +164,6 @@ describe Mongoid::QueryCache do
         end
       end
 
-      context 'when the first query has a collation', if: collation_supported? do
-
-        before do
-          Band.where(name: 'DEPECHE MODE').collation(locale: 'en_US', strength: 2).to_a
-        end
-
-        context "when the next query has the same collation" do
-
-          it "uses the cache" do
-            expect_no_queries do
-              Band.where(name: 'DEPECHE MODE').collation(locale: 'en_US', strength: 2).to_a
-            end
-          end
-        end
-
-        context "when the next query does not have the same collation" do
-
-          it "queries again" do
-            expect_query(1) do
-              Band.where(name: 'DEPECHE MODE').collation(locale: 'fr', strength: 2).to_a
-            end
-          end
-        end
-
-        context "when the next query does not have a collation" do
-
-          it "queries again" do
-            expect_query(1) do
-              Band.where(name: 'DEPECHE MODE').to_a
-            end
-          end
-        end
-      end
-
       context "when the first query has no limit" do
 
         let(:game) do
@@ -305,37 +360,6 @@ describe Mongoid::QueryCache do
     end
   end
 
-  context "when reloading a document" do
-
-    let!(:band_id) do
-      Band.create.id
-    end
-
-    context 'when query cache is disabled' do
-
-      before do
-        Mongoid::QueryCache.enabled = false
-      end
-
-      it "queries again" do
-        band = Band.find(band_id)
-        expect_query(1) do
-          band.reload
-        end
-      end
-    end
-
-    context 'when query cache is enabled' do
-
-      it "queries again" do
-        band = Band.find(band_id)
-        expect_query(1) do
-          band.reload
-        end
-      end
-    end
-  end
-
   context "when querying a very large collection" do
 
     before do

data/spec/mongoid/relations/accessors_spec.rb

@@ -728,7 +728,7 @@ describe Mongoid::Relations::Accessors do
         game.save
       end
 
-      it "keeps the the foreign key as ObjectID" do
+      it "keeps the foreign key as ObjectID" do
         expect(game.reload.person_id).to eq(person.id)
       end
     end

data/spec/mongoid/relations/auto_save_spec.rb

@@ -70,13 +70,44 @@ describe Mongoid::Relations::AutoSave do
 
         context "when saving a new parent document" do
 
-          before do
-            person.drugs << drug
-            person.save
+          context 'when persistence options are not set on the parent' do
+            before do
+              person.drugs << drug
+              person.save
+            end
+
+            it "saves the relation" do
+              expect(drug).to be_persisted
+            end
           end
 
-          it "saves the relation" do
-            expect(drug).to be_persisted
+          context 'when persistence options are set on the parent' do
+
+            let(:other_database) do
+              :other
+            end
+
+            after do
+              Person.with(database: other_database) do |person_class|
+                person_class.delete_all
+              end
+              Drug.with(database: other_database) do |drug_class|
+                drug_class.delete_all
+              end
+            end
+
+            before do
+              person.with(database: other_database) do |per|
+                per.drugs << drug
+                per.save
+              end
+            end
+
+            it 'saves the relation with the persistence options' do
+              Drug.with(database: other_database) do |drug_class|
+                expect(drug_class.count).to eq(1)
+              end
+            end
           end
         end
 
@@ -160,7 +191,9 @@ describe Mongoid::Relations::AutoSave do
           it "sends one insert" do
             account.name = "account"
             expect_query(1) do
-              person.with(write: {w:0}).save
+              person.with(write: {w:0}) do |_person|
+                _person.save
+              end
             end
           end
         end

data/spec/mongoid/relations/builders_spec.rb

@@ -185,20 +185,47 @@ describe Mongoid::Relations::Builders do
 
         context "when a document already exists" do
 
-          let!(:game_two) do
-            person.create_game(name: "Skyrim")
-          end
+          context "when the relation belongs_to is already set" do
 
-          it "replaces the existing document" do
-            expect(person.game).to eq(game_two)
-          end
+            let!(:game_one) do
+              game = person.create_game(name: "Starcraft")
+              game.person = person
+            end
+
+            let!(:game_two) do
+              person.create_game(name: "Skyrim")
+            end
+
+            it "replaces the existing document" do
+              expect(person.game).to eq(game_two)
+            end
+
+            it "persists the change" do
+              expect(person.game(true)).to eq(game_two)
+            end
 
-          it "persists the change" do
-            expect(person.game(true)).to eq(game_two)
+            it "removes the old document from the database" do
+              expect(Game.collection.find.count).to eq(1)
+            end
           end
 
-          it "removes the old document from the database" do
-            expect(Game.collection.find.count).to eq(1)
+          context 'when the relation belongs_to is not already set' do
+
+            let!(:game_two) do
+              person.create_game(name: "Skyrim")
+            end
+
+            it "replaces the existing document" do
+              expect(person.game).to eq(game_two)
+            end
+
+            it "persists the change" do
+              expect(person.game(true)).to eq(game_two)
+            end
+
+            it "removes the old document from the database" do
+              expect(Game.collection.find.count).to eq(1)
+            end
           end
         end
       end

data/spec/mongoid/relations/counter_cache_spec.rb

@@ -34,7 +34,7 @@ describe Mongoid::Relations::CounterCache do
       it "expect to raise an error" do
         expect {
           person.reset_counters(:not_exist)
-        }.to raise_error
+        }.to raise_error(NoMethodError)
       end
     end
 
@@ -85,6 +85,26 @@ describe Mongoid::Relations::CounterCache do
         expect(subscription.reload[:packs_count]).to eq(1)
       end
     end
+
+    context 'when there are persistence options set' do
+
+      let(:subscription) do
+        Subscription.new
+      end
+
+      before do
+        subscription.with(collection: 'other') do |sub|
+          sub.save
+          sub.packs.create
+        end
+      end
+
+      it 'applies the persistence options when resetting the counter' do
+        subscription.with(collection: 'other') do |sub|
+          expect(sub.reload[:packs_count]).to eq(1)
+        end
+      end
+    end
   end
 
   describe ".reset_counters" do
@@ -111,7 +131,7 @@ describe Mongoid::Relations::CounterCache do
       it "expect to raise an error" do
         expect {
           Person.reset_counters "1", :drugs
-        }.to raise_error
+        }.to raise_error(Mongoid::Errors::DocumentNotFound)
       end
     end
 
@@ -124,7 +144,7 @@ describe Mongoid::Relations::CounterCache do
       it "expect to raise an error" do
         expect {
           Person.reset_counters person.id, :not_exist
-        }.to raise_error
+        }.to raise_error(NoMethodError)
       end
     end
 
@@ -256,6 +276,26 @@ describe Mongoid::Relations::CounterCache do
         expect(person.reload.drugs_count).to eq(3)
       end
     end
+
+    context 'when there are persistence options set' do
+
+      let(:person) do
+        Person.new
+      end
+
+      before do
+        person.with(collection: 'other') do |per|
+          per.save
+          per.drugs.create
+        end
+      end
+
+      it 'applies the persistence options when resetting the counter' do
+        person.with(collection: 'other') do |per|
+          expect(per.drugs_count).to eq(1)
+        end
+      end
+    end
   end
 
   describe "#decrement_counter" do
@@ -286,6 +326,27 @@ describe Mongoid::Relations::CounterCache do
         expect(person.reload.drugs_count).to eq(0)
       end
     end
+
+    context 'when there are persistence options set' do
+
+      let(:person) do
+        Person.new
+      end
+
+      before do
+        person.with(collection: 'other') do |per|
+          per.save
+          drug = per.drugs.create
+          drug.destroy
+        end
+      end
+
+      it 'applies the persistence options when resetting the counter' do
+        person.with(collection: 'other') do |per|
+          expect(per.reload.drugs_count).to eq(0)
+        end
+      end
+    end
   end
 
   describe "#add_counter_cache_callbacks" do
@@ -351,18 +412,6 @@ describe Mongoid::Relations::CounterCache do
         it "updates the original object's counter cache" do
           expect(person1.reload.drugs_count).to eq(1)
         end
-
-        context 'when foreign_key differs from model name' do
-
-          let(:genre) { PostGenre.create }
-
-          let(:post) { Post.create }
-
-          it 'updates correct counter cache' do
-            post.update post_genre: genre
-            expect(genre.reload.posts_count).to eq 1
-          end
-        end
       end
     end