Skip to content

Commit

Permalink
Refactor: remove apparently dead code
Browse files Browse the repository at this point in the history
No tests broke when I removed this, and I was perplexed by what it might
have been intended to do, since it would have apparently set all
subsequent scenarios in a feature to be skipped if one of them failed.

I don't really understand why it didn't, but it's probably to do with
how step invocations are cached on steps, with the step invocations
being created anew when the next scenario is visited.
  • Loading branch information
mattwynne committed Mar 4, 2013
1 parent d0655cc commit b839dc7
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions lib/cucumber/ast/scenario.rb
Original file line number Diff line number Diff line change
Expand Up @@ -86,9 +86,6 @@ def status

def skip_invoke!
@steps.each{|step_invocation| step_invocation.skip_invoke!}
@feature.next_feature_element(self) do |next_one|
next_one.skip_invoke!
end
end

def to_sexp
Expand Down

0 comments on commit b839dc7

Please sign in to comment.