take_care 0.0.1 → 0.0.2

data/README.md

@@ -1,8 +1,8 @@
 # TakeCare
 
-Sidekiq wrapper for activerecord model
+Sidekiq wrapper for activerecord model (or any class which instances are fetched by #id)
 
-Method #take_care(method, *args) that delegates actual performing to sidekiq
+Method #take_care(method, *args) that delegates actual performing of own method to sidekiq
 
 ## Installation
 
@@ -35,6 +35,13 @@ human = Human.create(name: 'Alex')
 
 human.take_care :hard_work, 'heavy box', 'second heavy box' # This goes to sidekiq
 
+human.take_care_of :hard_work, 'heavy box', 'second heavy box' # Same using alias
+
+# Same using dynamic methods (method is defined after first call)
+human.take_care_hard_work 'heavy box', 'second heavy box'
+# Or
+human.take_care_of_hard_work 'heavy box', 'second heavy box'
+
 ```
 
 ## Contributing

data/lib/take_care/reliable.rb

@@ -1,8 +1,24 @@
 module TakeCare
   module Reliable
-    def take_care(method, *args)
-      TakeCare::Worker.perform_async(self.class.to_s, self.id, method, *args)
+    def take_care(method_name, *args)
+      TakeCare::Worker.perform_async(self.class.to_s, self.id, method_name, *args)
     end
     alias_method :take_care_of, :take_care
+
+    def respond_to?(method_name)
+      return false 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)
+      self.class.__send__(:define_method, method_name) do |*args|
+        self.take_care($1.to_sym, *args)
+      end
+      self.take_care($1.to_sym, *args)
+    end
   end
 end

data/lib/take_care/version.rb

@@ -1,3 +1,3 @@
 module TakeCare
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/spec/support/human.rb

@@ -1,8 +1,20 @@
-class Human
-  def id
-    4
-  end
+module HumanMacros
+  def redefine_human_class
+    before :each do
+      Object.__send__ :remove_const, "Human" if Object.const_defined? "Human"
+      Object.const_set("Human", Class.new do
+        include TakeCare::Reliable
+        def id
+          4
+        end
 
-  def hard_work(box1, box2)
+        def hard_work(box1, box2)
+        end
+      end)
+    end
   end
 end
+
+RSpec.configure do |config|
+  config.extend HumanMacros
+end

data/spec/take_care/reliable_spec.rb

@@ -1,26 +1,56 @@
 require "spec_helper"
 
 describe TakeCare::Reliable do
-  let(:human) { Human.new.extend(TakeCare::Reliable) }
+  redefine_human_class
+  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)
-      human.take_care :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)
-      human.take_care_of :hard_work, :box1, :box2
+      subject.take_care_of :hard_work, :box1, :box2
     end
   end
 
-  # context "Dinamic methods 'take_care_...'" do
-  #   it "delegates method calls to Worker" do
-  #     TakeCare::Worker.should_receive(:perform_async).with("Human", 4, :hard_work, :box1, :box2)
-  #     human.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
+      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
+        Human.class_eval { private; def take_care_hard_work; end }
+        expect { subject.take_care_hard_work :box1, :box2 }.to raise_error NameError
+      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 }
+
+      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
+      end
+    end
+  end
 end

data/spec/take_care/worker_spec.rb

@@ -1,12 +1,14 @@
 require "spec_helper"
 
 describe TakeCare::Worker do
+  redefine_human_class
+
   describe '#perform' do
     it "should include Sidekiq::Worker" do
       described_class.ancestors.should include Sidekiq::Worker
     end
 
-    it "finds class instance by id and calls mathod with received args" do
+    it "finds class instance by id and calls method with received args" do
       duty = double("Human instance")
       Human.should_receive(:find).with(4).and_return(duty)
       duty.should_receive(:hard_work).with(:box1, :box2)

metadata

@@ -1,46 +1,52 @@
 --- !ruby/object:Gem::Specification
 name: take_care
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
+  prerelease: 
 platform: ruby
 authors:
 - Alexander Avoyants
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-22 00:00:00.000000000 Z
+date: 2013-10-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sidekiq
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -48,6 +54,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -55,29 +62,33 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: Sidekiq wrapper for activerecord model
@@ -104,26 +115,33 @@ files:
 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'
+      segments:
+      - 0
+      hash: 4157606556185016579
 required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: 4157606556185016579
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.6
+rubygems_version: 1.8.23
 signing_key: 
-specification_version: 4
+specification_version: 3
 summary: Sidekiq wrapper for activerecord model (any class which instances fetched
   by id)
 test_files:

checksums.yaml

@@ -1,7 +0,0 @@
----
-SHA1:
-  metadata.gz: fa95c237faea234849c224d4c8e00e69fc37af63
-  data.tar.gz: 7a3006a04abcdb2afcccd02407cac1efbe068d14
-SHA512:
-  metadata.gz: fc341db5e388cdae6c349a1870d6429d137977ce0641fd9f4d4963e07385418c1d320a8830734ffd480c5fbcb5e516ae0902d0b9f74999a111c38bd9740dbcbe
-  data.tar.gz: ba4dec1f20f71aa8347ec9d1f9e9422e2f39a21a6ae1f0dd8984cd276c639e0ebe03057a47217977fc132f321a8fa045e63ebe80649cbc5c7e4ea7c12f61fa2b