[rb] update after hook in specs to reduce unnecessary browser restarts

Author: titusfortner

rb/spec/integration/selenium/webdriver/bidi/browsing_context_spec.rb

@@ -23,7 +23,7 @@ module Selenium
   module WebDriver
     class BiDi
       describe BrowsingContext, exclusive: {bidi: true}, only: {browser: %i[chrome edge firefox]} do
-        after { reset_driver! }
+        after { |example| reset_driver!(example: example) }
 
         it 'can create a browsing context for given id' do
           id = driver.window_handle

rb/spec/integration/selenium/webdriver/bidi/script_spec.rb

@@ -22,7 +22,7 @@
 module Selenium
   module WebDriver
     describe Script, exclusive: {bidi: true}, only: {browser: %i[chrome edge firefox]} do
-      after { reset_driver! }
+      after { |example| reset_driver!(example: example) }
 
       it 'errors when bidi not enabled' do
         reset_driver!(web_socket_url: false) do |driver|

rb/spec/integration/selenium/webdriver/bidi_spec.rb

@@ -22,7 +22,7 @@
 module Selenium
   module WebDriver
     describe BiDi, exclusive: {bidi: true}, only: {browser: %i[chrome edge firefox]} do
-      after { reset_driver! }
+      after { |example| reset_driver!(example: example) }
 
       it 'errors when bidi not enabled' do
         reset_driver!(web_socket_url: false) do |driver|

rb/spec/integration/selenium/webdriver/devtools_spec.rb

@@ -22,7 +22,7 @@
 module Selenium
   module WebDriver
     describe DevTools, exclusive: {browser: %i[chrome edge firefox]} do
-      after { reset_driver! }
+      after { |example| reset_driver!(example: example) }
 
       it 'sends commands' do
         driver.devtools.page.navigate(url: url_for('xhtmlTest.html'))

rb/spec/integration/selenium/webdriver/spec_helper.rb

@@ -64,7 +64,7 @@
 
   c.after do |example|
     result = example.execution_result
-    reset_driver! if result.exception || result.pending_exception
+    reset_driver! if example.exception || result.pending_exception
   end
 end
 

rb/spec/integration/selenium/webdriver/spec_support/test_environment.rb

@@ -62,6 +62,9 @@ def driver_instance(...)
         end
 
         def reset_driver!(time: 0, **opts, &block)
+          # do not reset if the test was marked skipped
+          return if opts.delete(:example)&.metadata&.fetch(:skip, nil)
+
           quit_driver
           sleep time
           driver_instance(**opts, &block)