langalex-couch_potato 0.1 → 0.1.1

data/lib/couch_potato/versioning.rb

@@ -36,7 +36,7 @@ module CouchPotato
         copy._id = nil
         copy._rev = nil
         copy.master_version_id = self._id
-        copy.save
+        copy.save_without_callbacks
         self.master_version_id ||= self.id
         self.version += 1
       end

data/spec/attributes_spec.rb

@@ -1,22 +1,42 @@
 require File.dirname(__FILE__) + '/spec_helper'
 
-describe 'attributes=' do
-  
-  class Plant
-    include CouchPotato::Persistence
-    property :leaf_count
+class Plant
+  include CouchPotato::Persistence
+  property :leaf_count
+end
+
+describe "attributes" do
+  before(:all) do
+    CouchPotato::Persistence.Db!
   end
   
-  it "should assign the attributes" do
-    plant = Plant.new 
-    plant.attributes = {:leaf_count => 1}
-    plant.leaf_count.should == 1
+  describe 'attributes=' do
+    it "should assign the attributes" do
+      plant = Plant.new 
+      plant.attributes = {:leaf_count => 1}
+      plant.leaf_count.should == 1
+    end
   end
-end
 
-describe "attributes" do
-  it "should return the attributes" do
-    plant = Plant.new :leaf_count => 1
-    plant.attributes .should == {:leaf_count => 1}
+  describe "attributes" do
+    it "should return the attributes" do
+      plant = Plant.new :leaf_count => 1
+      plant.attributes .should == {:leaf_count => 1}
+    end
   end
-end
+
+  describe 'update_attributes' do
+    it "should assign the attributes" do
+      plant = Plant.new 
+      plant.update_attributes :leaf_count => 1
+      plant.leaf_count.should == 1
+    end
+
+    it "should save the object" do
+      plant = Plant.new 
+      plant.update_attributes :leaf_count => 1
+      plant.should_not be_new_document
+    end
+  end
+end
+

data/spec/belongs_to_spec.rb

@@ -1,6 +1,10 @@
 require File.dirname(__FILE__) + '/spec_helper'
 
 describe 'belongs_to' do
+  before(:all) do
+    CouchPotato::Persistence.Db!
+  end
+  
   before(:each) do
     @commenter = Commenter.create!
   end
@@ -25,6 +29,20 @@ describe 'belongs_to' do
     c.commenter_id.should == @commenter.id
   end
   
+  it "should unassign the parent object" do
+    c = Comment.new :title => 'title', :commenter => stub('comenter', :id => 1)
+    c.commenter = nil
+    c.commenter.should be_nil
+    c.commenter_id.should be_nil
+  end
+  
+  it "should unassign the parent object id" do
+    c = Comment.new :title => 'title', :commenter => stub('comenter', :id => 1)
+    c.commenter_id = nil
+    c.commenter.should be_nil
+    c.commenter_id.should be_nil
+  end
+  
   it "should persist the link to the parent object" do
     c = Comment.new :title => 'title'
     c.commenter_id = @commenter.id

data/spec/callbacks_spec.rb

@@ -27,6 +27,9 @@ class CallbackRecorder
 end
 
 describe "multiple callbacks at once" do
+  before(:all) do
+    CouchPotato::Persistence.Db!
+  end
   class Monkey
     include CouchPotato::Persistence
     attr_accessor :eaten_banana, :eaten_apple
@@ -51,6 +54,9 @@ describe "multiple callbacks at once" do
 end
 
 describe 'create callbacks' do
+  before(:all) do
+    CouchPotato::Persistence.Db!
+  end
   
   before(:each) do
     @recorder = CallbackRecorder.new
@@ -131,41 +137,43 @@ describe 'create callbacks' do
 end
 
 describe "update callbacks" do
+  before(:all) do
+    CouchPotato::Persistence.Db!
+  end
   
   before(:each) do
     @recorder = CallbackRecorder.create! :required_property => 1
+    @recorder.required_property = 2
     @recorder.callbacks.clear
   end
   
   describe "successful update" do
     
-    it "should call before_validation_on_update" do
+    before(:each) do
       @recorder.save!
+    end
+    
+    it "should call before_validation_on_update" do
       @recorder.callbacks.should include(:before_validation_on_update)
     end
     
     it "should call before_validation_on_save" do
-      @recorder.save!
       @recorder.callbacks.should include(:before_validation_on_save)
     end
     
     it "should call before_save" do
-      @recorder.save!
       @recorder.callbacks.should include(:before_save)
     end
     
     it "should call after_save" do
-      @recorder.save!
       @recorder.callbacks.should include(:after_save)
     end
     
     it "should call before_update" do
-      @recorder.save!
       @recorder.callbacks.should include(:before_update)
     end
     
     it "should call after_update" do
-      @recorder.save!
       @recorder.callbacks.should include(:after_update)
     end
     
@@ -175,35 +183,30 @@ describe "update callbacks" do
     
     before(:each) do
        @recorder.required_property = nil
+       @recorder.save
     end
     
     it "should call before_validation_on_update" do
-      @recorder.save
       @recorder.callbacks.should include(:before_validation_on_update)
     end
     
     it "should call before_validation_on_save" do
-      @recorder.save
       @recorder.callbacks.should include(:before_validation_on_save)
     end
     
     it "should not call before_save" do
-      @recorder.save
       @recorder.callbacks.should_not include(:before_save)
     end
     
     it "should not call after_save" do
-      @recorder.save
       @recorder.callbacks.should_not include(:after_save)
     end
     
     it "should not call before_update" do
-      @recorder.save
       @recorder.callbacks.should_not include(:before_update)
     end
     
     it "should not call after_update" do
-      @recorder.save
       @recorder.callbacks.should_not include(:after_update)
     end
     
@@ -212,6 +215,10 @@ describe "update callbacks" do
 end
 
 describe "destroy callbacks" do
+  before(:all) do
+    CouchPotato::Persistence.Db!
+  end
+  
   before(:each) do
     @recorder = CallbackRecorder.create! :required_property => 1
     @recorder.callbacks.clear
@@ -226,4 +233,16 @@ describe "destroy callbacks" do
     @recorder.destroy
     @recorder.callbacks.should include(:after_destroy)
   end
+end
+
+describe 'save_without_callbacks' do
+  before(:all) do
+    CouchPotato::Persistence.Db!
+  end
+  
+  it "should not run any callbacks" do
+    @recorder = CallbackRecorder.new
+    @recorder.save_without_callbacks
+    @recorder.callbacks.should be_empty
+  end
 end

data/spec/create_spec.rb

@@ -1,6 +1,10 @@
 require File.dirname(__FILE__) + '/spec_helper'
 
 describe "create" do
+  before(:all) do
+    CouchPotato::Persistence.Db!
+  end
+  
   describe "succeeds" do
     before(:each) do
       @comment = Comment.new :title => 'my_title'
@@ -33,6 +37,7 @@ describe "create" do
   describe "fails" do
     before(:each) do
       CouchPotato::Persistence.Db.delete!
+      CouchPotato::Persistence.Db!
       @comment = Comment.new
       @comment.save
     end

data/spec/custom_view_spec.rb

@@ -0,0 +1,44 @@
+require File.dirname(__FILE__) + '/spec_helper'
+
+class Build
+  include CouchPotato::Persistence
+  
+  property :state
+  property :time
+  
+  view :timeline, :key => :time
+  view :minimal_timeline, :key => :time, :properties => [:state]
+end
+
+describe 'custom view' do
+  before(:each) do
+    CouchPotato::Persistence.Db.delete!
+    CouchPotato::Persistence.Db!
+  end
+  
+  it "should return instances of the class" do
+    Build.db.save({:state => 'success', :time => '2008-01-01'})
+    Build.timeline.map(&:class).should == [Build]
+  end
+  
+  it "should assign all properties to the objects by default" do
+    Build.db.save({:state => 'success', :time => '2008-01-01'})
+    Build.timeline.first.state.should == 'success'
+    Build.timeline.first.time.should == '2008-01-01'
+  end
+  
+  it "should assign the configured properties" do
+    Build.db.save({:state => 'success', :time => '2008-01-01'})
+    Build.minimal_timeline.first.state.should == 'success'
+  end
+  
+  it "should assign the id when properties are configured" do
+    id = Build.db.save({:state => 'success', :time => '2008-01-01'})['id']
+    Build.minimal_timeline.first._id.should == id
+  end
+  
+  it "should not assign the properties not configured" do
+    Build.db.save({:state => 'success', :time => '2008-01-01'})
+    Build.minimal_timeline.first.time.should be_nil
+  end
+end

data/spec/destroy_spec.rb

@@ -1,6 +1,10 @@
 require File.dirname(__FILE__) + '/spec_helper'
 
 describe 'destroy' do
+  before(:all) do
+    CouchPotato::Persistence.Db!
+  end
+  
   before(:each) do
     @comment = Comment.create! :title => 'title'
     @comment_id = @comment.id

data/spec/dirty_attributes_spec.rb

@@ -0,0 +1,82 @@
+require File.dirname(__FILE__) + '/spec_helper'
+
+class Plate
+  include CouchPotato::Persistence
+  
+  property :food
+end
+
+describe 'dirty attribute tracking' do
+  before(:all) do
+    CouchPotato::Persistence.Db!
+  end
+  
+  describe "save" do
+    it "should not save when nothing dirty" do
+      plate = Plate.create! :food => 'sushi'
+      old_rev = plate._rev
+      plate.save
+      plate._rev.should == old_rev
+    end
+    
+    it "should save when there are dirty attributes" do
+      plate = Plate.create! :food => 'sushi'
+      old_rev = plate._rev
+      plate.food = 'burger'
+      plate.save
+      plate._rev.should_not == old_rev
+    end
+  end
+  
+  describe "newly created object" do
+    
+    before(:each) do
+      @plate = Plate.new :food => 'sushi'
+    end
+    
+    describe "access old values" do
+      it "should return the old value" do
+        @plate.food = 'burger'
+        @plate.food_was.should == 'sushi'
+      end
+    end
+
+    describe "check for dirty" do
+      it "should return true if attribute changed" do
+        @plate.food = 'burger'
+        @plate.should be_food_changed
+      end
+
+      it "should return false if attribute not changed" do
+        @plate.should_not be_food_changed
+      end
+    end
+  end
+  
+  describe "object loaded from database" do
+    before(:each) do
+      @plate = Plate.create! :food => 'sushi'
+      @plate = Plate.get @plate._id
+    end
+    
+    describe "access old values" do
+      it "should return the old value" do
+        @plate.food = 'burger'
+        @plate.food_was.should == 'sushi'
+      end
+    end
+
+    describe "check for dirty" do
+      it "should return true if attribute changed" do
+        @plate.food = 'burger'
+        @plate.should be_food_changed
+      end
+
+      it "should return false if attribute not changed" do
+        @plate.should_not be_food_changed
+      end
+    end
+  end
+  
+  
+end

data/spec/find_spec.rb

@@ -1,6 +1,9 @@
 require File.dirname(__FILE__) + '/spec_helper'
 
 describe "find" do
+  before(:all) do
+    CouchPotato::Persistence.Db!
+  end
   before(:each) do
     @comment = Comment.create! :title => 'title'
   end
@@ -25,8 +28,8 @@ end
 describe 'first' do
   before(:each) do
     CouchPotato::Persistence.Db.delete!
+    CouchPotato::Persistence.Db!
     @comment = Comment.create! :title => 'title'
-    Comment.create! :title => 'title3'
   end
   
   it "should find the first matching object" do
@@ -41,7 +44,7 @@ end
 describe 'last' do
   before(:each) do
     CouchPotato::Persistence.Db.delete!
-    Comment.create! :title => 'title'
+    CouchPotato::Persistence.Db!
     @comment = Comment.create! :title => 'title'
   end
   
@@ -58,12 +61,16 @@ end
 describe 'all' do
   before(:each) do
     CouchPotato::Persistence.Db.delete!
+    CouchPotato::Persistence.Db!
     @comment = Comment.create! :title => 'title'
     @comment2 = Comment.create! :title => 'title'
   end
   
   it "should find the matching objects" do
-    Comment.all(:title =>'title').should == [@comment, @comment2]
+    comments = Comment.all(:title =>'title')
+    comments.size.should == 2
+    comments.should include(@comment)
+    comments.should include(@comment2)
   end
   
   it "should return [] if nothing found" do
@@ -74,6 +81,7 @@ end
 describe 'count' do
   before(:each) do
     CouchPotato::Persistence.Db.delete!
+    CouchPotato::Persistence.Db!
     @comment = Comment.create! :title => 'title'
     @comment2 = Comment.create! :title => 'title'
   end