bugsnag-maze-runner 9.18.1 → 9.19.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1bf3a12cf3a22fdf973e3fd8ac7880475395ab7cd6a449b7c7484ef9f6b9f1a0
-  data.tar.gz: 492d5ed9de4b7203253d641244c84f45f7e571ea4114e9059f608af179e64622
+  metadata.gz: 2ed96ef59a28728acac2405e11e0838fdee4e47b2971236e98a0142f2eb72211
+  data.tar.gz: 75c6d6c2c0c0a45928c2da61fe6b8deb584acc71ef7a841aeb0d0fa981c6b6c5
 SHA512:
-  metadata.gz: 50c855d588fa42abc11d7547a09ea3994c7d2da56873f62dfea3654a38841a6f67c4e47453f5cc51fbf98a8918187a1efe46ca6d89f987daf11eb8dc49355744
-  data.tar.gz: b2280139bc8e8ccf867c9c655d386f3be1d30990aa4ed392e343170338a2b95f173b6ae742ce513f5449d8d81e3a4d214101ce605b36c10a913f7e8aff5ea9a3
+  metadata.gz: b2871cd29e22a58fe4200a44beaffc56d385ffe7d6fc27ca7a0bc5c3391cc87fba32f3f12203059c0e54d1c79e15f5a43646ade01bef8a3919d8602b331ce0ff
+  data.tar.gz: e30d047f7f87f7d7f63985fa060f220e854b3d67ef3a2eb495b62c9f3a025da3291dce27b2472522430e0b05262ce4356d93c2871e8d73a8558ab31b407e0ced

data/bin/maze-runner

@@ -36,6 +36,7 @@ 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/error_captor'
 require_relative '../lib/maze/errors'
 require_relative '../lib/maze/generator'
 require_relative '../lib/maze/helper'

data/lib/features/support/internal_hooks.rb

@@ -186,6 +186,7 @@ ensure
   Maze::Runner.environment.clear
   Maze::Store.values.clear
   Maze::Aws::Sam.reset!
+  Maze::ErrorCaptor.reset
 end
 
 def output_received_requests(request_type)

data/lib/maze/bugsnag_config.rb

@@ -33,6 +33,7 @@ module Maze
           end
           config.middleware.use(AssertErrorMiddleware)
           config.middleware.use(AmbiguousErrorMiddleware)
+          config.middleware.use(ErrorCaptureMiddleware)
           config.add_metadata(:'buildkite', metadata)
           config.project_root = Dir.pwd
         end
@@ -88,5 +89,18 @@ module Maze
         @middleware.call(report)
       end
     end
+
+    class ErrorCaptureMiddleware
+
+      def initialize(middleware)
+        @middleware = middleware
+      end
+
+      def call(report)
+        Maze::ErrorCaptor.instance.add_captured_error(report.dup)
+
+        @middleware.call(report)
+      end
+    end
   end
 end

data/lib/maze/client/selenium/bb_browsers.yml

@@ -65,16 +65,16 @@ edge_latest:
   version: 'latest'
   resolution: '1920x1080'
 
-safari_16:
+safari_18:
   platform: 'macOS'
   osVersion: '13'
   browserName: 'safari'
-  version: '16'
+  version: '18'
   resolution: '2560x1920'
 
-safari_15:
+safari_17:
   platform: 'macOS'
   osVersion: '12'
   browserName: 'safari'
-  version: '15'
+  version: '17'
   resolution: '2560x1920'

data/lib/maze/error_captor.rb

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+module Maze
+  class ErrorCaptor
+    attr_reader :captured_errors
+
+    class << self
+
+      # Sets the last error report that BugSnag has captured
+      #
+      # @param report [Bugsnag::Report] The report to set
+      def add(report)
+        captured_errors << report
+      end
+
+      # Returns an array of the primary error classes of captured error reports
+      #
+      # @return [Array<String>] The error classes
+      def classes
+        captured_errors.map { |report| report.summary[:error_class] }
+      end
+
+      # Returns the primary error messages of the captured error reports
+      #
+      # @return [Array<String>] The error messages
+      def messages
+        captured_errors.map { |report| report.summary[:message] }
+      end
+
+      # Resets the captured errors array
+      def reset
+        @captured_errors = []
+      end
+
+      # Whether any error reports exist in the current context
+      #
+      # @return [Boolean] Whether a report exists
+      def empty?
+        captured_errors.empty?
+      end
+
+      def captured_errors
+        @captured_errors ||= []
+      end
+    end
+  end
+end

data/lib/maze/hooks/appium_hooks.rb

@@ -3,6 +3,13 @@ module Maze
   module Hooks
     # Hooks for Appium mode use
     class AppiumHooks < InternalHooks
+
+      APPIUM_DRIVER_FAILED_ERRORS = [
+        Selenium::WebDriver::Error::UnknownError,
+        Selenium::WebDriver::Error::ServerError,
+        Selenium::WebDriver::Error::WebDriverError
+      ]
+
       @client
 
       def before_all
@@ -23,21 +30,23 @@ module Maze
       end
 
       def after(scenario)
+        if scenario.failed?
+          Maze.driver = nil if driver_has_failed?
+          $logger.warn('The appium driver has failed, removing it to prevent future errors')
+        end
 
         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
-          write_device_logs(scenario) if scenario.failed?
-
           # Reset the server to ensure that test fixtures cannot fetch
           # commands from the previous scenario (in idempotent mode).
           begin
-            Maze.driver.terminate_app Maze.driver.app_id
-          rescue Selenium::WebDriver::Error::UnknownError
+            Maze.driver.terminate_app Maze.driver&.app_id
+          rescue Selenium::WebDriver::Error::UnknownError, Selenium::WebDriver::Error::InvalidSessionIdError
             if Maze.config.appium_version && Maze.config.appium_version.to_f < 2.0
               $logger.warn 'terminate_app failed, using the slower but more forceful close_app instead'
-              Maze.driver.close_app
+              Maze.driver&.close_app
             else
               $logger.warn 'terminate_app failed, future errors may occur if the application did not close remotely'
             end
@@ -73,6 +82,14 @@ module Maze
 
       private
 
+      def driver_has_failed?
+        return false if Maze::ErrorCaptor.empty?
+
+        Maze::ErrorCaptor.classes.any? do |error_class|
+          APPIUM_DRIVER_FAILED_ERRORS.include?(error_class)
+        end
+      end
+
       # Pulls the device logs using Appium and writes them to file in the maze_output folder
       def write_device_logs(scenario)
         log_name = case Maze::Helper.get_current_platform
@@ -81,9 +98,10 @@ module Maze
                    when 'ios'
                      'syslog'
                    end
-        logs = Maze.driver.get_log(log_name)
-
-        Maze::MazeOutput.new(scenario).write_device_logs(logs)
+        unless Maze.driver.nil?
+          logs = Maze.driver.get_log(log_name)
+          Maze::MazeOutput.new(scenario).write_device_logs(logs)
+        end
       end
     end
   end

data/lib/maze/maze_output.rb

@@ -90,7 +90,7 @@ module Maze
       end
     end
 
-    # Pulls the logs from the device if the scenario fails
+    # Writes a set of logs to a device logs file
     # @param logs [Array<String>] The lines of log to be written
     def write_device_logs(logs)
 

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.18.1'
+  VERSION = '9.19.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.18.1
+  version: 9.19.0
 platform: ruby
 authors:
 - Steve Kirkland
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-11-13 00:00:00.000000000 Z
+date: 2024-11-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: cucumber
@@ -474,6 +474,7 @@ files:
 - lib/maze/document_server.rb
 - lib/maze/driver/appium.rb
 - lib/maze/driver/browser.rb
+- lib/maze/error_captor.rb
 - lib/maze/errors.rb
 - lib/maze/generator.rb
 - lib/maze/helper.rb