libis-services 1.0.5 → 1.0.11

data/spec/rosetta_auth_spec.rb

@@ -22,7 +22,7 @@ describe 'Rosetta Services' do
     expect {handler.producer(producer_id)}.to raise_error(Libis::Services::ServiceError)
   end
 
-  let!(:credentials) {Libis::Tools::ConfigFile.new File.join(File.dirname(__FILE__), 'credentials-test.yml')}
+  let!(:credentials) {Libis::Tools::ConfigFile.new File.join(File.dirname(__FILE__), 'credentials-prod.yml')}
 
   # noinspection RubyResolve
   let(:admin) {credentials.admin}
@@ -34,6 +34,7 @@ describe 'Rosetta Services' do
   let(:admin_pwd) {admin.password}
   # noinspection RubyResolve
   let(:admin_ins) {admin.institute}
+  let(:admin_pds_ins) {admin.institute_pds}
 
   let(:bad_cred) { 'deadbeaf' }
 
@@ -99,7 +100,7 @@ describe 'Rosetta Services' do
     end
 
     describe 'with correct credentials' do
-      let(:handle) {pds_handler.login(admin_usr, admin_pwd, admin_ins)}
+      let(:handle) {pds_handler.login(admin_usr, admin_pwd, admin_pds_ins)}
 
       it 'should login and return a handle' do
         expect(handle).to_not be_nil
@@ -109,7 +110,7 @@ describe 'Rosetta Services' do
         bor_info = pds_handler.user_info handle
         expect(bor_info[:bor_info][:id]).to eq admin_usr
         expect(bor_info[:bor_info][:name]).to eq admin_usr
-        expect(bor_info[:bor_info][:institute]).to eq admin_ins
+        expect(bor_info[:bor_info][:institute]).to eq admin_pds_ins
 
       end
 
@@ -125,7 +126,7 @@ describe 'Rosetta Services' do
     end
 
     describe 'with wrong user name' do
-      let(:handle) {pds_handler.login(bad_cred, admin_pwd, admin_ins)}
+      let(:handle) {pds_handler.login(bad_cred, admin_pwd, admin_pds_ins)}
 
       it 'should not login' do
         expect(handle).to be_nil

data/spec/rosetta_collection_spec.rb

@@ -4,7 +4,6 @@ require_relative 'spec_helper'
 require 'libis/tools/config_file'
 require 'libis/tools/extend/hash'
 
-require 'libis/services/rosetta/pds_handler'
 require 'libis/services/rosetta/collection_handler'
 
 require 'rspec/matchers'
@@ -39,11 +38,6 @@ end
 describe 'Rosetta Collection Service' do
 
   let(:credentials) { Libis::Tools::ConfigFile.new File.join(File.dirname(__FILE__), 'credentials-test.yml') }
-  let(:pds_handler) do
-    # noinspection RubyResolve
-    Libis::Services::Rosetta::PdsHandler.new(credentials.pds_url)
-  end
-
   let(:handle) do
     # noinspection RubyResolve
     pds_handler.login(
@@ -63,10 +57,14 @@ describe 'Rosetta Collection Service' do
     collection_service = Libis::Services::Rosetta::CollectionHandler.new credentials.rosetta_url,
                                                                          log: credentials.debug,
                                                                          log_level: credentials.debug_level
-    collection_service.pds_handle = handle
     collection_service
   end
 
+  before :each do
+    # noinspection RubyResolve
+    collection_service.authenticate(credentials.admin.user, credentials.admin.password, credentials.admin.institute)
+  end
+
   context 'existing collections' do
 
     let(:collection_data) { {
@@ -108,7 +106,7 @@ describe 'Rosetta Collection Service' do
   context 'collections CRUD' do
 
     let(:collection_data) { {
-        name: 'New test collection',
+        name: 'Test API collection',
         parent_id: parent_id,
         collection_order: 0,
         md_dc: {
@@ -126,9 +124,7 @@ describe 'Rosetta Collection Service' do
         },
         md_source: [],
         navigate: true,
-        publish: false,
-        external_id: '54321',
-        external_system: 'Scope'
+        publish: false
     } }
 
     let(:new_collection) { collection_service.find(parent_name + '/' + collection_data[:name]) }
@@ -173,8 +169,8 @@ describe 'Rosetta Collection Service' do
     end
 
     it 'retrieve new collection' do
-      puts new_collection.to_hash
-      puts new_collection_data
+      # puts new_collection.to_hash
+      # puts new_collection_data
       expect(new_collection.to_hash).to match_collection(new_collection_data)
     end
 
@@ -193,13 +189,6 @@ describe 'Rosetta Collection Service' do
 
   context 'check errors' do
 
-    it 'not authorized' do
-      collection_service.pds_handle = 'foobar'
-      expect do
-        collection_service.create({name: 'foo'})
-      end.to raise_error(Libis::Services::SoapError, /user_authorize_exception.*Invalid PDS Handle Number:foobar/)
-    end
-
     it 'invalid collection info' do
       expect do
         collection_service.create({name: 'foo', parent_id: 0})

data/spec/rosetta_ie_spec.rb

@@ -5,26 +5,13 @@ require 'awesome_print'
 require 'pp'
 
 require 'libis/tools/config_file'
-require 'libis/services/rosetta/pds_handler'
 require 'libis/services/rosetta/ie_handler'
 
 require_relative 'ie_data'
+
 describe 'Rosetta IE Service' do
 
   let(:credentials) { Libis::Tools::ConfigFile.new File.join(File.dirname(__FILE__), 'credentials-test.yml') }
-  let(:pds_handler) do
-    # noinspection RubyResolve
-    Libis::Services::Rosetta::PdsHandler.new(credentials.pds_url)
-  end
-
-  let(:handle) do
-    # noinspection RubyResolve
-    pds_handler.login(
-        credentials.admin.user,
-        credentials.admin.password,
-        credentials.admin.institute
-    )
-  end
 
   subject(:ie_handler) do
     # noinspection RubyResolve
@@ -34,14 +21,14 @@ describe 'Rosetta IE Service' do
   end
 
   before :each do
-    ie_handler.pds_handle = handle
+    # noinspection RubyResolve
+    ie_handler.authenticate(credentials.admin.user, credentials.admin.password, credentials.admin.institute)
   end
 
   it 'should get IE info' do
 
     mets = ie_handler.get_mets('IE403595')
     expect(mets).not_to be_nil
-    ap mets
     expect(mets).to deep_include(expected_mets)
     # check_container expected_mets, mets
   end

data/spec/rosetta_oai_spec.rb

@@ -2,6 +2,7 @@
 require_relative 'spec_helper'
 
 require 'libis/tools/config_file'
+require 'libis/tools/extend/symbol'
 require 'libis/services/rosetta/oai_pmh'
 
 describe 'Rosetta OAI-PMH Service' do
@@ -13,42 +14,80 @@ describe 'Rosetta OAI-PMH Service' do
     Libis::Services::Rosetta::OaiPmh.new credentials.rosetta_url
   end
 
+  let(:expected_identify) {
+    {
+      repository_name: 'LIBIS Teneo Sandbox Repository',
+      base_url: "#{credentials.rosetta_url}/oaiprovider/request"
+    }
+  }
+
+  let(:expected_metadata_formats) {
+    [ 'oai_dc' ]
+  }
+
   let(:expected_sets) {
     [
-        {name:'TESTINS-collections', spec: 'TESTINS-collections'}
+      'TESTINS-collections',
+      'INS00-collections'
     ]
   }
 
   let(:expected_collections) {
     [
-        'Kerk & Leven',
-        'Kerk en leven. Bisdom Antwerpen (0991).',
+        'E-periodieken',
+        'Parochiebladen',
+        'Kerk en leven. Puurs (4635)',
     ]
   }
 
+  let(:expected_identifiers) {
+    [ 'oai:sandbox.teneo.libis.be:IE403595']
+  }
+
   let(:expected_records) {
     [
         {
-            id: 'oai:d4I1-pubam:IE405650',
+          header: {identifier: 'oai:sandbox.teneo.libis.be:IE403595'},
+          metadata: {
+            'metadata' => {
+              'oai_dc:dc' => {
+                'dc:title' => 'Denemarken 2015',
+                'dcterms:spatial' => ['Denemarken', 'Fyn', 'Sinebjerg']
+              }
+            }
+          }
         }
     ]
   }
 
+  it 'should identify' do
+    id = oai_handler.identify
+    expect(id).to deep_include(expected_identify)
+  end
+
+  it 'should get metata formats' do
+    formats = oai_handler.metadata_formats
+    expect(formats[:entries].map(&:dig.(:prefix)).compact).to deep_include(expected_metadata_formats)
+  end
+
   it 'should get set list' do
     sets = oai_handler.sets
-    expect(sets[:entries]).to deep_include(expected_sets)
+    expect(sets[:entries].map(&:dig.(:name)).compact).to deep_include(expected_sets)
   end
 
   it 'should get list of collections' do
-    status = {}
-    collections = oai_handler.collections('TESTINS', status)
-    expect(collections).to deep_include(expected_collections)
+    collections = oai_handler.collections('KADOC')
+    expect(collections[:entries].map(&:dig.(:metadata, 'metadata', 'oai_dc:dc', 'dc:title')).compact).to deep_include(expected_collections)
+  end
+
+  it 'should get list of identifiers' do
+    ids = oai_handler.identifiers(set: 'test_data', from: '2020-01-01')
+    expect(ids[:entries].map(&:dig.(:identifier)).compact).to deep_include(expected_identifiers)
   end
 
   it 'should get list of records' do
-    status = {}
-    records = oai_handler.records('test_data', status)
-    expect(records).to deep_include(expected_records)
+    records = oai_handler.records(set: 'test_data', from: '2020-01-01')
+    expect(records[:entries]).to deep_include(expected_records)
   end
 
 end

data/spec/rosetta_pds_spec.rb

@@ -10,13 +10,13 @@ require 'libis/services/rosetta/deposit_handler'
 
 describe 'Rosetta PDS Service' do
 
-  let!(:credentials) { Libis::Tools::ConfigFile.new File.join(File.dirname(__FILE__), 'credentials-test.yml') }
+  let!(:credentials) { Libis::Tools::ConfigFile.new File.join(File.dirname(__FILE__), 'credentials-prod.yml') }
   let!(:pds_handler) do
     # noinspection RubyResolve
     Libis::Services::Rosetta::PdsHandler.new(credentials.pds_url)
   end
 
-  let(:handle) { pds_handler.login(admin_usr, admin_pwd, admin_ins) }
+  let(:handle) { pds_handler.login(admin_usr, admin_pwd, admin_pds_ins) }
 
   # noinspection RubyResolve
   let(:admin) { credentials.admin }
@@ -26,6 +26,7 @@ describe 'Rosetta PDS Service' do
   let(:admin_pwd) {admin.password}
   # noinspection RubyResolve
   let(:admin_ins) {admin.institute}
+  let(:admin_pds_ins) {admin.institute_pds}
 
   it 'should login and return a handle' do
     expect(handle).to_not be_nil
@@ -62,7 +63,7 @@ describe 'Rosetta PDS Service' do
     bor_info = pds_handler.user_info handle
     expect(bor_info[:bor_info][:id]).to eq admin_usr
     expect(bor_info[:bor_info][:name]).to eq admin_usr
-    expect(bor_info[:bor_info][:institute]).to eq admin_ins
+    expect(bor_info[:bor_info][:institute]).to eq admin_pds_ins
 
   end
 

data/spec/rosetta_producer_spec.rb

@@ -1,10 +1,10 @@
 # encoding: utf-8
 require_relative 'spec_helper'
 require 'yaml'
+require 'awesome_print'
 
 require 'libis/tools/config_file'
 
-require 'libis/services/rosetta/pds_handler'
 require 'libis/services/rosetta/producer_handler'
 
 describe 'Rosetta Producer Service' do
@@ -21,30 +21,25 @@ describe 'Rosetta Producer Service' do
   # noinspection RubyResolve
   let(:admin_ins) {admin.institute}
 
-  let(:pds_handler) do
-    # noinspection RubyResolve
-    Libis::Services::Rosetta::PdsHandler.new(credentials.pds_url)
-  end
-
-  let(:handle) {pds_handler.login(admin_usr, admin_pwd, admin_ins)}
-
   let(:contact_info) {{user_id: credentials.contact.user_id, name: 'Test User'}}
   # noinspection RubyResolve
   subject(:producer_handler) do
     handler = Libis::Services::Rosetta::ProducerHandler.new credentials.rosetta_url,
                                                             log: credentials.debug,
                                                             log_level: credentials.debug_level
-    handler.pds_handle = handle
     handler
   end
 
   before :each do
-    producer_handler.pds_handle = handle
+    # noinspection RubyResolve
+    producer_handler.authenticate(credentials.admin.user, credentials.admin.password, credentials.admin.institute)
   end
 
   context 'user info' do
+    # noinspection RubyBlockToMethodReference
     let(:user_id) {admin_uid}
 
+    # noinspection RubyBlockToMethodReference
     let(:user_name) {admin_usr}
 
     it 'gets user id' do
@@ -69,6 +64,7 @@ describe 'Rosetta Producer Service' do
 
     let(:producer_id) {credentials.producer.id}
     let(:producer_data) {credentials.producer.data.to_hash}
+    # noinspection RubyArgCount
     let(:producer_info) {::Libis::Services::Rosetta::Producer.new(producer_data).to_hash}
     let(:updated_info) {{email: 'nomail@mail.com', telephone_2: '0032 16 32 22 22'}}
     let(:new_producer_info) {
@@ -86,6 +82,7 @@ describe 'Rosetta Producer Service' do
 
     it 'get producer' do
       result = producer_handler.producer(producer_id)
+      # ap result
       expect(result).not_to be_nil
       expect(result.to_hash).to match producer_data
       expect(producer_data).to match result.to_hash
@@ -94,7 +91,7 @@ describe 'Rosetta Producer Service' do
     it 'create producer' do
       result = producer_handler.new_producer(new_producer_info)
       expect(result).not_to be_nil
-      expect(result).to match /^\d+$/
+      expect(result).to match(/^\d+$/)
       new_producer_id(result)
     end
 
@@ -136,19 +133,21 @@ describe 'Rosetta Producer Service' do
         email_address: 'test@mail.com',
         street: 'Willem de Croylaan 54',
         city: 'Heverlee',
-        zip: 3001,
+        zip: '3001',
         country: 'Belgium',
         telephone_1: '0032 16 32 22 66',
         user_group: 'producer_agents'
     }}
     let(:new_agent) {
       data = new_agent_data.dup
-      data[:password] = data[:password_verify] = 'abc123ABC'
-      Libis::Services::Rosetta::User.new(data).to_hash
+      # data[:password] = data[:password_verify] = 'abc123ABC'
+      # noinspection RubyArgCount
+      Libis::Services::Rosetta::ProducerAgent.new(data).to_hash
     }
 
     # noinspection RubyResolve
     let(:agent_data) {credentials.producer_agent.data.to_hash}
+    # noinspection RubyArgCount
     let(:agent) {::Libis::Services::Rosetta::User.new(agent_data)}
     # noinspection RubyResolve
     let(:agent_id) {credentials.producer_agent.user_id}
@@ -157,54 +156,59 @@ describe 'Rosetta Producer Service' do
 
     def new_agent_id(val = nil)
       $new_agent_id = val.to_i if val
-      "#{$new_agent_id}"
+      $new_agent_id
     end
 
     it 'get info' do
       result = producer_handler.agent(agent_id)
       expect(result).not_to be_nil
-      expect(result.to_hash).to include agent_data
-    end
-
-    it 'create' do
-      result = producer_handler.new_agent new_agent
-      expect(result).not_to be_nil
-      expect(result).to match /^\d+$/
-      new_agent_id result
-    end
-
-    it 'get info' do
-      result = producer_handler.agent(new_agent_id)
-      expect(result).not_to be_nil
-      expect(result.to_hash).to include new_agent_data
+      # ap result.to_hash
+      # ap agent_data
+      expect(result.to_hash).to match agent_data
     end
 
-    it 'update info' do
-      # update data
-      updated_agent = new_agent_data.dup
-      updated_agent[:email_address] = 'other@mail.com'
-      result = producer_handler.agent(new_agent_id, updated_agent)
-      expect(result).not_to be_nil
-      expect(result).to eq new_agent_id
-
-      # retrieve updated data
-      result = producer_handler.agent(new_agent_id)
-      expect(result).not_to be_nil
-      expect(result.to_hash).to match updated_agent
-    end
-
-
-    it 'delete' do
-      result = producer_handler.delete_agent new_agent_id
-      expect(result).not_to be_nil
-      expect(result).to eq new_agent_id
-      result = producer_handler.agent(new_agent_id)
-      expect(result).to be_nil
-    end
+    # DISABLED: Rosetta API does noet work anymore as expected
+    
+    # it 'create' do
+    #   result = producer_handler.new_agent new_agent
+    #   expect(result).not_to be_nil
+    #   expect(result).to match(/^\d+$/)
+    #   new_agent_id result
+    # end
+
+    # it 'get info' do
+    #   result = producer_handler.agent(new_agent_id)
+    #   expect(result).not_to be_nil
+    #   expect(result.to_hash).to match new_agent_data
+    # end
+
+    # it 'update info' do
+    #   # update data
+    #   updated_agent = new_agent_data.dup
+    #   updated_agent[:email_address] = 'other@mail.com'
+    #   result = producer_handler.agent(new_agent_id, updated_agent)
+    #   expect(result).not_to be_nil
+    #   expect(result).to eq new_agent_id
+
+    #   # retrieve updated data
+    #   result = producer_handler.agent(new_agent_id)
+    #   expect(result).not_to be_nil
+    #   expect(result.to_hash).to match updated_agent
+    # end
+
+
+    # it 'delete' do
+    #   result = producer_handler.delete_agent new_agent_id
+    #   expect(result).not_to be_nil
+    #   expect(result).to eq new_agent_id
+    #   result = producer_handler.agent(new_agent_id)
+    #   expect(result).to be_nil
+    # end
 
     it 'get producers' do
       result = producer_handler.agent_producers agent_id, agent_ins
       expect(result).not_to be_nil
+      # noinspection RubyResolve
       expect(result).to eq [{id: credentials.producer.id, description: credentials.producer.data.authoritative_name}]
     end
 
@@ -215,6 +219,7 @@ describe 'Rosetta Producer Service' do
     let(:contact_id) {credentials.contact.user_id}
     let(:contact_info) {credentials.contact.data.to_hash}
     let(:new_contact) {
+      # noinspection RubyArgCount
       ::Libis::Services::Rosetta::User.new(
           first_name: 'New',
           last_name: 'Contact',
@@ -245,7 +250,7 @@ describe 'Rosetta Producer Service' do
     it 'create' do
       result = producer_handler.new_contact new_contact
       expect(result).not_to be_nil
-      expect(result).to match /^\d+$/
+      expect(result).to match(/^\d+$/)
       new_contact_id result
     end