mongoid 0.9.8 → 0.9.9

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

@@ -209,6 +209,23 @@ describe Mongoid::Associations::HasMany do
 
   end
 
+  describe "#nested_build" do
+
+    before do
+      @association = Mongoid::Associations::HasMany.new(
+        @document,
+        Mongoid::Associations::Options.new(:name => :addresses)
+      )
+    end
+
+    it "returns the newly built object in the association" do
+      @association.nested_build({ "0" => { :street => "Yet Another" } })
+      @association.size.should == 3
+      @association.last.street.should == "Yet Another"
+    end
+
+  end
+
   describe "#push" do
 
     before do

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

@@ -0,0 +1,130 @@
+require "spec_helper"
+
+describe Mongoid::Associations::HasOneRelated do
+
+  let(:document) { stub(:id => "1") }
+  let(:options) { Mongoid::Associations::Options.new(:name => :game) }
+
+  describe "#build" do
+
+    before do
+      @parent = stub(:id => "5", :class => Person)
+      Game.expects(:first).returns(nil)
+      @association = Mongoid::Associations::HasOneRelated.new(@parent, options)
+    end
+
+    it "adds a new object to the association" do
+      @association.build(:score => 100)
+      @association.score.should == 100
+    end
+
+    it "sets the parent object id on the child" do
+      @association.build(:score => 100)
+      @association.person_id.should == @parent.id
+    end
+
+  end
+
+  describe "#create" do
+
+    before do
+      @parent = stub(:id => "5", :class => Person)
+      Game.expects(:first).returns(nil)
+      Mongoid::Commands::Save.expects(:execute)
+      @association = Mongoid::Associations::HasOneRelated.new(@parent, options)
+    end
+
+    it "adds a new object to the association" do
+      @association.create(:score => 100)
+      @association.score.should == 100
+    end
+
+    it "sets the parent object id on the child" do
+      @association.create(:score => 100)
+      @association.person_id.should == @parent.id
+    end
+
+  end
+
+  describe ".initialize" do
+
+    before do
+      @person = Person.new
+      @game = stub
+    end
+
+    it "finds the association game by the parent key" do
+      Game.expects(:first).with(:conditions => { "person_id"=> @person.id }).returns(@game)
+      @person.game.should == @game
+    end
+
+  end
+
+  describe ".instantiate" do
+
+    it "delegates to new" do
+      Mongoid::Associations::HasOneRelated.expects(:new).with(document, options)
+      Mongoid::Associations::HasOneRelated.instantiate(document, options)
+    end
+
+  end
+
+  describe "#method_missing" do
+
+    before do
+      @person = Person.new
+      @game = stub
+    end
+
+    it "delegates to the documet" do
+      Game.expects(:first).with(:conditions => { "person_id"=> @person.id }).returns(@game)
+      @game.expects(:strange_method)
+      association = Mongoid::Associations::HasOneRelated.instantiate(@person, options)
+      association.strange_method
+    end
+
+  end
+
+  describe ".macro" do
+
+    it "returns :has_one_related" do
+      Mongoid::Associations::HasOneRelated.macro.should == :has_one_related
+    end
+
+  end
+
+  describe "#nil?" do
+
+    before do
+      @person = Person.new
+      @game = stub
+      Game.expects(:first).with(:conditions => { "person_id"=> @person.id }).returns(nil)
+    end
+
+    it "delegates to the document" do
+      association = Mongoid::Associations::HasOneRelated.instantiate(@person, options)
+      association.should be_nil
+    end
+
+  end
+
+  describe ".update" do
+
+    before do
+      @person = Person.new
+      @game = stub
+    end
+
+    it "sets the parent on the child association" do
+      @game.expects(:person=).with(@person)
+      Mongoid::Associations::HasOneRelated.update(@game, @person, options)
+    end
+
+    it "returns the child" do
+      @game.expects(:person=).with(@person)
+      Mongoid::Associations::HasOneRelated.update(@game, @person, options).should == @game
+    end
+
+  end
+
+end

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

@@ -80,6 +80,27 @@ describe Mongoid::Associations::HasOne do
 
   end
 
+  describe "#nested_build" do
+
+    context "when attributes provided" do
+
+      before do
+        @association = Mongoid::Associations::HasOne.new(
+          @document,
+          @attributes[:mixed_drink],
+          Mongoid::Associations::Options.new(:name => :mixed_drink)
+        )
+      end
+
+      it "replaces the existing has_one" do
+        drink = @association.nested_build({ :name => "Sapphire and Tonic" })
+        drink.name.should == "Sapphire and Tonic"
+      end
+
+    end
+
+  end
+
   describe ".instantiate" do
 
     context "when attributes exist" do
@@ -129,4 +150,36 @@ describe Mongoid::Associations::HasOne do
 
   end
 
+  describe "#valid?" do
+
+    context "when the document is nil" do
+
+      before do
+        @document = stub(:attributes => {})
+        @options = Mongoid::Associations::Options.new(:name => :name)
+        @association = Mongoid::Associations::HasOne.instantiate(@document, @options)
+      end
+
+      it "returns false" do
+        @association.valid?.should be_false
+      end
+
+    end
+
+    context "when the document is not nil" do
+
+      before do
+        @document = stub(:attributes => { :name => { :first_name => "Test" } }, :update => true)
+        @options = Mongoid::Associations::Options.new(:name => :name)
+        @association = Mongoid::Associations::HasOne.instantiate(@document, @options)
+      end
+
+      it "validates the document" do
+        @association.valid?.should be_true
+      end
+
+    end
+
+  end
+
 end

data/spec/unit/mongoid/associations_spec.rb

@@ -97,6 +97,14 @@ describe Mongoid::Associations do
       code.phone_number.should == phone_number
     end
 
+    context "when inverse_of not supplied" do
+
+      it "raises an error" do
+        lambda { Person.class_eval { belongs_to :nothing } }.should raise_error
+      end
+
+    end
+
     context "when navigating the graph" do
 
       before do
@@ -241,20 +249,39 @@ describe Mongoid::Associations do
 
   end
 
-  describe ".relates_to_one" do
+  describe ".belongs_to_related" do
+
+    before do
+      @game = Game.new
+    end
 
     it "creates an id field for the relationship" do
-      Person.new.should respond_to(:game_id)
+      @game.should respond_to(:person_id)
     end
 
-    it "creates a getter and setter for the relationship" do
-      Person.new.should respond_to(:game)
-      Person.new.should respond_to(:game=)
+    it "creates a getter for the parent" do
+      @game.should respond_to(:person)
+    end
+
+  end
+
+  describe ".has_one_related" do
+
+    before do
+      @person = Person.new
+    end
+
+    it "creates a getter for the relationship" do
+      @person.should respond_to(:game)
+    end
+
+    it "creates a setter for the relationship" do
+      @person.should respond_to(:game=)
     end
 
   end
 
-  describe ".relates_to_many" do
+  describe ".has_many_related" do
 
     it "creates a getter and setter for the relationship" do
       Person.new.should respond_to(:posts)
@@ -274,7 +301,7 @@ describe Mongoid::Associations do
       end
 
       it "saves each association" do
-        @related.expects(:quick_save).returns(@related)
+        @related.expects(:save).returns(@related)
         @person.update_associations(:posts)
       end
 
@@ -287,6 +314,7 @@ describe Mongoid::Associations do
       end
 
       it "does nothing" do
+        Post.expects(:find).returns([])
         @person.update_associations(:posts)
         @person.posts.first.should be_nil
       end
@@ -306,7 +334,7 @@ describe Mongoid::Associations do
       end
 
       it "saves each association" do
-        @related.expects(:quick_save).returns(@related)
+        @related.expects(:save).returns(@related)
         @person.update_association(:game)
       end
 

data/spec/unit/mongoid/attributes_spec.rb

@@ -2,6 +2,78 @@ require "spec_helper"
 
 describe Mongoid::Attributes do
 
+  describe ".accepts_nested_attributes_for" do
+
+    before do
+      @person = Person.new
+    end
+
+    it "adds a setter for the association attributes" do
+      @person.should respond_to(:addresses_attributes=)
+    end
+
+    describe "#association_attributes=" do
+
+      context "on a has many association" do
+
+        context "when association is empty" do
+
+          before do
+            @attributes = {
+              "0" => { "street" => "Folsom", "city" => "San Francisco" }
+            }
+            @person.addresses_attributes = @attributes
+          end
+
+          it "adds a new document to the association" do
+            address = @person.addresses.first
+            address.street.should == "Folsom"
+            address.city.should == "San Francisco"
+          end
+
+        end
+
+        context "when association is not empty" do
+
+          before do
+            @person = Person.new
+            @person.addresses.build(:street => "Broadway", :city => "New York")
+            @attributes = {
+              "0" => { "street" => "Folsom", "city" => "San Francisco" }
+            }
+            @person.addresses_attributes = @attributes
+          end
+
+          it "updates the existing attributes on the association" do
+            @person.addresses.size.should == 2
+          end
+
+        end
+
+      end
+
+      context "on a has one association" do
+
+        before do
+          @person = Person.new
+          @attributes = {
+            "first_name" => "Fernando", "last_name" => "Torres"
+          }
+          @person.name_attributes = @attributes
+        end
+
+        it "replaces the document on the association" do
+          name = @person.name
+          name.first_name.should == "Fernando"
+          name.last_name.should == "Torres"
+        end
+
+      end
+
+    end
+
+  end
+
   describe "#process" do
 
     context "when supplied hash has values" do
@@ -94,4 +166,150 @@ describe Mongoid::Attributes do
 
   end
 
+  describe "#read_attribute" do
+
+    context "when attribute does not exist" do
+
+      before do
+        @person = Person.new
+      end
+
+      it "returns the default value" do
+        @person.age.should == 100
+      end
+
+    end
+
+  end
+
+  describe "#remove_attribute" do
+
+    context "when the attribute exists" do
+
+      it "removes the attribute" do
+        person = Person.new(:title => "Sir")
+        person.remove_attribute(:title)
+        person.title.should be_nil
+      end
+
+    end
+
+    context "when the attribute does not exist" do
+
+      it "does nothing" do
+        person = Person.new
+        person.remove_attribute(:title)
+        person.title.should be_nil
+      end
+
+    end
+
+  end
+
+  describe "#write_attribute" do
+
+    context "when attribute does not exist" do
+
+      before do
+        @person = Person.new
+      end
+
+      it "returns the default value" do
+        @person.age = nil
+        @person.age.should == 100
+      end
+
+    end
+
+    context "when field has a default value" do
+
+      before do
+        @person = Person.new
+      end
+
+      it "should allow overwriting of the default value" do
+        @person.terms = true
+        @person.terms.should be_true
+      end
+
+    end
+
+  end
+
+  describe "#write_attributes" do
+
+    context "typecasting" do
+
+      before do
+        @person = Person.new
+        @attributes = { :age => "50" }
+      end
+
+      it "properly casts values" do
+        @person.write_attributes(@attributes)
+        @person.age.should == 50
+      end
+
+    end
+
+    context "on a parent document" do
+
+      context "when the parent has a has many through a has one" do
+
+        before do
+          @owner = PetOwner.new(:title => "Mr")
+          @pet = Pet.new(:name => "Fido")
+          @owner.pet = @pet
+          @vet_visit = VetVisit.new(:date => Date.today)
+          @pet.vet_visits = [@vet_visit]
+        end
+
+        it "does not overwrite child attributes if not in the hash" do
+          @owner.write_attributes({ :pet => { :name => "Bingo" } })
+          @owner.pet.name.should == "Bingo"
+          @owner.pet.vet_visits.size.should == 1
+        end
+
+      end
+
+    end
+
+    context "on a child document" do
+
+      context "when child is part of a has one" do
+
+        before do
+          @person = Person.new(:title => "Sir", :age => 30)
+          @name = Name.new(:first_name => "Test", :last_name => "User")
+          @person.name = @name
+        end
+
+        it "sets the child attributes on the parent" do
+          @name.write_attributes(:first_name => "Test2", :last_name => "User2")
+          @person.attributes[:name].should ==
+            { "_id" => "test-user", "first_name" => "Test2", "last_name" => "User2" }
+        end
+
+      end
+
+      context "when child is part of a has many" do
+
+        before do
+          @person = Person.new(:title => "Sir")
+          @address = Address.new(:street => "Test")
+          @person.addresses << @address
+        end
+
+        it "updates the child attributes on the parent" do
+          @address.write_attributes("street" => "Test2")
+          @person.attributes[:addresses].should ==
+            [ { "_id" => "test", "street" => "Test2" } ]
+        end
+
+      end
+
+    end
+
+  end
+
 end