diffbench 0.0.2 → 0.1.0

data/README.md

@@ -3,13 +3,12 @@
 Diffbench is tool the I end up during many many performance patches to:
 
 * Rails
-  * ActiveRecord
-  * ActiveModel
-  * ActiveSupport
-* Mail
+  * ActiveRecord - [#5467](https://github.com/rails/rails/pull/5467)
+  * ActiveModel - [#5431](https://github.com/rails/rails/pull/5431)
+  * ActiveSupport - [#4493](https://github.com/rails/rails/pull/4493)
+* Mail - [#396](https://github.com/mikel/mail/pull/369), [#366](https://github.com/mikel/mail/pull/366)
 
-It runs a same benchmark code before and after applying a patch
-TODO links to original PRs
+It runs a same benchmark code before and after applying a patch.
 
 ## Installation
 
@@ -32,10 +31,14 @@ DiffBench.bm do
     Mail::Header.new("X-Subscriber: 1111\n"* 1000)
   end
   report("headers parsing when tiny") do
-    Mail::Header.new("X-Subscriber: 1111\n"* 10)
+    10.times do
+      Mail::Header.new("X-Subscriber: 1111\n"* 10)
+    end
   end
   report("headers parsing when empty") do
-    Mail::Header.new("")
+    100.times do
+      Mail::Header.new("")
+    end
   end
 end
 ```
@@ -50,4 +53,26 @@ If the working tree is dirty than diffbench will run benchmark against dirty and
 If the working tree is not dirty than diffbench will run benchmark against current HEAD and commit previous to HEAD.
 
 
+Output:
+
+```
+Running benchmark with current working tree
+Checkout HEAD^
+Running benchmark with HEAD^
+Checkout to previous HEAD again
+
+                    user     system      total        real
+----------------------------------headers parsing when long
+After patch:    0.100000   0.000000   0.100000 (  0.089926)
+Before patch:   0.700000   0.000000   0.700000 (  0.697444)
+
+----------------------------------headers parsing when tiny
+After patch:    0.000000   0.000000   0.000000 (  0.009930)
+Before patch:   0.020000   0.000000   0.020000 (  0.024283)
+
+---------------------------------headers parsing when empty
+After patch:    0.010000   0.000000   0.010000 (  0.002160)
+Before patch:   0.000000   0.000000   0.000000 (  0.002354)
+```
+
 

data/VERSION

@@ -1 +1 @@
-0.0.2
+0.1.0

data/bin/diffbench

@@ -1,5 +1,10 @@
 #!/usr/bin/env ruby
 
+if File.exists?(File.join(File.expand_path('../..', __FILE__), '.git'))
+  diffbench_path = File.expand_path('../../lib', __FILE__)
+  $:.unshift(diffbench_path)
+end
+
 require 'diffbench'
 
 DiffBench.run(*ARGV)

data/diffbench.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "diffbench"
-  s.version = "0.0.2"
+  s.version = "0.1.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Bogdan Gusiev"]
-  s.date = "2012-03-16"
+  s.date = "2012-03-23"
   s.description = "Diffbench is gem designed to benchmark the performance patches. It can run specified benchmark file before and after some changes made and show performance comparation result"
   s.email = "agresso@gmail.com"
   s.executables = ["diffbench"]
@@ -33,6 +33,8 @@ Gem::Specification.new do |s|
     "features/support/env.rb",
     "lib/diff_bench.rb",
     "lib/diffbench.rb",
+    "spec/bench.rb",
+    "spec/code.rb",
     "spec/diffbench_spec.rb",
     "spec/spec_helper.rb"
   ]

data/lib/diffbench.rb

@@ -60,7 +60,10 @@ class DiffBench
     end
 
     def run_file
-      output = `ruby #{@file}`
+      output = `ruby -I#{File.dirname(__FILE__)} #{@file}`
+      if $?.to_i > 0
+        raise Error, "Error exit code: #{$?.to_i}"
+      end
       begin
         result = YAML.load(output) 
         raise Error, "Can not parse result of ruby script: \n #{output}" unless result.is_a?(Hash)

data/spec/bench.rb

@@ -0,0 +1,12 @@
+$LOAD_PATH.unshift "./lib"
+require 'diffbench'
+load File.dirname(__FILE__) + "/code.rb"
+
+DiffBench.bm do
+  report "Sleeper 1" do
+    Sleeper.run
+  end
+  report "Sleeper 2" do
+    Sleeper.run
+  end
+end

data/spec/code.rb

@@ -0,0 +1,8 @@
+class Sleeper
+
+  TIME = 0.2
+
+  def self.run
+    sleep(TIME)
+  end
+end

data/spec/diffbench_spec.rb

@@ -1,7 +1,46 @@
-require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+require 'spec_helper'
+require "fileutils"
 
-describe "Diffbench" do
-  it "fails" do
-    fail "hey buddy, you should probably rename this file and start specing for real"
+describe DiffBench do
+  let(:repo) do
+    "#{File.dirname(__FILE__)}/repo"
   end
+  let!(:git) do
+    FileUtils.rm_rf(repo)
+    FileUtils.mkdir(repo)
+    git = Git.init(repo)
+    FileUtils.cp("spec/code.rb", repo)
+    git.add("code.rb")
+    git.commit("Init")
+    git
+  end
+
+
+  describe "when git tree is dirty" do
+    before do
+      content = File.read("#{repo}/code.rb")
+      File.open("#{repo}/code.rb", "w") do |f|
+        f.write(content.gsub(/TIME = 0\.2/, "TIME = 0.1"))
+      end
+      FileUtils.cp("spec/bench.rb", "#{repo}/bench.rb")
+    end
+
+    it "should run benchmark with dirty tree and clean tree" do
+      puts `cd #{repo}; ./../../bin/diffbench bench.rb`
+    end
+
+    describe "when changes got commit" do
+
+      before(:each) do
+        git.add("code.rb")
+        git.commit("Commit")
+      end
+
+      it "should run benchmark with HEAD and HEAD^" do
+        puts `cd #{repo}; ./../../bin/diffbench bench.rb`
+      end
+    end
+  end
+
+  
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: diffbench
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.1.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-03-16 00:00:00.000000000 Z
+date: 2012-03-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: git
-  requirement: &16059340 !ruby/object:Gem::Requirement
+  requirement: &15252040 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *16059340
+  version_requirements: *15252040
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &16058500 !ruby/object:Gem::Requirement
+  requirement: &15251480 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: 2.8.0
   type: :development
   prerelease: false
-  version_requirements: *16058500
+  version_requirements: *15251480
 - !ruby/object:Gem::Dependency
   name: rdoc
-  requirement: &16057740 !ruby/object:Gem::Requirement
+  requirement: &15250800 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: '3.12'
   type: :development
   prerelease: false
-  version_requirements: *16057740
+  version_requirements: *15250800
 - !ruby/object:Gem::Dependency
   name: cucumber
-  requirement: &16056880 !ruby/object:Gem::Requirement
+  requirement: &15250180 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *16056880
+  version_requirements: *15250180
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &16072220 !ruby/object:Gem::Requirement
+  requirement: &15249420 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -65,10 +65,10 @@ dependencies:
         version: 1.1.0
   type: :development
   prerelease: false
-  version_requirements: *16072220
+  version_requirements: *15249420
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &16071080 !ruby/object:Gem::Requirement
+  requirement: &15248740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -76,10 +76,10 @@ dependencies:
         version: 1.8.3
   type: :development
   prerelease: false
-  version_requirements: *16071080
+  version_requirements: *15248740
 - !ruby/object:Gem::Dependency
   name: rcov
-  requirement: &16070060 !ruby/object:Gem::Requirement
+  requirement: &15248240 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,7 +87,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *16070060
+  version_requirements: *15248240
 description: Diffbench is gem designed to benchmark the performance patches. It can
   run specified benchmark file before and after some changes made and show performance
   comparation result
@@ -114,6 +114,8 @@ files:
 - features/support/env.rb
 - lib/diff_bench.rb
 - lib/diffbench.rb
+- spec/bench.rb
+- spec/code.rb
 - spec/diffbench_spec.rb
 - spec/spec_helper.rb
 homepage: http://github.com/bogdan/diffbench
@@ -131,7 +133,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 446831729209371677
+      hash: -1898630107307179820
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: