sidekiq 0.9.0 → 0.9.1

data/Changes.md

@@ -1,3 +1,8 @@
+0.9.1
+-----------
+
+- Fix missed deprecations, poor method name in web UI
+
 0.9.0
 -----------
 

data/lib/sidekiq.rb

@@ -3,6 +3,7 @@ require 'sidekiq/client'
 require 'sidekiq/worker'
 require 'sidekiq/rails'
 require 'sidekiq/redis_connection'
+require 'sidekiq/util'
 
 require 'sidekiq/extensions/action_mailer'
 require 'sidekiq/extensions/active_record'

data/lib/sidekiq/version.rb

@@ -1,3 +1,3 @@
 module Sidekiq
-  VERSION = "0.9.0"
+  VERSION = "0.9.1"
 end

data/lib/sidekiq/web.rb

@@ -38,7 +38,7 @@ module Sidekiq
     helpers do
       def workers
         @workers ||= begin
-          Sidekiq.redis.with_connection do |conn|
+          Sidekiq.redis do |conn|
             conn.smembers('workers').map do |w|
               msg = conn.get("worker:#{w}")
               msg = MultiJson.decode(msg) if msg
@@ -49,15 +49,15 @@ module Sidekiq
       end
 
       def processed
-        Sidekiq.redis.get('stat:processed') || 0
+        Sidekiq.redis { |conn| conn.get('stat:processed') } || 0
       end
 
       def failed
-        Sidekiq.redis.get('stat:failed') || 0
+        Sidekiq.redis { |conn| conn.get('stat:failed') } || 0
       end
 
       def queues
-        Sidekiq.redis.with_connection do |conn|
+        Sidekiq.redis do |conn|
           conn.smembers('queues').map do |q|
             [q, conn.llen("queue:#{q}") || 0]
           end.sort { |x,y| x[1] <=> y[1] }
@@ -65,14 +65,14 @@ module Sidekiq
       end
 
       def location
-        Sidekiq.redis.client.location
+        Sidekiq.redis { |conn| conn.client.location }
       end
 
       def root_path
         "#{env['SCRIPT_NAME']}/"
       end
 
-      def status
+      def current_status
         return 'down' if workers.size == 0
         return 'idle' if workers.size > 0 && workers.map { |x| x[1] }.compact.size == 0
         return 'active'
@@ -84,8 +84,9 @@ module Sidekiq
     end
 
     get "/queues/:name" do
+      halt 404 unless params[:name]
       @name = params[:name]
-      @messages = Sidekiq.redis.lrange("queue:#{@name}", 0, 10).map { |str| MultiJson.decode(str) }
+      @messages = Sidekiq.redis {|conn| conn.lrange("queue:#{@name}", 0, 10) }.map { |str| MultiJson.decode(str) }
       slim :queue
     end
   end

data/test/test_web.rb

@@ -47,5 +47,15 @@ class TestWeb < MiniTest::Unit::TestCase
       assert_match last_response.body, /foo/
     end
 
+    it 'handles queues with no name' do
+      get '/queues'
+      assert_equal 404, last_response.status
+    end
+
+    it 'handles queue view' do
+      get '/queues/default'
+      assert_equal 200, last_response.status
+    end
+
   end
 end

data/web/views/index.slim

@@ -1,6 +1,6 @@
 
 .hero-unit
-  h1 Sidekiq is #{status}
+  h1 Sidekiq is #{current_status}
   p Processed: #{processed}
   p Failed: #{failed}
   p Workers: #{workers.size}

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sidekiq
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.9.1
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2012-03-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis
-  requirement: &70241342627260 !ruby/object:Gem::Requirement
+  requirement: &70238239105760 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70241342627260
+  version_requirements: *70238239105760
 - !ruby/object:Gem::Dependency
   name: redis-namespace
-  requirement: &70241342626820 !ruby/object:Gem::Requirement
+  requirement: &70238239105320 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70241342626820
+  version_requirements: *70238239105320
 - !ruby/object:Gem::Dependency
   name: connection_pool
-  requirement: &70241342626320 !ruby/object:Gem::Requirement
+  requirement: &70238239104820 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: 0.9.0
   type: :runtime
   prerelease: false
-  version_requirements: *70241342626320
+  version_requirements: *70238239104820
 - !ruby/object:Gem::Dependency
   name: celluloid
-  requirement: &70241342625900 !ruby/object:Gem::Requirement
+  requirement: &70238239104400 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70241342625900
+  version_requirements: *70238239104400
 - !ruby/object:Gem::Dependency
   name: multi_json
-  requirement: &70241342625440 !ruby/object:Gem::Requirement
+  requirement: &70238239103940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70241342625440
+  version_requirements: *70238239103940
 - !ruby/object:Gem::Dependency
   name: minitest
-  requirement: &70241342625020 !ruby/object:Gem::Requirement
+  requirement: &70238239103520 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70241342625020
+  version_requirements: *70238239103520
 - !ruby/object:Gem::Dependency
   name: sinatra
-  requirement: &70241342624600 !ruby/object:Gem::Requirement
+  requirement: &70238239103100 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70241342624600
+  version_requirements: *70238239103100
 - !ruby/object:Gem::Dependency
   name: slim
-  requirement: &70241342624180 !ruby/object:Gem::Requirement
+  requirement: &70238239102680 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,10 +98,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70241342624180
+  version_requirements: *70238239102680
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70241342623760 !ruby/object:Gem::Requirement
+  requirement: &70238239102260 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -109,10 +109,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70241342623760
+  version_requirements: *70238239102260
 - !ruby/object:Gem::Dependency
   name: actionmailer
-  requirement: &70241342623340 !ruby/object:Gem::Requirement
+  requirement: &70238239101840 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -120,10 +120,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70241342623340
+  version_requirements: *70238239101840
 - !ruby/object:Gem::Dependency
   name: activerecord
-  requirement: &70241342622920 !ruby/object:Gem::Requirement
+  requirement: &70238239101420 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -131,7 +131,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70241342622920
+  version_requirements: *70238239101420
 description: Simple, efficient message processing for Ruby
 email:
 - mperham@gmail.com