mongoid 5.4.1 → 6.0.0.beta

data/spec/mongoid/extensions/integer_spec.rb

@@ -23,7 +23,7 @@ describe Mongoid::Extensions::Integer do
 
   describe ".demongoize" do
 
-    context "when the the value is an integer" do
+    context "when the value is an integer" do
 
       it "returns a integer" do
         expect(Integer.demongoize(number)).to eq(number)
@@ -133,4 +133,11 @@ describe Mongoid::Extensions::Integer do
       expect(number.mongoize).to eq(number)
     end
   end
+
+  describe "#numeric?" do
+
+    it "returns true" do
+      expect(number.numeric?).to eq(true)
+    end
+  end
 end

data/spec/mongoid/extensions/object_spec.rb

@@ -289,4 +289,15 @@ describe Mongoid::Extensions::Object do
       end
     end
   end
+
+  describe "#numeric?" do
+
+    let(:object) do
+      Object.new
+    end
+
+    it "returns false" do
+      expect(object.numeric?).to eq(false)
+    end
+  end
 end

data/spec/mongoid/extensions/string_spec.rb

@@ -301,6 +301,27 @@ describe Mongoid::Extensions::String do
         expect("blah").to_not be_numeric
       end
     end
+
+    context "when the string is NaN" do
+
+      it "returns true" do
+        expect("NaN").to be_numeric
+      end
+    end
+
+    context "when the string is Infinity" do
+
+      it "returns true" do
+        expect("Infinity").to be_numeric
+      end
+    end
+
+    context "when the string is -Infinity" do
+
+      it "returns true" do
+        expect("-Infinity").to be_numeric
+      end
+    end
   end
 
   describe "#singularize" do

data/spec/mongoid/extensions/time_spec.rb

@@ -204,12 +204,8 @@ describe Mongoid::Extensions::Time do
 
       context "when the string is an invalid time" do
 
-        let(:epoch) do
-          Time.utc(1970, 1, 1, 0, 0, 0, 0)
-        end
-
-        it "converts the time to epoch" do
-          expect(Time.mongoize("time")).to eq(epoch)
+        it "converts the time to nil" do
+          expect(Time.mongoize("time")).to eq(nil)
         end
       end
 
@@ -281,7 +277,7 @@ describe Mongoid::Extensions::Time do
       end
 
       it "does not alter seconds with fractions" do
-        expect(DateTime.mongoize(1.11).to_f).to eq(1.11)
+        expect(DateTime.mongoize(11.11).to_f).to eq(11.11)
       end
 
       context "when using the ActiveSupport time zone" do
@@ -344,7 +340,7 @@ describe Mongoid::Extensions::Time do
       end
 
       it "does not alter seconds with fractions" do
-        expect(Time.mongoize(102.25).to_f).to eq(102.25)
+        expect(Time.mongoize(102.63).to_f).to eq(102.63)
       end
     end
 

data/spec/mongoid/extensions/time_with_zone_spec.rb

@@ -204,12 +204,8 @@ describe Mongoid::Extensions::TimeWithZone do
 
       context "when the string is an invalid time" do
 
-        let(:epoch) do
-          Time.utc(1970, 1, 1, 0, 0, 0)
-        end
-
-        it "returns epoch" do
-          expect(ActiveSupport::TimeWithZone.mongoize("time")).to eq(epoch)
+        it "returns nil" do
+          expect(ActiveSupport::TimeWithZone.mongoize("time")).to eq(nil)
         end
       end
 

data/spec/mongoid/fields/localized_spec.rb

@@ -430,97 +430,6 @@ describe Mongoid::Fields::Localized do
           expect(value).to eq({ "de" => 100 })
         end
       end
-
-      context 'when the type is Boolean' do
-
-        before do
-          I18n.enforce_available_locales = false
-          ::I18n.locale = :de
-        end
-
-        after do
-          ::I18n.locale = :en
-        end
-
-        context "when the value is false" do
-
-          let(:field) do
-            described_class.new(:boolean_value, localize: true, type: Boolean)
-          end
-
-          let(:value) do
-            field.demongoize({ "de" => false })
-          end
-
-          it "returns the boolean value from the set locale" do
-            expect(value).to eq(false)
-          end
-        end
-
-        context "when the value is true" do
-
-          let(:field) do
-            described_class.new(:boolean_value, localize: true, type: Boolean)
-          end
-
-          let(:value) do
-            field.demongoize({"de" => true})
-          end
-
-          it "returns the boolean value from the set locale" do
-            expect(value).to eq(true)
-          end
-        end
-
-        context "when fallbacks are defined" do
-
-          before(:all) do
-            require "i18n/backend/fallbacks"
-            I18n::Backend::Simple.send(:include, I18n::Backend::Fallbacks)
-          end
-
-          context "when the lookup does not need to use fallbacks" do
-
-            context "when the value is false" do
-
-              before do
-                ::I18n.fallbacks[:de] = [:en, :es]
-              end
-
-              let(:field) do
-                described_class.new(:boolean_value, localize: true, type: Boolean)
-              end
-
-              let(:value) do
-                field.demongoize({"de" => false})
-              end
-
-              it "returns the boolean value from the set locale" do
-                expect(value).to eq(false)
-              end
-            end
-
-            context "when the value is true" do
-
-              before do
-                ::I18n.fallbacks[:de] = [:en, :es]
-              end
-
-              let(:field) do
-                described_class.new(:boolean_value, localize: true, type: Boolean)
-              end
-
-              let(:value) do
-                field.demongoize({"de" => true})
-              end
-
-              it "returns the boolean value from the set locale" do
-                expect(value).to eq(true)
-              end
-            end
-          end
-        end
-      end
     end
   end
 end

data/spec/mongoid/findable_spec.rb

@@ -476,7 +476,7 @@ describe Mongoid::Findable do
     end
   end
 
-  Origin::Selectable.forwardables.each do |method|
+  Mongoid::Criteria::Queryable::Selectable.forwardables.each do |method|
 
     describe "##{method}" do
 
@@ -485,4 +485,49 @@ describe Mongoid::Findable do
       end
     end
   end
+
+  context 'when Mongoid is configured to use activesupport time zone' do
+
+    before do
+      Mongoid.use_utc = false
+      Mongoid.use_activesupport_time_zone = true
+      Time.zone = "Asia/Kolkata"
+    end
+
+    let!(:time) do
+      Time.zone.now.tap do |t|
+        User.create(last_login: t, name: 'Tom')
+      end
+    end
+
+    it 'uses activesupport time zone' do
+      expect(User.distinct(:last_login).first.to_s).to eql(time.in_time_zone('Asia/Kolkata').to_s)
+    end
+
+    it 'loads other fields accurately' do
+      expect(User.distinct(:name)).to match_array(['Tom'])
+    end
+  end
+
+  context 'when Mongoid is not configured to use activesupport time zone' do
+
+    before do
+      Mongoid.use_utc = true
+      Mongoid.use_activesupport_time_zone = false
+    end
+
+    let!(:time) do
+      Time.now.tap do |t|
+        User.create(last_login: t, name: 'Tom')
+      end
+    end
+
+    it 'uses utc' do
+      expect(User.distinct(:last_login).first.to_s).to eql(time.utc.to_s)
+    end
+
+    it 'loads other fields accurately' do
+      expect(User.distinct(:name)).to match_array(['Tom'])
+    end
+  end
 end

data/spec/mongoid/indexable_spec.rb

@@ -58,7 +58,9 @@ describe Mongoid::Indexable do
       end
 
       let(:indexes) do
-        klass.with(database: "mongoid_optional").collection.indexes
+        klass.with(database: "mongoid_optional") do |klass|
+          klass.collection.indexes
+        end
       end
 
       it "creates the indexes" do
@@ -107,39 +109,13 @@ describe Mongoid::Indexable do
       end
 
       let(:indexes) do
-        klass.with(database: "mongoid_optional").collection.indexes
-      end
-
-      it "creates the indexes" do
-        expect(indexes.get(_type: 1)).to_not be_nil
-      end
-    end
-
-    context "when a collation option is specified", if: collation_supported? do
-
-      let(:klass) do
-        Class.new do
-          include Mongoid::Document
-          store_in collection: "test_db_indexes"
-          index({name: 1}, {collation: {locale: 'en_US', strength: 2}})
+        klass.with(database: "mongoid_optional") do |klass|
+          klass.collection.indexes
         end
       end
 
-      before do
-        klass.create_indexes
-      end
-
-      after do
-        klass.remove_indexes
-      end
-
-      let(:indexes) do
-        klass.collection.indexes
-      end
-
       it "creates the indexes" do
-        expect(indexes.get("name_1")["collation"]).to_not be_nil
-        expect(indexes.get("name_1")["collation"]["locale"]).to eq('en_US')
+        expect(indexes.get(_type: 1)).to_not be_nil
       end
     end
   end
@@ -177,7 +153,6 @@ describe Mongoid::Indexable do
       Class.new do
         include Mongoid::Document
         field :a, as: :authentication_token
-        store_in collection: :specs
       end
     end
 
@@ -286,21 +261,6 @@ describe Mongoid::Indexable do
       end
     end
 
-    context "when providing a collation option", if: collation_supported? do
-
-      before do
-        klass.index({ name: 1 }, collation: { locale: 'en_US', strength: 2 })
-      end
-
-      let(:options) do
-        klass.index_specification(name: 1).options
-      end
-
-      it "sets the index with a collation option" do
-        expect(options).to eq(collation: { locale: 'en_US', strength: 2 })
-      end
-    end
-
     context "when providing a compound index" do
 
       before do

data/spec/mongoid/interceptable_spec.rb

@@ -280,13 +280,18 @@ describe Mongoid::Interceptable do
       end
     end
 
-    context "callback returns false" do
+    context "callback aborts the callback chain" do
 
       before do
-        expect(artist).to receive(:before_create_stub).once.and_return(false)
+        Artist.before_create(:before_create_fail_stub)
+        expect(artist).to receive(:before_create_fail_stub).once.and_call_original
         artist.save
       end
 
+      after do
+        Artist.reset_callbacks(:create)
+      end
+
       it "does not get saved" do
         expect(artist.persisted?).to be false
       end
@@ -316,13 +321,18 @@ describe Mongoid::Interceptable do
         end
       end
 
-      context "when callback returns false" do
+      context "when callback halts the callback chain" do
 
         before do
-          expect(artist).to receive(:before_save_stub).once.and_return(false)
+          Artist.before_save(:before_save_fail_stub)
+        end
+
+        after do
+          Artist.reset_callbacks(:save)
         end
 
         it "the save returns false" do
+          expect(artist).to receive(:before_save_fail_stub).once.and_call_original
           expect(artist.save).to be false
         end
       end
@@ -343,7 +353,7 @@ describe Mongoid::Interceptable do
       context "when the callback returns true" do
 
         before do
-          expect(artist).to receive(:before_save_stub).once.and_return(true)
+          expect(artist).to receive(:before_update_stub).once.and_return(true)
         end
 
         it "the save returns true" do
@@ -351,13 +361,18 @@ describe Mongoid::Interceptable do
         end
       end
 
-      context "when the callback returns false" do
+      context "when the callback halts the callback chain" do
 
         before do
-          expect(artist).to receive(:before_save_stub).once.and_return(false)
+          Artist.before_update(:before_update_fail_stub)
+        end
+
+        after do
+          Artist.reset_callbacks(:update)
         end
 
         it "the save returns false" do
+          expect(artist).to receive(:before_update_fail_stub).once.and_call_original
           expect(artist.save).to be false
         end
       end
@@ -389,13 +404,18 @@ describe Mongoid::Interceptable do
       end
     end
 
-    context "when the callback returns false" do
+    context "when the callback halts the callback chain" do
 
       before do
-        expect(artist).to receive(:before_destroy_stub).once.and_return(false)
+        Artist.before_destroy(:before_destroy_fail_stub)
+      end
+
+      after do
+        Artist.reset_callbacks(:destroy)
       end
 
       it "the destroy returns false" do
+        expect(artist).to receive(:before_destroy_fail_stub).once.and_call_original
         expect(artist.destroy).to be false
       end
     end
@@ -1434,6 +1454,25 @@ describe Mongoid::Interceptable do
         end
       end
 
+      describe '#after_destroy' do
+
+        context 'when the parent is updated in a child after_destroy callback' do
+
+          let!(:person) do
+            Person.create!(ordered_posts: [OrderedPost.new])
+          end
+
+          before do
+            post = OrderedPost.first
+            post.destroy
+          end
+
+          it 'updates the parent' do
+            expect(person.reload.title).to eq('Minus one ordered post.')
+          end
+        end
+      end
+
       describe "#before_validation" do
 
         context "when the child is new" do
@@ -1544,7 +1583,7 @@ describe Mongoid::Interceptable do
 
     before(:all) do
       Person.before_save do |doc|
-        doc.mode != :prevent_save
+        throw(:abort) if doc.mode == :prevent_save
       end
     end