mongoid 0.4.5 → 0.4.7

data/spec/unit/mongoid/document_spec.rb

@@ -14,21 +14,6 @@ describe Mongoid::Document do
     @collection = nil
   end
 
-  describe "#aggregate" do
-
-    before do
-      @reduce = "function(obj, prev) { prev.count++; }"
-    end
-
-    it "returns documents grouped by the supplied fields" do
-      results = [{ "title" => "Sir", "count" => 30 }]
-      @collection.expects(:group).with([:title], nil, {:count => 0}, @reduce).returns(results)
-      grouped = Person.aggregate([:title], {})
-      grouped.first["count"].should == 30
-    end
-
-  end
-
   describe "#all" do
 
     before do
@@ -58,34 +43,6 @@ describe Mongoid::Document do
 
   end
 
-  describe "#association=" do
-
-    before do
-      @name = Name.new
-      @person = Person.new
-    end
-
-    it "parentizes the association" do
-      @person.name = @name
-      @name.parent.should == @person
-    end
-
-  end
-
-  describe "#belongs_to" do
-
-    it "creates a reader for the association" do
-      address = Address.new
-      address.should respond_to(:person)
-    end
-
-    it "creates a writer for the association" do
-      address = Address.new
-      address.should respond_to(:person=)
-    end
-
-  end
-
   describe "#collection" do
 
     before do
@@ -221,124 +178,58 @@ describe Mongoid::Document do
 
   end
 
-  describe "#group_by" do
-
-    before do
-      @reduce = "function(obj, prev) { prev.group.push(obj); }"
-    end
-
-    it "returns documents grouped by the supplied fields" do
-      results = [{ "title" => "Sir", "group" => [{ "title" => "Sir", "age" => 30 }] }]
-      @collection.expects(:group).with([:title], nil, { :group => [] }, @reduce).returns(results)
-      grouped = Person.group_by([:title], {})
-      people = grouped.first["group"]
-      people.first.should be_a_kind_of(Person)
-    end
-
-  end
-
-  describe "#has_many" do
+  describe "#index" do
 
-    it "adds a new Association to the collection" do
-      person = Person.new
-      person.addresses.should_not be_nil
-    end
+    context "when unique options are not provided" do
 
-    it "creates a reader for the association" do
-      person = Person.new
-      person.should respond_to(:addresses)
-    end
+      it "delegates to collection with unique => false" do
+        @collection.expects(:create_index).with(:title, :unique => false)
+        Person.index :title
+      end
 
-    it "creates a writer for the association" do
-      person = Person.new
-      person.should respond_to(:addresses=)
     end
 
-    context "when setting the association directly" do
-
-      before do
-        @attributes = { :title => "Sir",
-          :addresses => [
-            { :street => "Street 1" },
-            { :street => "Street 2" } ] }
-        @person = Person.new(@attributes)
-      end
+    context "when unique option is provided" do
 
-      it "sets the attributes for the association" do
-        address = Address.new(:street => "New Street")
-        @person.addresses = [address]
-        @person.addresses.first.street.should == "New Street"
+      it "delegates to collection with unique option" do
+        @collection.expects(:create_index).with(:title, :unique => true)
+        Person.index :title, :unique => true
       end
 
     end
 
   end
 
-  describe "#has_one" do
+  describe "#key" do
 
-    it "adds a new Association to the collection" do
-      person = Person.new
-      person.name.should_not be_nil
-    end
-
-    it "creates a reader for the association" do
-      person = Person.new
-      person.should respond_to(:name)
-    end
-
-    it "creates a writer for the association" do
-      person = Person.new
-      person.should respond_to(:name=)
-    end
-
-    context "when setting the association directly" do
+    context "when key is single field" do
 
       before do
-        @attributes = { :title => "Sir",
-          :name => { :first_name => "Test" } }
-        @person = Person.new(@attributes)
+        Address.key :street
+        @address = Address.new(:street => "Testing Street Name")
+        @address.expects(:collection).returns(@collection)
+        @collection.expects(:save)
       end
 
-      it "sets the attributes for the association" do
-        name = Name.new(:first_name => "New Name")
-        @person.name = name
-        @person.name.first_name.should == "New Name"
+      it "adds the callback for primary key generation" do
+        @address.save
+        @address.id.should == "testing-street-name"
       end
 
     end
 
-  end
-
-  describe "#has_timestamps" do
+    context "when key is composite" do
 
-    before do
-      @tester = Tester.new
-    end
-
-    it "adds created_on and last_modified to the document" do
-      fields = Tester.instance_variable_get(:@fields)
-      fields[:created_at].should_not be_nil
-      fields[:last_modified].should_not be_nil
-    end
-
-  end
-
-  describe "#index" do
-
-    context "when unique options are not provided" do
-
-      it "delegates to collection with unique => false" do
-        @collection.expects(:create_index).with(:title, :unique => false)
-        Person.index :title
+      before do
+        Address.key :street, :zip
+        @address = Address.new(:street => "Testing Street Name", :zip => "94123")
+        @address.expects(:collection).returns(@collection)
+        @collection.expects(:save)
       end
 
-    end
-
-    context "when unique option is provided" do
-
-      it "delegates to collection with unique option" do
-        @collection.expects(:create_index).with(:title, :unique => true)
-        Person.index :title, :unique => true
+      it "combines all fields" do
+        @address.save
+        @address.id.should == "testing-street-name-94123"
       end
 
     end
@@ -369,6 +260,35 @@ describe Mongoid::Document do
 
     end
 
+    context "with a primary key" do
+
+      context "when the value for the key exists" do
+
+        before do
+          Address.key :street
+          @address = Address.new(:street => "Test")
+        end
+
+        it "sets the primary key" do
+          @address.id.should == "test"
+        end
+
+      end
+
+      context "when the value for the key is nonexistant" do
+
+        before do
+          @address = Address.new(:street => "Test")
+        end
+
+        it "sets the primary key" do
+          @address.id.should be_nil
+        end
+
+      end
+
+    end
+
   end
 
   describe "#new_record?" do
@@ -402,32 +322,33 @@ describe Mongoid::Document do
   describe "#paginate" do
 
     before do
-      @cursor = stub(:count => 100, :collect => [])
+      @criteria = stub(:page => 1, :offset => "20")
     end
 
     context "when pagination parameters are passed" do
 
+      before do
+        @params = { :conditions => { :test => "Test" }, :page => 2, :per_page => 20 }
+      end
+
       it "delegates to will paginate with the results" do
-        @collection.expects(:find).with({ :test => "Test" }, { :sort => nil, :limit => 20, :skip => 20}).returns(@cursor)
-        Person.paginate(:conditions => { :test => "Test" }, :page => 2, :per_page => 20)
+        Mongoid::Criteria.expects(:translate).with(:all, @params).returns(@criteria)
+        @criteria.expects(:execute).with(Person).returns([])
+        Person.paginate(@params)
       end
 
     end
 
     context "when pagination parameters are not passed" do
 
-      it "delegates to will paginate with default values" do
-        @collection.expects(:find).with({ :test => "Test" }, { :sort => nil, :limit => 20, :skip => 0}).returns(@cursor)
-        Person.paginate(:conditions => { :test => "Test" })
+      before do
+        @params = { :conditions => { :test => "Test" }}
       end
 
-    end
-
-    context "when sorting paramters provided" do
-
-      it "adds the sorting parameters in the collection#find" do
-        @collection.expects(:find).with({ :test => "Test" }, { :sort => { :test => -1}, :limit => 20, :skip => 0}).returns(@cursor)
-        Person.paginate(:conditions => { :test => "Test" }, :sort => { :test => -1 })
+      it "delegates to will paginate with default values" do
+        Mongoid::Criteria.expects(:translate).with(:all, @params).returns(@criteria)
+        @criteria.expects(:execute).with(Person).returns([])
+        Person.paginate(:conditions => { :test => "Test" })
       end
 
     end
@@ -515,6 +436,45 @@ describe Mongoid::Document do
 
   end
 
+  describe "#write_attributes" do
+
+    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 ==
+            { :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 == [{ :street => "Test2" }]
+        end
+
+      end
+
+    end
+
+  end
+
   context "validations" do
 
     context "when defining using macros" do

data/spec/unit/mongoid/extensions/array/parentization_spec.rb

@@ -11,10 +11,12 @@ describe Mongoid::Extensions::Array::Parentization do
     end
 
     it "sets the parent on each element" do
+      @child.expects(:add_observer).with(@parent)
       @child.expects(:parent=).with(@parent)
-      @array.parentize(@parent)
+      @child.expects(:association_name=).with(:child)
+      @array.parentize(@parent, :child)
     end
 
   end
 
-end
+end

data/spec/unit/mongoid/extensions/hash/accessors_spec.rb

@@ -0,0 +1,87 @@
+require File.join(File.dirname(__FILE__), "/../../../../spec_helper.rb")
+
+describe Mongoid::Extensions::Hash::Accessors do
+
+  describe "#insert" do
+
+    before do
+      @hash = {
+        :_id => 1,
+        :title => "value",
+        :name => {
+          :_id => 2, :first_name => "Test", :last_name => "User"
+        },
+        :addresses => [
+          { :_id => 3, :street => "First Street" },
+          { :_id => 4, :street => "Second Street" }
+        ]
+      }
+    end
+
+    context "when writing a single attribute" do
+
+      context "when attribute exists" do
+
+        before do
+          @new = { :_id => 2, :first_name => "Test2", :last_name => "User2" }
+        end
+
+        it "updates the existing attribute" do
+          @hash.insert(:name, @new)
+          @hash[:name].should == @new
+        end
+
+      end
+
+      context "when attribute does not exist" do
+
+        before do
+          @hash.delete(:name)
+          @new = { :_id => 2, :first_name => "Test2", :last_name => "User2" }
+        end
+
+        it "updates the existing attribute" do
+          @hash.insert(:name, @new)
+          @hash[:name].should == @new
+        end
+
+      end
+
+    end
+
+    context "when writing to an array of attributes" do
+
+      context "when matching attribute exists" do
+
+        before do
+          @new = { :_id => 3, :street => "New Street" }
+        end
+
+        it "updates the matching attributes" do
+          @hash.insert(:addresses, @new)
+          @hash[:addresses].should include({:street => "New Street", :_id => 3})
+        end
+
+      end
+
+      context "when matching attribute does not exist" do
+
+        before do
+          @new = { :street => "New Street" }
+        end
+
+        it "updates the matching attributes" do
+          @hash.insert(:addresses, @new)
+          @hash[:addresses].should == [
+            { :_id => 3, :street => "First Street" },
+            { :_id => 4, :street => "Second Street" },
+            { :street => "New Street" }
+          ]
+        end
+      end
+
+    end
+
+  end
+
+end

data/spec/unit/mongoid/extensions/object/parentization_spec.rb

@@ -10,10 +10,10 @@ describe Mongoid::Extensions::Object::Parentization do
     end
 
     it "sets the parent on each element" do
-      @child.parentize(@parent)
+      @child.parentize(@parent, :child)
       @child.parent.should == @parent
     end
 
   end
 
-end
+end