mongoid 7.1.11 → 7.2.0.rc1

data/spec/mongoid/association/depending_spec.rb

@@ -489,7 +489,7 @@ describe Mongoid::Association::Depending do
           end
         end
 
-        context "when dependent is restrict_with_error" do
+        context "when dependent is restrict_with_exception" do
 
           context "when restricting a references many" do
 
@@ -825,44 +825,96 @@ describe Mongoid::Association::Depending do
 
   context 'when the strategy is :restrict_with_error' do
 
-    let(:person) do
-      Person.new
-    end
+    context "when restricting a one-to-many" do
+      
+      let(:person) do
+        Person.new
+      end
+  
+      let(:post) do
+        Post.new
+      end
 
-    let(:post) do
-      Post.new
-    end
+      let!(:association) do
+        Person.has_many :restrictable_posts, class_name: "Post", dependent: :restrict_with_error
+      end
 
-    let!(:association) do
-      Person.has_many :restrictable_posts, class_name: "Post", dependent: :restrict_with_error
-    end
+      after do
+        Person.dependents.delete(association)
+      end
 
-    after do
-      Person.dependents.delete(association)
-    end
+      context 'when there are related objects' do
 
-    context 'when there are related objects' do
+        before do
+          person.restrictable_posts << post
+        end
 
-      before do
-        person.restrictable_posts << post
+        it 'adds an error to the parent object' do
+          expect(person.delete).to be(false)
+  
+          person.errors[:restrictable_posts].first.should ==
+            "is not empty and prevents the document from being destroyed"
+        end
       end
 
-      it 'adds an error to the parent object' do
-        expect(person.delete).to be(false)
-        expect(person.errors[:restrictable_posts].first).to be(
-          Mongoid::Association::Depending::RESTRICT_ERROR_MSG)
+      context 'when there are no related objects' do
+
+        before do
+          expect(post).to receive(:delete).never
+          expect(post).to receive(:destroy).never
+        end
+
+        it 'deletes the object and leaves the other one intact' do
+          expect(person.delete).to be(true)
+        end
+      end
+
+      context 'when deleted inside a transaction' do
+        require_transaction_support
+        
+        before do
+          person.restrictable_posts << post
+        end
+
+        it 'doesn\'t raise an exception' do
+          person.with_session do |session|
+            session.with_transaction do 
+              expect { person.destroy }.to_not raise_error
+            end
+          end
+        end
       end
     end
 
-    context 'when there are no related objects' do
+    context "when restricting a many to many" do
 
-      before do
-        expect(post).to receive(:delete).never
-        expect(post).to receive(:destroy).never
+      let!(:association) do
+        Person.has_and_belongs_to_many :houses, dependent: :restrict_with_error
       end
 
-      it 'deletes the object and leaves the other one intact' do
-        expect(person.delete).to be(true)
+      after do
+        Person.dependents.delete(association)
+        Person.has_and_belongs_to_many :houses, validate: false
+      end
+
+      let(:person) do
+        Person.new houses: [House.new]
+      end
+
+      it "returns false" do
+        expect(person.destroy).to be false
+      end
+
+      context "when inside a transaction" do
+        require_transaction_support
+        
+        it 'doesn\'t raise an exception inside a transaction' do
+          person.with_session do |session|
+            session.with_transaction do
+              expect { person.destroy }.to_not raise_error
+            end
+          end
+        end
       end
     end
   end

data/spec/mongoid/association/embedded/embedded_in/proxy_spec.rb

@@ -507,54 +507,4 @@ describe Mongoid::Association::Embedded::EmbeddedIn::Proxy do
       end
     end
   end
-
-  context "when the same class is embedded multiple times" do
-
-    let(:customer) do
-      Customer.new
-    end
-
-    context "assignment after saving" do
-
-      it "correctly sets the association for the embedded class" do
-        pending 'MONGOID-5039'
-
-        customer.home_address = CustomerAddress.new
-        customer.work_address = CustomerAddress.new
-
-        expect(customer.home_address._association.store_as).to eq("home_address")
-        expect(customer.work_address._association.store_as).to eq("work_address")
-
-        expect(customer.home_address.instance_eval { _association.store_as }).to eq("home_address")
-        expect(customer.work_address.instance_eval { _association.store_as }).to eq("work_address")
-
-        customer.save!
-
-        customer.home_address = CustomerAddress.new
-        customer.work_address = CustomerAddress.new
-
-        expect(customer.home_address._association.store_as).to eq("home_address")
-        expect(customer.work_address._association.store_as).to eq("work_address")
-
-        expect(customer.home_address.instance_eval { _association.store_as }).to eq("home_address")
-        expect(customer.work_address.instance_eval { _association.store_as }).to eq("work_address")
-      end
-    end
-
-    context "inverse assignment" do
-
-      it "correctly sets the association for the embedded class" do
-        pending 'MONGOID-5039'
-
-        customer.work_address = CustomerAddress.new
-        customer.work_address.addressable = customer
-
-        expect(customer.home_address._association.store_as).to eq("home_address")
-        expect(customer.work_address._association.store_as).to eq("work_address")
-
-        expect(customer.home_address.instance_eval { _association.store_as }).to eq("home_address")
-        expect(customer.work_address.instance_eval { _association.store_as }).to eq("work_address")
-      end
-    end
-  end
 end

data/spec/mongoid/association/embedded/embeds_many/proxy_spec.rb

@@ -1402,6 +1402,39 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     end
   end
 
+  describe "#any?" do
+
+    let(:person) do
+      Person.create
+    end
+
+    context "when documents are persisted" do
+      before do
+        person.addresses.create(street: "Upper")
+      end
+
+      it "returns true" do
+        expect(person.addresses.any?).to be true
+      end
+    end
+    
+    context "when documents are not persisted" do
+      before do
+        person.addresses.build(street: "Bond")
+      end
+      
+      it "returns true" do
+        expect(person.addresses.any?).to be true
+      end
+    end
+
+    context "when documents are not created" do
+      it "returns false" do
+        expect(person.addresses.any?).to be false
+      end
+    end
+  end
+
   describe "#create" do
 
     context "when providing multiple attributes" do
@@ -2416,26 +2449,13 @@ describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
     end
 
     context "when providing a criteria class method" do
-      context "without keyword arguments" do
-
-        let(:addresses) do
-          person.addresses.california
-        end
 
-        it "applies the criteria to the documents" do
-          expect(addresses).to eq([ address_one ])
-        end
+      let(:addresses) do
+        person.addresses.california
       end
 
-      context "with keyword arguments" do
-
-        let(:addresses) do
-          person.addresses.city_and_state(city: "Sacramento", state: "CA")
-        end
-
-        it "applies the criteria to the documents" do
-          expect(addresses).to eq([])
-        end
+      it "applies the criteria to the documents" do
+        expect(addresses).to eq([ address_one ])
       end
     end
 

data/spec/mongoid/association/embedded/embeds_many_models.rb

@@ -3,6 +3,7 @@
 
 class EmmCongress
   include Mongoid::Document
+  include Mongoid::Timestamps
 
   embeds_many :legislators, class_name: 'EmmLegislator'
 

data/spec/mongoid/association/embedded/embeds_one_models.rb

@@ -3,6 +3,7 @@
 
 class EomParent
   include Mongoid::Document
+  include Mongoid::Timestamps
 
   embeds_one :child, class_name: 'EomChild'
 

data/spec/mongoid/association/referenced/belongs_to/proxy_spec.rb

@@ -1332,21 +1332,4 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
       end
     end
   end
-
-  describe "#method_missing" do
-      let!(:person) do
-        Person.create
-      end
-
-      let!(:game) do
-        Game.create(person: person)
-      end
-
-    it 'handles keyword args' do
-      expect do
-        game.person.set_personal_data(ssn: '123', age: 25)
-      end.not_to raise_error
-    end
-
-  end
 end