mongoid 3.1.4 → 3.1.5

data/spec/mongoid/equality_spec.rb

@@ -0,0 +1,241 @@
+# -*- coding: utf-8 -*-
+require "spec_helper"
+
+describe Mongoid::Equality do
+
+  let(:klass) do
+    Person
+  end
+
+  let(:person) do
+    Person.new
+  end
+
+  describe "#==" do
+
+    context "when comparable is not a document" do
+
+      let(:other) do
+        "Document"
+      end
+
+      it "returns false" do
+        person.should_not eq(other)
+      end
+    end
+
+    context "when comparable is a document" do
+
+      context "when it has the same id" do
+
+        context "when the classes are not the same" do
+
+          let(:other) do
+            Post.new
+          end
+
+          before do
+            other.id = person.id
+          end
+
+          it "returns false" do
+            person.should_not eq(other)
+          end
+        end
+
+        context "when the classes are the same" do
+
+          let(:other) do
+            Person.new
+          end
+
+          before do
+            other.id = person.id
+          end
+
+          it "returns true" do
+            person.should eq(other)
+          end
+        end
+      end
+
+      context "when it has a different id" do
+
+        let(:other) do
+          Person.new
+        end
+
+        context "when the instances are the same" do
+
+          it "returns true" do
+            person.should eq(person)
+          end
+        end
+
+        context "when the instances are different" do
+
+          it "returns false" do
+            person.should_not eq(other)
+          end
+        end
+      end
+    end
+  end
+
+  describe ".===" do
+
+    context "when comparable is an instance of this document" do
+
+      it "returns true" do
+        (klass === person).should be_true
+      end
+    end
+
+    context "when comparable is a relation of this document" do
+
+      let(:relation) do
+        Post.new(person: person).person
+      end
+
+      it "returns true" do
+        (klass === relation).should be_true
+      end
+    end
+
+    context "when comparable is the same class" do
+
+      it "returns true" do
+        (klass === Person).should be_true
+      end
+    end
+
+    context "when the comparable is a subclass" do
+
+      it "returns false" do
+        (Person === Doctor).should be_false
+      end
+    end
+
+    context "when the comparable is an instance of a subclass" do
+
+      it "returns true" do
+        (Person === Doctor.new).should be_true
+      end
+    end
+  end
+
+  describe "#===" do
+
+    context "when comparable is the same type" do
+
+      context "when the instance is different" do
+
+        it "returns false" do
+          (person === Person.new).should be_false
+        end
+      end
+
+      context "when the instance is the same" do
+
+        it "returns true" do
+          (person === person).should be_true
+        end
+      end
+    end
+
+    context "when the comparable is a subclass" do
+
+      it "returns false" do
+        (person === Doctor.new).should be_false
+      end
+    end
+
+    context "when comparing to a class" do
+
+      context "when the class is the same" do
+
+        it "returns true" do
+          (person === Person).should be_true
+        end
+      end
+
+      context "when the class is a subclass" do
+
+        it "returns false" do
+          (person === Doctor).should be_false
+        end
+      end
+
+      context "when the class is a superclass" do
+
+        it "returns true" do
+          (Doctor.new === Person).should be_true
+        end
+      end
+    end
+  end
+
+  describe "#<=>" do
+
+    let(:first) do
+      Person.new
+    end
+
+    let(:second) do
+      Person.new
+    end
+
+    it "compares based on the document id" do
+      (first <=> second ).should eq(-1)
+    end
+  end
+
+  describe "#eql?" do
+
+    context "when comparable is not a document" do
+
+      let(:other) do
+        "Document"
+      end
+
+      it "returns false" do
+        person.should_not be_eql(other)
+      end
+    end
+
+    context "when comparable is a document" do
+
+      let(:other) do
+        Person.new
+      end
+
+      context "when it has the same id" do
+
+        before do
+          other.id = person.id
+        end
+
+        it "returns true" do
+          person.should be_eql(other)
+        end
+      end
+
+      context "when it has a different id" do
+
+        context "when the instances are the same" do
+
+          it "returns true" do
+            person.should be_eql(person)
+          end
+        end
+
+        context "when the instances are different" do
+
+          it "returns false" do
+            person.should_not be_eql(other)
+          end
+        end
+      end
+    end
+  end
+end

data/spec/mongoid/errors/ambiguous_relationship_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe Mongoid::Errors::AmbiguousRelationship do
+
+  describe "#message" do
+
+    let(:error) do
+      described_class.new(Person, Drug, :person, [ :drugs, :evil_drugs ])
+    end
+
+    it "contains the problem in the message" do
+      error.message.should include(
+        "Ambiguous relations :drugs, :evil_drugs defined on Person."
+      )
+    end
+
+    it "contains the summary in the message" do
+      error.message.should include(
+        "When Mongoid attempts to set an inverse document of a relation in memory"
+      )
+    end
+
+    it "contains the resolution in the message" do
+      error.message.should include(
+        "On the :person relation on Drug you must add an :inverse_of option"
+      )
+    end
+  end
+end

data/spec/mongoid/errors/callback_spec.rb

@@ -0,0 +1,29 @@
+require "spec_helper"
+
+describe Mongoid::Errors::Callback do
+
+  describe "#message" do
+
+    let(:error) do
+      described_class.new(Post, :create!)
+    end
+
+    it "contains the problem in the message" do
+      error.message.should include(
+        "Calling create! on Post resulted in a false return from a callback."
+      )
+    end
+
+    it "contains the summary in the message" do
+      error.message.should include(
+        "If a before callback returns false when using Document.create!"
+      )
+    end
+
+    it "contains the resolution in the message" do
+      error.message.should include(
+        "Double check all before callbacks to make sure they are not"
+      )
+    end
+  end
+end

data/spec/mongoid/errors/delete_restriction_spec.rb

@@ -0,0 +1,29 @@
+require "spec_helper"
+
+describe Mongoid::Errors::DeleteRestriction do
+
+  describe "#message" do
+
+    let(:error) do
+      described_class.new(Person.new, :drugs)
+    end
+
+    it "contains the problem in the message" do
+      error.message.should include(
+        "Cannot delete Person because of dependent 'drugs'."
+      )
+    end
+
+    it "contains the summary in the message" do
+      error.message.should include(
+        "When defining 'drugs' with a :dependent => :restrict,"
+      )
+    end
+
+    it "contains the resolution in the message" do
+      error.message.should include(
+        "Don't attempt to delete the parent Person when it has children"
+      )
+    end
+  end
+end

data/spec/mongoid/errors/document_not_found_spec.rb

@@ -0,0 +1,104 @@
+require "spec_helper"
+
+describe Mongoid::Errors::DocumentNotFound do
+
+  describe "#message" do
+
+    context "when providing an id" do
+
+      let(:error) do
+        described_class.new(Person, 1, 1)
+      end
+
+      it "contains the problem in the message" do
+        error.message.should include(
+          "Document(s) not found for class Person with id(s) 1."
+        )
+      end
+
+      it "contains the summary in the message" do
+        error.message.should include(
+          "When calling Person.find with an id or array of ids"
+        )
+      end
+
+      it "contains the resolution in the message" do
+        error.message.should include(
+          "Search for an id that is in the database or set the"
+        )
+      end
+    end
+
+    context "when providing ids" do
+
+      let(:error) do
+        described_class.new(Person, [ 1, 2, 3 ], [ 1 ])
+      end
+
+      it "contains the problem in the message" do
+        error.message.should include(
+          "Document(s) not found for class Person with id(s) 1."
+        )
+      end
+
+      it "contains the summary in the message" do
+        error.message.should include(
+          "When calling Person.find with an id or array of ids"
+        )
+      end
+
+      it "contains the resolution in the message" do
+        error.message.should include(
+          "Search for an id that is in the database or set the"
+        )
+      end
+    end
+
+    context "when providing attributes" do
+
+      let(:error) do
+        described_class.new(Person, { name: "syd" }, nil)
+      end
+
+      it "contains the problem in the message" do
+        error.message.should include(
+          "Document not found for class Person with attributes {:name=>\"syd\"}."
+        )
+      end
+
+      it "contains the summary in the message" do
+        error.message.should include(
+          "When calling Person.find_by with a hash of attributes"
+        )
+      end
+
+      it "contains the resolution in the message" do
+        error.message.should include(
+          "Search for attributes that are in the database or set"
+        )
+      end
+    end
+  end
+
+  describe "#params" do
+
+    let(:error) do
+      described_class.new(Person, 1, 1)
+    end
+
+    it "returns the parameters passed to the find" do
+      error.params.should eq(1)
+    end
+  end
+
+  describe "#klass" do
+
+    let(:error) do
+      described_class.new(Person, 1, 1)
+    end
+
+    it "returns the model class" do
+      error.klass.should eq(Person)
+    end
+  end
+end

data/spec/mongoid/errors/eager_load_spec.rb

@@ -0,0 +1,29 @@
+require "spec_helper"
+
+describe Mongoid::Errors::EagerLoad do
+
+  describe "#message" do
+
+    let(:error) do
+      described_class.new(:ratable)
+    end
+
+    it "contains the problem in the message" do
+      error.message.should include(
+        "Eager loading :ratable is not supported since it is a polymorphic"
+      )
+    end
+
+    it "contains the summary in the message" do
+      error.message.should include(
+        "Mongoid cannot currently determine the classes it needs to eager"
+      )
+    end
+
+    it "contains the resolution in the message" do
+      error.message.should include(
+        "Don't attempt to perform this action and have patience"
+      )
+    end
+  end
+end