benchmark_harness 0.0.2 → 0.0.3

data/VERSION

@@ -1 +1 @@
-0.0.2
+0.0.3

data/benchmark_harness.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{benchmark_harness}
-  s.version = "0.0.2"
+  s.version = "0.0.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Daniel Higginbotham"]

data/lib/benchmark_harness.rb

@@ -43,14 +43,15 @@ class BenchmarkHarness
     else
       raise ArgumentError, "your signature must contain a method"
     end
-    
+        
     constant_name, method_name = signature.split(split_on)
+    method_name_safe = method_name.gsub(/[\W\D]/, '') # trip out ie !
     constant = get_constant(constant_name)
     
     method_definition = <<-END
-      alias :#{method_name}_pre_benchmark_harness :#{method_name}
+      alias :#{method_name_safe}_pre_benchmark_harness :#{method_name}
       def #{method_name}(*args, &block)
-        BenchmarkHarness.measure(:#{collection_name}){result = #{method_name}_pre_benchmark_harness(*args, &block)}
+        BenchmarkHarness.measure(:#{collection_name}){result = #{method_name_safe}_pre_benchmark_harness(*args, &block)}
       end
     END
     
@@ -69,7 +70,7 @@ class BenchmarkHarness
       if parent_constant.const_defined?(piece)
         parent_constant = parent_constant.const_get(piece)
       else
-        raise NameError, "#{name} contains an undefined constant"
+        raise NameError, "#{name} contains an undefined constant; it may be autoloaded"
       end
     end
     parent_constant

data/spec/benchmark_harness_spec.rb

@@ -95,6 +95,12 @@ describe "BenchmarkHarness" do
       NestedExample::OneLevelDown.new.subtract
       BenchmarkHarness.collections[:subtract].should_not be_empty
     end
+    
+    it "should wrap a method with an '!' in the name" do
+      BenchmarkHarness.wrap_method("NestedExample#important!", :important)
+      NestedExample.new.important!
+      BenchmarkHarness.collections[:important].should_not be_empty
+    end
   end
   
   describe "#flush" do

data/spec/fixtures/nested_example.rb

@@ -1,4 +1,4 @@
-module NestedExample
+class NestedExample
   class OneLevelDown
     class << self
       def add
@@ -10,4 +10,8 @@ module NestedExample
       1 - 1
     end
   end
+  
+  def important!
+    return nil
+  end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: benchmark_harness
 version: !ruby/object:Gem::Version 
-  hash: 27
+  hash: 25
   prerelease: false
   segments: 
   - 0
   - 0
-  - 2
-  version: 0.0.2
+  - 3
+  version: 0.0.3
 platform: ruby
 authors: 
 - Daniel Higginbotham