rubydora 0.5.6 → 0.5.7

data/History.textile

@@ -1,5 +1,8 @@
+h3. 0.5.7
+* Don't expect that a digital object necessarily has a repository. 
+
 h3. 0.5.6
-* Don't log 404 errors. It's just checking to see if datastreams exist"
+* Don't log 404 errors. It's just checking to see if datastreams exist
 
 
 h3. 0.5.5

data/VERSION

@@ -1 +1 @@
-0.5.6
+0.5.7

data/lib/rubydora/datastream.rb

@@ -154,6 +154,7 @@ module Rubydora
         ## Force a recheck of the profile if they've passed :validateChecksum and we don't have dsChecksumValid
         return @profile
       end
+      return @profile = {} unless digital_object.respond_to? :repository
       @profile = begin
         options = { :pid => pid, :dsid => dsid }
         options.merge!(opts)

data/spec/lib/datastream_spec.rb

@@ -443,51 +443,62 @@ describe Rubydora::Datastream do
   end
 
   describe "profile" do
-    before(:each) do
-      @datastream = Rubydora::Datastream.new @mock_object, 'dsid'
-    end
-    it "should accept a validateChecksum argument" do
-      @mock_repository.should_receive(:datastream).with(hash_including(:validateChecksum => true)).and_return <<-XML
-        <datastreamProfile>
-          <dsChecksumValid>true</dsChecksumValid>
-        </datastreamProfile>
-      XML
-      @datastream.profile(:validateChecksum=>true).should == {'dsChecksumValid' =>true}
-    end
-    it "should reraise Unauthorized errors" do
-      @mock_repository.should_receive(:datastream).and_raise(RestClient::Unauthorized)
-      lambda{@datastream.profile}.should raise_error(RestClient::Unauthorized)
+    describe "with a digital_object that doesn't have a repository" do
+      ### see UnsavedDigitalObject in ActiveFedora
+      before(:each) do
+        @datastream = Rubydora::Datastream.new stub(:foo), 'dsid'
+      end
+      it "should be empty if the digital_object doesn't have a repository" do
+        @datastream.profile.should == {}
+      end
     end
-    describe "once it has a profile" do
-      it "should use the profile from cache" do
-        @mock_repository.should_receive(:datastream).once.and_return <<-XML
-          <datastreamProfile>
-            <dsChecksumValid>true</dsChecksumValid>
-          </datastreamProfile>
-        XML
-        @datastream.profile().should == {'dsChecksumValid' =>true}
-        #second time should not trigger the mock, which demonstrates that the profile is coming from cache.
-        @datastream.profile().should == {'dsChecksumValid' =>true}
+    describe "with a digital_object that has a repository" do
+      before(:each) do
+        @datastream = Rubydora::Datastream.new @mock_object, 'dsid'
       end
-      it "should re-fetch and replace the profile when validateChecksum is passed in, and there is no dsChecksumValid in the existing profile" do
-        @mock_repository.should_receive(:datastream).once.and_return <<-XML
-          <datastreamProfile>
-            <dsLabel>The description of the content</dsLabel>
-          </datastreamProfile>
-        XML
-        @mock_repository.should_receive(:datastream).with(hash_including(:validateChecksum => true)).once.and_return <<-XML
+      it "should accept a validateChecksum argument" do
+        @mock_repository.should_receive(:datastream).with(hash_including(:validateChecksum => true)).and_return <<-XML
           <datastreamProfile>
-            <dsLabel>The description of the content</dsLabel>
             <dsChecksumValid>true</dsChecksumValid>
           </datastreamProfile>
         XML
-        @datastream.profile().should == {"dsLabel"=>"The description of the content"}
-        @datastream.profile(:validateChecksum=>true).should == {"dsLabel"=>"The description of the content", 'dsChecksumValid' =>true}
-        ## Third time should not trigger a mock, which demonstrates that the profile is coming from cache.
-        @datastream.profile(:validateChecksum=>true)
+        @datastream.profile(:validateChecksum=>true).should == {'dsChecksumValid' =>true}
+      end
+      it "should reraise Unauthorized errors" do
+        @mock_repository.should_receive(:datastream).and_raise(RestClient::Unauthorized)
+        lambda{@datastream.profile}.should raise_error(RestClient::Unauthorized)
       end
-    end
 
+      describe "once it has a profile" do
+        it "should use the profile from cache" do
+          @mock_repository.should_receive(:datastream).once.and_return <<-XML
+            <datastreamProfile>
+              <dsChecksumValid>true</dsChecksumValid>
+            </datastreamProfile>
+          XML
+          @datastream.profile().should == {'dsChecksumValid' =>true}
+          #second time should not trigger the mock, which demonstrates that the profile is coming from cache.
+          @datastream.profile().should == {'dsChecksumValid' =>true}
+        end
+        it "should re-fetch and replace the profile when validateChecksum is passed in, and there is no dsChecksumValid in the existing profile" do
+          @mock_repository.should_receive(:datastream).once.and_return <<-XML
+            <datastreamProfile>
+              <dsLabel>The description of the content</dsLabel>
+            </datastreamProfile>
+          XML
+          @mock_repository.should_receive(:datastream).with(hash_including(:validateChecksum => true)).once.and_return <<-XML
+            <datastreamProfile>
+              <dsLabel>The description of the content</dsLabel>
+              <dsChecksumValid>true</dsChecksumValid>
+            </datastreamProfile>
+          XML
+          @datastream.profile().should == {"dsLabel"=>"The description of the content"}
+          @datastream.profile(:validateChecksum=>true).should == {"dsLabel"=>"The description of the content", 'dsChecksumValid' =>true}
+          ## Third time should not trigger a mock, which demonstrates that the profile is coming from cache.
+          @datastream.profile(:validateChecksum=>true)
+        end
+      end
+    end
   end
 
   describe "to_api_params" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubydora
 version: !ruby/object:Gem::Version
-  version: 0.5.6
+  version: 0.5.7
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2012-03-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fastercsv
-  requirement: &70206995188040 !ruby/object:Gem::Requirement
+  requirement: &70357365893180 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70206995188040
+  version_requirements: *70357365893180
 - !ruby/object:Gem::Dependency
   name: rest-client
-  requirement: &70206995197140 !ruby/object:Gem::Requirement
+  requirement: &70357365887520 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70206995197140
+  version_requirements: *70357365887520
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: &70206995196540 !ruby/object:Gem::Requirement
+  requirement: &70357365901640 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70206995196540
+  version_requirements: *70357365901640
 - !ruby/object:Gem::Dependency
   name: mime-types
-  requirement: &70206995195920 !ruby/object:Gem::Requirement
+  requirement: &70357365901000 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70206995195920
+  version_requirements: *70357365901000
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70206995195340 !ruby/object:Gem::Requirement
+  requirement: &70357365900400 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70206995195340
+  version_requirements: *70357365900400
 - !ruby/object:Gem::Dependency
   name: activemodel
-  requirement: &70206995193380 !ruby/object:Gem::Requirement
+  requirement: &70357365899700 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70206995193380
+  version_requirements: *70357365899700
 - !ruby/object:Gem::Dependency
   name: savon
-  requirement: &70206995191240 !ruby/object:Gem::Requirement
+  requirement: &70357365897940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70206995191240
+  version_requirements: *70357365897940
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70206995217900 !ruby/object:Gem::Requirement
+  requirement: &70357365895460 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,10 +98,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70206995217900
+  version_requirements: *70357365895460
 - !ruby/object:Gem::Dependency
   name: shoulda
-  requirement: &70206995216300 !ruby/object:Gem::Requirement
+  requirement: &70357365893860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -109,10 +109,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70206995216300
+  version_requirements: *70357365893860
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &70206995210220 !ruby/object:Gem::Requirement
+  requirement: &70357365920960 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -120,10 +120,10 @@ dependencies:
         version: 1.0.14
   type: :development
   prerelease: false
-  version_requirements: *70206995210220
+  version_requirements: *70357365920960
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70206995229760 !ruby/object:Gem::Requirement
+  requirement: &70357365914820 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -131,10 +131,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70206995229760
+  version_requirements: *70357365914820
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: &70206995228500 !ruby/object:Gem::Requirement
+  requirement: &70357365934120 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -142,10 +142,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70206995228500
+  version_requirements: *70357365934120
 - !ruby/object:Gem::Dependency
   name: jettywrapper
-  requirement: &70206995223720 !ruby/object:Gem::Requirement
+  requirement: &70357365933200 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -153,7 +153,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70206995223720
+  version_requirements: *70357365933200
 description: ! 'Fedora Commons REST API ruby library : REQUIRES FCREPO 3.4+'
 email:
 - chris@cbeer.info
@@ -213,7 +213,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3539344283285420289
+      hash: -4514414545578735851
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -222,7 +222,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3539344283285420289
+      hash: -4514414545578735851
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.15