mongo_mapper 0.12.0 → 0.13.0.beta1

data/{test/functional/associations/test_many_embedded_polymorphic_proxy.rb → spec/functional/associations/many_embedded_polymorphic_proxy_spec.rb}

@@ -1,25 +1,24 @@
-require 'test_helper'
-require 'models'
+require 'spec_helper'
 
-class ManyEmbeddedPolymorphicProxyTest < Test::Unit::TestCase
-  def setup
+describe "ManyEmbeddedPolymorphicProxy" do
+  before do
     Catalog.collection.remove
     TrModels::Fleet.collection.remove
   end
 
-  should "default reader to empty array" do
+  it "should default reader to empty array" do
     catalog = Catalog.new
     catalog.medias.should == []
   end
 
-  should "allow adding to association like it was an array" do
+  it "should allow adding to association like it was an array" do
     catalog = Catalog.new
     catalog.medias << Video.new
     catalog.medias.push Video.new
     catalog.medias.size.should == 2
   end
 
-  should "be able to replace the association" do
+  it "should be able to replace the association" do
     catalog = Catalog.new
     catalog.medias = [Video.new('file' => 'video.mpg', 'length' => 3600)]
     catalog.save.should be_true
@@ -31,7 +30,7 @@ class ManyEmbeddedPolymorphicProxyTest < Test::Unit::TestCase
   end
 
   context "associating objects of non-SCI class" do
-    should "work on replacement" do
+    it "should work on replacement" do
       catalog = Catalog.new
       catalog.medias = [Human.new(:name => 'Frank'), Robot.new(:serial_number => '1B')]
 
@@ -51,7 +50,7 @@ class ManyEmbeddedPolymorphicProxyTest < Test::Unit::TestCase
       catalog.medias[1].class.should == Robot
     end
 
-    should "work on replacement with hashes" do
+    it "should work on replacement with hashes" do
       catalog = Catalog.new
       catalog.medias = [{:name => 'Frank', '_type' => 'Human'}, {:serial_number => '1B', '_type' => 'Robot'}]
 
@@ -71,7 +70,7 @@ class ManyEmbeddedPolymorphicProxyTest < Test::Unit::TestCase
       catalog.medias[1].class.should == Robot
     end
 
-    should "work with concatination" do
+    it "should work with concatination" do
       catalog = Catalog.new
       catalog.medias << Human.new(:name => 'Frank')
       catalog.medias << Robot.new(:serial_number => '1B')
@@ -94,11 +93,11 @@ class ManyEmbeddedPolymorphicProxyTest < Test::Unit::TestCase
   end
 
   context "count" do
-    should "default to 0" do
+    it "should default to 0" do
       Catalog.new.medias.count.should == 0
     end
 
-    should 'return correct count if any are embedded' do
+    it 'should return correct count if any are embedded' do
       catalog = Catalog.new
       catalog.medias = [
         Video.new('file' => 'video.mpg', 'length' => 3600),
@@ -112,7 +111,7 @@ class ManyEmbeddedPolymorphicProxyTest < Test::Unit::TestCase
     end
   end
 
-  should "store different associations" do
+  it "should store different associations" do
     catalog = Catalog.new
     catalog.medias = [
       Video.new('file' => 'video.mpg', 'length' => 3600),
@@ -133,7 +132,7 @@ class ManyEmbeddedPolymorphicProxyTest < Test::Unit::TestCase
   end
 
   context "With modularized models" do
-    should "set associations correctly" do
+    it "should set associations correctly" do
       fleet_attributes = {
         'name' => 'My Fleet',
         'transports' => [
@@ -170,19 +169,19 @@ class ManyEmbeddedPolymorphicProxyTest < Test::Unit::TestCase
       fleet.transports[2].year.should == 2008
     end
 
-    should "default reader to empty array" do
+    it "should default reader to empty array" do
       fleet = TrModels::Fleet.new
       fleet.transports.should == []
     end
 
-    should "allow adding to association like it was an array" do
+    it "should allow adding to association like it was an array" do
       fleet = TrModels::Fleet.new
       fleet.transports << TrModels::Car.new
       fleet.transports.push TrModels::Bus.new
       fleet.transports.size.should == 2
     end
 
-    should "be able to replace the association" do
+    it "should be able to replace the association" do
       fleet = TrModels::Fleet.new
       fleet.transports = [TrModels::Car.new('license_plate' => 'DCU2013', 'model' => 'Honda Civic')]
       fleet.save.should be_true
@@ -192,7 +191,7 @@ class ManyEmbeddedPolymorphicProxyTest < Test::Unit::TestCase
       fleet.transports[0].license_plate.should == 'DCU2013'
     end
 
-    should "store different associations" do
+    it "should store different associations" do
       fleet = TrModels::Fleet.new
       fleet.transports = [
         TrModels::Car.new('license_plate' => 'ABC1223', 'model' => 'Honda Civic', 'year' => 2003),
@@ -214,7 +213,7 @@ class ManyEmbeddedPolymorphicProxyTest < Test::Unit::TestCase
   end
 
   context "extending the association" do
-    should "work using a block passed to many" do
+    it "should work using a block passed to many" do
       catalog = Catalog.new
       medias = catalog.medias = [
         Video.new('file' => 'video.mpg', 'length' => 3600, :visible => true),
@@ -225,7 +224,7 @@ class ManyEmbeddedPolymorphicProxyTest < Test::Unit::TestCase
       catalog.medias.visible.should == [medias[0], medias[1]]
     end
 
-    should "work using many's :extend option" do
+    it "should work using many's :extend option" do
       fleet = TrModels::Fleet.new
       transports = fleet.transports = [
         TrModels::Car.new('license_plate' => 'ABC1223', 'model' => 'Honda Civic', 'year' => 2003, :purchased_on => 2.years.ago.to_date),

data/{test/functional/associations/test_many_embedded_proxy.rb → spec/functional/associations/many_embedded_proxy_spec.rb}

@@ -1,8 +1,7 @@
-require 'test_helper'
-require 'models'
+require 'spec_helper'
 
-class ManyEmbeddedProxyTest < Test::Unit::TestCase
-  def setup
+describe "ManyEmbeddedProxy" do
+  before do
     @comment_class = EDoc do
       key :name, String
       key :body, String
@@ -28,18 +27,18 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
     @owner_class.many :pets, :class => @pet_class
   end
 
-  should "default reader to empty array" do
+  it "should default reader to empty array" do
     @post_class.new.comments.should == []
   end
 
-  should "allow adding to association like it was an array" do
+  it "should allow adding to association like it was an array" do
     post = @post_class.new
     post.comments << @comment_class.new
     post.comments.push @comment_class.new
     post.comments.size.should == 2
   end
 
-  should "be embedded in document on save" do
+  it "should be embedded in document on save" do
     frank = @comment_class.new(:name => 'Frank', :body => 'Hi!')
     bill = @comment_class.new(:name => 'Bill', :body => 'Hi!')
     post = @post_class.new
@@ -55,7 +54,7 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
     post.comments[1].new?.should == false
   end
 
-  should "allow embedding arbitrarily deep" do
+  it "should allow embedding arbitrarily deep" do
     @klass = Doc()
     @klass.key :person, @person_class
 
@@ -72,7 +71,7 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
     doc.person.child.child.name.should == 'Linda'
   end
 
-  should "allow assignment of many embedded documents using a hash" do
+  it "should allow assignment of many embedded documents using a hash" do
     person_attributes = {
       'name' => 'Mr. Pet Lover',
       'pets' => [
@@ -103,7 +102,7 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
   end
 
   context "passing documents between versions of code" do
-    setup do
+    before do
       @old_klass = Doc do
         set_collection_name 'generic_parents'
         key :name, String
@@ -116,7 +115,7 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
       @updated_klass.many :pets, :class => @pet_class
     end
 
-    should "not break many embedded proxy" do
+    it "should not break many embedded proxy" do
       @old_klass.collection.drop
       created_by_new_code = @updated_klass.create!
       created_by_new_code.pets.should == []
@@ -132,12 +131,12 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
   end
 
   context "embedding many embedded documents" do
-    setup do
+    before do
       @klass = Doc()
       @klass.many :people, :class => @person_class
     end
 
-    should "persist all embedded documents" do
+    it "should persist all embedded documents" do
       meg = @person_class.new(:name => 'Meg', :pets => [
         @pet_class.new(:name => 'Sparky', :species => 'Dog'),
         @pet_class.new(:name => 'Koda', :species => 'Dog')
@@ -156,7 +155,7 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
       doc.people.first.pets[1].species.should == 'Dog'
     end
 
-    should "create a reference to the root document for all embedded documents before save" do
+    it "should create a reference to the root document for all embedded documents before save" do
       doc = @klass.new
       meg = @person_class.new(:name => 'Meg')
       pet = @pet_class.new(:name => 'Sparky', :species => 'Dog')
@@ -167,7 +166,7 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
       doc.people.first._root_document.should == doc
       doc.people.first.pets.first._root_document.should == doc
     end
-    should "create a reference to the owning document for all embedded documents before save" do
+    it "should create a reference to the owning document for all embedded documents before save" do
       doc = @klass.new
       meg = @person_class.new(:name => 'Meg')
       pet = @pet_class.new(:name => 'Sparky', :species => 'Dog')
@@ -179,7 +178,7 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
       doc.people.first.pets.first._parent_document.should == doc.people.first
     end
 
-    should "create a reference to the root document for all embedded documents" do
+    it "should create a reference to the root document for all embedded documents" do
       sparky = @pet_class.new(:name => 'Sparky', :species => 'Dog')
       meg    = @person_class.new(:name => 'Meg', :pets => [sparky])
       doc    = @klass.new
@@ -190,7 +189,7 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
       doc.people.first._root_document.should == doc
       doc.people.first.pets.first._root_document.should == doc
     end
-    should "create a reference to the owning document for all embedded documents" do
+    it "should create a reference to the owning document for all embedded documents" do
       doc = @klass.new
       meg = @person_class.new(:name => 'Meg')
       pet = @pet_class.new(:name => 'Sparky', :species => 'Dog')
@@ -204,7 +203,7 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
       doc.people.first.pets.first._parent_document.should == doc.people.first
     end
 
-    should "create embedded_in relationship for embedded docs" do
+    it "should create embedded_in relationship for embedded docs" do
       doc = @klass.new
       meg = @person_class.new(:name => 'Meg')
       pet = @pet_class.new(:name => 'Sparky', :species => 'Dog')
@@ -218,7 +217,7 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
     end
   end
 
-  should "allow finding by id" do
+  it "should allow finding by id" do
     sparky = @pet_class.new(:name => 'Sparky', :species => 'Dog')
     meg    = @owner_class.create(:name => 'Meg', :pets => [sparky])
 
@@ -227,11 +226,11 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
   end
 
   context "count" do
-    should "default to 0" do
+    it "should default to 0" do
       @owner_class.new.pets.count.should == 0
     end
 
-    should "return correct count if any are embedded" do
+    it "should return correct count if any are embedded" do
       owner = @owner_class.new(:name => 'Meg')
       owner.pets = [@pet_class.new, @pet_class.new]
       owner.pets.count.should == 2
@@ -242,7 +241,7 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
   end
 
   context "extending the association" do
-    setup do
+    before do
       @address_class = EDoc do
         key :address, String
         key :city, String
@@ -255,7 +254,7 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
       end
     end
 
-    should "work using a block passed to many" do
+    it "should work using a block passed to many" do
       @project_class.many :addresses, :class => @address_class do
         def find_all_by_state(state)
           find_all { |a| a.state == state }
@@ -270,7 +269,7 @@ class ManyEmbeddedProxyTest < Test::Unit::TestCase
       project.addresses.find_all_by_state("CA").should == [addr1, addr2]
     end
 
-    should "work using many's :extend option" do
+    it "should work using many's :extend option" do
       module FindByCity
         def find_by_city(city)
           find_all { |a| a.city == city }

data/{test/functional/associations/test_many_polymorphic_proxy.rb → spec/functional/associations/many_polymorphic_proxy_spec.rb}

@@ -1,21 +1,20 @@
-require 'test_helper'
-require 'models'
+require 'spec_helper'
 
-class ManyPolymorphicProxyTest < Test::Unit::TestCase
-  def setup
+describe "ManyPolymorphicProxy" do
+  before do
     Room.collection.remove
     Message.collection.remove
   end
 
-  should "default reader to empty array" do
+  it "should default reader to empty array" do
     Room.new.messages.should == []
   end
 
-  should "add type key to polymorphic class base" do
+  it "should add type key to polymorphic class base" do
     Message.keys.keys.should include('_type')
   end
 
-  should "allow adding to assiciation like it was an array" do
+  it "should allow adding to assiciation like it was an array" do
     room = Room.new
     room.messages << Enter.new
     room.messages.push Exit.new
@@ -23,7 +22,7 @@ class ManyPolymorphicProxyTest < Test::Unit::TestCase
     room.messages.size.should == 3
   end
 
-  should "be able to replace the association" do
+  it "should be able to replace the association" do
     room = Room.create(:name => 'Lounge')
 
     lambda {
@@ -42,7 +41,7 @@ class ManyPolymorphicProxyTest < Test::Unit::TestCase
     messages[2].body.should == 'John exited room'
   end
 
-  should "correctly store type when using <<, push and concat" do
+  it "should correctly store type when using <<, push and concat" do
     room = Room.new
     room.messages <<      Enter.new(:body => 'John entered the room', :position => 1)
     room.messages.push    Exit.new(:body => 'John entered the room', :position => 2)
@@ -56,19 +55,19 @@ class ManyPolymorphicProxyTest < Test::Unit::TestCase
   end
 
   context "build" do
-    should "assign foreign key" do
+    it "should assign foreign key" do
       room = Room.create
       message = room.messages.build
       message.room_id.should == room._id
     end
 
-    should "assign _type" do
+    it "should assign _type" do
       room = Room.create
       message = room.messages.build
       message._type.should == 'Message'
     end
 
-    should "allow assigning attributes" do
+    it "should allow assigning attributes" do
       room = Room.create
       message = room.messages.build(:body => 'Foo!')
       message.body.should == 'Foo!'
@@ -76,26 +75,26 @@ class ManyPolymorphicProxyTest < Test::Unit::TestCase
   end
 
   context "create" do
-    should "assign foreign key" do
+    it "should assign foreign key" do
       room = Room.create
       message = room.messages.create
       message.room_id.should == room._id
     end
 
-    should "assign _type" do
+    it "should assign _type" do
       room = Room.create
       message = room.messages.create
       message._type.should == 'Message'
     end
 
-    should "save record" do
+    it "should save record" do
       room = Room.create
       lambda {
         room.messages.create
       }.should change { Message.count }
     end
 
-    should "allow passing attributes" do
+    it "should allow passing attributes" do
       room = Room.create
       message = room.messages.create(:body => 'Foo!')
       message.body.should == 'Foo!'
@@ -103,7 +102,7 @@ class ManyPolymorphicProxyTest < Test::Unit::TestCase
   end
 
   context "count" do
-    should "work scoped to association" do
+    it "should work scoped to association" do
       room = Room.create
       3.times { room.messages.create }
 
@@ -114,7 +113,7 @@ class ManyPolymorphicProxyTest < Test::Unit::TestCase
       other_room.messages.count.should == 2
     end
 
-    should "work with conditions" do
+    it "should work with conditions" do
       room = Room.create
       room.messages.create(:body => 'Foo')
       room.messages.create(:body => 'Other 1')
@@ -125,7 +124,7 @@ class ManyPolymorphicProxyTest < Test::Unit::TestCase
   end
 
   context "Finding scoped to association" do
-    setup do
+    before do
       @lounge = Room.create(:name => 'Lounge')
       @lm1 = Message.create(:body => 'Loungin!', :position => 1)
       @lm2 = Message.create(:body => 'I love loungin!', :position => 2)
@@ -142,24 +141,24 @@ class ManyPolymorphicProxyTest < Test::Unit::TestCase
     end
 
     context "dynamic finders" do
-      should "work with single key" do
+      it "should work with single key" do
         @lounge.messages.find_by_position(1).should == @lm1
         @hall.messages.find_by_position(2).should == @hm2
       end
 
-      should "work with multiple keys" do
+      it "should work with multiple keys" do
         @lounge.messages.find_by_body_and_position('Loungin!', 1).should == @lm1
         @lounge.messages.find_by_body_and_position('Loungin!', 2).should be_nil
       end
 
-      should "raise error when using !" do
+      it "should raise error when using !" do
         lambda {
           @lounge.messages.find_by_position!(222)
         }.should raise_error(MongoMapper::DocumentNotFound)
       end
 
       context "find_or_create_by" do
-        should "not create document if found" do
+        it "should not create document if found" do
           lambda {
             message = @lounge.messages.find_or_create_by_body('Loungin!')
             message.room.should == @lounge
@@ -167,7 +166,7 @@ class ManyPolymorphicProxyTest < Test::Unit::TestCase
           }.should_not change { Message.count }
         end
 
-        should "create document if not found" do
+        it "should create document if not found" do
           lambda {
             message = @lounge.messages.find_or_create_by_body('Yo dawg!')
             message.room.should == @lounge
@@ -178,49 +177,49 @@ class ManyPolymorphicProxyTest < Test::Unit::TestCase
     end
 
     context "with #all" do
-      should "work" do
+      it "should work" do
         @lounge.messages.all(:order => "position").should == [@lm1, @lm2]
       end
 
-      should "work with conditions" do
+      it "should work with conditions" do
         messages = @lounge.messages.all(:body => 'Loungin!', :order => "position")
         messages.should == [@lm1]
       end
 
-      should "work with order" do
+      it "should work with order" do
         messages = @lounge.messages.all(:order => 'position desc')
         messages.should == [@lm2, @lm1]
       end
     end
 
     context "with #first" do
-      should "work" do
+      it "should work" do
         @lounge.messages.first(:order => "position asc").should == @lm1
       end
 
-      should "work with conditions" do
+      it "should work with conditions" do
         message = @lounge.messages.first(:body => 'I love loungin!', :order => "position asc")
         message.should == @lm2
       end
     end
 
     context "with #last" do
-      should "work" do
+      it "should work" do
         @lounge.messages.last(:order => "position asc").should == @lm2
       end
 
-      should "work with conditions" do
+      it "should work with conditions" do
         message = @lounge.messages.last(:body => 'Loungin!', :order => "position asc")
         message.should == @lm1
       end
     end
 
     context "with one id" do
-      should "work for id in association" do
+      it "should work for id in association" do
         @lounge.messages.find(@lm2._id).should == @lm2
       end
 
-      should "not work for id not in association" do
+      it "should not work for id not in association" do
         lambda {
           @lounge.messages.find!(@hm2._id)
         }.should raise_error(MongoMapper::DocumentNotFound)
@@ -228,53 +227,53 @@ class ManyPolymorphicProxyTest < Test::Unit::TestCase
     end
 
     context "with query options/criteria" do
-      should "work with order on association" do
+      it "should work with order on association" do
         @lounge.messages.should == [@lm1, @lm2]
       end
 
-      should "allow overriding the order provided to the association" do
+      it "should allow overriding the order provided to the association" do
         @lounge.messages.all(:order => 'position').should == [@lm1, @lm2]
       end
 
-      should "allow using conditions on association" do
+      it "should allow using conditions on association" do
         @hall.latest_messages.should == [@hm3, @hm2]
       end
     end
 
     context "with multiple ids" do
-      should "work for ids in association" do
+      it "should work for ids in association" do
         messages = @lounge.messages.find(@lm1._id, @lm2._id)
         messages.should == [@lm1, @lm2]
       end
 
-      should "not work for ids not in association" do
-        assert_raises(MongoMapper::DocumentNotFound) do
+      it "should not work for ids not in association" do
+        expect {
           @lounge.messages.find!(@lm1._id, @lm2._id, @hm2._id)
-        end
+        }.to raise_error(MongoMapper::DocumentNotFound)
       end
     end
 
     context "with #paginate" do
-      setup do
+      before do
         @messages = @hall.messages.paginate(:per_page => 2, :page => 1, :order => 'position asc')
       end
 
-      should "return total pages" do
+      it "should return total pages" do
         @messages.total_pages.should == 2
       end
 
-      should "return total entries" do
+      it "should return total entries" do
         @messages.total_entries.should == 3
       end
 
-      should "return the subject" do
+      it "should return the subject" do
         @messages.should == [@hm1, @hm2]
       end
     end
   end
 
   context "extending the association" do
-    should "work using a block passed to many" do
+    it "should work using a block passed to many" do
       room = Room.new(:name => "Amazing Room")
       messages = room.messages = [
         Enter.new(:body => 'John entered room',  :position => 3),
@@ -288,7 +287,7 @@ class ManyPolymorphicProxyTest < Test::Unit::TestCase
       room.messages.older.should == messages[3..5]
     end
 
-    should "work using many's :extend option" do
+    it "should work using many's :extend option" do
 
       room = Room.new(:name => "Amazing Room")
       accounts = room.accounts = [