take_care 0.0.3 → 0.0.4

checksums.yaml

@@ -0,0 +1,15 @@
+---
+!binary "U0hBMQ==":
+  metadata.gz: !binary |-
+    YTY1ZjQyYjYxYmJkN2NiOGM0YzZmOGNjYWIzOGRmZDRhZjY2ZTFiYQ==
+  data.tar.gz: !binary |-
+    OGVkMWJhNTY5YTM5YTE0MTkzNjBlZTQ3YjY4YjMxYjliN2I2ZWJlNw==
+SHA512:
+  metadata.gz: !binary |-
+    ZjQ3Y2Q2ODM0MmU1MDM0MTMzYzU4NzRiNjRjNWZlZTYzMWUyNjU5N2ZiZDdl
+    MGI2ZjgzOWNjNTBmOTkwZGQ5YzdlMGUxODRjZWVlMGYxYjUwMjJkZWVkMjA5
+    ZDhjMzRjNDY5ZjFlMjE1NWMyNGI0OTMyZDY1NWQ4YmNkZDE4NDA=
+  data.tar.gz: !binary |-
+    NGQ1MDM3M2VhZmFmMjJiOWJjMWI5YTNkOTU0MjgzMzAzMmE3ZjdlZDU3ZWI0
+    Yzg1ZjIyZGVkY2IzMWI0MzBkY2FmMGIxODVlZGUyMDhhYmViMjBlYjg2NWY4
+    ZDJlYWYwYmY5MjM1OTdlNTI2ZWJkZTU0NmE0NGQ1YmY4Y2Q3MTQ=

data/README.md

@@ -44,6 +44,28 @@ human.take_care_of_hard_work 'heavy box', 'second heavy box'
 
 ```
 
+There is also support for class methods for all kind of classes (not only ActiveRecord::Base).
+
+```rb
+class Human
+  include TakeCare::Reliable
+
+  def self.do_stuff(s1, s2)
+    # Code that needs to be executed in background
+  end
+end
+
+Human.take_care :do_stuff, 'arg1', 'arg2' # This goes to sidekiq
+
+Human.take_care_of :do_stuff, 'arg1', 'arg2' # Same using alias
+
+# Same using dynamic methods (method is defined after first call)
+Human.take_care_do_stuff 'arg1', 'arg2'
+# Or
+Human.take_care_of_do_stuff 'arg1', 'arg2'
+
+```
+
 ## Contributing
 
 1. Fork it

data/lib/take_care/reliable.rb

@@ -1,5 +1,7 @@
 module TakeCare
   module Reliable
+    extend ActiveSupport::Concern
+
     def take_care(method_name, *args)
       TakeCare::Worker.perform_async(self.class.to_s, self.id, method_name, *args)
     end
@@ -15,10 +17,34 @@ module TakeCare
         raise NoMethodError, "private method `#{method_name}' called for #{self}"
       end
       return super unless /\Atake_care_(?:of_)?(.+)\Z/ =~ method_name && self.respond_to?($1)
-      self.class.__send__(:define_method, method_name) do |*args|
-        self.take_care($1.to_sym, *args)
+      self.class.__send__(:define_method, method_name) do |*d_args|
+        self.take_care($1.to_sym, *d_args)
       end
       self.take_care($1.to_sym, *args)
     end
+
+    module ClassMethods
+      def take_care(method_name, *args)
+        TakeCare::WorkerC.perform_async(self.to_s, method_name, *args)
+      end
+      alias_method :take_care_of, :take_care
+
+      def respond_to?(method_name, include_private = false)
+        return include_private if self.class.private_method_defined?(method_name)
+        /\Atake_care_(?:of_)?(.+)\Z/ =~ method_name && self.respond_to?($1) || super
+      end
+
+      def method_missing(method_name, *args, &block)
+        if self.class.private_method_defined?(method_name)
+          raise NoMethodError, "private method `#{method_name}' called for #{self}"
+        end
+        return super unless /\Atake_care_(?:of_)?(.+)\Z/ =~ method_name && self.respond_to?($1)
+        eigenclass = class << self; self; end
+        eigenclass.__send__(:define_method, method_name) do |*d_args|
+          self.take_care($1.to_sym, *d_args)
+        end
+        self.take_care($1.to_sym, *args)
+      end
+    end
   end
 end

data/lib/take_care/version.rb

@@ -1,3 +1,3 @@
 module TakeCare
-  VERSION = "0.0.3"
+  VERSION = "0.0.4"
 end

data/lib/take_care/worker_c.rb

@@ -0,0 +1,12 @@
+require "sidekiq"
+require "active_support"
+
+module TakeCare
+  class WorkerC
+    include ::Sidekiq::Worker
+
+    def perform(class_name, class_method, *args)
+      class_name.constantize.__send__(class_method, *args)
+    end
+  end
+end

data/lib/take_care.rb

@@ -1,5 +1,6 @@
 require "take_care/version"
 require "take_care/worker"
+require "take_care/worker_c"
 require "take_care/reliable"
 
 module TakeCare

data/spec/support/human.rb

@@ -10,6 +10,19 @@ module HumanMacros
 
         def hard_work(box1, box2)
         end
+
+        private
+        def take_care_hard_work_in_guts(box1, box2)
+        end
+
+        class << self
+          def do_stuff(garbage1, garbage2)
+          end
+
+          private
+          def take_care_do_stuff_in_guts(garbage1, garbage2)
+          end
+        end
       end)
     end
   end

data/spec/take_care/reliable_spec.rb

@@ -2,59 +2,114 @@ require "spec_helper"
 
 describe TakeCare::Reliable do
   redefine_human_class
-  subject { Human.new }
+  context "Instance methods" do
+    subject { Human.new }
 
-  describe "#take_care" do
-    it "delegates method calls to Worker" do
-      TakeCare::Worker.should_receive(:perform_async).with("Human", 4, :hard_work, :box1, :box2)
-      subject.take_care :hard_work, :box1, :box2
-    end
-  end
-
-  describe "#take_care_of (alias of #take_care)" do
-    it "delegates method calls to Worker" do
-      TakeCare::Worker.should_receive(:perform_async).with("Human", 4, :hard_work, :box1, :box2)
-      subject.take_care_of :hard_work, :box1, :box2
+    describe "#take_care" do
+      it "delegates method calls to Worker" do
+        TakeCare::Worker.should_receive(:perform_async).with("Human", 4, :hard_work, :box1, :box2)
+        subject.take_care :hard_work, :box1, :box2
+      end
     end
-  end
 
-  context 'Dynamic methods "take_care_..."' do
-    describe "#method_missing" do
+    describe "#take_care_of (alias of #take_care)" do
       it "delegates method calls to Worker" do
         TakeCare::Worker.should_receive(:perform_async).with("Human", 4, :hard_work, :box1, :box2)
-        subject.take_care_hard_work :box1, :box2
+        subject.take_care_of :hard_work, :box1, :box2
       end
+    end
+
+    context 'Dynamic methods "take_care_..."' do
+      describe "#method_missing" do
+        it "delegates method calls to Worker" do
+          TakeCare::Worker.should_receive(:perform_async).with("Human", 4, :hard_work, :box1, :box2)
+          subject.take_care_hard_work :box1, :box2
+        end
 
-      it "defines dynamic method after first call (method missing should not be called after that)" do
-        TakeCare::Worker.stub(:perform_async)
-        subject.take_care_hard_work :box1, :box2
-        Human.instance_methods(false).should be_include :take_care_hard_work
+        it "defines dynamic method after first call (method missing should not be called after that)" do
+          TakeCare::Worker.stub(:perform_async)
+          subject.take_care_hard_work :box1, :box2
+          Human.instance_methods(false).should be_include :take_care_hard_work
+        end
+
+        it "second time acts same as first (dynamically defined method)" do
+          TakeCare::Worker.should_receive(:perform_async).with("Human", 4, :hard_work, :box1, :box2).twice
+          subject.take_care_hard_work :box1, :box2
+          subject.take_care_hard_work :box1, :box2
+        end
+
+        it "raises error if exists private method with same name" do
+          expect { subject.take_care_hard_work_in_guts :box1, :box2 }.to raise_error NameError
+        end
       end
 
-      it "second time acts same as first (dynamically defined method)" do
-        TakeCare::Worker.should_receive(:perform_async).with("Human", 4, :hard_work, :box1, :box2).twice
-        subject.take_care_hard_work :box1, :box2
-        subject.take_care_hard_work :box1, :box2
+      describe "#respond_to?" do
+        it { should be_respond_to :take_care_hard_work }
+        it { should be_respond_to :take_care_of_hard_work }
+
+        it "is false if exists private method with such name" do
+          should_not be_respond_to :take_care_hard_work_in_guts
+        end
+
+        it "is true if exists private method with such name and we are checking for private methods as well" do
+          should be_respond_to(:take_care_hard_work_in_guts, true)
+        end
       end
+    end
+  end
 
-      it "raises error if exists private method with same name" do
-        Human.class_eval { private; def take_care_hard_work; end }
-        expect { subject.take_care_hard_work :box1, :box2 }.to raise_error NameError
+  context "Class methods" do
+    subject { Human }
+
+    describe "#take_care" do
+      it "delegates class method calls to WorkerC" do
+        TakeCare::WorkerC.should_receive(:perform_async).with("Human", :do_stuff, :s1, :s2)
+        subject.take_care :do_stuff, :s1, :s2
       end
     end
 
-    describe "#respond_to?" do
-      it { should be_respond_to :take_care_hard_work }
-      it { should be_respond_to :take_care_of_hard_work }
+    describe "#take_care_of (alias of #take_care)" do
+      it "delegates class method calls to WorkerC" do
+        TakeCare::WorkerC.should_receive(:perform_async).with("Human", :do_stuff, :s1, :s2)
+        subject.take_care_of :do_stuff, :s1, :s2
+      end
+    end
 
-      it "is false if exists private method with such name" do
-        Human.class_eval { private; def take_care_hard_work; end }
-        should_not be_respond_to :take_care_hard_work
+    context 'Dynamic methods "take_care_..."' do
+      describe "#method_missing" do
+        it "delegates method calls to WorkerC" do
+          TakeCare::WorkerC.should_receive(:perform_async).with("Human", :do_stuff, :s1, :s2)
+          subject.take_care_do_stuff :s1, :s2
+        end
+
+        it "defines dynamic method after first call (method missing should not be called after that)" do
+          TakeCare::WorkerC.stub(:perform_async)
+          subject.take_care_do_stuff :s1, :s2
+          subject.methods(false).should be_include :take_care_do_stuff
+        end
+
+        it "second time acts same as first (dynamically defined method)" do
+          TakeCare::WorkerC.should_receive(:perform_async).with("Human", :do_stuff, :s1, :s2).twice
+          subject.take_care_do_stuff :s1, :s2
+          subject.take_care_do_stuff :s1, :s2
+        end
+
+        it "raises error if exists private method with same name" do
+          expect { subject.take_care_do_stuff_in_guts :s1, :s2 }.to raise_error NameError
+        end
       end
 
-      it "is true if exists private method with such name and we are checking for private methods as well" do
-        Human.class_eval { private; def take_care_hard_work; end }
-        should be_respond_to :take_care_hard_work, true
+      describe "#respond_to?" do
+        it { should be_respond_to :take_care_do_stuff }
+        it { should be_respond_to :take_care_of_do_stuff }
+
+        it "is false if exists private method with such name" do
+          should_not be_respond_to :take_care_do_stuff_in_guts
+        end
+
+        it "is true if exists private method with such name and we are checking for private methods as well" do
+          should be_respond_to :take_care_do_stuff_in_guts, true
+        end
       end
     end
   end

data/spec/take_care/worker_c_spec.rb

@@ -0,0 +1,16 @@
+require "spec_helper"
+
+describe TakeCare::WorkerC do
+  redefine_human_class
+
+  describe '#perform' do
+    it "should include Sidekiq::Worker" do
+      described_class.ancestors.should include Sidekiq::Worker
+    end
+
+    it "calls class method with received args" do
+      Human.should_receive(:do_stuff).with(:s1, :s2)
+      subject.perform("Human", :do_stuff, :s1, :s2)
+    end
+  end
+end

metadata

@@ -1,20 +1,18 @@
 --- !ruby/object:Gem::Specification
 name: take_care
 version: !ruby/object:Gem::Version
-  version: 0.0.3
-  prerelease: 
+  version: 0.0.4
 platform: ruby
 authors:
 - Alexander Avoyants
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-29 00:00:00.000000000 Z
+date: 2014-01-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sidekiq
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -22,7 +20,6 @@ dependencies:
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -30,7 +27,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -38,7 +34,6 @@ dependencies:
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -46,7 +41,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -54,7 +48,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -62,7 +55,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -70,7 +62,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -78,7 +69,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -86,7 +76,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -107,39 +96,41 @@ files:
 - lib/take_care/reliable.rb
 - lib/take_care/version.rb
 - lib/take_care/worker.rb
+- lib/take_care/worker_c.rb
 - spec/spec_helper.rb
 - spec/support/human.rb
 - spec/take_care/reliable_spec.rb
+- spec/take_care/worker_c_spec.rb
 - spec/take_care/worker_spec.rb
 - take_care.gemspec
 homepage: ''
 licenses:
 - MIT
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.23
+rubygems_version: 2.2.0
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Sidekiq wrapper for activerecord model (any class which instances fetched
   by id)
 test_files:
 - spec/spec_helper.rb
 - spec/support/human.rb
 - spec/take_care/reliable_spec.rb
+- spec/take_care/worker_c_spec.rb
 - spec/take_care/worker_spec.rb