hook_me_up 0.0.1 → 0.0.2

data/.rspec

@@ -1 +1,2 @@
 --color
+--format nested

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+## v0.0.2
+
+* Added options for disabling all or some of the hooks:
+	* :no_hook
+	* :no_before_hook
+	* :no_after_hook
+
 ## v0.0.1
 
 * initial release

data/README.md

@@ -25,27 +25,40 @@ Or install it yourself as:
 
 NOTE: `hook_me_up` call **must** come after your method definitions
 
+```ruby
+class SomeClass
+  include HookMeUp
 
-	class SomeClass
-		include HookMeUp
+	def some_method
+	end
 
-		def some_method
-		end
+	def some_other_method
+	end
 
-		def some_other_method
-		end
+	def before_hook
+	end
 
-		def before_hook
-		end
+	def after_hook
+	end
 
-		def after_hook
-		end
+	hook_me_up [:some_method, :some_other_method], :before => :before_hook, :after => :after_hook
+end
+```
 
-		hook_me_up [:some_method, :some_other_method], :before => :before_hook, :after => :after_hook
-	end
+### Disabling hooks for a specific call
+You can disable a hooks on specific calls with the following options:
+
+* :no_hook  
+* :no_before_hook  
+* :no_after_hook  
 
+```ruby
+some_method(args, no_hook: true) # No hooks will be called
+some_method(args, no_before_hook: true) # The before hook will be skipped
+some_method(args, no_after_hook: true)	# The After hook will be skipped
+```
 
-###You can pass lambda to the hooks instead of methods
+### You can pass lambda to the hooks instead of methods
 
 	hook_me_up :some_method, :before => lambda{ |sender, *args| sender.do_something(args) },
 			:after => lambda{ |sender, *args, result| sender.do_something_else(result) }

data/hook_me_up.gemspec

@@ -8,7 +8,7 @@ Gem::Specification.new do |gem|
   gem.version       = HookMeUp::VERSION
   gem.authors       = ["Chen Fisher"]
   gem.email         = ["chen.fisher@gmail.com"]
-  gem.description   = %q{Lets you hook any method class with :before and :after hooks}
+  gem.description   = %q{Lets you hook any method with :before and :after hooks}
   gem.summary       = %q{This gem lets you hook and method in any class with a before and after hooks}
   gem.homepage      = ""
 

data/lib/hook_me_up.rb

@@ -9,22 +9,45 @@ module HookMeUp
 				original_method = self.instance_method(method)
 
 				self.send(:define_method, method) do |*a|
-					if hooks[:before]
-						if hooks[:before].is_a? Proc
-							hooks[:before].call(self, *a)
-						else
-							self.send(hooks[:before], *a)
+					options = a.last
+					if options.is_a?(::Hash) && options[:no_hook] == true && a.pop	
+						result = original_method.bind(self).call(*a)
+					elsif options.is_a?(::Hash) && options[:no_before_hook] == true && a.pop
+						result = original_method.bind(self).call(*a)
+						if hooks[:after]
+							if hooks[:after].is_a? Proc
+								hooks[:after].call(self, *a, result)
+							else
+								self.send(hooks[:after], *a, result)
+							end
+						end
+					elsif options.is_a?(::Hash) && options[:no_after_hook] == true && a.pop
+						if hooks[:before]
+							if hooks[:before].is_a? Proc
+								hooks[:before].call(self, *a)
+							else
+								self.send(hooks[:before], *a)
+							end
+						end
+
+						result = original_method.bind(self).call(*a)
+					else
+						if hooks[:before]
+							if hooks[:before].is_a? Proc
+								hooks[:before].call(self, *a)
+							else
+								self.send(hooks[:before], *a)
+							end
 						end
-					end
-					
-					result = original_method.bind(self).call(*a)
 
+						result = original_method.bind(self).call(*a)
 
-					if hooks[:after]
-						if hooks[:after].is_a? Proc
-							hooks[:after].call(self, *a, result)
-						else
-							self.send(hooks[:after], *a, result)
+						if hooks[:after]
+							if hooks[:after].is_a? Proc
+								hooks[:after].call(self, *a, result)
+							else
+								self.send(hooks[:after], *a, result)
+							end
 						end
 					end
 
@@ -33,7 +56,7 @@ module HookMeUp
 			end
 		end
 	end
-	
+
 	module InstanceMethods
 		
 	end

data/lib/hook_me_up/version.rb

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

data/spec/hook_me_up_spec.rb

@@ -2,12 +2,10 @@ require 'spec_helper'
 require 'hook_me_up/sample_class'
 
 describe HookMeUp do
-	before(:each) do
-		@sample = SampleClass.new
-	end
+	let(:sample) { SampleClass.new }
 
 	it "hooks a before and after methods" do
-		class << @sample
+		class << sample
 			include HookMeUp
 
 			hook_me_up :method_one, :before => :before_hook, 
@@ -20,14 +18,14 @@ describe HookMeUp do
 			end
 		end
 
-		@sample.should_receive(:before_hook).with('args')
-		@sample.should_receive(:after_hook).with('args', 'args')
+		sample.should_receive(:before_hook).with('args')
+		sample.should_receive(:after_hook).with('args', 'args')
 
-		@sample.method_one('args').should eq 'args'
+		sample.method_one('args').should eq 'args'
 	end
 
 	it "can receive a block as a before or after hooks" do
-		class << @sample
+		class << sample
 			include HookMeUp
 
 			attr_accessor :before, :after
@@ -36,15 +34,15 @@ describe HookMeUp do
 				:after => lambda{ |sender, *args, result| sender.after = true }
 		end
 
-		@sample.method_one('args').should eq 'args'
+		sample.method_one('args').should eq 'args'
 
-		@sample.before.should eq true
-		@sample.after.should eq true
+		sample.before.should eq true
+		sample.after.should eq true
 	end
 
 
 	it "can receive multiple methods to hook" do
-		class << @sample
+		class << sample
 			include HookMeUp
 
 			attr_accessor :before, :after
@@ -54,19 +52,19 @@ describe HookMeUp do
 			def before_hook(*args)
 			end
 
-			def after_hook(*args, result)
+			def after_hook(*args)
 			end
 		end
 
-		@sample.should_receive(:before_hook).exactly(2)
-		@sample.should_receive(:after_hook).exactly(2)
+		sample.should_receive(:before_hook).exactly(2)
+		sample.should_receive(:after_hook).exactly(2)
 
-		@sample.method_one('args').should eq 'args'
-		@sample.method_two.should eq 'method_two'
+		sample.method_one('args').should eq 'args'
+		sample.method_two.should eq 'method_two'
 	end
 
 	it "should not call before or after if not defined" do
-		class << @sample
+		class << sample
 			include HookMeUp
 
 			hook_me_up :method_one, :before => :before_hook
@@ -78,9 +76,62 @@ describe HookMeUp do
 			end
 		end
 
-		@sample.should_receive(:before_hook).with('args')
-		@sample.should_not_receive(:after_hook)
+		sample.should_receive(:before_hook).with('args')
+		sample.should_not_receive(:after_hook)
+
+		sample.method_one('args').should eq 'args'
+	end
+
+	context "when disabled" do
+		before do
+			class << sample
+				include HookMeUp
+
+				hook_me_up :method_one, :before => :before_hook, :after => :after_hook
+
+				def before_hook(*args)
+				end
+
+				def after_hook(*args)
+				end
+			end
+		end
+
+		describe "no_hook" do
+			it "doesn't call before hook" do
+				sample.should_not_receive(:after_hook)
+				sample.method_one('args', no_hook: true).should eq 'args'
+			end
+
+			it "doesn't call after hook" do
+				sample.should_not_receive(:after_hook)
+				sample.method_one('args', no_hook: true).should eq 'args'
+			end
+		end
+
+		describe "no_before_hook" do
+			it "doesn't call before hook" do
+				sample.should_not_receive(:before_hook)
+				sample.method_one('args', no_before_hook: true).should eq 'args'
+			end
+
+			it "calls the after hook" do
+				sample.should_receive(:after_hook)
+				sample.method_one('args')
+			end
+		end
+
+		describe "no_after_hook" do
+			it "doesn't call after hook" do
+				sample.should_not_receive(:after_hook).with('args', 'args')
+				sample.method_one('args', no_after_hook: true).should eq 'args'
+			end
+
+			it "calls the before hook" do
+				sample.should_receive(:before_hook)
+				sample.method_one('args')
+			end
+		end
 
-		@sample.method_one('args').should eq 'args'
 	end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hook_me_up
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-10-19 00:00:00.000000000 Z
+date: 2012-10-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -27,7 +27,7 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
-description: Lets you hook any method class with :before and :after hooks
+description: Lets you hook any method with :before and :after hooks
 email:
 - chen.fisher@gmail.com
 executables: []