map some additional orcid pub types

app/models/publication.rb

@@ -258,6 +258,10 @@ def pubmed_pub?
     provenance == Settings.pubmed_source
   end
 
+  def orcid_pub?
+    provenance == Settings.orcid_source
+  end
+
   def wos_pub?
     provenance == Settings.wos_source
   end

config/settings.yml

@@ -124,6 +124,7 @@ cap_provenance: cap
 pubmed_source: pubmed
 sciencewire_source: sciencewire
 wos_source: wos
+orcid_source: orcid
 
 # checkin keys for honeybadger (actual keys are in shared_configs per environment as needed)
 # see https://app.honeybadger.io/projects/50046/check_ins

lib/orcid/publication_type_mapper.rb

@@ -4,12 +4,16 @@ module Orcid
   # Maps work / publication types between ORCID and SUL-PUB.
   class PublicationTypeMapper
     # Note that this is limited to the work types for which mapping is supported.
+    # Full list of work types at ORCID: see https://info.orcid.org/faq/what-work-types-does-orcid-support/
     PUB_TYPE_TO_WORK_TYPE = {
       'article' => 'journal-article',
       'book' => 'book',
       'caseStudy' => 'research-tool',
       'inbook' => 'book-chapter',
-      'inproceedings' => 'conference-paper'
+      'inproceedings' => 'conference-paper',
+      'otherPaper' => 'other',
+      'technicalReport' => 'report',
+      'workingPaper' => 'working-paper'
     }.freeze
 
     # @return [String] ORCID work type or nil if no matching

lib/orcid/work_mapper.rb

@@ -21,7 +21,7 @@ def map
         title: work.title,
         identifier: map_identifiers(work.self_external_ids),
         abstract: work.short_description,
-        provenance: 'orcid',
+        provenance: Settings.orcid_source,
         doi: work.external_id_value('doi'),
         isbn: work.external_id_value('isbn'),
         issn: work.external_id_value('issn'),

pub_hash_schema.yml

@@ -119,14 +119,14 @@ properties:
     enum:
       - article
       - book
-      - caseStudy # Legacy. Not in current code.
+      - caseStudy # Only set via manual pub submission (provenance = cap)
       - inproceedings
-      - inbook # Legacy. Not in current code.
-      - otherpaper # Legacy. Not in current code.
-      - otherPaper # Legacy. Not in current code.
-      - technicalReport # Legacy. Not in current code.
-      - workingPaper # Legacy. Not in current code.
-      - null # Legacy. Not in current code.
+      - inbook # Only set via manual pub submission (provenance = cap)
+      - otherpaper # Legacy. Not in current code, likely bad data.
+      - otherPaper # Only set via manual pub submission (provenance = cap) or via ORCID harvest (provenance = orcid)
+      - technicalReport # Only set via manual pub submission (provenance = cap) or via ORCID harvest (provenance = orcid)
+      - workingPaper # Only set via manual pub submission (provenance = cap) or via ORCID harvest (provenance = orcid)
+      - null # Legacy. Not in current code, likely bad data.
   wos_item_id:
     type: string
   wos_uid:

spec/lib/orcid/pub_mapper_spec.rb

@@ -121,14 +121,24 @@
 
   context 'when unmappable type' do
     let(:pub_hash) do
-      base_pub_hash.dup.tap { |pub_hash| pub_hash[:type] = 'workingPaper' }
+      base_pub_hash.dup.tap { |pub_hash| pub_hash[:type] = 'unknownType' }
     end
 
     it 'raises' do
       expect { work }.to raise_error(Orcid::PubMapper::PubMapperError, 'Unmapped publication type')
     end
   end
 
+  context 'when mappable type' do
+    let(:pub_hash) do
+      base_pub_hash.dup.tap { |pub_hash| pub_hash[:type] = 'workingPaper' }
+    end
+
+    it 'maps correctly' do
+      expect(work['type']).to eq('working-paper')
+    end
+  end
+
   context 'when missing title' do
     let(:pub_hash) { base_pub_hash.except(:title) }
 

spec/models/publication_spec.rb

@@ -474,6 +474,13 @@
       expect(pub).to be_harvested_pub
     end
 
+    it "returns true if the pub has a provenance of 'orcid'" do
+      pub.pub_hash = { provenance: 'orcid' }
+      expect(pub).not_to be_authoritative_pmid_source
+      expect(pub).to be_orcid_pub
+      expect(pub).to be_harvested_pub
+    end
+
     it "returns false if the pub does not have a provanance of 'pubmed' or 'sciencewire'" do
       pub.pub_hash = { provenance: 'cap' }
       expect(pub).not_to be_authoritative_pmid_source