mongoid 7.0.13 → 7.1.0.rc0

data/spec/app/models/vet_visit.rb

@@ -1,3 +1,6 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
 class VetVisit
   include Mongoid::Document
   field :date, type: Date

data/spec/app/models/video.rb

@@ -1,3 +1,6 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
 class Video
   include Mongoid::Document
   field :title, type: String

data/spec/app/models/video_game.rb

@@ -1 +1,4 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
 class VideoGame < Game; end

data/spec/app/models/weapon.rb

@@ -1,3 +1,6 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
 class Weapon
   include Mongoid::Document
 

data/spec/app/models/wiki_page.rb

@@ -1,3 +1,6 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
 class WikiPage
   include Mongoid::Document
   include Mongoid::Timestamps

data/spec/app/models/word.rb

@@ -1,3 +1,6 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
 class Word
   include Mongoid::Document
   field :name, type: String

data/spec/app/models/word_origin.rb

@@ -1,3 +1,6 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
 class WordOrigin
   include Mongoid::Document
 

data/spec/app/models/writer.rb

@@ -1,3 +1,6 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
 class Writer
   include Mongoid::Document
   field :speed, type: Integer, default: 0

data/spec/config/mongoid.yml

@@ -39,6 +39,7 @@ test:
     belongs_to_required_by_default: false
     app_name: 'testing'
     background_indexing: false
+    join_contexts: false
 test_with_max_staleness:
   clients:
     default:

data/spec/integration/associations/belongs_to_spec.rb

@@ -1,3 +1,6 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
 require 'spec_helper'
 require_relative '../../mongoid/association/referenced/has_one_models'
 

data/spec/integration/associations/embedded_spec.rb

@@ -3,63 +3,8 @@
 
 require 'spec_helper'
 require_relative '../../mongoid/association/embedded/embeds_many_models'
-require_relative '../../mongoid/association/embedded/embeds_one_models'
 
 describe 'embedded associations' do
-
-  describe 'parent association' do
-    let(:parent) do
-      parent_cls.new
-    end
-
-    context 'embeds_one' do
-
-      shared_examples 'is set' do
-        it 'is set' do
-          parent.child = child_cls.new
-          parent.child.parent.should == parent
-        end
-      end
-
-      context 'class_name set without leading ::' do
-        let(:parent_cls) { EomParent }
-        let(:child_cls) { EomChild }
-
-        it_behaves_like 'is set'
-      end
-
-      context 'class_name set with leading ::' do
-        let(:parent_cls) { EomCcParent }
-        let(:child_cls) { EomCcChild }
-
-        it_behaves_like 'is set'
-      end
-    end
-
-    context 'embeds_many' do
-
-      let(:child) { parent.legislators.new }
-
-      shared_examples 'is set' do
-        it 'is set' do
-          child.congress.should == parent
-        end
-      end
-
-      context 'class_name set without leading ::' do
-        let(:parent_cls) { EmmCongress }
-
-        it_behaves_like 'is set'
-      end
-
-      context 'class_name set with leading ::' do
-        let(:parent_cls) { EmmCcCongress }
-
-        it_behaves_like 'is set'
-      end
-    end
-  end
-
   context 'without default order' do
     let(:congress) do
       EmmCongress.create!

data/spec/integration/associations/foreign_key_spec.rb

@@ -0,0 +1,99 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
+require 'spec_helper'
+require_relative './foreign_key_spec_models'
+
+describe 'Association foreign key configuration' do
+  describe 'has_many/belongs_to' do
+    it 'creates child' do
+      company = ForeignKeySpec::Company.create!(c: 'test')
+      email = ForeignKeySpec::Email.create!(company: company)
+      expect(email.c_ref).to eq('test')
+    end
+
+    it 'looks up child' do
+      company = ForeignKeySpec::Company.create!(c: 'test')
+      email = ForeignKeySpec::Email.create!(company: company)
+
+      company = ForeignKeySpec::Company.find(company.id)
+      expect(company.emails.first.id).to eq(email.id)
+    end
+
+    it 'looks up parent' do
+      company = ForeignKeySpec::Company.create!(c: 'test')
+      email = ForeignKeySpec::Email.create!(company: company)
+
+      email = ForeignKeySpec::Email.find(email.id)
+      expect(email.company.id).to eq(company.id)
+    end
+
+    context 'with default scope' do
+      context 'using default scope' do
+        it 'looks up child' do
+          company = ForeignKeySpec::ScopedCompany.create!(c: 'test')
+          on_email = ForeignKeySpec::ScopedEmail.create!(company: company, s: 'on')
+          off_email = ForeignKeySpec::ScopedEmail.create!(company: company, s: 'off')
+
+          company = ForeignKeySpec::ScopedCompany.find(company.id)
+          expect(company.emails.length).to eq(1)
+          expect(company.emails.first.id).to eq(on_email.id)
+        end
+      end
+
+      context 'unscoped' do
+        it 'looks up child' do
+          company = ForeignKeySpec::ScopedCompany.create!(c: 'test')
+          on_email = ForeignKeySpec::ScopedEmail.create!(company: company, s: 'on')
+          off_email = ForeignKeySpec::ScopedEmail.create!(company: company, s: 'off')
+
+          company = ForeignKeySpec::ScopedCompany.find(company.id)
+          expect(company.emails.unscoped.length).to eq(2)
+          expect(company.emails.unscoped.map(&:id).sort).to eq([on_email.id, off_email.id].sort)
+        end
+      end
+    end
+  end
+
+  describe 'has_one/belongs_to' do
+    it 'creates child' do
+      company = ForeignKeySpec::Company.create!(c: 'test')
+      founder = ForeignKeySpec::Founder.create!(company: company)
+      expect(founder.c_ref).to eq('test')
+    end
+
+    it 'looks up child' do
+      company = ForeignKeySpec::Company.create!(c: 'test')
+      founder = ForeignKeySpec::Founder.create!(company: company)
+
+      company = ForeignKeySpec::Company.find(company.id)
+      expect(company.founder.id).to eq(founder.id)
+    end
+
+    it 'looks up parent' do
+      company = ForeignKeySpec::Company.create!(c: 'test')
+      founder = ForeignKeySpec::Founder.create!(company: company)
+
+      email = ForeignKeySpec::Founder.find(founder.id)
+      expect(founder.company.id).to eq(company.id)
+    end
+  end
+
+  describe 'has_and_belongs_to_many' do
+    it 'persists association on the other side' do
+      animal = ForeignKeySpec::Animal.create!(a: 'bear')
+      zoo = ForeignKeySpec::Zoo.create!(z: 'bz')
+      animal.zoos << zoo
+      animal.save!
+
+      # This write should not be necessary, but without it the test fails.
+      # https://jira.mongodb.org/browse/MONGOID-4648
+      zoo.animals << animal
+      zoo.save!
+
+      expect(zoo.animals).to eq([animal])
+      zoo = ForeignKeySpec::Zoo.find(zoo.id)
+      expect(zoo.animals).to eq([animal])
+    end
+  end
+end

data/spec/integration/associations/foreign_key_spec_models.rb

@@ -0,0 +1,65 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
+module ForeignKeySpec
+  class Company
+    include Mongoid::Document
+
+    field :c, type: String
+    has_many :emails, class_name: 'ForeignKeySpec::Email',
+      foreign_key: 'c_ref', primary_key: 'c'
+    has_one :founder, class_name: 'ForeignKeySpec::Founder',
+      foreign_key: 'c_ref', primary_key: 'c'
+  end
+
+  class Email
+    include Mongoid::Document
+
+    field :c_ref, type: String
+    belongs_to :company, class_name: 'ForeignKeySpec::Company',
+      foreign_key: 'c_ref', primary_key: 'c'
+  end
+
+  class Founder
+    include Mongoid::Document
+
+    field :c_ref, type: String
+    belongs_to :company, class_name: 'ForeignKeySpec::Company',
+      foreign_key: 'c_ref', primary_key: 'c'
+  end
+
+  class Animal
+    include Mongoid::Document
+
+    field :a, type: String
+    has_and_belongs_to_many :zoos, class_name: 'ForeignKeySpec::Zoo',
+      foreign_key: 'z_refs', primary_key: 'z'
+  end
+
+  class Zoo
+    include Mongoid::Document
+
+    field :z, type: String
+    has_and_belongs_to_many :animals, class_name: 'ForeignKeySpec::Animal',
+      foreign_key: 'a_refs', primary_key: 'a'
+  end
+
+  class ScopedCompany
+    include Mongoid::Document
+
+    field :c, type: String
+    has_many :emails, class_name: 'ForeignKeySpec::ScopedEmail',
+      foreign_key: 'c_ref', primary_key: 'c'
+  end
+
+  class ScopedEmail
+    include Mongoid::Document
+
+    field :c_ref, type: String
+    belongs_to :company, class_name: 'ForeignKeySpec::ScopedCompany',
+      foreign_key: 'c_ref', primary_key: 'c'
+
+    field :s, type: String
+    default_scope -> { where(s: 'on') }
+  end
+end

data/spec/integration/associations/reverse_population_spec.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
+require 'spec_helper'
+require_relative './reverse_population_spec_models'
+
+describe 'Association reverse population' do
+  describe 'has_many/belongs_to' do
+    it 'populates child in parent' do
+      company = ReversePopulationSpec::Company.create!
+      email = ReversePopulationSpec::Email.create!(company: company)
+      expect(company.emails).to eq([email])
+    end
+  end
+
+  describe 'has_one/belongs_to' do
+    it 'populates child in parent' do
+      company = ReversePopulationSpec::Company.create!
+      founder = ReversePopulationSpec::Founder.create!(company: company)
+      expect(company.founder).to eq(founder)
+    end
+  end
+
+  describe 'has_and_belongs_to_many' do
+    it 'persists association on the other side' do
+      animal = ReversePopulationSpec::Animal.create!
+      zoo = ReversePopulationSpec::Zoo.create!
+      animal.zoos << zoo
+      animal.save!
+      expect(zoo.animals).to eq([animal])
+      zoo = ReversePopulationSpec::Zoo.find(zoo.id)
+      expect(zoo.animals).to eq([animal])
+    end
+  end
+end

data/spec/integration/associations/reverse_population_spec_models.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+# encoding: utf-8
+
+module ReversePopulationSpec
+  class Company
+    include Mongoid::Document
+
+    has_many :emails, class_name: 'ReversePopulationSpec::Email'
+    has_one :founder, class_name: 'ReversePopulationSpec::Founder'
+  end
+
+  class Email
+    include Mongoid::Document
+
+    belongs_to :company, class_name: 'ReversePopulationSpec::Company'
+  end
+
+  class Founder
+    include Mongoid::Document
+
+    belongs_to :company, class_name: 'ReversePopulationSpec::Company'
+  end
+
+  class Animal
+    include Mongoid::Document
+
+    field :a, type: String
+    has_and_belongs_to_many :zoos, class_name: 'ReversePopulationSpec::Zoo'
+  end
+
+  class Zoo
+    include Mongoid::Document
+
+    field :z, type: String
+    has_and_belongs_to_many :animals, class_name: 'ReversePopulationSpec::Animal'
+  end
+end

data/spec/integration/criteria/default_scope_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+describe 'Criteria and default scope' do
+
+  context 'order in query' do
+    let(:query) do
+      Acolyte.order(status: :desc)
+    end
+
+    let(:sort_options) do
+      query.options[:sort]
+    end
+
+    it 'is added after order of default scope' do
+      sort_options.should == {'status' => -1, 'name' => 1}
+
+      # Keys in Ruby are ordered
+      sort_options.keys.should == %w(name status)
+    end
+  end
+end

data/spec/integration/criteria/logical_spec.rb

@@ -0,0 +1,81 @@
+require 'spec_helper'
+
+describe 'Criteria logical operations' do
+  let!(:ap) { Band.create!(name: 'Astral Projection', origin: 'SFX', genres: ['Goa', 'Psy']) }
+  let!(:im) { Band.create!(name: 'Infected Mushroom', origin: 'Computers', genres: ['Psy']) }
+  let!(:sp) { Band.create!(name: 'Sun Project', genres: ['Goa']) }
+
+  describe 'and' do
+    it 'combines conditions on different fields given as hashes' do
+      bands = Band.where(name: /Proj/).and(genres: 'Psy')
+      expect(bands.to_a).to eq([ap])
+    end
+
+    it 'combines conditions on different fields given as scopes' do
+      bands = Band.where(name: /Proj/).and(Band.where(genres: 'Psy'))
+      expect(bands.to_a).to eq([ap])
+    end
+
+    it 'combines conditions on same field given as hashes' do
+      bands = Band.where(name: /Proj/).and(name: /u/)
+      expect(bands.to_a).to eq([sp])
+    end
+
+    it 'combines conditions on same field given as scopes' do
+      bands = Band.where(name: /Proj/).and(Band.where(name: /u/))
+      expect(bands.to_a).to eq([sp])
+    end
+  end
+
+  describe 'or' do
+    it 'combines conditions on different fields given as hashes' do
+      bands = Band.where(name: /Sun/).or(origin: 'SFX')
+      expect(bands.order_by(name: 1).to_a).to eq([ap, sp])
+    end
+
+    it 'combines conditions on different fields given as scopes' do
+      bands = Band.where(name: /Sun/).or(Band.where(origin: 'SFX'))
+      expect(bands.order_by(name: 1).to_a).to eq([ap, sp])
+    end
+
+    it 'combines conditions on same field given as hashes' do
+      bands = Band.where(name: /jecti/).or(name: /ush/)
+      expect(bands.order_by(name: 1).to_a).to eq([ap, im])
+    end
+
+    it 'combines conditions on same field given as scopes' do
+      bands = Band.where(name: /jecti/).or(Band.where(name: /ush/))
+      expect(bands.order_by(name: 1).to_a).to eq([ap, im])
+    end
+  end
+
+  describe 'not' do
+    context 'hash argument with string value' do
+      let(:actual) do
+        Band.not(name: 'test').selector
+      end
+
+      let(:expected) do
+        {'name' => {'$ne' => 'test'}}
+      end
+
+      it 'expands to use $ne' do
+        expect(actual).to eq(expected)
+      end
+    end
+
+    context 'hash argument with regexp value' do
+      let(:actual) do
+        Band.not(name: /test/).selector
+      end
+
+      let(:expected) do
+        {'name' => {'$not' => /test/}}
+      end
+
+      it 'expands to use $not' do
+        expect(actual).to eq(expected)
+      end
+    end
+  end
+end