mongo_mapper 0.12.0 → 0.13.0.beta1

data/{test/functional/associations/test_one_embedded_polymorphic_proxy.rb → spec/functional/associations/one_embedded_polymorphic_proxy_spec.rb}

@@ -1,24 +1,23 @@
-require 'test_helper'
-require 'models'
+require 'spec_helper'
 
-class OneEmbeddedPolymorhpicProxyTest < Test::Unit::TestCase
-  def setup
+describe "OneEmbeddedPolymorhpicProxy" do
+  before do
     @post_class = Doc('Post') do
       key :title, String
     end
   end
 
-  should "default to nil" do
+  it "should default to nil" do
     @post_class.one :author, :polymorphic => true, :class => Robot
     @post_class.new.author.should be_nil
   end
 
-  should "return nil instead of a proxy" do
+  it "should return nil instead of a proxy" do
     @post_class.one :author, :polymorphic => true, :class => Robot
     nil.should === @post_class.new.author
   end
 
-  should "be able to build" do
+  it "should be able to build" do
     @post_class.one :author, :polymorphic => true, :class => Robot
     post = @post_class.create
     author = post.build_author(:serial_number => "1B")
@@ -29,29 +28,29 @@ class OneEmbeddedPolymorhpicProxyTest < Test::Unit::TestCase
     post.author.post.should == post
   end
 
-  should "allow assignment of associated document using a hash" do
+  it "should allow assignment of associated document using a hash" do
     @post_class.one :author, :polymorphic => :true, :class => Robot
-  
+
     post = @post_class.new('author' => { 'name' => 'Frank', '_type' => 'Human' })
     post.author.name.should == 'Frank'
     post.author.class.should == Human
-  
+
     post.save.should be_true
     post.reload
-  
+
     post.author.name.should == 'Frank'
     post.author.class.should == Human
   end
 
   context "replacing the association" do
     context "with an object" do
-      setup do
+      before do
         @post_class.one :author, :polymorphic => true, :class => Robot
         @post = @post_class.create
         @human = Human.new(:name => 'Frank')
       end
 
-      should "work" do
+      it "should work" do
         @post.author = @human
         @post.save
         @post.reload
@@ -65,7 +64,7 @@ class OneEmbeddedPolymorhpicProxyTest < Test::Unit::TestCase
         @post.author.should == new_human
       end
 
-      should "generate a new proxy instead of modifying the existing one" do
+      it "should generate a new proxy instead of modifying the existing one" do
         @post.author = @human
         @post.save
         @post.reload
@@ -82,19 +81,19 @@ class OneEmbeddedPolymorhpicProxyTest < Test::Unit::TestCase
         original_author.name.should == 'Frank'
       end
 
-      should "assign _type" do
+      it "should assign _type" do
         @post.author = @human
         @post.author._type.should == "Human"
       end
     end
 
     context "with a Hash" do
-      setup do
+      before do
         @post_class.one :author, :polymorphic => true, :class => Robot
         @post = @post_class.create
       end
 
-      should "convert to an object of the class and work" do
+      it "should convert to an object of the class and work" do
         @post.author = {'serial_number' => '1B'}
         @post.save
         @post.reload
@@ -106,7 +105,7 @@ class OneEmbeddedPolymorhpicProxyTest < Test::Unit::TestCase
         @post.author.serial_number.should == '2C'
       end
 
-      should "convert to an object of _type if given" do
+      it "should convert to an object of _type if given" do
         @post.author = {'name' => 'Frank', '_type' => 'Human'}
         @post.author.name.should == 'Frank'
         @post.author.class.should == Human
@@ -117,7 +116,7 @@ class OneEmbeddedPolymorhpicProxyTest < Test::Unit::TestCase
         @post.author.class.should == Human
       end
 
-      should "assign _type" do
+      it "should assign _type" do
         @post.author = {'name' => 'Frank', '_type' => 'Human'}
         @post.save
         @post.reload
@@ -126,7 +125,7 @@ class OneEmbeddedPolymorhpicProxyTest < Test::Unit::TestCase
     end
   end
 
-  should "unset the association" do
+  it "should unset the association" do
     @post_class.one :author, :polymorphic => true, :class => Robot
     post = @post_class.create
     human = Human.new
@@ -136,7 +135,7 @@ class OneEmbeddedPolymorhpicProxyTest < Test::Unit::TestCase
     post.author.should == nil
   end
 
-  should "set modularized associated models correctly" do
+  it "should set modularized associated models correctly" do
     @post_class.one :author, :polymorphic => true, :class => Robot
 
     post = @post_class.new('author' => {'_type' => 'TrModels::Ambulance', 'license_plate' => 'GGG123', 'icu' => true})
@@ -152,7 +151,7 @@ class OneEmbeddedPolymorhpicProxyTest < Test::Unit::TestCase
     post.author.icu.should be_true
   end
 
-  should "not have problem loading root document if embedded one is nil" do
+  it "should not have problem loading root document if embedded one is nil" do
     @post_class.one :author, :polymorphic => true, :class => Robot
     post = @post_class.create
 
@@ -161,7 +160,7 @@ class OneEmbeddedPolymorhpicProxyTest < Test::Unit::TestCase
     }.should_not raise_error
   end
 
-  should "load the parent and root documents for nested embedded documents" do
+  it "should load the parent and root documents for nested embedded documents" do
     @address_class = EDoc('Address') do
       key :city, String
       key :state, String
@@ -169,31 +168,31 @@ class OneEmbeddedPolymorhpicProxyTest < Test::Unit::TestCase
     @author_class = EDoc('EmbeddedAuthor')
     @author_class.one :address, :polymorphic => true, :class => @address_class
     @post_class.one :author, :polymorphic => true, :class => @author_class
-  
+
     post = @post_class.create(:title => 'Post Title', :author => { :name => 'Frank', :address => { :city => 'Boston', :state => 'MA' } })
-  
+
     post.author.address._parent_document.should == post.author
     post.author.address._root_document.should == post
   end
-  
-  should "have boolean method for testing presence" do
+
+  it "should have boolean method for testing presence" do
     @post_class.one :author, :polymorphic => true, :class => Robot
-  
+
     post = @post_class.new
     post.author?.should be_false
-  
+
     post.author = Human.new(:name => 'Frank')
     post.author?.should be_true
   end
 
-  should "initialize id for nested embedded document created from hash" do
+  it "should initialize id for nested embedded document created from hash" do
     @address_class = EDoc('Address') do
       key :city, String
       key :state, String
     end
     @author_class = EDoc('EmbeddedAuthor')
     @author_class.one :address, :polymorphic => true, :class => @address_class
-    @post_class.one :author, :polymorphic => true, :class => @author_class  
+    @post_class.one :author, :polymorphic => true, :class => @author_class
 
     post = @post_class.create(:title => 'Post Title', :author => {
       :name => 'Frank',
@@ -202,7 +201,7 @@ class OneEmbeddedPolymorhpicProxyTest < Test::Unit::TestCase
         :state => 'MA'
       }
     })
-  
+
     post.author.address.id.should_not be_nil
   end
 end

data/{test/functional/associations/test_one_embedded_proxy.rb → spec/functional/associations/one_embedded_proxy_spec.rb}

@@ -1,7 +1,7 @@
-require 'test_helper'
+require 'spec_helper'
 
-class OneEmbeddedProxyTest < Test::Unit::TestCase
-  def setup
+describe "OneEmbeddedProxy" do
+  before do
     @post_class = Doc('Post') do
       key :title, String
     end
@@ -11,12 +11,12 @@ class OneEmbeddedProxyTest < Test::Unit::TestCase
     end
   end
 
-  should "default to nil" do
+  it "should default to nil" do
     @post_class.one :author, :class => @author_class
     @post_class.new.author.should be_nil
   end
 
-  should "be able to build" do
+  it "should be able to build" do
     @post_class.one :author, :class => @author_class
 
     post = @post_class.create
@@ -28,7 +28,7 @@ class OneEmbeddedProxyTest < Test::Unit::TestCase
     post.author.post.should == post
   end
 
-  should "be able to replace the association" do
+  it "should be able to replace the association" do
     @post_class.one :author, :class => @author_class
 
     post = @post_class.new
@@ -45,7 +45,7 @@ class OneEmbeddedProxyTest < Test::Unit::TestCase
     post.author.should == new_author
   end
 
-  should "not have problem loading root document if embedded one is nil" do
+  it "should not have problem loading root document if embedded one is nil" do
     @post_class.one :author, :class => @author_class
     post = @post_class.create
 
@@ -54,7 +54,7 @@ class OneEmbeddedProxyTest < Test::Unit::TestCase
     }.should_not raise_error
   end
 
-  should "load the parent and root documents for nested embedded documents" do
+  it "should load the parent and root documents for nested embedded documents" do
     @address_class = EDoc('Address') do
       key :city, String
       key :state, String
@@ -68,7 +68,7 @@ class OneEmbeddedProxyTest < Test::Unit::TestCase
     post.author.address._root_document.should == post
   end
 
-  should "have boolean method for testing presence" do
+  it "should have boolean method for testing presence" do
     @post_class.one :author, :class => @author_class
 
     post = @post_class.new
@@ -78,7 +78,7 @@ class OneEmbeddedProxyTest < Test::Unit::TestCase
     post.author?.should be_true
   end
 
-  should "initialize id for nested embedded document created from hash" do
+  it "should initialize id for nested embedded document created from hash" do
     @address_class = EDoc('Address') do
       key :city, String
       key :state, String

data/{test/functional/associations/test_one_proxy.rb → spec/functional/associations/one_proxy_spec.rb}

@@ -1,25 +1,25 @@
-require 'test_helper'
-require 'models'
+require 'spec_helper'
 
-class OneProxyTest < Test::Unit::TestCase
-  def setup
+describe "OneProxy" do
+  before do
     @post_class = Doc('Post')
     @author_class = Doc do
+      key :name, String
       key :post_id, ObjectId
     end
   end
 
-  should "default to nil" do
+  it "should default to nil" do
     @post_class.one :author, :class => @author_class
     @post_class.new.author.nil?.should be_true
   end
 
-  should "return nil instead of a proxy" do
+  it "should return nil instead of a proxy" do
     @post_class.one :author, :class => @author_class
     nil.should === @post_class.new.author
   end
 
-  should "allow assignment of associated document using a hash" do
+  it "should allow assignment of associated document using a hash" do
     @post_class.one :author, :class => @author_class
 
     post = @post_class.new('author' => { 'name' => 'Frank' })
@@ -33,7 +33,7 @@ class OneProxyTest < Test::Unit::TestCase
 
   context "replacing the association" do
     context "with an object of the class" do
-      should "work" do
+      it "should work" do
         @post_class.one :author, :class => @author_class
 
         post = @post_class.new
@@ -49,7 +49,7 @@ class OneProxyTest < Test::Unit::TestCase
         post.author.should == new_author
       end
 
-      should "generate a new proxy instead of modifying the existing one" do
+      it "should generate a new proxy instead of modifying the existing one" do
         @post_class.one :author, :class => @author_class
 
         post = @post_class.new
@@ -71,7 +71,7 @@ class OneProxyTest < Test::Unit::TestCase
     end
 
     context "with a Hash" do
-      should "convert to an object of the class and work" do
+      it "should convert to an object of the class and work" do
         @post_class.one :author, :class => @author_class
 
         post = @post_class.new
@@ -85,35 +85,35 @@ class OneProxyTest < Test::Unit::TestCase
         post.author.name.should == 'Emily'
       end
     end
-    
+
     context "with :dependent" do
       context "=> delete" do
-        setup do
+        before 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
+
+        it "should call delete on the existing document" do
+          @author_class.any_instance.should_receive(:delete).once
           @post.author = @author_class.new
         end
-        
-        should "remove the existing document from the database" do
+
+        it "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
+
+        it "should do nothing if it's the same document" do
+          @author_class.any_instance.should_receive(:delete).never
           @post.author = @author
-        end        
+        end
       end
-    
+
       context "=> destory" do
-        setup do
+        before do
           @post_class.one :author, :class => @author_class, :dependent => :destroy
 
           @post = @post_class.create
@@ -121,32 +121,32 @@ class OneProxyTest < Test::Unit::TestCase
           @post.author = @author
         end
 
-        should "call destroy the existing document" do
-          @author_class.any_instance.expects(:destroy).once
+        it "should call destroy the existing document" do
+          @author_class.any_instance.should_receive(:destroy).once
           @post.author = @author_class.new
         end
-        
-        should "remove the existing document from the database" do
+
+        it "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
+
+        it "should do nothing if it's the same document" do
+          @author_class.any_instance.should_receive(:destroy).never
           @post.author = @author
-        end        
+        end
       end
 
       context "=> nullify" do
-        setup do
+        before 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
+
+        it "should nullify the existing document" do
           @author.reload
           @author.post_id.should == @post.id
 
@@ -155,16 +155,16 @@ class OneProxyTest < Test::Unit::TestCase
           @author.reload
           @author.post_id.should be_nil
         end
-        
-        should "work when it's the same document" do
+
+        it "should work when it's the same document" do
           old_author = @post.author
           @post.author = @author
           old_author.should == @post.author
-        end        
+        end
       end
-      
+
       context "unspecified" do
-        should "nullify the existing document" do
+        it "should nullify the existing document" do
           @post_class.one :author, :class => @author_class
 
           post = @post_class.create
@@ -180,30 +180,30 @@ class OneProxyTest < Test::Unit::TestCase
         end
       end
     end
-    
+
     context "with nil" do
-      setup do
+      before do
         @post_class.one :author, :class => @author_class
 
         @post = @post_class.new
-        @author = @author_class.new(:name => 'Frank')        
+        @author = @author_class.new(:name => 'Frank')
         @post.author = @author
       end
-      
-      should "nullify the existing document" do
+
+      it "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
+      it "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
+  it "should have boolean method for testing presence" do
     @post_class.one :author, :class => @author_class
 
     post = @post_class.new
@@ -213,7 +213,7 @@ class OneProxyTest < Test::Unit::TestCase
     post.author?.should be_true
   end
 
-  should "work with criteria" do
+  it "should work with criteria" do
     @post_class.one :primary_author, :class => @author_class, :primary => true
     @post_class.one :author, :class => @author_class, :primary => false
 
@@ -225,7 +225,7 @@ class OneProxyTest < Test::Unit::TestCase
     post.primary_author.should == primary
   end
 
-  should "unset the association" do
+  it "should unset the association" do
     @post_class.one :author, :class => @author_class
     post = @post_class.create
     author = @author_class.create
@@ -234,10 +234,10 @@ 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
+      before do
         @post_class.one :author, :class => @author_class, :dependent => :destroy
 
         @post = @post_class.create
@@ -245,12 +245,12 @@ class OneProxyTest < Test::Unit::TestCase
         @post.author = @author
       end
 
-      should "should call destroy on the associated documents" do
-        @author_class.any_instance.expects(:destroy).once
+      it "should should call destroy on the associated documents" do
+        @author_class.any_instance.should_receive(:destroy).once
         @post.destroy
       end
-      
-      should "should remove the associated documents" do
+
+      it "should should remove the associated documents" do
         @author_class.count.should == 1
         @post.destroy
         @post.author.should == nil
@@ -259,7 +259,7 @@ class OneProxyTest < Test::Unit::TestCase
     end
 
     context "=> delete" do
-      setup do
+      before do
         @post_class.one :author, :class => @author_class, :dependent => :delete
 
         @post = @post_class.create
@@ -267,12 +267,12 @@ class OneProxyTest < Test::Unit::TestCase
         @post.author = @author
       end
 
-      should "should call delete the associated documents" do
-        @author_class.any_instance.expects(:delete).once
+      it "should should call delete the associated documents" do
+        @author_class.any_instance.should_receive(:delete).once
         @post.destroy
       end
 
-      should "remove the associated documents" do
+      it "should remove the associated documents" do
         @author_class.count.should == 1
         @post.destroy
         @post.author.should == nil
@@ -281,7 +281,7 @@ class OneProxyTest < Test::Unit::TestCase
     end
 
     context "=> nullify" do
-      should "should nullify the relationship but not destroy the associated document" do
+      it "should should nullify the relationship but not destroy the associated document" do
         @post_class.one :author, :class => @author_class, :dependent => :nullify
 
         post = @post_class.create
@@ -297,9 +297,9 @@ class OneProxyTest < Test::Unit::TestCase
         author.post_id.should == nil
       end
     end
-    
+
     context "unspecified" do
-      should "should nullify the relationship but not destroy the associated document" do
+      it "should should nullify the relationship but not destroy the associated document" do
         @post_class.one :author, :class => @author_class
 
         post = @post_class.create
@@ -316,65 +316,88 @@ class OneProxyTest < Test::Unit::TestCase
       end
     end
   end
-  
 
-  should "be able to build" do
-    @post_class.one :author, :class => @author_class
-
-    post = @post_class.create
-    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
-    post.author.post_id.should == post.id
-  end
+  context "when building associations" do
+    before do
+      @post_class.one :author, :class => @author_class
+    end
+    let(:post) { @post_class.create }
+
+    context "#build" do
+      it "should work" 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
+        post.author.post_id.should == post.id
+      end
 
-  should "be able to create" do
-    @post_class.one :author, :class => @author_class
+      it "should allow a block" do
+        author = post.build_author do |doc|
+          doc.name = "John"
+        end
+        author.name.should == "John"
+      end
+    end
 
-    post = @post_class.create
-    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
-    post.author.post_id.should == post.id
-  end
+    context "#create" do
+      it "should work" 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
+        post.author.post_id.should == post.id
+      end
 
-  context "#create!" do
-    setup do
-      @author_class.key :name, String, :required => true
-      @post_class.one :author, :class => @author_class
+      it "should allow a block" do
+        author = post.create_author do |doc|
+          doc.name = "John"
+        end
+        author.name.should == "John"
+      end
     end
 
-    should "raise exception if invalid" do
-      post = @post_class.create
-      assert_raises(MongoMapper::DocumentNotValid) do
-        post.create_author!
+
+    context "#create!" do
+      before do
+        @author_class.key :name, String, :required => true
+      end
+
+      it "should raise exception if invalid" do
+        expect {
+          post.create_author!
+        }.to raise_error(MongoMapper::DocumentNotValid)
+      end
+
+      it "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
+        post.author.post_id.should == post.id
       end
-    end
 
-    should "work if valid" do
-      post = @post_class.create
-      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
-      post.author.post_id.should == post.id
+      it "should accept a block" do
+        author = post.create_author! do |doc|
+          doc.name = "John"
+        end
+        author.name.should == "John"
+      end
     end
   end
 
   context "namespaced foreign keys" do
-    setup do
+    before do
       News::Paper.one :article, :class_name => 'News::Article'
       News::Article.belongs_to :paper, :class_name => 'News::Paper'
 
       @paper = News::Paper.create
     end
 
-    should "properly infer the foreign key" do
+    it "should properly infer the foreign key" do
       article = @paper.create_article
       article.should respond_to(:paper_id)
       article.paper_id.should == @paper.id