mongoid 5.1.6 → 5.2.0.rc0

data/lib/mongoid/version.rb

@@ -1,4 +1,4 @@
 # encoding: utf-8
 module Mongoid
-  VERSION = "5.1.6"
+  VERSION = "5.2.0.rc0"
 end

data/lib/rails/generators/mongoid/config/templates/mongoid.yml

@@ -130,6 +130,10 @@ development:
     # environment. The Mongoid logger will be set to the Rails logger
     # otherwise.(default: :info)
     # log_level: :info
+
+    # Application name that is printed to the mongodb logs upon establishing a
+    # connection in server versions >= 3.4. Note that the name cannot exceed 128 bytes.
+    # app_name: MyApplicationName
 test:
   clients:
     default:

data/spec/app/models/band.rb

@@ -13,6 +13,7 @@ class Band
   field :upserted, type: Mongoid::Boolean, default: false
   field :created, type: DateTime
   field :sales, type: BigDecimal
+  field :decimal, type: BSON::Decimal128
   field :y, as: :years, type: Integer
   field :founded, type: Date
   field :deleted, type: Boolean

data/spec/config/mongoid.yml

@@ -13,6 +13,10 @@ test:
           tag_sets:
             - use: web
         max_pool_size: 1
+    reports:
+      database: reports
+      hosts:
+        - <%=ENV["MONGOID_SPEC_HOST"]%>:<%=ENV["MONGOID_SPEC_PORT"]%>
   options:
     include_root_in_json: false
     include_type_for_serialization: false
@@ -22,3 +26,4 @@ test:
     use_activesupport_time_zone: true
     use_utc: false
     log_level: :warn
+    app_name: 'testing'

data/spec/mongoid/clients/factory_spec.rb

@@ -36,6 +36,10 @@ describe Mongoid::Clients::Factory do
           it "sets the cluster's seeds" do
             expect(cluster.addresses.first.to_s).to eq("127.0.0.1:27017")
           end
+
+          it "sets the platform to Mongoid's platform constant" do
+            expect(client.options[:platform]).to eq(Mongoid::PLATFORM_DETAILS)
+          end
         end
 
         context "when the configuration has no ports" do
@@ -280,5 +284,9 @@ describe Mongoid::Clients::Factory do
     it "sets the write concern" do
       expect(client.write_concern).to be_a(Mongo::WriteConcern::Acknowledged)
     end
+
+    it "sets the platform to Mongoid's platform constant" do
+      expect(client.options[:platform]).to eq(Mongoid::PLATFORM_DETAILS)
+    end
   end
 end

data/spec/mongoid/clients_spec.rb

@@ -408,6 +408,14 @@ describe Mongoid::Clients do
       it "returns the default client" do
         expect(mongo_client.options[:database].to_s).to eq(database_id)
       end
+
+      it "sets the platform to Mongoid's platform constant" do
+        expect(mongo_client.options[:platform]).to eq(Mongoid::PLATFORM_DETAILS)
+      end
+
+      it "sets the app_name to the config value" do
+        expect(mongo_client.options[:app_name]).to eq('testing')
+      end
     end
 
     context "when no client exists with the key" do
@@ -416,6 +424,10 @@ describe Mongoid::Clients do
         Band.store_in(client: :nonexistent)
       end
 
+      after do
+        Band.reset_storage_options!
+      end
+
       let(:band) do
         Band.new
       end
@@ -426,6 +438,63 @@ describe Mongoid::Clients do
         }.to raise_error(Mongoid::Errors::NoClientConfig)
       end
     end
+
+    context "when getting a client by name", if: testing_locally? do
+
+      let(:file) do
+        File.join(File.dirname(__FILE__), "..", "config", "mongoid.yml")
+      end
+
+      before do
+        described_class.clear
+        Mongoid.load!(file, :test)
+        Band.store_in(client: :reports)
+      end
+
+      after do
+        mongo_client.close
+        Mongoid::Config.reset
+        Band.reset_storage_options!
+      end
+
+      let!(:band) do
+        Band.store_in(client: :reports)
+      end
+
+      let!(:mongo_client) do
+        Band.new.mongo_client
+      end
+
+      it "uses the reports client" do
+        expect(mongo_client.options[:database].to_s).to eq('reports')
+      end
+
+      it "sets the platform to Mongoid's platform constant" do
+        expect(mongo_client.options[:platform]).to eq(Mongoid::PLATFORM_DETAILS)
+      end
+
+      it "sets the app_name to the config value" do
+        expect(mongo_client.options[:app_name]).to eq('testing')
+      end
+    end
+
+    context 'when the app_name is not set in the config' do
+
+      before do
+        Mongoid::Config.reset
+        Mongoid.configure do |config|
+          config.load_configuration(CONFIG)
+        end
+      end
+
+      let(:mongo_client) do
+        Band.new.mongo_client
+      end
+
+      it 'does not set the Mongoid.app_name option' do
+        expect(mongo_client.options.has_key?(:app_name)).to be(false)
+      end
+    end
   end
 
   describe ".mongo_client", if: non_legacy_server? do
@@ -457,13 +526,21 @@ describe Mongoid::Clients do
         Mongoid.clients[:default][:database] = database_id
       end
 
-      let!(:mongo_client) do
+      let(:mongo_client) do
         Band.mongo_client
       end
 
       it "returns the default client" do
         expect(mongo_client.options[:database].to_s).to eq(database_id)
       end
+
+      it "sets the platform to Mongoid's platform constant" do
+        expect(mongo_client.options[:platform]).to eq(Mongoid::PLATFORM_DETAILS)
+      end
+
+      it "sets the app_name to the config value" do
+        expect(mongo_client.options[:app_name]).to eq('testing')
+      end
     end
 
     context "when no client exists with the key" do

data/spec/mongoid/config_spec.rb

@@ -82,6 +82,37 @@ describe Mongoid::Config do
     end
   end
 
+  context 'when the app_name is set in the config' do
+
+    let(:conf) do
+      CONFIG.merge(options: {app_name: 'admin-reporting'})
+    end
+
+    before do
+      Mongoid.configure do |config|
+        config.load_configuration(conf)
+      end
+    end
+
+    it 'sets the Mongoid.app_name to the provided value' do
+      expect(Mongoid.app_name).to eq('admin-reporting')
+    end
+  end
+
+  context 'when the app_name is not set in the config' do
+
+    before do
+      Mongoid::Config.reset
+      Mongoid.configure do |config|
+        config.load_configuration(CONFIG)
+      end
+    end
+
+    it 'does not set the Mongoid.app_name option' do
+      expect(Mongoid.app_name).to be_nil
+    end
+  end
+
   describe "#load!" do
 
     before(:all) do

data/spec/mongoid/contextual/atomic_spec.rb

@@ -73,6 +73,29 @@ describe Mongoid::Contextual::Atomic do
         expect(smiths.reload.members).to eq([ "Dave" ])
       end
     end
+
+    context 'when the criteria has a collation', if: collation_supported? do
+
+      let(:criteria) do
+        Band.where(members: [ "DAVE" ]).collation(locale: 'en_US', strength: 2)
+      end
+
+      let(:context) do
+        Mongoid::Contextual::Mongo.new(criteria)
+      end
+
+      before do
+        context.add_to_set(members: "Dave", genres: "Electro")
+      end
+
+      it "does not add duplicates" do
+        expect(depeche_mode.reload.members).to eq([ "Dave" ])
+      end
+
+      it "adds multiple operations" do
+        expect(depeche_mode.reload.genres).to eq([ "Electro" ])
+      end
+    end
   end
 
   describe "#bit" do
@@ -121,6 +144,29 @@ describe Mongoid::Contextual::Atomic do
         expect(depeche_mode.reload.likes).to eq(14)
       end
     end
+
+    context 'when the criteria has a collation', if: collation_supported? do
+
+      let!(:depeche_mode) do
+        Band.create(members: [ "Dave" ], likes: 60)
+      end
+
+      let(:criteria) do
+        Band.where(members: [ "DAVE" ]).collation(locale: 'en_US', strength: 2)
+      end
+
+      let(:context) do
+        Mongoid::Contextual::Mongo.new(criteria)
+      end
+
+      before do
+        context.bit(likes: { and: 13, or: 10 })
+      end
+
+      it "performs the bitwise operation on initialized fields" do
+        expect(depeche_mode.reload.likes).to eq(14)
+      end
+    end
   end
 
   describe "#inc" do
@@ -177,6 +223,29 @@ describe Mongoid::Contextual::Atomic do
         expect(beatles.reload.y).to eq(3)
       end
     end
+
+    context 'when the criteria has a collation', if: collation_supported? do
+
+      let!(:depeche_mode) do
+        Band.create(members: [ "Dave" ])
+      end
+
+      let(:criteria) do
+        Band.where(members: [ "DAVE" ]).collation(locale: 'en_US', strength: 2)
+      end
+
+      let(:context) do
+        Mongoid::Contextual::Mongo.new(criteria)
+      end
+
+      before do
+        context.inc(years: 1)
+      end
+
+      it "incs the value and read from alias" do
+        expect(depeche_mode.reload.years).to eq(1)
+      end
+    end
   end
 
   describe "#pop" do
@@ -226,6 +295,29 @@ describe Mongoid::Contextual::Atomic do
         expect(smiths.reload.members).to be_nil
       end
     end
+
+    context 'when the criteria has a collation', if: collation_supported? do
+
+      let!(:depeche_mode) do
+        Band.create(members: [ "Dave" ])
+      end
+
+      let(:criteria) do
+        Band.where(members: [ "DAVE" ]).collation(locale: 'en_US', strength: 2)
+      end
+
+      let(:context) do
+        Mongoid::Contextual::Mongo.new(criteria)
+      end
+
+      before do
+        context.pop(members: 1)
+      end
+
+      it "pops the last element off the array" do
+        expect(depeche_mode.reload.members).to be_empty
+      end
+    end
   end
 
   describe "#pull" do
@@ -257,129 +349,276 @@ describe Mongoid::Contextual::Atomic do
     it "does not error on non existent fields" do
       expect(smiths.reload.members).to be_nil
     end
+
+    context 'when the criteria has a collation', if: collation_supported? do
+
+      let!(:depeche_mode) do
+        Band.create(members: [ "Dave" ])
+      end
+
+      let(:criteria) do
+        Band.where(members: [ "DAVE" ]).collation(locale: 'en_US', strength: 2)
+      end
+
+      let(:context) do
+        Mongoid::Contextual::Mongo.new(criteria)
+      end
+
+      before do
+        context.pull(members: "DAVE")
+      end
+
+      it "pulls when the value is found" do
+        expect(depeche_mode.reload.members).to be_empty
+      end
+    end
   end
 
   describe "#pull_all" do
 
-    let!(:depeche_mode) do
-      Band.create(members: [ "Dave", "Alan", "Fletch" ])
-    end
+    context 'when the criteria does not have a collation' do
 
-    let!(:smiths) do
-      Band.create
-    end
+      let!(:depeche_mode) do
+        Band.create(members: [ "Dave", "Alan", "Fletch" ])
+      end
 
-    let(:criteria) do
-      Band.all
-    end
+      let!(:smiths) do
+        Band.create
+      end
 
-    let(:context) do
-      Mongoid::Contextual::Mongo.new(criteria)
-    end
+      let(:criteria) do
+        Band.all
+      end
 
-    before do
-      context.pull_all(members: [ "Alan", "Dave" ])
-    end
+      let(:context) do
+        Mongoid::Contextual::Mongo.new(criteria)
+      end
+
+      before do
+        context.pull_all(members: [ "Alan", "Dave" ])
+      end
 
-    it "pulls when the values are found" do
-      expect(depeche_mode.reload.members).to eq([ "Fletch" ])
+      it "pulls when the values are found" do
+        expect(depeche_mode.reload.members).to eq([ "Fletch" ])
+      end
+
+      it "does not error on non existent fields" do
+        expect(smiths.reload.members).to be_nil
+      end
     end
 
-    it "does not error on non existent fields" do
-      expect(smiths.reload.members).to be_nil
+    context 'when the criteria has a collation', if: collation_supported? do
+
+      let!(:depeche_mode) do
+        Band.create(members: [ "Dave", "Alan", "Fletch" ])
+      end
+
+      let(:criteria) do
+        Band.all.collation(locale: 'en_US', strength: 2)
+      end
+
+      let(:context) do
+        Mongoid::Contextual::Mongo.new(criteria)
+      end
+
+      before do
+        context.pull_all(members: [ "ALAN", "DAVE" ])
+      end
+
+      it "pulls when the value is found" do
+        expect(depeche_mode.reload.members).to eq([ "Fletch" ])
+      end
     end
   end
 
   describe "#push" do
 
-    let!(:depeche_mode) do
-      Band.create(members: [ "Dave" ])
-    end
+    context 'when the criteria does not have a collation' do
 
-    let!(:smiths) do
-      Band.create
-    end
+      let!(:depeche_mode) do
+        Band.create(members: [ "Dave" ])
+      end
 
-    let(:criteria) do
-      Band.all
-    end
+      let!(:smiths) do
+        Band.create
+      end
 
-    let(:context) do
-      Mongoid::Contextual::Mongo.new(criteria)
-    end
+      let(:criteria) do
+        Band.all
+      end
 
-    before do
-      context.push(members: "Alan")
-    end
+      let(:context) do
+        Mongoid::Contextual::Mongo.new(criteria)
+      end
+
+      before do
+        context.push(members: "Alan")
+      end
+
+      it "pushes the value to existing arrays" do
+        expect(depeche_mode.reload.members).to eq([ "Dave", "Alan" ])
+      end
 
-    it "pushes the value to existing arrays" do
-      expect(depeche_mode.reload.members).to eq([ "Dave", "Alan" ])
+      it "pushes to non existent fields" do
+        expect(smiths.reload.members).to eq([ "Alan" ])
+      end
     end
 
-    it "pushes to non existent fields" do
-      expect(smiths.reload.members).to eq([ "Alan" ])
+    context 'when the criteria has a collation', if: collation_supported? do
+
+      let!(:depeche_mode) do
+        Band.create(members: [ "Dave" ])
+      end
+
+      let!(:smiths) do
+        Band.create
+      end
+
+      let(:criteria) do
+        Band.where(members: ['DAVE']).collation(locale: 'en_US', strength: 2)
+      end
+
+      let(:context) do
+        Mongoid::Contextual::Mongo.new(criteria)
+      end
+
+      before do
+        context.push(members: "Alan")
+      end
+
+      it "pushes the value to existing arrays" do
+        expect(depeche_mode.reload.members).to eq([ "Dave", "Alan" ])
+      end
+
+      it "pushes to non existent fields" do
+        expect(smiths.reload.members).to be_nil
+      end
     end
   end
 
   describe "#push_all" do
 
-    let!(:depeche_mode) do
-      Band.create(members: [ "Dave" ])
-    end
+    context 'when the criteria does not have a collation' do
 
-    let!(:smiths) do
-      Band.create
-    end
+      let!(:depeche_mode) do
+        Band.create(members: [ "Dave" ])
+      end
 
-    let(:criteria) do
-      Band.all
-    end
+      let!(:smiths) do
+        Band.create
+      end
 
-    let(:context) do
-      Mongoid::Contextual::Mongo.new(criteria)
-    end
+      let(:criteria) do
+        Band.all
+      end
 
-    before do
-      context.push_all(members: [ "Alan", "Fletch" ])
-    end
+      let(:context) do
+        Mongoid::Contextual::Mongo.new(criteria)
+      end
+
+      before do
+        context.push_all(members: [ "Alan", "Fletch" ])
+      end
 
-    it "pushes the values to existing arrays" do
-      expect(depeche_mode.reload.members).to eq([ "Dave", "Alan", "Fletch" ])
+      it "pushes the values to existing arrays" do
+        expect(depeche_mode.reload.members).to eq([ "Dave", "Alan", "Fletch" ])
+      end
+
+      it "pushes to non existent fields" do
+        expect(smiths.reload.members).to eq([ "Alan", "Fletch" ])
+      end
     end
 
-    it "pushes to non existent fields" do
-      expect(smiths.reload.members).to eq([ "Alan", "Fletch" ])
+    context 'when the criteria has a collation', if: collation_supported? do
+
+      let!(:depeche_mode) do
+        Band.create(members: [ "Dave" ])
+      end
+
+      let!(:smiths) do
+        Band.create
+      end
+
+      let(:criteria) do
+        Band.where(members: ['DAVE']).collation(locale: 'en_US', strength: 2)
+      end
+
+      let(:context) do
+        Mongoid::Contextual::Mongo.new(criteria)
+      end
+
+      before do
+        context.push_all(members: [ "Alan", "Fletch" ])
+      end
+
+      it "pushes the values to existing arrays" do
+        expect(depeche_mode.reload.members).to eq([ "Dave", "Alan", "Fletch" ])
+      end
     end
   end
 
   describe "#rename" do
 
-    let!(:depeche_mode) do
-      Band.create(members: [ "Dave" ])
-    end
+    context 'when the criteria does not have a collation' do
 
-    let!(:smiths) do
-      Band.create
-    end
+      let!(:depeche_mode) do
+        Band.create(members: [ "Dave" ])
+      end
 
-    let(:criteria) do
-      Band.all
-    end
+      let!(:smiths) do
+        Band.create
+      end
 
-    let(:context) do
-      Mongoid::Contextual::Mongo.new(criteria)
-    end
+      let(:criteria) do
+        Band.all
+      end
 
-    before do
-      context.rename(members: :artists)
-    end
+      let(:context) do
+        Mongoid::Contextual::Mongo.new(criteria)
+      end
 
-    it "renames existing fields" do
-      expect(depeche_mode.reload.artists).to eq([ "Dave" ])
+      before do
+        context.rename(members: :artists)
+      end
+
+      it "renames existing fields" do
+        expect(depeche_mode.reload.artists).to eq([ "Dave" ])
+      end
+
+      it "does not rename non existent fields" do
+        expect(smiths.reload).to_not respond_to(:artists)
+      end
     end
 
-    it "does not rename non existent fields" do
-      expect(smiths.reload).to_not respond_to(:artists)
+    context 'when the criteria has a collation', if: collation_supported? do
+
+      let!(:depeche_mode) do
+        Band.create(members: [ "Dave" ])
+      end
+
+      let!(:smiths) do
+        Band.create
+      end
+
+      let(:criteria) do
+        Band.where(members: ['DAVE']).collation(locale: 'en_US', strength: 2)
+      end
+
+      let(:context) do
+        Mongoid::Contextual::Mongo.new(criteria)
+      end
+
+      before do
+        context.rename(members: :artists)
+      end
+
+      it "renames existing fields" do
+        expect(depeche_mode.reload.artists).to eq([ "Dave" ])
+      end
+
+      it "does not rename non existent fields" do
+        expect(smiths.reload).to_not respond_to(:artists)
+      end
     end
   end
 
@@ -509,5 +748,32 @@ describe Mongoid::Contextual::Atomic do
         end
       end
     end
+
+    context 'when the criteria has a collation', if: collation_supported? do
+
+      let!(:depeche_mode) do
+        Band.create(name: "Depeche Mode", years: 10)
+      end
+
+      let!(:new_order) do
+        Band.create(name: "New Order", years: 10)
+      end
+
+      let(:criteria) do
+        Band.where(name: 'DEPECHE MODE').collation(locale: 'en_US', strength: 2)
+      end
+
+      let(:context) do
+        Mongoid::Contextual::Mongo.new(criteria)
+      end
+
+      before do
+        context.unset(:name)
+      end
+
+      it "unsets the unaliased field" do
+        expect(depeche_mode.reload.name).to be_nil
+      end
+    end
   end
 end