mongo_mapper 0.9.2 → 0.10.0

data/test/functional/associations/test_many_documents_proxy.rb

@@ -93,6 +93,144 @@ class ManyDocumentsProxyTest < Test::Unit::TestCase
         project.statuses[0].name.should == "ready"
       end
     end
+    
+    context "with :dependent" do
+      setup do
+        @broker_class = Doc('Broker')
+        @property_class = Doc('Property') do
+          key :broker_id, ObjectId
+          belongs_to :broker
+        end
+      end
+      
+      context "=> destroy" do
+        setup do
+          @broker_class.many :properties, :class => @property_class, :dependent => :destroy
+    
+          @broker = @broker_class.create(:name => "Bob")
+          @property1 = @property_class.create
+          @property2 = @property_class.create
+          @property3 = @property_class.create
+          @broker.properties << @property1
+          @broker.properties << @property2
+          @broker.properties << @property3
+        end
+            
+        should "call destroy the existing documents" do
+          @broker.properties[0].expects(:destroy).once
+          @broker.properties[1].expects(:destroy).once
+          @broker.properties[2].expects(:destroy).once
+          @broker.properties = [@property_class.new]
+        end
+        
+        should "remove the existing document from the database" do
+          @property_class.count.should == 3
+          @broker.properties = []
+          @property_class.count.should == 0
+        end
+        
+        should "skip over documents that are the same" do
+          @broker.properties[0].expects(:destroy).never
+          @broker.properties[1].expects(:destroy).once
+          @broker.properties[2].expects(:destroy).never
+          @broker.properties = [@property3, @property1]
+        end
+      end
+      
+      context "=> delete_all" do
+        setup do
+          @broker_class.many :properties, :class => @property_class, :dependent => :delete_all
+    
+          @broker = @broker_class.create(:name => "Bob")
+          @property1 = @property_class.create
+          @property2 = @property_class.create
+          @property3 = @property_class.create
+          @broker.properties << @property1
+          @broker.properties << @property2
+          @broker.properties << @property3
+        end
+            
+        should "call delete the existing documents" do
+          @broker.properties[0].expects(:delete).once
+          @broker.properties[1].expects(:delete).once
+          @broker.properties[2].expects(:delete).once
+          @broker.properties = [@property_class.new]
+        end
+        
+        should "remove the existing document from the database" do
+          @property_class.count.should == 3
+          @broker.properties = []
+          @property_class.count.should == 0
+        end
+        
+        should "skip over documents that are the same" do
+          @broker.properties[0].expects(:delete).never
+          @broker.properties[1].expects(:delete).once
+          @broker.properties[2].expects(:delete).never
+          @broker.properties = [@property3, @property1]
+        end
+      end
+      
+      context "=> nullify" do
+        setup do
+          @broker_class.many :properties, :class => @property_class, :dependent => :nullify
+    
+          @broker = @broker_class.create(:name => "Bob")
+          @property1 = @property_class.create
+          @property2 = @property_class.create
+          @property3 = @property_class.create
+          @broker.properties << @property1
+          @broker.properties << @property2
+          @broker.properties << @property3
+        end
+            
+        should "nullify the existing documents" do
+          @property1.reload.broker_id.should == @broker.id
+          @property2.reload.broker_id.should == @broker.id
+          @property3.reload.broker_id.should == @broker.id
+
+          @broker.properties = [@property_class.new]
+          
+          @property1.reload.broker_id.should be_nil
+          @property2.reload.broker_id.should be_nil
+          @property3.reload.broker_id.should be_nil
+        end
+        
+        should "skip over documents that are the same" do
+          @broker.properties = [@property3, @property1]
+
+          @property1.reload.broker_id.should == @broker.id
+          @property2.reload.broker_id.should be_nil
+          @property3.reload.broker_id.should == @broker.id
+        end
+        
+        should "work" do
+          old_properties = @broker.properties
+          @broker.properties = [@property1, @property2, @property3]
+          old_properties.should == @broker.properties
+        end
+      end
+      
+      context "unspecified" do
+        should "nullify the existing documents" do
+          @broker_class.many :properties, :class => @property_class
+    
+          @broker = @broker_class.create(:name => "Bob")
+          @property1 = @property_class.create
+          @property2 = @property_class.create
+          @property3 = @property_class.create
+          @broker.properties << @property1
+          @broker.properties << @property2
+          @broker.properties << @property3
+
+          @broker.properties = [@property_class.new]
+          
+          @property1.reload.broker_id.should be_nil
+          @property2.reload.broker_id.should be_nil
+          @property3.reload.broker_id.should be_nil
+        end
+      end
+    end
   end
 
   context "using <<, push and concat" do
@@ -624,6 +762,29 @@ class ManyDocumentsProxyTest < Test::Unit::TestCase
         Property.count.should == 3
       end
     end
+    
+    context "unspecified" do
+      setup do
+        Property.key :thing_id, ObjectId
+        Property.belongs_to :thing
+        Thing.has_many :properties, :dependent => :nullify
+
+        @thing = Thing.create(:name => "Tree")
+        @property1 = Property.create
+        @property2 = Property.create
+        @property3 = Property.create
+        @thing.properties << @property1
+        @thing.properties << @property2
+        @thing.properties << @property3
+      end
+
+      should "should nullify relationship but not destroy associated documents" do
+        @thing.properties.count.should == 3
+        @thing.destroy
+        @thing.properties.count.should == 0
+        Property.count.should == 3
+      end
+    end
   end
 
   context "namespaced foreign keys" do

data/test/functional/associations/test_one_as_proxy.rb

@@ -0,0 +1,485 @@
+require 'test_helper'
+require 'models'
+
+class OneAsProxyTest < Test::Unit::TestCase
+  def setup
+    @post_class  = Doc('Post')
+    @author_class = Doc('Author')
+  end
+
+  should "default to nil" do
+    @post_class.one :author, :as => :authorable, :class => @author_class
+    @post_class.new.author.nil?.should be_true
+  end
+
+  should "return nil instead of a proxy" do
+    @post_class.one :author, :as => :authorable, :class => @author_class
+    nil.should === @post_class.new.author
+  end
+
+  should "allow assignment of associated document using a hash" do
+    @post_class.one :author, :as => :authorable, :class => @author_class
+
+    post = @post_class.new('author' => { 'name' => 'Frank' })
+    post.author.name.should == 'Frank'
+
+    post.save.should be_true
+    post.reload
+
+    post.author.name.should == 'Frank'
+  end
+
+  should "only return a document with the correct type" do
+    @post_class.one :author, :as => :authorable, :class => @author_class
+    post = @post_class.create
+
+    author = @author_class.create(:authorable_id => post.id, :authorable_type => "Nada")
+    post.reload.author.should be_nil
+
+    author = @author_class.create(:authorable_id => post.id, :authorable_type => "Post")
+    post.reload.author.should == author
+  end
+
+  context "replacing the association" do
+    context "with an object of the class" do
+      setup do
+        @post_class.one :author, :as => :authorable, :class => @author_class
+        @post = @post_class.new
+        @author = @author_class.new(:name => 'Frank')
+      end
+
+      should "work" do
+        @post.author = @author
+        @post.reload
+
+        @post.author.should == @author
+        @post.author.nil?.should be_false
+
+        new_author = @author_class.new(:name => 'Emily')
+        @post.author = new_author
+        @post.author.should == new_author
+      end
+
+      should "generate a new proxy instead of modifying the existing one" do
+        @post.author = @author
+        @post.reload
+
+        @post.author.should == @author
+        @post.author.nil?.should be_false
+
+        original_author = @post.author
+        original_author.name.should == 'Frank'
+        new_author = @author_class.new(:name => 'Emily')
+        @post.author = new_author
+        @post.author.should == new_author
+
+        original_author.name.should == 'Frank'
+      end
+
+      should "assign foreign key" do
+        @post.author = @author
+        @post.author.authorable_id.should == @post.id
+      end
+
+      should "assign _type" do
+        @post.author = @author
+        @post.author.authorable_type.should == "Post"
+      end
+    end
+
+    context "with a Hash" do
+      setup do
+        @post_class.one :author, :as => :authorable, :class => @author_class
+        @post = @post_class.new
+      end
+
+      should "convert to an object of the class and work" do
+        @post.author = {'name' => 'Frank'}
+        @post.reload
+
+        @post.author.name.should == 'Frank'
+        @post.author.nil?.should be_false
+
+        @post.author = {'name' => 'Emily'}
+        @post.author.name.should == 'Emily'
+      end
+
+      should "assign foreign key" do
+        @post.author = {'name' => 'Frank'}
+        @post.author.authorable_id.should == @post.id
+      end
+
+      should "assign _type" do
+        @post.author = {'name' => 'Frank'}
+        @post.author.authorable_type.should == "Post"
+      end
+    end
+
+    context "with :dependent" do
+      context "=> delete" do
+        setup do
+          @post_class.one :author, :as => :authorable, :class => @author_class, :dependent => :delete
+
+          @post = @post_class.create
+          @author = @author_class.new
+          @post.author = @author
+        end
+
+        should "call delete on the existing document" do
+          @author_class.any_instance.expects(:delete).once
+          @post.author = @author_class.new
+        end
+
+        should "remove the existing document from the database" do
+          @post.author = @author_class.new
+          lambda { @author.reload }.should raise_error(MongoMapper::DocumentNotFound)
+        end
+
+        should "do nothing if it's the same document" do
+          @author_class.any_instance.expects(:delete).never
+          @post.author = @author
+        end
+      end
+
+      context "=> destory" do
+        setup do
+          @post_class.one :author, :as => :authorable, :class => @author_class, :dependent => :destroy
+
+          @post = @post_class.create
+          @author = @author_class.new
+          @post.author = @author
+        end
+
+        should "call destroy the existing document" do
+          @author_class.any_instance.expects(:destroy).once
+          @post.author = @author_class.new
+        end
+
+        should "remove the existing document from the database" do
+          @post.author = @author_class.new
+          lambda { @author.reload }.should raise_error(MongoMapper::DocumentNotFound)
+        end
+
+        should "do nothing if it's the same document" do
+          @author_class.any_instance.expects(:destroy).never
+          @post.author = @author
+        end
+      end
+
+      context "=> nullify" do
+        setup do
+          @post_class.one :author, :as => :authorable, :class => @author_class, :dependent => :nullify
+
+          @post = @post_class.create
+          @author = @author_class.new
+          @post.author = @author
+        end
+
+        should "nullify the existing document" do
+          @author.reload
+          @author.authorable_id.should == @post.id
+
+          @post.author = @author_class.new
+
+          @author.reload
+          @author.authorable_id.should be_nil
+        end
+
+        should "work when it's the same document" do
+          old_author = @post.author
+          @post.author = @author
+          old_author.should == @post.author
+        end
+
+        should "nullify _type" do
+          @post.author = @author_class.new
+          @author.reload
+          @author.authorable_type.should be_nil
+        end
+      end
+
+      context "unspecified" do
+        setup do
+          @post_class.one :author, :as => :authorable, :class => @author_class
+
+          @post = @post_class.create
+          @author = @author_class.new
+          @post.author = @author
+        end
+
+        should "nullify the existing document" do
+          @author.reload
+          @author.authorable_id.should == @post.id
+
+          @post.author = @author_class.new
+
+          @author.reload
+          @author.authorable_id.should be_nil
+        end
+
+        should "nullify _type" do
+          @post.author = @author_class.new
+          @author.reload
+          @author.authorable_type.should be_nil
+        end
+      end
+    end
+
+    context "with nil" do
+      setup do
+        @post_class.one :author, :as => :authorable, :class => @author_class
+
+        @post = @post_class.new
+        @author = @author_class.new(:name => 'Frank')
+        @post.author = @author
+      end
+
+      should "nullify the existing document" do
+        @post.author = nil
+        @author.reload
+        @author.authorable_id.should be_nil
+      end
+
+      should "set the target to nil" do
+        @post.author = nil
+        @post.author.should == nil
+      end
+
+      should "nullify _type" do
+        @post.author = nil
+        @author.reload
+        @author.authorable_type.should be_nil
+      end
+    end
+  end
+
+  should "have boolean method for testing presence" do
+    @post_class.one :author, :as => :authorable, :class => @author_class
+
+    post = @post_class.new
+    post.author?.should be_false
+
+    post.author = @author_class.new(:name => 'Frank')
+    post.author?.should be_true
+  end
+
+  should "work with criteria" do
+    @post_class.one :primary_author, :as => :authorable, :class => @author_class, :primary => true
+    @post_class.one :author, :as => :authorable, :class => @author_class, :primary => false
+
+    post = @post_class.create
+    author = @author_class.create(:name => 'Frank', :primary => false, :authorable_id => post.id, :authorable_type => 'Post')
+    primary = @author_class.create(:name => 'Bill', :primary => true, :authorable_id => post.id, :authorable_type => 'Post')
+    post.reload
+    post.author.should == author
+    post.primary_author.should == primary
+  end
+
+  should "unset the association" do
+    @post_class.one :author, :as => :authorable, :class => @author_class
+    post = @post_class.create
+    author = @author_class.create
+    post.update_attributes!(:author => author)
+    post.reload
+    post.author = nil
+    post.author.nil?.should be_true
+  end
+
+  context "destroying parent with :dependent" do
+    context "=> destroy" do
+      setup do
+        @post_class.one :author, :as => :authorable, :class => @author_class, :dependent => :destroy
+
+        @post = @post_class.create
+        @author = @author_class.new
+        @post.author = @author
+      end
+
+      should "should call destroy on the associated documents" do
+        @author_class.any_instance.expects(:destroy).once
+        @post.destroy
+      end
+
+      should "should remove the associated documents" do
+        @author_class.count.should == 1
+        @post.destroy
+        @post.author.should == nil
+        @author_class.count.should == 0
+      end
+    end
+
+    context "=> delete" do
+      setup do
+        @post_class.one :author, :as => :authorable, :class => @author_class, :dependent => :delete
+
+        @post = @post_class.create
+        @author = @author_class.new
+        @post.author = @author
+      end
+
+      should "should call delete the associated documents" do
+        @author_class.any_instance.expects(:delete).once
+        @post.destroy
+      end
+
+      should "remove the associated documents" do
+        @author_class.count.should == 1
+        @post.destroy
+        @post.author.should == nil
+        @author_class.count.should == 0
+      end
+    end
+
+    context "=> nullify" do
+      setup do
+        @post_class.one :author, :as => :authorable, :class => @author_class, :dependent => :nullify
+
+        @post = @post_class.create
+        @author = @author_class.new
+        @post.author = @author
+      end
+
+      should "should nullify the relationship but not destroy the associated document" do
+        @author_class.count.should == 1
+        @post.destroy
+        @post.author.should == nil
+        @author_class.count.should == 1
+
+        @author_class.first.should == @author
+        @author.authorable_id.should == nil
+      end
+
+      should "nullify _type" do
+        @post.destroy
+        @author.authorable_type.should be_nil
+      end
+    end
+
+    context "unspecified" do
+      setup do
+        @post_class.one :author, :as => :authorable, :class => @author_class
+
+        @post = @post_class.create
+        @author = @author_class.new
+        @post.author = @author
+      end
+
+      should "should nullify the relationship but not destroy the associated document" do
+        @author_class.count.should == 1
+        @post.destroy
+        @post.author.should == nil
+        @author_class.count.should == 1
+
+        @author_class.first.should == @author
+        @author.authorable_id.should == nil
+      end
+
+      should "nullify _type" do
+        @post.destroy
+        @author.authorable_type.should be_nil
+      end
+    end
+  end
+
+  context "#build" do
+    setup do
+      @post_class.one :author, :as => :authorable, :class => @author_class
+      @post = @post_class.create
+    end
+
+    should "be able to build with attributes" do
+      author = @post.build_author(:name => 'John')
+      @post.author.should be_instance_of(@author_class)
+      @post.author.should be_new
+      @post.author.name.should == 'John'
+      @post.author.should == author
+    end
+
+    should "assign foreign key" do
+      @post.build_author
+      @post.author.authorable_id.should == @post.id
+    end
+
+    should "assign _type" do
+        @post.build_author
+        @post.author.authorable_type.should == 'Post'
+    end
+  end
+
+  context "#create" do
+     setup do
+      @post_class.one :author, :as => :authorable, :class => @author_class
+      @post = @post_class.create
+    end
+
+    should "be able to create" do
+      author = @post.create_author(:name => 'John')
+      @post.author.should be_instance_of(@author_class)
+      @post.author.should_not be_new
+      @post.author.name.should == 'John'
+      @post.author.should == author
+    end
+
+    should "assign foreign key" do
+      @post.create_author
+      @post.author.authorable_id.should == @post.id
+    end
+
+    should "assign _type" do
+      @post.create_author
+      @post.author.authorable_type.should == 'Post'
+    end
+  end
+
+  context "#create!" do
+    setup do
+      @author_class.key :name, String, :required => true
+      @post_class.one :author, :as => :authorable, :class => @author_class
+      @post = @post_class.create
+    end
+
+    should "raise exception if invalid" do
+      assert_raises(MongoMapper::DocumentNotValid) do
+        @post.create_author!
+      end
+    end
+
+    should "work if valid" do
+      author = @post.create_author!(:name => 'John')
+      @post.author.should be_instance_of(@author_class)
+      @post.author.should_not be_new
+      @post.author.name.should == 'John'
+      @post.author.should == author
+    end
+
+    should "assign foreign key if valid" do
+      @post.create_author!(:name => 'John')
+      @post.author.authorable_id.should == @post.id
+    end
+
+    should "assign _type if valid" do
+      @post.create_author!(:name => 'John')
+      @post.author.authorable_type.should == 'Post'
+    end
+  end
+
+  context "namespaced foreign keys" do
+    setup do
+      News::Paper.one :article, :as => 'articleable', :class_name => 'News::Article'
+      News::Article.belongs_to :articleable, :polymorphic => true
+
+      @paper = News::Paper.create
+    end
+
+    should "work" do
+      @paper.create_article
+      @paper.article.class.should == News::Article
+    end
+
+    should "properly infer the foreign key" do
+      article = @paper.create_article
+      article.should respond_to(:articleable_id)
+      article.articleable_id.should == @paper.id
+    end
+  end
+end