Merge branch 'main' into tiered-notifies

CHANGELOG.md

@@ -4,6 +4,10 @@
 
 - Ensure selenium related errors are always reported [706](https://github.com/bugsnag/maze-runner/pull/706)
 
+## Fixes
+
+- Fix issue where the incorrect `buildId` was being pulled from `Maze.driver.session_capabilities` [707](https://github.com/bugsnag/maze-runner/pull/707)
+
 # 9.20.0 - 2024/11/21
 
 ## Enhancements

lib/maze.rb

@@ -7,6 +7,7 @@
 # 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.21.0'
 
   class << self

lib/maze/client/appium/base_client.rb

@@ -23,11 +23,16 @@ def start_session
                                  Maze.driver.session_capabilities['appPackage']
                                when 'ios'
                                  unless app_id = Maze.driver.session_capabilities['CFBundleIdentifier']
-                                    app_id = Maze.driver.session_capabilities['bundleID']
+                                    app_id = Maze.driver.session_capabilities['bundleId']
                                  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
+
           # Ensure the device is unlocked
           begin
             Maze.driver.unlock