AO3-6911: prefer lazy loaded locale, fix route

Author: boyer-victor

app/controllers/application_controller.rb

@@ -28,7 +28,7 @@ def raise_not_found
   rescue_from Rack::Timeout::RequestTimeoutException, with: :raise_timeout
 
   def raise_timeout
-    redirect_to "/timeout"
+    redirect_to timeout_error_path
   end
 
   helper :all # include all helpers, all the time

app/controllers/errors_controller.rb

@@ -6,10 +6,11 @@ class ErrorsController < ApplicationController
   end
 
   def auth_error
-    @page_subtitle = "Auth Error"
+    @page_subtitle = t(".subtitle")
   end
 
   def timeout_error
-    @page_subtitle = "Timeout Error"
+    @page_subtitle = t(".subtitle")
+    render "timeout_error", status: 200, formats: :html
   end
 end

app/views/errors/timeout.html.erb

@@ -0,0 +1,3 @@
+<h2 class="heading"><%= t(".title") %></h2>
+<h3 class="heading"><%= t(".subtitle") %></h3>
+<p><%= t(".html", contact_support_link: link_to(t(".contact_support_link"), new_feedback_report_path)) %></p>

app/views/errors/timeout_error.html.erb

@@ -90,6 +90,11 @@ en:
       error: Sorry, that comment could not be unhidden.
       permission_denied: Sorry, you don't have permission to unhide that comment.
       success: Comment successfully unhidden!
+  errors:
+    timeout_error:
+      subtitle: "Timeout Error"
+    auth_error:
+      subtitle: "Authentication Error"
   external_works:
     update:
       successfully_updated: External work was successfully updated.

config/locales/controllers/en.yml

@@ -612,10 +612,11 @@ en:
         revealed_html: A translation of %{work_link} by %{creator_link}
         unrevealed: A translation of a work in an unrevealed collection
   errors:
-    timeout:
+    timeout_error:
       title: Timeout
       subtitle: The page was responding too slowly. Please try again after a few minutes.
       html: If you still get this error after a few tries, please %{contact_support_link}.
+      contact_support_link: contact Support
   feedbacks:
     new:
       abuse:

config/locales/views/en.yml

@@ -53,7 +53,7 @@
   get '/422', to: 'errors#422'
   get '/500', to: 'errors#500'
   get '/auth_error', to: 'errors#auth_error'
-  get "/timeout", to: "errors#timeout"
+  get "/timeout_error", to: "errors#timeout_error"
 
   #### DOWNLOADS ####
 

config/routes.rb

@@ -29,7 +29,7 @@
     end
   end
 
-  describe "auth_error" do
+  describe "GET #auth_error" do
     it "returns an HTML auth error page" do
       get :auth_error
       expect(response.status).to eq(200)