bugsnag-maze-runner 9.36.0 → 10.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fea2efe5108b520669c020cb35a9e03a4ab9bc7f88a6136ed866aab64641a4af
-  data.tar.gz: 6940e0f5213b8c62bc73bb2888dc81f7946fbbffcb06316b499946a03f7580b8
+  metadata.gz: d59bb98b603f22702c5da83fc40f70e12dcba5553b1177075dea770c97a154a3
+  data.tar.gz: d0e404e19355099925754b022ac30f00d56050beb8d1740e8c1ced2b32705d4f
 SHA512:
-  metadata.gz: 4243f48064e91757b1fb549bff897e0d9d9398076ac4e1fb0ac0512d391b3aa317038c042d679b80403337de4d06651d565ea2af564c045a5be146241e72efda
-  data.tar.gz: '0580471cf0ca9ab9de0e278806357c00d20adeacf4b5f577ae7b3fc6e4c1c8cc203bb0cccc9479d865757c1fe6f8cee34965d95d7ecb283842069cb7605d0afc'
+  metadata.gz: 11e455f30a38488c0169b7f21536c295ab007ed8dfae1afcce8fe34fb3461ee2e7d93b6db35fb746989ca28bd7ece7fa97ead621b0eb421764a4e03e6c9e2772
+  data.tar.gz: 970798d5594c687c269f9d3043fc70e86808d63098fdc9017a226bb71c08422dc6cf55634c025f877c52cd750c39ad80d5b625d8f5184e304f94883d034b75cf

data/bin/maze-runner

@@ -29,7 +29,6 @@ require_relative '../lib/maze/client/appium/base_client'
 require_relative '../lib/maze/client/appium/bb_client'
 require_relative '../lib/maze/client/appium/bb_devices'
 require_relative '../lib/maze/client/appium/bs_client'
-require_relative '../lib/maze/client/appium/bs_legacy_client'
 require_relative '../lib/maze/client/appium/bs_devices'
 require_relative '../lib/maze/client/appium/local_client'
 require_relative '../lib/maze/client/appium/session_metadata'
@@ -42,7 +41,6 @@ require_relative '../lib/maze/client/selenium/local_client'
 require_relative '../lib/maze/aws_public_ip'
 require_relative '../lib/maze/compare'
 require_relative '../lib/maze/docker'
-require_relative '../lib/maze/document_server'
 require_relative '../lib/maze/errors'
 require_relative '../lib/maze/generator'
 require_relative '../lib/maze/helper'

data/lib/features/steps/app_automator_steps.rb

@@ -49,38 +49,9 @@ When('I send the app to the background for {int} second(s)') do |timeout|
   Maze::Api::Appium::AppManager.new.background(timeout)
 end
 
-# Clears a given element
-# Requires a running Appium driver
-#
-# @step_input element_id [String] The locator id
-# @deprecated To be removed in the next major version
-When('I clear the element {string}') do |element_id|
-  Maze.driver.clear_element(element_id)
-end
-
-# Sends keys to a given element
-# Requires a running Appium driver
-#
-# @step_input keys [String] The keys to send to the element
-# @step_input element_id [String] The locator id
-# @deprecated To be removed in the next major version
-When('I send the keys {string} to the element {string}') do |keys, element_id|
-  Maze.driver.send_keys_to_element(element_id, keys)
-end
-
 # Set the device orientation to either portrait or landscape
 # Requires a running Appium driver
 When('I set the device orientation to {orientation}') do |orientation|
   Maze::Api::Appium::DeviceManager.new.set_rotation orientation
 end
 
-# Sends keys to a given element, clearing it first
-# Requires a running Appium driver
-#
-# @step_input keys [String] The keys to send to the element
-# @step_input element_id [String] The locator id
-# @deprecated To be removed in the next major version
-When('I clear and send the keys {string} to the element {string}') do |keys, element_id|
-  Maze.driver.clear_and_send_keys_to_element(element_id, keys)
-end
-

data/lib/features/steps/error_reporting_steps.rb

@@ -136,14 +136,6 @@ Then('the event {string} equals {float} to {int} decimal place(s)') do |field, f
   step "the error payload field \"events.0.#{field}\" equals #{float_value} to #{places} decimal places"
 end
 
-# Tests whether a value in the first event entry equals an integer.
-#
-# @step_input field [String] The relative location of the value to test
-# @step_input value [Integer] The integer to test against
-Then('the event {string} equals {int}') do |field, value|
-  step "the error payload field \"events.0.#{field}\" equals #{value}"
-end
-
 # Tests whether a value in the first event entry starts with a string.
 #
 # @step_input field [String] The relative location of the value to test

data/lib/features/steps/payload_steps.rb

@@ -76,16 +76,6 @@ Then('the {request_type} payload field {string} is not null') do |request_type,
   Maze.check.not_nil(Maze::Helper.read_key_path(list.current[:body], field_path))
 end
 
-# Tests that a payload element equals an integer.
-#
-# @step_input request_type [String] The type of request (error, session, build, etc)
-# @step_input field_path [String] Path to the tested element
-# @step_input int_value [Integer] The value to test against
-Then('the {request_type} payload field {string} equals {int}') do |request_type, field_path, int_value|
-  Maze.check.equal(int_value,
-                   Maze::Helper.read_key_path(Maze::Server.list_for(request_type).current[:body], field_path))
-end
-
 # Tests that a payload element equals a float.
 #
 # @step_input request_type [String] The type of request (error, session, build, etc)

data/lib/features/support/internal_hooks.rb

@@ -12,16 +12,16 @@ BeforeAll do
 
   # Infer mode of operation from config, one of:
   # - Appium (using either remote or local devices)
-  # - Browser (Selenium with local or remote browsers)
+  # - Selenium (Selenium with local or remote browsers)
   # - Command (the software under test is invoked with a system call)
-  # TODO Consider making this a specific command line option defaulting to Appium
-  is_appium = [:bs, :bb, :local].include?(Maze.config.farm) && !Maze.config.app.nil?
-  is_browser = !Maze.config.browser.nil?
+  is_appium = ([:bs, :bb, :local].include?(Maze.config.farm) && !Maze.config.app.nil?) ||
+    (Maze.config.farm == :bb && !Maze.config.device.nil? && !Maze.config.browser.nil?)
+  is_browser = Maze.config.device.nil? && !Maze.config.browser.nil?
   if is_appium
     Maze.mode = :appium
     Maze.internal_hooks = Maze::Hooks::AppiumHooks.new
   elsif is_browser
-    Maze.mode = :browser
+    Maze.mode = :selenium
     Maze.internal_hooks = Maze::Hooks::BrowserHooks.new
   else
     Maze.mode = :command
@@ -57,20 +57,11 @@ BeforeAll do
   # Call any blocks registered by the client
   Maze.hooks.call_before_all
 
-  # Start document server, if asked for
-  # This must happen after any client hooks have run, so that they can set the server root
-  Maze::DocumentServer.start unless Maze.config.document_server_root.nil?
-
   # Determine public IP if enabled
   if Maze.config.aws_public_ip
     public_ip = Maze::AwsPublicIp.new
     Maze.public_address = public_ip.address
     $logger.info "Public address: #{Maze.public_address}"
-
-    unless Maze.config.document_server_root.nil?
-      Maze.public_document_server_address = public_ip.document_server_address
-      $logger.info "Public document server address: #{Maze.public_document_server_address}"
-    end
   end
 
   # An initial setup for total success status
@@ -102,7 +93,7 @@ Before do |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?
+  if (Maze.mode == :appium || Maze.mode == :selenium) && Maze.driver.failed?
     $logger.debug "Failing scenario because the #{Maze.mode.to_s} driver failed: #{Maze.driver.failure_reason}"
     scenario.fail('Cannot run scenario - driver failed')
   end
@@ -146,9 +137,6 @@ After do |scenario|
   # Call any blocks registered by the client
   Maze.hooks.call_after scenario
 
-  # Stop document server if started by the Cucumber step
-  Maze::DocumentServer.manual_stop
-
   # Stop terminating server if started by the Cucumber step
   Maze::TerminatingServer.stop
 
@@ -253,7 +241,7 @@ After do |scenario|
 
   # 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?
+  if (Maze.mode == :appium || Maze.mode == :selenium) && 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

data/lib/maze/aws_public_ip.rb

@@ -5,29 +5,17 @@ module Maze
   class AwsPublicIp
     attr_reader :host
     attr_reader :port
-    attr_reader :document_server_port
 
     def address
       "#{@ip}:#{@port}"
     end
 
-    def document_server_address
-      return nil if @document_server_port.nil?
-
-      "#{@ip}:#{@document_server_port}"
-    end
-
     def initialize
       # This class is only relevant on Buildkite
       return unless ENV['BUILDKITE']
 
       @ip = determine_public_ip
       @port = determine_public_port Maze.config.port
-
-      unless Maze.config.document_server_root.nil?
-        @document_server_port = determine_public_port Maze.config.document_server_port
-      end
-
     end
 
     # Determines the public IP address of the running AWS instance

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

@@ -29,21 +29,20 @@ module Maze
           start_driver(Maze.config)
 
           # Set bundle/app id for later use
-          Maze.driver.app_id = case Maze::Helper.get_current_platform
-                               when 'android'
-                                 Maze.driver.session_capabilities['appPackage']
-                               when 'ios'
-                                 unless app_id = Maze.driver.session_capabilities['CFBundleIdentifier']
-                                    app_id = Maze.driver.session_capabilities['bundleId']
+          unless Maze.config.app.nil?
+            Maze.driver.app_id = case Maze::Helper.get_current_platform
+                                 when 'android'
+                                   Maze.driver.session_capabilities['appPackage']
+                                 when 'ios'
+                                   unless app_id = Maze.driver.session_capabilities['CFBundleIdentifier']
+                                      app_id = Maze.driver.session_capabilities['bundleId']
+                                   end
+                                   app_id
                                  end
-                                 app_id
-                               end
-
-          if Maze.driver.app_id.nil?
-            $logger.error "Failed to determine app id."
-            $logger.debug "session_capabilities: #{Maze.driver.session_capabilities.inspect}"
           end
 
+          $logger.error "Failed to determine app id." if Maze.driver.app_id.nil?
+
           # Log the device information after it's started
           write_device_info
 

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

@@ -5,7 +5,7 @@ module Maze
         def prepare_session
           config = Maze.config
           config.app = Maze::Client::BitBarClientUtils.upload_app config.access_key,
-                                                                  config.app
+                                                                  config.app if config.app
           if Maze::Client::BitBarClientUtils.use_local_tunnel?
             Maze::Client::BitBarClientUtils.start_local_tunnel config.sb_local,
                                                                config.username,
@@ -51,7 +51,7 @@ module Maze
         end
 
         def start_scenario
-          unless Maze.config.legacy_driver?
+          unless Maze.config.browser
             # Write Maze's address to file and push to the device
             maze_address = Maze.public_address || "local:#{Maze.config.port}"
             Maze::Api::Appium::FileManager.new.write_app_file(JSON.generate({ maze_address: maze_address }),
@@ -83,6 +83,10 @@ module Maze
           capabilities.deep_merge! BitBarClientUtils.dashboard_capabilities
           capabilities.deep_merge! BitBarDevices.get_available_device(config.device)
           capabilities['bitbar:options']['appiumVersion'] = config.appium_version unless config.appium_version.nil?
+          unless config.browser.nil?
+            capabilities['bitbar:options']['acceptInsecureCerts'] = true
+            capabilities['browserName'] = config.browser
+          end
           capabilities.deep_merge! JSON.parse(config.capabilities_option)
 
           capabilities

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

@@ -31,12 +31,10 @@ module Maze
         end
 
         def start_scenario
-          unless Maze.config.legacy_driver?
-            # Write Maze's address to file and push to the device
-            maze_address = "bs-local.com:#{Maze.config.port}"
-            Maze::Api::Appium::FileManager.new.write_app_file(JSON.generate({ maze_address: maze_address }),
-                                                              FIXTURE_CONFIG)
-          end
+          # Write Maze's address to file and push to the device
+          maze_address = "bs-local.com:#{Maze.config.port}"
+          Maze::Api::Appium::FileManager.new.write_app_file(JSON.generate({ maze_address: maze_address }),
+                                                            FIXTURE_CONFIG)
 
           super
         end

data/lib/maze/client/appium.rb

@@ -5,14 +5,7 @@ module Maze
         client_class =
           case Maze.config.farm
           when :bb then BitBarClient
-          when :bs
-            if Maze.config.legacy_driver?
-              $logger.info 'Using the Legacy (JWP) Appium client'
-              BrowserStackLegacyClient
-            else
-              $logger.info 'Using the W3C Appium client'
-              BrowserStackClient
-            end
+          when :bs then BrowserStackClient
           when :local then LocalClient
           end
 

data/lib/maze/client/selenium/bs_client.rb

@@ -23,45 +23,23 @@ module Maze
         end
 
         def create_capabilities(config)
-          if config.legacy_driver?
-            capabilities = ::Selenium::WebDriver::Remote::Capabilities.new
-            capabilities['browserstack.local'] = 'true'
-            capabilities['browserstack.localIdentifier'] = Maze.run_uuid
-            capabilities['browserstack.console'] = 'errors'
-            capabilities['acceptInsecureCerts'] = 'true'
-            capabilities.merge! JSON.parse(config.capabilities_option)
-            capabilities.merge! project_name_capabilities
-            add_browser_capabilities(config, capabilities)
-          else
-            raw_capabilities = {
-              'acceptInsecureCerts' => true,
-              'bstack:options' => {
-                'local' => 'true',
-                'localIdentifier' => Maze.run_uuid
-              }
+          raw_capabilities = {
+            'acceptInsecureCerts' => true,
+            'bstack:options' => {
+              'local' => 'true',
+              'localIdentifier' => Maze.run_uuid
             }
-            raw_capabilities.deep_merge! JSON.parse(config.capabilities_option)
-            raw_capabilities.merge! project_name_capabilities
-            add_browser_capabilities(config, raw_capabilities)
-            capabilities = ::Selenium::WebDriver::Remote::Capabilities.new raw_capabilities
-          end
+          }
+          raw_capabilities.deep_merge! JSON.parse(config.capabilities_option)
+          raw_capabilities.merge! project_name_capabilities
+          add_browser_capabilities(config, raw_capabilities)
+          capabilities = ::Selenium::WebDriver::Remote::Capabilities.new raw_capabilities
           config.capabilities = capabilities
         end
 
         def add_browser_capabilities(config, capabilities)
           browsers = YAML.safe_load(File.read("#{__dir__}/bs_browsers.yml"))
-          browser = browsers[config.browser]
-
-          if config.legacy_driver?
-            # Convert W3S capabilities to JSON-WP
-            capabilities['browser'] = browser['browserName']
-            capabilities['browser_version'] = browser['browserVersion']
-            capabilities['device'] = browser['device']
-            capabilities['os'] = browser['os']
-            capabilities['os_version'] = browser['osVersion']
-          else
-            capabilities.deep_merge! browser
-          end
+          capabilities.deep_merge! browsers[config.browser]
         end
 
         def stop_session

data/lib/maze/configuration.rb

@@ -15,7 +15,6 @@ module Maze
       self.span_timestamp_validation = true
       self.unmanaged_traces_mode = false
       self.client_mode_validation = true
-      @legacy_driver = false
     end
 
     #
@@ -41,12 +40,6 @@ module Maze
     # Document server root
     attr_accessor :document_server_root
 
-    # Document server bind address
-    attr_accessor :document_server_bind_address
-
-    # Document server port
-    attr_accessor :document_server_port
-
     #
     # Common configuration
     #
@@ -194,15 +187,6 @@ module Maze
     # URL of the Selenium server
     attr_accessor :selenium_server_url
 
-    # Whether the legacy (JSON-WP) Appium driver should be used
-    def legacy_driver?
-      @legacy_driver
-    end
-
-    def legacy_driver=(value)
-      @legacy_driver = value
-    end
-
     #
     # Local testing specific configuration
     #

data/lib/maze/driver/appium.rb

@@ -40,8 +40,6 @@ module Maze
         # Timers
         @find_element_timer = Maze.timers.add 'Appium - find element'
         @click_element_timer = Maze.timers.add 'Appium - click element'
-        @clear_element_timer = Maze.timers.add 'Appium - clear element'
-        @send_keys_timer = Maze.timers.add 'Appium - send keys to element'
 
         super({
           'caps' => @capabilities,
@@ -82,6 +80,33 @@ module Maze
         @failure_reason = reason
       end
 
+      def javascript?
+        return false if Maze.config.browser.nil?
+        @driver.execute_script('return true')
+      rescue Selenium::WebDriver::Error::UnsupportedOperationError
+        false
+      end
+
+
+      # Provided for mobile browsers - check if the browser supports local storage
+      def local_storage?
+        # Assume we can use local storage if we aren't able to verify by running JavaScript
+        return true unless javascript?
+
+        result = @driver.execute_script <<-JAVASCRIPT
+      try {
+        window.localStorage.setItem('__localstorage_test__', 1234)
+        window.localStorage.removeItem('__localstorage_test__')
+        return true
+      } catch (err) {
+        return err
+      }
+        JAVASCRIPT
+
+        result == true
+      end
+
+
       # Checks for an element, waiting until it is present or the method times out
       #
       # @param element_id [String] the element to search for
@@ -168,20 +193,6 @@ module Maze
         raise e
       end
 
-      # Clears a given element
-      #
-      # @param element_id [String] the element to clear
-      def clear_element(element_id)
-        element = find_element_timed(element_id)
-        @clear_element_timer.time do
-          element.clear
-        end
-      rescue Selenium::WebDriver::Error::ServerError => e
-        # Assume the remote appium session has stopped, so crash out of the session
-        fail_driver(e)
-        raise e
-      end
-
       # Gets the application hierarchy XML
       def page_source
         @driver.page_source
@@ -192,21 +203,6 @@ module Maze
         @driver.unlock
       end
 
-      # Sends keys to a given element
-      #
-      # @param element_id [String] the element to send text to
-      # @param text [String] the text to send
-      def send_keys_to_element(element_id, text)
-        element = find_element_timed(element_id)
-        @send_keys_timer.time do
-          element.send_keys(text)
-        end
-      rescue Selenium::WebDriver::Error::ServerError => e
-        # Assume the remote appium session has stopped, so crash out of the session
-        fail_driver(e)
-        raise e
-      end
-
       # Sets the rotation of the device
       #
       # @param orientation [Symbol] :portrait or :landscape
@@ -218,40 +214,6 @@ module Maze
         @driver.window_size
       end
 
-      # Send keys to the device without a specific element
-      #
-      # @param text [String] the text to send
-      def send_keys(text)
-        @driver.send_keys(text)
-      end
-
-      # Sends keys to a given element, clearing it first
-      #
-      # @param element_id [String] the element to clear and send text to
-      # @param text [String] the text to send
-      def clear_and_send_keys_to_element(element_id, text)
-        element = find_element_timed(element_id)
-        @clear_element_timer.time do
-          element.clear
-        end
-
-        @send_keys_timer.time do
-          element.send_keys(text)
-        end
-      rescue Selenium::WebDriver::Error::ServerError => e
-        # Assume the remote appium session has stopped, so crash out of the session
-        fail_driver(e)
-        raise e
-      end
-
-      # Reset the currently running application after a given timeout
-      #
-      # @param timeout [Number] the amount of time in seconds to wait before resetting
-      def reset_with_timeout(timeout = 0.1)
-        sleep(timeout)
-        reset
-      end
-
       def device_info
         driver.execute_script('mobile:deviceInfo')
       end

data/lib/maze/driver/browser.rb

@@ -58,7 +58,7 @@ module Maze
 
       # check if Selenium supports running javascript in the current browser
       def javascript?
-        @driver.execute_script('return true')
+        !Maze.config.browser.nil? && @driver.execute_script('return true')
       rescue Selenium::WebDriver::Error::UnsupportedOperationError
         false
       end
@@ -128,15 +128,9 @@ module Maze
           $logger.info "Starting Selenium driver"
           time = Time.now
           if driver_for == :remote
-            if Maze.config.legacy_driver?
-              driver = ::Selenium::WebDriver.for :remote,
-                                                 url: selenium_url,
-                                                 desired_capabilities: @capabilities
-            else
-              driver = ::Selenium::WebDriver.for :remote,
-                                                 url: selenium_url,
-                                                 capabilities: @capabilities
-            end
+            driver = ::Selenium::WebDriver.for :remote,
+                                               url: selenium_url,
+                                               capabilities: @capabilities
           else
             if driver_for == :chrome
               options = Selenium::WebDriver::Options.chrome

data/lib/maze/helper.rb

@@ -89,7 +89,7 @@ module Maze
       # Returns the current platform all lower-case.
       # @return Typically 'ios', 'android', 'mac' or 'browser'
       def get_current_platform
-        if Maze.mode == :browser
+        if Maze.mode == :selenium || (Maze.mode == :appium && Maze.config.browser)
           os = 'browser'
         else
           os = case Maze.config.farm

data/lib/maze/hooks/appium_hooks.rb

@@ -29,7 +29,7 @@ module Maze
         if Maze.config.os == 'macos'
           # 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
+        elsif [:bb, :bs, :local].include?(Maze.config.farm) && Maze.config.browser.nil?
           close_fallback = Maze.config.appium_version && Maze.config.appium_version.to_f < 2.0
           begin
             manager.terminate(!close_fallback)
@@ -45,7 +45,7 @@ module Maze
           # 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
+          manager.activate if Maze.config.browser.nil?
         end
       rescue => error
         # Notify and re-raise for Cucumber to handle

data/lib/maze/option/parser.rb

@@ -85,14 +85,6 @@ module Maze
                 'Document server root',
                 short: :none,
                 type: :string
-            opt Option::DS_BIND_ADDRESS,
-                'Document server bind address',
-                short: :none,
-                type: :string
-            opt Option::DS_PORT,
-                'Document server port',
-                short: :none,
-                default: 9340
 
             text ''
             text 'Appium options:'

data/lib/maze/option/processor.rb

@@ -29,8 +29,6 @@ module Maze
 
           # Document server options
           config.document_server_root = options[Maze::Option::DS_ROOT]
-          config.document_server_bind_address = options[Maze::Option::DS_BIND_ADDRESS]
-          config.document_server_port = options[Maze::Option::DS_PORT]
 
           # Logger options
           config.file_log = options[Maze::Option::FILE_LOG]
@@ -52,7 +50,6 @@ module Maze
           config.capabilities_option = options[Maze::Option::CAPABILITIES]
           config.app_activity = options[Maze::Option::APP_ACTIVITY]
           config.app_package = options[Maze::Option::APP_PACKAGE]
-          config.legacy_driver = !ENV['USE_LEGACY_DRIVER'].nil?
           config.start_tunnel = options[Maze::Option::TUNNEL]
 
           # Farm specific options
@@ -63,11 +60,7 @@ module Maze
             if !device_option.empty?
               config.device = device_option.first
               config.device_list = device_option.drop(1)
-              if config.legacy_driver?
-                config.os_version = Maze::Client::Appium::BrowserStackDevices::DEVICE_HASH[config.device]['os_version'].to_f
-              else
-                config.os_version = Maze::Client::Appium::BrowserStackDevices::DEVICE_HASH[config.device]['platformVersion'].to_f
-              end
+              config.os_version = Maze::Client::Appium::BrowserStackDevices::DEVICE_HASH[config.device]['platformVersion'].to_f
             elsif !browser_option.empty?
               config.browser = browser_option.first
               config.browser_list = browser_option.drop(1)
@@ -84,7 +77,12 @@ module Maze
             device_option = options[Maze::Option::DEVICE]
             browser_option = options[Maze::Option::BROWSER]
             browser_version = options[Maze::Option::BROWSER_VERSION]
-            if !device_option.empty?
+            if !device_option.empty? && !browser_option.empty?
+              config.device = device_option.first
+              config.device_list = device_option.drop(1)
+              config.browser = browser_option.first
+              config.browser_list = browser_option.drop(1)
+            elsif !device_option.empty?
               config.device = device_option.first
               config.device_list = device_option.drop(1)
             elsif !browser_option.empty?

data/lib/maze/option/validator.rb

@@ -113,7 +113,8 @@ module Maze
         # Device
         if browser.empty? && device.empty?
           errors << "Either --#{Option::BROWSER} or --#{Option::DEVICE} must be specified"
-        elsif !browser.empty?
+        elsif !browser.empty? && device.empty?
+          # Desktop browsers (Selenium)
           browsers = YAML.safe_load(File.read("#{__dir__}/../client/selenium/bb_browsers.yml"))
 
           rejected_browsers = browser.reject { |br| browsers.include? br }
@@ -129,7 +130,8 @@ module Maze
             errors << "Browser types '#{rejected_browsers.join(', ')}' unknown on BitBar.  Must be one of: #{browser_list}."
           end
 
-        elsif !device.empty?
+        elsif browser.empty? && !device.empty?
+          # Mobile app testing
           app = Maze::Helper.read_at_arg_file options[Option::APP]
           if app.nil?
             errors << "--#{Option::APP} must be provided when running on a device"
@@ -140,6 +142,8 @@ module Maze
               errors << "app file '#{app}' not found" unless File.exist?(app)
             end
           end
+        else
+          # TODO - Mobile browser testing
         end
       end
 

data/lib/maze/option.rb

@@ -3,10 +3,6 @@
 # Provides the set of Maze Runner command line options
 module Maze
   module Option
-    # Document server options
-    DS_BIND_ADDRESS = 'document-server-bind-address'
-    DS_PORT = 'document-server-port'
-
     # Server options
     HTTPS = 'https'
     BIND_ADDRESS = 'bind-address'

data/lib/maze/plugins/global_retry_plugin.rb

@@ -23,7 +23,7 @@ module Maze
           configuration.options[:retry] = 0
 
           # Guard to check if the case should be retried
-          should_retry = event.result.failed? && Maze::RetryHandler.should_retry?(test_case, event)
+          should_retry = event.result.failed? && Maze::RetryHandler.should_retry?(test_case)
 
           next unless should_retry
 

data/lib/maze/retry_handler.rb

@@ -14,25 +14,11 @@ module Maze
       # Determines whether a failed test_case should be restarted
       #
       # @param test_case [Cucumber::RunningTestCase] The current test_case or scenario
-      # @param event [Cucumber::Core::Event] The triggering event
-      def should_retry?(test_case, event)
+      def should_retry?(test_case)
         # Only retry if the option is set and we haven't already retried
         return false if !Maze.config.enable_retries || retried_previously?(test_case)
 
-        if retry_on_driver_error?(event)
-          if Maze.mode == :appium
-            if Maze.config.farm.eql?(:bb)
-              Maze::Hooks::ErrorCodeHook.exit_code = Maze::Api::ExitCode::APPIUM_SESSION_FAILURE
-              return false
-            else
-              $logger.warn "Retrying #{test_case.name} due to appium driver error: #{event.result.exception}"
-              Maze.driver.restart
-            end
-          elsif Maze.mode == :browser
-            $logger.warn "Retrying #{test_case.name} due to selenium driver error: #{event.result.exception}"
-            Maze.driver.refresh
-          end
-        elsif retry_on_tag?(test_case)
+        if retry_on_tag?(test_case)
           $logger.warn "Retrying #{test_case.name} due to retry tag"
         elsif Maze.dynamic_retry
           $logger.warn "Retrying #{test_case.name} due to dynamic retry set"
@@ -53,12 +39,6 @@ module Maze
         global_retried[test_case] += 1
       end
 
-      def retry_on_driver_error?(event)
-        error_class = event.result.exception.class
-        maze_errors = Maze::Error::ERROR_CODES
-        Maze.driver && maze_errors.include?(error_class) && maze_errors[error_class][:retry]
-      end
-
       def retry_on_tag?(test_case)
         test_case.tags.any? do |tag|
           RETRY_TAGS.include?(tag.name)

data/lib/maze.rb

@@ -8,11 +8,11 @@ require_relative 'maze/timers'
 # providing an alternative to the proliferation of global variables or singletons.
 module Maze
 
-  VERSION = '9.36.0'
+  VERSION = '10.1.0'
 
   class << self
     attr_accessor :check, :driver, :internal_hooks, :mode, :start_time, :dynamic_retry, :public_address,
-                  :public_document_server_address, :run_uuid, :scenario
+                  :run_uuid, :scenario
 
     def config
       @config ||= Maze::Configuration.new

metadata

@@ -1,14 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: bugsnag-maze-runner
 version: !ruby/object:Gem::Version
-  version: 9.36.0
+  version: 10.1.0
 platform: ruby
 authors:
 - Steve Kirkland
+- Josh Edney
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-08-26 00:00:00.000000000 Z
+date: 2025-09-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: cucumber
@@ -16,14 +17,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '10.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '10.0'
 - !ruby/object:Gem::Dependency
   name: os
   requirement: !ruby/object:Gem::Requirement
@@ -72,14 +73,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.7.0
+        version: 1.9.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.7.0
+        version: 1.9.0
 - !ruby/object:Gem::Dependency
   name: appium_lib
   requirement: !ruby/object:Gem::Requirement
@@ -136,20 +137,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '6.24'
-- !ruby/object:Gem::Dependency
-  name: cucumber-expressions
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 6.0.0
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 6.0.0
 - !ruby/object:Gem::Dependency
   name: curb
   requirement: !ruby/object:Gem::Requirement
@@ -390,7 +377,7 @@ dependencies:
         version: 0.9.6
 description: Automation steps and mock server to validaterequest payloads response.
 email:
-- steve@bugsnag.com
+- notifiers@bugsnag.com
 executables:
 - bb-failed-sessions
 - benchmarks-to-datadog
@@ -421,8 +408,6 @@ files:
 - lib/features/steps/breadcrumb_steps.rb
 - lib/features/steps/browser_steps.rb
 - lib/features/steps/build_api_steps.rb
-- lib/features/steps/deprecated_steps.rb
-- lib/features/steps/document_server_steps.rb
 - lib/features/steps/error_reporting_steps.rb
 - lib/features/steps/feature_flag_steps.rb
 - lib/features/steps/header_steps.rb
@@ -461,7 +446,6 @@ files:
 - lib/maze/client/appium/bb_devices.rb
 - lib/maze/client/appium/bs_client.rb
 - lib/maze/client/appium/bs_devices.rb
-- lib/maze/client/appium/bs_legacy_client.rb
 - lib/maze/client/appium/local_client.rb
 - lib/maze/client/appium/session_metadata.rb
 - lib/maze/client/bb_api_client.rb
@@ -477,7 +461,6 @@ files:
 - lib/maze/compare.rb
 - lib/maze/configuration.rb
 - lib/maze/docker.rb
-- lib/maze/document_server.rb
 - lib/maze/driver/appium.rb
 - lib/maze/driver/browser.rb
 - lib/maze/error_monitor/assert_error_middleware.rb
@@ -550,7 +533,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.0
+      version: '3.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -560,5 +543,5 @@ requirements: []
 rubygems_version: 3.5.3
 signing_key:
 specification_version: 4
-summary: Bugsnag API request validation harness
+summary: Bugsnag SDK test harness
 test_files: []

data/lib/features/steps/deprecated_steps.rb

@@ -1,8 +0,0 @@
-# @!group Deprecated steps
-
-# Waits for a given number of spans to be received, which may be spread across one or more trace requests.
-#
-# @step_input span_count [Integer] The number of spans to wait for
-When('I wait for {int} span(s)') do |span_count|
-  assert_received_spans Maze::Server.list_for('traces'), span_count
-end

data/lib/features/steps/document_server_steps.rb

@@ -1,7 +0,0 @@
-# @!group Document steps
-
-# Starts the document server manually.  It will be stopped automatically at the end of each scenario
-# (if started in this way).
-When('I start the document server') do
-  Maze::DocumentServer.manual_start
-end

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

@@ -1,31 +0,0 @@
-module Maze
-  module Client
-    module Appium
-      class BrowserStackLegacyClient < BrowserStackClient
-        def device_capabilities
-          config = Maze.config
-          capabilities = {
-            'app' => config.app,
-            'browserstack.console' => 'errors',
-            'deviceOrientation' => 'portrait',
-            'noReset' => 'true'
-          }
-          device_caps = Maze::Client::Appium::BrowserStackDevices::DEVICE_HASH[config.device]
-          capabilities.deep_merge! device_caps
-          capabilities.deep_merge! JSON.parse(config.capabilities_option)
-          capabilities.merge! project_name_capabilities
-          capabilities['browserstack.appium_version'] = config.appium_version unless config.appium_version.nil?
-          unless device_caps['platformName'] == 'android' && device_caps['platformVersion'].to_i <= 6
-            capabilities['disableAnimations'] = 'true'
-          end
-          if Maze.config.start_tunnel
-            capabilities['browserstack.localIdentifier'] = Maze.run_uuid
-            capabilities['browserstack.local'] = 'true'
-          end
-
-          capabilities
-        end
-      end
-    end
-  end
-end

data/lib/maze/document_server.rb

@@ -1,46 +0,0 @@
-# frozen_string_literal: true
-
-require 'webrick'
-
-module Maze
-  # HTTP server for a given document root
-  class DocumentServer
-    class << self
-      # Start the document server.  This is intended to be called only once per test run.
-      # Use manual_start for finer grained control.
-      def start
-        @thread = Thread.new do
-          options = {
-            DocumentRoot: Maze.config.document_server_root,
-            Port: Maze.config.document_server_port,
-            Logger: $logger,
-            AccessLog: []
-          }
-          options[:BindAddress] = Maze.config.document_server_bind_address unless Maze.config.document_server_bind_address.nil?
-          server = WEBrick::HTTPServer.new(options)
-          server.mount '/reflect', Servlets::ReflectiveServlet
-
-          $logger.info "Starting document server for root: #{Maze.config.document_server_root}"
-          server.start
-        end
-      end
-
-      # Starts the document server "manually" (via a Cucumber step as opposed to command line option)
-      def manual_start
-        if !@thread.nil? && @thread.alive?
-          $logger.warn 'Document Server has already been started on the command line, ignoring manual start'
-          return
-        end
-        @manual_start = true
-        start
-      end
-
-      def manual_stop
-        return unless  @manual_start
-
-        @thread.kill
-        @manual_start = false
-      end
-    end
-  end
-end