peeek 1.0.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 90ae3ee8f84b1d47140d8ebd28943835875550be
-  data.tar.gz: 8a7bdc5de0df596be2e309ee950112e89ff94166
+  metadata.gz: 4cf3690abfc7e18ba772e5813bd248c6e8f77e31
+  data.tar.gz: 97b35ef5d8c535fc08db01336fbc0ee319be02f9
 SHA512:
-  metadata.gz: f6ffdc7048653dc766511b66272c730a1f37ac04a1aa7efe35efa4a834916f758d5766ad5c07da75954cb5a63c4a067788014514ca2edcd94756654f5b13dc75
-  data.tar.gz: d32b0ca28b0520cf773aa1f1f22c678168cdf6211dcac657796d1dc73fd85819db714df49e7e2ca8c4a9cd8fa78b1a62f80818afbddd3fb9c786c44649ef55e3
+  metadata.gz: 19ddd27a103fe0c1c52484f1525e73743ed7755bdfbc323a3de52263bc4ba2aee2694280ac50ccdeb0d5fbac44d412812c3d745c44c560c81a594c1ed2fd767d
+  data.tar.gz: 31f272cd92df1acf44b5e16877a30c1c7975e0fec030322c7cb8308449b69ef421153af4b0cf8048eff5c9f4fa967b2e2eebc18f0c5793ce9be78d06bcf45b5a

data/README.md

@@ -96,7 +96,7 @@ Hook to methods at head of the block, and return the calls, then use
 
 ### Filter calls
 
-`Peeek#calls` or `Peeek.local` return an instance of `Peeek::Calls`. It has
+`Peeek#calls` or `Peeek.capture` return an instance of `Peeek::Calls`. It has
 implemented methods to filter by attributes of the call.
 
     require 'peeek'

data/lib/peeek/hook.rb

@@ -118,7 +118,11 @@ class Peeek
 
     # Link the hook to the method.
     def link
-      @original_method = @linker.link(&method(:call)) unless linked?
+      unless linked?
+        @original_method = @linker.target_method
+        @linker.link(&method(:call))
+      end
+
       self
     end
 

data/lib/peeek/hook/instance.rb

@@ -11,6 +11,12 @@ class Peeek
         METHOD_PREFIX
       end
 
+      # @attribute [r] target_method
+      # @return [UnboundMethod] the instance method of the object
+      def target_method
+        @object.instance_method(@method_name)
+      end
+
       # Determine if the instance method is defined in the object.
       #
       # @return whether the instance method is defined in the object
@@ -25,12 +31,9 @@ class Peeek
       # @yieldparam [Object] receiver object that received the call
       # @yieldparam [Array] args arguments at the call
       # @yieldreturn [Object] return value of the original method
-      # @return [UnboundMethod] the original method
       def link
         raise ArgumentError, 'block not supplied' unless block_given?
-        original_method = @object.instance_method(@method_name)
         define_method { |*args| yield caller, self, args }
-        original_method
       end
 
       # Unlink the hook from the instance method.

data/lib/peeek/hook/singleton.rb

@@ -11,6 +11,12 @@ class Peeek
         METHOD_PREFIX
       end
 
+      # @attribute [r] target_method
+      # @return [Method] the method of the object
+      def target_method
+        @object.method(@method_name)
+      end
+
       # Determine if the method is defined in the object.
       #
       # @return whether the method is defined in the object
@@ -25,12 +31,9 @@ class Peeek
       # @yieldparam [Object] receiver object that received the call
       # @yieldparam [Array] args arguments at the call
       # @yieldreturn [Object] return value of the original method
-      # @return [Method] original method
       def link
         raise ArgumentError, 'block not supplied' unless block_given?
-        original_method = @object.method(@method_name)
         define_method { |*args| yield caller, self, args }
-        original_method
       end
 
       # Unlink the hook from the method.

data/lib/peeek/version.rb

@@ -1,3 +1,3 @@
 class Peeek
-  VERSION = '1.0.2'
+  VERSION = '1.0.3'
 end

data/spec/peeek/hook/instance_spec.rb

@@ -11,6 +11,15 @@ describe Peeek::Hook::Instance, '#method_prefix' do
   end
 end
 
+describe Peeek::Hook::Instance, '#target_method' do
+  it 'returns the instance method of the object' do
+    linker = sample_instance_linker
+    method = linker.target_method
+    method.owner.should == String
+    method.name.should == :%
+  end
+end
+
 describe Peeek::Hook::Instance, '#defined?' do
   it 'is true if the method is defined in the object' do
     linker = sample_instance_linker
@@ -58,10 +67,6 @@ describe Peeek::Hook::Instance, '#link' do
     lambda { '%s (%d)' % ['Koyomi', 18] }.should raise_error('exception')
   end
 
-  it 'returns the original method' do
-    @linker.link { }.should == @original_method
-  end
-
   it 'raises ArgumentError if a block not given' do
     lambda { @linker.link }.should raise_error(ArgumentError, 'block not supplied')
   end

data/spec/peeek/hook/singleton_spec.rb

@@ -11,6 +11,15 @@ describe Peeek::Hook::Singleton, '#method_prefix' do
   end
 end
 
+describe Peeek::Hook::Singleton, '#target_method' do
+  it 'returns the method of the object' do
+    linker = sample_singleton_linker
+    method = linker.target_method
+    method.receiver.should == Regexp
+    method.name.should == :quote
+  end
+end
+
 describe Peeek::Hook::Singleton, '#defined?' do
   it 'is true if the method is defined in the object' do
     linker = sample_singleton_linker
@@ -53,10 +62,6 @@ describe Peeek::Hook::Singleton, '#link' do
     lambda { Regexp.quote('.') }.should raise_error('exception')
   end
 
-  it 'returns the original method' do
-    @linker.link { }.should == @original_method
-  end
-
   it 'raises ArgumentError if a block not given' do
     lambda { @linker.link }.should raise_error(ArgumentError, 'block not supplied')
   end

data/spec/peeek/hook_spec.rb

@@ -182,6 +182,14 @@ describe Peeek::Hook, '#link' do
     @hook.link
   end
 
+  it "is linked before call #{described_class}::Linker#link" do
+    @linker.stub!(:link).with { }.and_return do |&block|
+      @hook.should be_linked
+    end
+
+    @hook.link
+  end
+
   it 'links the hook to the method' do
     @hook.should_not be_linked # assert
     @hook.link

data/spec/spec_helper.rb

@@ -24,7 +24,7 @@ end
 
 def instance_linker_stub(object, method_name)
   original_method = object.instance_method(method_name)
-  linker = stub('Peeek::Hook::Linker', :link => original_method, :unlink => nil)
+  linker = stub('Peeek::Hook::Linker', :target_method => original_method, :link => nil, :unlink => nil)
   [linker, original_method]
 end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: peeek
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors:
 - Takahiro Kondo
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-08 00:00:00.000000000 Z
+date: 2013-04-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -101,7 +101,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.0
+rubygems_version: 2.0.3
 signing_key: 
 specification_version: 4
 summary: Peek at calls of a method