should_be_faster 0.0.2 → 0.0.3

data/lib/should_be_faster.rb

@@ -1,25 +1,12 @@
 require 'benchmark'
 require 'spec'
 require 'spec/matchers'
+require 'spec/matchers/be'
 
 def _benchmark(code, iter)
   Benchmark.measure { iter.times { code.call } }
 end
 
-Spec::Matchers.define(:be_faster_than) do |rhs_code, options|
-  options ||= {}
-  options[:matcher] = self
-  options[:faster]  = true
-  Spec::Matchers::BenchmarkComparison.new(rhs_code, options).benchmark_comparison
-end
-
-Spec::Matchers.define(:be_slower_than) do |rhs_code, options|
-  options ||= {}
-  options[:matcher] = self
-  options[:faster]  = false
-  Spec::Matchers::BenchmarkComparison.new(rhs_code, options).benchmark_comparison
-end
-
 module Spec
   module Matchers
     class BenchmarkComparison
@@ -28,7 +15,7 @@ module Spec
         @options  = options
         @options[:iterations] ||= 100
         @options[:factor]     ||= 1
-        @options[:faster]       = @options[:faster] ? true : false  # I hate ||= not working with booleans
+        @options[:faster]       = @options[:faster]
       end
 
       def benchmark_comparison
@@ -76,7 +63,7 @@ module Spec
       end
     end
 
-    class BePredicate < Be
+    module BeFasterThan
       def times
         @factor = @args[0]
         self
@@ -102,5 +89,24 @@ module Spec
         end
       end
     end
+
   end
 end
+
+Spec::Matchers::BePredicate.send(:include, Spec::Matchers::BeFasterThan)
+Spec::Matchers::BeSameAs.send(:include, Spec::Matchers::BeFasterThan)
+
+Spec::Matchers.define(:be_faster_than) do |rhs_code, options|
+  options ||= {}
+  options[:matcher] = self
+  options[:faster]  = true
+  Spec::Matchers::BenchmarkComparison.new(rhs_code, options).benchmark_comparison
+end
+
+Spec::Matchers.define(:be_slower_than) do |rhs_code, options|
+  options ||= {}
+  options[:matcher] = self
+  options[:faster]  = false
+  Spec::Matchers::BenchmarkComparison.new(rhs_code, options).benchmark_comparison
+end
+

data/spec/should_be_faster_spec.rb

@@ -9,13 +9,24 @@ describe 'should_be_faster' do
     Spec::Matchers::BenchmarkComparison.new(nil, options)
   end
 
-  it 'should take an options hash to specify iterations' do
-    options = mock('hash')
-    options.stub!(:[])
-    options.stub!(:[]=)
-    options.stub!(:[]).with(:iterations).and_return(20)
-    Spec::Matchers::BenchmarkComparison.new(nil, options)
-    options[:iterations].should == 20
+  context 'with an options hash' do
+    it 'should allow you to specify iterations' do
+      options = mock('hash')
+      options.stub!(:[])
+      options.stub!(:[]=)
+      options.stub!(:[]).with(:iterations).and_return(20)
+      Spec::Matchers::BenchmarkComparison.new(nil, options)
+      options[:iterations].should == 20
+    end
+
+    it 'should allow you to specify factor' do
+      options = mock('hash')
+      options.stub!(:[])
+      options.stub!(:[]=)
+      options.stub!(:[]).with(:factor).and_return(20)
+      Spec::Matchers::BenchmarkComparison.new(nil, options)
+      options[:factor].should == 20
+    end
   end
 
   context 'when ensuring code is' do
@@ -29,9 +40,21 @@ describe 'should_be_faster' do
         @fast.should be_faster_than(@slow)
       end
 
+      it 'should ensure lhs  < rhs by x times' do
+        @fast.should be_faster_than(@slow, :factor => 2)
+      end
+
+      it 'should ensure lhs  < rhs by y secs'
+
       it 'should ensure lhs !> rhs' do
         @slow.should_not be_faster_than(@fast)
       end
+
+      it 'should ensure lhs !> rhs by x times' do
+        @slow.should_not be_faster_than(@fast, :factor => 2)
+      end
+
+      it 'should ensure lhs !> rhs by y secs'
     end
 
     context 'slower_than' do
@@ -39,9 +62,21 @@ describe 'should_be_faster' do
         @slow.should be_slower_than(@fast)
       end
 
+      it 'should ensure lhs  > rhs by x times' do
+        @slow.should be_slower_than(@fast, :factor => 2)
+      end
+
+      it 'should ensure lhs  > rhs by y secs'
+
       it 'should ensure lhs !< rhs' do
         @fast.should_not be_slower_than(@slow)
       end
+
+      it 'should ensure lhs !< rhs by x times' do
+        @fast.should_not be_slower_than(@slow, :factor => 2)
+      end
+
+      it 'should ensure lhs !< rhs by y secs'
     end
 
     context 'at_least(x).times.faster_than' do
@@ -63,5 +98,25 @@ describe 'should_be_faster' do
         @fast.should_not be_at_least(2).times.slower_than(@slow)
       end
     end
+
+    context 'x.times.slower_than' do
+      it 'should ensure lhs  > rhs' do
+        @slow.should be(2).times.slower_than(@fast)
+      end
+
+      it 'should ensure lhs !< rhs' do
+        @fast.should_not be(2).times.slower_than(@slow)
+      end
+    end
+
+    context 'x.times.faster_than' do
+      it 'should ensure lhs  > rhs' do
+        @fast.should be(2).times.faster_than(@slow)
+      end
+
+      it 'should ensure lhs !< rhs' do
+        @slow.should_not be(2).times.faster_than(@fast)
+      end
+    end
   end
 end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 0
-  - 2
-  version: 0.0.2
+  - 3
+  version: 0.0.3
 platform: ruby
 authors: 
 - David Turnbull
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-04-11 00:00:00 +10:00
+date: 2010-05-06 00:00:00 +10:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency