hydra-remote_identifier 0.6.3 → 0.6.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0044fca4960997fe2ea9ae11d9ce753b07f3572f
-  data.tar.gz: 43ff13cdbc11ccacefe3ec707105d8211e78cb1d
+  metadata.gz: 88b472a25583380ecb5baf994bfc73fe3d5d5a65
+  data.tar.gz: 336e74b11670464b4cb545dabca319dd4f02f82d
 SHA512:
-  metadata.gz: 16b800a78b49c35ed8a2a4a4e04630a5a03e96c77e30431e2d99c392dc8b72741252a3641b1ebf49c3e5b9eb1d905f07d76dc4c18e9b019adfcfcd28310cd063
-  data.tar.gz: 63305ed2a5047673401fa0dc20e5f4d49b06833e02d66cb239362aa530afc712676511ca9b94c216acf9dd377287f2f0d073bfb43d1e27a11ef043de112c206c
+  metadata.gz: 667c0f7acb4ceebd6f02114775600bd32c22919940e3a8471afd86a098cbc4e61cb20e01cf03624ec42d8f92630ad7e67dbf08a69e3b3d7a977c993df30c539c
+  data.tar.gz: 3f05bbeae5546cd40d7485c056bbe831a3963de741f807c51ccc09445ff3baa7d7b37feca60ffa086cd0cc42b6107a6d535b6515c83a518f4d77f063a0d4f2cc

data/lib/hydra/remote_identifier/configuration.rb

@@ -1,52 +1,54 @@
 require File.expand_path('../remote_services', __FILE__)
 require File.expand_path('../registration', __FILE__)
 
-module Hydra::RemoteIdentifier
+module Hydra
+  module RemoteIdentifier
 
-  # Configuration is responsible for exposing the available RemoteServices
-  # and configuring those RemoteServices.
-  class Configuration
+    # Configuration is responsible for exposing the available RemoteServices
+    # and configuring those RemoteServices.
+    class Configuration
 
-    def initialize(options = {})
-      @remote_service_namespace_container = options.fetch(:remote_service_namespace_container, RemoteServices)
-      @registration_builder = options.fetch(:registration_builder, Registration)
-      @remote_services = {}
-    end
-    attr_reader :remote_service_namespace_container, :registration_builder
-    private :remote_service_namespace_container, :registration_builder
-    attr_reader :remote_services
+      def initialize(options = {})
+        @remote_service_namespace_container = options.fetch(:remote_service_namespace_container, RemoteServices)
+        @registration_builder = options.fetch(:registration_builder, Registration)
+        @remote_services = {}
+      end
+      attr_reader :remote_service_namespace_container, :registration_builder
+      private :remote_service_namespace_container, :registration_builder
+      attr_reader :remote_services
 
-    def find_remote_service(service_name)
-      remote_services.fetch(service_name.to_sym)
-    end
+      def find_remote_service(service_name)
+        remote_services.fetch(service_name.to_sym)
+      end
 
-    # @param service_name [#to_s]
-    # @param args - passed through to RemoteService initializer
-    #
-    # @yieldparam [Registration]
-    def remote_service(service_name, *args)
-      remote_service = find_or_store_remote_service(service_name, *args)
-      registration = registration_builder.new(remote_service)
-      yield(registration)
-      remote_service
-    end
+      # @param service_name [#to_s]
+      # @param args - passed through to RemoteService initializer
+      #
+      # @yieldparam [Registration]
+      def remote_service(service_name, *args)
+        remote_service = find_or_store_remote_service(service_name, *args)
+        registration = registration_builder.new(remote_service)
+        yield(registration)
+        remote_service
+      end
 
-    private
-    def find_or_store_remote_service(service_name, *args)
-      remote_services[service_name.to_sym] ||= remote_service_class_lookup(service_name).new(*args)
-      remote_services.fetch(service_name.to_sym)
-    end
+      private
+      def find_or_store_remote_service(service_name, *args)
+        remote_services[service_name.to_sym] ||= remote_service_class_lookup(service_name).new(*args)
+        remote_services.fetch(service_name.to_sym)
+      end
 
-    def remote_service_class_lookup(string)
-      remote_service_class_name = string.to_s.gsub(/(?:^|_)([a-z])/) { $1.upcase }
-      if remote_service_namespace_container.const_defined?(remote_service_class_name)
-        remote_service_namespace_container.const_get(remote_service_class_name)
-      else
-        raise NotImplementedError.new(
-          "Unable to find #{self} remote_service '#{string}'. Consider creating #{remote_service_namespace_container}::#{remote_service_class_name}"
-        )
+      def remote_service_class_lookup(string)
+        remote_service_class_name = string.to_s.gsub(/(?:^|_)([a-z])/) { $1.upcase }
+        if remote_service_namespace_container.const_defined?(remote_service_class_name)
+          remote_service_namespace_container.const_get(remote_service_class_name)
+        else
+          raise NotImplementedError.new(
+            "Unable to find #{self} remote_service '#{string}'. Consider creating #{remote_service_namespace_container}::#{remote_service_class_name}"
+          )
+        end
       end
     end
-  end
 
+  end
 end

data/lib/hydra/remote_identifier/exceptions.rb

@@ -1,46 +1,48 @@
-module Hydra::RemoteIdentifier
-  class InvalidServiceMapping < RuntimeError
-    def initialize(errors)
-      super(errors.join(". ") << '.')
+module Hydra
+  module RemoteIdentifier
+    class InvalidServiceMapping < RuntimeError
+      def initialize(errors)
+        super(errors.join(". ") << '.')
+      end
     end
-  end
 
-  class RemoteServiceError < RuntimeError
-    def initialize(exception, uri, payload)
-      @exception = exception
-      @uri = uri
-      @payload = payload
-    end
+    class RemoteServiceError < RuntimeError
+      def initialize(exception, uri, payload)
+        @exception = exception
+        @uri = uri
+        @payload = payload
+      end
 
-    def to_s
-      text = ""
-      text << "response.code: #{response_code.inspect}\n"
-      text << "response.body: #{response_body.inspect}\n"
-      text << "\n"
-      text << "request.uri: #{request_sanitized_uri.inspect}\n"
-      text << "request.payload: #{request_sanitized_paylod.inspect}\n"
-    end
+      def to_s
+        text = ""
+        text << "response.code: #{response_code.inspect}\n"
+        text << "response.body: #{response_body.inspect}\n"
+        text << "\n"
+        text << "request.uri: #{request_sanitized_uri.inspect}\n"
+        text << "request.payload: #{request_sanitized_paylod.inspect}\n"
+      end
 
-    private
-    def request_sanitized_paylod
-      @payload
-    end
+      private
+      def request_sanitized_paylod
+        @payload
+      end
 
-    def request_sanitized_uri
-      uri = URI.parse(@uri.to_s)
-      x = "#{uri.scheme}://"
-      x << "xxx:xxx@" if uri.user || uri.password
-      x << File.join(uri.host, uri.path)
-      x << '?' << uri.query if uri.query
-      x
-    end
+      def request_sanitized_uri
+        uri = URI.parse(@uri.to_s)
+        x = "#{uri.scheme}://"
+        x << "xxx:xxx@" if uri.user || uri.password
+        x << File.join(uri.host, uri.path)
+        x << '?' << uri.query if uri.query
+        x
+      end
 
-    def response_code
-      @exception.http_code rescue NoMethodError @exception.to_s
-    end
+      def response_code
+        @exception.http_code rescue NoMethodError @exception.to_s
+      end
 
-    def response_body
-      @exception.http_body rescue NoMethodError 'unknown'
+      def response_body
+        @exception.http_body rescue NoMethodError 'unknown'
+      end
     end
   end
-end
+end

data/lib/hydra/remote_identifier/mapper.rb

@@ -1,97 +1,99 @@
 require File.expand_path('../exceptions', __FILE__)
 
-module Hydra::RemoteIdentifier
+module Hydra
+  module RemoteIdentifier
 
-  # The Mapper is responsible for transforming a target, via a Map, into an
-  # acceptable format for a Minter
-  class Mapper
+    # The Mapper is responsible for transforming a target, via a Map, into an
+    # acceptable format for a Minter
+    class Mapper
 
-    attr_reader :map
-    def initialize(service_class, map_builder = Map, &mapping_block)
-      @map = map_builder.new(service_class, &mapping_block)
-    end
-
-    def call(target, wrapper_builder = Wrapper)
-      wrapper_builder.new(map, target)
-    end
+      attr_reader :map
+      def initialize(service_class, map_builder = Map, &mapping_block)
+        @map = map_builder.new(service_class, &mapping_block)
+      end
 
-    # The Wrapper provides the getting and setting behavior for a target based on a Map
-    class Wrapper
-      attr_reader :map, :target
-      def initialize(map, target)
-        @map, @target = map, target
+      def call(target, wrapper_builder = Wrapper)
+        wrapper_builder.new(map, target)
       end
 
-      def extract_payload
-        map._getters.each_with_object({}) do |(key, getter), mem|
-          mem[key] = extract_attribute_for(target, getter, key)
-          mem
+      # The Wrapper provides the getting and setting behavior for a target based on a Map
+      class Wrapper
+        attr_reader :map, :target
+        def initialize(map, target)
+          @map, @target = map, target
         end
-      end
 
-      def set_identifier(identifier)
-        update_target(map._setter, target, identifier)
-      end
+        def extract_payload
+          map._getters.each_with_object({}) do |(key, getter), mem|
+            mem[key] = extract_attribute_for(target, getter, key)
+            mem
+          end
+        end
+
+        def set_identifier(identifier)
+          update_target(map._setter, target, identifier)
+        end
 
-      private
+        private
 
-      def update_target(setter, target, value)
-        if setter.respond_to?(:call)
-          setter.call(target, value)
-        elsif setter.is_a?(Symbol) || setter.is_a?(String)
-          target.send(setter, value)
-        elsif setter.is_a?(Hash)
-          datastream = setter.fetch(:at)
-          field = setter.fetch(:in)
-          target.datastreams["#{datastream}"].send("#{field}=", value)
+        def update_target(setter, target, value)
+          if setter.respond_to?(:call)
+            setter.call(target, value)
+          elsif setter.is_a?(Symbol) || setter.is_a?(String)
+            target.send(setter, value)
+          elsif setter.is_a?(Hash)
+            datastream = setter.fetch(:at)
+            field = setter.fetch(:in)
+            target.datastreams["#{datastream}"].send("#{field}=", value)
+          end
         end
-      end
 
-      def extract_attribute_for(target, getter, field_name)
-        if getter.respond_to?(:call)
-          getter.call(target)
-        elsif getter.is_a?(::Symbol) || getter.is_a?(::String)
-          target.send(getter)
-        elsif getter.is_a?(::Hash)
-          datastream = getter.fetch(:at)
-          field = getter.fetch(:in, field_name)
-          target.datastreams["#{datastream}"].send("#{field}")
+        def extract_attribute_for(target, getter, field_name)
+          if getter.respond_to?(:call)
+            getter.call(target)
+          elsif getter.is_a?(::Symbol) || getter.is_a?(::String)
+            target.send(getter)
+          elsif getter.is_a?(::Hash)
+            datastream = getter.fetch(:at)
+            field = getter.fetch(:in, field_name)
+            target.datastreams["#{datastream}"].send("#{field}")
+          end
         end
       end
-    end
 
-    # The Map is responsible for defining which attributes on the target map
-    # to the attributes expected in the RemoteService as well as defining
-    # how the RemoteService can update the target
-    class Map < BasicObject
-      attr_reader :service_class, :_getters, :_setter
-      def initialize(service_class, &config)
-        @service_class = service_class
-        @_setter = nil
-        @_getters = {}
-        @errors = []
-        config.call(self)
-        if @_setter.nil?
-          @errors << "Missing :set_identifier"
+      # The Map is responsible for defining which attributes on the target map
+      # to the attributes expected in the RemoteService as well as defining
+      # how the RemoteService can update the target
+      class Map < BasicObject
+        attr_reader :service_class, :_getters, :_setter
+        def initialize(service_class, &config)
+          @service_class = service_class
+          @_setter = nil
+          @_getters = {}
+          @errors = []
+          config.call(self)
+          if @_setter.nil?
+            @errors << "Missing :set_identifier"
+          end
+          ::Kernel.raise ::Hydra::RemoteIdentifier::InvalidServiceMapping.new(@errors) if @errors.any?
         end
-        ::Kernel.raise ::Hydra::RemoteIdentifier::InvalidServiceMapping.new(@errors) if @errors.any?
-      end
 
-      def inspect
-        "#<Hydra::RemoteIdentifier::Mapper::Map for #{service_class} (#{__FILE__})>"
-      end
+        def inspect
+          "#<Hydra::RemoteIdentifier::Mapper::Map for #{service_class} (#{__FILE__})>"
+        end
 
-      def method_missing(method_name, *args, &block)
-        if method_name == :set_identifier
-          @_setter = args.first || block
-        elsif service_class.valid_attribute?(method_name)
-          @_getters[method_name] = args.first || block
-        else
-          @errors << "Invalid mapping #{method_name}"
+        def method_missing(method_name, *args, &block)
+          if method_name == :set_identifier
+            @_setter = args.first || block
+          elsif service_class.valid_attribute?(method_name)
+            @_getters[method_name] = args.first || block
+          else
+            @errors << "Invalid mapping #{method_name}"
+          end
         end
       end
+
     end
 
   end
-
 end

data/lib/hydra/remote_identifier/minter.rb

@@ -1,35 +1,37 @@
-module Hydra::RemoteIdentifier
+module Hydra
+  module RemoteIdentifier
 
-  # The Minter is responsible for passing the target's payload to the
-  # RemoteService then setting the target's identifier based on the response
-  # from the remote_service
-  class Minter
+    # The Minter is responsible for passing the target's payload to the
+    # RemoteService then setting the target's identifier based on the response
+    # from the remote_service
+    class Minter
 
-    def self.call(coordinator, target)
-      new(coordinator, target).call
-    end
+      def self.call(coordinator, target)
+        new(coordinator, target).call
+      end
 
-    attr_reader :service, :target
+      attr_reader :service, :target
 
-    def initialize(service, target)
-      @service, @target = service, target
-    end
+      def initialize(service, target)
+        @service, @target = service, target
+      end
 
-    def call
-      update_target(service.call(payload))
-    end
+      def call
+        update_target(service.call(payload))
+      end
 
-    private
+      private
 
-    def payload
-      target.extract_payload
-    end
+      def payload
+        target.extract_payload
+      end
+
+      def update_target(identifier)
+        target.set_identifier(identifier)
+        identifier
+      end
 
-    def update_target(identifier)
-      target.set_identifier(identifier)
-      identifier
     end
 
   end
-
 end

data/lib/hydra/remote_identifier/minting_coordinator.rb

@@ -1,27 +1,29 @@
 require File.expand_path('../mapper', __FILE__)
 require File.expand_path('../minter', __FILE__)
 
-module Hydra::RemoteIdentifier
+module Hydra
+  module RemoteIdentifier
 
-  # The Minting
-  class MintingCoordinator
-    attr_reader :remote_service, :mapper
-    def initialize(remote_service, mapper_builder = Mapper, &map_config)
-      @remote_service = remote_service
-      @mapper = mapper_builder.new(remote_service, &map_config)
-    end
+    # The Minting
+    class MintingCoordinator
+      attr_reader :remote_service, :mapper
+      def initialize(remote_service, mapper_builder = Mapper, &map_config)
+        @remote_service = remote_service
+        @mapper = mapper_builder.new(remote_service, &map_config)
+      end
 
-    # Responsible for passing attributes from the target, as per the map, to
-    # the service and assigning the result of the service to the target, as per
-    # the map.
-    def call(target, minter = Minter)
-      minter.call(remote_service, wrap(target))
-    end
+      # Responsible for passing attributes from the target, as per the map, to
+      # the service and assigning the result of the service to the target, as per
+      # the map.
+      def call(target, minter = Minter)
+        minter.call(remote_service, wrap(target))
+      end
 
-    private
-    def wrap(target)
-      mapper.call(target)
+      private
+      def wrap(target)
+        mapper.call(target)
+      end
     end
-  end
 
+  end
 end

data/lib/hydra/remote_identifier/railtie.rb

@@ -1,12 +1,14 @@
-module Hydra::RemoteIdentifier
-  class Railtie < Rails::Railtie
-    generators do
-      require 'generators/hydra/remote_identifier/install_generator'
-      require 'generators/hydra/remote_identifier/doi_generator'
-    end
+module Hydra
+  module RemoteIdentifier
+    class Railtie < Rails::Railtie
+      generators do
+        require 'generators/hydra/remote_identifier/install_generator'
+        require 'generators/hydra/remote_identifier/doi_generator'
+      end
 
-    config.to_prepare do
-      Hydra::RemoteIdentifier.send(:configure!)
+      config.to_prepare do
+        Hydra::RemoteIdentifier.send(:configure!)
+      end
     end
   end
-end
+end

data/lib/hydra/remote_identifier/registration.rb

@@ -1,43 +1,45 @@
 require 'active_support/core_ext/class/attribute'
 require File.expand_path('../minting_coordinator', __FILE__)
 
-module Hydra::RemoteIdentifier
+module Hydra
+  module RemoteIdentifier
 
-  # The Registration is responsible for connecting a RemoteService and a Target
-  # to a particular Map
-  class Registration
-    attr_reader :remote_service, :minting_coordinator
-    def initialize(remote_service, minting_coordinator = MintingCoordinator, &map)
-      @remote_service = remote_service
-      @minting_coordinator = minting_coordinator
-    end
+    # The Registration is responsible for connecting a RemoteService and a Target
+    # to a particular Map
+    class Registration
+      attr_reader :remote_service, :minting_coordinator
+      def initialize(remote_service, minting_coordinator = MintingCoordinator, &map)
+        @remote_service = remote_service
+        @minting_coordinator = minting_coordinator
+      end
 
-    # @param target_classes [Array]
-    # @yieldparam map [Map]
-    def register(*target_classes, &map)
-      if map.nil?
-        raise RuntimeError, "You attempted to register the remote service #{remote_service} for #{target_classes} without a map"
+      # @param target_classes [Array]
+      # @yieldparam map [Map]
+      def register(*target_classes, &map)
+        if map.nil?
+          raise RuntimeError, "You attempted to register the remote service #{remote_service} for #{target_classes} without a map"
+        end
+        Array(target_classes).flatten.compact.each {|target_class|
+          register_target(target_class, &map)
+        }
       end
-      Array(target_classes).flatten.compact.each {|target_class|
-        register_target(target_class, &map)
-      }
-    end
 
-    private
+      private
 
-    def register_target(target_class, &map)
-      target_class.module_exec(remote_service) do |service|
-        unless target_class.respond_to?(:registered_remote_identifier_minters)
-          class_attribute :registered_remote_identifier_minters
-        end
-        attr_accessor service.accessor_name
-        define_method("#{service.accessor_name}?") do
-          instance_variable_get("@#{service.accessor_name}").to_i != 0
+      def register_target(target_class, &map)
+        target_class.module_exec(remote_service) do |service|
+          unless target_class.respond_to?(:registered_remote_identifier_minters)
+            class_attribute :registered_remote_identifier_minters
+          end
+          attr_accessor service.accessor_name
+          define_method("#{service.accessor_name}?") do
+            instance_variable_get("@#{service.accessor_name}").to_i != 0
+          end
         end
+        target_class.registered_remote_identifier_minters ||= []
+        target_class.registered_remote_identifier_minters += [minting_coordinator.new(remote_service, &map)]
       end
-      target_class.registered_remote_identifier_minters ||= []
-      target_class.registered_remote_identifier_minters += [minting_coordinator.new(remote_service, &map)]
     end
-  end
 
+  end
 end

data/lib/hydra/remote_identifier/remote_service.rb

@@ -1,66 +1,68 @@
 require 'active_support/core_ext/string/inflections'
-module Hydra::RemoteIdentifier
+module Hydra
+  module RemoteIdentifier
 
-  # The RemoteService is responsible for delivering a payload to a remote
-  # identification minting service and returning an identifier.
-  #
-  # It is responsible for assisting the construction and validation of a remote
-  # payload.
-  class RemoteService
+    # The RemoteService is responsible for delivering a payload to a remote
+    # identification minting service and returning an identifier.
+    #
+    # It is responsible for assisting the construction and validation of a remote
+    # payload.
+    class RemoteService
 
-    def name
-      self.class.to_s.demodulize.underscore.to_sym
-    end
+      def name
+        self.class.to_s.demodulize.underscore.to_sym
+      end
 
-    def to_s
-      name.to_s
-    end
+      def to_s
+        name.to_s
+      end
 
-    def registered?(target)
-      Hydra::RemoteIdentifier.registered?(self, target)
-    end
+      def registered?(target)
+        Hydra::RemoteIdentifier.registered?(self, target)
+      end
 
-    def mint(target)
-      Hydra::RemoteIdentifier.mint(self, target)
-    end
+      def mint(target)
+        Hydra::RemoteIdentifier.mint(self, target)
+      end
 
-    # @param identifier[#to_s] - An identifier that was created by this remote
-    #   service
-    # @returns [URI] - The URI for that identifier
-    def remote_uri_for(identifier)
-      raise NotImplementedError,
-        "You must implement #{self.class}#remote_uri_for(identifier)"
-    end
+      # @param identifier[#to_s] - An identifier that was created by this remote
+      #   service
+      # @returns [URI] - The URI for that identifier
+      def remote_uri_for(identifier)
+        raise NotImplementedError,
+          "You must implement #{self.class}#remote_uri_for(identifier)"
+      end
 
-    # When mapping a Target to a RemoteService, this is the name of the
-    # :attr_accessor that will be created on the Target; Helpful for form
-    # construction.
-    #
-    # @returns [Symbol]
-    def accessor_name
-      "mint_#{name}".to_sym
-    end
+      # When mapping a Target to a RemoteService, this is the name of the
+      # :attr_accessor that will be created on the Target; Helpful for form
+      # construction.
+      #
+      # @returns [Symbol]
+      def accessor_name
+        "mint_#{name}".to_sym
+      end
 
-    # @param identifier[#to_s] - The name of an attribute that is
-    #
-    # @returns [boolean] - Is this a valid attribute to send as part of the
-    #   payload for RemoteService#call
-    def valid_attribute?(attribute_name)
-      raise NotImplementedError,
-        "You must implement #{self.class}#valid_attribute?"
-    end
+      # @param identifier[#to_s] - The name of an attribute that is
+      #
+      # @returns [boolean] - Is this a valid attribute to send as part of the
+      #   payload for RemoteService#call
+      def valid_attribute?(attribute_name)
+        raise NotImplementedError,
+          "You must implement #{self.class}#valid_attribute?"
+      end
+
+      # @param payload[Hash] - A map with key/value pairs of valid attribute names
+      #   and corresponding values that will be used to create the remote
+      #   identifier
+      # @return - The remote identifier that was created
+      #
+      # @see RemoteService#valid_attribute?
+      def call(payload)
+        raise NotImplementedError,
+          "You must implement #{self.class}#call"
+      end
 
-    # @param payload[Hash] - A map with key/value pairs of valid attribute names
-    #   and corresponding values that will be used to create the remote
-    #   identifier
-    # @return - The remote identifier that was created
-    #
-    # @see RemoteService#valid_attribute?
-    def call(payload)
-      raise NotImplementedError,
-        "You must implement #{self.class}#call"
     end
 
   end
-
-end
+end

data/lib/hydra/remote_identifier/remote_services/doi.rb

@@ -11,20 +11,22 @@ module Hydra::RemoteIdentifier
         username: 'apitest',
         password: 'apitest',
         shoulder: 'doi:10.5072/FK2',
-        url: "https://ezid.lib.purdue.edu/ezid/"
+        url: "https://ezid.lib.purdue.edu/",
+        resolver_url: 'http://dx.doi.org/'
       }
 
-      attr_reader :username, :password, :shoulder, :url
+      attr_reader :username, :password, :shoulder, :url, :resolver_url
       def initialize(options = {})
         configuration = options.with_indifferent_access
         @username = configuration.fetch(:username)
         @password = configuration.fetch(:password)
         @shoulder = configuration.fetch(:shoulder)
         @url = configuration.fetch(:url)
+        @resolver_url = configuration.fetch(:resolver_url) { default_resolver_url }
       end
 
       def remote_uri_for(identifier)
-        URI.parse(File.join(url, identifier))
+        URI.parse(File.join(resolver_url, identifier))
       end
 
       REQUIRED_ATTRIBUTES = ['target', 'creator', 'title', 'publisher', 'publicationyear' ].freeze
@@ -62,6 +64,10 @@ module Hydra::RemoteIdentifier
         data << "datacite.publicationyear: #{payload.fetch(:publicationyear)}"
         data.join("\n")
       end
+
+      def default_resolver_url
+        'http://dx.doi.org/'
+      end
     end
   end
 end

data/lib/hydra/remote_identifier/version.rb

@@ -1,5 +1,5 @@
 module Hydra
   module RemoteIdentifier
-    VERSION = "0.6.3"
+    VERSION = "0.6.4"
   end
 end

data/run-tests

@@ -0,0 +1,4 @@
+#!/usr/bin/env bash
+
+bundle install
+rspec

data/spec/fixtures/cassettes/doi-create.yml

@@ -2,7 +2,7 @@
 http_interactions:
 - request:
     method: post
-    uri: https://apitest:apitest@ezid.lib.purdue.edu/ezid/shoulder/doi:10.5072/FK2
+    uri: https://apitest:apitest@ezid.lib.purdue.edu/shoulder/doi:10.5072/FK2
     body:
       encoding: UTF-8
       string: |-
@@ -34,7 +34,7 @@ http_interactions:
       Vary:
       - Cookie
       Set-Cookie:
-      - sessionid=649cfcff023c40d4c8f1420673837ae6; Path=/ezid/
+      - sessionid=649cfcff023c40d4c8f1420673837ae6; Path=/
       Content-Length:
       - '53'
       Content-Type:
@@ -46,7 +46,7 @@ http_interactions:
   recorded_at: Thu, 03 Oct 2013 17:50:54 GMT
 - request:
     method: post
-    uri: https://apitest:apitest@ezid.lib.purdue.edu/ezid/shoulder/doi:10.5072/FK2
+    uri: https://apitest:apitest@ezid.lib.purdue.edu/shoulder/doi:10.5072/FK2
     body:
       encoding: UTF-8
       string: |-

data/spec/fixtures/cassettes/doi-integration.yml

@@ -2,7 +2,7 @@
 http_interactions:
 - request:
     method: post
-    uri: https://apitest:apitest@ezid.lib.purdue.edu/ezid/shoulder/doi:10.5072/FK2
+    uri: https://apitest:apitest@ezid.lib.purdue.edu/shoulder/doi:10.5072/FK2
     body:
       encoding: UTF-8
       string: |-
@@ -34,7 +34,7 @@ http_interactions:
       Vary:
       - Cookie
       Set-Cookie:
-      - sessionid=995dadb966d7013e9251daf61b6122ba; Path=/ezid/
+      - sessionid=995dadb966d7013e9251daf61b6122ba; Path=/
       Content-Length:
       - '53'
       Content-Type:
@@ -46,7 +46,7 @@ http_interactions:
   recorded_at: Thu, 03 Oct 2013 19:10:25 GMT
 - request:
     method: post
-    uri: https://apitest:apitest@ezid.lib.purdue.edu/ezid/shoulder/doi:10.5072/FK2
+    uri: https://apitest:apitest@ezid.lib.purdue.edu/shoulder/doi:10.5072/FK2
     body:
       encoding: UTF-8
       string: |-

data/spec/lib/hydra/remote_identifier/remote_services/doi_spec.rb

@@ -35,7 +35,7 @@ module Hydra::RemoteIdentifier
       end
 
       context '.remote_uri_for' do
-        let(:expected_uri) { URI.parse(File.join(subject.url, expected_doi))}
+        let(:expected_uri) { URI.parse(File.join(subject.resolver_url, expected_doi))}
         it 'should be based on configuration' do
           expect(subject.remote_uri_for(expected_doi)).to eq(expected_uri)
         end

data/spec/lib/hydra/remote_identifier_spec.rb

@@ -109,7 +109,7 @@ module Hydra::RemoteIdentifier
     context '.remote_uri_for' do
       it {
         expect(Hydra::RemoteIdentifier.remote_uri_for(:doi, expected_doi)).
-        to eq(URI.parse(File.join(doi_options.fetch(:url), expected_doi)))
+        to eq(URI.parse(File.join(doi_options.fetch(:resolver_url), expected_doi)))
       }
     end
 
@@ -150,8 +150,25 @@ module Hydra::RemoteIdentifier
       it 'returns false if the target is not configured for identifiers' do
         expect(Hydra::RemoteIdentifier.mint(:doi, double)).to eq(false)
       end
+
     end
+    if ENV['DOI_API_USERNAME'] && ENV['DOI_API_PASSWORD']
+
+      let(:doi_options) {
+        {
+          username: ENV['DOI_API_USERNAME'],
+          password: ENV['DOI_API_PASSWORD'],
+          shoulder: 'doi:10.5072/FK2',
+          url: "https://ezid.lib.purdue.edu/"
+        }
+      }
 
+      context 'against live sandbox' do
+        it 'works!' do
+          Hydra::RemoteIdentifier.mint(:doi, target)
+        end
+      end
+    end
   end
 
-end
+end

data/spec/spec_helper.rb

@@ -11,6 +11,7 @@ VCR.configure do |config|
   config.cassette_library_dir = 'spec/fixtures/cassettes'
   config.hook_into :webmock
   config.configure_rspec_metadata!
+  config.allow_http_connections_when_no_cassette = true
 end
 
 # This file was generated by the `rspec --init` command. Conventionally, all

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hydra-remote_identifier
 version: !ruby/object:Gem::Version
-  version: 0.6.3
+  version: 0.6.4
 platform: ruby
 authors:
 - Jeremy Friesen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-03-14 00:00:00.000000000 Z
+date: 2014-05-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -118,6 +118,7 @@ files:
 - lib/hydra/remote_identifier/remote_services.rb
 - lib/hydra/remote_identifier/remote_services/doi.rb
 - lib/hydra/remote_identifier/version.rb
+- run-tests
 - spec/fixtures/cassettes/doi-create.yml
 - spec/fixtures/cassettes/doi-integration.yml
 - spec/lib/hydra/remote_identifier/configuration_spec.rb