mongoid 8.0.7 → 8.0.9

data/spec/mongoid/validatable/associated_spec.rb

@@ -37,12 +37,18 @@ describe Mongoid::Validatable::AssociatedValidator do
           User.new(name: "test")
         end
 
-        let(:description) do
+        let(:description1) do
+          Description.new
+        end
+
+        let(:description2) do
           Description.new
         end
 
         before do
-          user.descriptions << description
+          user.descriptions << description1
+          user.descriptions << description2
+          user.valid?
         end
 
         it "only validates the parent once" do
@@ -50,12 +56,16 @@ describe Mongoid::Validatable::AssociatedValidator do
         end
 
         it "adds the errors from the relation" do
-          user.valid?
           expect(user.errors[:descriptions]).to_not be_nil
         end
 
+        it 'reports all failed validations' do
+          errors = user.descriptions.flat_map { |d| d.errors[:details] }
+          expect(errors.length).to be == 2
+        end
+
         it "only validates the child once" do
-          expect(description).to_not be_valid
+          expect(description1).to_not be_valid
         end
       end
 
@@ -75,7 +85,6 @@ describe Mongoid::Validatable::AssociatedValidator do
         end
 
         it "does not run validation on them" do
-          expect(description).to receive(:valid?).never
           expect(user).to be_valid
         end
 
@@ -84,14 +93,14 @@ describe Mongoid::Validatable::AssociatedValidator do
     end
   end
 
-  describe "#validate_each" do
+  describe "#validate" do
 
     let(:person) do
       Person.new
     end
 
     let(:validator) do
-      described_class.new(attributes: person.attributes)
+      described_class.new(attributes: person.relations.keys)
     end
 
     context "when the association is a one to one" do
@@ -99,7 +108,7 @@ describe Mongoid::Validatable::AssociatedValidator do
       context "when the association is nil" do
 
         before do
-          validator.validate_each(person, :name, nil)
+          validator.validate(person)
         end
 
         it "adds no errors" do
@@ -108,14 +117,9 @@ describe Mongoid::Validatable::AssociatedValidator do
       end
 
       context "when the association is valid" do
-
-        let(:associated) do
-          double(valid?: true, flagged_for_destroy?: false)
-        end
-
         before do
-          expect(associated).to receive(:validated?).and_return(false)
-          validator.validate_each(person, :name, associated)
+          person.name = Name.new(first_name: 'A', last_name: 'B')
+          validator.validate(person)
         end
 
         it "adds no errors" do
@@ -125,13 +129,9 @@ describe Mongoid::Validatable::AssociatedValidator do
 
       context "when the association is invalid" do
 
-        let(:associated) do
-          double(valid?: false, flagged_for_destroy?: false)
-        end
-
         before do
-          expect(associated).to receive(:validated?).and_return(false)
-          validator.validate_each(person, :name, associated)
+          person.name = Name.new(first_name: 'Jamis', last_name: 'Buck')
+          validator.validate(person)
         end
 
         it "adds errors to the parent document" do
@@ -149,7 +149,7 @@ describe Mongoid::Validatable::AssociatedValidator do
       context "when the association is empty" do
 
         before do
-          validator.validate_each(person, :addresses, [])
+          validator.validate(person)
         end
 
         it "adds no errors" do
@@ -159,13 +159,9 @@ describe Mongoid::Validatable::AssociatedValidator do
 
       context "when the association has invalid documents" do
 
-        let(:associated) do
-          double(valid?: false, flagged_for_destroy?: false)
-        end
-
         before do
-          expect(associated).to receive(:validated?).and_return(false)
-          validator.validate_each(person, :addresses, [ associated ])
+          person.addresses << Address.new(street: '123')
+          validator.validate(person)
         end
 
         it "adds errors to the parent document" do
@@ -175,13 +171,10 @@ describe Mongoid::Validatable::AssociatedValidator do
 
       context "when the association has all valid documents" do
 
-        let(:associated) do
-          double(valid?: true, flagged_for_destroy?: false)
-        end
-
         before do
-          expect(associated).to receive(:validated?).and_return(false)
-          validator.validate_each(person, :addresses, [ associated ])
+          person.addresses << Address.new(street: '123 First St')
+          person.addresses << Address.new(street: '456 Second St')
+          validator.validate(person)
         end
 
         it "adds no errors" do

data/spec/support/models/name.rb

@@ -4,6 +4,8 @@ class Name
   include Mongoid::Document
   include Mongoid::Attributes::Dynamic
 
+  validate :is_not_jamis
+
   field :_id, type: String, overwrite: true, default: ->{
     "#{first_name}-#{last_name}"
   }
@@ -23,4 +25,12 @@ class Name
   def set_parent=(set = false)
     self.parent_title = namable.title if set
   end
+
+  private
+
+  def is_not_jamis
+    if first_name == 'Jamis' && last_name == 'Buck'
+      errors.add(:base, :invalid)
+    end
+  end
 end

metadata

@@ -1,41 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: mongoid
 version: !ruby/object:Gem::Version
-  version: 8.0.7
+  version: 8.0.9
 platform: ruby
 authors:
 - The MongoDB Ruby Team
-autorequire: 
 bindir: bin
-cert_chain:
-- |
-  -----BEGIN CERTIFICATE-----
-  MIIEeDCCAuCgAwIBAgIBATANBgkqhkiG9w0BAQsFADBBMREwDwYDVQQDDAhkYngt
-  cnVieTEXMBUGCgmSJomT8ixkARkWB21vbmdvZGIxEzARBgoJkiaJk/IsZAEZFgNj
-  b20wHhcNMjMwMTMxMTE1NjM1WhcNMjQwMTMxMTE1NjM1WjBBMREwDwYDVQQDDAhk
-  YngtcnVieTEXMBUGCgmSJomT8ixkARkWB21vbmdvZGIxEzARBgoJkiaJk/IsZAEZ
-  FgNjb20wggGiMA0GCSqGSIb3DQEBAQUAA4IBjwAwggGKAoIBgQC0/Veq9l47cTfX
-  tQ+kHq2NOCwJuJGt1iXWQ/vH/yp7pZ/bLej7gPDl2CfIngAXRjM7r1FkR9ya7VAm
-  IneBFcVU3HhpIXWi4ByXGjBOXFD1Dfbz4C4zedIWRk/hNzXa+rQY4KPwpOwG/hZg
-  id+rSXWSbNlkyN97XfonweVh7JsIa9X/2JY9ADYjhCfEZF+b0+Wl7+jgwzLWb46I
-  0WH0bZBIZ0BbKAwUXIgvq5mQf9PzukmMVYCwnkJ/P4wrHO22HuwnbMyvJuGjVwqi
-  j1NRp/2vjmKBFWxIfhlSXEIiqAmeEVNXzhPvTVeyo+rma+7R3Bo+4WHkcnPpXJJZ
-  Jd63qXMvTB0GplEcMJPztWhrJOmcxIOVoQyigEPSQT8JpzFVXby4SGioizv2eT7l
-  VYSiCHuc3yEDyq5M+98WGX2etbj6esYtzI3rDevpIAHPB6HQmtoJIA4dSl3gjFb+
-  D+YQSuB2qYu021FI9zeY9sbZyWysEXBxhwrmTk+XUV0qz+OQZkMCAwEAAaN7MHkw
-  CQYDVR0TBAIwADALBgNVHQ8EBAMCBLAwHQYDVR0OBBYEFH4nnr4tYlatU57RbExW
-  jG86YM5nMB8GA1UdEQQYMBaBFGRieC1ydWJ5QG1vbmdvZGIuY29tMB8GA1UdEgQY
-  MBaBFGRieC1ydWJ5QG1vbmdvZGIuY29tMA0GCSqGSIb3DQEBCwUAA4IBgQAVSlgM
-  nFDWCCNLOCqG5/Lj4U62XoALkdCI+OZ30+WrA8qiRLSL9ZEziVK9AV7ylez+sriQ
-  m8XKZKsCN5ON4+zXw1S+6Ftz/R4zDg7nTb9Wgw8ibzsoiP6e4pRW3Fls3ZdaG4pW
-  +qMTbae9OiSrgI2bxNTII+v+1FcbQjOlMu8HPZ3ZfXnurXPgN5GxSyyclZI1QONO
-  HbUoKHRirZu0F7JCvQQq4EkSuLWPplRJfYEeJIYm05zhhFeEyqea2B/TTlCtXa42
-  84vxXsxGzumuO8F2Q9m6/p95sNhqCp0B/SkKXIrRGJ7FBzupoORNRXHviS2OC3ty
-  4lwUzOlLTF/yO0wwYYfmtQOALQwKnW838vbYthMXvTjxB0EgVZ5PKto99WbjsXzy
-  wkeAWhd5b+5JS0zgDL4SvGB8/W2IY+y0zELkojBMgJPyrpAWHL/WSsSBMuhyI2Pv
-  xxaBVLklnJJ/qCCOZ3lG2MyVc/Nb0Mmq8ygWNsfwHmKKYuuWcviit0D0Tek=
-  -----END CERTIFICATE-----
-date: 2023-10-23 00:00:00.000000000 Z
+cert_chain: []
+date: 2025-01-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -858,29 +830,6 @@ files:
 - spec/mongoid_spec.rb
 - spec/rails/controller_extension/controller_runtime_spec.rb
 - spec/rails/mongoid_spec.rb
-- spec/shared/LICENSE
-- spec/shared/bin/get-mongodb-download-url
-- spec/shared/bin/s3-copy
-- spec/shared/bin/s3-upload
-- spec/shared/lib/mrss/child_process_helper.rb
-- spec/shared/lib/mrss/cluster_config.rb
-- spec/shared/lib/mrss/constraints.rb
-- spec/shared/lib/mrss/docker_runner.rb
-- spec/shared/lib/mrss/eg_config_utils.rb
-- spec/shared/lib/mrss/event_subscriber.rb
-- spec/shared/lib/mrss/lite_constraints.rb
-- spec/shared/lib/mrss/server_version_registry.rb
-- spec/shared/lib/mrss/session_registry.rb
-- spec/shared/lib/mrss/session_registry_legacy.rb
-- spec/shared/lib/mrss/spec_organizer.rb
-- spec/shared/lib/mrss/utils.rb
-- spec/shared/share/Dockerfile.erb
-- spec/shared/share/haproxy-1.conf
-- spec/shared/share/haproxy-2.conf
-- spec/shared/shlib/config.sh
-- spec/shared/shlib/distro.sh
-- spec/shared/shlib/server.sh
-- spec/shared/shlib/set_env.sh
 - spec/spec_helper.rb
 - spec/support/authorization.rb
 - spec/support/client_registry.rb
@@ -1163,7 +1112,6 @@ metadata:
   documentation_uri: https://docs.mongodb.com/mongoid/
   homepage_uri: https://mongoid.org/
   source_code_uri: https://github.com/mongodb/mongoid
-post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -1178,8 +1126,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 1.3.6
 requirements: []
-rubygems_version: 3.4.21
-signing_key: 
+rubygems_version: 3.6.3
 specification_version: 4
 summary: Elegant Persistence in Ruby for MongoDB.
 test_files:
@@ -1573,29 +1520,6 @@ test_files:
 - spec/mongoid_spec.rb
 - spec/rails/controller_extension/controller_runtime_spec.rb
 - spec/rails/mongoid_spec.rb
-- spec/shared/LICENSE
-- spec/shared/bin/get-mongodb-download-url
-- spec/shared/bin/s3-copy
-- spec/shared/bin/s3-upload
-- spec/shared/lib/mrss/child_process_helper.rb
-- spec/shared/lib/mrss/cluster_config.rb
-- spec/shared/lib/mrss/constraints.rb
-- spec/shared/lib/mrss/docker_runner.rb
-- spec/shared/lib/mrss/eg_config_utils.rb
-- spec/shared/lib/mrss/event_subscriber.rb
-- spec/shared/lib/mrss/lite_constraints.rb
-- spec/shared/lib/mrss/server_version_registry.rb
-- spec/shared/lib/mrss/session_registry.rb
-- spec/shared/lib/mrss/session_registry_legacy.rb
-- spec/shared/lib/mrss/spec_organizer.rb
-- spec/shared/lib/mrss/utils.rb
-- spec/shared/share/Dockerfile.erb
-- spec/shared/share/haproxy-1.conf
-- spec/shared/share/haproxy-2.conf
-- spec/shared/shlib/config.sh
-- spec/shared/shlib/distro.sh
-- spec/shared/shlib/server.sh
-- spec/shared/shlib/set_env.sh
 - spec/spec_helper.rb
 - spec/support/authorization.rb
 - spec/support/client_registry.rb

data/spec/shared/LICENSE

@@ -1,20 +0,0 @@
-Copyright (c) 2020 MongoDB, Inc.
-
-Permission is hereby granted, free of charge, to any person obtaining
-a copy of this software and associated documentation files (the
-"Software"), to deal in the Software without restriction, including
-without limitation the rights to use, copy, modify, merge, publish,
-distribute, sublicense, and/or sell copies of the Software, and to
-permit persons to whom the Software is furnished to do so, subject to
-the following conditions:
-
-The above copyright notice and this permission notice shall be
-included in all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
-EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
-NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
-LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
-OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
-WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/spec/shared/bin/get-mongodb-download-url

@@ -1,17 +0,0 @@
-#!/usr/bin/env ruby
-
-desired_version, arch = ARGV
-if arch.nil?
-  STDERR.puts "Usage: get-mongodb-download-url desired-version arch"
-  exit 1
-end
-
-$: << File.join(File.dirname(__FILE__), '../lib')
-require 'mrss/server_version_registry'
-
-begin
-  puts Mrss::ServerVersionRegistry.new(desired_version, arch).download_url
-rescue Mrss::ServerVersionRegistry::Error => exc
-  STDERR.puts "Error: #{exc}"
-  exit 2
-end

data/spec/shared/bin/s3-copy

@@ -1,45 +0,0 @@
-#!/usr/bin/env ruby
-
-require 'optparse'
-require 'aws-sdk-s3'
-
-options = {}
-OptionParser.new do |opts|
-  opts.banner = "Usage: s3-copy options"
-
-  opts.on("-r", "--region=REGION", "AWS region to use (default us-east-1)") do |v|
-    options[:region] = v
-  end
-
-  opts.on("-p", "--param=KEY=VALUE", "Specify parameter for new files") do |v|
-    options[:params] ||= {}
-    k, v = v.split('=', 2)
-    options[:params][k.to_sym] = v
-  end
-
-  opts.on("-f", "--from=BUCKET:PATH", "Bucket name and key (or path) to copy from") do |v|
-    options[:from] = v
-  end
-
-  opts.on("-t", "--to=BUCKET:PATH", "Bucket name and key (or path) to write to (may be specified more than once)") do |v|
-    options[:to] ||= []
-    options[:to] << v
-  end
-end.parse!
-
-ENV['AWS_REGION'] ||= options[:region] || 'us-east-1'
-
-bucket, key = options.fetch(:from).split(':', 2)
-
-s3 = Aws::S3::Client.new
-
-options.fetch(:to).each do |dest|
-  STDERR.puts "Copying to #{dest}"
-  dbucket, dkey = dest.split(':', 2)
-  s3.copy_object(
-    bucket: dbucket,
-    key: dkey,
-    copy_source: "/#{bucket}/#{key}",
-    **options[:params] || {},
-  )
-end

data/spec/shared/bin/s3-upload

@@ -1,69 +0,0 @@
-#!/usr/bin/env ruby
-
-require 'optparse'
-require 'aws-sdk-s3'
-
-options = {}
-OptionParser.new do |opts|
-  opts.banner = "Usage: s3-upload options"
-
-  opts.on("-r", "--region=REGION", "AWS region to use (default us-east-1)") do |v|
-    options[:region] = v
-  end
-
-  opts.on("-p", "--param=KEY=VALUE", "Specify parameter for S3 upload") do |v|
-    options[:params] ||= {}
-    k, v = v.split('=', 2)
-    options[:params][k.to_sym] = v
-  end
-
-  opts.on("-f", "--file=PATH", "Path to the file to upload, - to upload standard input") do |v|
-    options[:file] = v
-  end
-
-  opts.on("-w", "--write=BUCKET:PATH", "Bucket name and key (or path) to upload to") do |v|
-    options[:write] = v
-  end
-
-  opts.on("-c", "--copy=BUCKET:PATH", "Bucket name and key (or path) to copy to (may be specified more than once)") do |v|
-    options[:copy] ||= []
-    options[:copy] << v
-  end
-end.parse!
-
-ENV['AWS_REGION'] ||= options[:region] || 'us-east-1'
-
-def upload(f, options)
-  s3 = Aws::S3::Client.new
-  write = options.fetch(:write)
-  STDERR.puts "Writing #{write}"
-  bucket, key = write.split(':', 2)
-  s3.put_object(
-    body: f.read,
-    bucket: bucket,
-    key: key,
-    **options[:params] || {},
-  )
-  if copy = options[:copy]
-    copy.each do |dest|
-      STDERR.puts "Copying to #{dest}"
-      dbucket, dkey = dest.split(':', 2)
-      s3.copy_object(
-        bucket: dbucket,
-        key: dkey,
-        copy_source: "/#{bucket}/#{key}",
-        **options[:params] || {},
-      )
-    end
-  end
-end
-
-if options[:file] == '-'
-  upload(STDIN, options)
-elsif options[:file]
-  File.open(options[:file]) do |f|
-    upload(f, options)
-  end
-else
-  upload(STDIN, options)
-end

data/spec/shared/lib/mrss/child_process_helper.rb

@@ -1,80 +0,0 @@
-# frozen_string_literal: true
-# encoding: utf-8
-
-autoload :ChildProcess, 'childprocess'
-autoload :Tempfile, 'tempfile'
-
-module Mrss
-  module ChildProcessHelper
-    class SpawnError < StandardError; end
-
-    module_function def call(cmd, env: nil, cwd: nil)
-      process = ChildProcess.new(*cmd)
-      process.io.inherit!
-      if cwd
-        process.cwd = cwd
-      end
-      if env
-        env.each do |k, v|
-          process.environment[k.to_s] = v
-        end
-      end
-      process.start
-      process.wait
-      process
-    end
-
-    module_function def check_call(cmd, env: nil, cwd: nil)
-      process = call(cmd, env: env, cwd: cwd)
-      unless process.exit_code == 0
-        raise SpawnError, "Failed to execute: #{cmd}"
-      end
-    end
-
-    module_function def get_output(cmd, env: nil, cwd: nil)
-      process = ChildProcess.new(*cmd)
-      process.io.inherit!
-      if cwd
-        process.cwd = cwd
-      end
-      if env
-        env.each do |k, v|
-          process.environment[k.to_s] = v
-        end
-      end
-
-      output = ''
-      r, w = IO.pipe
-
-      begin
-        process.io.stdout = w
-        process.start
-        w.close
-
-        thread = Thread.new do
-          begin
-            loop do
-              output << r.readpartial(16384)
-            end
-          rescue EOFError
-          end
-        end
-
-        process.wait
-        thread.join
-      ensure
-        r.close
-      end
-
-      [process, output]
-    end
-
-    module_function def check_output(*args)
-      process, output = get_output(*args)
-      unless process.exit_code == 0
-        raise SpawnError,"Failed to execute: #{args}"
-      end
-      output
-    end
-  end
-end