sidetiq 0.6.2 → 0.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b04b7cc75ee4bfc199cd0d844008ddbe233488f5
-  data.tar.gz: be839f3d4817872e6d968c49838e738ef096d0b5
+  metadata.gz: 1c946fd72cae8f96d3079762cfe6f113e7c2eab2
+  data.tar.gz: 50e1c485a7d3f8d1947b6c15b7fe2d386cf6e44d
 SHA512:
-  metadata.gz: 6cbcf675522a069a327442266facd020fd24a62798ca3466827ba340d6a78a90f04b01cb029732248d9d33933677847cb3f19949dbdbc00760b1a82c629925d7
-  data.tar.gz: 5a3118fac8bbc228f6e3ba656344a746c12b622f7204c0de6948ffd67ace5f61992a7b86672e2ba0ccdd65e28422660222dda8101e5b95df95d12bd63fdb097a
+  metadata.gz: 0d51d953cdbd0ac1ec2543561258ef77f2d657de4bb84232d3f7a0a70e90d189c914f65aea5a54543a3d806e94278f9e2b17217301a85021ed91c3413a65dcc2
+  data.tar.gz: f3ca33fb55398a6c55289dfaf436a53de17c189723619e902be0c78770a6f5066dc12fd4f3b3b820431dc052d4219c1aabdfd4947ddd7cfa661f4959aad34da3

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+0.6.3
+-----
+
+- Don't pass classes without #perform defined to handler
+- Don't try to connect to redis from Schduleable unless
+  Sidekiq.server? returns true.
+
 0.6.2
 -----
 

data/lib/sidetiq/api.rb

@@ -1,9 +1,13 @@
 module Sidetiq
   # Public: Sidetiq API methods.
   module API
-    # Public: Returns an Array of workers including Sidetiq::Schedulable.
+    # Public: Returns an Array of workers including
+    # Sidetiq::Schedulable. Excludes classes which don't define
+    # a #perform method.
     def workers
-      Sidetiq::Schedulable.subclasses(true)
+      Sidetiq::Schedulable.subclasses(true).select do |klass|
+        klass.method_defined?(:perform)
+      end
     end
 
     # Public: Returns a Hash of Sidetiq::Schedule instances.

data/lib/sidetiq/schedulable.rb

@@ -42,12 +42,14 @@ module Sidetiq
         schedule.set_options(options)
 
         # deleting schedulable keys if schedule changed since last reccurence definition
-        old_description = get_schedulable_key("schedule_description")
-        if old_description != schedule.to_s
-          get_schedulable_keys.map do |key|
-            Sidekiq.redis_pool.with { |r| r.del(key) }
+        if Sidekiq.server?
+          old_description = get_schedulable_key("schedule_description")
+          if old_description != schedule.to_s
+            get_schedulable_keys.map do |key|
+              Sidekiq.redis_pool.with { |r| r.del(key) }
+            end
+            set_schedulable_key("schedule_description", schedule.to_s)
           end
-          set_schedulable_key("schedule_description", schedule.to_s)
         end
       end
 

data/lib/sidetiq/version.rb

@@ -8,7 +8,7 @@ module Sidetiq
     MINOR = 6
 
     # Public: Sidetiq patch level.
-    PATCH = 2
+    PATCH = 3
 
     # Public: Sidetiq version suffix.
     SUFFIX = nil

data/test/test_subclass_tracking.rb

@@ -1,7 +1,10 @@
 require_relative 'helper'
 
 class TestSubclassTracking < Sidetiq::TestCase
-  class Foo
+  class Base
+  end
+
+  class Foo < Base
     extend Sidetiq::SubclassTracking
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sidetiq
 version: !ruby/object:Gem::Version
-  version: 0.6.2
+  version: 0.6.3
 platform: ruby
 authors:
 - Tobias Svensson
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-09-22 00:00:00.000000000 Z
+date: 2014-09-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sidekiq