mongoid 7.1.6 → 7.2.2

data/spec/integration/associations/embedded_spec.rb

@@ -172,4 +172,98 @@ describe 'embedded associations' do
       end
     end
   end
+
+  describe 'updated_at on parent' do
+    context 'when child is added' do
+      context 'embeds_one' do
+        let(:parent) { EomParent.create! }
+
+        it 'updates' do
+          pending 'https://jira.mongodb.org/browse/MONGOID-3252'
+
+          first_updated_at = parent.updated_at
+
+          parent.child = EomChild.new
+          parent.save!
+
+          parent.updated_at.should > first_updated_at
+        end
+      end
+
+      context 'embeds_many' do
+        let(:parent) { EmmCongress.create! }
+
+        it 'updates' do
+          pending 'https://jira.mongodb.org/browse/MONGOID-3252'
+
+          first_updated_at = parent.updated_at
+
+          parent.legislators << EmmLegislator.new
+          parent.save!
+
+          parent.updated_at.should > first_updated_at
+        end
+      end
+    end
+
+    context 'when child is updated' do
+      context 'embeds_one' do
+        let(:parent) { EomParent.create!(child: EomChild.new) }
+
+        it 'updates' do
+          first_updated_at = parent.updated_at
+
+          parent.child.a = 42
+          parent.save!
+
+          parent.updated_at.should > first_updated_at
+        end
+      end
+
+      context 'embeds_many' do
+        let(:parent) { EmmCongress.create!(legislators: [EmmLegislator.new]) }
+
+        it 'updates' do
+          first_updated_at = parent.updated_at
+
+          parent.legislators.first.a = 42
+          parent.save!
+
+          parent.updated_at.should > first_updated_at
+        end
+      end
+    end
+
+    context 'when child is removed' do
+      context 'embeds_one' do
+        let(:parent) { EomParent.create!(child: EomChild.new) }
+
+        it 'updates' do
+          pending 'https://jira.mongodb.org/browse/MONGOID-3252'
+
+          first_updated_at = parent.updated_at
+
+          parent.child = nil
+          parent.save!
+
+          parent.updated_at.should > first_updated_at
+        end
+      end
+
+      context 'embeds_many' do
+        let(:parent) { EmmCongress.create!(legislators: [EmmLegislator.new]) }
+
+        it 'updates' do
+          pending 'https://jira.mongodb.org/browse/MONGOID-3252'
+
+          first_updated_at = parent.updated_at
+
+          parent.legislators.clear
+          parent.save!
+
+          parent.updated_at.should > first_updated_at
+        end
+      end
+    end
+  end
 end

data/spec/integration/associations/has_many_spec.rb

@@ -2,33 +2,86 @@
 # encoding: utf-8
 
 require 'spec_helper'
+require 'mongoid/association/referenced/has_many_models'
 
 describe 'has_many associations' do
   context 'destroying parent in transaction with dependent child' do
     require_transaction_support
 
-    let(:artist) { Artist.create! }
-    let(:album) { Album.create!(artist: artist) }
+    let(:company) { HmmCompany.create! }
+    let(:address) { HmmAddress.create!(company: company) }
 
     before do
-      Artist.delete_all
-      Album.delete_all
-
-      album
+      Company.delete_all
+      Address.delete_all
     end
 
-    it 'works' do
-      Artist.count.should == 1
-      Album.count.should == 1
+    context 'dependent: :destroy' do
+      before do
+        HmmCompany.class_eval do
+          has_one :address, class_name: 'HmmAddress', dependent: :destroy
+        end
+      end
+
+      it 'destroys' do
+        address
+
+        HmmCompany.count.should == 1
+        HmmAddress.count.should == 1
 
-      artist.with_session do |session|
-        session.with_transaction do
-          artist.destroy
+        company.with_session do |session|
+          session.with_transaction do
+            company.destroy!
+          end
         end
+
+        HmmCompany.count.should == 0
+        HmmAddress.count.should == 0
+      end
+    end
+
+    context 'dependent: :restrict_with_error' do
+      before do
+        HmmCompany.class_eval do
+          has_one :address, class_name: 'HmmAddress', dependent: :restrict_with_error
+        end
+      end
+
+      it 'destroys' do
+        address
+
+        HmmCompany.count.should == 1
+        HmmAddress.count.should == 1
+
+        lambda do
+          company.with_session do |session|
+            session.with_transaction do
+              company.destroy!
+            end
+          end
+        end.should raise_error(Mongoid::Errors::DocumentNotDestroyed)
+
+        HmmCompany.count.should == 1
+        HmmAddress.count.should == 1
       end
+    end
+  end
 
-      Artist.count.should == 0
-      Album.count.should == 0
+  context 'when child does not have parent association' do
+    context 'Child.new' do
+      it 'creates a child instance' do
+        HmmBusSeat.new.should be_a(HmmBusSeat)
+      end
+    end
+
+    context 'assignment to child in parent' do
+      let(:parent) { HmmBus.new }
+
+      it 'raises InverseNotFound' do
+        lambda do
+          parent.seats << HmmBusSeat.new
+        end.should raise_error(Mongoid::Errors::InverseNotFound)
+      end
     end
   end
 

data/spec/integration/associations/has_one_spec.rb

@@ -2,33 +2,86 @@
 # encoding: utf-8
 
 require 'spec_helper'
+require 'mongoid/association/referenced/has_one_models'
 
 describe 'has_one associations' do
   context 'destroying parent in transaction with dependent child' do
     require_transaction_support
 
-    let(:person) { Person.create! }
-    let(:game) { Game.create!(person: person) }
+    let(:college) { HomCollege.create! }
+    let(:address) { HomAddress.create!(college: college) }
 
     before do
-      Person.delete_all
-      Game.delete_all
-
-      game
+      HomCollege.delete_all
+      HomAddress.delete_all
     end
 
-    it 'works' do
-      Person.count.should == 1
-      Game.count.should == 1
+    context 'dependent: :destroy' do
+      before do
+        HomCollege.class_eval do
+          has_one :address, class_name: 'HomAddress', dependent: :destroy
+        end
+      end
+
+      it 'destroys' do
+        address
+
+        HomCollege.count.should == 1
+        HomAddress.count.should == 1
 
-      Person.with_session do |session|
-        session.with_transaction do
-          person.destroy
+        HomCollege.with_session do |session|
+          session.with_transaction do
+            college.destroy!
+          end
         end
+
+        HomCollege.count.should == 0
+        HomAddress.count.should == 0
+      end
+    end
+
+    context 'dependent: :restrict_with_error' do
+      before do
+        HomCollege.class_eval do
+          has_one :address, class_name: 'HomAddress', dependent: :restrict_with_error
+        end
+      end
+
+      it 'does not destroy' do
+        address
+
+        HomCollege.count.should == 1
+        HomAddress.count.should == 1
+
+        lambda do
+          HomCollege.with_session do |session|
+            session.with_transaction do
+              college.destroy!
+            end
+          end
+        end.should raise_error(Mongoid::Errors::DocumentNotDestroyed)
+
+        HomCollege.count.should == 1
+        HomAddress.count.should == 1
       end
+    end
+  end
 
-      Person.count.should == 0
-      Game.count.should == 0
+  context 'when child does not have parent association' do
+    context 'Child.new' do
+      it 'creates a child instance' do
+        HomBusDriver.new.should be_a(HomBusDriver)
+      end
+    end
+
+    context 'assignment to child in parent' do
+      let(:parent) { HomBus.new }
+
+      it 'raises InverseNotFound' do
+        lambda do
+          parent.driver = HomBusDriver.new
+        end.should raise_error(Mongoid::Errors::InverseNotFound)
+      end
     end
   end
 

data/spec/integration/associations/nested_attributes_assignment_spec.rb

@@ -0,0 +1,116 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe 'nested attributes assignment' do
+  context 'when creating parent document' do
+    context 'when setting deeply nested attributes' do
+      context 'embeds_many' do
+        let(:truck) { Truck.new }
+
+        it 'persists all documents' do
+          truck.attributes = {
+            capacity: 1,
+            crates_attributes: {
+              '0' => {
+                volume: 2,
+                toys_attributes: {
+                  '0' => {
+                    name: 'Bear',
+                  },
+                },
+              },
+            },
+          }
+
+          truck.save!
+
+          _truck = Truck.find(truck.id)
+          _truck.capacity.should == 1
+          _truck.crates.length.should == 1
+          _truck.crates.first.volume.should == 2
+          _truck.crates.first.toys.length.should == 1
+          _truck.crates.first.toys.first.name.should == 'Bear'
+        end
+      end
+    end
+  end
+
+  context 'when updating parent document' do
+    context 'when setting deeply nested attributes' do
+      context 'embeds_many' do
+        let(:truck) do
+          Truck.create!(
+            capacity: 1,
+            crates: [Crate.new(
+              volume: 2,
+              toys: [Toy.new(name: 'Bear')],
+            )],
+          )
+        end
+
+        context 'updating embedded documents' do
+
+          it 'persists all documents' do
+            truck.attributes = {
+              capacity: 2,
+              crates_attributes: {
+                '0' => {
+                  id: truck.crates.first.id,
+                  volume: 3,
+                  toys_attributes: {
+                    '0' => {
+                      id: truck.crates.first.toys.first.id,
+                      name: 'Rhino',
+                    },
+                  },
+                },
+              },
+            }
+
+            truck.save!
+
+            _truck = Truck.find(truck.id)
+            _truck.capacity.should == 2
+            _truck.crates.length.should == 1
+            _truck.crates.first.volume.should == 3
+            _truck.crates.first.toys.length.should == 1
+            _truck.crates.first.toys.first.name.should == 'Rhino'
+          end
+        end
+
+        context 'adding embedded documents' do
+
+          it 'persists all changes' do
+            truck.attributes = {
+              capacity: 2,
+              crates_attributes: {
+                '0' => {
+                  volume: 3,
+                  toys_attributes: {
+                    '0' => {
+                      name: 'Rhino',
+                    },
+                  },
+                },
+              },
+            }
+
+            truck.save!
+
+            _truck = Truck.find(truck.id)
+            _truck.capacity.should == 2
+            _truck.crates.length.should == 2
+            _truck.crates.first.volume.should == 2
+            _truck.crates.first.toys.length.should == 1
+            _truck.crates.first.toys.first.name.should == 'Bear'
+            _truck.crates.last.volume.should == 3
+            _truck.crates.last.toys.length.should == 1
+            _truck.crates.last.toys.last.name.should == 'Rhino'
+          end
+        end
+      end
+    end
+  end
+end

data/spec/integration/atomic/modifiers_spec.rb

@@ -0,0 +1,117 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
+require "spec_helper"
+
+describe Mongoid::Atomic::Modifiers do
+
+  let(:modifiers) do
+    described_class.new
+  end
+
+  context 'when performing multiple operations with similar keys' do
+
+    let(:pushes) do
+      { "addresses.0.locations" => { "street" => "Bond St" } }
+    end
+
+    let(:similar1) do
+      { "dresses" => { "color" => "red" } }
+    end
+
+    let(:similar2) do
+      { "ses.0.foo" => { "baz" => "qux" } }
+    end
+
+    before do
+      modifiers.send(operation, pushes)
+      modifiers.send(operation, similar1)
+      modifiers.send(operation, similar2)
+    end
+
+    context "push" do
+      let(:operation) { :push }
+
+      it "adds all modifiers to top level" do
+        expect(modifiers).to eq({ "$push" => {
+                                    "addresses.0.locations" => { '$each' => [{ "street" => "Bond St" }] },
+                                    "dresses" => { '$each' => [{ "color" => "red" }] },
+                                    "ses.0.foo" => { '$each' => [{ "baz" => "qux" }] }
+                                 } })
+      end
+    end
+
+    context "pull" do
+      let(:operation) { :pull }
+
+      it "adds all modifiers to top level" do
+        expect(modifiers).to eq({ "$pull" => {
+                                    "addresses.0.locations" => { "street" => "Bond St" },
+                                    "dresses" => { "color" => "red" },
+                                    "ses.0.foo" => { "baz" => "qux" },
+                                 } })
+      end
+    end
+
+    context "pull_all" do
+      let(:operation) { :pull_all }
+
+      it "adds all modifiers to top level" do
+        expect(modifiers).to eq({ "$pullAll" => {
+                                    "addresses.0.locations" => { "street" => "Bond St" },
+                                    "dresses" => { "color" => "red" },
+                                    "ses.0.foo" => { "baz" => "qux" },
+                                 } })
+      end
+    end
+
+    context "add_to_set" do
+      let(:operation) { :add_to_set }
+
+      it "adds all modifiers to top level" do
+        expect(modifiers).to eq({ "$addToSet" => {
+                                    "addresses.0.locations" => { '$each' => { "street" => "Bond St" } },
+                                    "dresses" => { '$each' => { "color" => "red" } },
+                                    "ses.0.foo" => { '$each' => { "baz" => "qux" } }
+                                 } })
+      end
+    end
+
+    context "set" do
+      let(:operation) { :set }
+
+      it "adds all modifiers to top level" do
+        expect(modifiers).to eq({ "$set" => {
+                                    "addresses.0.locations" => { "street" => "Bond St" },
+                                    "dresses" => { "color" => "red" },
+                                    "ses.0.foo" => { "baz" => "qux" },
+                                 } })
+      end
+    end
+
+    context "unset" do
+      let(:operation) { :unset }
+
+      let(:pushes) do
+        [:foobar]
+      end
+
+      let(:similar1) do
+        [:foo]
+      end
+
+      let(:similar2) do
+        [:bar]
+      end
+
+      it "adds all modifiers to top level" do
+        expect(modifiers).to eq("$unset" => {
+          foobar: true,
+          foo: true,
+          bar: true,
+        })
+      end
+    end
+  end
+
+end