Fix presentation of the Queue in the Recurring Jobs page

Fixes mattyr#38

Author: Fryguy

Rakefile

@@ -22,14 +22,19 @@ task :web do
     }
   end
 
-  # throw a fake job in
+  # throw some fake jobs in
   Sidecloq.configure do |config|
     sched = Sidecloq::Schedule.from_hash({
       my_scheduled_job: {
         class: 'DoWork',
         cron: '* * * * *',
         queue: 'default'
+      },
+      my_scheduled_job2: {
+        class: 'DoWorkWithQueue',
+        cron: '* * * * *'
       }
+
     })
     sched.save_redis
     config[:schedule] = sched
@@ -39,6 +44,11 @@ task :web do
     include Sidekiq::Worker
   end
 
+  class DoWorkWithQueue
+    include Sidekiq::Worker
+    sidekiq_options queue: "not_default"
+  end
+
   require 'sidekiq/web'
   require 'sidecloq/web'
   require 'securerandom'

lib/sidecloq/web.rb

@@ -5,7 +5,15 @@ module Web
 
     def self.registered(app)
       app.get '/recurring' do
-        @schedule = Schedule.from_redis
+        @job_specs = Schedule.from_redis.job_specs
+        @job_specs.each_value do |job_spec|
+          job_spec['cron'] ||= job_spec['every']
+
+          job_spec['queue'] ||= begin
+            klass = Object.const_get(job_spec['class'])
+            (klass.sidekiq_options_hash && klass.sidekiq_options_hash.fetch('queue', 'default')) || 'default'
+          end
+        end
 
         erb File.read(File.join(VIEW_PATH, 'recurring.erb'))
       end

test/helper.rb

@@ -52,6 +52,11 @@ class DummyJob
   include Sidekiq::Worker
 end
 
+class DummyJobWithQueue
+  include Sidekiq::Worker
+  sidekiq_options :queue => "dummy"
+end
+
 require 'active_job'
 
 ActiveJob::Base.queue_adapter = :sidekiq

web/views/recurring.erb

@@ -14,13 +14,13 @@
     </thead>
 
     <tbody>
-      <% @schedule.job_specs.each do |name, job_spec| %>
+      <% @job_specs.each do |name, job_spec| %>
         <tr>
           <td><%= name %></td>
-          <td><%= job_spec.fetch 'cron', job_spec['every'] %></td>
+          <td><%= job_spec['cron'] %></td>
           <td><%= job_spec['class'] %></td>
           <td>
-            <a href="<%= root_path %>queues/<%= job_spec.fetch('queue', 'default') %>"><%= job_spec.fetch('queue', 'default') %></a>
+            <a href="<%= root_path %>queues/<%= job_spec['queue'] %>"><%= job_spec['queue'] %></a>
           </td>
           <td><%= job_spec['args'] %></td>
           <td>