Merge pull request #706 from bugsnag/tiered-notifies

Use normal maze-runner API key in the event of selenium failures

CHANGELOG.md

@@ -1,4 +1,8 @@
-# 9.20.1- 2024/11/27
+# 9.21.0 - 2024/11/28
+
+## Enhancements
+
+- Ensure selenium related errors are always reported [706](https://github.com/bugsnag/maze-runner/pull/706)
 
 ## Fixes
 

lib/maze.rb

@@ -7,7 +7,8 @@
 # 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.20.1'
+
+  VERSION = '9.21.0'
 
   class << self
     attr_accessor :check, :driver, :internal_hooks, :mode, :start_time, :dynamic_retry, :public_address,

lib/maze/plugins/bugsnag_reporting_plugin.rb

@@ -25,8 +25,7 @@ def test_case(test_case)
           next unless event.test_case.eql?(test_case) && event.result.failed?
 
           Bugsnag.notify(event.result.exception) do |bsg_event|
-
-            bsg_event.api_key = ENV['MAZE_SCENARIO_BUGSNAG_API_KEY']
+            bsg_event.api_key = ENV['MAZE_SCENARIO_BUGSNAG_API_KEY'] unless selenium_report?(event.result.exception)
 
             unless @last_test_step.nil?
 
@@ -54,6 +53,10 @@ def test_case(test_case)
 
         super
       end
+
+      def selenium_report?(exception)
+        exception.class.to_s.include?('Selenium')
+      end
     end
   end
 end