harvestdor 0.0.14 → 0.1.0

data/lib/harvestdor/purl_xml.rb

@@ -2,7 +2,7 @@ require 'nokogiri'
 
 module Harvestdor
   # Mixin:  code to retrieve Purl public xml pieces
-  
+
   RDF_NAMESPACE = 'http://www.w3.org/1999/02/22-rdf-syntax-ns#'
   OAI_DC_NAMESPACE = 'http://www.openarchives.org/OAI/2.0/oai_dc/'
   MODS_NAMESPACE = 'http://www.loc.gov/mods/v3'
@@ -28,14 +28,14 @@ module Harvestdor
     begin
       ng_doc = Nokogiri::XML(open("#{purl_url}/#{druid}.xml"))
       raise Harvestdor::Errors::MissingPublicXml.new(druid) if !ng_doc || ng_doc.children.empty?
-      ng_doc 
+      ng_doc
     rescue OpenURI::HTTPError
       raise Harvestdor::Errors::MissingPurlPage.new(druid)
     end
   end
 
   # the contentMetadata for this fedora object, from the purl xml
-  # @param [Object] object a String containing a druid (e.g. ab123cd4567), or 
+  # @param [Object] object a String containing a druid (e.g. ab123cd4567), or
   #  a Nokogiri::XML::Document containing the public_xml for an object
   # @param [String] purl_url url for the purl server.  default is Harvestdor::PURL_DEFAULT
   # @return [Nokogiri::XML::Document] the contentMetadata for the fedora object
@@ -45,14 +45,14 @@ module Harvestdor
       # preserve namespaces, etc for the node
       ng_doc = Nokogiri::XML(pub_xml_ng_doc.root.xpath('/publicObject/contentMetadata').to_xml)
       raise Harvestdor::Errors::MissingContentMetadata.new(object.inspect) if !ng_doc || ng_doc.children.empty?
-      ng_doc 
+      ng_doc
     rescue
       raise Harvestdor::Errors::MissingContentMetadata.new(object.inspect)
     end
   end
 
   # the identityMetadata for this fedora object, from the purl xml
-  # @param [Object] object a String containing a druid (e.g. ab123cd4567), or 
+  # @param [Object] object a String containing a druid (e.g. ab123cd4567), or
   #  a Nokogiri::XML::Document containing the public_xml for an object
   # @param [String] purl_url url for the purl server.  default is Harvestdor::PURL_DEFAULT
   # @return [Nokogiri::XML::Document] the identityMetadata for the fedora object
@@ -62,14 +62,14 @@ module Harvestdor
       # preserve namespaces, etc for the node
       ng_doc = Nokogiri::XML(pub_xml_ng_doc.root.xpath('/publicObject/identityMetadata').to_xml)
       raise Harvestdor::Errors::MissingIdentityMetadata.new(object.inspect) if !ng_doc || ng_doc.children.empty?
-      ng_doc 
+      ng_doc
     rescue
       raise Harvestdor::Errors::MissingIdentityMetadata.new(object.inspect)
     end
   end
 
   # the rightsMetadata for this fedora object, from the purl xml
-  # @param [Object] object a String containing a druid (e.g. ab123cd4567), or 
+  # @param [Object] object a String containing a druid (e.g. ab123cd4567), or
   #  a Nokogiri::XML::Document containing the public_xml for an object
   # @param [String] purl_url url for the purl server.  default is Harvestdor::PURL_DEFAULT
   # @return [Nokogiri::XML::Document] the rightsMetadata for the fedora object
@@ -86,7 +86,7 @@ module Harvestdor
   end
 
   # the RDF for this fedora object, from the purl xml
-  # @param [Object] object a String containing a druid (e.g. ab123cd4567), or 
+  # @param [Object] object a String containing a druid (e.g. ab123cd4567), or
   #  a Nokogiri::XML::Document containing the public_xml for an object
   # @param [String] purl_url url for the purl server.  default is Harvestdor::PURL_DEFAULT
   # @return [Nokogiri::XML::Document] the RDF for the fedora object
@@ -103,7 +103,7 @@ module Harvestdor
   end
 
   # the Dublin Core for this fedora object, from the purl xml
-  # @param [Object] object a String containing a druid (e.g. ab123cd4567), or 
+  # @param [Object] object a String containing a druid (e.g. ab123cd4567), or
   #  a Nokogiri::XML::Document containing the public_xml for an object
   # @param [String] purl_url url for the purl server.  default is Harvestdor::PURL_DEFAULT
   # @return [Nokogiri::XML::Document] the dc for the fedora object
@@ -119,73 +119,14 @@ module Harvestdor
     end
   end
 
-
-  class Client
-    
-    # the public xml for this fedora object, from the purl server
-    # @param [String] druid e.g. ab123cd4567, in the purl url
-    # @return [Nokogiri::XML::Document] the MODS metadata for the fedora object
-    def mods druid
-      Harvestdor.mods(druid, config.purl)
-    end
-
-    # the public xml for this fedora object, from the purl xml
-    # @param [String] druid e.g. ab123cd4567, in the purl url
-    # @return [Nokogiri::XML::Document] the public xml for the fedora object
-    def public_xml druid
-      Harvestdor.public_xml(druid, config.purl)
-    end
-
-    # the contentMetadata for this fedora object, from the purl xml
-    # @param [Object] object a String containing a druid (e.g. ab123cd4567), or 
-    #  a Nokogiri::XML::Document containing the public_xml for an object
-    # @return [Nokogiri::XML::Document] the contentMetadata for the fedora object
-    def content_metadata object
-      Harvestdor.content_metadata(object, config.purl)
-    end
-
-    # the identityMetadata for this fedora object, from the purl xml
-    # @param [Object] object a String containing a druid (e.g. ab123cd4567), or 
-    #  a Nokogiri::XML::Document containing the public_xml for an object
-    # @return [Nokogiri::XML::Document] the identityMetadata for the fedora object
-    def identity_metadata object
-      Harvestdor.identity_metadata(object, config.purl)
-    end
-
-    # the rightsMetadata for this fedora object, from the purl xml
-    # @param [Object] object a String containing a druid (e.g. ab123cd4567), or 
-    #  a Nokogiri::XML::Document containing the public_xml for an object
-    # @return [Nokogiri::XML::Document] the rightsMetadata for the fedora object
-    def rights_metadata object
-      Harvestdor.rights_metadata(object, config.purl)
-    end
-
-    # the RDF for this fedora object, from the purl xml
-    # @param [Object] object a String containing a druid (e.g. ab123cd4567), or 
-    #  a Nokogiri::XML::Document containing the public_xml for an object
-    # @return [Nokogiri::XML::Document] the RDF for the fedora object
-    def rdf object
-      Harvestdor.rdf(object, config.purl)
-    end
-
-    # the Dublin Core for this fedora object, from the purl xml
-    # @param [Object] object a String containing a druid (e.g. ab123cd4567), or 
-    #  a Nokogiri::XML::Document containing the public_xml for an object
-    # @return [Nokogiri::XML::Document] the dc for the fedora object
-    def dc object
-      Harvestdor.dc(object, config.purl)
-    end
-    
-  end # class Client
-
   protected #--------------------------------------------
-  
-  # @param [Object] object a String containing a druid (e.g. ab123cd4567), or 
+
+  # @param [Object] object a String containing a druid (e.g. ab123cd4567), or
   #  a Nokogiri::XML::Document containing the public_xml for an object
   # @param [String] purl_url url for the purl server.  default is Harvestdor::PURL_DEFAULT
   # @return [Nokogiri::XML::Document] the public xml for a DOR object
   def self.pub_xml(object, purl_url = Harvestdor::PURL_DEFAULT)
-    case 
+    case
       when object.instance_of?(String)
         # it's a druid
         pub_xml_ng_doc = Harvestdor.public_xml(object, purl_url)
@@ -194,7 +135,7 @@ module Harvestdor
       else
         raise "expected String or Nokogiri::XML::Document for first argument, got #{object.class}"
     end
-    pub_xml_ng_doc    
+    pub_xml_ng_doc
   end
 
 end # module Harvestdor

data/lib/harvestdor/version.rb

@@ -1,3 +1,3 @@
 module Harvestdor
-  VERSION = "0.0.14"
+  VERSION = "0.1.0"
 end

data/lib/harvestdor.rb

@@ -1,121 +1,25 @@
 require 'harvestdor/errors'
-require 'harvestdor/oai_harvest'
 require 'harvestdor/purl_xml'
 require 'harvestdor/version'
+require 'harvestdor/client'
 # external gems
 require 'confstruct'
-require 'oai'
 # stdlib
 require 'logger'
 require 'open-uri'
 require 'yaml'
 
 module Harvestdor
-  
+
   LOG_NAME_DEFAULT = "harvestdor.log"
   LOG_DIR_DEFAULT = File.join(File.dirname(__FILE__), "..", "logs")
   PURL_DEFAULT = 'http://purl.stanford.edu'
-  HTTP_OPTIONS_DEFAULT = { 'ssl' => { 
-                              'verify' => false 
-                            }, 
-                           'request' => {
-                              'timeout' => 60, # open/read timeout (seconds)
-                              'open_timeout' => 60 # connection open timeout (seconds)
-                            }
-                          }
-  OAI_CLIENT_DEBUG_DEFAULT = false
-  OAI_REPOSITORY_URL_DEFAULT = 'https://dor-oaiprovider-prod.stanford.edu/oai'
-  DEFAULT_METADATA_PREFIX = 'mods'
-  DEFAULT_FROM_DATE = nil
-  DEFAULT_UNTIL_DATE = nil
-  DEFAULT_SET = nil
-  
-  class Client
-    
-    # Set default values for the construction of Harvestdor::Client objects
-    def self.default_config
-      @class_config ||= Confstruct::Configuration.new({
-        :log_dir => LOG_DIR_DEFAULT,
-        :log_name => LOG_NAME_DEFAULT,
-        :purl => PURL_DEFAULT,
-        :http_options => HTTP_OPTIONS_DEFAULT,
-        :oai_repository_url => OAI_REPOSITORY_URL_DEFAULT,
-        :oai_client_debug => OAI_CLIENT_DEBUG_DEFAULT,
-        :default_metadata_prefix => DEFAULT_METADATA_PREFIX,
-        :default_from_date => DEFAULT_FROM_DATE,
-        :default_until_date => DEFAULT_UNTIL_DATE,
-        :default_set => DEFAULT_SET
-      })
-    end
-    
-    # Initialize a new instance of Harvestdor::Client
-    # @param Hash options
-    # @example
-    #   client = Harvestdor::Client.new({ # Example with all possible options
-    #      :log_dir => File.join(File.dirname(__FILE__), "..", "logs"),
-    #      :log_name => 'harvestdor.log',
-    #      :purl => 'http://purl.stanford.edu',
-    #      :http_options => { 'ssl' => { 
-    #                          'verify' => false 
-    #                          }, 
-    #                         'request' => {
-    #                            'timeout' => 30, # open/read timeout (seconds)
-    #                            'open_timeout' => 30 # connection open timeout (seconds)
-    #                          }
-    #                        },
-    #      :oai_repository_url => 'https://dor-oaiprovider-prod.stanford.edu/oai', # The OAI repository to connect to
-    #      :oai_client_debug => false,
-    #      :default_metadata_prefix => 'mods',
-    #      :default_from_date => '2012-12-01', 
-    #      :default_until_date => '2014-12-01',
-    #      :default_set => nil, 
-    #   })
-    def initialize options = {}
-      config.configure(YAML.load_file(options[:config_yml_path])) if options[:config_yml_path]
-      config.configure options
-      yield(config) if block_given?
-    end
-    
-    def config
-      @config ||= Confstruct::Configuration.new(self.class.default_config)
-    end
-    
-    # @return OAI::Client an instantiated OAI::Client object, based on config options
-    def oai_client
-      @oai_client ||= OAI::Client.new config.oai_repository_url, :debug => config.oai_client_debug, :http => oai_http_client
-    end
-
-    def logger
-      @logger ||= self.class.logger(config.log_dir, config.log_name)
-    end
-
-    protected #---------------------------------------------------------------------
-    
-    def oai_http_client
-      logger.info "Constructing OAI http client with faraday options #{config.http_options.to_hash.inspect}"
-      @oai_http_client ||= Faraday.new config.oai_repository_url, config.http_options.to_hash
-    end
-
-    # Global, memoized, lazy initialized instance of a logger
-    # @param [String] log_dir directory for to get log file
-    # @param [String] log_name name of log file
-    def self.logger(log_dir, log_name)
-      Dir.mkdir(log_dir) unless File.directory?(log_dir) 
-      @logger ||= Logger.new(File.join(log_dir, log_name), 'daily')
-    end
-
-  end # class Client
-  
-  # @param [Object] arg OAI::Header object or OAI::Record object or String (oai identifier)
-  # @return [String] the druid part of an OAI identifier in an OAI header, e.g.  bb134cc1324
-  def self.druid(arg)
-    oai_id = arg
-    if arg.is_a?(OAI::Header)
-      oai_id = arg.identifier
-    elsif arg.is_a?(OAI::Record)
-      oai_id = arg.header.identifier
-    end
-    oai_id.split('druid:').last
-  end
-
+  HTTP_OPTIONS_DEFAULT = { 'ssl' => {
+    'verify' => false
+  },
+  'request' => {
+    'timeout' => 60, # open/read timeout (seconds)
+    'open_timeout' => 60 # connection open timeout (seconds)
+  }
+}
 end # module Harvestdor

data/spec/config/example.yml

@@ -0,0 +1,16 @@
+# log_dir:  directory for log file  (default logs, relative to harvestdor gem path)
+log_dir: spec/test_logs
+
+# log_name: name of log file  (default: harvestdor.log)
+
+# purl: url for the DOR purl server (used to get ContentMetadata, etc.)
+# purl: http://purl-test.stanford.edu
+
+# Additional options to pass to Faraday http client (https://github.com/technoweenie/faraday)
+# timeouts are in seconds;  timeout -> open/read, open_timeout -> connection open
+http_options:
+  ssl:
+    verify: false
+  request:
+    timeout: 121
+    open_timeout: 122

data/spec/harvestdor_client_spec.rb

@@ -3,7 +3,7 @@ require "spec_helper"
 describe Harvestdor::Client do
 
   before(:all) do
-    @config_yml_path = File.join(File.dirname(__FILE__), "config", "oai.yml")
+    @config_yml_path = File.join(File.dirname(__FILE__), "config", "example.yml")
     @client_via_yml_only = Harvestdor::Client.new({:config_yml_path => @config_yml_path})
     require 'yaml'
     @yaml = YAML.load_file(@config_yml_path)
@@ -11,68 +11,44 @@ describe Harvestdor::Client do
 
   describe "initialization" do
     before(:all) do
-      @from_date = '2012-11-29'
-      @repo_url = 'http://my_oai_repo.org/oai'
+      @some_args = Harvestdor::Client.new.config
     end
+
     context "attributes passed in hash argument" do
-      before(:all) do
-        @some_args = Harvestdor::Client.new({:default_from_date => @from_date, :oai_repository_url => @repo_url}).config
-      end
-      it "should set the attributes to the passed values" do
-        expect(@some_args.oai_repository_url).to eql(@repo_url)
-        expect(@some_args.default_from_date).to eql(@from_date)
-      end
       it "should keep the defaults for attributes not in the hash argument" do
         expect(@some_args.log_name).to eql(Harvestdor::LOG_NAME_DEFAULT)
         expect(@some_args.log_dir).to eql(Harvestdor::LOG_DIR_DEFAULT)
         expect(@some_args.purl).to eql(Harvestdor::PURL_DEFAULT)
         expect(@some_args.http_options).to eql(Confstruct::Configuration.new(Harvestdor::HTTP_OPTIONS_DEFAULT))
-        expect(@some_args.oai_client_debug).to eql(Harvestdor::OAI_CLIENT_DEBUG_DEFAULT)
-        expect(@some_args.default_metadata_prefix).to eql(Harvestdor::DEFAULT_METADATA_PREFIX)
-        expect(@some_args.default_until_date).to eql(Harvestdor::DEFAULT_UNTIL_DATE)
-        expect(@some_args.default_set).to eql(Harvestdor::DEFAULT_SET)
       end
     end
-    
+
     context "config_yml_path in hash argument" do
       before(:all) do
         @config_via_yml_only = @client_via_yml_only.config
       end
       it "should set attributes in yml file over defaults" do
         expect(@config_via_yml_only.log_dir).to eql(@yaml['log_dir'])
-        expect(@config_via_yml_only.oai_repository_url).to eql(@yaml['oai_repository_url'])
-        expect(@config_via_yml_only.default_from_date).to eql(@yaml['default_from_date'])
-        expect(@config_via_yml_only.default_metadata_prefix).to eql(@yaml['default_metadata_prefix'])
         expect(@config_via_yml_only.http_options.request.timeout).to eql(@yaml['http_options']['request']['timeout'])
       end
       it "should keep the defaults for attributes not present in yml file nor a config yml file" do
         expect(@config_via_yml_only.log_name).to eql(Harvestdor::LOG_NAME_DEFAULT)
         expect(@config_via_yml_only.purl).to eql(Harvestdor::PURL_DEFAULT)
-        expect(@config_via_yml_only.default_until_date).to eql(Harvestdor::DEFAULT_UNTIL_DATE)
-        expect(@config_via_yml_only.default_set).to eql(Harvestdor::DEFAULT_SET)
       end
       context "and some hash arguments" do
         before(:all) do
-          @config_via_yml_plus = Harvestdor::Client.new({:config_yml_path => @config_yml_path, 
-            :default_from_date => @from_date, :oai_repository_url => @repo_url}).config
-        end
-        it "should favor hash arg attribute values over yml file values" do
-          expect(@config_via_yml_plus.oai_repository_url).to eql(@repo_url)
-          expect(@config_via_yml_plus.default_from_date).to eql(@from_date)
+          @config_via_yml_plus = Harvestdor::Client.new({:config_yml_path => @config_yml_path}).config
         end
         it "should favor yml file values over defaults" do
           expect(@config_via_yml_plus.log_dir).to eql(@yaml['log_dir'])
-          expect(@config_via_yml_plus.default_metadata_prefix).to eql(@yaml['default_metadata_prefix'])
           expect(@config_via_yml_plus.http_options.timeout).to eql(@yaml['http_options']['timeout'])
         end
         it "should keep the defaults for attributes not present in yml file" do
           expect(@config_via_yml_plus.log_name).to eql(Harvestdor::LOG_NAME_DEFAULT)
-          expect(@config_via_yml_plus.default_until_date).to eql(Harvestdor::DEFAULT_UNTIL_DATE)
-          expect(@config_via_yml_plus.default_set).to eql(Harvestdor::DEFAULT_SET)
         end
       end
     end
-    
+
     context "without hash arguments" do
       it "should keep the defaults for all attributes" do
         no_args = Harvestdor::Client.new.config
@@ -80,16 +56,10 @@ describe Harvestdor::Client do
         expect(no_args.log_dir).to eql(Harvestdor::LOG_DIR_DEFAULT)
         expect(no_args.purl).to eql(Harvestdor::PURL_DEFAULT)
         expect(no_args.http_options).to eql(Confstruct::Configuration.new(Harvestdor::HTTP_OPTIONS_DEFAULT))
-        expect(no_args.oai_client_debug).to eql(Harvestdor::OAI_CLIENT_DEBUG_DEFAULT)
-        expect(no_args.oai_repository_url).to eql(Harvestdor::OAI_REPOSITORY_URL_DEFAULT)
-        expect(no_args.default_metadata_prefix).to eql(Harvestdor::DEFAULT_METADATA_PREFIX)
-        expect(no_args.default_from_date).to eql(Harvestdor::DEFAULT_FROM_DATE)
-        expect(no_args.default_until_date).to eql(Harvestdor::DEFAULT_UNTIL_DATE)
-        expect(no_args.default_set).to eql(Harvestdor::DEFAULT_SET)
       end
     end
   end # initialize client
-  
+
   it "should allow direct setting of configuration attributes" do
     conf = Harvestdor::Client.new.config
     expect(conf.log_dir).to eql(Harvestdor::LOG_DIR_DEFAULT)
@@ -103,33 +73,5 @@ describe Harvestdor::Client do
       expect(File.exists?(File.join(@yaml['log_dir'], Harvestdor::LOG_NAME_DEFAULT))).to eql(true)
     end
   end
-  
-  context "oai_client" do
-    before(:all) do
-      @client = Harvestdor::Client.new
-      @default_oai_client = Harvestdor::Client.new.oai_client
-    end
-    
-    it "oai_client should return an OAI::Client object based on config data" do
-      expect(@default_oai_client).to be_an_instance_of(OAI::Client)
-    end 
-    
-    it "oai_client should have an http_client" do
-      expect(@default_oai_client.instance_variable_get(:@http_client)).to be_an_instance_of(Faraday::Connection)
-    end
 
-    context "oai_http_client (protected method)" do
-	    before(:all) do
-	      @http_client = @client.send(:oai_http_client)
-	    end
-	    it "should be a Faraday object" do
-	      expect(@http_client).to be_an_instance_of(Faraday::Connection)
-	    end
-	    it "should have the oai_provider url from config" do
-	      uri_obj = @http_client.url_prefix
-	      expect(@client.config.oai_repository_url).to match(Regexp.new(uri_obj.host + uri_obj.path))
-	    end
-	  end
-  end # context oai_client
-  
 end

data/spec/purl_xml_spec.rb

@@ -12,10 +12,10 @@ describe Harvestdor::Client do
     @rdf_xml = "<rdf:RDF xmlns:rdf='http://www.w3.org/1999/02/22-rdf-syntax-ns#'><rdf:Description rdf:about=\"info:fedora/druid:#{@druid}\">relationship!</rdf:Description></rdf:RDF>"
     @dc_xml = "<oai_dc:dc xmlns:oai_dc='#{Harvestdor::OAI_DC_NAMESPACE}'><oai_dc:title>hoo ha</oai_dc:title</oai_dc:dc>"
     @pub_xml = "<publicObject id='druid:#{@druid}'>#{@id_md_xml}#{@cntnt_md_xml}#{@rights_md_xml}#{@rdf_xml}#{@dc_xml}</publicObject>"
-    @ng_pub_xml = Nokogiri::XML(@pub_xml) 
-    @fake_druid = 'oo000oo0000'     
+    @ng_pub_xml = Nokogiri::XML(@pub_xml)
+    @fake_druid = 'oo000oo0000'
   end
-  
+
   it "#mods returns a Nokogiri::XML::Document from the purl mods" do
     VCR.use_cassette('purl_mods') do
       x = Harvestdor.mods(@druid, @purl)
@@ -23,7 +23,7 @@ describe Harvestdor::Client do
       expect(x.root.name).to eql('mods')
       expect(x.root.namespace.href).to eql(Harvestdor::MODS_NAMESPACE)
     end
-  end    
+  end
 
   context "#public_xml" do
     it "#public_xml retrieves entire public xml as a Nokogiri::XML::Document when called with druid" do
@@ -44,7 +44,7 @@ describe Harvestdor::Client do
       expect { Harvestdor.public_xml(@fake_druid, @purl) }.to raise_error(Harvestdor::Errors::MissingPublicXml)
     end
   end
-  
+
   context "#pub_xml" do
     it "retrieves public_xml via fetch when first arg is a druid" do
       expect(Harvestdor).to receive(:public_xml).with(@druid, @purl)
@@ -57,7 +57,7 @@ describe Harvestdor::Client do
       expect { Harvestdor.pub_xml(Array.new)}.to raise_error(RuntimeError, "expected String or Nokogiri::XML::Document for first argument, got Array")
     end
   end
-  
+
   context "#content_metadata" do
     it "returns a Nokogiri::XML::Document from the public xml fetched with druid" do
       VCR.use_cassette('content_metadata') do
@@ -76,9 +76,9 @@ describe Harvestdor::Client do
     it "raises MissingContentMetadata error if there is no contentMetadata in the public_xml for the druid" do
       pub_xml = "<publicObject id='druid:#{@druid}'>#{@id_md_xml}#{@rights_md_xml}</publicObject>"
       expect { Harvestdor.content_metadata(Nokogiri::XML(pub_xml)) }.to raise_error(Harvestdor::Errors::MissingContentMetadata)
-    end 
+    end
   end
-  
+
   context "#identity_metadata" do
     it "returns a Nokogiri::XML::Document from the public xml fetched with druid" do
       VCR.use_cassette('identity_metadata') do
@@ -98,9 +98,9 @@ describe Harvestdor::Client do
     it "raises MissingIdentityMetadata error if there is no identityMetadata in the public_xml for the druid" do
       pub_xml = "<publicObject id='druid:#{@druid}'>#{@cntnt_md_xml}#{@rights_md_xml}</publicObject>"
       expect { Harvestdor.identity_metadata(Nokogiri::XML(pub_xml)) }.to raise_error(Harvestdor::Errors::MissingIdentityMetadata)
-    end  
+    end
   end
-  
+
   context "#rights_metadata" do
     it "#rights_metadata returns a Nokogiri::XML::Document from the public xml fetched with druid" do
       VCR.use_cassette('rights_metadata') do
@@ -118,9 +118,9 @@ describe Harvestdor::Client do
     it "raises MissingRightsMetadata error if there is no identityMetadata in the public_xml for the druid" do
       pub_xml = "<publicObject id='druid:#{@druid}'>#{@cntnt_md_xml}#{@id_md_xml}</publicObject>"
       expect { Harvestdor.rights_metadata(Nokogiri::XML(pub_xml)) }.to raise_error(Harvestdor::Errors::MissingRightsMetadata)
-    end  
+    end
   end
-  
+
   context "#rdf" do
     it "returns a Nokogiri::XML::Document from the public xml fetched with druid" do
       VCR.use_cassette('rdf') do
@@ -140,9 +140,9 @@ describe Harvestdor::Client do
     it "raises MissingRDF error if there is no RDF in the public_xml for the druid" do
       pub_xml = "<publicObject id='druid:#{@druid}'>#{@cntnt_md_xml}#{@id_md_xml}</publicObject>"
       expect { Harvestdor.rdf(Nokogiri::XML(pub_xml)) }.to raise_error(Harvestdor::Errors::MissingRDF)
-    end  
+    end
   end
-  
+
   context "#dc" do
     it "returns a Nokogiri::XML::Document from the public xml fetched with druid" do
       VCR.use_cassette('dc') do
@@ -162,9 +162,9 @@ describe Harvestdor::Client do
     it "raises MissingDC error if there is no DC in the public_xml for the druid" do
       pub_xml = "<publicObject id='druid:#{@druid}'>#{@cntnt_md_xml}#{@id_md_xml}</publicObject>"
       expect { Harvestdor.dc(Nokogiri::XML(pub_xml)) }.to raise_error(Harvestdor::Errors::MissingDC)
-    end  
+    end
   end
-  
+
   context "Harvestdor:Client calls methods with config.purl" do
     before(:all) do
       @client = Harvestdor::Client.new({:purl_url => 'http://thisone.org'})