mongoid 7.1.5 → 7.2.1

data/spec/integration/matcher_operator_data/regex.yml

@@ -0,0 +1,174 @@
+- name: string - matches
+  document:
+    title: King
+  query:
+    title:
+      $regex: K..g
+  matches: true
+
+- name: string - does not match
+  document:
+    title: Queen
+  query:
+    title:
+      $regex: K..g
+  matches: false
+
+- name: native regexp - matches
+  document:
+    title: King
+  query:
+    title:
+      $regex: !ruby/regexp /K..g/
+  matches: true
+
+- name: native regexp - does not match
+  document:
+    title: Queen
+  query:
+    title:
+      $regex: !ruby/regexp /K..g/
+  matches: false
+
+- name: bson regexp - matches
+  document:
+    title: King
+  query:
+    title:
+      $regex: !ruby/object:BSON::Regexp::Raw
+        pattern: K..g
+        options: ''
+  matches: true
+
+- name: bson regexp - does not match
+  document:
+    title: Queen
+  query:
+    title:
+      $regex: !ruby/object:BSON::Regexp::Raw
+        pattern: K..g
+        options: ''
+  matches: false
+
+- name: string argument - matches
+  document:
+    title: King
+  query:
+    title:
+      $regex:
+        K..g
+  matches: true
+
+- name: string argument - does not match
+  document:
+    title: King
+  query:
+    title:
+      $regex:
+        K...g
+  matches: false
+
+- name: nil argument
+  document:
+    position: ceo
+  query:
+    position:
+      $regex: ~
+  error: true
+
+- name: empty array argument
+  document:
+    position: ceo
+  query:
+    position:
+      $regex: []
+  error: true
+
+- name: empty hash argument
+  document:
+    position: ceo
+  query:
+    position:
+      $regex: {}
+  error: true
+
+- name: numeric argument
+  document:
+    position: ceo
+  query:
+    position:
+      $regex: 42
+  error: true
+
+- name: numeric field value - matches if stringified
+  document:
+    title: 42
+  query:
+    title:
+      $regex:
+        '42'
+  matches: false
+
+- name: array field value - matches an element
+  document:
+    title:
+      - foo
+  query:
+    title:
+      $regex: foo
+  matches: true
+
+- name: array field value - does not match an element
+  document:
+    title:
+      - foo
+  query:
+    title:
+      $regex: bar
+  matches: false
+
+- name: empty array field value
+  document:
+    title: []
+  query:
+    title:
+      $regex: bar
+  matches: false
+
+- name: true field value
+  document:
+    title: true
+  query:
+    title:
+      $regex:
+        'true'
+  matches: false
+
+- name: false field value
+  document:
+    title: false
+  query:
+    title:
+      $regex:
+        'false'
+  matches: false
+
+- name: nil field value
+  document:
+    title: ~
+  query:
+    title:
+      $regex:
+        ''
+  matches: false
+
+- name: range
+  document:
+    name: 42
+  query:
+    name:
+      $regex: !ruby/range
+        begin: 1
+        end: 2
+        excl: false
+  error: true

data/spec/integration/matcher_operator_data/regex_options.yml

@@ -0,0 +1,72 @@
+- name: string with i option - matches
+  document:
+    title: King
+  query:
+    title:
+      $regex: k..g
+      $options: i
+  matches: true
+
+- name: string with i option - does not match
+  document:
+    title: King
+  query:
+    title:
+      $regex: kkkg
+      $options: i
+  matches: false
+
+- name: string with m option - matches
+  document:
+    title: "King\nSir"
+  query:
+    title:
+      $regex: ^Sir
+      $options: m
+  matches: true
+
+- name: string with m option - does not match
+  document:
+    title: "King\nSir"
+  query:
+    title:
+      $regex: ^ir
+      $options: m
+  matches: false
+
+- name: options given without regex
+  document:
+    title: King
+  query:
+    title:
+      $options: i
+  error: true
+
+- name: options are given first
+  document:
+    title: King
+  query:
+    title:
+      $options: i
+      $regex: k..g
+  matches: true
+
+- name: regex with i option - matches
+  document:
+    title: King
+  query:
+    title:
+      $regex: !ruby/regexp /k..g/
+      $options: i
+  matches: true
+  pending: https://jira.mongodb.org/browse/MONGOID-4936
+
+- name: regex with i option - does not match
+  document:
+    title: King
+  query:
+    title:
+      $regex: !ruby/regexp /kkkg/
+      $options: i
+  matches: false
+  pending: https://jira.mongodb.org/browse/MONGOID-4936

data/spec/integration/matcher_operator_data/size.yml

@@ -0,0 +1,174 @@
+- name: correct size
+  document:
+    products: [1, 2, 3]
+  query:
+    products:
+      $size: 3
+  matches: true
+
+- name: incorrect size
+  document:
+    products: [1, 2, 3]
+  query:
+    products:
+      $size: 2
+  matches: false
+
+- name: empty array - correct size
+  document:
+    products: []
+  query:
+    products:
+      $size: 0
+  matches: true
+
+- name: empty array - incorrect size
+  document:
+    products: []
+  query:
+    products:
+      $size: 1
+  matches: false
+
+- name: condition is an array
+  document:
+    products: [1, 2, 3]
+  query:
+    products:
+      - $size: 3
+  matches: false
+
+- name: field value is a scalar
+  document:
+    hello: world
+  query:
+    hello:
+      $size: 5
+  matches: false
+
+- name: field value is a hash
+  document:
+    hello:
+      foo: bar
+  query:
+    hello:
+      $size: 1
+  matches: false
+
+- name: nil argument
+  document:
+    position: ceo
+  query:
+    position:
+      $size: ~
+  error: true
+
+- name: empty array argument
+  document:
+    position: ceo
+  query:
+    position:
+      $size: []
+  error: true
+
+- name: empty hash argument
+  document:
+    position: ceo
+  query:
+    position:
+      $size: {}
+  error: true
+
+- name: string argument
+  document:
+    position: ceo
+  query:
+    position:
+      $size: foo
+  error: true
+  min_server_version: 3.4
+
+- name: floating point argument
+  document:
+    position: ceo
+  query:
+    position:
+      $size: 2.5
+  error: true
+  min_server_version: 3.4
+
+- name: negative integer argument
+  document:
+    position: ceo
+  query:
+    position:
+      $size: -2
+  error: true
+  min_server_version: 3.4
+
+- name: empty array field value
+  document:
+    position: []
+  query:
+    position:
+      $size: 1
+  matches: false
+
+- name: empty hash field value
+  document:
+    position: {}
+  query:
+    position:
+      $size: 1
+  matches: false
+
+- name: numeric field value
+  document:
+    position: 42
+  query:
+    position:
+      $size: 1
+  matches: false
+
+- name: string field value
+  document:
+    position: foo
+  query:
+    position:
+      $size: 1
+  matches: false
+
+- name: true field value
+  document:
+    position: true
+  query:
+    position:
+      $size: 0
+  matches: false
+
+- name: false field value
+  document:
+    position: false
+  query:
+    position:
+      $size: 0
+  matches: false
+
+- name: null field value
+  document:
+    position: ~
+  query:
+    position:
+      $size: 0
+  matches: false
+
+- name: range
+  document:
+    name: 42
+  query:
+    name:
+      $size: !ruby/range
+        begin: 1
+        end: 2
+        excl: false
+  error: true

data/spec/integration/matcher_operator_spec.rb

@@ -0,0 +1,100 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
+require 'spec_helper'
+
+def mop_error?(spec, kind)
+  unless %w(matcher driver dsl).include?(kind)
+    raise ArgumentError, "Bogus kind: #{kind}"
+  end
+
+  spec['error'] == true || spec['error'] == kind ||
+    spec['error'].is_a?(Array) && spec['error'].include?(kind)
+end
+
+describe 'Matcher operators' do
+  Dir[File.join(File.dirname(__FILE__), 'matcher_operator_data', '*.yml')].sort.each do |path|
+    context File.basename(path) do
+      specs = YAML.load(File.read(path))
+
+      specs.each do |spec|
+        context spec['name'] do
+
+          if spec['pending']
+            before do
+              # Cannot use `pending` here because some of the queries may work
+              # as specified (e.g. when Mongoid and server behavior differ).
+              skip spec['pending'].to_s
+            end
+          end
+
+          if spec['min_server_version']
+            min_server_version spec['min_server_version'].to_s
+          end
+
+          let(:query) { spec.fetch('query') }
+          let(:result) { spec.fetch('matches') }
+
+          context 'embedded matching' do
+            let(:document) { Mop.new(spec.fetch('document')) }
+
+            if mop_error?(spec, 'matcher')
+              it 'produces an error' do
+                lambda do
+                  document._matches?(query)
+                end.should raise_error(Mongoid::Errors::InvalidQuery)
+              end
+            else
+              it 'produces the correct result' do
+                document._matches?(query).should be result
+              end
+            end
+          end
+
+          context 'server query' do
+            let!(:document) { Mop.create(spec.fetch('document')) }
+
+            context 'via driver' do
+              if mop_error?(spec, 'driver')
+                it 'produces an error' do
+                  begin
+                    Mop.collection.find(query).any?
+                  rescue Mongo::Error::OperationFailure
+                  rescue Mongo::Error::InvalidDocument
+                  rescue BSON::Error::UnserializableClass
+                  else
+                    fail "Expected an exception to be raised"
+                  end
+                end
+              else
+                it 'produces the correct result' do
+                  Mop.collection.find(query).any?.should be result
+                end
+              end
+            end
+
+            context 'via Mongoid DSL' do
+              if mop_error?(spec, 'dsl')
+                it 'produces an error' do
+                  begin
+                    Mop.where(query).any?
+                  rescue Mongo::Error::OperationFailure
+                  rescue BSON::Error::UnserializableClass
+                  rescue Mongoid::Errors::InvalidQuery
+                  rescue Mongoid::Errors::CriteriaArgumentRequired
+                  else
+                    fail "Expected the query to raise an error"
+                  end
+                end
+              else
+                it 'produces the correct result' do
+                  Mop.where(query).any?.should be result
+                end
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end