Simplify Runtime API by using EventBus for StepMatches

cucumber · Sep 4, 2015 · 8bb7860 · 8bb7860
1 parent 752fe0d
commit 8bb7860
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/lib/cucumber/formatter/usage.rb b/lib/cucumber/formatter/usage.rb
@@ -16,12 +16,16 @@ def initialize(runtime, path_or_io, options)
         @options = options
         @stepdef_to_match = Hash.new { |h, stepdef_key| h[stepdef_key] = [] }
         @total_duration = 0
+        @matches = {}
+        runtime.configuration.on_event :step_match do |event|
+          @matches[event.test_step.source] = event.step_match
+        end
       end
 
       def after_test_step(test_step, result)
         return if HookQueryVisitor.new(test_step).hook?
 
-        step_match = @runtime.step_match(test_step.source.last.name)
+        step_match = @matches[test_step.source]
         step_definition = step_match.step_definition
         stepdef_key = StepDefKey.new(step_definition.regexp_source, step_definition.location)
         unless @stepdef_to_match[stepdef_key].map { |key| key[:location] }.include? test_step.location

diff --git a/lib/cucumber/runtime.rb b/lib/cucumber/runtime.rb
@@ -87,10 +87,6 @@ def steps(status = nil)
       @results.steps(status)
     end
 
-    def step_match(step_name, name_to_report=nil) #:nodoc:
-      @support_code.step_match(step_name, name_to_report)
-    end
-
     def unmatched_step_definitions
       @support_code.unmatched_step_definitions
     end