mongoid 7.2.1 → 9.0.0

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

@@ -1,7 +1,8 @@
 # frozen_string_literal: true
-# encoding: utf-8
+# rubocop:todo all
 
 require "spec_helper"
+require_relative './selectable_shared_examples'
 
 class FieldWithSerializer
   def evolve(object)
@@ -19,43 +20,6 @@ describe Mongoid::Criteria::Queryable::Selectable do
     Mongoid::Query.new("id" => "_id")
   end
 
-  shared_examples_for "returns a cloned query" do
-
-    it "returns a cloned query" do
-      expect(selection).to_not equal(query)
-    end
-  end
-
-  shared_examples_for 'requires an argument' do
-    context "when provided no argument" do
-
-      let(:selection) do
-        query.send(query_method)
-      end
-
-      it "raises ArgumentError" do
-        expect do
-          selection.selector
-        end.to raise_error(ArgumentError)
-      end
-    end
-  end
-
-  shared_examples_for 'requires a non-nil argument' do
-    context "when provided nil" do
-
-      let(:selection) do
-        query.send(query_method, nil)
-      end
-
-      it "raises CriteriaArgumentRequired" do
-        expect do
-          selection.selector
-        end.to raise_error(Mongoid::Errors::CriteriaArgumentRequired, /#{query_method}/)
-      end
-    end
-  end
-
   shared_examples_for 'supports merge strategies' do
 
     context 'when the field is not aliased' do
@@ -518,7 +482,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
 
         it "adds the $elemMatch expression" do
           expect(selection.selector).to eq({
-            "users" => { "$elemMatch" => { name: "value" }}
+            "users" => { "$elemMatch" => { "name" => "value" }}
           })
         end
 
@@ -562,8 +526,8 @@ describe Mongoid::Criteria::Queryable::Selectable do
 
         it "adds the $elemMatch expression" do
           expect(selection.selector).to eq({
-            "users" => { "$elemMatch" => { name: "value" }},
-            "comments" => { "$elemMatch" => { text: "value" }}
+            "users" => { "$elemMatch" => { "name" => "value" }},
+            "comments" => { "$elemMatch" => { "text" => "value" }}
           })
         end
 
@@ -585,8 +549,8 @@ describe Mongoid::Criteria::Queryable::Selectable do
 
         it "adds the $elemMatch expression" do
           expect(selection.selector).to eq({
-            "users" => { "$elemMatch" => { name: "value" }},
-            "comments" => { "$elemMatch" => { text: "value" }}
+            "users" => { "$elemMatch" => { "name" => "value" }},
+            "comments" => { "$elemMatch" => { "text" => "value" }}
           })
         end
 
@@ -603,9 +567,10 @@ describe Mongoid::Criteria::Queryable::Selectable do
             elem_match(users: { state: "new" })
         end
 
-        it "overrides the $elemMatch expression" do
+        it "adds an $elemMatch expression" do
           expect(selection.selector).to eq({
-            "users" => { "$elemMatch" => { state: "new" }}
+            "users" => { "$elemMatch" => { "name" => "value" } },
+            "$and" => [ { "users" => { "$elemMatch" => { "state" => "new" } } } ],
           })
         end
 
@@ -732,127 +697,119 @@ describe Mongoid::Criteria::Queryable::Selectable do
     end
   end
 
-  %i(geo_spatial geo_spacial).each do |meth|
-    describe "#geo_spacial" do
+  describe "#geo_spatial" do
 
-      let(:query_method) { meth }
+    let(:query_method) { :geo_spatial }
 
-      it_behaves_like 'requires an argument'
-      it_behaves_like 'requires a non-nil argument'
+    it_behaves_like 'requires an argument'
+    it_behaves_like 'requires a non-nil argument'
 
-      context "when provided a criterion" do
+    context "when provided a criterion" do
 
-        context "when the geometry is a point intersection" do
+      context "when the geometry is a point intersection" do
 
-          let(:selection) do
-            query.public_send(query_method, :location.intersects_point => [ 1, 10 ])
-          end
+        let(:selection) do
+          query.geo_spatial(:location.intersects_point => [ 1, 10 ])
+        end
 
-          it "adds the $geoIntersects expression" do
-            expect(selection.selector).to eq({
-              "location" => {
-                "$geoIntersects" => {
-                  "$geometry" => {
-                    "type" => "Point",
-                    "coordinates" => [ 1, 10 ]
-                  }
+        it "adds the $geoIntersects expression" do
+          expect(selection.selector).to eq({
+            "location" => {
+              "$geoIntersects" => {
+                "$geometry" => {
+                  "type" => "Point",
+                  "coordinates" => [ 1, 10 ]
                 }
               }
-            })
-          end
-
-          it_behaves_like "returns a cloned query"
+            }
+          })
         end
 
-        context "when the geometry is a line intersection" do
+        it_behaves_like "returns a cloned query"
+      end
 
-          let(:selection) do
-            query.public_send(query_method, :location.intersects_line => [[ 1, 10 ], [ 2, 10 ]])
-          end
+      context "when the geometry is a line intersection" do
 
-          it "adds the $geoIntersects expression" do
-            expect(selection.selector).to eq({
-              "location" => {
-                "$geoIntersects" => {
-                  "$geometry" => {
-                    "type" => "LineString",
-                    "coordinates" => [[ 1, 10 ], [ 2, 10 ]]
-                  }
+        let(:selection) do
+          query.geo_spatial(:location.intersects_line => [[ 1, 10 ], [ 2, 10 ]])
+        end
+
+        it "adds the $geoIntersects expression" do
+          expect(selection.selector).to eq({
+            "location" => {
+              "$geoIntersects" => {
+                "$geometry" => {
+                  "type" => "LineString",
+                  "coordinates" => [[ 1, 10 ], [ 2, 10 ]]
                 }
               }
-            })
-          end
-
-          it_behaves_like "returns a cloned query"
+            }
+          })
         end
 
-        context "when the geometry is a polygon intersection" do
+        it_behaves_like "returns a cloned query"
+      end
 
-          let(:selection) do
-            query.public_send(query_method,
-              :location.intersects_polygon => [[[ 1, 10 ], [ 2, 10 ], [ 1, 10 ]]]
-            )
-          end
+      context "when the geometry is a polygon intersection" do
 
-          it "adds the $geoIntersects expression" do
-            expect(selection.selector).to eq({
-              "location" => {
-                "$geoIntersects" => {
-                  "$geometry" => {
-                    "type" => "Polygon",
-                    "coordinates" => [[[ 1, 10 ], [ 2, 10 ], [ 1, 10 ]]]
-                  }
+        let(:selection) do
+          query.geo_spatial(:location.intersects_polygon => [[[ 1, 10 ], [ 2, 10 ], [ 1, 10 ]]])
+        end
+
+        it "adds the $geoIntersects expression" do
+          expect(selection.selector).to eq({
+            "location" => {
+              "$geoIntersects" => {
+                "$geometry" => {
+                  "type" => "Polygon",
+                  "coordinates" => [[[ 1, 10 ], [ 2, 10 ], [ 1, 10 ]]]
                 }
               }
-            })
-          end
+            }
+          })
+        end
 
-          it_behaves_like "returns a cloned query"
+        it_behaves_like "returns a cloned query"
+      end
+
+      context "when the geometry is within a polygon" do
+
+        let(:selection) do
+          query.geo_spatial(:location.within_polygon => [[[ 1, 10 ], [ 2, 10 ], [ 1, 10 ]]])
         end
 
-        context "when the geometry is within a polygon" do
+        it "adds the $geoIntersects expression" do
+          expect(selection.selector).to eq({
+            "location" => {
+              "$geoWithin" => {
+                "$geometry" => {
+                  "type" => "Polygon",
+                  "coordinates" => [[[ 1, 10 ], [ 2, 10 ], [ 1, 10 ]]]
+                }
+              }
+            }
+          })
+        end
 
+        context "when used with the $box operator ($geoWithin query) " do
           let(:selection) do
-            query.public_send(query_method,
-              :location.within_polygon => [[[ 1, 10 ], [ 2, 10 ], [ 1, 10 ]]]
-            )
+            query.geo_spatial(:location.within_box => [[ 1, 10 ], [ 2, 10 ]])
           end
 
           it "adds the $geoIntersects expression" do
             expect(selection.selector).to eq({
               "location" => {
                 "$geoWithin" => {
-                  "$geometry" => {
-                    "type" => "Polygon",
-                    "coordinates" => [[[ 1, 10 ], [ 2, 10 ], [ 1, 10 ]]]
-                  }
+                  "$box" => [
+                    [ 1, 10 ], [ 2, 10 ]
+                  ]
                 }
               }
             })
           end
-
-          context "when used with the $box operator ($geoWithin query) " do
-            let(:selection) do
-              query.public_send(query_method,
-                :location.within_box => [[ 1, 10 ], [ 2, 10 ]]
-              )
-            end
-
-            it "adds the $geoIntersects expression" do
-              expect(selection.selector).to eq({
-                "location" => {
-                  "$geoWithin" => {
-                    "$box" => [
-                      [ 1, 10 ], [ 2, 10 ]
-                    ]
-                  }
-                }
-              })
-            end
-          end
-
-          it_behaves_like "returns a cloned query"
         end
+
+        it_behaves_like "returns a cloned query"
       end
     end
   end
@@ -928,10 +885,11 @@ describe Mongoid::Criteria::Queryable::Selectable do
           query.gt(first: 10).gt(first: 15)
         end
 
-        it "overwrites the first $gt selector" do
+        it "adds a second $gt selector" do
           expect(selection.selector).to eq({
-            "first" => { "$gt" => 15 }
-          })
+            "first" => { "$gt" => 10 },
+            "$and" => [{ "first" => { "$gt" => 15 } }]
+            })
         end
 
         it "returns a cloned query" do
@@ -1012,9 +970,10 @@ describe Mongoid::Criteria::Queryable::Selectable do
           query.gte(first: 10).gte(first: 15)
         end
 
-        it "overwrites the first $gte selector" do
+        it "adds a second $gte selector" do
           expect(selection.selector).to eq({
-            "first" =>  { "$gte" => 15 }
+            "first" =>  { "$gte" => 10 },
+            "$and" => [ { "first" => { "$gte" => 15 } } ]
           })
         end
 
@@ -1189,9 +1148,10 @@ describe Mongoid::Criteria::Queryable::Selectable do
           query.lt(first: 10).lt(first: 15)
         end
 
-        it "overwrites the first $lt selector" do
+        it "adds a second $lt selector" do
           expect(selection.selector).to eq({
-            "first" =>  { "$lt" => 15 }
+            "first" =>  { "$lt" => 10 },
+            "$and" => [ { "first" => { "$lt" => 15 } } ]
           })
         end
 
@@ -1273,9 +1233,10 @@ describe Mongoid::Criteria::Queryable::Selectable do
           query.lte(first: 10).lte(first: 15)
         end
 
-        it "overwrites the first $lte selector" do
+        it "adds a second $lte selector" do
           expect(selection.selector).to eq({
-            "first" =>  { "$lte" => 15 }
+            "first" =>  { "$lte" => 10 },
+            "$and" => [ { "first" => { "$lte" => 15 } } ]
           })
         end
 
@@ -1964,7 +1925,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
       end
 
       # MongoDB server can only handle one text expression at a time,
-      # per https://docs.mongodb.com/manual/reference/operator/query/text/.
+      # per https://www.mongodb.com/docs/manual/reference/operator/query/text/.
       # Nonetheless we test that the query is built correctly when
       # a user supplies more than one text condition.
       it 'merges conditions' do
@@ -1978,534 +1939,6 @@ describe Mongoid::Criteria::Queryable::Selectable do
     end
   end
 
-  describe "#where" do
-
-    let(:query_method) { :where }
-
-    context "when provided no criterion" do
-
-      let(:selection) do
-        query.where
-      end
-
-      it "does not add any criterion" do
-        expect(selection.selector).to eq({})
-      end
-
-      it "returns the query" do
-        expect(selection).to eq(query)
-      end
-
-      it "returns a cloned query" do
-        expect(selection).to_not equal(query)
-      end
-    end
-
-    it_behaves_like 'requires a non-nil argument'
-
-    context "when provided a string" do
-
-      let(:selection) do
-        query.where("this.value = 10")
-      end
-
-      it "adds the $where criterion" do
-        expect(selection.selector).to eq({ "$where" => "this.value = 10" })
-      end
-
-      it "returns a cloned query" do
-        expect(selection).to_not equal(query)
-      end
-
-      context 'when multiple calls with string argument are made' do
-
-        let(:selection) do
-          query.where("this.value = 10").where('foo.bar')
-        end
-
-        it 'combines conditions' do
-          expect(selection.selector).to eq(
-            "$where" => "this.value = 10", '$and' => [{'$where' => 'foo.bar'}],
-          )
-        end
-      end
-
-      context 'when called with string argument and with hash argument' do
-
-        let(:selection) do
-          query.where("this.value = 10").where(foo: 'bar')
-        end
-
-        it 'combines conditions' do
-          expect(selection.selector).to eq(
-            "$where" => "this.value = 10", 'foo' => 'bar',
-          )
-        end
-      end
-
-      context 'when called with hash argument and with string argument' do
-
-        let(:selection) do
-          query.where(foo: 'bar').where("this.value = 10")
-        end
-
-        it 'combines conditions' do
-          expect(selection.selector).to eq(
-            'foo' => 'bar', "$where" => "this.value = 10",
-          )
-        end
-      end
-    end
-
-    context "when provided a single criterion" do
-
-      context "when the value needs no evolution" do
-
-        let(:selection) do
-          query.where(name: "Syd")
-        end
-
-        it "adds the criterion to the selection" do
-          expect(selection.selector).to eq({ "name" => "Syd" })
-        end
-      end
-
-      context "when the value must be evolved" do
-
-        before(:all) do
-          class Document
-            def id
-              13
-            end
-            def self.evolve(object)
-              object.id
-            end
-          end
-        end
-
-        after(:all) do
-          Object.send(:remove_const, :Document)
-        end
-
-        context "when the key needs evolution" do
-
-          let(:query) do
-            Mongoid::Query.new({ "user" => "user_id" })
-          end
-
-          let(:document) do
-            Document.new
-          end
-
-          let(:selection) do
-            query.where(user: document)
-          end
-
-          it "alters the key and value" do
-            expect(selection.selector).to eq({ "user_id" => document.id })
-          end
-        end
-
-        context 'when the field is a String and the value is a BSON::Regexp::Raw' do
-
-          let(:raw_regexp) do
-            BSON::Regexp::Raw.new('^Em')
-          end
-
-          let(:selection) do
-            Login.where(_id: raw_regexp)
-          end
-
-          it 'does not convert the bson raw regexp object to a String' do
-            expect(selection.selector).to eq({ "_id" => raw_regexp })
-          end
-        end
-      end
-    end
-
-    context "when provided complex criterion" do
-
-      context "when performing an $all" do
-
-        context "when performing a single query" do
-
-          let(:selection) do
-            query.where(:field.all => [ 1, 2 ])
-          end
-
-          it "adds the $all criterion" do
-            expect(selection.selector).to eq({ "field" => { "$all" => [ 1, 2 ] }})
-          end
-
-          it "returns a cloned query" do
-            expect(selection).to_not eq(query)
-          end
-        end
-      end
-
-      context "when performing an $elemMatch" do
-
-        context "when the value is not complex" do
-
-          let(:selection) do
-            query.where(:field.elem_match => { key: 1 })
-          end
-
-          it "adds the $elemMatch criterion" do
-            expect(selection.selector).to eq(
-              { "field" => { "$elemMatch" => { key: 1 } }}
-            )
-          end
-
-          it "returns a cloned query" do
-            expect(selection).to_not eq(query)
-          end
-        end
-
-        context "when the value is complex" do
-
-          let(:selection) do
-            query.where(:field.elem_match => { :key.gt => 1 })
-          end
-
-          it "adds the $elemMatch criterion" do
-            expect(selection.selector).to eq(
-              { "field" => { "$elemMatch" => { "key" => { "$gt" => 1 }}}}
-            )
-          end
-
-          it "returns a cloned query" do
-            expect(selection).to_not eq(query)
-          end
-        end
-      end
-
-      context "when performing an $exists" do
-
-        context "when providing boolean values" do
-
-          let(:selection) do
-            query.where(:field.exists => true)
-          end
-
-          it "adds the $exists criterion" do
-            expect(selection.selector).to eq(
-              { "field" => { "$exists" => true }}
-            )
-          end
-
-          it "returns a cloned query" do
-            expect(selection).to_not eq(query)
-          end
-        end
-
-        context "when providing string values" do
-
-          let(:selection) do
-            query.where(:field.exists => "t")
-          end
-
-          it "adds the $exists criterion" do
-            expect(selection.selector).to eq(
-              { "field" => { "$exists" => true }}
-            )
-          end
-
-          it "returns a cloned query" do
-            expect(selection).to_not eq(query)
-          end
-        end
-      end
-
-      context "when performing a $gt" do
-
-        let(:selection) do
-          query.where(:field.gt => 10)
-        end
-
-        it "adds the $gt criterion" do
-          expect(selection.selector).to eq(
-            { "field" => { "$gt" => 10 }}
-          )
-        end
-
-        it "returns a cloned query" do
-          expect(selection).to_not eq(query)
-        end
-      end
-
-      context "when performing a $gte" do
-
-        let(:selection) do
-          query.where(:field.gte => 10)
-        end
-
-        it "adds the $gte criterion" do
-          expect(selection.selector).to eq(
-            { "field" => { "$gte" => 10 }}
-          )
-        end
-
-        it "returns a cloned query" do
-          expect(selection).to_not eq(query)
-        end
-      end
-
-      context "when performing an $in" do
-
-        let(:selection) do
-          query.where(:field.in => [ 1, 2 ])
-        end
-
-        it "adds the $in criterion" do
-          expect(selection.selector).to eq({ "field" => { "$in" => [ 1, 2 ] }})
-        end
-
-        it "returns a cloned query" do
-          expect(selection).to_not eq(query)
-        end
-      end
-
-      context "when performing a $lt" do
-
-        let(:selection) do
-          query.where(:field.lt => 10)
-        end
-
-        it "adds the $lt criterion" do
-          expect(selection.selector).to eq(
-            { "field" => { "$lt" => 10 }}
-          )
-        end
-
-        it "returns a cloned query" do
-          expect(selection).to_not eq(query)
-        end
-      end
-
-      context "when performing a $lte" do
-
-        let(:selection) do
-          query.where(:field.lte => 10)
-        end
-
-        it "adds the $lte criterion" do
-          expect(selection.selector).to eq(
-            { "field" => { "$lte" => 10 }}
-          )
-        end
-
-        it "returns a cloned query" do
-          expect(selection).to_not eq(query)
-        end
-      end
-
-      context "when performing a $mod" do
-
-        let(:selection) do
-          query.where(:field.mod => [ 10, 1 ])
-        end
-
-        it "adds the $lte criterion" do
-          expect(selection.selector).to eq(
-            { "field" => { "$mod" => [ 10, 1 ]}}
-          )
-        end
-
-        it "returns a cloned query" do
-          expect(selection).to_not eq(query)
-        end
-      end
-
-      context "when performing a $ne" do
-
-        let(:selection) do
-          query.where(:field.ne => 10)
-        end
-
-        it "adds the $ne criterion" do
-          expect(selection.selector).to eq(
-            { "field" => { "$ne" => 10 }}
-          )
-        end
-
-        it "returns a cloned query" do
-          expect(selection).to_not eq(query)
-        end
-      end
-
-      context "when performing a $near" do
-
-        let(:selection) do
-          query.where(:field.near => [ 1, 1 ])
-        end
-
-        it "adds the $near criterion" do
-          expect(selection.selector).to eq(
-            { "field" => { "$near" => [ 1, 1 ] }}
-          )
-        end
-
-        it "returns a cloned query" do
-          expect(selection).to_not eq(query)
-        end
-      end
-
-      context "when performing a $nearSphere" do
-
-        let(:selection) do
-          query.where(:field.near_sphere => [ 1, 1 ])
-        end
-
-        it "adds the $nearSphere criterion" do
-          expect(selection.selector).to eq(
-            { "field" => { "$nearSphere" => [ 1, 1 ] }}
-          )
-        end
-
-        it "returns a cloned query" do
-          expect(selection).to_not eq(query)
-        end
-      end
-
-      context "when performing a $nin" do
-
-        let(:selection) do
-          query.where(:field.nin => [ 1, 2 ])
-        end
-
-        it "adds the $nin criterion" do
-          expect(selection.selector).to eq({ "field" => { "$nin" => [ 1, 2 ] }})
-        end
-
-        it "returns a cloned query" do
-          expect(selection).to_not eq(query)
-        end
-      end
-
-      context "when performing a $not" do
-
-        let(:selection) do
-          query.where(:field.not => /test/)
-        end
-
-        it "adds the $not criterion" do
-          expect(selection.selector).to eq({ "field" => { "$not" => /test/ }})
-        end
-
-        it "returns a cloned query" do
-          expect(selection).to_not eq(query)
-        end
-      end
-
-      context "when performing a $size" do
-
-        context "when providing an integer" do
-
-          let(:selection) do
-            query.where(:field.with_size => 10)
-          end
-
-          it "adds the $size criterion" do
-            expect(selection.selector).to eq(
-              { "field" => { "$size" => 10 }}
-            )
-          end
-
-          it "returns a cloned query" do
-            expect(selection).to_not eq(query)
-          end
-        end
-
-        context "when providing a string" do
-
-          let(:selection) do
-            query.where(:field.with_size => "10")
-          end
-
-          it "adds the $size criterion" do
-            expect(selection.selector).to eq(
-              { "field" => { "$size" => 10 }}
-            )
-          end
-
-          it "returns a cloned query" do
-            expect(selection).to_not eq(query)
-          end
-        end
-      end
-
-      context "when performing a $type" do
-
-        let(:selection) do
-          query.where(:field.with_type => 10)
-        end
-
-        it "adds the $type criterion" do
-          expect(selection.selector).to eq(
-            { "field" => { "$type" => 10 }}
-          )
-        end
-
-        it "returns a cloned query" do
-          expect(selection).to_not eq(query)
-        end
-      end
-    end
-
-    context 'when using an MQL logical operator manually' do
-      let(:base_query) do
-        query.where(test: 1)
-      end
-
-      let(:selection) do
-        base_query.where(mql_operator => ['hello' => 'world'])
-      end
-
-      shared_examples_for 'adds conditions to existing query' do
-        it 'adds conditions to existing query' do
-          selection.selector.should == {
-            'test' => 1,
-            mql_operator => ['hello' => 'world'],
-          }
-        end
-      end
-
-      shared_examples_for 'adds conditions to existing query with an extra $and' do
-        it 'adds conditions to existing query' do
-          selection.selector.should == {
-            'test' => 1,
-            mql_operator => ['hello' => 'world'],
-          }
-        end
-      end
-
-      context '$or' do
-        let(:mql_operator) { '$or' }
-
-        it_behaves_like 'adds conditions to existing query with an extra $and'
-      end
-
-      context '$nor' do
-        let(:mql_operator) { '$nor' }
-
-        it_behaves_like 'adds conditions to existing query with an extra $and'
-      end
-
-      context '$not' do
-        let(:mql_operator) { '$not' }
-
-        it_behaves_like 'adds conditions to existing query'
-      end
-
-      context '$and' do
-        let(:mql_operator) { '$and' }
-
-        it_behaves_like 'adds conditions to existing query'
-      end
-    end
-  end
-
   describe Symbol do
 
     describe "#all" do
@@ -2514,7 +1947,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.all
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2533,7 +1966,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.elem_match
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2552,7 +1985,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.exists
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2571,7 +2004,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.gt
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2590,7 +2023,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.gte
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2609,7 +2042,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.in
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2628,7 +2061,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.lt
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2647,7 +2080,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.lte
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2666,7 +2099,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.mod
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2685,7 +2118,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.ne
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2704,7 +2137,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.near
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2723,7 +2156,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.near_sphere
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2742,7 +2175,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.nin
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2781,7 +2214,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.with_size
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2800,7 +2233,7 @@ describe Mongoid::Criteria::Queryable::Selectable do
         :field.with_type
       end
 
-      it "returns a selecton key" do
+      it "returns a selection key" do
         expect(key).to be_a(Mongoid::Criteria::Queryable::Key)
       end
 
@@ -2859,4 +2292,37 @@ describe Mongoid::Criteria::Queryable::Selectable do
       end
     end
   end
+
+  describe "chained operators" do
+    [ :eq, :elem_match, :gt, :gte, :lt, :lte, :mod, :ne, :near, :near_sphere ].each do |meth|
+
+      context "when chaining the #{meth} method when using the same field" do
+        let(:op) do
+          {
+            eq: "$eq",
+            elem_match: "$elemMatch",
+            gt: "$gt",
+            gte: "$gte",
+            lt: "$lt",
+            lte: "$lte",
+            mod: "$mod",
+            ne: "$ne",
+            near: "$near",
+            near_sphere: "$nearSphere"
+          }[meth]
+        end
+
+        let(:criteria) do
+          Band.send(meth, {views: 1}).send(meth, {views:2})
+        end
+
+        it "is and-ed with the previous operators" do
+          expect(criteria.selector).to eq({
+            "views" => { op => 1 },
+            "$and" => [{ "views" => { op => 2 } }]
+          })
+        end
+      end
+    end
+  end
 end