hydra-remote_identifier 0.2.2 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 00609ca00a1aa3d266e11024d3e68b755716d609
-  data.tar.gz: d9fcb87871a336c9850fd913585296cc75951200
+  metadata.gz: 3ab26b285d7aa330263fb086b0bc280448b9e7b0
+  data.tar.gz: cbcd41fd1cf8a074c30a1a7c5ff1c48028b8aef6
 SHA512:
-  metadata.gz: cbfdc77f38b4d7031246bd72e525701cb2888882f304bc376800d76bb52e1fec61890f9941e60ec61567bb4e92749cca9928e458d85170b49e9c30faf0fff203
-  data.tar.gz: 56fd2f04082c9b23390e900e07ef27273053c955a4ded0c0b9bf5c938173fab3fbc99d5fef7085ff656d6a23a5103708c3ce2ad4fe6cf44227fcf277e8a23d3f
+  metadata.gz: 36d568e724f9b87055c8ac453f64d25b57b0c221d478fce22ef03468b797d6758ed5ca0c571f627993eef923006b1d62a525e8a63f254c0a76a91dfd0914a20a
+  data.tar.gz: 46403c9ec84143ec5ae66c3ba072a582858d9b493281ff502b208d8c72a30230205b40dd7e5d3e55e87341d3aaabc6674523a139debfe6fd5d1f7d96f4fb8945

data/lib/hydra/remote_identifier.rb

@@ -35,16 +35,25 @@ module Hydra::RemoteIdentifier
     #     Hydra::RemoteIdentifier.with_registered_remote_service(:doi, book) do |remote_service|
     #       Hydra::RemoteIdentifier.mint(:doi, book)
     #     end
-    def with_registered_remote_service(service_name, target)
+    def with_registered_remote_service(remote_service_name, target)
       # @TODO - the registered remote identifier is more than a bit off;
       # but it continues to work
       target.registered_remote_identifier_minters.each {|minter|
-        if minter.remote_service.name.to_s == service_name.to_s
+        if minter.remote_service.name.to_s == remote_service_name.to_s
           yield(minter.remote_service)
         end
       }
     end
 
+    def mint_if_applicable(remote_service_name, target)
+      remote_service = configuration.find_remote_service(remote_service_name)
+      if target.public_send(remote_service.accessor_name).to_i != 0
+        target.registered_remote_identifier_minters.each do |minter|
+          minter.call(target) if minter.remote_service == remote_service
+        end
+      end
+    end
+
     # Using the RemoteService mint the corresponding remote identifier for
     # the target. You must first configure the RemoteService and target's class
     # to define the attribute map. See Hydra::RemoteIdentifier.configure

data/lib/hydra/remote_identifier/remote_service.rb

@@ -12,6 +12,14 @@ module Hydra::RemoteIdentifier
       self.class.to_s.demodulize.underscore.to_sym
     end
 
+    def mint(target)
+      Hydra::RemoteIdentifier.mint(name, target)
+    end
+
+    def mint_if_applicable(target)
+      Hydra::RemoteIdentifier.mint_if_applicable(name, target)
+    end
+
     def accessor_name
       "mint_#{name}".to_sym
     end

data/lib/hydra/remote_identifier/version.rb

@@ -1,5 +1,5 @@
 module Hydra
   module RemoteIdentifier
-    VERSION = "0.2.2"
+    VERSION = "0.3.0"
   end
 end

data/spec/lib/hydra/remote_identifier/remote_service_spec.rb

@@ -18,6 +18,20 @@ module Hydra::RemoteIdentifier
       it { expect { subject.valid_attribute?(:attribute_name) }.to raise_error NotImplementedError }
     end
 
+    context '#mint' do
+      it {
+        Hydra::RemoteIdentifier.should_receive(:mint).with(:remote_service, :target)
+        subject.mint(:target)
+      }
+    end
+
+    context '#mint_if_applicable' do
+      it {
+        Hydra::RemoteIdentifier.should_receive(:mint_if_applicable).with(:remote_service, :target)
+        subject.mint_if_applicable(:target)
+      }
+    end
+
   end
 
 end

data/spec/lib/hydra/remote_identifier_spec.rb

@@ -53,9 +53,15 @@ module Hydra::RemoteIdentifier
         }.to yield_with_args(RemoteServices::Doi)
       end
 
+      it 'should not yield the service if non is registered' do
+        expect {|block|
+          Hydra::RemoteIdentifier.with_registered_remote_service(:alternate, target, &block)
+        }.to_not yield_control
+      end
+
     end
 
-    describe '.register' do
+    describe '.mint' do
 
       it 'works!', VCR::SpecSupport.merge(record: :new_episodes, cassette_name: 'doi-integration') do
         expect {
@@ -65,6 +71,24 @@ module Hydra::RemoteIdentifier
 
     end
 
+    describe '.mint_if_applicable' do
+
+      it 'does not mint if accessor is not set' do
+        target.mint_doi = '0'
+        expect {
+          Hydra::RemoteIdentifier.mint_if_applicable(:doi, target)
+        }.to_not change(target, :set_identifier)
+      end
+
+      it 'mints if accessor is set to non-zero', VCR::SpecSupport.merge(record: :new_episodes, cassette_name: 'doi-integration') do
+        target.mint_doi = '1'
+        expect {
+          Hydra::RemoteIdentifier.mint_if_applicable(:doi, target)
+        }.to change(target, :set_identifier).from(nil).to(expected_doi)
+      end
+
+    end
+
   end
 
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hydra-remote_identifier
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.3.0
 platform: ruby
 authors:
 - Jeremy Friesen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-07 00:00:00.000000000 Z
+date: 2013-10-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport