mongoid 5.4.1 → 6.1.1

data/spec/mongoid/criteria/queryable/smash_spec.rb

@@ -0,0 +1,30 @@
+require "spec_helper"
+
+describe Mongoid::Criteria::Queryable::Smash do
+
+  let(:smash) do
+    described_class.new(ns: :namespace)
+  end
+
+  describe "#[]" do
+
+    before do
+      smash.store(:namespace, :default)
+      smash.store(:some_field, 42)
+    end
+
+    context "when accessing aliased field" do
+
+      it "returns value for original field" do
+        expect(smash[:ns]).to eq(:default)
+      end
+    end
+
+    context "when accessing non-aliased field" do
+
+      it "returns value for the field" do
+        expect(smash[:some_field]).to eq(42)
+      end
+    end
+  end
+end

data/spec/mongoid/criteria_spec.rb

@@ -412,7 +412,7 @@ describe Mongoid::Criteria do
       end
 
       let(:criteria) do
-        Band.where(name: "Depeche Mode").asc(:name).includes(:records)
+        Band.where(name: "Depeche Mode").asc(:name).includes(:records).read(mode: :secondary)
       end
 
       before do
@@ -433,7 +433,7 @@ describe Mongoid::Criteria do
       end
 
       it "contains equal options" do
-        expect(clone.options).to eq({ sort: { "name" => 1 }})
+        expect(clone.options).to eq({ sort: { "name" => 1 }, read: { mode: :secondary } })
       end
 
       it "clones the options" do
@@ -467,6 +467,10 @@ describe Mongoid::Criteria do
       it "sets the context to nil" do
         expect(clone.instance_variable_get(:@context)).to be_nil
       end
+
+      it 'does not convert the option keys to string from symbols' do
+        expect(clone.options[:read][:mode]).to eq(:secondary)
+      end
     end
   end
 
@@ -590,7 +594,8 @@ describe Mongoid::Criteria do
 
     let!(:match) do
       Band.create(name: "Depeche Mode").tap do |band|
-        band.records.create(name: "101")
+        r = band.records
+        r.create(name: "101")
       end
     end
 
@@ -2803,6 +2808,93 @@ describe Mongoid::Criteria do
         end
       end
     end
+
+    context 'when the field is localized' do
+
+      before do
+        I18n.locale = :en
+        d = Dictionary.create(description: 'english-text')
+        I18n.locale = :de
+        d.description = 'deutsch-text'
+        d.save
+      end
+
+      after do
+        I18n.locale = :en
+      end
+
+      context 'when entire field is included' do
+
+        let(:dictionary) do
+          Dictionary.only(:description).first
+
+        end
+
+        it 'loads all translations' do
+          expect(dictionary.description_translations.keys).to include('de', 'en')
+        end
+
+        it 'returns the field value for the current locale' do
+          I18n.locale = :en
+          expect(dictionary.description).to eq('english-text')
+          I18n.locale = :de
+          expect(dictionary.description).to eq('deutsch-text')
+        end
+      end
+
+      context 'when a specific locale is included' do
+
+        let(:dictionary) do
+          Dictionary.only(:'description.de').first
+        end
+
+        it 'loads translations only for the included locale' do
+          expect(dictionary.description_translations.keys).to include('de')
+          expect(dictionary.description_translations.keys).to_not include('en')
+        end
+
+        it 'returns the field value for the included locale' do
+          I18n.locale = :en
+          expect(dictionary.description).to be_nil
+          I18n.locale = :de
+          expect(dictionary.description).to eq('deutsch-text')
+        end
+      end
+
+      context 'when entire field is excluded' do
+
+        let(:dictionary) do
+          Dictionary.without(:description).first
+        end
+
+        it 'does not load all translations' do
+          expect(dictionary.description_translations.keys).to_not include('de', 'en')
+        end
+
+        it 'raises an ActiveModel::MissingAttributeError when attempting to access the field' do
+          expect{dictionary.description}.to raise_error ActiveModel::MissingAttributeError
+        end
+      end
+
+      context 'when a specific locale is excluded' do
+
+        let(:dictionary) do
+          Dictionary.without(:'description.de').first
+        end
+
+        it 'does not load excluded translations' do
+          expect(dictionary.description_translations.keys).to_not include('de')
+          expect(dictionary.description_translations.keys).to include('en')
+        end
+
+        it 'returns nil for excluded translations' do
+          I18n.locale = :en
+          expect(dictionary.description).to eq('english-text')
+          I18n.locale = :de
+          expect(dictionary.description).to be_nil
+        end
+      end
+    end
   end
 
   [ :or, :any_of ].each do |method|
@@ -2890,7 +2982,7 @@ describe Mongoid::Criteria do
         end
 
         it "returns the values" do
-          expect(plucked).to eq([ "Depeche Mode", "Tool", "Photek" ])
+          expect(plucked).to contain_exactly("Depeche Mode", "Tool", "Photek")
         end
 
         context "when subsequently executing the criteria without a pluck" do
@@ -2955,14 +3047,14 @@ describe Mongoid::Criteria do
         end
       end
 
-      context "when plucking mult-fields" do
+      context "when plucking multi-fields" do
 
         let(:plucked) do
           Band.where(:name.exists => true).pluck(:name, :likes)
         end
 
         it "returns the values" do
-          expect(plucked).to eq([ ["Depeche Mode", 3], ["Tool", 3], ["Photek", 1] ])
+          expect(plucked).to contain_exactly(["Depeche Mode", 3], ["Tool", 3], ["Photek", 1])
         end
       end
 
@@ -2973,7 +3065,7 @@ describe Mongoid::Criteria do
         end
 
         it "returns the duplicates" do
-          expect(plucked).to eq([ 3, 3, 1 ])
+          expect(plucked).to contain_exactly(3, 3, 1)
         end
       end
     end
@@ -3035,6 +3127,43 @@ describe Mongoid::Criteria do
         end
       end
     end
+
+    context 'when plucking a localized field' do
+
+      before do
+        I18n.locale = :en
+        d = Dictionary.create(description: 'english-text')
+        I18n.locale = :de
+        d.description = 'deutsch-text'
+        d.save
+      end
+
+      after do
+        I18n.locale = :en
+      end
+
+      context 'when plucking the entire field' do
+
+        let(:plucked) do
+          Dictionary.all.pluck(:description)
+        end
+
+        it 'returns all translations' do
+          expect(plucked.first).to eq({'en' => 'english-text', 'de' => 'deutsch-text'})
+        end
+      end
+
+      context 'when plucking a specific locale' do
+
+        let(:plucked) do
+          Dictionary.all.pluck(:'description.de')
+        end
+
+        it 'returns the specific translations' do
+          expect(plucked.first).to eq({'de' => 'deutsch-text'})
+        end
+      end
+    end
   end
 
   describe "#respond_to?" do
@@ -3314,17 +3443,6 @@ describe Mongoid::Criteria do
         end
       end
 
-      context 'when the criteria contains a BSON::Regexp::Raw' do
-
-        let(:criteria) do
-          Band.where(name: BSON::Regexp::Raw.new("^Depeche"))
-        end
-
-        it "returns the matching documents" do
-          expect(criteria).to eq([ match ])
-        end
-      end
-
       context "when the criteria is an exact fk array match" do
 
         let(:id_one) do
@@ -3502,8 +3620,21 @@ describe Mongoid::Criteria do
 
   describe "#with" do
 
-    let!(:criteria) do
-      Band.where(name: "Depeche Mode").with(collection: "artists")
+    let!(:criteria_and_collection) do
+      collection = nil
+      criteria = Band.where(name: "Depeche Mode").with(collection: "artists") do |crit|
+        collection = crit.collection
+        crit
+      end
+      [ criteria, collection ]
+    end
+
+    let(:criteria) do
+      criteria_and_collection[0]
+    end
+
+    let(:collection) do
+      criteria_and_collection[1]
     end
 
     it "retains the criteria selection" do
@@ -3511,7 +3642,7 @@ describe Mongoid::Criteria do
     end
 
     it "sets the persistence options" do
-      expect(criteria.persistence_options).to eq(collection: "artists")
+      expect(collection.name).to eq("artists")
     end
   end
 

data/spec/mongoid/document_spec.rb

@@ -122,75 +122,6 @@ describe Mongoid::Document do
     end
   end
 
-  describe "#cache_key" do
-
-    let(:document) do
-      Dokument.new
-    end
-
-    context "when the document is new" do
-
-      it "has a new key name" do
-        expect(document.cache_key).to eq("dokuments/new")
-      end
-    end
-
-    context "when persisted" do
-
-      before do
-        document.save
-      end
-
-      context "with updated_at" do
-
-        let!(:updated_at) do
-          document.updated_at.utc.to_s(:nsec)
-        end
-
-        it "has the id and updated_at key name" do
-          expect(document.cache_key).to eq("dokuments/#{document.id}-#{updated_at}")
-        end
-      end
-
-      context "without updated_at, with Timestamps" do
-
-        before do
-          document.updated_at = nil
-        end
-
-        it "has the id key name" do
-          expect(document.cache_key).to eq("dokuments/#{document.id}")
-        end
-      end
-    end
-
-    context "when model dont have Timestamps" do
-
-      let(:artist) do
-        Artist.create!
-      end
-
-      it "should have the id key name" do
-        expect(artist.cache_key).to eq("artists/#{artist.id}")
-      end
-    end
-
-    context "when model has Short Timestamps" do
-
-      let(:agent) do
-        ShortAgent.create!
-      end
-
-      let!(:updated_at) do
-        agent.updated_at.utc.to_s(:nsec)
-      end
-
-      it "has the id and updated_at key name" do
-        expect(agent.cache_key).to eq("short_agents/#{agent.id}-#{updated_at}")
-      end
-    end
-  end
-
   describe "#identity" do
 
     let(:person) do
@@ -519,14 +450,26 @@ describe Mongoid::Document do
       expect(person.as_document).to have_key("name")
     end
 
+    it "includes embeds one attributes as a symbol" do
+      expect(person.as_document).to have_key(:name)
+    end
+
     it "includes embeds many attributes" do
       expect(person.as_document).to have_key("addresses")
     end
 
+    it "includes embeds many attributes as a symbol" do
+      expect(person.as_document).to have_key(:addresses)
+    end
+
     it "includes second level embeds many attributes" do
       expect(person.as_document["addresses"].first).to have_key("locations")
     end
 
+    it "includes second level embeds many attributes as a symbol" do
+      expect(person.as_document["addresses"].first).to have_key(:locations)
+    end
+
     context "with relation define store_as option in embeded_many" do
 
       let!(:phone) do
@@ -537,6 +480,10 @@ describe Mongoid::Document do
         expect(person.as_document).to have_key("mobile_phones")
       end
 
+      it 'includes the store_as key association as a symbol' do
+        expect(person.as_document).to have_key(:mobile_phones)
+      end
+
       it 'should not include the key of association' do
         expect(person.as_document).to_not have_key("phones")
       end
@@ -675,7 +622,7 @@ describe Mongoid::Document do
 
         it "freezes attributes" do
           expect(person.freeze).to eq(person)
-          expect { person.title = "something" }.to raise_error
+          expect { person.title = "something" }.to raise_error(RuntimeError)
         end
       end
 
@@ -689,7 +636,7 @@ describe Mongoid::Document do
           person.freeze
           expect {
             person.title = "something"
-          }.to raise_error
+          }.to raise_error(RuntimeError)
         end
       end
     end
@@ -1215,22 +1162,20 @@ describe Mongoid::Document do
 
   context "when marshalling the document" do
 
-    let(:person) do
-      Person.new.tap do |person|
-        person.addresses.extension
-      end
+    let(:agency) do
+      Agency.new
     end
 
-    let!(:account) do
-      person.create_account(name: "savings")
+    let!(:agent) do
+      agency.agents.build(title: "VIP")
     end
 
     describe Marshal, ".dump" do
 
       it "successfully dumps the document" do
         expect {
-          Marshal.dump(person)
-          Marshal.dump(account)
+          Marshal.dump(agency)
+          Marshal.dump(agent)
         }.not_to raise_error
       end
     end
@@ -1238,7 +1183,7 @@ describe Mongoid::Document do
     describe Marshal, ".load" do
 
       it "successfully loads the document" do
-        expect { Marshal.load(Marshal.dump(person)) }.not_to raise_error
+        expect(Marshal.load(Marshal.dump(agency))).to eq(agency)
       end
     end
   end
@@ -1253,22 +1198,26 @@ describe Mongoid::Document do
 
       describe "#fetch" do
 
-        let!(:person) do
-          Person.new
+        let(:agency) do
+          Agency.new
+        end
+
+        let(:agent) do
+          agency.agents.build(title: "VIP", address: address)
         end
 
-        let!(:account) do
-          person.create_account(name: "savings")
+        let(:address) do
+          Address.new(city: 'Berlin')
         end
 
         it "stores the parent object" do
-          expect(cache.fetch("key") { person }).to eq(person)
-          expect(cache.fetch("key")).to eq(person)
+          expect(cache.fetch("key") { agency }).to eq(agency)
+          expect(cache.fetch("key")).to eq(agency)
         end
 
         it "stores the embedded object" do
-          expect(cache.fetch("key") { account }).to eq(account)
-          expect(cache.fetch("key")).to eq(account)
+          expect(cache.fetch("key") { agent }).to eq(agent)
+          expect(cache.fetch("key").address).to eq(agent.address)
         end
       end
     end