rubydora 0.5.4 → 0.5.5

data/History.textile

@@ -1,3 +1,6 @@
+h3. 0.5.5
+* Raise Unauthorized errors
+
 h3. 0.5.4
 * Query Benchmarking 
 * Custom exception classes

data/VERSION

@@ -1 +1 @@
-0.5.4
+0.5.5

data/lib/rubydora/datastream.rb

@@ -160,7 +160,9 @@ module Rubydora
         options[:asOfDateTime] = asOfDateTime if asOfDateTime
         options[:validateChecksum] = true if repository.config[:validateChecksum]
         self.profile_xml_to_hash(repository.datastream(options))
-      rescue
+      rescue RestClient::Unauthorized => e
+        raise e
+      rescue StandardError
         {}
       end
     end

data/lib/rubydora/rest_api_client.rb

@@ -205,6 +205,10 @@ module Rubydora
         logger.error "Unable to connect to Fedora at #{@client.url}"
         raise e
       rescue RestClient::ResourceNotFound => e
+        logger.error "ResourceNotFound at #{client.url}/#{datastream_url(pid, dsid, options)}"
+        raise e
+      rescue RestClient::Unauthorized => e
+        logger.error "Unauthorized at #{client.url}/#{datastream_url(pid, dsid, options)}"
         raise e
       rescue RestClient::InternalServerError => e
         logger.error e.response

data/spec/lib/datastream_spec.rb

@@ -454,6 +454,10 @@ describe Rubydora::Datastream do
       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)
+    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

data/spec/lib/rest_api_client_spec.rb

@@ -107,19 +107,28 @@ describe Rubydora::RestApiClient do
 
   it "datastream" do
      RestClient::Request.should_receive(:execute).with(hash_including(:url => "http://example.org/objects/mypid/datastreams?format=xml"))
+    logger.should_receive(:debug) # squelch message "Loaded datastream list for mypid (time)"
     @mock_repository.datastream :pid => 'mypid'
   end
 
   it "datastream" do
      RestClient::Request.should_receive(:execute).with(hash_including(:url => "http://example.org/objects/mypid/datastreams/aaa?format=xml"))
+    logger.should_receive(:debug) # squelch message "Loaded datastream mypid/aaa (time)"
     @mock_repository.datastream :pid => 'mypid', :dsid => 'aaa'
   end
 
   it "should raise not found exception when getting a datastream" do
      RestClient::Request.should_receive(:execute).with(hash_including(:url => "http://example.org/objects/mypid/datastreams/aaa?format=xml")).and_raise( RestClient::ResourceNotFound)
+    logger.should_receive(:error).with("ResourceNotFound at http://example.org/objects/mypid/datastreams/aaa?format=xml")
     lambda {@mock_repository.datastream :pid => 'mypid', :dsid => 'aaa'}.should raise_error RestClient::ResourceNotFound
   end
 
+  it "should raise Unauthorized exception when getting a datastream" do
+     RestClient::Request.should_receive(:execute).with(hash_including(:url => "http://example.org/objects/mypid/datastreams/aaa?format=xml")).and_raise( RestClient::Unauthorized)
+    logger.should_receive(:error).with("Unauthorized at http://example.org/objects/mypid/datastreams/aaa?format=xml")
+    lambda {@mock_repository.datastream :pid => 'mypid', :dsid => 'aaa'}.should raise_error RestClient::Unauthorized
+  end
+
   it "datastream_dissemination" do
      RestClient::Request.should_receive(:execute).with(hash_including(:url => "http://example.org/objects/mypid/datastreams/aaa/content"))
     @mock_repository.datastream_dissemination :pid => 'mypid', :dsid => 'aaa'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubydora
 version: !ruby/object:Gem::Version
-  version: 0.5.4
+  version: 0.5.5
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-03-03 00:00:00.000000000 Z
+date: 2012-03-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fastercsv
-  requirement: &70336833980200 !ruby/object:Gem::Requirement
+  requirement: &70154491151220 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70336833980200
+  version_requirements: *70154491151220
 - !ruby/object:Gem::Dependency
   name: rest-client
-  requirement: &70336833973900 !ruby/object:Gem::Requirement
+  requirement: &70154491145140 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70336833973900
+  version_requirements: *70154491145140
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: &70336833994120 !ruby/object:Gem::Requirement
+  requirement: &70154491159360 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70336833994120
+  version_requirements: *70154491159360
 - !ruby/object:Gem::Dependency
   name: mime-types
-  requirement: &70336833991880 !ruby/object:Gem::Requirement
+  requirement: &70154491158840 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70336833991880
+  version_requirements: *70154491158840
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70336833989860 !ruby/object:Gem::Requirement
+  requirement: &70154491158180 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70336833989860
+  version_requirements: *70154491158180
 - !ruby/object:Gem::Dependency
   name: activemodel
-  requirement: &70336833987740 !ruby/object:Gem::Requirement
+  requirement: &70154491157400 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70336833987740
+  version_requirements: *70154491157400
 - !ruby/object:Gem::Dependency
   name: savon
-  requirement: &70336834009180 !ruby/object:Gem::Requirement
+  requirement: &70154491154800 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70336834009180
+  version_requirements: *70154491154800
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70336834008120 !ruby/object:Gem::Requirement
+  requirement: &70154491152940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,10 +98,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70336834008120
+  version_requirements: *70154491152940
 - !ruby/object:Gem::Dependency
   name: shoulda
-  requirement: &70336834006880 !ruby/object:Gem::Requirement
+  requirement: &70154491180420 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -109,10 +109,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70336834006880
+  version_requirements: *70154491180420
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &70336834005220 !ruby/object:Gem::Requirement
+  requirement: &70154491177460 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -120,10 +120,10 @@ dependencies:
         version: 1.0.14
   type: :development
   prerelease: false
-  version_requirements: *70336834005220
+  version_requirements: *70154491177460
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70336834001740 !ruby/object:Gem::Requirement
+  requirement: &70154491193220 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -131,10 +131,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70336834001740
+  version_requirements: *70154491193220
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: &70336834022680 !ruby/object:Gem::Requirement
+  requirement: &70154491191840 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -142,10 +142,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70336834022680
+  version_requirements: *70154491191840
 - !ruby/object:Gem::Dependency
   name: jettywrapper
-  requirement: &70336834021440 !ruby/object:Gem::Requirement
+  requirement: &70154491190920 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -153,7 +153,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70336834021440
+  version_requirements: *70154491190920
 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: -1588477717339381900
+      hash: -3533612321507909257
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -222,10 +222,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1588477717339381900
+      hash: -3533612321507909257
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.17
+rubygems_version: 1.8.15
 signing_key: 
 specification_version: 3
 summary: Fedora Commons REST API ruby library