Second experiment

This tries implementing a second (concurrent) experiment, which would rely on knowing how often a user clicks an outbound link.

That means we need an interstitial redirect in order to conclude the experiment, which happens in the new outbound path.

PLEASE NOTE: This isn't working yet, because the redirect is getting caught by Turbo, it seems, and violating CORS policies. As a result, we're just logging the outbound URLs to the console.

Author: matt-bernhardt

app/controllers/record_controller.rb

@@ -3,6 +3,19 @@ class RecordController < ApplicationController
 
   include RecordHelper
 
+  # The out method is used for concluding A/B testing experiments that rely on the user following outbound links.
+  #
+  # @param url String a URL where the URL will be redirected
+  def out
+    if params[:url].present?
+      ab_finished(:result_format)
+
+      redirect_to params[:url], allow_other_host: true
+    else
+      redirect_to '/'
+    end
+  end
+
   def view
     id = params[:id]
     index = ENV.fetch('TIMDEX_INDEX', nil)

app/controllers/search_controller.rb

@@ -15,6 +15,9 @@ def results
     # if we are loading results, the user submitted the form - so this experiment is finished
     ab_finished(:ui_colors)
 
+    # Start experimental result format
+    @fulfillment = ab_test(:result_format, 'link', 'button')
+
     # inject session preference for boolean type if it is present
     params[:booleanType] = cookies[:boolean_type] || 'AND'
 

app/helpers/search_helper.rb

@@ -18,6 +18,7 @@ def format_highlight_label(field_name)
 
   def link_to_result(result)
     if result[:source_link].present?
+      Rails.logger.debug(outbound_path(url: result[:source_link]))
       link_to(result[:title], result[:source_link])
     else
       result[:title]

app/views/record/out.html.erb

@@ -0,0 +1 @@
+<p>This would redirect you.</p>

app/views/search/_result_primo.html.erb

@@ -6,6 +6,7 @@
     <h3 class="record-title">
       <span class="sr">Title: </span>
       <% if result[:links]&.find { |link| link['kind'] == 'full record' } %>
+        <% Rails.logger.debug(outbound_path(url: result[:links].find { |link| link['kind'] == 'full record' }['url'])) %>
         <%= link_to(result[:title], result[:links].find { |link| link['kind'] == 'full record' }['url']) %>
       <% else %>
         <%= result[:title] %>

config/routes.rb

@@ -5,6 +5,8 @@
 
   get 'lookup', to: 'libkey#lookup'
 
+  get 'out', to: 'record#out', as: "outbound"
+  
   get 'record/(:id)',
       to: 'record#view',
       as: 'record',

test/controllers/record_controller_test.rb

@@ -36,4 +36,19 @@ class RecordControllerTest < ActionDispatch::IntegrationTest
       assert_select 'main', /(.*)#{message}(.*)/
     end
   end
+
+  test 'outbound route redirects to home without a URL parameter' do
+    get '/out'
+
+    assert_response :redirect
+    assert_redirected_to root_url
+  end
+
+  test 'outbound route redirects to a URL if provided' do
+    target = 'https://example.org'
+    get "/out?url=#{target}"
+
+    assert_response :redirect
+    assert_redirected_to target
+  end
 end