Only show the eyebrow in multi-source tabs

Why are these changes being introduced:

* It is believed that displaying the eyebrow in single-source tabs
  (e.g., Alma, CDI, ASpace) is redundant and adds visual clutter.
  Removing the eyebrow in these contexts should streamline the user
  experience.

Relevant ticket(s):

* https://mitlibraries.atlassian.net/browse/USE-251

How does this address that need:

* adds `multi_source_tab?` helper method to determine if the active tab
  includes multiple sources
* conditions display of eyebrow in search result partials using this
  helper

Document any side effects to this change:

* we will need to remember to update the `multi_source_tab?` method
  as tabs change over time

Author: JPrevost

app/helpers/record_helper.rb

@@ -186,6 +186,12 @@ def location(loc)
     "<strong>#{loc[:name]}</strong> #{loc[:collection]} (#{loc[:call_number]})"
   end
 
+  # Returns true if the active tab includes multiple sources.
+  # As our tabs adjust over time, we will need to revisit this logic.
+  def multi_source_tab?
+    %w[all primo timdex website].include?(@active_tab)
+  end
+
   private
 
   def render_kind_value(list)

app/views/search/_result.html.erb

@@ -1,6 +1,8 @@
 <li class="result use">
   <div class="result-content">
-    <p class="eyebrow"><%= result[:eyebrow] %></p>
+    <% if multi_source_tab? && result[:eyebrow].present? %>
+      <p class="eyebrow"><%= result[:eyebrow] %></p>
+    <% end %>
     <h3 class="record-title">
       <span class="sr">Title: </span>
       <%= link_to_result(result) %>

app/views/search/_result_primo.html.erb

@@ -1,6 +1,8 @@
 <li class="result primo">
   <div class="result-content">
-    <p class="eyebrow"><%= result[:eyebrow] %></p>
+    <% if multi_source_tab? && result[:eyebrow].present? %>
+      <p class="eyebrow"><%= result[:eyebrow] %></p>
+    <% end %>
     <h3 class="record-title">
       <span class="sr">Title: </span>
       <% if result[:links]&.find { |link| link['kind'] == 'full record' } %>

test/helpers/record_helper_test.rb

@@ -387,4 +387,18 @@ class RecordHelperTest < ActionView::TestCase
     expected = "<i class='fa-sharp fa-solid fa-question' aria-hidden='true''></i>"
     assert_equal expected, icon('question')
   end
+
+  test 'multi_source_tab? returns true for tabs that combine sources' do
+    %w[all primo timdex website].each do |tab|
+      @active_tab = tab
+      assert multi_source_tab?
+    end
+  end
+
+  test 'multi_source_tab? returns false for other tabs' do
+    %w[alma cdi aspace timdex_alma fake_tab].each do |tab|
+      @active_tab = tab
+      assert_not multi_source_tab?
+    end
+  end
 end