mongoid 1.2.8 → 1.2.9

data/spec/unit/mongoid/associations/has_many_spec.rb

@@ -129,14 +129,16 @@ describe Mongoid::Associations::HasMany do
           @document,
           Mongoid::Associations::Options.new(:name => :addresses)
         )
-        @address = Address.new(:street => "Yet Another")
+        @address = mock(:parentize => true, :write_attributes => true)
+        Address.expects(:instantiate).returns(@address)
+        @address.expects(:run_callbacks).with(:before_create)
+        @address.expects(:run_callbacks).with(:after_create)
       end
 
       it "builds and saves a new object" do
-        Mongoid::Commands::Save.expects(:execute).returns(true)
+        @address.expects(:save).returns(true)
         address = @association.create({ :street => "Yet Another" })
-        address.should be_a_kind_of(Address)
-        address.street.should == "Yet Another"
+        address.should == @address
       end
 
     end
@@ -148,13 +150,16 @@ describe Mongoid::Associations::HasMany do
           @document,
           Mongoid::Associations::Options.new(:name => :shapes)
         )
+        @circle = mock(:parentize => true, :write_attributes => true)
+        Circle.expects(:instantiate).returns(@circle)
+        @circle.expects(:run_callbacks).with(:before_create)
+        @circle.expects(:run_callbacks).with(:after_create)
       end
 
       it "instantiates a class of that type" do
-        Mongoid::Commands::Save.expects(:execute).returns(true)
+        @circle.expects(:save).returns(true)
         circle = @association.create({ :radius => 100 }, Circle)
-        circle.should be_a_kind_of(Circle)
-        circle.radius.should == 100
+        circle.should == @circle
       end
 
     end

data/spec/unit/mongoid/associations_spec.rb

@@ -453,6 +453,22 @@ describe Mongoid::Associations do
 
     end
 
+    context "when using object ids" do
+
+      before do
+        Mongoid.use_object_ids = true
+      end
+
+      after do
+        Mongoid.use_object_ids = false
+      end
+
+      it "sets the foreign key as an object id" do
+        Game.expects(:field).with("person_id", :type => Mongo::ObjectID)
+        Game.belongs_to_related :person
+      end
+    end
+
   end
 
   describe ".has_one_related" do

data/spec/unit/mongoid/attributes_spec.rb

@@ -142,6 +142,17 @@ describe Mongoid::Attributes do
 
   describe "#process" do
 
+    context "when passing non accessible fields" do
+
+      before do
+        @person = Person.new(:owner_id => 6)
+      end
+
+      it "does not set the value" do
+        @person.owner_id.should be_nil
+      end
+    end
+
     context "when attributes dont have fields defined" do
 
       before do
@@ -336,6 +347,18 @@ describe Mongoid::Attributes do
 
     end
 
+    context "when attribute is not accessible" do
+
+      before do
+        @person = Person.new
+        @person.owner_id = 5
+      end
+
+      it "returns the value" do
+        @person.read_attribute(:owner_id).should == 5
+      end
+    end
+
   end
 
   describe "#remove_attribute" do

data/spec/unit/mongoid/commands/destroy_spec.rb

@@ -29,10 +29,13 @@ describe Mongoid::Commands::Destroy do
         @parent = Person.new
         @address = Address.new(:street => "Genoa Pl")
         @parent.addresses << @address
+        @collection = mock
+        Person.expects(:collection).returns(@collection)
       end
 
       it "removes the document from the parent attributes" do
         @parent.addresses.should == [@address]
+        @collection.expects(:save).returns(true)
         Mongoid::Commands::Destroy.execute(@address)
         @parent.addresses.should be_empty
       end

data/spec/unit/mongoid/commands_spec.rb

@@ -86,10 +86,9 @@ describe Mongoid::Commands do
         @person = Person.new
       end
 
-      it "returns false" do
+      it "lets the error bubble up" do
         Mongoid::Commands::Save.expects(:execute).raises(Mongo::OperationFailure.new("Operation Failed"))
-        @person.save
-        @person.errors.on(:mongoid).should == "Operation Failed"
+        lambda { @person.save }.should raise_error
       end
 
     end
@@ -204,14 +203,8 @@ describe Mongoid::Commands do
         Mongoid::Commands::Save.expects(:execute).raises(Mongo::OperationFailure.new("Operation Failed"))
       end
 
-      it "returns the document with errors" do
-        person = Person.create
-        person.errors.on(:mongoid).should == "Operation Failed"
-      end
-
-      it "keeps the document's new record flag" do
-        person = Person.create
-        person.should be_a_new_record
+      it "lets the error bubble up" do
+        lambda { Person.create }.should raise_error
       end
 
     end

data/spec/unit/mongoid/contexts/enumerable_spec.rb

@@ -145,6 +145,22 @@ describe Mongoid::Contexts::Enumerable do
 
   end
 
+  describe "#iterate" do
+    before do
+      @criteria.where(:street => "Bourke Street")
+      @criteria.documents = @docs
+      @context = Mongoid::Contexts::Enumerable.new(@criteria)
+    end
+
+    it "executes the criteria" do
+      acc = []
+      @context.iterate do |doc|
+        acc << doc
+      end
+      acc.should == [@melbourne]
+    end
+  end
+
   describe "#last" do
 
     it "returns the last matching in the enumerable" do

data/spec/unit/mongoid/contexts/mongo_spec.rb

@@ -27,6 +27,43 @@ describe Mongoid::Contexts::Mongo do
 
   end
 
+  describe "blank?" do
+
+    before do
+      @criteria = Mongoid::Criteria.new(Game)
+      @criteria.where(:test => 'Testing')
+      @context = Mongoid::Contexts::Mongo.new(@criteria)
+    end
+
+    context "when a document exists" do
+
+      before do
+        @doc = mock
+        @collection = mock
+        Game.expects(:collection).returns(@collection)
+        @collection.expects(:find_one).with({ :test => "Testing" }, { :fields => [ :_id ] }).returns(@doc)
+      end
+
+      it "returns false" do
+        @context.blank?.should be_false
+      end
+    end
+
+    context "when a document does not exist" do
+
+      before do
+        @doc = mock
+        @collection = mock
+        Game.expects(:collection).returns(@collection)
+        @collection.expects(:find_one).with({ :test => "Testing" }, { :fields => [ :_id ] }).returns(nil)
+      end
+
+      it "returns true" do
+        @context.blank?.should be_true
+      end
+    end
+  end
+
   describe "#count" do
 
     before do
@@ -214,6 +251,65 @@ describe Mongoid::Contexts::Mongo do
     end
   end
 
+  describe "#iterate" do
+    before do
+      @criteria = Mongoid::Criteria.new(Person)
+      @context = Mongoid::Contexts::Mongo.new(@criteria)
+      @person = Person.new(:title => "Sir")
+      @cursor = stub('cursor')
+      @cursor.stubs(:each).yields(@person)
+    end
+
+    context "when not caching" do
+
+      it "executes the criteria" do
+        @context.expects(:execute).returns(@cursor)
+        @context.iterate do |person|
+          person.should == @person
+        end
+
+      end
+
+    end
+
+    context "when caching" do
+      before do
+        @criteria.cache
+      end
+
+      it "executes the criteria" do
+        @context.expects(:execute).returns(@cursor)
+        @context.iterate do |person|
+          person.should == @person
+        end
+      end
+
+      it "executes only once and it caches the result" do
+        @context.expects(:execute).once.returns(@cursor)
+        @context.iterate do |person|
+          person.should == @person
+        end
+        @context.iterate do |person|
+          person.should == @person
+        end
+      end
+
+      it "executes even if there is no block" do
+        @context.expects(:execute).once.returns(@cursor)
+        @context.iterate
+      end
+
+      it "caches even if there is no block" do
+        @context.expects(:execute).once.returns(@cursor)
+        @context.iterate
+        @context.iterate do |person|
+          person.should == @person
+        end
+      end
+    end
+
+  end
+
   describe "#last" do
 
     before do

data/spec/unit/mongoid/criteria_spec.rb

@@ -127,10 +127,10 @@ describe Mongoid::Criteria do
       @criteria.instance_variable_set(:@context, @context)
     end
 
-    context "when the count is 0" do
+    context "when the context is blank" do
 
       before do
-        @context.expects(:count).returns(0)
+        @context.expects(:blank?).returns(true)
       end
 
       it "returns true" do
@@ -138,10 +138,10 @@ describe Mongoid::Criteria do
       end
     end
 
-    context "when the count is greater than 0" do
+    context "when the context is not blank" do
 
       before do
-        @context.expects(:count).returns(10)
+        @context.expects(:blank?).returns(false)
       end
 
       it "returns false" do
@@ -274,6 +274,13 @@ describe Mongoid::Criteria do
       @cursor = stub(:count => 10)
     end
 
+    it "delegates to the context#iterate" do
+      @context = stub('context')
+      @criteria.stubs(:context).returns(@context)
+      @context.expects(:iterate)
+      @criteria.each
+    end
+
     context "when the criteria has not been executed" do
 
       before do

data/spec/unit/mongoid/document_spec.rb

@@ -499,6 +499,17 @@ describe Mongoid::Document do
 
   end
 
+  describe "#persisted?" do
+
+    before do
+      @person = Person.new
+    end
+
+    it "delegates to new_record?" do
+      @person.persisted?.should be_false
+    end
+  end
+
   describe "#_parent" do
 
     before do

data/spec/unit/mongoid/extensions/big_decimal/conversions_spec.rb

@@ -0,0 +1,22 @@
+require "spec_helper"
+
+describe Mongoid::Extensions::BigDecimal::Conversions do
+
+  let(:number) do
+    BigDecimal.new("123456.789")
+  end
+
+  describe "#get" do
+
+    it "converts the string to a big decimal" do
+      BigDecimal.get("123456.789").should == number
+    end
+  end
+
+  describe "#set" do
+
+    it "converts the big decimal to a string" do
+      BigDecimal.set(number).should == "123456.789"
+    end
+  end
+end

data/spec/unit/mongoid/extensions/binary/conversions_spec.rb

@@ -0,0 +1,22 @@
+require "spec_helper"
+
+describe Mongoid::Extensions::Binary::Conversions do
+
+  let(:bin) do
+    Binary.new
+  end
+
+  describe "#get" do
+
+    it "returns self" do
+      Binary.get(bin).should == bin
+    end
+  end
+
+  describe "#set" do
+
+    it "returns self" do
+      Binary.set(bin).should == bin
+    end
+  end
+end

data/spec/unit/mongoid/{enslavement_spec.rb → extras_spec.rb}

@@ -1,10 +1,63 @@
 require "spec_helper"
 
-describe Mongoid::Enslavement do
+describe Mongoid::Extras do
 
   before do
     @klass = Class.new do
-      include Mongoid::Enslavement
+      include Mongoid::Extras
+    end
+  end
+
+  describe ".cache" do
+
+    before do
+      @klass.cache
+    end
+
+    it "sets the cached boolean on the class" do
+      @klass.cached.should be_true
+    end
+
+  end
+
+  describe ".cached" do
+
+    it "defaults to false" do
+      @klass.cached.should be_false
+    end
+  end
+
+  describe ".cached?" do
+
+    context "when the class is cached" do
+
+      before do
+        @klass.cache
+      end
+
+      it "returns true" do
+        @klass.should be_cached
+      end
+    end
+
+    context "when the class is not cached" do
+
+      it "returns false" do
+        @klass.should_not be_cached
+      end
+    end
+
+  end
+
+  describe "#cached?" do
+
+    before do
+      @klass.cache
+      @doc = @klass.new
+    end
+
+    it "returns the class cached? value" do
+      @doc.should be_cached
     end
   end