bugsnag-maze-runner 9.27.2 → 9.27.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b280c22e5ebba0999aab175bdff85961db4eecda551b9ef7af9b86ce3e9a2dfb
-  data.tar.gz: 8ed9875673535f2753586d57b1c70352700a522e728b6e1a25de54175028e82e
+  metadata.gz: 87adf6fe40bf417708320af55a4cc71c2e8656e7f79bdd60902b473c2d7aa2b7
+  data.tar.gz: 47b1d556a9bf643ada71d5476cffe5f82ca5db36c08238911f964c3ac4d9c4fe
 SHA512:
-  metadata.gz: 184bbd13f45aedd6eaa04fe00afa119b8f8d23978e78674acced363a8af30858f94b3dc32414ec45973127bfff9880bd3577829fce0ae53e3cff621c78ef267a
-  data.tar.gz: c06d158038c59513ca71e63d1d0ea0a2a5462500e5236eccf54c4371631e943ccb8e641becf01f7b5aca37d6d7b67468b7285f0f42695ee50f5e0ab738f5d3f2
+  metadata.gz: ad05007ba4678a5a45811cae40ddbe6be9fce37d9a719520d7ee6494c436b9328f6ff649467286c4a241412c646ce5c9e83858fc565f5f69fdc3264f0a8bb478
+  data.tar.gz: 310739ae63a5583838503902c78e51030eb2ed1ccf175715f0270d5a5946cd4c94f48f62de3f8fbef758a6d9da4298b23fee4f8e553359f7e2aba91be4f0f7ad

data/lib/features/support/internal_hooks.rb

@@ -100,13 +100,16 @@ end
 
 # Before each scenario
 Before do |scenario|
+  $logger.debug "Before hook - scenario.status: #{scenario.status}"
   next if scenario.status == :skipped
 
   Maze.scenario = Maze::Api::Cucumber::Scenario.new(scenario)
 
   # Skip scenario if the driver it needs has failed
+  $logger.debug "Before hook - Maze.driver&.failed?: #{Maze.driver&.failed?}"
   if (Maze.mode == :appium || Maze.mode == :browser) && Maze.driver.failed?
-    skip_this_scenario
+    $logger.debug "Failing scenario because the #{Maze.mode.to_s} driver failed: #{Maze.driver.failure_reason}"
+    scenario.fail('Cannot run scenario - driver failed')
   end
 
   # Default to no dynamic retry
@@ -134,6 +137,7 @@ end
 
 # General processing to be run after each scenario
 After do |scenario|
+  $logger.debug "After hook 1 - scenario.status: #{scenario.status}"
   next if scenario.status == :skipped
 
   # If we're running on macos, take a screenshot if the scenario fails
@@ -227,22 +231,32 @@ end
 #
 # Furthermore, this hook should appear after the general hook as they are executed in reverse order by Cucumber.
 After do |scenario|
+  $logger.debug "After hook 2 - scenario.status: #{scenario.status}"
   next if scenario.status == :skipped
 
   # Call any pre_complete hooks registered by the client
   Maze.hooks.call_pre_complete scenario
 
+  # Fail the scenario if there are any invalid requests
   unless Maze::Server.invalid_requests.size_all == 0
     msg = "#{Maze::Server.invalid_requests.size_all} invalid request(s) received during scenario"
     Maze.scenario.mark_as_failed msg
   end
 
+  # Fail the scenario if the driver failed, if the scenario hasn't already failed
+  $logger.debug "After hook 2 - Maze.driver&.failed?: #{Maze.driver&.failed?}"
+  if (Maze.mode == :appium || Maze.mode == :browser) && Maze.driver.failed? && !scenario.failed?
+    $logger.debug "Marking scenario as failed because driver failed: #{Maze.driver.failure_reason}"
+    Maze.scenario.mark_as_failed Maze.driver.failure_reason
+  end
+
   Maze.scenario.complete
 end
 
 # Test all requests against schemas or extra validation rules.  These will only run if the schema/validation is
 # specified for the specific endpoint
 After do |scenario|
+  $logger.debug "After hook 3 - scenario.status: #{scenario.status}"
   next if scenario.status == :skipped
 
   ['error', 'session', 'build', 'trace'].each do |endpoint|

data/lib/maze/api/appium/app_manager.rb

@@ -18,14 +18,17 @@ module Maze
           @driver.activate_app(@driver.app_id)
           true
         rescue Selenium::WebDriver::Error::ServerError, Selenium::WebDriver::Error::UnknownError => e
+          $logger.error "Failed to activate app: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
           fail_driver(e.message)
           raise e
         end
 
-        # Terminates the app
+        # Terminates the app.  If terminate fails then clients may wish to ty the legacy close method, so an option
+        # is provided to not fail the Appium driver.
+        # @fail_driver [Boolean] Whether to fail the Appium driver if the app cannot be terminated
         # @returns [Boolean] Whether the app was successfully closed
-        def terminate
+        def terminate(fail_driver = true)
           if failed_driver?
             $logger.error 'Cannot terminate the app - Appium driver failed.'
             return false
@@ -34,8 +37,9 @@ module Maze
           @driver.terminate_app(@driver.app_id)
           true
         rescue Selenium::WebDriver::Error::ServerError, Selenium::WebDriver::Error::UnknownError => e
+          $logger.error "Failed to terminate app: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
-          fail_driver(e.message)
+          fail_driver(e.message) if fail_driver
           raise e
         end
 
@@ -50,6 +54,7 @@ module Maze
           @driver.launch_app
           true
         rescue Selenium::WebDriver::Error::ServerError, Selenium::WebDriver::Error::UnknownError => e
+          $logger.error "Failed to launch app: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
           fail_driver(e.message)
           raise e
@@ -66,6 +71,7 @@ module Maze
           @driver.close_app
           true
         rescue Selenium::WebDriver::Error::ServerError, Selenium::WebDriver::Error::UnknownError => e
+          $logger.error "Failed to close app: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
           fail_driver(e.message)
           raise e
@@ -81,6 +87,7 @@ module Maze
 
           @driver.app_state(@driver.app_id)
         rescue Selenium::WebDriver::Error::ServerError, Selenium::WebDriver::Error::UnknownError => e
+          $logger.error "Failed to get app state: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
           fail_driver(e.message)
           raise e

data/lib/maze/api/appium/device_manager.rb

@@ -19,6 +19,7 @@ module Maze
           @driver.unlock
           true
         rescue Selenium::WebDriver::Error::ServerError => e
+          $logger.error "Failed to unlock the device: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
           fail_driver(e.message)
           raise e
@@ -35,6 +36,7 @@ module Maze
           @driver.back
           true
         rescue Selenium::WebDriver::Error::ServerError => e
+          $logger.error "Failed to press back: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
           fail_driver(e.message)
           raise e
@@ -52,6 +54,7 @@ module Maze
 
           @driver.get_log(log_type)
         rescue Selenium::WebDriver::Error::ServerError => e
+          $logger.error "Failed to get logs: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
           fail_driver(e.message)
           raise e
@@ -69,6 +72,7 @@ module Maze
           @driver.set_rotation(orientation)
           true
         rescue Selenium::WebDriver::Error::ServerError => e
+          $logger.error "Failed to set the device rotation: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
           fail_driver(e.message)
           raise e
@@ -84,6 +88,7 @@ module Maze
 
           JSON.generate(@driver.device_info)
         rescue Selenium::WebDriver::Error::ServerError => e
+          $logger.error "Failed to get the device info: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
           fail_driver(e.message)
           raise e

data/lib/maze/api/appium/file_manager.rb

@@ -35,6 +35,7 @@ module Maze
           $logger.error "Error writing file to device: #{e.message}"
           false
         rescue Selenium::WebDriver::Error::ServerError => e
+          $logger.error "Error writing file to device: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
           fail_driver(e.message)
           raise e
@@ -72,6 +73,7 @@ module Maze
           $logger.error "Error reading file from device: #{e.message}"
           nil
         rescue Selenium::WebDriver::Error::ServerError => e
+          $logger.error "Error reading file from device: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
           fail_driver(e.message)
           raise e

data/lib/maze/api/appium/ui_manager.rb

@@ -22,6 +22,7 @@ module Maze
 
           @driver.wait_for_element(element_id, timeout, retry_if_stale)
         rescue Selenium::WebDriver::Error::ServerError => e
+          $logger.error "Error waiting for element #{element_id}: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
           fail_driver(e.message)
           raise e
@@ -41,6 +42,7 @@ module Maze
           @driver.click_element(element_id)
           true
         rescue Selenium::WebDriver::Error::ServerError => e
+          $logger.error "Error clicking element #{element_id}: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
           fail_driver(e.message)
           raise e
@@ -59,6 +61,7 @@ module Maze
 
           @driver.click_element_if_present(element_id)
         rescue Selenium::WebDriver::Error::ServerError => e
+          $logger.error "Error clicking element #{element_id}: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
           fail_driver(e.message)
           raise e

data/lib/maze/hooks/appium_hooks.rb

@@ -24,23 +24,27 @@ module Maze
       end
 
       def after(scenario)
+        $logger.debug "Appium after hook"
+
         manager = Maze::Api::Appium::AppManager.new
         if Maze.config.os == 'macos'
-          # Close the app - without the sleep, launching the app for the next scenario intermittently fails
+          # Close the app - without the sleep launching the app for the next scenario intermittently fails
           system("killall -KILL #{Maze.config.app} && sleep 1")
         elsif [:bb, :bs, :local].include? Maze.config.farm
-          # Reset the server to ensure that test fixtures cannot fetch
-          # commands from the previous scenario (in idempotent mode).
+          close_fallback = Maze.config.appium_version && Maze.config.appium_version.to_f < 2.0
           begin
-            manager.terminate
+            manager.terminate(!close_fallback)
           rescue Selenium::WebDriver::Error::UnknownError, Selenium::WebDriver::Error::InvalidSessionIdError
-            if Maze.config.appium_version && Maze.config.appium_version.to_f < 2.0
+            if close_fallback
               $logger.warn 'terminate_app failed, using the slower but more forceful close_app instead'
               manager.close
             else
-              $logger.warn 'terminate_app failed, future errors may occur if the application did not close remotely'
+              $logger.warn 'terminate_app failed, future errors may occur if the application did not close properly'
             end
           end
+
+          # Reset the server before relaunching the app to ensure that test fixtures cannot fetch
+          # commands from the previous scenario (in idempotent mode).
           Maze::Server.reset!
           manager.activate
         end

data/lib/maze.rb

@@ -8,7 +8,7 @@ require_relative 'maze/timers'
 # providing an alternative to the proliferation of global variables or singletons.
 module Maze
 
-  VERSION = '9.27.2'
+  VERSION = '9.27.4'
 
   class << self
     attr_accessor :check, :driver, :internal_hooks, :mode, :start_time, :dynamic_retry, :public_address,

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bugsnag-maze-runner
 version: !ruby/object:Gem::Version
-  version: 9.27.2
+  version: 9.27.4
 platform: ruby
 authors:
 - Steve Kirkland
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-04-08 00:00:00.000000000 Z
+date: 2025-04-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: cucumber