jnunemaker-mongomapper 0.3.2 → 0.3.3

data/test/functional/test_document.rb

@@ -11,16 +11,48 @@ class DocumentTest < Test::Unit::TestCase
       key :last_name, String
       key :age, Integer
     end
-    
+
     @document.collection.clear
   end
+
+  context "Saving a document with a custom id" do
+    should "clear custom id flag when saved" do
+      doc = @document.new(:id => '1234')
+      doc.using_custom_id?.should be_true
+      doc.save.should be_true
+      doc.using_custom_id?.should be_false
+    end
+  end
+  
+  context "Loading a document from the database with keys that are not defined" do
+    setup do
+      @id = XGen::Mongo::Driver::ObjectID.new.to_s
+      @document.collection.insert({
+        :_id            => @id,
+        :first_name     => 'John', 
+        :last_name      => 'Nunemaker', 
+        :age            => 27, 
+        :favorite_color => 'red',
+        :skills         => ['ruby', 'rails', 'javascript', 'xhtml', 'css']
+      })
+    end
+
+    should "assign all keys from database" do
+      doc = @document.find(@id)
+      doc.first_name.should == 'John'
+      doc.last_name.should == 'Nunemaker'
+      doc.age.should == 27
+      doc.favorite_color.should == 'red'
+      doc.skills.should == ['ruby', 'rails', 'javascript', 'xhtml', 'css']
+    end
+  end
   
   context "Document Class Methods" do
     context "Using key with type Array" do
       setup do
         @document.key :tags, Array
       end
-      
+
       should "give correct default" do
         doc = @document.new
         doc.tags.should == []
@@ -53,7 +85,7 @@ class DocumentTest < Test::Unit::TestCase
         doc.tags.should == ["foo"]
       end
 
-      should_eventually "work with << then save" do
+      should "work with << then save" do
         doc = @document.new
         doc.tags << "foo"
         doc.tags << "bar"
@@ -62,7 +94,7 @@ class DocumentTest < Test::Unit::TestCase
         @document.find(doc.id).tags.should == %w(foo bar)
       end
     end
-  
+
     context "Using key with type Hash" do
       setup do
         @document.key :foo, Hash
@@ -72,14 +104,14 @@ class DocumentTest < Test::Unit::TestCase
         doc = @document.new
         doc.foo.should == {}
       end
-      
+
       should "work with []=" do
         doc = @document.new
         doc.foo["quux"] = "bar"
         doc.foo["quux"].should == "bar"
         doc.foo.should == { "quux" => "bar" }
       end
-      
+
       should "work with indifferent access" do
         doc = @document.new
         doc.foo = {:baz => 'bar'}
@@ -91,47 +123,51 @@ class DocumentTest < Test::Unit::TestCase
         doc = @document.new
         doc.foo = {:baz => 'bar'}
         doc.save
-        
+
         doc = @document.find(doc.id)
         doc.foo[:baz].should == 'bar'
         doc.foo['baz'].should == 'bar'
       end
     end
-  
+
     context "Saving a document with an embedded document" do
       setup do
         @document.class_eval do
           key :foo, Address
         end
       end
-  
+
       should "embed embedded document" do
         address = Address.new(:city => 'South Bend', :state => 'IN')
         doc = @document.new(:foo => address)
         doc.save
         doc.foo.city.should == 'South Bend'
         doc.foo.state.should == 'IN'
-  
+
         from_db = @document.find(doc.id)
         from_db.foo.city.should == 'South Bend'
         from_db.foo.state.should == 'IN'
       end
     end
-  
+
     context "Creating a single document" do
       setup do
         @doc_instance = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
       end
-  
+
       should "create a document in correct collection" do
         @document.count.should == 1
       end
-  
+
       should "automatically set id" do
         @doc_instance.id.should_not be_nil
         @doc_instance.id.size.should == 24
       end
-  
+
+      should "no longer be new?" do
+        @doc_instance.new?.should be_false
+      end
+
       should "return instance of document" do
         @doc_instance.should be_instance_of(@document)
         @doc_instance.first_name.should == 'John'
@@ -139,7 +175,7 @@ class DocumentTest < Test::Unit::TestCase
         @doc_instance.age.should == 27
       end
     end
-    
+
     context "Creating a document with no attributes provided" do
       setup do
         @document = Class.new do
@@ -147,14 +183,14 @@ class DocumentTest < Test::Unit::TestCase
         end
         @document.collection.clear
       end
-      
+
       should "create the document" do
         lambda {
           @document.create
         }.should change { @document.count }.by(1)
       end
     end
-    
+
     context "Creating multiple documents" do
       setup do
         @doc_instances = @document.create([
@@ -162,396 +198,579 @@ class DocumentTest < Test::Unit::TestCase
           {:first_name => 'Steve', :last_name => 'Smith', :age => '28'},
         ])
       end
-  
+
       should "create multiple documents" do
         @document.count.should == 2
       end
-  
+
       should "return an array of doc instances" do
         @doc_instances.map do |doc_instance|
           doc_instance.should be_instance_of(@document)
         end
       end
     end
-  
+
     context "Updating a document" do
       setup do
         doc = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
         @doc_instance = @document.update(doc.id, {:age => 40})
       end
-  
+
       should "update attributes provided" do
         @doc_instance.age.should == 40
       end
-  
+
       should "not update existing attributes that were not set to update" do
         @doc_instance.first_name.should == 'John'
         @doc_instance.last_name.should == 'Nunemaker'
       end
-  
+
       should "not create new document" do
         @document.count.should == 1
       end
     end
-  
+
     should "raise error when updating single doc if not provided id and attributes" do
       doc = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
       lambda { @document.update }.should raise_error(ArgumentError)
       lambda { @document.update(doc.id) }.should raise_error(ArgumentError)
       lambda { @document.update(doc.id, [1]) }.should raise_error(ArgumentError)
     end
-  
+
     context "Updating multiple documents" do
       setup do
         @doc1 = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
         @doc2 = @document.create({:first_name => 'Steve', :last_name => 'Smith', :age => '28'})
-  
+
         @doc_instances = @document.update({
           @doc1.id => {:age => 30},
           @doc2.id => {:age => 30},
         })
       end
-  
+
       should "not create any new documents" do
         @document.count.should == 2
       end
-  
+
       should "should return an array of doc instances" do
         @doc_instances.map do |doc_instance|
           doc_instance.should be_instance_of(@document)
         end
       end
-  
+
       should "update the documents" do
         @document.find(@doc1.id).age.should == 30
         @document.find(@doc2.id).age.should == 30
       end
     end
-  
+
     should "raise error when updating multiple documents if not a hash" do
       lambda { @document.update([1, 2]) }.should raise_error(ArgumentError)
     end
-  
+
     context "Finding documents" do
       setup do
         @doc1 = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
         @doc2 = @document.create({:first_name => 'Steve', :last_name => 'Smith', :age => '28'})
         @doc3 = @document.create({:first_name => 'Steph', :last_name => 'Nunemaker', :age => '26'})
       end
-  
+
       should "raise document not found if nothing provided" do
         lambda { @document.find }.should raise_error(MongoMapper::DocumentNotFound)
       end
-  
+
       context "with a single id" do
         should "work" do
           @document.find(@doc1.id).should == @doc1
         end
-  
+
         should "raise error if document not found" do
-          lambda { @document.find(MongoID.new) }.should raise_error(MongoMapper::DocumentNotFound)
-        end
-        
-        should "raise error if id is illegal" do
-          lambda { @document.find(1) }.should raise_error(MongoMapper::IllegalID)
+          lambda { @document.find(123) }.should raise_error(MongoMapper::DocumentNotFound)
         end
       end
-      
+
       context "with multiple id's" do
         should "work as arguments" do
           @document.find(@doc1.id, @doc2.id).should == [@doc1, @doc2]
         end
-  
+
         should "work as array" do
           @document.find([@doc1.id, @doc2.id]).should == [@doc1, @doc2]
         end
       end
-  
+
       context "with :all" do
         should "find all documents" do
           @document.find(:all, :order => 'first_name').should == [@doc1, @doc3, @doc2]
         end
-  
+
         should "be able to add conditions" do
           @document.find(:all, :conditions => {:first_name => 'John'}).should == [@doc1]
         end
       end
-  
+
       context "with #all" do
         should "find all documents based on criteria" do
           @document.all(:order => 'first_name').should == [@doc1, @doc3, @doc2]
-          @document.all(:conditions => {:last_name => 'Nunemaker'}).should == [@doc1, @doc3]
+          @document.all(:conditions => {:last_name => 'Nunemaker'}, :order => 'age desc').should == [@doc1, @doc3]
         end
       end
-  
+
       context "with :first" do
         should "find first document" do
           @document.find(:first, :order => 'first_name').should == @doc1
         end
       end
-  
+
       context "with #first" do
         should "find first document based on criteria" do
           @document.first(:order => 'first_name').should == @doc1
           @document.first(:conditions => {:age => 28}).should == @doc2
         end
       end
-  
+
       context "with :last" do
         should "find last document" do
-          @document.find(:last).should == @doc3
+          @document.find(:last, :order => 'age desc').should == @doc2
         end
       end
-  
+
       context "with #last" do
         should "find last document based on criteria" do
-          @document.last.should == @doc3
+          @document.last(:order => 'age desc').should == @doc2
           @document.last(:conditions => {:age => 28}).should == @doc2
         end
       end
+
+      context "with :find_by" do
+        should "find document based on argument" do
+          @document.find_by_first_name('John').should == @doc1
+          @document.find_by_last_name('Nunemaker', :order => 'age desc').should == @doc1
+          @document.find_by_age(27).should == @doc1
+        end
+
+        should "not raise error" do
+          @document.find_by_first_name('Mongo').should be_nil
+        end
+
+        should "define a method for each key" do
+          @document.methods(false).select { |e| e =~ /^find_by_/ }.size == @document.keys.size
+        end
+      end
+
+      context "with dynamic finders" do
+        should "find document based on all arguments" do
+          @document.find_by_first_name_and_last_name_and_age('John', 'Nunemaker', 27).should == @doc1
+        end
+
+        should "not find the document if an argument is wrong" do
+          @document.find_by_first_name_and_last_name_and_age('John', 'Nunemaker', 28).should be_nil
+        end
+
+        should "find all documents based on arguments" do
+          docs = @document.find_all_by_last_name('Nunemaker')
+          docs.should be_kind_of(Array)
+          docs.should include(@doc1)
+          docs.should include(@doc3)
+        end
+
+        should "find last document based on arguments" do
+          doc = @document.find_last_by_last_name('Nunemaker', :order => 'age desc')
+          doc.should == @doc1
+        end
+
+        should "initialize document with given arguments" do
+          doc = @document.find_or_initialize_by_first_name_and_last_name('David', 'Cuadrado')
+          doc.should be_new
+          doc.first_name.should == 'David'
+        end
+
+        should "not initialize document if document is found" do
+          doc = @document.find_or_initialize_by_first_name('John')
+          doc.should_not be_new
+        end
+
+        should "create document with given arguments" do
+          doc = @document.find_or_create_by_first_name_and_last_name('David', 'Cuadrado')
+          doc.should_not be_new
+          doc.first_name.should == 'David'
+        end
+
+        should "raise error if document is not found" do
+          lambda {@document.find_by_first_name_and_last_name!(1,2)}.should raise_error(MongoMapper::DocumentNotFound)
+        end
+      end
     end # finding documents
-  
+
     context "Finding document by id" do
       setup do
         @doc1 = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
         @doc2 = @document.create({:first_name => 'Steve', :last_name => 'Smith', :age => '28'})
       end
-  
+
       should "be able to find by id" do
         @document.find_by_id(@doc1.id).should == @doc1
         @document.find_by_id(@doc2.id).should == @doc2
       end
-  
+
       should "return nil if document not found" do
-        @document.find_by_id(MongoID.new).should be(nil)
+        @document.find_by_id(1234).should be(nil)
       end
     end
-  
+
     context "Deleting a document" do
       setup do
         @doc1 = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
         @doc2 = @document.create({:first_name => 'Steve', :last_name => 'Smith', :age => '28'})
         @document.delete(@doc1.id)
       end
-  
+
       should "remove document from collection" do
         @document.count.should == 1
       end
-  
+
       should "not remove other documents" do
         @document.find(@doc2.id).should_not be(nil)
       end
     end
-  
+
     context "Deleting multiple documents" do
       should "work with multiple arguments" do
         @doc1 = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
         @doc2 = @document.create({:first_name => 'Steve', :last_name => 'Smith', :age => '28'})
         @doc3 = @document.create({:first_name => 'Steph', :last_name => 'Nunemaker', :age => '26'})
         @document.delete(@doc1.id, @doc2.id)
-  
+
         @document.count.should == 1
       end
-  
+
       should "work with array as argument" do
         @doc1 = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
         @doc2 = @document.create({:first_name => 'Steve', :last_name => 'Smith', :age => '28'})
         @doc3 = @document.create({:first_name => 'Steph', :last_name => 'Nunemaker', :age => '26'})
         @document.delete([@doc1.id, @doc2.id])
-  
+
         @document.count.should == 1
       end
     end
-  
+
     context "Deleting all documents" do
       setup do
         @doc1 = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
         @doc2 = @document.create({:first_name => 'Steve', :last_name => 'Smith', :age => '28'})
         @doc3 = @document.create({:first_name => 'Steph', :last_name => 'Nunemaker', :age => '26'})
       end
-  
+
       should "remove all documents when given no conditions" do
         @document.delete_all
         @document.count.should == 0
       end
-  
+
       should "only remove matching documents when given conditions" do
         @document.delete_all({:first_name => 'John'})
         @document.count.should == 2
       end
-  
+
       should "convert the conditions to mongo criteria" do
         @document.delete_all(:age => [26, 27])
         @document.count.should == 1
       end
     end
-  
+
     context "Destroying a document" do
       setup do
         @doc1 = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
         @doc2 = @document.create({:first_name => 'Steve', :last_name => 'Smith', :age => '28'})
         @document.destroy(@doc1.id)
       end
-  
+
       should "remove document from collection" do
         @document.count.should == 1
       end
-  
+
       should "not remove other documents" do
         @document.find(@doc2.id).should_not be(nil)
       end
     end
-  
+
     context "Destroying multiple documents" do
       should "work with multiple arguments" do
         @doc1 = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
         @doc2 = @document.create({:first_name => 'Steve', :last_name => 'Smith', :age => '28'})
         @doc3 = @document.create({:first_name => 'Steph', :last_name => 'Nunemaker', :age => '26'})
         @document.destroy(@doc1.id, @doc2.id)
-  
+
         @document.count.should == 1
       end
-  
+
       should "work with array as argument" do
         @doc1 = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
         @doc2 = @document.create({:first_name => 'Steve', :last_name => 'Smith', :age => '28'})
         @doc3 = @document.create({:first_name => 'Steph', :last_name => 'Nunemaker', :age => '26'})
         @document.destroy([@doc1.id, @doc2.id])
-  
+
         @document.count.should == 1
       end
     end
-  
+
     context "Destroying all documents" do
       setup do
         @doc1 = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
         @doc2 = @document.create({:first_name => 'Steve', :last_name => 'Smith', :age => '28'})
         @doc3 = @document.create({:first_name => 'Steph', :last_name => 'Nunemaker', :age => '26'})
       end
-  
+
       should "remove all documents when given no conditions" do
         @document.destroy_all
         @document.count.should == 0
       end
-  
+
       should "only remove matching documents when given conditions" do
         @document.destroy_all(:first_name => 'John')
         @document.count.should == 2
         @document.destroy_all(:age => 26)
         @document.count.should == 1
       end
-  
+
       should "convert the conditions to mongo criteria" do
         @document.destroy_all(:age => [26, 27])
         @document.count.should == 1
       end
     end
-  
+
+    context ":dependent" do
+      setup do
+        # FIXME: make use of already defined models
+        class ::Property
+          include MongoMapper::Document
+        end
+        Property.delete_all
+
+        class ::Thing
+          include MongoMapper::Document
+          key :name, String
+        end
+        Thing.delete_all
+      end
+
+      teardown do
+        Object.send :remove_const, 'Property' if defined?(::Property)
+        Object.send :remove_const, 'Thing' if defined?(::Thing)
+      end
+
+      context "many" do
+        context "=> destroy" do
+          setup do
+            Property.belongs_to :thing, :dependent => :destroy
+            Thing.many :properties, :dependent => :destroy
+
+            @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 destroy the associated documents" do
+            @thing.properties.count.should == 3
+            @thing.destroy
+            @thing.properties.count.should == 0
+            Property.count.should == 0
+          end
+        end
+
+        context "=> delete_all" do
+          setup do
+            Property.belongs_to :thing
+            Thing.has_many :properties, :dependent => :delete_all
+
+            @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 delete associated documents" do
+            @thing.properties.count.should == 3
+            @thing.destroy
+            @thing.properties.count.should == 0
+            Property.count.should == 0
+          end
+        end
+
+        context "=> nullify" do
+          setup do
+            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 "belongs_to" do
+        context "=> destroy" do
+          setup do
+            Property.belongs_to :thing, :dependent => :destroy
+            Thing.has_many :properties
+
+            @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 "destroy the thing" do
+            Thing.count.should == 1
+            @property1.destroy
+            Thing.count.should == 0
+            @property1.thing.should be_frozen
+          end
+        end
+      end
+    end
+
     context "Counting documents in collection" do
       setup do
         @doc1 = @document.create({:first_name => 'John', :last_name => 'Nunemaker', :age => '27'})
         @doc2 = @document.create({:first_name => 'Steve', :last_name => 'Smith', :age => '28'})
         @doc3 = @document.create({:first_name => 'Steph', :last_name => 'Nunemaker', :age => '26'})
       end
-  
+
       should "count all with no arguments" do
         @document.count.should == 3
       end
-  
+
       should "return 0 if there are no documents in the collection" do
         @document.delete_all
         @document.count.should == 0
       end
-  
+
       should "return 0 if the collection does not exist" do
         klass = Class.new do
           include MongoMapper::Document
           collection 'foobarbazwickdoesnotexist'
         end
-  
+
         klass.count.should == 0
       end
-  
+
       should "return count for matching documents if conditions provided" do
         @document.count(:age => 27).should == 1
       end
-  
+
       should "convert the conditions to mongo criteria" do
         @document.count(:age => [26, 27]).should == 2
       end
     end
-  
+
     context "Indexing" do
       setup do
         @document.collection.drop_indexes
       end
-  
+
       should "allow creating index for a key" do
         index_name = nil
         lambda {
           index_name = @document.ensure_index :first_name
         }.should change { @document.collection.index_information.size }.by(1)
-        
+
         index_name.should == 'first_name_1'
         index = @document.collection.index_information[index_name]
         index.should_not be_nil
         index.should include(['first_name', 1])
       end
-  
+
       should "allow creating unique index for a key" do
         @document.collection.expects(:create_index).with(:first_name, true)
         @document.ensure_index :first_name, :unique => true
       end
-  
+
       should "allow creating index on multiple keys" do
         index_name = nil
         lambda {
           index_name = @document.ensure_index [[:first_name, 1], [:last_name, -1]]
         }.should change { @document.collection.index_information.size }.by(1)
-        
+
         index_name.should == 'first_name_1_last_name_-1'
-        
+
         index = @document.collection.index_information[index_name]
         index.should_not be_nil
         index.should include(['first_name', 1])
         index.should include(['last_name', -1])
       end
-  
+
       should "work with :index shortcut when defining key" do
         @document.expects(:ensure_index).with('father').returns(nil)
         @document.key :father, String, :index => true
       end
     end
   end # Document Class Methods
-  
+
   context "Saving a new document" do
     setup do
       @doc = @document.new(:first_name => 'John', :age => '27')
       @doc.save
     end
-  
+
     should "insert document into the collection" do
       @document.count.should == 1
     end
-  
+
     should "assign an id for the document" do
       @doc.id.should_not be(nil)
       @doc.id.size.should == 24
     end
-  
+
     should "save attributes" do
       @doc.first_name.should == 'John'
       @doc.age.should == 27
     end
-  
+
     should "update attributes in the database" do
       from_db = @document.find(@doc.id)
       from_db.should == @doc
       from_db.first_name.should == 'John'
       from_db.age.should == 27
     end
+
+    should "allow to add custom attributes to the document" do
+      @doc = @document.new(:first_name => 'David', :age => '26', :gender => 'male', :tags => [1, "2"])
+      @doc.save
+      from_db = @document.find(@doc.id)
+      from_db.gender.should == 'male'
+      from_db.tags.should == [1, "2"]
+    end
+
+    should "allow to use custom methods to assign properties" do
+      person = RealPerson.new(:realname => "David")
+      person.save
+      from_db = RealPerson.find(person.id)
+      from_db.name.should == "David"
+    end
   end
-  
+
   context "Saving an existing document" do
     setup do
       @doc = @document.create(:first_name => 'John', :age => '27')
@@ -559,66 +778,80 @@ class DocumentTest < Test::Unit::TestCase
       @doc.age = 30
       @doc.save
     end
-  
+
     should "not insert document into collection" do
       @document.count.should == 1
     end
-      
+
     should "update attributes" do
       @doc.first_name.should == 'Johnny'
       @doc.age.should == 30
     end
-  
+
     should "update attributes in the database" do
       from_db = @document.find(@doc.id)
       from_db.first_name.should == 'Johnny'
       from_db.age.should == 30
     end
+
+    should "allow to update custom attributes" do
+      @doc = @document.new(:first_name => 'David', :age => '26', :gender => 'male')
+      @doc.gender = 'Male'
+      @doc.save
+      from_db = @document.find(@doc.id)
+      from_db.gender.should == 'Male'
+    end
   end
-  
+
   context "Calling update attributes on a new document" do
     setup do
       @doc = @document.new(:first_name => 'John', :age => '27')
       @doc.update_attributes(:first_name => 'Johnny', :age => 30)
     end
-  
+
     should "insert document into the collection" do
       @document.count.should == 1
     end
-  
+
     should "assign an id for the document" do
       @doc.id.should_not be(nil)
       @doc.id.size.should == 24
     end
-  
+
     should "save attributes" do
       @doc.first_name.should == 'Johnny'
       @doc.age.should == 30
     end
-  
+
     should "update attributes in the database" do
       from_db = @document.find(@doc.id)
       from_db.should == @doc
       from_db.first_name.should == 'Johnny'
       from_db.age.should == 30
     end
+
+    should "allow to update custom attributes" do
+      @doc.update_attributes(:gender => 'mALe')
+      from_db = @document.find(@doc.id)
+      from_db.gender.should == 'mALe'
+    end
   end
-  
+
   context "Updating an existing document using update attributes" do
     setup do
       @doc = @document.create(:first_name => 'John', :age => '27')
       @doc.update_attributes(:first_name => 'Johnny', :age => 30)
     end
-  
+
     should "not insert document into collection" do
       @document.count.should == 1
     end
-  
+
     should "update attributes" do
       @doc.first_name.should == 'Johnny'
       @doc.age.should == 30
     end
-  
+
     should "update attributes in the database" do
       from_db = @document.find(@doc.id)
       from_db.first_name.should == 'Johnny'
@@ -626,38 +859,56 @@ class DocumentTest < Test::Unit::TestCase
     end
   end
   
+  context "update_attributes" do
+    setup do
+      @document.key :foo, String, :required => true
+    end
+    
+    should "return true if document valid" do
+      @document.new.update_attributes(:foo => 'bar').should be_true
+    end
+    
+    should "return false if document not valid" do
+      @document.new.update_attributes({}).should be_false
+    end
+  end
+
   context "Destroying a document that exists" do
     setup do
       @doc = @document.create(:first_name => 'John', :age => '27')
       @doc.destroy
     end
-  
+
     should "remove the document from the collection" do
       @document.count.should == 0
     end
-  
+
     should "raise error if assignment is attempted" do
       lambda { @doc.first_name = 'Foo' }.should raise_error(TypeError)
     end
+
+    should "do nothing if destroy is called again" do
+      @doc.destroy.should be_false
+    end
   end
-  
+
   context "Destroying a document that is a new" do
     setup do
       setup do
         @doc = @document.new(:first_name => 'John Nunemaker', :age => '27')
         @doc.destroy
       end
-  
+
       should "not affect collection count" do
         @document.collection.count.should == 0
       end
-  
+
       should "raise error if assignment is attempted" do
         lambda { @doc.first_name = 'Foo' }.should raise_error(TypeError)
       end
     end
   end
-  
+
   context "timestamping" do
     should "set created_at and updated_at on create" do
       doc = @document.new(:first_name => 'John', :age => 27)
@@ -667,7 +918,7 @@ class DocumentTest < Test::Unit::TestCase
       doc.created_at.should_not be(nil)
       doc.updated_at.should_not be(nil)
     end
-  
+
     should "set updated_at on field update but leave created_at alone" do
       doc = @document.create(:first_name => 'John', :age => 27)
       old_created_at = doc.created_at
@@ -677,17 +928,17 @@ class DocumentTest < Test::Unit::TestCase
       doc.created_at.should == old_created_at
       doc.updated_at.should_not == old_updated_at
     end
-    
+
     should "set updated_at on document update but leave created_at alone" do
       doc = @document.create(:first_name => 'John', :age => 27)
       old_created_at = doc.created_at
       old_updated_at = doc.updated_at
       sleep 1 # this annoys me
       @document.update(doc._id, { :first_name => 'Johnny' })
-      
+
       from_db = @document.find(doc.id)
       from_db.created_at.to_i.should == old_created_at.to_i
       from_db.updated_at.to_i.should_not == old_updated_at.to_i
     end
   end
-end
+end