mongoid 7.1.1 → 8.1.0

data/spec/mongoid/criteria/queryable/extensions/big_decimal_spec.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-# encoding: utf-8
 
 require "spec_helper"
 
@@ -7,55 +6,163 @@ describe BigDecimal do
 
   describe ".evolve" do
 
-    context "when provided a big decimal" do
+    context 'when map_big_decimal_to_decimal128 is false' do
+      config_override :map_big_decimal_to_decimal128, false
 
-      let(:big_decimal) do
-        BigDecimal("123456.789")
+      context "when provided a big decimal" do
+
+        let(:big_decimal) do
+          BigDecimal("123456.789")
+        end
+
+        it "returns the decimal as a string" do
+          expect(described_class.evolve(big_decimal)).to eq(big_decimal.to_s)
+        end
       end
 
-      it "returns the decimal as a string" do
-        expect(described_class.evolve(big_decimal)).to eq(big_decimal.to_s)
+      context "when provided a non big decimal" do
+
+        it "returns the object as a string" do
+          expect(described_class.evolve("testing")).to eq("testing")
+        end
+      end
+
+      context "when provided an array of big decimals" do
+
+        let(:bd_one) do
+          BigDecimal("123456.789")
+        end
+
+        let(:bd_two) do
+          BigDecimal("123333.789")
+        end
+
+        let(:array) do
+          [ bd_one, bd_two ]
+        end
+
+        let(:evolved) do
+          described_class.evolve(array)
+        end
+
+        it "returns the array as strings" do
+          expect(evolved).to eq([ bd_one.to_s, bd_two.to_s ])
+        end
+
+        it "does not evolve in place" do
+          expect(evolved).to_not equal(array)
+        end
+      end
+
+      context "when provided nil" do
+
+        it "returns nil" do
+          expect(described_class.evolve(nil)).to be_nil
+        end
+      end
+
+      context "when provided a numeric" do
+        it "returns a string" do
+          expect(described_class.evolve(1)).to eq('1')
+        end
+      end
+
+      context "when provided a bogus value" do
+        it "returns the value" do
+          expect(described_class.evolve(:bogus)).to eq(:bogus)
+        end
       end
-    end
 
-    context "when provided a non big decimal" do
+      context "when provided a bogus string" do
+        it "returns the string" do
+          expect(described_class.evolve("bogus")).to eq("bogus")
+        end
+      end
 
-      it "returns the object as a string" do
-        expect(described_class.evolve("testing")).to eq("testing")
+      context "when provided a valid string" do
+        it "returns the string" do
+          expect(described_class.evolve("1")).to eq("1")
+        end
       end
     end
 
-    context "when provided an array of big decimals" do
+    context 'when map_big_decimal_to_decimal128 is true' do
+      config_override :map_big_decimal_to_decimal128, true
 
-      let(:bd_one) do
-        BigDecimal("123456.789")
+      context "when provided a big decimal" do
+
+        let(:big_decimal) do
+          BigDecimal("123456.789")
+        end
+
+        it "returns the decimal as a BSON::Decimal128 object" do
+          expect(described_class.evolve(big_decimal)).to eq(BSON::Decimal128.new(big_decimal))
+        end
       end
 
-      let(:bd_two) do
-        BigDecimal("123333.789")
+      context "when provided a non big decimal" do
+
+        it "returns the object as a string" do
+          expect(described_class.evolve("testing")).to eq("testing")
+        end
       end
 
-      let(:array) do
-        [ bd_one, bd_two ]
+      context "when provided an array of big decimals" do
+
+        let(:bd_one) do
+          BigDecimal("123456.789")
+        end
+
+        let(:bd_two) do
+          BigDecimal("123333.789")
+        end
+
+        let(:array) do
+          [ bd_one, bd_two ]
+        end
+
+        let(:evolved) do
+          described_class.evolve(array)
+        end
+
+        it "returns the array as BSON::Decimal128s" do
+          expect(evolved).to eq([ BSON::Decimal128.new(bd_one), BSON::Decimal128.new(bd_two) ])
+        end
+
+        it "does not evolve in place" do
+          expect(evolved).to_not equal(array)
+        end
       end
 
-      let(:evolved) do
-        described_class.evolve(array)
+      context "when provided nil" do
+
+        it "returns nil" do
+          expect(described_class.evolve(nil)).to be_nil
+        end
       end
 
-      it "returns the array as strings" do
-        expect(evolved).to eq([ bd_one.to_s, bd_two.to_s ])
+      context "when provided a numeric" do
+        it "returns a string" do
+          expect(described_class.evolve(1)).to eq(BSON::Decimal128.new('1'))
+        end
       end
 
-      it "does not evolve in place" do
-        expect(evolved).to_not equal(array)
+      context "when provided a bogus value" do
+        it "returns the value" do
+          expect(described_class.evolve(:bogus)).to eq(:bogus)
+        end
       end
-    end
 
-    context "when provided nil" do
+      context "when provided a bogus string" do
+        it "returns the string" do
+          expect(described_class.evolve("bogus")).to eq("bogus")
+        end
+      end
 
-      it "returns nil" do
-        expect(described_class.evolve(nil)).to be_nil
+      context "when provided a valid string" do
+        it "returns the string" do
+          expect(described_class.evolve("1")).to eq("1")
+        end
       end
     end
   end

data/spec/mongoid/criteria/queryable/extensions/boolean_spec.rb

@@ -1,9 +1,8 @@
 # frozen_string_literal: true
-# encoding: utf-8
 
 require "spec_helper"
 
-describe Boolean do
+describe Mongoid::Boolean do
 
   describe ".evolve" do
 

data/spec/mongoid/criteria/queryable/extensions/date_spec.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-# encoding: utf-8
 
 require "spec_helper"
 
@@ -323,6 +322,17 @@ describe Date do
       end
     end
 
+    context "when provided an invalid string" do
+
+      let(:evolved) do
+        described_class.evolve("bogus")
+      end
+
+      it "returns that string" do
+        expect(evolved).to eq("bogus")
+      end
+    end
+
     context "when provided nil" do
 
       it "returns nil" do

data/spec/mongoid/criteria/queryable/extensions/date_time_spec.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-# encoding: utf-8
 
 require "spec_helper"
 
@@ -414,6 +413,17 @@ describe DateTime do
       end
     end
 
+    context "when provided an invalid string" do
+
+      let(:evolved) do
+        described_class.evolve("bogus")
+      end
+
+      it "returns that string" do
+        expect(evolved).to eq("bogus")
+      end
+    end
+
     context "when provided nil" do
 
       it "returns nil" do

data/spec/mongoid/criteria/queryable/extensions/float_spec.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-# encoding: utf-8
 
 require "spec_helper"
 

data/spec/mongoid/criteria/queryable/extensions/hash_spec.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-# encoding: utf-8
 
 require "spec_helper"
 
@@ -312,19 +311,4 @@ describe Hash do
       end
     end
   end
-
-  describe "#update_values" do
-
-    let(:hash) do
-      { field: "1" }
-    end
-
-    before do
-      hash.update_values(&:to_i)
-    end
-
-    it "updates each value in the hash" do
-      expect(hash).to eq({ field: 1 })
-    end
-  end
 end

data/spec/mongoid/criteria/queryable/extensions/integer_spec.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-# encoding: utf-8
 
 require "spec_helper"
 

data/spec/mongoid/criteria/queryable/extensions/nil_class_spec.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-# encoding: utf-8
 
 require "spec_helper"
 

data/spec/mongoid/criteria/queryable/extensions/numeric_spec.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-# encoding: utf-8
 
 require "spec_helper"
 
@@ -12,42 +11,108 @@ describe Mongoid::Criteria::Queryable::Extensions::Numeric::ClassMethods do
   end
 
   describe "#__numeric__" do
-
     let(:actual) { host.__numeric__(str) }
 
     context "when the string is a whole number" do
-
       let(:str) { '123' }
 
-      it "returns the value as integer" do
+      it "returns the value as an integer" do
         expect(actual).to eq(123)
+        expect(actual).to be_a Integer
       end
     end
 
     context "when the string is a floating point number" do
-
       let(:str) { '123.45' }
 
       it "returns the value as a float" do
         expect(actual).to eq(123.45)
+        expect(actual).to be_a Float
       end
     end
 
-    context "when the string is a dot only" do
+    context "when the string is a floating point number with a leading dot" do
+      let(:str) { '.45' }
 
+      it "returns the value as a float" do
+        expect(actual).to eq(0.45)
+        expect(actual).to be_a Float
+      end
+    end
+
+    context "when the string is a dot only" do
       let(:str) { '.' }
 
       it "returns zero" do
         expect(actual).to eq(0)
+        expect(actual).to be_a Integer
       end
     end
 
-    context "when the string is a number with fractional part consisting of zeros" do
+    context "when the string is a number with a trailing dot" do
+      let(:str) { '123.' }
 
+      it "returns zero" do
+        expect(actual).to eq(123)
+        expect(actual).to be_a Integer
+      end
+    end
+
+    context "when the string is a number with fractional part consisting of zeros" do
       let(:str) { '12.000' }
 
-      it "returns the value as integer" do
+      it "returns the value as an integer" do
         expect(actual).to eq(12)
+        expect(actual).to be_a Integer
+      end
+    end
+
+    context "when the string is a number with leading dot then zeros" do
+      let(:str) { '.000' }
+
+      it "returns the value as an integer" do
+        expect(actual).to eq(0)
+        expect(actual).to be_a Integer
+      end
+    end
+
+    context "when the string is non-numeric" do
+      let(:str) { 'foo' }
+
+      it "raises ArgumentError" do
+        expect { actual }.to raise_error(ArgumentError)
+      end
+    end
+
+    context "when the string is non-numeric with leading dot" do
+      let(:str) { '.foo' }
+
+      it "raises ArgumentError" do
+        expect { actual }.to raise_error(ArgumentError)
+      end
+    end
+
+    context "when the string is non-numeric with trailing dot" do
+      let(:str) { 'foo.' }
+
+      it "raises ArgumentError" do
+        expect { actual }.to raise_error(ArgumentError)
+      end
+    end
+
+    context "when the string is non-numeric with trailing dot and zeroes" do
+      let(:str) { 'foo.000' }
+
+      it "raises ArgumentError" do
+        expect { actual }.to raise_error(ArgumentError)
+      end
+    end
+
+    context "when the string is empty" do
+      let(:str) { '' }
+
+      it "raises ArgumentError" do
+        expect { actual }.to raise_error(ArgumentError)
       end
     end
   end

data/spec/mongoid/criteria/queryable/extensions/object_spec.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-# encoding: utf-8
 
 require "spec_helper"