Fix to avoid prefetches that break tests.

Author: justinlittman

app/components/search/item_result_component.html.erb

@@ -5,7 +5,8 @@
         <% component.with_caption do %>
           <%= render Search::ResultHeadingComponent.new(index:) do |component| %>
             <%= component.with_link do %>
-              <%= helpers.link_to(title, object_path(druid:), data: { turbo_frame: '_top' }) %>
+              <%# Pre-fetch here breaks tests because can be triggered unintentionally. %>
+              <%= helpers.link_to(title, object_path(druid:), data: { turbo_frame: '_top', turbo_prefetch: !Rails.env.test? }) %>
             <% end %>
           <% end %>
         <% end %>

app/components/search/item_result_component.rb

@@ -24,8 +24,9 @@ def content_type_values
     end
 
     def admin_policy_values
+      # Pre-fetch here breaks tests because can be triggered unintentionally.
       [helpers.link_to(result.apo_title, object_path(druid: result.apo_druid),
-                       data: { turbo_frame: '_top' })]
+                       data: { turbo_frame: '_top', turbo_prefetch: !Rails.env.test? })]
     end
 
     def collections_values
@@ -65,7 +66,7 @@ def access_rights_values
     def collection_links
       result.collection_druids.map.with_index do |collection_druid, index|
         helpers.link_to(result.collection_titles[index], object_path(druid: collection_druid),
-                        data: { turbo_frame: '_top' })
+                        data: { turbo_frame: '_top', turbo_prefetch: false })
       end
     end
 

spec/spec_helper.rb

@@ -20,15 +20,15 @@
 SKIPPABLE_TEST_COVERAGE_CONCERNS = %w[assets javascript views].freeze
 
 SimpleCov.start(:rails) do
-  add_filter '/lib/tasks/'
+  skip '/lib/tasks/'
 
   # Use SimpleCov groups to break down test coverage per application concern, e.g.,
   # controllers, models, & jobs. See https://github.com/simplecov-ruby/simplecov#groups
   Dir.glob('app/*').each do |concern_path|
     concern = File.basename(concern_path)
     next if SKIPPABLE_TEST_COVERAGE_CONCERNS.include?(concern)
 
-    add_group concern.capitalize, concern_path
+    group concern.capitalize, concern_path
   end
 
   if ENV['CI']