method_profiler 1.0.1 → 1.1.0

data/.travis.yml

@@ -0,0 +1,7 @@
+language: ruby
+rvm:
+  - 1.8.7
+  - 1.9.3
+  - jruby
+  - rbx
+  - ree

data/Gemfile

@@ -1,3 +1,5 @@
 source :rubygems
 
 gemspec
+
+gem 'rake', :group => :test

data/README.md

@@ -1,3 +1,5 @@
+[![Build Status](https://secure.travis-ci.org/change/method_profiler.png)](http://travis-ci.org/change/method_profiler) [![endorse](http://api.coderwall.com/jimmycuadra/endorsecount.png)](http://coderwall.com/jimmycuadra)
+
 # MethodProfiler
 
 **MethodProfiler** collects performance information about the methods in your objects and creates reports to help you identify slow methods. The collected data can be sorted in various ways, converted into an array, or pretty printed as a table.
@@ -25,31 +27,33 @@ The resulting chart includes each method, the minimum time it took to run, the m
 
 ```
 MethodProfiler results for: MyClass
-+------------------------+-----------+------------+--------------+-------------+
-| Method                 | Min Time  | Max Time   | Average Time | Total Calls |
-+------------------------+-----------+------------+--------------+-------------+
-| #accusamus_est         | 32.086 ms | 324.140 ms | 160.252 ms   | 3           |
-| #autem_iste!           | 29.607 ms | 318.592 ms | 156.221 ms   | 3           |
-| #distinctio_eos        | 29.477 ms | 318.471 ms | 156.097 ms   | 3           |
-| #laborum_fugit         | 18.388 ms | 291.900 ms | 140.580 ms   | 3           |
-| #suscipit_architecto   | 12.036 ms | 272.279 ms | 130.247 ms   | 3           |
-| #et_fugit              | 2.794 ms  | 11.658 ms  | 6.185 ms     | 3           |
-| #porro_rerum           | 0.097 ms  | 12.096 ms  | 1.031 ms     | 43          |
-| #provident_molestiae   | 0.005 ms  | 3.997 ms   | 0.871 ms     | 6           |
-| .nisi_inventore        | 0.368 ms  | 1.329 ms   | 0.849 ms     | 2           |
-| #quis_temporibus       | 0.104 ms  | 6.696 ms   | 0.713 ms     | 24          |
-| #labore_voluptatum     | 0.004 ms  | 8.834 ms   | 0.447 ms     | 86          |
-| #quia_est              | 0.004 ms  | 3.667 ms   | 0.327 ms     | 24          |
-| #ut_reiciendis         | 0.004 ms  | 2.842 ms   | 0.250 ms     | 24          |
-| #sint_quasi            | 0.066 ms  | 1.836 ms   | 0.166 ms     | 24          |
-| #sed_at                | 0.067 ms  | 0.119 ms   | 0.078 ms     | 24          |
-| #repellendus_suscipit  | 0.054 ms  | 0.128 ms   | 0.067 ms     | 24          |
-| #quas_nesciunt         | 0.024 ms  | 0.026 ms   | 0.025 ms     | 3           |
-| #iure_quis             | 0.006 ms  | 0.232 ms   | 0.014 ms     | 48          |
-| #dicta_ipsam           | 0.005 ms  | 0.068 ms   | 0.013 ms     | 24          |
-| #perspiciatis_aut      | 0.006 ms  | 0.006 ms   | 0.006 ms     | 3           |
-| #aperiam_laborum       | 0.004 ms  | 0.009 ms   | 0.005 ms     | 34          |
-+------------------------+-----------+------------+--------------+-------------+
++-----------------------+-----------+------------+--------------+------------+-------------+
+| Method                | Min Time  | Max Time   | Average Time | Total Time | Total Calls |
++-----------------------+-----------+------------+--------------+------------+-------------+
+| #accusamus_est        | 28.722 ms | 393.649 ms | 150.543 ms   | 451.628 ms | 3           |
+| #autem_iste!          | 26.220 ms | 387.026 ms | 146.644 ms   | 439.933 ms | 3           |
+| #distinctio_eos       | 26.095 ms | 386.903 ms | 146.520 ms   | 439.559 ms | 3           |
+| #laborum_fugit        | 14.887 ms | 351.369 ms | 127.564 ms   | 382.692 ms | 3           |
+| #suscipit_architecto  | 9.876 ms  | 269.339 ms | 96.440 ms    | 289.319 ms | 3           |
+| #et_fugit             | 0.005 ms  | 63.101 ms  | 10.704 ms    | 64.225 ms  | 6           |
+| #porro_rerum          | 2.970 ms  | 15.137 ms  | 7.126 ms     | 21.378 ms  | 3           |
+| #provident_molestiae  | 0.097 ms  | 17.860 ms  | 1.134 ms     | 27.225 ms  | 24          |
+| #nisi_inventore       | 0.098 ms  | 15.076 ms  | 1.044 ms     | 54.272 ms  | 52          |
+| #quis_temporibus      | 0.004 ms  | 11.908 ms  | 0.643 ms     | 15.430 ms  | 24          |
+| .labore_voluptatum    | 0.440 ms  | 0.470 ms   | 0.455 ms     | 0.910 ms   | 2           |
+| #quia_est             | 0.004 ms  | 11.133 ms  | 0.453 ms     | 47.092 ms  | 104         |
+| #ut_reiciendis        | 0.004 ms  | 5.626 ms   | 0.346 ms     | 8.302 ms   | 24          |
+| #sint_quasi           | 0.062 ms  | 2.152 ms   | 0.188 ms     | 4.504 ms   | 24          |
+| #sed_at               | 0.065 ms  | 0.150 ms   | 0.085 ms     | 2.034 ms   | 24          |
+| #repellendus_suscipit | 0.051 ms  | 0.122 ms   | 0.070 ms     | 1.684 ms   | 24          |
+| .quas_nesciunt        | 0.058 ms  | 0.124 ms   | 0.062 ms     | 4.303 ms   | 69          |
+| #iure_quis            | 0.021 ms  | 0.025 ms   | 0.023 ms     | 0.069 ms   | 3           |
+| #dicta_ipsam          | 0.006 ms  | 0.266 ms   | 0.017 ms     | 0.798 ms   | 48          |
+| #perspiciatis_aut     | 0.004 ms  | 0.068 ms   | 0.013 ms     | 0.314 ms   | 24          |
+| .aperiam_laborum      | 0.005 ms  | 0.015 ms   | 0.006 ms     | 0.438 ms   | 69          |
+| #voluptas_ratione     | 0.005 ms  | 0.007 ms   | 0.006 ms     | 0.018 ms   | 3           |
+| #ex_voluptas          | 0.004 ms  | 0.010 ms   | 0.005 ms     | 0.212 ms   | 41          |
++-----------------------+-----------+------------+--------------+------------+-------------+
 ```
 
 ## Reporting

data/lib/method_profiler/profiler.rb

@@ -38,8 +38,8 @@ module MethodProfiler
 
       @obj.singleton_class.module_eval do
         singleton_methods_to_wrap.each do |method|
-          define_method("#{method}_with_profiling") do |*args|
-            profiler.send(:profile, method, true) { send("#{method}_without_profiling", *args) }
+          define_method("#{method}_with_profiling") do |*args, &block|
+            profiler.send(:profile, method, true) { send("#{method}_without_profiling", *args, &block) }
           end
 
           alias_method "#{method}_without_profiling", method
@@ -49,8 +49,8 @@ module MethodProfiler
 
       @obj.module_eval do
         instance_methods_to_wrap.each do |method|
-          define_method("#{method}_with_profiling") do |*args|
-            profiler.send(:profile, method) { send("#{method}_without_profiling", *args) }
+          define_method("#{method}_with_profiling") do |*args, &block|
+            profiler.send(:profile, method) { send("#{method}_without_profiling", *args, &block) }
           end
 
           alias_method "#{method}_without_profiling", method
@@ -85,13 +85,15 @@ module MethodProfiler
 
       @data.each do |method, records|
         total_calls = records.size
-        average = records.reduce(:+) / total_calls
+        total_time = records.reduce(:+)
+        average = total_time / total_calls
         results << {
           :method => method,
           :min => records.min,
           :max => records.max,
           :average => average,
-          :total_calls => total_calls
+          :total_time => total_time,
+          :total_calls => total_calls,
         }
       end
 

data/lib/method_profiler/report.rb

@@ -6,7 +6,7 @@ module MethodProfiler
   #
   class Report
     # Fields that can be passed to {#sort_by}.
-    FIELDS = [:method, :min, :max, :average, :total_calls]
+    FIELDS = [:method, :min, :max, :average, :total_time, :total_calls]
 
     # Directions that can be passed to {#order}.
     DIRECTIONS = [:asc, :ascending, :desc, :descending]
@@ -73,13 +73,15 @@ module MethodProfiler
             :min => "Min Time",
             :max => "Max Time",
             :average => "Average Time",
-            :total_calls => "Total Calls"
+            :total_time => "Total Time",
+            :total_calls => "Total Calls",
           },
-          :fields => [:method, :min, :max, :average, :total_calls],
+          :fields => [:method, :min, :max, :average, :total_time, :total_calls],
           :filters => {
             :min => :to_milliseconds,
             :max => :to_milliseconds,
-            :average => :to_milliseconds
+            :average => :to_milliseconds,
+            :total_time => :to_milliseconds,
           },
           :description => false
         )

data/method_profiler.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
   s.name        = "method_profiler"
-  s.version     = "1.0.1"
+  s.version     = "1.1.0"
   s.authors     = ["Jimmy Cuadra"]
   s.email       = ["jimmy@jimmycuadra.com"]
   s.homepage    = "https://github.com/change/method_profiler"

data/spec/method_profiler/profiler_spec.rb

@@ -11,14 +11,30 @@ describe MethodProfiler::Profiler do
     petition.should_not respond_to(:foo_with_profiling_with_profiling)
   end
 
-  it "class methods should properly return values" do
+  it "returns correct values for class methods" do
     Petition.guys.should == "sup"
   end
 
-  it "instance method should properly return values" do
+  it "returns correct values for instance methods" do
     petition.baz.should == "blah"
   end
 
+  it "yields to implicit blocks" do
+    petition.method_with_implicit_block {|v| v }.should == "implicit"
+  end
+
+  it "calls explicit blocks" do
+    petition.method_with_explicit_block {|v| v }.should == "explicit"
+  end
+
+  it "yields to implicit blocks with arguments" do
+    petition.method_with_implicit_block_and_args(1,2,3) {|v| v }.should == [1,2,3]
+  end
+
+  it "calls explicit blocks with arguments" do
+    petition.method_with_explicit_block_and_args(1,2,3) {|v| v }.should == [1,2,3]
+  end
+
   describe "#report" do
     it "returns a new Report object" do
       profiler.report.should be_an_instance_of MethodProfiler::Report

data/spec/support/petition.rb

@@ -15,4 +15,20 @@ class Petition
   def baz
     "blah"
   end
+
+  def method_with_implicit_block
+    yield "implicit"
+  end
+
+  def method_with_explicit_block(&block)
+    block.call "explicit"
+  end
+
+  def method_with_implicit_block_and_args(*args)
+    yield args
+  end
+
+  def method_with_explicit_block_and_args(*args, &block)
+    block.call args
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: method_profiler
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.1.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-03-06 00:00:00.000000000 Z
+date: 2012-03-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hirb
-  requirement: &70256884010560 !ruby/object:Gem::Requirement
+  requirement: &70291604515080 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: 0.6.0
   type: :runtime
   prerelease: false
-  version_requirements: *70256884010560
+  version_requirements: *70291604515080
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70256884010160 !ruby/object:Gem::Requirement
+  requirement: &70291604514680 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70256884010160
+  version_requirements: *70291604514680
 - !ruby/object:Gem::Dependency
   name: simplecov
-  requirement: &70256884009700 !ruby/object:Gem::Requirement
+  requirement: &70291604514220 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,7 +43,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70256884009700
+  version_requirements: *70291604514220
 description: MethodProfiler observes your code and generates reports about the methods
   that were run and how long they took.
 email:
@@ -53,6 +53,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- .travis.yml
 - Gemfile
 - LICENSE
 - README.md