toystore-mongo 0.8.0 → 0.8.1

data/lib/toy/mongo/querying.rb

@@ -81,8 +81,14 @@ module Toy
       def atomic_update_attributes(attrs={})
         self.attributes = attrs
         if valid?
-          atomic_update('$set' => persistable_changes)
-          true
+          run_callbacks(:save) do
+            run_callbacks(:update) do
+              criteria = {'_id' => id}
+              update   = {'$set' => persistable_changes}
+              store.client.update(criteria, update, {:safe => store.options[:safe]})
+              true
+            end
+          end
         else
           false
         end

data/lib/toy/mongo/version.rb

@@ -1,5 +1,5 @@
 module Toy
   module Mongo
-    VERSION = '0.8.0'
+    VERSION = '0.8.1'
   end
 end

data/spec/toy/mongo/querying_spec.rb

@@ -135,9 +135,41 @@ describe Toy::Mongo::Querying do
     end
 
     it "only persists changes" do
-      @user.should_receive(:atomic_update).with('$set' => {'name' => 'Frank'})
+      query = {'_id' => @user.id}
+      update = {'$set' => {'name' => 'Frank'}}
+      @user.store.client.should_receive(:update).with(query, update, {:safe => nil})
       @user.atomic_update_attributes(:name => 'Frank')
     end
+
+    it "persists changes with safe option from store" do
+      User.store(:mongo, STORE, :safe => true)
+      query = {'_id' => @user.id}
+      update = {'$set' => {'name' => 'Frank'}}
+      @user.store.client.should_receive(:update).with(query, update, {:safe => true})
+      @user.atomic_update_attributes(:name => 'Frank')
+    end
+
+    it "runs callbacks in correct order" do
+      doc = User.create.tap(&:clear_history)
+      doc.name = 'John Nunemaker'
+      doc.atomic_update_attributes
+      doc.history.should == [:before_save, :before_update, :after_update, :after_save]
+    end
+
+    it "persists changes that happen in callbacks" do
+      User.class_eval do
+        attribute :name_lower, String
+
+        before_save do |record|
+          record.name_lower = name
+        end
+      end
+
+      user = User.create(:name => 'Frank')
+      user.atomic_update_attributes('name' => 'John')
+      user.name_lower.should == 'John'
+      user.reload.name_lower.should == 'John'
+    end
   end
 
   describe "#get" do

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: toystore-mongo
 version: !ruby/object:Gem::Version 
-  hash: 63
+  hash: 61
   prerelease: 
   segments: 
   - 0
   - 8
-  - 0
-  version: 0.8.0
+  - 1
+  version: 0.8.1
 platform: ruby
 authors: 
 - John Nunemaker
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-09-23 00:00:00 Z
+date: 2011-09-27 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: plucky