Load libkey lookup results in UI

** Why are these changes being introduced:

Now that there's an integration with libkey via an internal path, the
search results page needs to consult that when relevant results are
received from our discovery APIs.

** Relevant ticket(s):

* https://mitlibraries.atlassian.net/browse/use-232

** How does this address that need:

This adds fields for DOIs and PMIDs to the Primo record normalizer, on
the assumption that any given record has at most one of each of these.

If none is found, the field is left out of the normalized record.

If one is found that value is added to the normalized record.

If more than one is found, a message is sent to Sentry so we can look
further. This will help us understand how widespread the condition is.

(Any record might have _both_ a DOI and PMID, this is not problematic).

When a record has one of these values, we call the internal lookup route
via a stimulus controller. If the lookup results in an integration link
(not all DOIs or PMIDs will turn out this way), the link is added as the
first link in the UI.

If a record has both a DOI and a PMID, only the DOI is used.

I've written tests for the normalizer model, to confirm that it handles
cases with zero, one, and mulitple of each field. The tests don't deal
with the Sentry integration, however.

** Document any side effects to this change:

The integration with Sentry is a little iffy. I'm not thrilled about
directly calling Sentry from a model like this, but I want to be able
to use a consistent message text, while defining tags on each message
so that we can get the record IDs and DOIs/PMIDs to help us investigate
further.

Author: matt-bernhardt

app/models/normalize_primo_record.rb

@@ -18,6 +18,8 @@ def normalize
       links:,
       citation:,
       identifier:,
+      doi:,
+      pmid:,
       summary:,
       publisher:,
       location:,
@@ -142,6 +144,30 @@ def identifier
     @record['pnx']['control']['recordid'].join
   end
 
+  def doi
+    return unless @record['pnx']['addata'] && @record['pnx']['addata']['doi']
+
+    if @record['pnx']['addata']['doi'].length > 1
+      Sentry.set_tags('mitlib.recordId': identifier || 'empty record id')
+      Sentry.set_tags('mitlib.dois': @record['pnx']['addata']['doi'])
+      Sentry.capture_message('Multiple DOIs found in one record')
+    end
+
+    @record['pnx']['addata']['doi'].first
+  end
+
+  def pmid
+    return unless @record['pnx']['addata'] && @record['pnx']['addata']['pmid']
+
+    if @record['pnx']['addata']['pmid'].length > 1
+      Sentry.set_tags('mitlib.recordId': identifier || 'empty record id')
+      Sentry.set_tags('mitlib.pmids': @record['pnx']['addata']['pmid'])
+      Sentry.capture_message('Multiple PMIDs found in one record')
+    end
+
+    @record['pnx']['addata']['pmid'].first
+  end
+
   def summary
     return unless @record['pnx']['display']['description']
 

app/views/search/_result_primo.html.erb

@@ -35,6 +35,11 @@
     </div>
 
     <div class="result-get">
+      <% if Libkey.enabled? && result[:doi].present? %>
+        <%= render(partial: 'trigger_libkey', locals: { kind: 'doi', identifier: result[:doi] }) %>
+      <% elsif Libkey.enabled? && result[:pmid].present? %>
+        <%= render(partial: 'trigger_libkey', locals: { kind: 'pmid', identifier: result[:pmid] }) %>
+      <% end %>
       <% if result[:links].present? %>
         <% result[:links].each do |link| %>
           <%= link_to link['kind'].titleize, link['url'], class: 'button' %>

app/views/search/_trigger_libkey.html.erb

@@ -0,0 +1,8 @@
+<% return unless Libkey.enabled? %>
+
+<% data_url = "/lookup?type=#{kind}&identifier=#{identifier}" %>
+
+<span class="tacos-container"
+     data-controller="content-loader"
+     data-content-loader-url-value=<%= data_url %>>
+</span>

test/fixtures/primo/full_record.json

@@ -17,7 +17,9 @@
       "pages": ["123-145"],
       "jtitle": ["Journal of Testing"],
       "isbn": ["9781234567890", "1234567890"],
-      "pub": ["MIT Press"]
+      "pub": ["MIT Press"],
+      "doi": ["one"],
+      "pmid": ["one"]
     },
     "facets": {
       "frbrtype": ["5"],

test/models/normalize_primo_record_test.rb

@@ -106,6 +106,44 @@ def cdi_record
     assert_equal 'alma991000000001234567', normalized[:identifier]
   end
 
+  test 'normalizes doi' do
+    normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
+    assert_equal normalized[:doi], 'one'
+  end
+
+  test 'handles missing doi' do
+    normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
+    assert_nil normalized[:doi]
+  end
+
+  test 'multiple dois normalize to the first one' do
+    temp_record = full_record
+    temp_record['pnx']['addata']['doi'] = %w[three two one]
+
+    normalized = NormalizePrimoRecord.new(temp_record, 'test').normalize
+
+    assert_equal normalized[:doi], 'three'
+  end
+
+  test 'normalizes pmid' do
+    normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
+    assert_equal normalized[:pmid], 'one'
+  end
+
+  test 'handles missing pmid' do
+    normalized = NormalizePrimoRecord.new(minimal_record, 'test').normalize
+    assert_nil normalized[:pmid]
+  end
+
+  test 'multiple pmids normalize to the first one' do
+    temp_record = full_record
+    temp_record['pnx']['addata']['pmid'] = %w[three two one]
+
+    normalized = NormalizePrimoRecord.new(temp_record, 'test').normalize
+
+    assert_equal normalized[:pmid], 'three'
+  end
+
   test 'normalizes summary' do
     normalized = NormalizePrimoRecord.new(full_record, 'test').normalize
     assert_equal 'A comprehensive study of testing methodologies', normalized[:summary]