rubydora 1.7.1 → 1.7.3

data/spec/lib/digital_object_spec.rb

@@ -2,7 +2,9 @@ require 'spec_helper'
 
 describe Rubydora::DigitalObject do
   before do
-    @mock_repository = Rubydora::Fc3Service.new({})
+    @mock_api = Rubydora::Fc3Service.new({})
+    @mock_api.stub(:repository_profile, {"repositoryVersion" => "3.4"})
+    @mock_repository = Rubydora::Repository.new({}, @mock_api)
   end
   describe "profile" do
     before(:each) do
@@ -10,7 +12,7 @@ describe Rubydora::DigitalObject do
     end
 
     it "should convert object profile to a simple hash" do
-      @mock_repository.should_receive(:object).with(:pid => 'pid').and_return("<objectProfile><a>1</a><b>2</b><objModels><model>3</model><model>4</model></objectProfile>")
+      @mock_api.should_receive(:object).with(:pid => 'pid').and_return("<objectProfile><a>1</a><b>2</b><objModels><model>3</model><model>4</model></objectProfile>")
       h = @object.profile
 
       h.should have_key("a")
@@ -23,28 +25,28 @@ describe Rubydora::DigitalObject do
     end
 
     it "should be frozen (to prevent modification)" do
-      @mock_repository.should_receive(:object).with(:pid => 'pid').and_return("<objectProfile><a>1</a><b>2</b><objModels><model>3</model><model>4</model></objectProfile>")
+      @mock_api.should_receive(:object).with(:pid => 'pid').and_return("<objectProfile><a>1</a><b>2</b><objModels><model>3</model><model>4</model></objectProfile>")
       h = @object.profile
 
       expect { h['asdf'] = 'asdf' }.to raise_error
     end
 
     it "should return nil for empty profile fields" do
-      @mock_repository.should_receive(:object).with(:pid => 'pid').and_return("<objectProfile><a></a></objectProfile>")
+      @mock_api.should_receive(:object).with(:pid => 'pid').and_return("<objectProfile><a></a></objectProfile>")
       @object.profile['a'].should be_nil
     end
 
     it "should throw exceptions that arise" do
-      @mock_repository.should_receive(:object).with(:pid => 'pid').and_raise(Net::HTTPBadResponse)
+      @mock_api.should_receive(:object).with(:pid => 'pid').and_raise(Net::HTTPBadResponse)
       expect { @object.profile }.to raise_error(Net::HTTPBadResponse)
     end
   end
 
   describe "initialize" do
     before(:each) do
-      @mock_repository.stub(:object) { raise RestClient::ResourceNotFound }
+      @mock_api.stub(:object) { raise RestClient::ResourceNotFound }
     end
-    subject { Rubydora::DigitalObject.new 'pid', @mock_repository }
+    subject { Rubydora::DigitalObject.new 'pid', @mock_api }
 
     it "should load a DigitalObject instance" do
       expect(subject).to be_a_kind_of(Rubydora::DigitalObject)
@@ -60,15 +62,15 @@ describe Rubydora::DigitalObject do
 
     it "should call ingest on save" do
       subject.stub(:datastreams) { {} }
-      expect(@mock_repository).to receive(:ingest).with(hash_including(:pid => 'pid')).and_return('pid')
+      expect(@mock_api).to receive(:ingest).with(hash_including(:pid => 'pid')).and_return('pid')
       subject.save
     end
 
     describe "without a provided pid" do
-      subject { Rubydora::DigitalObject.new nil, @mock_repository }
+      subject { Rubydora::DigitalObject.new nil, @mock_api }
       it "should create a new Fedora object with a generated PID if no PID is provided" do 
-        @mock_repository.should_receive(:ingest).with(hash_including(:pid => nil)).and_return('pid')
-        @mock_repository.should_receive(:datastreams).with(hash_including(:pid => 'pid')).and_raise(RestClient::ResourceNotFound)
+        @mock_api.should_receive(:ingest).with(hash_including(:pid => nil)).and_return('pid')
+        @mock_api.should_receive(:datastreams).with(hash_including(:pid => 'pid')).and_raise(RestClient::ResourceNotFound)
         subject.save
         subject.pid.should == 'pid'
       end
@@ -77,29 +79,29 @@ describe Rubydora::DigitalObject do
 
   describe "create" do
     it "should call the Fedora REST API to create a new object" do
-      @mock_repository.should_receive(:ingest).with(instance_of(Hash)).and_return("pid")
-      obj = Rubydora::DigitalObject.create "pid", { :a => 1, :b => 2}, @mock_repository
+      @mock_api.should_receive(:ingest).with(instance_of(Hash)).and_return("pid")
+      obj = Rubydora::DigitalObject.create "pid", { :a => 1, :b => 2}, @mock_api
       obj.should be_a_kind_of(Rubydora::DigitalObject)
     end
 
     it "should return a new object with the Fedora response pid when no pid is provided" do
-      @mock_repository.should_receive(:ingest).with(instance_of(Hash)).and_return("pid")
-      obj = Rubydora::DigitalObject.create "new", { :a => 1, :b => 2}, @mock_repository
+      @mock_api.should_receive(:ingest).with(instance_of(Hash)).and_return("pid")
+      obj = Rubydora::DigitalObject.create "new", { :a => 1, :b => 2}, @mock_api
       obj.should be_a_kind_of(Rubydora::DigitalObject)
       obj.pid.should == "pid"
     end
   end
 
-  describe "retreive" do
-    before(:each) do
-      @mock_repository.stub :datastreams do |hash|
-        "<objectDatastreams><datastream dsid='a'></datastream>><datastream dsid='b'></datastream>><datastream dsid='c'></datastream></objectDatastreams>"
+  describe "retreive datastreams" do
+    describe "without profiles (fedora < 3.6)" do 
+      before(:each) do
+        @mock_api.stub :datastreams do |hash|
+          "<objectDatastreams><datastream dsid='a'></datastream><datastream dsid='b'></datastream><datastream dsid='c'></datastream></objectDatastreams>"
+        end
+        @object = Rubydora::DigitalObject.new 'pid', @mock_api
+        @object.stub(:new_record? => false)
       end
-      @object = Rubydora::DigitalObject.new 'pid', @mock_repository
-      @object.stub(:new_record? => false)
-    end
 
-    describe "datastreams" do
       it "should provide a hash populated by the existing datastreams" do
 
         @object.datastreams.should have_key("a")
@@ -108,7 +110,7 @@ describe Rubydora::DigitalObject do
       end
 
       it "should allow other datastreams to be added" do
-        @mock_repository.should_receive(:datastream).with(:pid => 'pid', :dsid => 'z').and_raise(RestClient::ResourceNotFound)
+        @mock_api.should_receive(:datastream).with(:pid => 'pid', :dsid => 'z').and_raise(RestClient::ResourceNotFound)
 
         @object.datastreams.length.should == 3
 
@@ -127,29 +129,92 @@ describe Rubydora::DigitalObject do
 
       it "should provide a way to override the type of datastream object to use" do
         class MyCustomDatastreamClass < Rubydora::Datastream; end
-        object = Rubydora::DigitalObject.new 'pid', @mock_repository
+        object = Rubydora::DigitalObject.new 'pid', @mock_api
         object.stub(:datastream_object_for) do |dsid|
           MyCustomDatastreamClass.new(self, dsid)
         end
 
         object.datastreams['asdf'].should be_a_kind_of(MyCustomDatastreamClass)
       end
-      
     end
+    describe "with profiles (fedora >= 3.6)" do 
+      before(:each) do
+        @mock_api.stub :datastreams do |hash|
+          "<objectDatastreams>
+             <datastreamProfile dsID='a'><dsLabel>Test label</dsLabel></datastreamProfile>
+             <datastreamProfile dsID='b'></datastreamProfile>
+             <datastreamProfile dsID='c'></datastreamProfile>
+           </objectDatastreams>"
+        end
+        @object = Rubydora::DigitalObject.new 'pid', @mock_api
+        @object.stub(:new_record? => false)
+      end
+
+      it "should provide a hash populated by the existing datastreams" do
+        @object.datastreams.should have_key("a")
+        @object.datastreams.should have_key("b")
+        @object.datastreams.should have_key("c")
+      end
+      it "should load the profile attributes" do
+        expect(@object['a'].label).to eq 'Test label'
+      end
+      it "should not set the new datastream as changed" do
+        expect(@object['a']).to_not be_changed
+      end
+    end
+  end
+
+  describe "retrieved with batch ds profiles" do
+    before(:each) do
+      @mock_api.stub(:datastreams).and_return <<-XML
+      <objectDatastreams>
+        <datastreamProfile pid="pid" dsID="a">
+          <dsLocation>some:uri</dsLocation>
+          <dsLabel>label</dsLabel>
+          <dsChecksumValid>true</dsChecksumValid>
+        </datastreamProfile>
+        <datastreamProfile pid="pid" dsID="b">
+          <dsLocation>some:uri</dsLocation>
+          <dsLabel>label</dsLabel>
+          <dsChecksumValid>true</dsChecksumValid>
+        </datastreamProfile>
+        <datastreamProfile pid="pid" dsID="c">
+          <dsLocation>some:uri</dsLocation>
+          <dsLabel>label</dsLabel>
+          <dsChecksumValid>true</dsChecksumValid>
+        </datastreamProfile>
+      </objectDatastreams>
+      XML
+     @object = Rubydora::DigitalObject.new 'pid', @mock_api
+      @object.stub(:new_record? => false)
+    end
+    describe "datastreams" do
+      it "should provide a hash populated by the existing datastreams" do
 
+        @object.datastreams.should have_key("a")
+        @object.datastreams["a"].new?.should be_false
+        @object.datastreams["a"].changed?.should be_false
+        @object.datastreams.should have_key("b")
+        @object.datastreams["b"].new?.should be_false
+        @object.datastreams["b"].changed?.should be_false
+        @object.datastreams.should have_key("c")
+        @object.datastreams["c"].new?.should be_false
+        @object.datastreams["c"].changed?.should be_false
+      end
+    end
   end
 
   describe "update" do
 
     before(:each) do
-      @mock_repository.stub(:object) { <<-XML
+      @mock_api.stub(:object) { <<-XML
       <objectProfile>
         <objLabel>label</objLabel>
       </objectProfile>
       XML
       }
 
-      @object = Rubydora::DigitalObject.new 'pid', @mock_repository
+      @object = Rubydora::DigitalObject.new 'pid', @mock_api
     end
 
     it "should not say changed if the value is set the same" do
@@ -164,30 +229,31 @@ describe Rubydora::DigitalObject do
 
   describe "save" do
     before(:each) do
-      @mock_repository.stub(:object) { <<-XML
+      @original_modified = "2011-01-02:05:15:45.1Z"
+      @mock_api.stub(:object) { <<-XML
       <objectProfile>
-        <not>empty</not>
+        <objLastModDate>2011-01-02:05:15:45.100Z</objLastModDate>
       </objectProfile>
       XML
       }
 
-      @object = Rubydora::DigitalObject.new 'pid', @mock_repository
+      @object = Rubydora::DigitalObject.new 'pid', @mock_api
     end
 
     describe "saving an object's datastreams" do
       before do
         @new_ds = double(Rubydora::Datastream)
-        @new_ds.stub(:new? => true, :changed? => true, :content_changed? => true, :content => 'XXX')
+        @new_ds.stub(:new? => true, :changed? => true, :content_changed? => true, :content => 'XXX', :dsCreateDate => '12345')
         @new_empty_ds = double(Rubydora::Datastream)
-        @new_empty_ds.stub(:new? => true, :changed? => false, :content_changed? => false, :content => nil)
+        @new_empty_ds.stub(:new? => true, :changed? => false, :content_changed? => false, :content => nil, :dsCreateDate => '12345')
         @existing_ds = double(Rubydora::Datastream)
-        @existing_ds.stub(:new? => false, :changed? => false, :content_changed? => false, :content => 'YYY')
+        @existing_ds.stub(:new? => false, :changed? => false, :content_changed? => false, :content => 'YYY', :dsCreateDate => '12345')
         @changed_attr_ds = double(Rubydora::Datastream)
-        @changed_attr_ds.stub(:new? => false, :changed? => true, :content_changed? => false, :content => 'YYY')
+        @changed_attr_ds.stub(:new? => false, :changed? => true, :content_changed? => false, :content => 'YYY', :dsCreateDate => '12345')
         @changed_ds = double(Rubydora::Datastream)
-        @changed_ds.stub(:new? => false, :changed? => true, :content_changed? => true, :content => 'ZZZ')
+        @changed_ds.stub(:new? => false, :changed? => true, :content_changed? => true, :content => 'ZZZ', :dsCreateDate => '2012-01-02:05:15:45.100Z')
         @changed_empty_ds = double(Rubydora::Datastream)
-        @changed_empty_ds.stub(:new? => false, :changed? => true, :content_changed? => true, :content => nil)
+        @changed_empty_ds.stub(:new? => false, :changed? => true, :content_changed? => true, :content => nil, :dsCreateDate => '12345')
 
       end
       it "should save a new datastream with content" do
@@ -200,6 +266,8 @@ describe Rubydora::DigitalObject do
         @object.stub(:datastreams) { { :changed_ds => @changed_ds } }
         @changed_ds.should_receive(:save)
         @object.save
+        # object date should be canonicalized and updated
+        @object.lastModifiedDate.should == '2012-01-02:05:15:45.1Z'
       end
 
       it "should save a datastream whose attributes have changed" do
@@ -230,34 +298,51 @@ describe Rubydora::DigitalObject do
     it "should save all changed attributes" do
       @object.label = "asdf"
       @object.should_receive(:datastreams).and_return({})
-      @mock_repository.should_receive(:modify_object).with(hash_including(:pid => 'pid'))
+      @mock_api.should_receive(:modify_object).with(hash_including(:pid => 'pid'))
+      @object.save
+      expect(@object).to_not be_changed, "#{@object.changes.inspect}"
+    end
+
+    it "updates the modification time" do
+      ds = double(Rubydora::Datastream)
+      ds.stub(:changed? => false)
+      @object.stub(:datastreams) { { :ds => ds } }
+
+      @object.lastModifiedDate.should == @original_modified
+      mod_time = "2012-01-02:05:15:00.1Z"
+      @mock_api.should_receive(:modify_object).and_return(mod_time)
+
+      @object.label = "asdf"
       @object.save
+      @object.lastModifiedDate.should == mod_time
+      expect(@object).to_not be_changed, "#{@object.changes.inspect}"
     end
+
   end
 
   describe "delete" do
     before(:each) do
-      @object = Rubydora::DigitalObject.new 'pid', @mock_repository
+      @object = Rubydora::DigitalObject.new 'pid', @mock_api
     end
 
     it "should call the Fedora REST API" do
-      @mock_repository.should_receive(:purge_object).with({:pid => 'pid'})
+      @mock_api.should_receive(:purge_object).with({:pid => 'pid'})
       @object.delete
     end
   end
 
   describe "models" do
     before(:each) do
-      @mock_repository.stub(:object) { <<-XML
+      @mock_api.stub(:object) { <<-XML
       <objectProfile>
       </objectProfile>
       XML
       }
-      @object = Rubydora::DigitalObject.new 'pid', @mock_repository
+      @object = Rubydora::DigitalObject.new 'pid', @mock_api
     end
 
     it "should add models to fedora" do
-      @mock_repository.should_receive(:add_relationship) do |params|
+      @mock_api.should_receive(:add_relationship) do |params|
         params.should have_key(:subject)
         params[:predicate].should == 'info:fedora/fedora-system:def/model#hasModel'
         params[:object].should == 'asdf'
@@ -267,7 +352,7 @@ describe Rubydora::DigitalObject do
 
     it "should remove models from fedora" do
       @object.stub(:profile).and_return({"objModels" => ['asdf']})
-      @mock_repository.should_receive(:purge_relationship) do |params|
+      @mock_api.should_receive(:purge_relationship) do |params|
         params.should have_key(:subject)
         params[:predicate].should == 'info:fedora/fedora-system:def/model#hasModel'
         params[:object].should == 'asdf'
@@ -277,8 +362,8 @@ describe Rubydora::DigitalObject do
 
     it "should be able to handle complete model replacemenet" do
       @object.stub(:profile).and_return({"objModels" => ['asdf']})
-      @mock_repository.should_receive(:add_relationship).with(instance_of(Hash))
-      @mock_repository.should_receive(:purge_relationship).with(instance_of(Hash))
+      @mock_api.should_receive(:add_relationship).with(instance_of(Hash))
+      @mock_api.should_receive(:purge_relationship).with(instance_of(Hash))
       @object.models = '1234'
 
     end
@@ -286,60 +371,60 @@ describe Rubydora::DigitalObject do
 
   describe "relations" do
     before(:each) do
-      @mock_repository.stub(:object) { <<-XML
+      @mock_api.stub(:object) { <<-XML
       <objectProfile>
       </objectProfile>
       XML
       }
-      @object = Rubydora::DigitalObject.new 'pid', @mock_repository
+      @object = Rubydora::DigitalObject.new 'pid', @mock_api
     end
 
     it "should fetch related objects using sparql" do
-      @mock_repository.should_receive(:find_by_sparql_relationship).with('info:fedora/pid', 'info:fedora/fedora-system:def/relations-external#hasPart').and_return([1])
+      @mock_api.should_receive(:find_by_sparql_relationship).with('info:fedora/pid', 'info:fedora/fedora-system:def/relations-external#hasPart').and_return([1])
       @object.parts.should == [1]
     end
 
     it "should add related objects" do
-      @mock_repository.should_receive(:add_relationship) do |params|
+      @mock_api.should_receive(:add_relationship) do |params|
         params.should have_key(:subject)
         params[:predicate].should == 'info:fedora/fedora-system:def/relations-external#hasPart'
         params[:object].should == 'asdf'
       end
       @mock_object = double(Rubydora::DigitalObject)
       @mock_object.should_receive(:fqpid).and_return('asdf')
-      @mock_repository.should_receive(:find_by_sparql_relationship).with('info:fedora/pid', 'info:fedora/fedora-system:def/relations-external#hasPart').and_return([])
+      @mock_api.should_receive(:find_by_sparql_relationship).with('info:fedora/pid', 'info:fedora/fedora-system:def/relations-external#hasPart').and_return([])
       @object.parts << @mock_object
     end
 
     it "should remove related objects" do
-      @mock_repository.should_receive(:purge_relationship) do |params|
+      @mock_api.should_receive(:purge_relationship) do |params|
         params.should have_key(:subject)
         params[:predicate].should == 'info:fedora/fedora-system:def/relations-external#hasPart'
         params[:object].should == 'asdf'
       end
       @mock_object = double(Rubydora::DigitalObject)
       @mock_object.should_receive(:fqpid).and_return('asdf')
-      @mock_repository.should_receive(:find_by_sparql_relationship).with('info:fedora/pid', 'info:fedora/fedora-system:def/relations-external#hasPart').and_return([@mock_object])
+      @mock_api.should_receive(:find_by_sparql_relationship).with('info:fedora/pid', 'info:fedora/fedora-system:def/relations-external#hasPart').and_return([@mock_object])
       @object.parts.delete(@mock_object)
     end
   end
 
   describe "versions" do
     before(:each) do
-      @mock_repository.stub(:object) { <<-XML
+      @mock_api.stub(:object) { <<-XML
       <objectProfile>
       </objectProfile>
       XML
       }
 
-      @mock_repository.stub(:object_versions) { <<-XML
+      @mock_api.stub(:object_versions) { <<-XML
       <fedoraObjectHistory>
         <objectChangeDate>2011-09-26T20:41:02.450Z</objectChangeDate>
         <objectChangeDate>2011-10-11T21:17:48.124Z</objectChangeDate>
       </fedoraObjectHistory>
       XML
       }
-      @object = Rubydora::DigitalObject.new 'pid', @mock_repository
+      @object = Rubydora::DigitalObject.new 'pid', @mock_api
     end
 
     it "should have a list of previous versions" do
@@ -352,7 +437,7 @@ describe Rubydora::DigitalObject do
     end
 
     it "should lookup content of datastream using the asOfDateTime parameter" do
-      @mock_repository.should_receive(:datastreams).with(hash_including(:asOfDateTime => '2011-09-26T20:41:02.450Z')).and_return('')
+      @mock_api.should_receive(:datastreams).with(hash_including(:asOfDateTime => '2011-09-26T20:41:02.450Z')).and_return('')
       Rubydora::Datastream.should_receive(:new).with(anything, 'my_ds', hash_including(:asOfDateTime => '2011-09-26T20:41:02.450Z'))
       ds = @object.versions.first['my_ds']
     end
@@ -361,7 +446,7 @@ describe Rubydora::DigitalObject do
 
   describe "to_api_params" do
     before(:each) do
-      @object = Rubydora::DigitalObject.new 'pid', @mock_repository
+      @object = Rubydora::DigitalObject.new 'pid', @mock_api
     end
     it "should compile parameters to hash" do
       @object.send(:to_api_params).should == {}
@@ -369,7 +454,7 @@ describe Rubydora::DigitalObject do
   end
 
   shared_examples "an object attribute" do
-    subject { Rubydora::DigitalObject.new 'pid', @mock_repository }
+    subject { Rubydora::DigitalObject.new 'pid', @mock_api }
 
     describe "getter" do
       it "should return the value" do
@@ -383,7 +468,7 @@ describe Rubydora::DigitalObject do
       end
 
       it "should fall-back to the set of default attributes" do
-        @mock_repository.should_receive(:object).with(:pid=>"pid").and_raise(RestClient::ResourceNotFound)
+        @mock_api.should_receive(:object).with(:pid=>"pid").and_raise(RestClient::ResourceNotFound)
         Rubydora::DigitalObject::OBJ_DEFAULT_ATTRIBUTES.should_receive(:[]).with(method.to_sym) { 'zxcv'} 
         subject.send(method).should == 'zxcv'
       end
@@ -394,7 +479,7 @@ describe Rubydora::DigitalObject do
         subject.stub(:datastreams => [])
       end
       it "should mark the object as changed after setting" do
-        @mock_repository.should_receive(:object).with(:pid=>"pid").and_raise(RestClient::ResourceNotFound)
+        @mock_api.should_receive(:object).with(:pid=>"pid").and_raise(RestClient::ResourceNotFound)
         subject.send("#{method}=", 'new_value')
         subject.should be_changed
       end
@@ -405,8 +490,8 @@ describe Rubydora::DigitalObject do
       end
 
       it "should appear in the save request" do 
-        @mock_repository.should_receive(:ingest).with(hash_including(method.to_sym => 'new_value'))
-        @mock_repository.should_receive(:object).with(:pid=>"pid").and_raise(RestClient::ResourceNotFound)
+        @mock_api.should_receive(:ingest).with(hash_including(method.to_sym => 'new_value'))
+        @mock_api.should_receive(:object).with(:pid=>"pid").and_raise(RestClient::ResourceNotFound)
         subject.send("#{method}=", 'new_value')
         subject.save
       end
@@ -414,7 +499,7 @@ describe Rubydora::DigitalObject do
   end
 
   describe "#state" do
-    subject { Rubydora::DigitalObject.new 'pid', @mock_repository }
+    subject { Rubydora::DigitalObject.new 'pid', @mock_api }
 
     describe "getter" do
       it "should return the value" do
@@ -428,7 +513,7 @@ describe Rubydora::DigitalObject do
       end
 
       it "should fall-back to the set of default attributes" do
-        @mock_repository.should_receive(:object).with(:pid=>"pid").and_raise(RestClient::ResourceNotFound)
+        @mock_api.should_receive(:object).with(:pid=>"pid").and_raise(RestClient::ResourceNotFound)
         Rubydora::DigitalObject::OBJ_DEFAULT_ATTRIBUTES.should_receive(:[]).with(:state) { 'zxcv'} 
         subject.state.should == 'zxcv'
       end
@@ -439,7 +524,7 @@ describe Rubydora::DigitalObject do
         subject.stub(:datastreams => [])
       end
       it "should mark the object as changed after setting" do
-        @mock_repository.should_receive(:object).with(:pid=>"pid").and_raise(RestClient::ResourceNotFound)
+        @mock_api.should_receive(:object).with(:pid=>"pid").and_raise(RestClient::ResourceNotFound)
         subject.state= 'D'
         subject.should be_changed
       end
@@ -455,8 +540,8 @@ describe Rubydora::DigitalObject do
       end
 
       it "should appear in the save request" do 
-        @mock_repository.should_receive(:ingest).with(hash_including(:state => 'A'))
-        @mock_repository.should_receive(:object).with(:pid=>"pid").and_raise(RestClient::ResourceNotFound)
+        @mock_api.should_receive(:ingest).with(hash_including(:state => 'A'))
+        @mock_api.should_receive(:object).with(:pid=>"pid").and_raise(RestClient::ResourceNotFound)
         subject.state='A'
         subject.save
       end

data/spec/lib/integration_test_spec.rb

@@ -72,10 +72,11 @@ describe "Integration testing against a live Fedora repository", :integration =>
     obj.label.should == 'asdf'
     obj.label = 'qwerty'
     obj.save
-
+    # get 'cached' lastModifiedDate
+    e_date = obj.lastModifiedDate
     obj = @repository.find('test:3')
     obj.label.should == 'qwerty'
-
+    obj.lastModifiedDate.should == e_date
   end
 
   describe "datastream stuff" do
@@ -120,6 +121,15 @@ describe "Integration testing against a live Fedora repository", :integration =>
     ds.controlGroup.should == "M"
     ds.size.should be > 100
   end
+
+  it "should not mark existing datastreams as changed on load" do
+    obj = @repository.find('fedora-system:ContentModel-3.0')
+    obj.datastreams.each do |k,v|
+      v.changed?.should be_false
+      v.new?.should be_false
+    end
+  end
+
   end
 
   it "should delete datastreams" do
@@ -135,6 +145,7 @@ describe "Integration testing against a live Fedora repository", :integration =>
     obj.save
 
     obj.datastreams["new_ds"].new?.should == false
+    obj.datastreams["new_ds"].changed?.should == false
     obj.datastreams["empty_ds"].new?.should == true
   end
 
@@ -164,6 +175,18 @@ describe "Integration testing against a live Fedora repository", :integration =>
     ds.content
   end
 
+  it "should cache object's lastModifiedDate from ds changes" do
+    @repository.ping.should == true
+    obj = @repository.find_or_initialize('test:1')
+    obj.save
+    obj.label = "abc"
+    obj.datastreams["my_ds"].content = "XXX"
+    obj.save
+    obj.label = "123"
+    obj.save
+  end
+
+
   describe "with transactions" do
     it "should work on ingest" do
        @repository.find('transactions:1').delete rescue nil

data/spec/lib/profile_parser_spec.rb

@@ -0,0 +1,20 @@
+require 'spec_helper'
+
+describe Rubydora::ProfileParser do
+  describe ".canonicalize_date_string" do
+    it "should correctly trim trailing zeroes in w3c date lexical representations" do
+      test_cases = {
+        "2014-02-13T19:44:30.630Z" => "2014-02-13T19:44:30.63Z",
+        "2014-02-13T19:44:30.600Z" => "2014-02-13T19:44:30.6Z",
+        "2014-02-13T19:44:30.000Z" => "2014-02-13T19:44:30Z",
+        "2014-02-13T19:44:30.01Z" => "2014-02-13T19:44:30.01Z",
+        "2014-02-13T19:44:30.001Z" => "2014-02-13T19:44:30.001Z",
+        "2014-02-13T20:40:43.470Z" => "2014-02-13T20:40:43.47Z"
+      }
+      test_cases.each do |input, expected|
+        actual = Rubydora::ProfileParser.canonicalize_date_string( input)
+        actual.should == expected
+      end
+    end
+  end
+end

data/spec/lib/repository_spec.rb

@@ -100,4 +100,11 @@ describe Rubydora::Repository do
     end
   end
 
+  describe "delegation of methods to api" do
+    it "should delegate :datastream_url" do
+      @repository.api.should_receive(:datastream_url).with("foo:bar", "descMetadata", {})
+      @repository.datastream_url("foo:bar", "descMetadata", {})
+    end
+  end
+
 end