Addresses PR feedback

* Moves defaults handling from MissionControl::Jobs::Application to
  MissionControl::Jobs::Server to minimize impact of this new
  feature on the existing implementation and tests.

  This fixes the unit test failure that I missed in the previous commit.

* Uses @server rather than looking it up via params in the JobsController.
  This allows reverting all changes to the JobsController.

Author: hms

app/controllers/mission_control/jobs/jobs_controller.rb

@@ -2,7 +2,6 @@ class MissionControl::Jobs::JobsController < MissionControl::Jobs::ApplicationCo
   include MissionControl::Jobs::JobScoped, MissionControl::Jobs::JobFilters
 
   skip_before_action :set_job, only: :index
-  before_action :set_backtrace_cleaner, only: :show
 
   def index
     @job_class_names = jobs_with_status.job_class_names
@@ -38,12 +37,4 @@ def filtered_jobs
     def jobs_status
       params[:status].presence&.inquiry
     end
-
-    def set_backtrace_cleaner
-      @backtrace_cleaner = @application.servers[server_id]&.backtrace_cleaner
-    end
-
-    def server_id
-      params[:server_id]
-    end
 end

app/helpers/mission_control/jobs/jobs_helper.rb

@@ -15,9 +15,9 @@ def clean_backtrace?
     params["clean_backtrace"] == "true"
   end
 
-  def failed_job_backtrace(job)
-    if clean_backtrace? && @backtrace_cleaner
-      @backtrace_cleaner.clean(job.last_execution_error.backtrace).join("\n")
+  def failed_job_backtrace(job, server)
+    if clean_backtrace? && server&.backtrace_cleaner
+      server.backtrace_cleaner.clean(job.last_execution_error.backtrace).join("\n")
     else
       job.last_execution_error.backtrace.join("\n")
     end

app/views/mission_control/jobs/jobs/_error_information.html.erb

@@ -14,9 +14,9 @@
     </tbody>
   </table>
 
-  <% if @backtrace_cleaner %>
+  <% if @server.backtrace_cleaner %>
     <%= render "mission_control/jobs/jobs/failed/backtrace_toggle", application: @application, job: job %>
   <% end %>
 
-  <pre class="is-family-monospace mb-4 backtrace-content"><%= failed_job_backtrace(job) %></pre>
+  <pre class="is-family-monospace mb-4 backtrace-content"><%= failed_job_backtrace(job, @server) %></pre>
 <% end %>

lib/mission_control/jobs/application.rb

@@ -12,7 +12,6 @@ def initialize(name:)
   def add_servers(queue_adapters_by_name)
     queue_adapters_by_name.each do |name, queue_adapter|
       adapter, cleaner = queue_adapter
-      cleaner ||= MissionControl::Jobs.backtrace_cleaner
 
       servers << MissionControl::Jobs::Server.new(name: name.to_s, queue_adapter: adapter,
         backtrace_cleaner: cleaner, application: self)

lib/mission_control/jobs/server.rb

@@ -6,11 +6,11 @@ class MissionControl::Jobs::Server
 
   attr_reader :name, :queue_adapter, :application, :backtrace_cleaner
 
-  def initialize(name:, queue_adapter:, application:, backtrace_cleaner:)
+  def initialize(name:, queue_adapter:, application:, backtrace_cleaner: nil)
     super(name: name)
     @queue_adapter = queue_adapter
     @application = application
-    @backtrace_cleaner = backtrace_cleaner
+    @backtrace_cleaner = backtrace_cleaner || MissionControl::Jobs.backtrace_cleaner
   end
 
   def activating(&block)