couchobject 0.5.0 → 0.6.0

data/spec/persistable/timestamps_spec.rb

@@ -0,0 +1,76 @@
+require File.dirname(__FILE__) + '/persistable_helper.rb'
+
+describe CouchObject::Persistable, "timestamp related actions:" do
+  before(:each) do
+    @bike = Bike.new
+    @timestampclass = WithTimeStamp.new
+    
+    @db = mock("mock db")
+    
+    content = HTTPResponse.new(JSON.unparse({
+      "_id" => "123BAC", 
+      "_rev" => "946B7D1C", 
+      "created_at" => Time.new,
+      "updated_at" => Time.new,
+      "class" => "WithTimeStamp",
+      "attributes" => {
+        "wheels" => 3
+      }
+    }))
+    
+    CouchObject::Response.stub!(:body).and_return(content)
+    
+    @empty_response = {}    
+    @ok_response = {"ok" => true}
+    @document_response = CouchObject::Response.new(content)
+  end
+
+  it "timestamps should be nil for classes that don't include them" do
+    @bike.created_at.should == nil
+    @bike.updated_at.should == nil
+  end
+  
+  it "timestamps should be set when saving an object" do
+    CouchObject::Database.should_receive(:open).and_return(@db)
+    timestampclass_json = @timestampclass.to_json
+    
+    @timestampclass.stub!(:to_json).and_return(timestampclass_json)
+    @db.should_receive(:post).
+      with("", timestampclass_json).and_return(@document_response)
+
+    @timestampclass.save("foo")
+    
+    @timestampclass.created_at.should_not == nil
+    @timestampclass.updated_at.should_not == nil
+  end
+
+  it "should update updated_at when saving an object again" do
+    CouchObject::Database.stub!(:open).and_return(@db)
+    timestampclass_json = @timestampclass.to_json
+    
+    @timestampclass.stub!(:to_json).and_return(timestampclass_json)
+    @db.should_receive(:post).
+      with("", timestampclass_json).and_return(@document_response)
+    @db.should_receive(:put).
+      with("123BAC", timestampclass_json).and_return(@document_response)
+
+    @timestampclass.save("foo")
+    updated_at = @timestampclass.updated_at
+    
+    @timestampclass.save
+    
+    @timestampclass.updated_at.should_not == updated_at    
+  end
+
+  it "loaded object should have their timestamps set" do
+    CouchObject::Database.should_receive(:open).and_return(@db)
+    @db.should_receive(:get).with("123BAC").and_return(@document_response)
+
+    timestampclass = WithTimeStamp.get_by_id("123BAC", "foo")
+
+    timestampclass.class.should == WithTimeStamp
+    timestampclass.created_at.should_not == nil
+    timestampclass.updated_at.should_not == nil
+  end
+end
+

data/spec/persistable/unsaved_changes.rb

@@ -0,0 +1,211 @@
+require File.dirname(__FILE__) + '/persistable_helper.rb'
+
+describe CouchObject::Persistable, "for loading objects:" do
+  before(:each) do
+    @bike = Bike.new
+    @with_location = WithStorageLocation.new
+    
+    @db = mock("mock db")
+    
+    @content_motor_bike = %{
+      {
+        "_id":"123BAC",
+        "_rev":"946B7D1C",
+        "class":"OtherMotorBike",
+        "attributes":
+        {
+          "wheels":2
+        }
+      }
+    }
+
+    @normal_content_motor_bike = %{
+      {
+        "_id":"123BAC",
+        "_rev":"946B7D1C",
+        "class":"MotorBike",
+        "attributes":
+        {
+          "wheels":2
+        }
+      }
+    }
+  
+    content = HTTPResponse.new(JSON.unparse({
+      "_id" => "123BAC", 
+      "_rev" => "946B7D1C", 
+    }))        
+    @document_response = CouchObject::Response.new(content)
+
+    @apartment_building_content = %{ {"_id":"A26EDC59D99D50C2CB2615404D357E49","_rev":"1756574922","class":"OtherApartmentBuilding","attributes":{}}
+    }
+
+    @apartment_building_inhabitants = %{{"total_rows":2,"offset":0,"rows":[{"id":"241757E4D7D2272C78048B4007399837","key":["12A64B5156002BFD66F23F559C5AD5FA","inhabitants"],"value":{"_id":"241757E4D7D2272C78048B4007399837","_rev":"2508041460","class":"OtherInhabitant","attributes":{},"belongs_to":{"inhabitants":"A26EDC59D99D50C2CB2615404D357E49"}}},{"id":"E94C93DEDB94BFC30B9AB396D4874D52","key":["12A64B5156002BFD66F23F559C5AD5FA","inhabitants"],"value":{"_id":"E94C93DEDB94BFC30B9AB396D4874D52","_rev":"960098874","class":"OtherInhabitant","attributes":{},"belongs_to":{"inhabitants":"A26EDC59D99D50C2CB2615404D357E49"}}}]}
+    }
+    
+    CouchObject::Database.stub!(:open).and_return(@db)
+  end
+ 
+  it "should return true for new objects" do
+    mbike = OtherMotorBike.new
+    mbike.unsaved_changes?.should == true
+    mbike.use_smart_save.should == true
+  end
+  
+  it "should return true for new objects that haven't activated the smart save feature" do
+    mbike = MotorBike.new
+    mbike.unsaved_changes?.should == true
+    mbike.use_smart_save.should == false
+  end
+  
+  it "should know if an object has unsaved changes" do 
+    response = HTTPResponse.new(@content_motor_bike)
+    
+    @db.should_receive(:get).with("123BAC").and_return(response)
+    
+    mbike = MotorBike.get_by_id("123BAC", "foo")
+    mbike.id.should == "123BAC"
+    mbike.use_smart_save.should == true
+    
+    mbike.unsaved_changes?.should == false
+    mbike.wheels = 3
+    mbike.unsaved_changes?.should == true
+    
+  end
+
+  it "should know if an object has unsaved for an object where the instance is forced into smart save mode" do 
+    response = HTTPResponse.new(@normal_content_motor_bike)
+    
+    @db.should_receive(:get).with("123BAC").and_return(response)
+    
+    mbike = MotorBike.get_with_smart_save("123BAC", "foo")
+    mbike.id.should == "123BAC"
+    mbike.use_smart_save.should == true
+    
+    mbike.unsaved_changes?.should == false
+    mbike.wheels = 3
+    mbike.unsaved_changes?.should == true
+    
+  end
+
+  it "should save an object that does not unsaved changes if it hasn't activated smart_save" do
+    response = HTTPResponse.new(@normal_content_motor_bike)
+    
+    @db.should_receive(:get).with("123BAC").and_return(response)
+    @db.should_receive(:put).and_return(@document_response)
+    
+    mbike = MotorBike.get_by_id("123BAC", "foo")
+    mbike.use_smart_save.should == false
+
+    # unsaved_changes should always be true for classes that don't have
+    # smart_save activated
+    mbike.unsaved_changes?.should == true
+    mbike.save
+
+  end
+  
+  it "should not save an object that does not have unsaved changes" do
+    response = HTTPResponse.new(@content_motor_bike)
+    
+    @db.should_receive(:get).with("123BAC").and_return(response)
+    @db.should_not_receive(:put)
+    
+    mbike = OtherMotorBike.get_by_id("123BAC", "foo")
+
+    mbike.unsaved_changes?.should == false
+    mbike.save
+
+  end
+
+  it "should not have unsaved_changes after having been saved" do
+    response = HTTPResponse.new(@content_motor_bike)
+    
+    @db.should_receive(:get).with("123BAC").and_return(response)
+    @db.should_receive(:put).and_return(@document_response)
+    
+    mbike = OtherMotorBike.get_by_id("123BAC", "foo")
+
+    mbike.unsaved_changes?.should == false
+    mbike.wheels = 3
+    mbike.unsaved_changes?.should == true
+    mbike.save
+    mbike.unsaved_changes?.should == false
+
+  end
+
+  it "should only save relations that need to be saved" do
+
+    response = HTTPResponse.new(@apartment_building_content)
+    response_relation = HTTPResponse.new(@apartment_building_inhabitants)
+    
+    CouchObject::Database.should_receive(:open).exactly(3).and_return(@db)
+    
+    @db.should_receive(:get).with("foo").and_return(response)
+      
+    # has_many relations are loaded
+    @db.should_receive(:get).
+      with("_view/couch_object_has_many_relations/related_documents?" + \
+      "key=[%22A26EDC59D99D50C2CB2615404D357E49%22,%22inhabitants%22]").
+      and_return(response_relation)
+    
+    @db.should_receive(:put).once.and_return(@document_response)
+    
+        
+    apartment_building = OtherApartmentBuilding.get_by_id("foo")
+    apartment_building.location.should_not == nil
+        
+    # The content should be loaded on first request!
+    apartment_building.inhabitants.size.should == 2
+    apartment_building.inhabitants.first.change
+    apartment_building.inhabitants.first.unsaved_changes?.should == true
+    apartment_building.inhabitants.last.unsaved_changes?.should == false
+    apartment_building.unsaved_changes?.should == false
+    apartment_building.save
+    
+  end
+
+  it "a class in a belongs_to relationship should notice if its parent changes" do
+
+    response = HTTPResponse.new(@apartment_building_content)
+    response_relation = HTTPResponse.new(@apartment_building_inhabitants)
+    
+    CouchObject::Database.should_receive(:open).exactly(4).and_return(@db)
+    
+    @db.should_receive(:get).with("foo").and_return(response)
+      
+    # has_many relations are loaded
+    @db.should_receive(:get).
+      with("_view/couch_object_has_many_relations/related_documents?" + \
+      "key=[%22A26EDC59D99D50C2CB2615404D357E49%22,%22inhabitants%22]").
+      and_return(response_relation)
+
+    # For the second apartment building
+    @db.should_receive(:post).once.and_return(@document_response)    
+    # And it's inhabitant which should be updated with the new parent
+    @db.should_receive(:put).once.and_return(@document_response)
+    
+    apartment_building = OtherApartmentBuilding.get_by_id("foo")
+    second_apartment_building = OtherApartmentBuilding.new
+    second_apartment_building.unsaved_changes?.should == true
+        
+    # The content should be loaded on first request!
+    apartment_building.inhabitants.size.should == 2
+    apartment_building.inhabitants.first.unsaved_changes?.should == false
+    apartment_building.inhabitants.last.unsaved_changes?.should == false
+    apartment_building.unsaved_changes?.should == false
+
+    second_apartment_building.inhabitants << apartment_building.inhabitants.first
+    second_apartment_building.inhabitants.size.should == 1
+    apartment_building.inhabitants.size.should == 1
+    apartment_building.inhabitants.first.unsaved_changes?.should == false
+    second_apartment_building.inhabitants.first.unsaved_changes?.should == true
+    
+    # Should not save anything at all...
+    apartment_building.save
+    
+    # Should save both documents
+    second_apartment_building.save
+    
+  end
+
+end

data/spec/persistable/unsaved_changes_spec.rb

@@ -0,0 +1,211 @@
+require File.dirname(__FILE__) + '/persistable_helper.rb'
+
+describe CouchObject::Persistable, "for loading objects:" do
+  before(:each) do
+    @bike = Bike.new
+    @with_location = WithStorageLocation.new
+    
+    @db = mock("mock db")
+    
+    @content_motor_bike = %{
+      {
+        "_id":"123BAC",
+        "_rev":"946B7D1C",
+        "class":"OtherMotorBike",
+        "attributes":
+        {
+          "wheels":2
+        }
+      }
+    }
+
+    @normal_content_motor_bike = %{
+      {
+        "_id":"123BAC",
+        "_rev":"946B7D1C",
+        "class":"MotorBike",
+        "attributes":
+        {
+          "wheels":2
+        }
+      }
+    }
+  
+    content = HTTPResponse.new(JSON.unparse({
+      "_id" => "123BAC", 
+      "_rev" => "946B7D1C", 
+    }))        
+    @document_response = CouchObject::Response.new(content)
+
+    @apartment_building_content = %{ {"_id":"A26EDC59D99D50C2CB2615404D357E49","_rev":"1756574922","class":"OtherApartmentBuilding","attributes":{}}
+    }
+
+    @apartment_building_inhabitants = %{{"total_rows":2,"offset":0,"rows":[{"id":"241757E4D7D2272C78048B4007399837","key":["12A64B5156002BFD66F23F559C5AD5FA","inhabitants"],"value":{"_id":"241757E4D7D2272C78048B4007399837","_rev":"2508041460","class":"OtherInhabitant","attributes":{},"belongs_to":{"inhabitants":"A26EDC59D99D50C2CB2615404D357E49"}}},{"id":"E94C93DEDB94BFC30B9AB396D4874D52","key":["12A64B5156002BFD66F23F559C5AD5FA","inhabitants"],"value":{"_id":"E94C93DEDB94BFC30B9AB396D4874D52","_rev":"960098874","class":"OtherInhabitant","attributes":{},"belongs_to":{"inhabitants":"A26EDC59D99D50C2CB2615404D357E49"}}}]}
+    }
+    
+    CouchObject::Database.stub!(:open).and_return(@db)
+  end
+ 
+  it "should return true for new objects" do
+    mbike = OtherMotorBike.new
+    mbike.unsaved_changes?.should == true
+    mbike.use_smart_save.should == true
+  end
+  
+  it "should return true for new objects that haven't activated the smart save feature" do
+    mbike = MotorBike.new
+    mbike.unsaved_changes?.should == true
+    mbike.use_smart_save.should == false
+  end
+  
+  it "should know if an object has unsaved changes" do 
+    response = HTTPResponse.new(@content_motor_bike)
+    
+    @db.should_receive(:get).with("123BAC").and_return(response)
+    
+    mbike = MotorBike.get_by_id("123BAC", "foo")
+    mbike.id.should == "123BAC"
+    mbike.use_smart_save.should == true
+    
+    mbike.unsaved_changes?.should == false
+    mbike.wheels = 3
+    mbike.unsaved_changes?.should == true
+    
+  end
+
+  it "should know if an object has unsaved for an object where the instance is forced into smart save mode" do 
+    response = HTTPResponse.new(@normal_content_motor_bike)
+    
+    @db.should_receive(:get).with("123BAC").and_return(response)
+    
+    mbike = MotorBike.get_with_smart_save("123BAC", "foo")
+    mbike.id.should == "123BAC"
+    mbike.use_smart_save.should == true
+    
+    mbike.unsaved_changes?.should == false
+    mbike.wheels = 3
+    mbike.unsaved_changes?.should == true
+    
+  end
+
+  it "should save an object that does not unsaved changes if it hasn't activated smart_save" do
+    response = HTTPResponse.new(@normal_content_motor_bike)
+    
+    @db.should_receive(:get).with("123BAC").and_return(response)
+    @db.should_receive(:put).and_return(@document_response)
+    
+    mbike = MotorBike.get_by_id("123BAC", "foo")
+    mbike.use_smart_save.should == false
+
+    # unsaved_changes should always be true for classes that don't have
+    # smart_save activated
+    mbike.unsaved_changes?.should == true
+    mbike.save
+
+  end
+  
+  it "should not save an object that does not have unsaved changes" do
+    response = HTTPResponse.new(@content_motor_bike)
+    
+    @db.should_receive(:get).with("123BAC").and_return(response)
+    @db.should_not_receive(:put)
+    
+    mbike = OtherMotorBike.get_by_id("123BAC", "foo")
+
+    mbike.unsaved_changes?.should == false
+    mbike.save
+
+  end
+
+  it "should not have unsaved_changes after having been saved" do
+    response = HTTPResponse.new(@content_motor_bike)
+    
+    @db.should_receive(:get).with("123BAC").and_return(response)
+    @db.should_receive(:put).and_return(@document_response)
+    
+    mbike = OtherMotorBike.get_by_id("123BAC", "foo")
+
+    mbike.unsaved_changes?.should == false
+    mbike.wheels = 3
+    mbike.unsaved_changes?.should == true
+    mbike.save
+    mbike.unsaved_changes?.should == false
+
+  end
+
+  it "should only save relations that need to be saved" do
+
+    response = HTTPResponse.new(@apartment_building_content)
+    response_relation = HTTPResponse.new(@apartment_building_inhabitants)
+    
+    CouchObject::Database.should_receive(:open).exactly(3).and_return(@db)
+    
+    @db.should_receive(:get).with("foo").and_return(response)
+      
+    # has_many relations are loaded
+    @db.should_receive(:get).
+      with("_view/couch_object_has_many_relations/related_documents?" + \
+      "key=[%22A26EDC59D99D50C2CB2615404D357E49%22,%22inhabitants%22]").
+      and_return(response_relation)
+    
+    @db.should_receive(:put).once.and_return(@document_response)
+    
+        
+    apartment_building = OtherApartmentBuilding.get_by_id("foo")
+    apartment_building.location.should_not == nil
+        
+    # The content should be loaded on first request!
+    apartment_building.inhabitants.size.should == 2
+    apartment_building.inhabitants.first.change
+    apartment_building.inhabitants.first.unsaved_changes?.should == true
+    apartment_building.inhabitants.last.unsaved_changes?.should == false
+    apartment_building.unsaved_changes?.should == false
+    apartment_building.save
+    
+  end
+
+  it "a class in a belongs_to relationship should notice if its parent changes" do
+
+    response = HTTPResponse.new(@apartment_building_content)
+    response_relation = HTTPResponse.new(@apartment_building_inhabitants)
+    
+    CouchObject::Database.should_receive(:open).exactly(4).and_return(@db)
+    
+    @db.should_receive(:get).with("foo").and_return(response)
+      
+    # has_many relations are loaded
+    @db.should_receive(:get).
+      with("_view/couch_object_has_many_relations/related_documents?" + \
+      "key=[%22A26EDC59D99D50C2CB2615404D357E49%22,%22inhabitants%22]").
+      and_return(response_relation)
+
+    # For the second apartment building
+    @db.should_receive(:post).once.and_return(@document_response)    
+    # And it's inhabitant which should be updated with the new parent
+    @db.should_receive(:put).once.and_return(@document_response)
+    
+    apartment_building = OtherApartmentBuilding.get_by_id("foo")
+    second_apartment_building = OtherApartmentBuilding.new
+    second_apartment_building.unsaved_changes?.should == true
+        
+    # The content should be loaded on first request!
+    apartment_building.inhabitants.size.should == 2
+    apartment_building.inhabitants.first.unsaved_changes?.should == false
+    apartment_building.inhabitants.last.unsaved_changes?.should == false
+    apartment_building.unsaved_changes?.should == false
+
+    second_apartment_building.inhabitants << apartment_building.inhabitants.first
+    second_apartment_building.inhabitants.size.should == 1
+    apartment_building.inhabitants.size.should == 1
+    apartment_building.inhabitants.first.unsaved_changes?.should == false
+    second_apartment_building.inhabitants.first.unsaved_changes?.should == true
+    
+    # Should not save anything at all...
+    apartment_building.save
+    
+    # Should save both documents
+    second_apartment_building.save
+    
+  end
+
+end