method_profiler 1.0.0 → 1.0.1

data/lib/method_profiler/core_ext/object.rb

@@ -0,0 +1,9 @@
+# Backports Object#singleton_class from Ruby 1.9.2
+class Object
+  # Returns the singleton class of an object.
+  #
+  # @return [Object] The object's singleton class.
+  def singleton_class
+    class << self; self end
+  end
+end if RUBY_VERSION < "1.9.2"

data/lib/method_profiler/profiler.rb

@@ -1,4 +1,5 @@
 require 'method_profiler/report'
+require 'method_profiler/core_ext/object'
 
 require 'benchmark'
 
@@ -73,7 +74,8 @@ module MethodProfiler
     def profile(method, singleton = false, &block)
       method_name = singleton ? ".#{method}" : "##{method}"
       result = nil
-      elapsed_time = benchmark(result, &block).to_s.match(/\(\s*([^\)]+)\)/)[1].to_f
+      elapsed_time, result = benchmark(result, &block)
+      elapsed_time = elapsed_time.to_s.match(/\(\s*([^\)]+)\)/)[1].to_f
       @data[method_name] << elapsed_time
       result
     end
@@ -85,11 +87,11 @@ module MethodProfiler
         total_calls = records.size
         average = records.reduce(:+) / total_calls
         results << {
-          method: method,
-          min: records.min,
-          max: records.max,
-          average: average,
-          total_calls: total_calls
+          :method => method,
+          :min => records.min,
+          :max => records.max,
+          :average => average,
+          :total_calls => total_calls
         }
       end
 
@@ -99,7 +101,8 @@ module MethodProfiler
     private
 
     def benchmark(result, &block)
-      Benchmark.measure { result = block.call }
+      elapsed_time = Benchmark.measure { result = block.call }
+      return elapsed_time, result
     end
   end
 end

data/lib/method_profiler/report.rb

@@ -68,20 +68,20 @@ module MethodProfiler
         "MethodProfiler results for: #{@obj}",
         Hirb::Helpers::Table.render(
           to_a,
-          headers: {
-            method: "Method",
-            min: "Min Time",
-            max: "Max Time",
-            average: "Average Time",
-            total_calls: "Total Calls"
+          :headers => {
+            :method => "Method",
+            :min => "Min Time",
+            :max => "Max Time",
+            :average => "Average Time",
+            :total_calls => "Total Calls"
           },
-          fields: [:method, :min, :max, :average, :total_calls],
-          filters: {
-            min: :to_milliseconds,
-            max: :to_milliseconds,
-            average: :to_milliseconds
+          :fields => [:method, :min, :max, :average, :total_calls],
+          :filters => {
+            :min => :to_milliseconds,
+            :max => :to_milliseconds,
+            :average => :to_milliseconds
           },
-          description: false
+          :description => false
         )
       ].join("\n")
     end

data/method_profiler.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
   s.name        = "method_profiler"
-  s.version     = "1.0.0"
+  s.version     = "1.0.1"
   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,6 +11,14 @@ describe MethodProfiler::Profiler do
     petition.should_not respond_to(:foo_with_profiling_with_profiling)
   end
 
+  it "class methods should properly return values" do
+    Petition.guys.should == "sup"
+  end
+
+  it "instance method should properly return values" do
+    petition.baz.should == "blah"
+  end
+
   describe "#report" do
     it "returns a new Report object" do
       profiler.report.should be_an_instance_of MethodProfiler::Report

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: method_profiler
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-03-04 00:00:00.000000000 Z
+date: 2012-03-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hirb
-  requirement: &70181816945620 !ruby/object:Gem::Requirement
+  requirement: &70256884010560 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: 0.6.0
   type: :runtime
   prerelease: false
-  version_requirements: *70181816945620
+  version_requirements: *70256884010560
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70181816945220 !ruby/object:Gem::Requirement
+  requirement: &70256884010160 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70181816945220
+  version_requirements: *70256884010160
 - !ruby/object:Gem::Dependency
   name: simplecov
-  requirement: &70181816944760 !ruby/object:Gem::Requirement
+  requirement: &70256884009700 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,7 +43,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70181816944760
+  version_requirements: *70256884009700
 description: MethodProfiler observes your code and generates reports about the methods
   that were run and how long they took.
 email:
@@ -58,6 +58,7 @@ files:
 - README.md
 - Rakefile
 - lib/method_profiler.rb
+- lib/method_profiler/core_ext/object.rb
 - lib/method_profiler/hirb.rb
 - lib/method_profiler/profiler.rb
 - lib/method_profiler/report.rb