test: ✅ update should run on step

Author: gingerbenw

test/browser/features/steps/browser-steps.rb

@@ -216,27 +216,32 @@ def check_attribute_equal_if_present(field, attribute, attr_type, expected)
 
 Then(/^on ((?:[A-Za-z]+ versions (?:>=?|<=?|==) [0-9.]+(?:, )?)+):$/) do |browser_specs, steps_to_run|
   spec_matcher = /^([A-Za-z]+) versions (>=?|<=?|==) ([0-9.]+)$/
+  run_steps = false
 
   browser_specs.split(", ").each do |browser_spec|
     browser_spec.scan(spec_matcher) do |name, operator, version|
       should_run_steps = $browser.name.casecmp?(name) && $browser.version.send(operator, version.to_i)
 
       # make sure this step is debuggable!
-      $logger.debug("#{$browser.name} == #{name} && v#{$browser.version} #{operator} #{version}? #{should_run_steps}")
-
+      $logger.debug("#{$browser.name} == #{name} && v#{$browser.version} #{operator} #{version}? #{should_run_steps}")     
+      
       if should_run_steps
-        steps_to_run.each_line(chomp: true) do |step_to_run|
-          step(step_to_run)
-        end
-      else
-        indent = " " * 4
-        # e.g. "a step\nanother step\n" -> "    1) a step\n    2) another step"
-        steps_indented = steps_to_run.each_line.map.with_index(1) { |step, i| "#{indent}#{i}) #{step.chomp}" }.join("\n")
-
-        $logger.info("Skipping steps on #{$browser.name} v#{$browser.version}:\n#{steps_indented}")
+        run_steps = true
       end
     end
   end
+  
+  if run_steps
+    steps_to_run.each_line(chomp: true) do |step_to_run|
+      step(step_to_run)
+    end
+  else   
+    indent = " " * 4
+    # e.g. "a step\nanother step\n" -> "    1) a step\n    2) another step"
+    steps_indented = steps_to_run.each_line.map.with_index(1) { |step, i| "#{indent}#{i}) #{step.chomp}" }.join("\n")
+
+    $logger.info("Skipping steps on #{$browser.name} v#{$browser.version}:\n#{steps_indented}")
+  end
 end
 
 module Maze