mongoid 7.0.13 → 8.0.8

data/spec/mongoid/interceptable_spec_models.rb

@@ -0,0 +1,307 @@
+module InterceptableSpec
+  class CallbackRegistry
+    def initialize
+      @calls = []
+    end
+
+    def record_call(cls, cb)
+      @calls << [cls, cb]
+    end
+
+    attr_reader :calls
+  end
+
+  module CallbackTracking
+    extend ActiveSupport::Concern
+
+    included do
+      %i(
+        validation save create update
+      ).each do |what|
+        %i(before after).each do |whn|
+          send("#{whn}_#{what}", "#{whn}_#{what}_stub".to_sym)
+          define_method("#{whn}_#{what}_stub") do
+            callback_registry&.record_call(self.class, "#{whn}_#{what}".to_sym)
+          end
+        end
+        unless what == :validation
+          send("around_#{what}", "around_#{what}_stub".to_sym)
+          define_method("around_#{what}_stub") do |&block|
+            callback_registry&.record_call(self.class, "around_#{what}_open".to_sym)
+            block.call
+            callback_registry&.record_call(self.class, "around_#{what}_close".to_sym)
+          end
+        end
+      end
+    end
+  end
+
+  class CbHasOneParent
+    include Mongoid::Document
+
+    has_one :child, autosave: true, class_name: "CbHasOneChild", inverse_of: :parent
+
+    def initialize(callback_registry)
+      @callback_registry = callback_registry
+      super()
+    end
+
+    attr_accessor :callback_registry
+
+    def insert_as_root
+      @callback_registry&.record_call(self.class, :insert_into_database)
+      super
+    end
+
+    include CallbackTracking
+  end
+
+  class CbHasOneChild
+    include Mongoid::Document
+
+    belongs_to :parent, class_name: "CbHasOneParent", inverse_of: :child
+
+    def initialize(callback_registry)
+      @callback_registry = callback_registry
+      super()
+    end
+
+    attr_accessor :callback_registry
+
+    include CallbackTracking
+  end
+
+  class CbHasManyParent
+    include Mongoid::Document
+
+    has_many :children, autosave: true, class_name: "CbHasManyChild", inverse_of: :parent
+
+    def initialize(callback_registry)
+      @callback_registry = callback_registry
+      super()
+    end
+
+    attr_accessor :callback_registry
+
+    def insert_as_root
+      @callback_registry&.record_call(self.class, :insert_into_database)
+      super
+    end
+
+    include CallbackTracking
+  end
+
+  class CbHasManyChild
+    include Mongoid::Document
+
+    belongs_to :parent, class_name: "CbHasManyParent", inverse_of: :children
+
+    def initialize(callback_registry)
+      @callback_registry = callback_registry
+      super()
+    end
+
+    attr_accessor :callback_registry
+
+    include CallbackTracking
+  end
+
+  class CbEmbedsOneParent
+    include Mongoid::Document
+
+    field :name
+
+    embeds_one :child, cascade_callbacks: true, class_name: "CbEmbedsOneChild", inverse_of: :parent
+
+    def initialize(callback_registry)
+      @callback_registry = callback_registry
+      super()
+    end
+
+    attr_accessor :callback_registry
+
+    def insert_as_root
+      @callback_registry&.record_call(self.class, :insert_into_database)
+      super
+    end
+
+    include CallbackTracking
+  end
+
+  class CbEmbedsOneChild
+    include Mongoid::Document
+
+    field :age
+
+    embedded_in :parent, class_name: "CbEmbedsOneParent", inverse_of: :child
+
+    def initialize(callback_registry)
+      @callback_registry = callback_registry
+      super()
+    end
+
+    attr_accessor :callback_registry
+
+    include CallbackTracking
+  end
+
+  class CbEmbedsManyParent
+    include Mongoid::Document
+
+    embeds_many :children, cascade_callbacks: true, class_name: "CbEmbedsManyChild", inverse_of: :parent
+
+    def initialize(callback_registry)
+      @callback_registry = callback_registry
+      super()
+    end
+
+    attr_accessor :callback_registry
+
+    def insert_as_root
+      @callback_registry&.record_call(self.class, :insert_into_database)
+      super
+    end
+
+    include CallbackTracking
+  end
+
+  class CbEmbedsManyChild
+    include Mongoid::Document
+
+    embedded_in :parent, class_name: "CbEmbedsManyParent", inverse_of: :children
+
+    def initialize(callback_registry)
+      @callback_registry = callback_registry
+      super()
+    end
+
+    attr_accessor :callback_registry
+
+    include CallbackTracking
+  end
+
+  class CbParent
+    include Mongoid::Document
+
+    def initialize(callback_registry)
+      @callback_registry = callback_registry
+      super()
+    end
+
+    attr_accessor :callback_registry
+
+    embeds_many :cb_children
+    embeds_many :cb_cascaded_children, cascade_callbacks: true
+
+    include CallbackTracking
+  end
+
+  class CbChild
+    include Mongoid::Document
+
+    embedded_in :cb_parent
+
+    def initialize(callback_registry, options)
+      @callback_registry = callback_registry
+      super(options)
+    end
+
+    attr_accessor :callback_registry
+
+    include CallbackTracking
+  end
+
+  class CbCascadedChild
+    include Mongoid::Document
+
+    embedded_in :cb_parent
+
+    def initialize(callback_registry, options)
+      @callback_registry = callback_registry
+      super(options)
+    end
+
+    attr_accessor :callback_registry
+
+    include CallbackTracking
+  end
+end
+
+class InterceptableBand
+  include Mongoid::Document
+
+  has_many :songs, class_name: "InterceptableSong"
+  field :name
+end
+
+class InterceptableSong
+  include Mongoid::Document
+
+  belongs_to :band, class_name: "InterceptableBand"
+  field :band_name, default: -> { band.name }
+  field :name
+end
+
+class InterceptablePlane
+  include Mongoid::Document
+
+  has_many :wings, class_name: "InterceptableWing"
+end
+
+class InterceptableWing
+  include Mongoid::Document
+
+  belongs_to :plane, class_name: "InterceptablePlane"
+  has_one :engine, autobuild: true, class_name: "InterceptableEngine"
+
+  field :_id, type: String, default: -> { 'hello-wing' }
+
+  field :p_id, type: String, default: -> { plane&.id }
+  field :e_id, type: String, default: -> { engine&.id }
+end
+
+class InterceptableEngine
+  include Mongoid::Document
+
+  belongs_to :wing, class_name: "InterceptableWing"
+
+  field :_id, type: String, default: -> { "hello-engine-#{wing&.id}" }
+end
+
+class InterceptableCompany
+  include Mongoid::Document
+
+  has_many :users, class_name: "InterceptableUser"
+  has_many :shops, class_name: "InterceptableShop"
+end
+
+class InterceptableShop
+  include Mongoid::Document
+
+  embeds_one :address, class_name: "InterceptableAddress"
+  belongs_to :company, class_name: "InterceptableCompany"
+
+  after_initialize :build_address1
+
+  def build_address1
+    self.address ||= Address.new
+  end
+end
+
+class InterceptableAddress
+  include Mongoid::Document
+  embedded_in :shop, class_name: "InterceptableShop"
+end
+
+class InterceptableUser
+  include Mongoid::Document
+
+  belongs_to :company, class_name: "InterceptableCompany"
+
+  validate :break_mongoid
+
+  def break_mongoid
+    company.shop_ids
+  end
+end
+

data/spec/mongoid/loggable_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Mongoid::Loggable do

data/spec/mongoid/matcher/extract_attribute_data/numeric_keys.yml

@@ -0,0 +1,104 @@
+- name: string key - matches
+  document:
+    # Document field names in MongoDB are always strings.
+    '42': foo
+  # The paths are also always strings.
+  key: '42'
+  
+  result:
+    - foo
+
+- name: string key - does not match
+  document:
+    # Document field names in MongoDB are always strings.
+    '42': foo
+  # The paths are also always strings.
+  key: '4'
+  
+  result: []
+
+- name: array value - matches
+  document:
+    '42':
+      - foo
+  key: '42'
+  
+  result:
+    - ['foo']
+
+- name: subfield of numeric key - matches
+  document:
+    '42':
+      foo: bar
+  key: 42.foo
+  
+  result: 
+    - bar
+
+- name: numeric key under array
+  document:
+    a:
+      -
+        '42': bar
+
+  key: a.42
+
+  result: 
+    - bar
+
+- name: multiple values under numeric key under array
+  document: &numeric-key-under-array
+    a:
+      -
+        '42': bar
+      -
+        '42': foo
+
+  key: a.42
+
+  result: 
+    - bar
+    - foo
+
+- name: array indexing with values under numeric key under array
+  document: *numeric-key-under-array
+
+  key: a.1
+
+  result: 
+    - 
+      '42': foo
+
+- name: multiple values under array under numeric key
+  document:
+    a:
+      '42':
+        -
+          foo: bar
+        -       
+          foo: baq
+
+  key: a.42.foo
+
+  result: 
+    - bar
+    - baq
+
+- name: path component interpretable as both array index and hash key
+  document: 
+    a:
+      -
+        '1': bar
+      -
+        '1': foo
+
+  key: a.1
+
+  result: 
+    # The order of these results isn't guaranteed:
+    # 1 interpreted as the array index:
+    -
+      '1': foo
+    # 1 interpreted as hash key:
+    - bar
+    - foo

data/spec/mongoid/matcher/extract_attribute_data/traversal.yml

@@ -0,0 +1,239 @@
+- name: scalar retrieval
+  document:
+    name: foo
+  key: name
+  
+  result: 
+    - foo
+
+- name: scalar leaf
+  document: &foo-bar-foo-bar-2
+    foo:
+      bar:
+        foo:
+          bar:
+            2
+  key: foo.bar.foo.bar
+  
+  result:
+    - 2
+
+- name: scalar non-leaf
+  document: *foo-bar-foo-bar-2
+  key: foo.bar
+  
+  result:
+    -
+      foo:
+        bar:
+          2
+
+- name: null leaf
+  document:
+    name:
+  key: name
+  
+  result: 
+    - ~
+
+- name: array leaf
+  document: &foo-bar-foo-bar-array-2
+    foo:
+      bar:
+        foo:
+          bar:
+            - 2
+  key: foo.bar.foo.bar
+  
+  result:
+    - [2]
+
+- name: scalar non-leaf
+  document: *foo-bar-foo-bar-array-2
+  key: foo.bar
+  
+  result:
+    -
+      foo:
+        bar:
+          - 2
+
+- name: array non-leaf
+  document: &foo-bar-array-foo-bar-2
+    foo:
+      -
+        bar:
+          foo:
+            bar:
+              2
+  key: foo.bar.foo.bar
+  
+  result:
+    - 2
+
+- name: array non-leaf with non-leaf query
+  document: *foo-bar-array-foo-bar-2
+  key: foo.bar
+  
+  result:
+    -
+      foo:
+        bar:
+          2
+
+- name: array non-leaf with query for missing non-leaf key
+  document: *foo-bar-array-foo-bar-2
+  key: foo.missing
+  
+  result: []
+
+- name: path past end of document
+  document: &foo-hello-world
+    foo:
+      hello: world
+  key: foo.bar.foo.bar
+  
+  result: []
+
+- name: query for missing leaf key
+  document: *foo-hello-world
+  key: foo.bar
+  
+  result: []
+
+- name: numeric path component when traversing hash
+  document:
+    foo:
+      "10":
+        42
+  key: "foo.10"
+
+  result: 
+    - 42
+
+- name: array index - hash leaf
+  document: &foo-one-two
+    foo:
+      - one: 1
+      - two: 2
+    bar: [3, 4]
+  key: foo.1.two
+  
+  result: 
+    - 2
+
+- name: array index - hash non-leaf
+  document: *foo-one-two
+  key: foo.1
+  
+  result:
+    -
+      two: 2
+
+- name: array index - scalar leaf
+  document: *foo-one-two
+  key: bar.1
+  
+  result: 
+    - 4
+
+- name: array index - scalar leaf - missing
+  document: *foo-one-two
+  key: bar.2
+  
+  result: []
+
+- name: array index - scalar leaf - missing with further hash subscription
+  document: *foo-one-two
+  key: bar.2.any
+  
+  result: []
+
+- name: non-sequential nested arrays - hash leaf - one value
+  document: &books-authors
+    books:
+      -
+        authors:
+          - name: Steve
+      -
+        authors:
+          - name: Boris
+          - name: Pasha
+  key: books.0.authors.name
+  
+  result:
+    - Steve
+
+- name: non-sequential nested arrays - hash non-leaf - one value
+  document: *books-authors
+  key: books.0
+  
+  result:
+    -
+      authors:
+        - name: Steve
+
+- name: non-sequential nested arrays - hash non-leaf - multiple values
+  document: *books-authors
+  key: books.1
+  
+  result:
+    -
+      authors:
+        - name: Boris
+        - name: Pasha
+
+- name: sequential nested arrays - numerically indexed leaf
+  document: &groups
+    groups:
+      - [1]
+  key: groups.0.0
+  
+  result: 
+    - 1
+
+- name: sequential nested arrays - numerically indexed leaf - missing
+  document: *groups
+  key: groups.0.1
+  
+  result: []
+
+- name: sequential nested arrays - missing hash path through non-leaf
+  document: *groups
+  key: groups.missing
+  
+  result: []
+
+- name: sequential nested arrays - missing hash path through leaf
+  document: *groups
+  key: groups.0.missing
+  
+  result: []
+
+- name: sequential nested arrays - missing hash path past leaf
+  document: *groups
+  key: groups.0.0.missing
+  
+  result: []
+
+- name: mix of arrays and hashes in an array
+  document:
+    foo:
+      -
+        -
+          bar: 1
+      - bar: 2
+  key: foo.bar
+  
+  result: 
+    - 2
+
+- name: numeric index through a primitive value not supporting subscription operator
+  document:
+    occupants:
+      -
+        age: 20
+  key:
+    occupants.0.age.0
+    
+  result: []

data/spec/mongoid/matcher/extract_attribute_spec.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe 'Matcher.extract_attribute' do
+  Dir[File.join(File.dirname(__FILE__), 'extract_attribute_data', '*.yml')].sort.each do |path|
+    context File.basename(path) do
+      specs = YAML.safe_load(File.read(path), aliases: true)
+
+      specs.each do |spec|
+        context spec['name'] do
+
+          if spec['pending']
+            pending spec['pending'].to_s
+          end
+
+          let(:document) do
+            spec['document']
+          end
+
+          let(:key) { spec['key'] }
+
+          let(:actual) do
+            Mongoid::Matcher.extract_attribute(document, key)
+          end
+
+          let(:expected) { spec.fetch('result') }
+
+          it 'has the expected result' do
+            actual.should == expected
+          end
+        end
+      end
+    end
+  end
+end