mongo_mapper 0.9.2 → 0.10.0

data/test/functional/associations/test_one_proxy.rb

@@ -85,6 +85,122 @@ class OneProxyTest < Test::Unit::TestCase
         post.author.name.should == 'Emily'
       end
     end
+    
+    context "with :dependent" do
+      context "=> delete" do
+        setup do
+          @post_class.one :author, :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, :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, :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.post_id.should == @post.id
+
+          @post.author = @author_class.new
+
+          @author.reload
+          @author.post_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        
+      end
+      
+      context "unspecified" do
+        should "nullify the existing document" do
+          @post_class.one :author, :class => @author_class
+
+          post = @post_class.create
+          author = @author_class.new
+          post.author = author
+          author.reload
+          author.post_id.should == post.id
+
+          post.author = @author_class.new
+
+          author.reload
+          author.post_id.should be_nil
+        end
+      end
+    end
+    
+    context "with nil" do
+      setup do
+        @post_class.one :author, :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.post_id.should be_nil
+      end
+
+      should "set the target to nil" do
+        @post.author = nil
+        @post.author.should == nil
+      end
+    end
   end
 
   should "have boolean method for testing presence" do
@@ -118,44 +234,89 @@ class OneProxyTest < Test::Unit::TestCase
     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, :class => @author_class, :dependent => :destroy
+
+        @post = @post_class.create
+        @author = @author_class.new
+        @post.author = @author
+      end
 
-  should "work with :dependent delete" do
-    @post_class.one :author, :class => @author_class, :dependent => :delete
+      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
 
-    post = @post_class.create
-    author = @author_class.new
-    post.author = author
-    post.reload
+    context "=> delete" do
+      setup do
+        @post_class.one :author, :class => @author_class, :dependent => :delete
 
-    @author_class.any_instance.expects(:delete).once
-    post.author = @author_class.new
-  end
+        @post = @post_class.create
+        @author = @author_class.new
+        @post.author = @author
+      end
 
-  should "work with :dependent destroy" do
-    @post_class.one :author, :class => @author_class, :dependent => :destroy
+      should "should call delete the associated documents" do
+        @author_class.any_instance.expects(:delete).once
+        @post.destroy
+      end
 
-    post = @post_class.create
-    author = @author_class.new
-    post.author = author
-    post.reload
+      should "remove the associated documents" do
+        @author_class.count.should == 1
+        @post.destroy
+        @post.author.should == nil
+        @author_class.count.should == 0
+      end
+    end
 
-    @author_class.any_instance.expects(:destroy).once
-    post.author = @author_class.new
-  end
+    context "=> nullify" do
+      should "should nullify the relationship but not destroy the associated document" do
+        @post_class.one :author, :class => @author_class, :dependent => :nullify
 
-  should "work with :dependent nullify" do
-    @post_class.one :author, :class => @author_class, :dependent => :nullify
+        post = @post_class.create
+        author = @author_class.new
+        post.author = author
 
-    post = @post_class.create
-    author = @author_class.new
-    post.author = author
-    post.reload
+        @author_class.count.should == 1
+        post.destroy
+        post.author.should == nil
+        @author_class.count.should == 1
 
-    post.author = @author_class.new
+        @author_class.first.should == author
+        author.post_id.should == nil
+      end
+    end
+    
+    context "unspecified" do
+      should "should nullify the relationship but not destroy the associated document" do
+        @post_class.one :author, :class => @author_class
 
-    author.reload
-    author.post_id.should be_nil
+        post = @post_class.create
+        author = @author_class.new
+        post.author = author
+
+        @author_class.count.should == 1
+        post.destroy
+        post.author.should == nil
+        @author_class.count.should == 1
+
+        @author_class.first.should == author
+        author.post_id.should == nil
+      end
+    end
   end
+  
 
   should "be able to build" do
     @post_class.one :author, :class => @author_class

data/test/functional/test_dirty.rb

@@ -106,6 +106,15 @@ class DirtyTest < Test::Unit::TestCase
       doc = @document.new
       doc.value_changed?.should be_false
     end
+
+    should "be false if the same ObjectId was assigned in String format" do
+      @document.key :doc_id, ObjectId
+
+      doc = @document.create!(:doc_id => BSON::ObjectId.new)
+      doc.changed?.should be_false
+      doc.doc_id = doc.doc_id.to_s
+      doc.changed?.should be_false
+    end
   end
 
   context "changes" do

data/test/functional/test_document.rb

@@ -245,6 +245,11 @@ class DocumentTest < Test::Unit::TestCase
       @instance.destroy
       assert_raises(MongoMapper::DocumentNotFound) { @instance.reload }
     end
+
+    should "clear keys that were removed from the database" do
+      @instance.unset(:age)
+      @instance.reload.age.should be_nil
+    end
   end
 
   context "database has keys not defined in model" do

data/test/support/railtie.rb

@@ -0,0 +1,4 @@
+module Railtie
+  # We could get AS::Dependencies to autocreate this module
+  # but let's keep it simple
+end

data/test/support/railtie/autoloaded.rb

@@ -0,0 +1,2 @@
+class Railtie::Autoloaded < Railtie::Parent
+end

data/test/support/railtie/not_autoloaded.rb

@@ -0,0 +1,3 @@
+class Railtie::NotAutoloaded < Railtie::Parent
+
+end

data/test/support/railtie/parent.rb

@@ -0,0 +1,3 @@
+class Railtie::Parent
+  extend ActiveSupport::DescendantsTracker
+end

data/test/unit/associations/test_one_association.rb

@@ -15,4 +15,22 @@ class OneAssociationTest < Test::Unit::TestCase
       base.embeddable?.should be_false
     end
   end
+
+  context "proxy_class" do
+    should "be OneProxy for one" do
+      base = OneAssociation.new(:status)
+      base.proxy_class.should == OneProxy
+    end
+
+    should "be OneAsProxy for one with :as option" do
+      base = OneAssociation.new(:message, :as => :messagable)
+      base.proxy_class.should == OneAsProxy
+    end
+
+    should "be OneEmbeddedProxy for one embedded" do
+      base = OneAssociation.new(:media)
+      base.proxy_class.should == OneEmbeddedProxy
+    end
+  end
+
 end

data/test/unit/test_extensions.rb

@@ -137,6 +137,10 @@ class SupportTest < Test::Unit::TestCase
     should "leave nil values nil" do
       Float.to_mongo(nil).should == nil
     end
+
+    should "leave blank values nil" do
+      Float.to_mongo('').should == nil
+    end
   end
 
   context "Hash.from_mongo" do

data/test/unit/test_keys.rb

@@ -14,28 +14,6 @@ class KeyTest < Test::Unit::TestCase
     end
   end
 
-  context ".new with no id and _id of type string" do
-    should "not error" do
-      lambda {
-        klass = Doc() do
-          key :_id, String
-        end
-        silence_stderr { klass.new.id.should_not be_nil }
-      }.should_not raise_error
-    end
-  end
-
-  context ".new with no id and _id of type object id" do
-    should "not error" do
-      lambda {
-        klass = Doc() do
-          key :_id, ObjectId
-        end
-        silence_stderr { klass.new.should_not be_nil }
-      }.should_not raise_error
-    end
-  end
-
   context ".key?(:symbol)" do
     should "be true if document has key" do
       Address.key?(:city).should be_true

data/test/unit/test_plugins.rb

@@ -5,7 +5,7 @@ class PluginsTest < Test::Unit::TestCase
     Class.new { extend MongoMapper::Plugins }.plugins.should == []
   end
 
-  context "ActiveSupport::Concern" do
+  context "a plugin" do
     module MyConcern
       extend ActiveSupport::Concern
 
@@ -88,49 +88,4 @@ class PluginsTest < Test::Unit::TestCase
       end
     end
   end
-
-  context "deprecated plugin" do
-    module DeprecatedPlugin
-      def self.configure(model)
-        model.class_eval { attr_accessor :from_configure }
-      end
-
-      module ClassMethods
-        def class_foo
-          'class_foo'
-        end
-      end
-
-      module InstanceMethods
-        def instance_foo
-          'instance_foo'
-        end
-      end
-    end
-
-    setup do
-      silence_stderr do
-        @document = Class.new do
-          extend MongoMapper::Plugins
-          plugin DeprecatedPlugin
-        end
-      end
-    end
-
-    should "include instance methods" do
-      @document.new.instance_foo.should == 'instance_foo'
-    end
-
-    should "extend class methods" do
-      @document.class_foo.should == 'class_foo'
-    end
-
-    should "pass model to configure" do
-      @document.new.should respond_to(:from_configure)
-    end
-
-    should "add plugin to plugins" do
-      @document.plugins.should include(DeprecatedPlugin)
-    end
-  end
 end

data/test/unit/test_railtie.rb

@@ -0,0 +1,61 @@
+require 'test_helper'
+require "rails"
+require 'mongo_mapper/railtie'
+
+class TestRailtie < Test::Unit::TestCase
+
+  def expect_descendants(expectation)
+    # Keep expectation a string so we don't accidentally load in a class
+    Railtie::Parent.descendants.map(&:to_s).sort.should == expectation.sort
+  end
+
+  def run_initializer(mod, name)
+    initializer = mod.initializers.detect do |i|
+      i.name == name
+    end
+    initializer.block.arity == -1 ? initializer.run : initializer.run(FakeRails)
+    # mongo_mapper.prepare_dispatcher takes a Rails app as its one arg,
+    # set_clear_dependencies_hook takes no args
+  end
+
+  def load_autoloaded_class
+    Railtie::Autoloaded.presence
+  end
+
+  class FakeRails
+    def self.config
+      return Class.new { def cache_classes ; false ; end }.new
+    end
+  end
+
+  context "Railtie" do
+    include Rails::Application::Bootstrap
+
+    setup do
+      require 'support/railtie'
+      require 'support/railtie/parent'
+      require 'support/railtie/not_autoloaded'
+
+      ActiveSupport::Dependencies.autoload_paths << File.join(File.dirname(__FILE__), '..', 'support')
+
+      # These initializers don't actually run anything, they just register cleanup and prepare hooks
+      run_initializer Rails::Application::Bootstrap, :set_clear_dependencies_hook
+      run_initializer MongoMapper::Railtie, 'mongo_mapper.prepare_dispatcher'
+    end
+
+    should "not clear ActiveSupport::DescendantsTracker" do
+      expect_descendants %w( Railtie::NotAutoloaded )
+      load_autoloaded_class
+      expect_descendants %w( Railtie::NotAutoloaded Railtie::Autoloaded )
+
+      ActionDispatch::Reloader.cleanup! # cleanup 'last request'
+
+      expect_descendants %w( Railtie::NotAutoloaded )
+      load_autoloaded_class
+      expect_descendants %w( Railtie::NotAutoloaded Railtie::Autoloaded )
+
+      ActionDispatch::Reloader.prepare! # prepare 'next request'
+      expect_descendants %w( Railtie::NotAutoloaded Railtie::Autoloaded )
+    end
+  end
+end