mongoid 9.0.11 → 9.1.0

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

@@ -1,15 +1,10 @@
 # frozen_string_literal: true
-# rubocop:todo all
-
-require "spec_helper"
 
+require 'spec_helper'
 
 describe Mongoid::Criteria::Queryable::Extensions::Regexp do
-
-  describe ".evolve" do
-
-    context "when provided a regexp" do
-
+  describe '.evolve' do
+    context 'when provided a regexp' do
       let(:regexp) do
         /\A[123]/
       end
@@ -18,30 +13,27 @@ describe Mongoid::Criteria::Queryable::Extensions::Regexp do
         Regexp.evolve(regexp)
       end
 
-      it "returns the regexp" do
+      it 'returns the regexp' do
         expect(evolved).to eq(regexp)
       end
     end
 
-    context "when provided a string" do
-
+    context 'when provided a string' do
       let(:regexp) do
-        "\\A[123]"
+        '\\A[123]'
       end
 
       let(:evolved) do
         Regexp.evolve(regexp)
       end
 
-      it "returns the converted regexp" do
+      it 'returns the converted regexp' do
         expect(evolved).to eq(/\A[123]/)
       end
     end
 
-    context "when provided an array" do
-
-      context "when the elements are regexps" do
-
+    context 'when provided an array' do
+      context 'when the elements are regexps' do
         let(:regexp) do
           /\A[123]/
         end
@@ -54,26 +46,25 @@ describe Mongoid::Criteria::Queryable::Extensions::Regexp do
           Regexp.evolve(array)
         end
 
-        it "returns the regexps" do
+        it 'returns the regexps' do
           expect(evolved).to eq([ regexp ])
         end
 
-        it "does not evolve in place" do
-          expect(evolved).to_not equal(array)
+        it 'does not evolve in place' do
+          expect(evolved).not_to equal(array)
         end
       end
 
-      context "when the elements are strings" do
-
+      context 'when the elements are strings' do
         let(:regexp) do
-          "\\A[123]"
+          '\\A[123]'
         end
 
         let(:evolved) do
           Regexp.evolve([ regexp ])
         end
 
-        it "returns the regexps" do
+        it 'returns the regexps' do
           expect(evolved).to eq([ /\A[123]/ ])
         end
       end

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

@@ -1,40 +1,33 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "spec_helper"
+require 'spec_helper'
 
 describe Set do
-
-  describe ".evolve" do
-
-    context "when provided a set" do
-
+  describe '.evolve' do
+    context 'when provided a set' do
       let(:set) do
-        ::Set.new([ 1, 2, 3 ])
+        Set.new([ 1, 2, 3 ])
       end
 
-      it "returns an array" do
+      it 'returns an array' do
         expect(described_class.evolve(set)).to eq([ 1, 2, 3 ])
       end
     end
 
-    context "when provided an array" do
-
-      it "returns an array" do
+    context 'when provided an array' do
+      it 'returns an array' do
         expect(described_class.evolve([ 1, 2, 3 ])).to eq([ 1, 2, 3 ])
       end
     end
 
-    context "when provided another object" do
-
-      it "returns the object" do
-        expect(described_class.evolve("testing")).to eq("testing")
+    context 'when provided another object' do
+      it 'returns the object' do
+        expect(described_class.evolve('testing')).to eq('testing')
       end
     end
 
-    context "when provided nil" do
-
-      it "returns nil" do
+    context 'when provided nil' do
+      it 'returns nil' do
         expect(described_class.evolve(nil)).to be_nil
       end
     end

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

@@ -1,82 +1,73 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "spec_helper"
+require 'spec_helper'
 
 describe String do
-
-  describe "#__evolve_date__" do
-
-    context "when the string is verbose" do
-
+  describe '#__evolve_date__' do
+    context 'when the string is verbose' do
       let(:date) do
-        "1st Jan 2010"
+        '1st Jan 2010'
       end
 
       let(:evolved) do
         date.__evolve_date__
       end
 
-      it "returns the strings as a times" do
+      it 'returns the strings as a times' do
         expect(evolved).to eq(Time.new(2010, 1, 1, 0, 0, 0, 0).utc)
       end
     end
 
-    context "when the string is in international format" do
-
+    context 'when the string is in international format' do
       let(:date) do
-        "2010-1-1"
+        '2010-1-1'
       end
 
       let(:evolved) do
         date.__evolve_date__
       end
 
-      it "returns the strings as a times" do
+      it 'returns the strings as a times' do
         expect(evolved).to eq(Time.new(2010, 1, 1, 0, 0, 0, 0).utc)
       end
     end
   end
 
-  describe "#__evolve_time__" do
-
-    context "when the string is verbose" do
-
+  describe '#__evolve_time__' do
+    context 'when the string is verbose' do
       let(:date) do
-        "1st Jan 2010 12:00:00+01:00"
+        '1st Jan 2010 12:00:00+01:00'
       end
 
       let(:evolved) do
         date.__evolve_time__
       end
 
-      it "returns the string as a utc time" do
+      it 'returns the string as a utc time' do
         expect(evolved).to eq(Time.new(2010, 1, 1, 11, 0, 0, 0).utc)
       end
     end
 
-    context "when the string is in international format" do
-
+    context 'when the string is in international format' do
       let(:date) do
-        "2010-01-01 12:00:00+01:00"
+        '2010-01-01 12:00:00+01:00'
       end
 
       let(:evolved) do
         date.__evolve_time__
       end
 
-      it "returns the string as a utc time" do
+      it 'returns the string as a utc time' do
         expect(evolved).to eq(Time.new(2010, 1, 1, 11, 0, 0, 0).utc)
       end
     end
 
-    context "when the string without timezone" do
-
-      context "when setting ActiveSupport time zone" do
-        time_zone_override "Tokyo"
+    context 'when the string without timezone' do
+      context 'when setting ActiveSupport time zone' do
+        time_zone_override 'Tokyo'
 
         let(:date) do
-          "2010-01-01 5:00:00"
+          '2010-01-01 5:00:00'
         end
 
         let(:evolved) do
@@ -90,92 +81,83 @@ describe String do
     end
   end
 
-  describe "#__sort_option__" do
-
-    context "when the string contains ascending" do
-
+  describe '#__sort_option__' do
+    context 'when the string contains ascending' do
       let(:option) do
-        "field_one ascending, field_two ascending".__sort_option__
+        'field_one ascending, field_two ascending'.__sort_option__
       end
 
-      it "returns the ascending sort option hash" do
+      it 'returns the ascending sort option hash' do
         expect(option).to eq({ field_one: 1, field_two: 1 })
       end
     end
 
-    context "when the string contains asc" do
-
+    context 'when the string contains asc' do
       let(:option) do
-        "field_one asc, field_two asc".__sort_option__
+        'field_one asc, field_two asc'.__sort_option__
       end
 
-      it "returns the ascending sort option hash" do
+      it 'returns the ascending sort option hash' do
         expect(option).to eq({ field_one: 1, field_two: 1 })
       end
     end
 
-    context "when the string contains ASCENDING" do
-
+    context 'when the string contains ASCENDING' do
       let(:option) do
-        "field_one ASCENDING, field_two ASCENDING".__sort_option__
+        'field_one ASCENDING, field_two ASCENDING'.__sort_option__
       end
 
-      it "returns the ascending sort option hash" do
+      it 'returns the ascending sort option hash' do
         expect(option).to eq({ field_one: 1, field_two: 1 })
       end
     end
 
-    context "when the string contains ASC" do
-
+    context 'when the string contains ASC' do
       let(:option) do
-        "field_one ASC, field_two ASC".__sort_option__
+        'field_one ASC, field_two ASC'.__sort_option__
       end
 
-      it "returns the ascending sort option hash" do
+      it 'returns the ascending sort option hash' do
         expect(option).to eq({ field_one: 1, field_two: 1 })
       end
     end
 
-    context "when the string contains descending" do
-
+    context 'when the string contains descending' do
       let(:option) do
-        "field_one descending, field_two descending".__sort_option__
+        'field_one descending, field_two descending'.__sort_option__
       end
 
-      it "returns the descending sort option hash" do
+      it 'returns the descending sort option hash' do
         expect(option).to eq({ field_one: -1, field_two: -1 })
       end
     end
 
-    context "when the string contains desc" do
-
+    context 'when the string contains desc' do
       let(:option) do
-        "field_one desc, field_two desc".__sort_option__
+        'field_one desc, field_two desc'.__sort_option__
       end
 
-      it "returns the descending sort option hash" do
+      it 'returns the descending sort option hash' do
         expect(option).to eq({ field_one: -1, field_two: -1 })
       end
     end
 
-    context "when the string contains DESCENDING" do
-
+    context 'when the string contains DESCENDING' do
       let(:option) do
-        "field_one DESCENDING, field_two DESCENDING".__sort_option__
+        'field_one DESCENDING, field_two DESCENDING'.__sort_option__
       end
 
-      it "returns the descending sort option hash" do
+      it 'returns the descending sort option hash' do
         expect(option).to eq({ field_one: -1, field_two: -1 })
       end
     end
 
-    context "when the string contains DESC" do
-
+    context 'when the string contains DESC' do
       let(:option) do
-        "field_one DESC, field_two DESC".__sort_option__
+        'field_one DESC, field_two DESC'.__sort_option__
       end
 
-      it "returns the descending sort option hash" do
+      it 'returns the descending sort option hash' do
         expect(option).to eq({ field_one: -1, field_two: -1 })
       end
     end
@@ -183,6 +165,7 @@ describe String do
 
   describe '#__expr_part__' do
     subject(:specified) { 'field'.__expr_part__(value) }
+
     let(:value) { 10 }
 
     it 'returns the expression with the value' do
@@ -228,7 +211,7 @@ describe String do
         let(:value) { 'test' }
 
         it 'returns the expression with the value negated' do
-          expect(specified).to eq({ 'field' => { '$ne' => 'test' }})
+          expect(specified).to eq({ 'field' => { '$ne' => 'test' } })
         end
       end
     end

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

@@ -1,14 +1,11 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "spec_helper"
+require 'spec_helper'
 
 describe Symbol do
-
-  describe ".add_key" do
-
+  describe '.add_key' do
     before do
-      described_class.add_key(:fubar, :union, "$fu", "$bar") do |value|
+      described_class.add_key(:fubar, :union, '$fu', '$bar') do |value|
         value.to_s
       end
     end
@@ -21,94 +18,83 @@ describe Symbol do
       :testing.fubar
     end
 
-    it "adds the method to symbol" do
+    it 'adds the method to symbol' do
       expect(fubar).to be_a(Mongoid::Criteria::Queryable::Key)
     end
 
-    it "sets the key name" do
+    it 'sets the key name' do
       expect(fubar.name).to eq(:testing)
     end
 
-    it "sets the key strategy" do
+    it 'sets the key strategy' do
       expect(fubar.strategy).to eq(:__union__)
     end
 
-    it "sets the key operator" do
-      expect(fubar.operator).to eq("$fu")
+    it 'sets the key operator' do
+      expect(fubar.operator).to eq('$fu')
     end
 
-    it "sets the additional key operator" do
-      expect(fubar.expanded).to eq("$bar")
+    it 'sets the additional key operator' do
+      expect(fubar.expanded).to eq('$bar')
     end
 
-    it "sets the transform block" do
+    it 'sets the transform block' do
       expect(fubar.block).to be
     end
   end
 
-  describe ".evolve" do
-
-    context "when provided nil" do
-
-      it "returns nil" do
+  describe '.evolve' do
+    context 'when provided nil' do
+      it 'returns nil' do
         expect(described_class.evolve(nil)).to be_nil
       end
     end
 
-    context "when provided a string" do
-
-      it "returns the string as a symbol" do
-        expect(described_class.evolve("test")).to eq(:test)
+    context 'when provided a string' do
+      it 'returns the string as a symbol' do
+        expect(described_class.evolve('test')).to eq(:test)
       end
     end
   end
 
-  describe "#__expr_part__" do
-
+  describe '#__expr_part__' do
     let(:specified) do
       :field.__expr_part__(10)
     end
 
-    it "returns the string with the value" do
+    it 'returns the string with the value' do
       expect(specified).to eq({ field: 10 })
     end
 
-    context "with a regexp" do
-
+    context 'with a regexp' do
       let(:specified) do
         :field.__expr_part__(/test/)
       end
 
-      it "returns the symbol with the value" do
+      it 'returns the symbol with the value' do
         expect(specified).to eq({ field: /test/ })
       end
-
     end
 
-    context "when negated" do
-
-      context "with a regexp" do
-
+    context 'when negated' do
+      context 'with a regexp' do
         let(:specified) do
           :field.__expr_part__(/test/, true)
         end
 
-        it "returns the symbol with the value negated" do
-          expect(specified).to eq({ field: { "$not" => /test/ } })
+        it 'returns the symbol with the value negated' do
+          expect(specified).to eq({ field: { '$not' => /test/ } })
         end
-
       end
 
-      context "with anything else" do
-
+      context 'with anything else' do
         let(:specified) do
           :field.__expr_part__('test', true)
         end
 
-        it "returns the symbol with the value negated" do
-          expect(specified).to eq({ field: { "$ne" => "test" }})
+        it 'returns the symbol with the value negated' do
+          expect(specified).to eq({ field: { '$ne' => 'test' } })
         end
-
       end
     end
   end