bugsnag-maze-runner 9.27.3 → 9.28.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0311fd0e499a676775d0b02bc1d5a9a73fba218845a531b60e4c2848bfd2f138
-  data.tar.gz: b38995cabfda41bc015b509ca10185198be47ad65210c7bcb24748c58278679d
+  metadata.gz: 0cd6f7e392e3c048cf4775a2715f3d87f692bc0a34234ff31a887dbf45c9e01d
+  data.tar.gz: af10924777104f0532f7294f49350244280938fb022ecea647f8756902c3be43
 SHA512:
-  metadata.gz: 48a00d3048487f3382bd4b7daeda0a3b27e3cce03037f1bd595d7e1217bdcf21428bd45af8c6d7097f858f655f434d8d8c1d740fb8ca66b78af7dbf88a4e9f9f
-  data.tar.gz: 40b2f8ba974a603f680a99baa2dced5b3645641e7d6f6eae69188bc5fd3052543c9dffca252bd44fab76bcac7832b6e35144eb116fc43e0d5fe93289584804a3
+  metadata.gz: 95163d0e29cba6dc81ae339604626396233c4d7aeb2cf779f8e1f91e9877eb09c9e3d722d90822ec356b6cef28e6df0175b575a05bde7665820cfdfc5bea63e9
+  data.tar.gz: c61f7b8579b2e918d402b0736ab872c56cff6843df8e87386a4f1bfec849f2a83dd4dea900eb005dc518a2af0fecadf33717c0d79f2a064f7bee0be30e372a1f

data/lib/features/steps/app_automator_steps.rb

@@ -38,7 +38,7 @@ end
 # Sends the app to the background indefinitely
 # Requires a running Appium driver
 When('I send the app to the background') do
-  Maze.driver.background_app(-1)
+  Maze::Api::Appium::AppManager.new.background(-1)
 end
 
 # Sends the app to the background for a number of seconds
@@ -46,7 +46,7 @@ end
 #
 # @step_input timeout [Integer] The amount of time the app is in the background in seconds
 When('I send the app to the background for {int} second(s)') do |timeout|
-  Maze.driver.background_app(timeout)
+  Maze::Api::Appium::AppManager.new.background(timeout)
 end
 
 # Clears a given element

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,6 +231,7 @@ 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
@@ -238,8 +243,10 @@ After do |scenario|
     Maze.scenario.mark_as_failed msg
   end
 
-  # Fail the scenario if the Appium driver failed
-  if Maze.mode == :appium && Maze.driver.failed?
+  # 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
 
@@ -249,6 +256,7 @@ 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

@@ -26,7 +26,7 @@ module Maze
 
         # 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
+        # @param 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(fail_driver = true)
           if failed_driver?
@@ -43,6 +43,24 @@ module Maze
           raise e
         end
 
+        # Instructs Appium to background the app
+        # @param seconds [Integers] The number of seconds to background the app for, or -1 for indefinitely
+        # @returns [Boolean] Whether the instruction to Appium was successfully made
+        def background(seconds = -1)
+          if failed_driver?
+            $logger.error 'Cannot background the app - Appium driver failed.'
+            return false
+          end
+
+          @driver.background_app(seconds)
+          true
+        rescue Selenium::WebDriver::Error::ServerError, Selenium::WebDriver::Error::UnknownError => e
+          $logger.error "Failed to background app: #{e.message}"
+          # Assume the remote appium session has stopped, so crash out of the session
+          fail_driver(e.message)
+          raise e
+        end
+
         # Launches the app (legacy method).
         # @returns [Boolean] Whether the app was successfully launched
         def launch

data/lib/maze/client/appium/bb_client.rb

@@ -38,6 +38,8 @@ module Maze
             interval = 10
           elsif error.message.include? '\'platformVersion\' must be a valid version number.'
             interval = 10
+          elsif error.message.include?('Device model with name') && error.message.include?('is currently unavailable')
+            interval = 10
           else
             # Do not retry in any other case
           end

data/lib/maze/hooks/appium_hooks.rb

@@ -24,6 +24,8 @@ 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
@@ -37,7 +39,7 @@ module Maze
               $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
 

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.3'
+  VERSION = '9.28.0'
 
   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.3
+  version: 9.28.0
 platform: ruby
 authors:
 - Steve Kirkland
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-04-10 00:00:00.000000000 Z
+date: 2025-04-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: cucumber