mongoid 7.0.12 → 7.3.3

data/spec/integration/associations/embeds_many_spec.rb

@@ -0,0 +1,68 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe 'embeds_many associations' do
+
+  context 're-associating the same object' do
+    context 'with dependent: destroy' do
+      let(:canvas) do
+        Canvas.create!(shapes: [Shape.new])
+      end
+
+      let!(:shape) { canvas.shapes.first }
+
+      it 'does not destroy the dependent object' do
+        canvas.shapes = [shape]
+        canvas.save!
+        canvas.reload
+        canvas.shapes.should == [shape]
+      end
+    end
+  end
+
+  context 'clearing association when parent is not saved' do
+    let!(:parent) { Canvas.create!(shapes: [Shape.new]) }
+
+    let(:unsaved_parent) { Canvas.new(id: parent.id, shapes: [Shape.new]) }
+
+    context "using #clear" do
+      it 'deletes the target from the database' do
+        unsaved_parent.shapes.clear
+
+        unsaved_parent.shapes.should be_empty
+
+        unsaved_parent.new_record?.should be true
+        parent.reload
+        parent.shapes.should be_empty
+      end
+    end
+
+    shared_examples 'does not delete the target from the database' do
+      it 'does not delete the target from the database' do
+        unsaved_parent.shapes.should be_empty
+
+        unsaved_parent.new_record?.should be true
+        parent.reload
+        parent.shapes.length.should == 1
+      end
+    end
+
+    context "using #delete_all" do
+      before do
+        unsaved_parent.shapes.delete_all
+      end
+
+      include_examples 'does not delete the target from the database'
+    end
+
+    context "using #destroy_all" do
+      before do
+        unsaved_parent.shapes.destroy_all
+      end
+
+      include_examples 'does not delete the target from the database'
+    end
+  end
+end

data/spec/integration/associations/embeds_one_spec.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe 'embeds_one associations' do
+
+  context 're-associating the same object' do
+    context 'with dependent: destroy' do
+      let(:canvas) do
+        Canvas.create!(palette: Palette.new)
+      end
+
+      let!(:palette) { canvas.palette }
+
+      it 'does not destroy the dependent object' do
+        canvas.palette = canvas.palette
+        canvas.save!
+        canvas.reload
+        canvas.palette.should == palette
+      end
+    end
+  end
+end

data/spec/integration/associations/foreign_key_spec.rb

@@ -0,0 +1,99 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
+require 'spec_helper'
+require_relative './foreign_key_spec_models'
+
+describe 'Association foreign key configuration' do
+  describe 'has_many/belongs_to' do
+    it 'creates child' do
+      company = ForeignKeySpec::Company.create!(c: 'test')
+      email = ForeignKeySpec::Email.create!(company: company)
+      expect(email.c_ref).to eq('test')
+    end
+
+    it 'looks up child' do
+      company = ForeignKeySpec::Company.create!(c: 'test')
+      email = ForeignKeySpec::Email.create!(company: company)
+
+      company = ForeignKeySpec::Company.find(company.id)
+      expect(company.emails.first.id).to eq(email.id)
+    end
+
+    it 'looks up parent' do
+      company = ForeignKeySpec::Company.create!(c: 'test')
+      email = ForeignKeySpec::Email.create!(company: company)
+
+      email = ForeignKeySpec::Email.find(email.id)
+      expect(email.company.id).to eq(company.id)
+    end
+
+    context 'with default scope' do
+      context 'using default scope' do
+        it 'looks up child' do
+          company = ForeignKeySpec::ScopedCompany.create!(c: 'test')
+          on_email = ForeignKeySpec::ScopedEmail.create!(company: company, s: 'on')
+          off_email = ForeignKeySpec::ScopedEmail.create!(company: company, s: 'off')
+
+          company = ForeignKeySpec::ScopedCompany.find(company.id)
+          expect(company.emails.length).to eq(1)
+          expect(company.emails.first.id).to eq(on_email.id)
+        end
+      end
+
+      context 'unscoped' do
+        it 'looks up child' do
+          company = ForeignKeySpec::ScopedCompany.create!(c: 'test')
+          on_email = ForeignKeySpec::ScopedEmail.create!(company: company, s: 'on')
+          off_email = ForeignKeySpec::ScopedEmail.create!(company: company, s: 'off')
+
+          company = ForeignKeySpec::ScopedCompany.find(company.id)
+          expect(company.emails.unscoped.length).to eq(2)
+          expect(company.emails.unscoped.map(&:id).sort).to eq([on_email.id, off_email.id].sort)
+        end
+      end
+    end
+  end
+
+  describe 'has_one/belongs_to' do
+    it 'creates child' do
+      company = ForeignKeySpec::Company.create!(c: 'test')
+      founder = ForeignKeySpec::Founder.create!(company: company)
+      expect(founder.c_ref).to eq('test')
+    end
+
+    it 'looks up child' do
+      company = ForeignKeySpec::Company.create!(c: 'test')
+      founder = ForeignKeySpec::Founder.create!(company: company)
+
+      company = ForeignKeySpec::Company.find(company.id)
+      expect(company.founder.id).to eq(founder.id)
+    end
+
+    it 'looks up parent' do
+      company = ForeignKeySpec::Company.create!(c: 'test')
+      founder = ForeignKeySpec::Founder.create!(company: company)
+
+      email = ForeignKeySpec::Founder.find(founder.id)
+      expect(founder.company.id).to eq(company.id)
+    end
+  end
+
+  describe 'has_and_belongs_to_many' do
+    it 'persists association on the other side' do
+      animal = ForeignKeySpec::Animal.create!(a: 'bear')
+      zoo = ForeignKeySpec::Zoo.create!(z: 'bz')
+      animal.zoos << zoo
+      animal.save!
+
+      # This write should not be necessary, but without it the test fails.
+      # https://jira.mongodb.org/browse/MONGOID-4648
+      zoo.animals << animal
+      zoo.save!
+
+      expect(zoo.animals).to eq([animal])
+      zoo = ForeignKeySpec::Zoo.find(zoo.id)
+      expect(zoo.animals).to eq([animal])
+    end
+  end
+end

data/spec/integration/associations/foreign_key_spec_models.rb

@@ -0,0 +1,65 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
+module ForeignKeySpec
+  class Company
+    include Mongoid::Document
+
+    field :c, type: String
+    has_many :emails, class_name: 'ForeignKeySpec::Email',
+      foreign_key: 'c_ref', primary_key: 'c'
+    has_one :founder, class_name: 'ForeignKeySpec::Founder',
+      foreign_key: 'c_ref', primary_key: 'c'
+  end
+
+  class Email
+    include Mongoid::Document
+
+    field :c_ref, type: String
+    belongs_to :company, class_name: 'ForeignKeySpec::Company',
+      foreign_key: 'c_ref', primary_key: 'c'
+  end
+
+  class Founder
+    include Mongoid::Document
+
+    field :c_ref, type: String
+    belongs_to :company, class_name: 'ForeignKeySpec::Company',
+      foreign_key: 'c_ref', primary_key: 'c'
+  end
+
+  class Animal
+    include Mongoid::Document
+
+    field :a, type: String
+    has_and_belongs_to_many :zoos, class_name: 'ForeignKeySpec::Zoo',
+      foreign_key: 'z_refs', primary_key: 'z'
+  end
+
+  class Zoo
+    include Mongoid::Document
+
+    field :z, type: String
+    has_and_belongs_to_many :animals, class_name: 'ForeignKeySpec::Animal',
+      foreign_key: 'a_refs', primary_key: 'a'
+  end
+
+  class ScopedCompany
+    include Mongoid::Document
+
+    field :c, type: String
+    has_many :emails, class_name: 'ForeignKeySpec::ScopedEmail',
+      foreign_key: 'c_ref', primary_key: 'c'
+  end
+
+  class ScopedEmail
+    include Mongoid::Document
+
+    field :c_ref, type: String
+    belongs_to :company, class_name: 'ForeignKeySpec::ScopedCompany',
+      foreign_key: 'c_ref', primary_key: 'c'
+
+    field :s, type: String
+    default_scope -> { where(s: 'on') }
+  end
+end

data/spec/integration/associations/has_many_spec.rb

@@ -0,0 +1,129 @@
+# frozen_string_literal: true
+# 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(:company) { HmmCompany.create! }
+    let(:address) { HmmAddress.create!(company: company) }
+
+    before do
+      Company.delete_all
+      Address.delete_all
+    end
+
+    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
+
+        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
+
+  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
+
+  context 're-associating the same object' do
+    context 'with dependent: destroy' do
+      let(:wiki_page) do
+        WikiPage.create!
+      end
+
+      let!(:comment) do
+        Comment.create!(wiki_page: wiki_page, title: 'hi') do
+          wiki_page.reload
+        end
+      end
+
+      it 'does not destroy the dependent object' do
+        wiki_page.comments.should == [comment]
+        wiki_page.comments = [comment]
+        wiki_page.save!
+        wiki_page.reload
+        wiki_page.comments.should == [comment]
+      end
+    end
+
+    context 'without dependent: destroy' do
+      let(:series) do
+        Series.create!
+      end
+
+      let!(:book) do
+        Book.create!(series: series).tap do
+          series.reload
+        end
+      end
+
+      it 'does not destroy the dependent object' do
+        series.books.should == [book]
+        series.books = [book]
+        series.save!
+        series.reload
+        series.books.should == [book]
+      end
+    end
+  end
+end

data/spec/integration/associations/has_one_spec.rb

@@ -0,0 +1,177 @@
+# frozen_string_literal: true
+# 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(:college) { HomCollege.create! }
+    let(:address) { HomAddress.create!(college: college) }
+
+    before do
+      HomCollege.delete_all
+      HomAddress.delete_all
+    end
+
+    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
+
+        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
+
+  context 'when calling methods on target' do
+    let(:parent) do
+      HomCollege.create!.tap do |college|
+        HomAccreditation.create!(college: college)
+      end
+    end
+
+    shared_examples 'delegates to the field' do |reloaded: false|
+      context 'non-conflicting field name' do
+        it 'delegates to the field' do
+          parent.accreditation.price.should == 42
+        end
+
+        context 'using send' do
+          it 'delegates to the field' do
+            parent.accreditation.send(:price).should == 42
+          end
+        end
+      end
+
+      context 'field name that conflicts with Kernel' do
+        it 'delegates to the field' do
+          parent.accreditation.format.should == 'fmt'
+        end
+
+        context 'using send' do
+          it 'delegates to the field' do
+            if reloaded
+              pending 'MONGOID-4018'
+            end
+
+            parent.accreditation.send(:format).should == 'fmt'
+          end
+        end
+      end
+    end
+
+    include_examples 'delegates to the field'
+
+    context 'after reloading parent' do
+      before do
+        parent.reload
+      end
+
+      include_examples 'delegates to the field', reloaded: true
+    end
+  end
+
+  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
+
+  context 're-associating the same object' do
+    context 'with dependent: destroy' do
+      let(:person) do
+        Person.create!
+      end
+
+      let!(:game) do
+        Game.create!(person: person) do
+          person.reload
+        end
+      end
+
+      it 'does not destroy the dependent object' do
+        person.game.should == game
+        person.game = person.game
+        person.save!
+        person.reload
+        person.game.should == game
+      end
+    end
+
+    context 'without dependent: destroy' do
+      let(:person) do
+        Person.create!
+      end
+
+      let!(:account) do
+        Account.create!(person: person, name: 'foo').tap do
+          person.reload
+        end
+      end
+
+      it 'does not destroy the dependent object' do
+        person.account.should == account
+        person.account = person.account
+        person.save!
+        person.reload
+        person.account.should == account
+      end
+    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