mongoid 7.0.2 → 8.1.1

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

@@ -1,4 +1,7 @@
+# frozen_string_literal: true
+
 require "spec_helper"
+require_relative '../belongs_to_models.rb'
 
 describe Mongoid::Association::Referenced::BelongsTo::Proxy do
 
@@ -7,7 +10,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
   end
 
   let(:person) do
-    Person.create
+    Person.create!
   end
 
   describe "#=" do
@@ -59,15 +62,15 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
     context "when referencing a document from an embedded document" do
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       let(:address) do
-        person.addresses.create(street: "Wienerstr")
+        person.addresses.create!(street: "Wienerstr")
       end
 
       let(:account) do
-        Account.create(name: "1", number: 1000000)
+        Account.create!(name: "1", number: 1000000)
       end
 
       before do
@@ -83,7 +86,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
       end
 
       it "allows saving of the embedded document" do
-        expect(address.save).to be true
+        expect(address.save!).to be true
       end
     end
 
@@ -124,6 +127,23 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           it "does not save the target" do
             expect(person).to_not be_persisted
           end
+
+          it "drops private visibility from private methods" do
+            # This is unfortunate but it appears that Ruby does not
+            # have a way to distinguish how method_missing was invoked
+            # (i.e. via an explicit send or method call).
+            # See https://jira.mongodb.org/browse/MONGOID-5009
+            game.person.secret_name.should == 'secret'
+          end
+
+          it "allows private methods to be invoked" do
+            game.person.send(:secret_name).should == 'secret'
+          end
+
+          it "properly exposes delegated methods visibility" do
+            expect(defined?(game.person.id)).to eq("method")
+            expect(defined?(game.person.secret_name)).to be_nil
+          end
         end
 
         context "when the child is not a new record" do
@@ -133,7 +153,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           end
 
           let(:game) do
-            Game.create
+            Game.create!
           end
 
           before do
@@ -167,16 +187,16 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
         context "when the parent is a subclass" do
 
           let(:canvas) do
-            Canvas::Test.create
+            Canvas::Test.create!
           end
 
           let(:comment) do
-            Comment.create(title: "test")
+            Comment.create!(title: "test")
           end
 
           before do
             comment.commentable = canvas
-            comment.save
+            comment.save!
           end
 
           it "sets the correct value in the type field" do
@@ -230,7 +250,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           end
 
           let(:rating) do
-            Rating.create
+            Rating.create!
           end
 
           before do
@@ -266,6 +286,11 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
 
         context 'when the child has persistence options set' do
 
+          before do
+            Person.collection.client['other-posts'].delete_many
+            Person.collection.client['other-people'].delete_many
+          end
+
           let(:person) do
             Person.new
           end
@@ -275,11 +300,11 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           end
 
           before do
-            post.with(collection: 'other-posts') do |p|
-              person.with(collection: 'other-people') do |per|
-                p.person = person
-                per.save
-                p.save
+            post.with(collection: 'other-posts') do |post|
+              person.with(collection: 'other-people') do |person|
+                post.person = person
+                person.save!
+                post.save!
               end
             end
           end
@@ -336,7 +361,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           end
 
           let(:post) do
-            Post.create
+            Post.create!
           end
 
           before do
@@ -428,7 +453,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
             end
 
             let(:rating) do
-              Rating.create
+              Rating.create!
             end
 
             before do
@@ -481,7 +506,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           Account.belongs_to :person, dependent: :destroy
           Person.has_one :account
           person.account = account
-          person.save
+          person.save!
         end
 
         after :all do
@@ -505,7 +530,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           context "when child is destroyed" do
 
             before do
-              account.delete
+              account.destroy
             end
 
             it "deletes child" do
@@ -532,7 +557,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           Drug.belongs_to :person, dependent: :destroy
           Person.has_many :drugs
           person.drugs = [drug]
-          person.save
+          person.save!
         end
 
         after :all do
@@ -545,7 +570,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           context "when child is destroyed" do
 
             before do
-              drug.delete
+              drug.destroy
             end
 
             it "deletes child" do
@@ -563,15 +588,15 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
     context "when dependent is delete" do
 
       let(:account) do
-        Account.create
+        Account.create!(name: 'Foobar')
       end
 
       let(:drug) do
-        Drug.create
+        Drug.create!
       end
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       context "when relation is has_one" do
@@ -587,7 +612,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           Account.belongs_to :person, dependent: :delete_all
           Person.has_one :account
           person.account = account
-          person.save
+          person.save!
         end
 
         after :all do
@@ -597,10 +622,10 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
 
         context "when parent is persisted" do
 
-          context "when child is deleted" do
+          context "when child is destroyed" do
 
             before do
-              account.delete
+              account.destroy
             end
 
             it "deletes child" do
@@ -627,7 +652,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           Drug.belongs_to :person, dependent: :delete_all
           Person.has_many :drugs
           person.drugs = [drug]
-          person.save
+          person.save!
         end
 
         after :all do
@@ -640,7 +665,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           context "when child is destroyed" do
 
             before do
-              drug.delete
+              drug.destroy
             end
 
             it "deletes child" do
@@ -658,15 +683,15 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
     context "when dependent is nullify" do
 
       let(:account) do
-        Account.create
+        Account.create!(name: 'Foobar')
       end
 
       let(:drug) do
-        Drug.create
+        Drug.create!
       end
 
       let(:person) do
-        Person.create
+        Person.create!
       end
 
       context "when relation is has_one" do
@@ -682,15 +707,15 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           Account.belongs_to :person, dependent: :nullify
           Person.has_one :account
           person.account = account
-          person.save
+          person.save!
         end
 
         context "when parent is persisted" do
 
-          context "when child is deleted" do
+          context "when child is destroyed" do
 
             before do
-              account.delete
+              account.destroy
             end
 
             it "deletes child" do
@@ -721,7 +746,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           Drug.belongs_to :person, dependent: :nullify
           Person.has_many :drugs
           person.drugs = [drug]
-          person.save
+          person.save!
         end
 
         context "when parent exists" do
@@ -729,7 +754,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           context "when child is destroyed" do
 
             before do
-              drug.delete
+              drug.destroy
             end
 
             it "deletes child" do
@@ -807,11 +832,11 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
         context "when the parent is not a new record" do
 
           let(:person) do
-            Person.create
+            Person.create!
           end
 
           let(:game) do
-            Game.create
+            Game.create!
           end
 
           before do
@@ -876,7 +901,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
             end
 
             let(:rating) do
-              Rating.create
+              Rating.create!
             end
 
             before do
@@ -939,7 +964,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           end
 
           let(:post) do
-            Post.create
+            Post.create!
           end
 
           before do
@@ -998,7 +1023,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
           end
 
           let(:rating) do
-            Rating.create
+            Rating.create!
           end
 
           before do
@@ -1140,11 +1165,11 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
   context "when replacing the relation with another" do
 
     let!(:person) do
-      Person.create
+      Person.create!
     end
 
     let!(:post) do
-      Post.create(title: "test")
+      Post.create!(title: "test")
     end
 
     let!(:game) do
@@ -1153,7 +1178,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
 
     before do
       post.person = game.person
-      post.save
+      post.save!
     end
 
     it "clones the relation" do
@@ -1200,17 +1225,17 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
   context "when the document belongs to a has one and has many" do
 
     let(:movie) do
-      Movie.create(name: "Infernal Affairs")
+      Movie.create!(name: "Infernal Affairs")
     end
 
     let(:account) do
-      Account.create(name: "Leung")
+      Account.create!(name: "Leung")
     end
 
     context "when creating the document" do
 
       let(:comment) do
-        Comment.create(movie: movie, account: account)
+        Comment.create!(title: 'My Title', movie: movie, account: account)
       end
 
       it "sets the correct has one" do
@@ -1226,20 +1251,20 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
   context "when reloading the relation" do
 
     let!(:person_one) do
-      Person.create
+      Person.create!
     end
 
     let!(:person_two) do
-      Person.create(title: "Sir")
+      Person.create!(title: "Sir")
     end
 
     let!(:game) do
-      Game.create(name: "Starcraft 2")
+      Game.create!(name: "Starcraft 2")
     end
 
     before do
       game.person = person_one
-      game.save
+      game.save!
     end
 
     context "when the relation references the same document" do
@@ -1266,7 +1291,7 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
 
       before do
         game.person_id = person_two.id
-        game.save
+        game.save!
       end
 
       let(:reloaded) do
@@ -1286,13 +1311,13 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
   context "when creating with a reference to an integer id parent" do
 
     let!(:jar) do
-      Jar.create do |doc|
+      Jar.create! do |doc|
         doc._id = 1
       end
     end
 
     let(:cookie) do
-      Cookie.create(jar_id: "1")
+      Cookie.create!(jar_id: "1")
     end
 
     it "allows strings to be passed as the id" do
@@ -1309,15 +1334,15 @@ describe Mongoid::Association::Referenced::BelongsTo::Proxy do
     context "when the relation exists" do
 
       let!(:person_one) do
-        Person.create
+        Person.create!
       end
 
       let!(:person_two) do
-        Person.create
+        Person.create!
       end
 
       let!(:game) do
-        Game.create(person: person_one)
+        Game.create!(person: person_one)
       end
 
       before do
@@ -1329,4 +1354,35 @@ 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
+
+  # This is a very specific case, see MONGOID-5089 for more details.
+  context "when required is false, child is an orphan, and parent has explicit _id" do
+    let(:comment) { BTMComment.create! }
+    let(:article) do
+      BTMArticle.new(
+        comment_ids: [comment.id],
+        id: 1
+      )
+    end
+
+    it "uses the correct explicit id" do
+      expect(article.comments.first.article_id).to eq(1)
+    end
+  end
 end

data/spec/mongoid/association/referenced/belongs_to_models.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+class BTMArticle
+  include Mongoid::Document
+  has_many :comments, class_name: "BTMComment"
+end
+
+class BTMComment
+  include Mongoid::Document
+  belongs_to :article, class_name: "BTMArticle", required: false
+end

data/spec/mongoid/association/referenced/belongs_to_query_spec.rb

@@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+require_relative './has_many_models'
+
+describe Mongoid::Association::Referenced::BelongsTo do
+  context 'when projecting with #only' do
+    before do
+      school = HmmSchool.create!(district: 'foo', team: 'Bulldogs')
+      HmmStudent.create!(school: school, name: 'Dave', grade: 10)
+    end
+
+    let(:student) do
+      HmmStudent.where(name: 'Dave').only(:school_id, 'school._id', 'school.district').first
+    end
+
+    let(:school) { student.school }
+
+    it 'populates specified fields only' do
+      pending 'https://jira.mongodb.org/browse/MONGOID-4704'
+
+      expect(school.district).to eq('foo')
+      # has a default value specified in the model
+      expect do
+        school.team
+      end.to raise_error(ActiveModel::MissingAttributeError)
+      expect(student.attributes.keys).to eq(['_id', 'name'])
+    end
+
+    # Delete this test when https://jira.mongodb.org/browse/MONGOID-4704 is
+    # implemented and above test is unpended
+    it 'fetches all fields' do
+      expect(school.district).to eq('foo')
+      expect(school.team).to eq('Bulldogs')
+    end
+  end
+
+  context 'when projecting with #only while having similar inverse_of candidates' do
+    before do
+      alice = HmmOwner.create!(name: 'Alice')
+      bob = HmmOwner.create!(name: 'Bob')
+
+      HmmPet.create!(name: 'Rex', current_owner: bob, previous_owner: alice)
+    end
+
+    let(:pet) { HmmPet.where(name: 'Rex').only(:name, :previous_owner_id, 'previous_owner.name').first }
+
+    it 'populates specified fields' do
+      expect(pet.name).to eq('Rex')
+      expect(pet.previous_owner.name).to eq('Alice')
+    end
+
+    it 'does not try to load the inverse for an association that explicitly prevents it' do
+      expect { pet.previous_owner.name }.not_to raise_error
+    end
+  end
+end