tick 0.1.1 → 0.1.2

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    tick (0.1.0)
+    tick (0.1.1)
       rainbow
 
 GEM

data/README.md

@@ -15,10 +15,24 @@ Installation
 
 Usge 
 --------
+include Tick 
 
+    class Foo 
+      include Tick 
+    end
+
+Tick your method define method 
+  
     def foo 
     end 
     tick :foo  
+
+Sample output without color
+
+    TICK: method 'foo' in class 'Foo'  (0.1 s)
+
+Customize message for each method 
+
     tick :foo, :message => "benchmark for foo"
     tick :foo, :message => lambda {|class_name, method_name| " #{class_name}-#{method_name}"}
 

data/lib/tick.rb

@@ -87,10 +87,10 @@ module Tick
 
   module ClassMethods 
     def tick(method_name, options = {})
-      alias_method "#{method_name}_without_tick", method_name  
-      define_method method_name do
-        result = nil 
-        if Tick.enabled 
+      if Tick.enabled
+        alias_method "#{method_name}_without_tick", method_name  
+        define_method method_name do
+          result = nil 
           sec = Benchmark.realtime  { result = self.send("#{method_name}_without_tick") } 
 
           desc = nil 
@@ -102,10 +102,8 @@ module Tick
 
           time = Tick.time_message.call(sec)
           _log_benchmark(desc, time)
-        else  
-          result = self.send("#{method_name}_without_tick")
+          result
         end
-        result
       end
     end
   end

data/lib/tick/version.rb

@@ -1,3 +1,3 @@
 module Tick
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

data/spec/tick_spec.rb

@@ -44,6 +44,7 @@ describe Tick do
     Tick.enabled.should be false
   end
 
+
   it "can set custom desc color" do 
     Tick.desc_color = "#FFC482" 
     Tick.desc_color.should == "#FFC482"
@@ -101,6 +102,17 @@ describe "A class include Tick" do
     lambda {@klass.send(:tick, :xxx)}.should raise_error
   end
 
+  it "do not rewrite method if turn of tick" do 
+    Tick.enabled = false 
+    @klass.class_eval do  
+      def default
+        sleep 0.5
+      end
+    end
+    @klass.send(:tick, :default) 
+    @klass.new.should_not respond_to :default_without_tick  
+  end
+
   describe "tick with options" do 
     before(:each) do 
       @klass.class_eval do  
@@ -144,15 +156,6 @@ describe "A class include Tick" do
       mock(@instance)._log_benchmark(Tick.desc_message.call("TestClass","default"), anything)
       @instance.default
     end
-
-    it "should not do benchmark if Tick is turn off" do 
-      dont_allow(Benchmark).realtime(anything)
-      old_value = Tick.enabled
-      Tick.enabled = false
-      @instance.default
-      Tick.enabled = old_value
-    end
-
   end
 
   describe "#_log_benchmark" do 
@@ -168,7 +171,7 @@ describe "A class include Tick" do
 
     it "log sec in color" do 
       desc = "TICK: method 'default' in class ''"
-      time = "(0.1 ms)"
+      time = "(0.1 s)"
       message = ""
       message << desc
       message << "  "

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: tick
 version: !ruby/object:Gem::Version 
-  hash: 25
+  hash: 31
   prerelease: false
   segments: 
   - 0
   - 1
-  - 1
-  version: 0.1.1
+  - 2
+  version: 0.1.2
 platform: ruby
 authors: 
 - allenwei
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-01-01 00:00:00 +08:00
+date: 2011-01-03 00:00:00 +08:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency