uberhook 0.0.1 → 0.0.2

data/README.md

@@ -1,5 +1,7 @@
 # Uberhook
 
+[![Build Status](https://secure.travis-ci.org/mjio/uberhook.png)](http://travis-ci.org/mjio/uberhook)
+
 The Uberhook gem allows you to define `before` and `after` hooks for your classes and models.
 
 ## Installation

data/lib/uberhook.rb

@@ -48,9 +48,16 @@ module Uberhook
 
     def define_hook(type, target, callback)
       alias_method "#{target}_without_callbacks", target
-      define_method target do
-        send callback
-        send "#{target}_without_callbacks"
+      if type == :pre
+        define_method target do
+          send callback
+          send "#{target}_without_callbacks"
+        end
+      elsif type == :post
+        define_method target do
+          send "#{target}_without_callbacks"
+          send callback
+        end
       end
     end
 

data/test/uberhook_test.rb

@@ -3,9 +3,10 @@ require 'test_helper'
 class UberhookTest < Uberhook::TestCase
   def setup
     @car = Class.new do
+      attr_reader :status
       include Uberhook
-      def drive; end
-      def start; end
+      def drive; @status = :drive; end
+      def start; @status = :start; end
       def stop; end
     end
   end
@@ -16,7 +17,7 @@ class UberhookTest < Uberhook::TestCase
       before :drive, :start
     end
   end
-  
+
   test 'should register :post hook' do
     @car.expects(:register_hook).with(:post, :drive, :stop).once
     @car.class_eval do
@@ -32,41 +33,39 @@ class UberhookTest < Uberhook::TestCase
       def two; end
     end
   end
-  
+
   test 'should not define :pre hook when target is not defined' do
     @car.expects(:define_hook).times(0)
     @car.class_eval do
       register_hook :pre, :two, :one
     end
   end
-  
+
   test 'should not define :post hook when target is not defined' do
     @car.expects(:define_hook).times(0)
     @car.class_eval do
       register_hook :post, :one, :two
     end
   end
-  
+
   test 'should run callback before the hook method' do
     @car.class_eval do
       register_hook :pre, :drive, :start
     end
     beatle = @car.new
-    # TODO how to test before call
     beatle.expects(:start).once
     beatle.drive
   end
-  
+
   test 'should run callback after the hook method' do
     @car.class_eval do
-      register_hook :post, :drive, :start
+      register_hook :post, :start, :drive
     end
     beatle = @car.new
-    # TODO how to test after call
-    beatle.expects(:start).once
-    beatle.drive
+    beatle.expects(:drive).once
+    beatle.start
   end
-  
+
   test 'should run :pre callback when hook method gets defined' do
     @car.class_eval do
       register_hook :pre, :two, :one
@@ -74,11 +73,21 @@ class UberhookTest < Uberhook::TestCase
       def two; end
     end
     beatle = @car.new
-    # TODO how to test before call
     beatle.expects(:one).once
     beatle.two
   end
-  
+
+  test 'should run :pre callback after hook method gets defined' do
+    @car.class_eval do
+      register_hook :pre, :two, :one
+      def one; @status = :one; end
+      def two; @status = :two; end
+    end
+    beatle = @car.new
+    beatle.two
+    assert_equal :two, beatle.status
+  end
+
   test 'should run :post callback when hook method gets defined' do
     @car.class_eval do
       register_hook :post, :one, :two
@@ -86,8 +95,18 @@ class UberhookTest < Uberhook::TestCase
       def two; end
     end
     beatle = @car.new
-    # TODO how to test after call
     beatle.expects(:two).once
     beatle.one
   end
+
+  test 'should run :post callback after hook method gets defined' do
+    @car.class_eval do
+      register_hook :post, :one, :two
+      def one; @status = :one; end
+      def two; @status = :two; end
+    end
+    beatle = @car.new
+    beatle.one
+    assert_equal :two, beatle.status
+  end
 end

data/uberhook.gemspec

@@ -1,14 +1,14 @@
 Gem::Specification.new do |gem|
   gem.name          = 'uberhook'
-  gem.version       = '0.0.1'
+  gem.version       = '0.0.2'
   gem.authors       = ['Martin Jagusch']
   gem.email         = ['_@mj.io']
   gem.description   = 'Allows you to define :before and :after hooks for your classes and models.'
   gem.summary       = ':before and :after hooks'
   gem.homepage      = 'https://github.com/mjio/uberhook'
 
-  gem.add_development_dependency 'rake', '~> 0.9.2.2'
-  gem.add_development_dependency 'mocha', '~> 0.12.1'
+  gem.add_development_dependency 'rake'
+  gem.add_development_dependency 'mocha'
 
   gem.files         = `git ls-files`.split($/)
   gem.test_files    = gem.files.grep(%r{^test/})

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: uberhook
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,40 +9,40 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-08-02 00:00:00.000000000 Z
+date: 2012-08-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
-        version: 0.9.2.2
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
-        version: 0.9.2.2
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: mocha
   requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
-        version: 0.12.1
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
-        version: 0.12.1
+        version: '0'
 description: Allows you to define :before and :after hooks for your classes and models.
 email:
 - _@mj.io