bugsnag-maze-runner 9.7.0 → 9.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f948d5bf3980cb7514127f60fbecb2ffdb4484c8d19a484e39ebccaaaebda535
-  data.tar.gz: efb5c22b4c81824d63c2762ea0792ad6316dc8744660042b4c1d6357eb8b7a5c
+  metadata.gz: b6101c495b7debb939e4bb83f21dbc015c13f7bc58ce797d69fb90e618c6be6d
+  data.tar.gz: 69f5fdd5eec4c1f5bc8e9ed2aaa188531a6f54e80ec26471873ef93e7ecb1ecd
 SHA512:
-  metadata.gz: 100b68e6b60360ea1a0bad3eb51c2b7e28fce9328e0ebc5a8bfeba089621714cfb834d7c4ae568f4c88c7399ff2e2b0fef351cbf8dd97397b3fb4da5cd39644b
-  data.tar.gz: ba9d6bbf2e966981d14be78688c119967278d662b5c0cc166fedba5302c829a3e9b2e93a0c4dfefc30893c5a9cebd09e1f7408de7c71ee7986d3fe89995eb748
+  metadata.gz: '0438524dc2173c1f31b323cca182e651ea1fc095727ee4658c5bb14baa986b5a17230fcd64460dd9eb9e805469c84eae2d3eadb85ae47668501a70dd1211e090'
+  data.tar.gz: bcf7dfcf91d8093099733839d06b38b1cea25548617eebad4f88a01910d8f64d6e0db26de0de5ebe38977e1cfa2b64d3fed80f8bc693d5ec6071c8dc633f5bb0

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

@@ -34,6 +34,10 @@ module Maze
             interval = 10
           elsif error.message.include? 'Could not proxy command to the remote server'
             interval = 10
+          elsif error.message.include? 'Could not find a connected Android device'
+            interval = 10
+          elsif error.message.include? '\'platformVersion\' must be a valid version number.'
+            interval = 10
           else
             # Do not retry in any other case
           end
@@ -54,26 +58,24 @@ module Maze
         end
 
         def device_capabilities
-          # Doubling up on capabilities in both the `appium:options` and `appium` sub dictionaries.
-          # See PLAT-11087
           config = Maze.config
           common_caps = {
             'noReset' => true,
             'newCommandTimeout' => 600
           }
           capabilities = {
-            'appium:options' => common_caps,
-            'appium' => common_caps,
             'bitbar:options' => {
-              # Some capabilities probably belong in the top level
-              # of the hash, but BitBar picks them up from here.
               'apiKey' => config.access_key,
               'app' => config.app,
               'findDevice' => false,
               'testTimeout' => 7200
             }
           }
-          capabilities.deep_merge! common_caps
+          if Maze.config.appium_version && Maze.config.appium_version.to_f < 2.0
+            capabilities.merge!(common_caps)
+          else
+            capabilities['appium:options'] = common_caps
+          end
           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?

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

@@ -18,22 +18,44 @@ module Maze
             if device_group_ids
               # Device group found - find a free device in it
               $logger.trace "Got group ids #{device_group_ids} for #{device_or_group_names}"
-              group_count, device = api_client.find_device_in_groups(device_group_ids)
-              if device.nil?
-                raise 'There are no devices available'
+              if device_group_ids.size > 1
+                group_id = false
+                group_count, device = api_client.find_device_in_groups(device_group_ids)
+                if device.nil?
+                  raise 'There are no devices available'
+                else
+                  $logger.info "#{group_count} device(s) currently available in group(s) '#{device_or_group_names}'"
+                end
               else
-                $logger.info "#{group_count} device(s) currently available in group(s) '#{device_or_group_names}'"
+                # Since there is only one group, we can use it verbatim
+                $logger.info "Using device group #{device_or_group_names}"
+                group_id = true
+                device_name = device_group_ids.first
               end
             else
               # See if there is a device with the given name
               device = api_client.find_device device_or_group_names
             end
 
-            device_name = device['displayName']
-            platform = device['platform'].downcase
-            platform_version = device['softwareVersion']['releaseVersion']
+            # If a single device has been identified use that to determine other characteristics
+            if device
+              device_name = device['displayName']
+              platform = device['platform'].downcase
+              platform_version = device['softwareVersion']['releaseVersion']
 
-            $logger.info "Selected device: #{device_name} (#{platform} #{platform_version})"
+              $logger.info "Selected device: #{device_name} (#{platform} #{platform_version})"
+            else
+              # If a device group has been identified, extrapolate characteristics from the group name
+              if android_match = Regexp.new('(ANDROID|android)_(\d{1,2})').match(device_or_group_names)
+                platform = 'android'
+                platform_version = android_match[2]
+              elsif ios_match = Regexp.new('(IOS|ios)_(\d{1,2})').match(device_or_group_names)
+                platform = 'ios'
+                platform_version = ios_match[2]
+              end
+
+              $logger.info "Selected device group: #{device_or_group_names} (#{platform} #{platform_version})"
+            end
 
             # TODO: Setting the config here is rather a side effect and factoring it out would be better.
             #   For now, though, it means not having to provide the --os and --os-version options on the command line.
@@ -42,9 +64,9 @@ module Maze
 
             case platform
             when 'android'
-              make_android_hash(device_name)
+              make_android_hash(device_name, group_id)
             when 'ios'
-              make_ios_hash(device_name)
+              make_ios_hash(device_name, group_id)
             else
               throw "Invalid device platform specified #{platform}"
             end
@@ -90,9 +112,7 @@ module Maze
             end
           end
 
-          def make_android_hash(device)
-            # Tripling up on capabilities in the `appium:options`, `appium` sub dictionaries and base dictionary.
-            # See PLAT-11087
+          def android_base_hash
             appium_options = {
               'automationName' => 'UiAutomator2',
               'autoGrantPermissions' => true,
@@ -103,20 +123,31 @@ module Maze
             appium_options['appPackage'] = Maze.config.app_package unless Maze.config.app_package.nil?
             hash = {
               'platformName' => 'Android',
-              'deviceName' => 'Android Phone',
-              'appium:options' => appium_options,
-              'appium' => appium_options,
-              'bitbar:options' => {
-                'device' => device,
-              }
+              'deviceName' => 'Android Phone'
             }
-            hash.merge!(appium_options)
+            if Maze.config.appium_version && Maze.config.appium_version.to_f < 2.0
+              hash.merge!(appium_options)
+            else
+              hash['appium:options'] = appium_options
+            end
+            hash.dup
+          end
+
+          def make_android_hash(device, group_id = false)
+            hash = android_base_hash
+            if group_id
+              hash['bitbar:options'] = {
+                'deviceGroupId' => device
+              }
+            else
+              hash['bitbar:options'] = {
+                'device' => device
+              }
+            end
             hash.freeze
           end
 
-          def make_ios_hash(device)
-            # Tripling up on capabilities in the `appium:options`, `appium` sub dictionaries and base dictionary.
-            # See PLAT-11087
+          def ios_base_hash
             appium_options = {
               'automationName' => 'XCUITest',
               'shouldTerminateApp' => 'true',
@@ -125,13 +156,26 @@ module Maze
             hash = {
               'platformName' => 'iOS',
               'deviceName' => 'iPhone device',
-              'appium:options' => appium_options,
-              'appium' => appium_options,
-              'bitbar:options' => {
+            }
+            if Maze.config.appium_version && Maze.config.appium_version.to_f < 2.0
+              hash.merge!(appium_options)
+            else
+              hash['appium:options'] = appium_options
+            end
+            hash.dup
+          end
+
+          def make_ios_hash(device, group_id = false)
+            hash = ios_base_hash
+            if group_id
+              hash['bitbar:options'] = {
+                'deviceGroupId' => device
+              }
+            else
+              hash['bitbar:options'] = {
                 'device' => device
               }
-            }
-            hash.merge!(appium_options)
+            end
             hash.freeze
           end
         end

data/lib/maze/server.rb

@@ -242,6 +242,7 @@ module Maze
             server.mount '/logs', Servlets::LogServlet
             server.mount '/metrics', Servlets::Servlet, :metrics
             server.mount '/reflect', Servlets::ReflectiveServlet
+            server.mount '/docs', WEBrick::HTTPServlet::FileHandler, Maze.config.document_server_root unless Maze.config.document_server_root.nil?
             server.start
           rescue StandardError => e
             Bugsnag.notify e

data/lib/maze.rb

@@ -7,7 +7,7 @@ require_relative 'maze/timers'
 # Glues the various parts of MazeRunner together that need to be accessed globally,
 # providing an alternative to the proliferation of global variables or singletons.
 module Maze
-  VERSION = '9.7.0'
+  VERSION = '9.8.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.7.0
+  version: 9.8.0
 platform: ruby
 authors:
 - Steve Kirkland
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-04-08 00:00:00.000000000 Z
+date: 2024-05-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: cucumber
@@ -240,14 +240,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 6.12.0
+        version: '7.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 6.12.0
+        version: '7.0'
 - !ruby/object:Gem::Dependency
   name: markdown
   requirement: !ruby/object:Gem::Requirement