mongoid 6.4.2 → 7.0.3

data/spec/mongoid/{relations/embedded/many_spec.rb → association/embedded/embeds_many/proxy_spec.rb}

@@ -1,6 +1,6 @@
 require "spec_helper"
 
-describe Mongoid::Relations::Embedded::Many do
+describe Mongoid::Association::Embedded::EmbedsMany::Proxy do
 
   [ :<<, :push ].each do |method|
 
@@ -40,8 +40,8 @@ describe Mongoid::Relations::Embedded::Many do
           expect(address._parent).to eq(person)
         end
 
-        it "sets the metadata on the child" do
-          expect(address.__metadata).to_not be_nil
+        it "sets the association metadata on the child" do
+          expect(address._association).to_not be_nil
         end
 
         it "sets the index on the child" do
@@ -52,6 +52,21 @@ describe Mongoid::Relations::Embedded::Many do
           expect(added).to eq(person.addresses)
         end
 
+        context 'when the child is already related to the parent' do
+
+          let(:message) do
+            Message.new(person: person)
+          end
+
+          before do
+            person.messages << message
+          end
+
+          it "appends only once to the target" do
+            expect(person.messages).to eq([ message ])
+          end
+        end
+
         context "with a limiting default scope" do
 
           context "when the document matches the scope" do
@@ -65,7 +80,7 @@ describe Mongoid::Relations::Embedded::Many do
             end
 
             it "appends to the target" do
-              expect(person.appointments.target).to eq([ active ])
+              expect(person.appointments._target).to eq([ active ])
             end
 
             it "appends to the _unscoped" do
@@ -84,7 +99,7 @@ describe Mongoid::Relations::Embedded::Many do
             end
 
             it "doesn't append to the target" do
-              expect(person.appointments.target).to_not eq([ inactive ])
+              expect(person.appointments._target).to_not eq([ inactive ])
             end
 
             it "appends to the _unscoped" do
@@ -180,8 +195,8 @@ describe Mongoid::Relations::Embedded::Many do
             expect(child_role._parent).to eq(parent_role)
           end
 
-          it "sets the metadata on the child" do
-            expect(child_role.__metadata).to_not be_nil
+          it "sets the association metadata on the child" do
+            expect(child_role._association).to_not be_nil
           end
 
           it "sets the index on the child" do
@@ -221,21 +236,21 @@ describe Mongoid::Relations::Embedded::Many do
         context "assign parent first" do
           before do
             message.person = person
-            message.receviers.send(method, person)
+            message.receivers.send(method, person)
           end
 
           it "appends to the relation array" do
-            expect(message.receviers).to include(person)
+            expect(message.receivers).to include(person)
           end
         end
 
         context "not assign parent" do
           before do
-            message.receviers.send(method, person)
+            message.receivers.send(method, person)
           end
 
           it "appends to the relation array" do
-            expect(message.receviers).to include(person)
+            expect(message.receivers).to include(person)
           end
         end
       end
@@ -282,8 +297,8 @@ describe Mongoid::Relations::Embedded::Many do
         expect(address._parent).to eq(person)
       end
 
-      it "sets the metadata on the child" do
-        expect(address.__metadata).to_not be_nil
+      it "sets the association metadata on the child" do
+        expect(address._association).to_not be_nil
       end
 
       it "sets the index on the child" do
@@ -433,8 +448,8 @@ describe Mongoid::Relations::Embedded::Many do
 
       let(:addresses) do
         [
-          Address.new(street: "1st St"),
-          Address.new(street: "2nd St")
+            Address.new(street: "1st St"),
+            Address.new(street: "2nd St")
         ]
       end
 
@@ -651,8 +666,8 @@ describe Mongoid::Relations::Embedded::Many do
           expect(child_role._parent).to eq(parent_role)
         end
 
-        it "sets the metadata on the child" do
-          expect(child_role.__metadata).to_not be_nil
+        it "sets the association metadata on the child" do
+          expect(child_role._association).to_not be_nil
         end
 
         it "sets the index on the child" do
@@ -1028,8 +1043,8 @@ describe Mongoid::Relations::Embedded::Many do
           expect(address._parent).to eq(person)
         end
 
-        it "sets the metadata on the child" do
-          expect(address.__metadata).to_not be_nil
+        it "sets the association metadata on the child" do
+          expect(address._association).to_not be_nil
         end
 
         it "sets the index on the child" do
@@ -1071,8 +1086,8 @@ describe Mongoid::Relations::Embedded::Many do
           expect(child_role._parent).to eq(parent_role)
         end
 
-        it "sets the metadata on the child" do
-          expect(child_role.__metadata).to_not be_nil
+        it "sets the association metadata on the child" do
+          expect(child_role._association).to_not be_nil
         end
 
         it "sets the index on the child" do
@@ -1092,9 +1107,9 @@ describe Mongoid::Relations::Embedded::Many do
 
         let(:address) do
           person.addresses.send(
-            method,
-            street: "Bond",
-            locations_attributes: { "1" => { "name" => "Home" } }
+              method,
+              street: "Bond",
+              locations_attributes: { "1" => { "name" => "Home" } }
           )
         end
 
@@ -1116,35 +1131,6 @@ describe Mongoid::Relations::Embedded::Many do
     end
   end
 
-  describe ".builder" do
-
-    let(:base) do
-      Person.new
-    end
-
-    let(:target) do
-      [ address ]
-    end
-
-    let(:metadata) do
-      Person.relations["addresses"]
-    end
-
-    let(:relation) do
-      described_class.new(base, target, metadata)
-    end
-
-    let(:document) do
-      Address.new
-    end
-
-    it "returns the many builder" do
-      expect(
-        described_class.builder(base, metadata, document)
-      ).to be_a(Mongoid::Relations::Builders::Embedded::Many)
-    end
-  end
-
   describe "#clear" do
 
     context "when the parent has been persisted" do
@@ -1260,8 +1246,8 @@ describe Mongoid::Relations::Embedded::Many do
         expect(address._parent).to eq(person)
       end
 
-      it "sets the metadata on the child" do
-        expect(address.__metadata).to_not be_nil
+      it "sets the association metadata on the child" do
+        expect(address._association).to_not be_nil
       end
 
       it "sets the index on the child" do
@@ -1367,8 +1353,8 @@ describe Mongoid::Relations::Embedded::Many do
           expect(child_role._parent).to eq(parent_role)
         end
 
-        it "sets the metadata on the child" do
-          expect(child_role.__metadata).to_not be_nil
+        it "sets the association metadata on the child" do
+          expect(child_role._association).to_not be_nil
         end
 
         it "sets the index on the child" do
@@ -1474,8 +1460,8 @@ describe Mongoid::Relations::Embedded::Many do
         expect(address._parent).to eq(person)
       end
 
-      it "sets the metadata on the child" do
-        expect(address.__metadata).to_not be_nil
+      it "sets the association metadata on the child" do
+        expect(address._association).to_not be_nil
       end
 
       it "sets the index on the child" do
@@ -1573,8 +1559,8 @@ describe Mongoid::Relations::Embedded::Many do
         expect(address._parent).to eq(person)
       end
 
-      it "sets the metadata on the child" do
-        expect(address.__metadata).to_not be_nil
+      it "sets the association metadata on the child" do
+        expect(address._association).to_not be_nil
       end
 
       it "sets the index on the child" do
@@ -1596,10 +1582,10 @@ describe Mongoid::Relations::Embedded::Many do
 
       context 'when the presence of the embedded relation is validated' do
 
-        before do
-          class Book
-            validates :pages, presence: true
-          end
+        around do |example|
+          Book.validates :pages, presence: true
+          example.run
+          Book.reset_callbacks(:validate)
         end
 
         let(:book) do
@@ -1805,8 +1791,8 @@ describe Mongoid::Relations::Embedded::Many do
 
           let!(:deleted) do
             person.addresses.send(
-              method,
-              { street: "Bond" }
+                method,
+                { street: "Bond" }
             )
           end
 
@@ -1853,8 +1839,8 @@ describe Mongoid::Relations::Embedded::Many do
 
           let!(:deleted) do
             person.addresses.send(
-              method,
-              { street: "Bond" }
+                method,
+                { street: "Bond" }
             )
           end
 
@@ -1939,8 +1925,8 @@ describe Mongoid::Relations::Embedded::Many do
 
           let!(:deleted) do
             person.addresses.send(
-              method,
-              conditions: { street: "Bond" }
+                method,
+                conditions: { street: "Bond" }
             )
           end
 
@@ -1986,13 +1972,6 @@ describe Mongoid::Relations::Embedded::Many do
     end
   end
 
-  describe ".foreign_key_suffix" do
-
-    it "returns nil" do
-      expect(described_class.foreign_key_suffix).to be_nil
-    end
-  end
-
   describe "#exists?" do
 
     let!(:person) do
@@ -2177,27 +2156,28 @@ describe Mongoid::Relations::Embedded::Many do
       end
     end
 
-    context "when the child belongs to another document" do
-
-      let(:product) do
-        Product.create
-      end
-
-      let(:purchase) do
-        Purchase.create
-      end
-
-      let(:line_item) do
-        purchase.line_items.find_or_create_by(
-          product_id: product.id,
-          product_type: product.class.name
-        )
-      end
-
-      it "properly creates the document" do
-        expect(line_item.product).to eq(product)
-      end
-    end
+    # todo: why should this pass?
+    # context "when the child belongs to another document" do
+    #
+    #   let(:product) do
+    #     Product.create
+    #   end
+    #
+    #   let(:purchase) do
+    #     Purchase.create
+    #   end
+    #
+    #   let(:line_item) do
+    #     purchase.line_items.find_or_create_by(
+    #         product_id: product.id,
+    #         product_type: product.class.name
+    #     )
+    #   end
+    #
+    #   it "properly creates the document" do
+    #     expect(line_item.product).to eq(product)
+    #   end
+    # end
   end
 
   describe "#find_or_create_by!" do
@@ -2251,27 +2231,28 @@ describe Mongoid::Relations::Embedded::Many do
       end
     end
 
-    context "when the child belongs to another document" do
-
-      let(:product) do
-        Product.create
-      end
-
-      let(:purchase) do
-        Purchase.create
-      end
-
-      let(:line_item) do
-        purchase.line_items.find_or_create_by(
-            product_id: product.id,
-            product_type: product.class.name
-        )
-      end
-
-      it "properly creates the document" do
-        expect(line_item.product).to eq(product)
-      end
-    end
+    # todo: why should this pass?
+    # context "when the child belongs to another document" do
+    #
+    #   let(:product) do
+    #     Product.create
+    #   end
+    #
+    #   let(:purchase) do
+    #     Purchase.create
+    #   end
+    #
+    #   let(:line_item) do
+    #     purchase.line_items.find_or_create_by(
+    #         product_id: product.id,
+    #         product_type: product.class.name
+    #     )
+    #   end
+    #
+    #   it "properly creates the document" do
+    #     expect(line_item.product).to eq(product)
+    #   end
+    # end
   end
 
   describe "#find_or_initialize_by" do
@@ -2317,13 +2298,6 @@ describe Mongoid::Relations::Embedded::Many do
     end
   end
 
-  describe ".macro" do
-
-    it "returns embeds_many" do
-      expect(described_class.macro).to eq(:embeds_many)
-    end
-  end
-
   describe "#max" do
 
     let(:person) do
@@ -2388,17 +2362,17 @@ describe Mongoid::Relations::Embedded::Many do
 
     let!(:address_one) do
       person.addresses.create(
-        street: "Market",
-        state: "CA",
-        services: [ "1", "2" ]
+          street: "Market",
+          state: "CA",
+          services: [ "1", "2" ]
       )
     end
 
     let!(:address_two) do
       person.addresses.create(
-        street: "Madison",
-        state: "NY",
-        services: [ "1", "2" ]
+          street: "Madison",
+          state: "NY",
+          services: [ "1", "2" ]
       )
     end
 
@@ -2527,19 +2501,6 @@ describe Mongoid::Relations::Embedded::Many do
     end
   end
 
-  describe ".nested_builder" do
-
-    let(:metadata) do
-      Person.relations["addresses"]
-    end
-
-    it "returns the many nested builder class" do
-      expect(
-        described_class.nested_builder(metadata, {}, {})
-      ).to be_a(Mongoid::Relations::Builders::NestedAttributes::Many)
-    end
-  end
-
   describe "#pop" do
 
     let(:person) do
@@ -2817,7 +2778,7 @@ describe Mongoid::Relations::Embedded::Many do
       end
     end
 
-    Mongoid::Relations::Embedded::Many.public_instance_methods.each do |method|
+    Mongoid::Association::Embedded::EmbedsMany::Proxy.public_instance_methods.each do |method|
 
       context "when checking #{method}" do
 
@@ -2911,8 +2872,8 @@ describe Mongoid::Relations::Embedded::Many do
 
         before do
           person.addresses.
-            where(street: "Hobrecht").
-            update_all(number: 26, post_code: "12437")
+              where(street: "Hobrecht").
+              update_all(number: 26, post_code: "12437")
         end
 
         it "resets the matching dirty flags" do
@@ -2934,25 +2895,6 @@ describe Mongoid::Relations::Embedded::Many do
     end
   end
 
-  describe ".valid_options" do
-
-    it "returns the valid options" do
-      expect(described_class.valid_options).to eq(
-        [
-          :as, :cascade_callbacks, :cyclic, :order,
-          :store_as, :before_add, :after_add, :before_remove, :after_remove
-        ]
-      )
-    end
-  end
-
-  describe ".validation_default" do
-
-    it "returns true" do
-      expect(described_class.validation_default).to be true
-    end
-  end
-
   context "when deeply embedding documents" do
 
     context "when updating the bottom level" do
@@ -3823,9 +3765,9 @@ describe Mongoid::Relations::Embedded::Many do
 
     let!(:person) do
       Person.create(
-        addresses: [
-          { locations: [{ name: "home" }]}
-        ]
+          addresses: [
+              { locations: [{ name: "home" }]}
+          ]
       )
     end
 
@@ -3837,9 +3779,9 @@ describe Mongoid::Relations::Embedded::Many do
 
       before do
         from_db.update_attributes(
-          addresses: [
-            { locations: [{ name: "work" }]}
-          ]
+            addresses: [
+                { locations: [{ name: "work" }]}
+            ]
         )
       end
 
@@ -3900,8 +3842,8 @@ describe Mongoid::Relations::Embedded::Many do
 
     before do
       person.update_attributes(
-        appointments: [ appointment_one.as_document, appointment_two.as_document ],
-        symptoms: [ symptom_one.as_document, symptom_two.as_document ]
+          appointments: [ appointment_one.as_document, appointment_two.as_document ],
+          symptoms: [ symptom_one.as_document, symptom_two.as_document ]
       )
     end
 
@@ -4249,8 +4191,8 @@ describe Mongoid::Relations::Embedded::Many do
 
     before do
       band.collection.
-        find(_id: band.id).
-        update_one("$set" => { records: [{ name: "Moderat" }]})
+          find(_id: band.id).
+          update_one("$set" => { records: [{ name: "Moderat" }]})
     end
 
     context "when loading the documents" do
@@ -4347,7 +4289,7 @@ describe Mongoid::Relations::Embedded::Many do
           zone.soa = soa_1
         end
 
-        it "properly sets the metadata" do
+        it "properly sets the association metadata" do
           expect(zone.soa = soa_2).to eq(soa_2)
         end
       end
@@ -4362,7 +4304,7 @@ describe Mongoid::Relations::Embedded::Many do
           zone.soa = soa_1
         end
 
-        it "properly sets the metadata" do
+        it "properly sets the association metadata" do
           expect(zone.soa.delete).to be true
         end
       end
@@ -4388,7 +4330,7 @@ describe Mongoid::Relations::Embedded::Many do
           zone.soa = soa_1
         end
 
-        it "properly sets the metadata" do
+        it "properly sets the association" do
           expect(zone.soa = soa_2).to eq(soa_2)
         end
       end
@@ -4403,7 +4345,7 @@ describe Mongoid::Relations::Embedded::Many do
           zone.soa = soa_1
         end
 
-        it "properly sets the metadata" do
+        it "properly sets the association" do
           expect(zone.soa.delete).to be true
         end
       end