mongoid 4.0.2 → 5.0.0

data/spec/mongoid/relations/referenced/many_spec.rb

@@ -292,7 +292,7 @@ describe Mongoid::Relations::Referenced::Many do
           it "raises an error" do
             expect {
               person.posts.send(method, post)
-            }.to raise_error(Moped::Errors::OperationFailure)
+            }.to raise_error(Mongo::Error::OperationFailure)
           end
         end
       end
@@ -1536,7 +1536,7 @@ describe Mongoid::Relations::Referenced::Many do
               person.posts.create do |doc|
                 doc._id = existing.id
               end
-            }.to raise_error(Moped::Errors::OperationFailure)
+            }.to raise_error(Mongo::Error::OperationFailure)
           end
         end
       end
@@ -2164,30 +2164,24 @@ describe Mongoid::Relations::Referenced::Many do
 
   describe "#find" do
 
-    context "when the identity map is enabled" do
+    context "when iterating after the find" do
 
-      context "when the document is in the map" do
-
-        let(:person) do
-          Person.create
-        end
-
-        before do
-          person.posts.create(title: "Test")
-        end
+      let(:person) do
+        Person.create!
+      end
 
-        context "when the document does not belong to the relation" do
+      let(:post_id) do
+        person.posts.first.id
+      end
 
-          let!(:post) do
-            Post.create(title: "testing")
-          end
+      before do
+        5.times { person.posts.create! }
+      end
 
-          it "raises an error" do
-            expect {
-              person.posts.find(post.id)
-            }.to raise_error(Mongoid::Errors::DocumentNotFound)
-          end
-        end
+      it "does not change the in memory size" do
+        expect {
+          person.posts.find(post_id)
+        }.not_to change { person.posts.to_a.size }
       end
     end
 
@@ -2807,6 +2801,35 @@ describe Mongoid::Relations::Referenced::Many do
     end
   end
 
+  describe "#last" do
+
+    let(:person) do
+      Person.create!
+    end
+
+    let!(:persisted_post) do
+      person.posts.create!
+    end
+
+    context "when a new document is added" do
+
+      let!(:new_post) do
+        person.posts.new
+      end
+
+      context "when the target is subsequently loaded" do
+
+        before do
+          person.posts.entries
+        end
+
+        it "returns the expected last document" do
+          expect(person.posts.last).to eq(new_post)
+        end
+      end
+    end
+  end
+
   describe ".macro" do
 
     it "returns has_many" do
@@ -3344,7 +3367,7 @@ describe Mongoid::Relations::Referenced::Many do
 
       before do
         Post.collection.find({ _id: post_one.id }).
-          update({ "$set" => { title: "reloaded" }})
+          update_one({ "$set" => { title: "reloaded" }})
       end
 
       let(:reloaded) do
@@ -3687,6 +3710,7 @@ describe Mongoid::Relations::Referenced::Many do
   end
 
   context "when accessing a relation named parent" do
+
     let!(:parent) do
       Odd.create(name: "odd parent")
     end

data/spec/mongoid/relations/referenced/many_to_many_spec.rb

@@ -3274,7 +3274,7 @@ describe Mongoid::Relations::Referenced::ManyToMany do
 
       before do
         Preference.collection.find({ _id: preference_one.id }).
-          update({ "$set" => { name: "reloaded" }})
+          update_one({ "$set" => { name: "reloaded" }})
       end
 
       let(:reloaded) do

data/spec/mongoid/relations/referenced/one_spec.rb

@@ -1178,7 +1178,7 @@ describe Mongoid::Relations::Referenced::One do
 
       before do
         Game.collection.find({ _id: game_one.id }).
-          update({ "$set" => { name: "Diablo 2" }})
+          update_one({ "$set" => { name: "Diablo 2" }})
       end
 
       let(:reloaded) do

data/spec/mongoid/relations/targets/enumerable_spec.rb

@@ -122,6 +122,13 @@ describe Mongoid::Relations::Targets::Enumerable do
       end
     end
 
+    context "when compared to an enumerable class" do
+
+      it "returns true" do
+        expect(enumerable === described_class).to be true
+      end
+    end
+
     context "when compared to a different class" do
 
       it "returns false" do
@@ -1186,7 +1193,7 @@ describe Mongoid::Relations::Targets::Enumerable do
     context "when the enumerable is not loaded" do
 
       let(:criteria) do
-        Post.where(person_id: person.id)
+        Post.asc(:_id).where(person_id: person.id)
       end
 
       let(:enumerable) do
@@ -1272,7 +1279,7 @@ describe Mongoid::Relations::Targets::Enumerable do
         context "when accessing from a reloaded child" do
 
           it "returns the last document" do
-            expect(post_one.reload.person.posts.last).to eq(post_two)
+            expect(post_one.reload.person.posts.asc(:_id).last).to eq(post_two)
           end
         end
       end

data/spec/mongoid/reloadable_spec.rb

@@ -115,7 +115,7 @@ describe Mongoid::Reloadable do
           before do
             Person.collection.find(
               { "_id" => person.id }
-            ).update({ "$set" => { "addresses.0.number" => 3 }})
+            ).update_one({ "$set" => { "addresses.0.number" => 3 }})
           end
 
           let!(:reloaded) do
@@ -143,7 +143,7 @@ describe Mongoid::Reloadable do
 
           before do
             Person.collection.find({ "_id" => person.id }).
-              update({ "$set" => { "name.last_name" => "Vicious" }})
+              update_one({ "$set" => { "name.last_name" => "Vicious" }})
           end
 
           let!(:reloaded) do
@@ -176,7 +176,7 @@ describe Mongoid::Reloadable do
 
         before do
           Person.collection.find({ "_id" => person.id }).
-            update({ "$set" => { "addresses.0.locations.0.name" => "work" }})
+            update_one({ "$set" => { "addresses.0.locations.0.name" => "work" }})
         end
 
         let!(:reloaded) do
@@ -209,7 +209,7 @@ describe Mongoid::Reloadable do
 
       before do
         Person.collection.find({ "_id" => person.id }).
-          update({ "$set" => { "addresses" => [] }})
+          update_one({ "$set" => { "addresses" => [] }})
         person.reload
       end
 
@@ -232,7 +232,7 @@ describe Mongoid::Reloadable do
 
         before do
           Game.collection.find({ "_id" => game.id }).
-            update({ "$set" => { "score" => 75 }})
+            update_one({ "$set" => { "score" => 75 }})
           person.reload
         end
 
@@ -251,7 +251,7 @@ describe Mongoid::Reloadable do
 
           before do
             Person.collection.find({ "_id" => person.id }).
-              update({ "$set" => { "title" => "Mam" }})
+              update_one({ "$set" => { "title" => "Mam" }})
             game.reload
           end
 

data/spec/mongoid/scopable_spec.rb

@@ -144,11 +144,11 @@ describe Mongoid::Scopable do
       end
 
       before do
-        Band.scope_stack.push(criteria)
+        Mongoid::Threaded.current_scope = criteria
       end
 
       after do
-        Band.scope_stack.clear
+        Mongoid::Threaded.current_scope = nil
       end
 
       it "returns the criteria on the stack" do
@@ -567,13 +567,9 @@ describe Mongoid::Scopable do
 
       context "when both scopes are or queries" do
 
-        let(:time) do
-          Time.now
-        end
-
         before do
           Band.scope(:xxx, ->{ Band.any_of({ :aaa.gt => 0 }, { :bbb.gt => 0 }) })
-          Band.scope(:yyy, ->{ Band.any_of({ :ccc => nil }, { :ccc.gt => time }) })
+          Band.scope(:yyy, ->{ Band.any_of({ :ccc => nil }, { :ccc.gt => 1 }) })
         end
 
         after do
@@ -592,7 +588,7 @@ describe Mongoid::Scopable do
           expect(criteria.selector).to eq({
             "$or" => [
               { "ccc" => nil },
-              { "ccc" => { "$gt" => time }},
+              { "ccc" => { "$gt" => 1.0 }},
               { "aaa" => { "$gt" => 0.0 }},
               { "bbb" => { "$gt" => 0.0 }}
             ]
@@ -644,33 +640,34 @@ describe Mongoid::Scopable do
         expect(circle_scope_keys).to match_array([:located_at, :with_radius])
       end
     end
-  end
-
-  describe ".scope_stack" do
-
-    context "when the scope stack has not been accessed" do
-
-      it "returns an empty array" do
-        expect(Band.scope_stack).to eq([])
-      end
-    end
 
-    context "when a criteria exists on the current thread" do
-
-      let(:criteria) do
-        Band.where(active: true)
-      end
+    context "when calling a scope defined in a parent class" do
 
       before do
-        Mongoid::Threaded.scope_stack[Band.object_id] = [ criteria ]
+        Shape.class_eval do
+          scope :visible, -> { large }
+          scope :large, -> { all }
+        end
+        Circle.class_eval do
+          scope :large, -> { where(radius: 5) }
+        end
       end
 
       after do
-        Mongoid::Threaded.scope_stack[Band.object_id].clear
+        class << Shape
+          undef_method :visible
+          undef_method :large
+        end
+        Shape._declared_scopes.clear
+
+        class << Circle
+          undef_method :large
+        end
+        Circle._declared_scopes.clear
       end
 
-      it "returns the criteria in the array" do
-        expect(Band.scope_stack).to eq([ criteria ])
+      it "uses sublcass context for all the other used scopes" do
+        expect(Circle.visible.selector).to eq("radius" => 5)
       end
     end
   end
@@ -784,6 +781,22 @@ describe Mongoid::Scopable do
           expect(unscoped.selector).to be_empty
         end
       end
+
+      context "when default scope is in a super class" do
+
+        before do
+          Band.scope(:active, ->{ Band.where(active: true) })
+        end
+
+        let(:unscoped) do
+          class U2 < Band; end
+          U2.unscoped.active
+        end
+
+        it "clears default scope" do
+          expect(unscoped.selector).to eq({ "active" => true })
+        end
+      end
     end
 
     context "when used with a block" do
@@ -888,7 +901,7 @@ describe Mongoid::Scopable do
 
     it "pops the criteria off the stack" do
       Band.with_scope(criteria) {}
-      expect(Band.scope_stack).to be_empty
+      expect(Mongoid::Threaded.current_scope).to be_nil
     end
   end
 

data/spec/mongoid/selectable_spec.rb

@@ -82,10 +82,8 @@ describe Mongoid::Selectable do
           person.create_name(first_name: "test", last_name: "user")
         end
 
-        it "returns the hash with the selector" do
-          expect(name.atomic_selector).to eq(
-            { "_id" => person.id, "name._id" => name.id }
-          )
+        it "returns the hash with the root selector" do
+          expect(name.atomic_selector).to eq({ "_id" => person.id })
         end
       end
 
@@ -95,10 +93,8 @@ describe Mongoid::Selectable do
           person.addresses.create(street: "kreuzbergstr")
         end
 
-        it "returns the hash with the selector" do
-          expect(address.atomic_selector).to eq(
-            { "_id" => person.id, "addresses._id" => address.id }
-          )
+        it "returns the hash with the root selector" do
+          expect(address.atomic_selector).to eq({ "_id" => person.id })
         end
 
         context "when the document's id changes" do
@@ -118,14 +114,8 @@ describe Mongoid::Selectable do
             address.locations.create
           end
 
-          it "returns a hash with the selector" do
-            expect(location.atomic_selector).to eq(
-              {
-                "_id" => person.id,
-                "addresses._id" => address.id,
-                "addresses.0.locations._id" => location.id
-              }
-            )
+          it "returns a hash with the root selector" do
+            expect(location.atomic_selector).to eq({ "_id" => person.id })
           end
         end
       end

data/spec/mongoid/tasks/database_rake_spec.rb

@@ -45,7 +45,7 @@ shared_context "rails rake task" do
   end
 end
 
-describe "db:drop" do
+describe "db:drop", if: non_legacy_server? do
   include_context "rake task"
   include_context "rails rake task"
 
@@ -58,7 +58,7 @@ describe "db:drop" do
   end
 end
 
-describe "db:purge" do
+describe "db:purge", if: non_legacy_server? do
   include_context "rake task"
   include_context "rails rake task"
 
@@ -71,7 +71,7 @@ describe "db:purge" do
   end
 end
 
-describe "db:seed" do
+describe "db:seed", if: non_legacy_server? do
   include_context "rake task"
   include_context "rails rake task"
 
@@ -85,7 +85,7 @@ describe "db:seed" do
   end
 end
 
-describe "db:setup" do
+describe "db:setup", if: non_legacy_server? do
   include_context "rake task"
   include_context "rails rake task"
 
@@ -117,7 +117,7 @@ describe "db:setup" do
   end
 end
 
-describe "db:reset" do
+describe "db:reset", if: non_legacy_server? do
   include_context "rake task"
   include_context "rails rake task"
 
@@ -135,7 +135,7 @@ describe "db:reset" do
   end
 end
 
-describe "db:create" do
+describe "db:create", if: non_legacy_server? do
   include_context "rake task"
   include_context "rails rake task"
 
@@ -144,7 +144,7 @@ describe "db:create" do
   end
 end
 
-describe "db:migrate" do
+describe "db:migrate", if: non_legacy_server? do
   include_context "rake task"
   include_context "rails rake task"
 
@@ -153,7 +153,7 @@ describe "db:migrate" do
   end
 end
 
-describe "db:test:prepare" do
+describe "db:test:prepare", if: non_legacy_server? do
   include_context "rake task"
   include_context "rails rake task"
 
@@ -174,7 +174,7 @@ describe "db:test:prepare" do
   end
 end
 
-describe "db:mongoid:create_indexes" do
+describe "db:mongoid:create_indexes", if: non_legacy_server? do
   include_context "rake task"
 
   it_behaves_like "create_indexes"
@@ -198,7 +198,7 @@ describe "db:mongoid:create_indexes" do
   end
 end
 
-describe "db:mongoid:remove_undefined_indexes" do
+describe "db:mongoid:remove_undefined_indexes", if: non_legacy_server? do
   include_context "rake task"
 
   it "receives remove_undefined_indexes" do
@@ -224,7 +224,7 @@ describe "db:mongoid:remove_undefined_indexes" do
   end
 end
 
-describe "db:mongoid:remove_indexes" do
+describe "db:mongoid:remove_indexes", if: non_legacy_server? do
   include_context "rake task"
 
   it "receives remove_indexes" do
@@ -250,7 +250,7 @@ describe "db:mongoid:remove_indexes" do
   end
 end
 
-describe "db:mongoid:drop" do
+describe "db:mongoid:drop", if: non_legacy_server? do
   include_context "rake task"
 
   it "works" do
@@ -266,7 +266,7 @@ describe "db:mongoid:drop" do
   end
 end
 
-describe "db:mongoid:purge" do
+describe "db:mongoid:purge", if: non_legacy_server? do
   include_context "rake task"
 
   it "receives a purge" do

data/spec/mongoid/tasks/database_spec.rb

@@ -100,7 +100,7 @@ describe "Mongoid::Tasks::Database" do
     context "with extra index on model collection" do
 
       before(:each) do
-        User.collection.indexes.create(account_expires: 1)
+        User.collection.indexes.create_one(account_expires: 1)
       end
 
       let(:names) do
@@ -121,7 +121,7 @@ describe "Mongoid::Tasks::Database" do
 
     before(:each) do
       Mongoid::Tasks::Database.create_indexes(models)
-      indexes.create(account_expires: 1)
+      indexes.create_one(account_expires: 1)
       Mongoid::Tasks::Database.remove_undefined_indexes(models)
     end
 

data/spec/mongoid/threaded_spec.rb

@@ -106,13 +106,6 @@ describe Mongoid::Threaded do
     end
   end
 
-  describe "#scope_stack" do
-
-    it "returns the default with the scope stack key" do
-      expect(described_class.scope_stack).to be_a(Hash)
-    end
-  end
-
   describe "#begin_validate" do
 
     let(:person) do

data/spec/mongoid/traversable_spec.rb

@@ -71,14 +71,14 @@ describe Mongoid::Traversable do
     context "when the document is a subclass" do
 
       it "returns true" do
-        expect(Circle).to be_hereditary
+        expect(Circle.hereditary?).to be true
       end
     end
 
     context "when the document is not a subclass" do
 
       it "returns false" do
-        expect(Shape).to_not be_hereditary
+        expect(Shape.hereditary?).to be false
       end
     end
   end

data/spec/mongoid/validatable/uniqueness_spec.rb

@@ -147,6 +147,35 @@ describe Mongoid::Validatable::UniquenessValidator do
                 Dictionary.reset_callbacks(:validate)
               end
 
+              context "when no attribute is set" do
+
+                context "when no document with no value exists in the database" do
+
+                  let(:dictionary) do
+                    Dictionary.new
+                  end
+
+                  it "returns true" do
+                    expect(dictionary).to be_valid
+                  end
+                end
+
+                context "when a document with no value exists in the database" do
+
+                  before do
+                    Dictionary.create
+                  end
+
+                  let(:dictionary) do
+                    Dictionary.new
+                  end
+
+                  it "returns false" do
+                    expect(dictionary).to_not be_valid
+                  end
+                end
+              end
+
               context "when the attribute is unique" do
 
                 context "when single localization" do
@@ -2382,7 +2411,7 @@ describe Mongoid::Validatable::UniquenessValidator do
       Person.reset_callbacks(:validate)
     end
 
-    it "transfers the options to the cloned session" do
+    it "transfers the options to the cloned client" do
       expect {
         Person.create!(ssn: "132-11-1111", username: "asdfsdfA")
       }.to raise_error

data/spec/mongoid_spec.rb

@@ -31,37 +31,35 @@ describe Mongoid do
     end
   end
 
-  describe ".default_session" do
+  describe ".default_client" do
 
-    it "returns the default session" do
-      expect(Mongoid.default_session).to eq(Mongoid::Sessions.default)
+    it "returns the default client" do
+      expect(Mongoid.default_client).to eq(Mongoid::Clients.default)
     end
   end
 
-  describe ".disconnect_sessions" do
+  describe ".disconnect_clients" do
 
-    let(:sessions) do
-      Mongoid::Threaded.sessions.values
+    let(:clients) do
+      Mongoid::Clients.clients.values
     end
 
     before do
       Band.all.entries
-      Mongoid.disconnect_sessions
     end
 
-    it "disconnects from all active sessions" do
-      sessions.each do |session|
-        session.cluster.nodes.each do |node|
-          expect(node.send(:connected?)).to be false
-        end
+    it "disconnects from all active clients" do
+      clients.each do |client|
+        expect(client.cluster).to receive(:disconnect!).and_call_original
       end
+      Mongoid.disconnect_clients
     end
   end
 
-  describe ".session" do
+  describe ".client" do
 
-    it "returns the named session" do
-      expect(Mongoid.session(:default)).to eq(Mongoid::Sessions.default)
+    it "returns the named client" do
+      expect(Mongoid.client(:default)).to eq(Mongoid::Clients.default)
     end
   end