Merge branch 'main' into integration/v10

Author: Steve Kirkland

CHANGELOG.md

@@ -6,6 +6,16 @@
 - Remove deprecated Cucumber steps for sending keys to elements [751](https://github.com/bugsnag/maze-runner/pull/751)
 - Remove document server on port 9340 [752](https://github.com/bugsnag/maze-runner/pull/752)
 
+# 9.31.0 - 2025/05/15
+
+## Enhancements
+
+- Add step `I discard all {request_type}`' [756](https://github.com/bugsnag/maze-runner/pull/756)
+
+## Fixes
+
+- Notify errors that fail the Appium driver [757](https://github.com/bugsnag/maze-runner/pull/757)
+
 # 9.30.3 - 2025/05/14
 
 ## Fixes

bin/maze-runner

@@ -93,7 +93,6 @@ require_relative '../lib/maze/hooks/logger_hooks'
 require_relative '../lib/maze/driver/appium'
 require_relative '../lib/maze/driver/browser'
 
-require_relative '../lib/maze/plugins/bugsnag_reporting_plugin'
 require_relative '../lib/maze/plugins/error_code_plugin'
 require_relative '../lib/maze/plugins/global_retry_plugin'
 require_relative '../lib/maze/plugins/datadog_metrics_plugin'

lib/features/steps/request_assertion_steps.rb

@@ -131,6 +131,15 @@ def assert_received_requests(request_count, list, list_name, precise = true, max
   Maze::Server.list_for(request_type).next
 end
 
+# Moves to the end of the request list
+#
+# @step_input request_type [String] The type of request (error, session, build, etc)
+Then('I discard all {request_type}') do |request_type|
+  raise "No #{request_type} to discard" if Maze::Server.list_for(request_type).current.nil?
+
+  Maze::Server.list_for(request_type).end
+end
+
 Then('the received errors match:') do |table|
   # Checks that each request matches one of the event fields
   requests = Maze::Server.errors.remaining

lib/features/support/internal_hooks.rb

@@ -84,9 +84,6 @@
 
   # Add step logging
   config.filters << Maze::Plugins::LoggingScenariosPlugin.new(config)
-
-  # Add bugsnag failed scenario reporting only if ENV['MAZE_SCENARIO_BUGSNAG_API_KEY'] is present
-  config.filters << Maze::Plugins::BugsnagReportingPlugin.new(config) unless ENV['MAZE_SCENARIO_BUGSNAG_API_KEY'].nil?
 end
 
 # Before each scenario

lib/maze.rb

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

lib/maze/api/appium/app_manager.rb

@@ -20,7 +20,7 @@ def activate
         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)
+          fail_driver(e)
           raise e
         end
 
@@ -39,7 +39,7 @@ def terminate(fail_driver = 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) if fail_driver
+          fail_driver(e) if fail_driver
           raise e
         end
 
@@ -57,7 +57,7 @@ def background(seconds = -1)
         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)
+          fail_driver(e)
           raise e
         end
 
@@ -74,7 +74,7 @@ def launch
         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)
+          fail_driver(e)
           raise e
         end
 
@@ -91,7 +91,7 @@ def close
         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)
+          fail_driver(e)
           raise e
         end
 
@@ -107,7 +107,7 @@ def state
         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)
+          fail_driver(e)
           raise e
         end
       end

lib/maze/api/appium/device_manager.rb

@@ -21,7 +21,7 @@ def unlock
         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)
+          fail_driver(e)
           raise e
         end
 
@@ -38,7 +38,7 @@ def back
         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)
+          fail_driver(e)
           raise e
         end
 
@@ -56,7 +56,7 @@ def 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)
+          fail_driver(e)
           raise e
         end
 
@@ -74,7 +74,7 @@ def set_rotation(orientation)
         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)
+          fail_driver(e)
           raise e
         end
 
@@ -90,7 +90,7 @@ def 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)
+          fail_driver(e)
           raise e
         end
       end

lib/maze/api/appium/file_manager.rb

@@ -37,7 +37,7 @@ def write_app_file(contents, filename)
         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)
+          fail_driver(e)
           raise e
         end
 
@@ -75,7 +75,7 @@ def read_app_file(filename, directory = 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)
+          fail_driver(e)
           raise e
         end
 
@@ -105,7 +105,7 @@ def read_app_folder
         rescue Selenium::WebDriver::Error::ServerError => e
           $logger.error "Error reading folder from device: #{e.message}"
           # Assume the remote appium session has stopped, so crash out of the session
-          fail_driver(e.message)
+          fail_driver(e)
           raise e
         end
       end

lib/maze/api/appium/manager.rb

@@ -11,8 +11,9 @@ def failed_driver?
           @driver.failed?
         end
 
-        def fail_driver(message)
-          @driver.fail_driver(message)
+        def fail_driver(exception)
+          Bugsnag.notify(exception)
+          @driver.fail_driver(exception.message)
         end
       end
     end

lib/maze/api/appium/ui_manager.rb

@@ -24,7 +24,7 @@ def wait_for_element(element_id, timeout = 15, retry_if_stale = true)
         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)
+          fail_driver(e)
           raise e
         end
 
@@ -44,7 +44,7 @@ def click_element(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)
+          fail_driver(e)
           raise e
         end
 
@@ -63,7 +63,7 @@ def 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)
+          fail_driver(e)
           raise e
         end
       end