test(event) error options end to end tests

Author: YYChen01988

bugsnag-android-core/src/main/java/com/bugsnag/android/ErrorOptions.kt

@@ -21,12 +21,12 @@ class CaptureOptions(
     /**
      * Controls which custom metadata tabs are included.
      * - null: all metadata tabs captured
-     * - empty set: only app and device tabs captured
+     * - empty set: no custom metadata captured
      * - set of names: app, device, and specified tabs captured
      *
      * Note: app and device tabs are always captured.
      */
-    var metadata: Set<String>? = null,
+    var metadata: MutableSet<String>? = null,
 
     /** Whether to capture stacktrace. Defaults to true. */
     var stacktrace: Boolean = true,
@@ -60,7 +60,7 @@ class CaptureOptions(
          */
         @JvmStatic
         @JvmOverloads
-        fun captureOnly(fields: Int, metadata: Set<String>? = null): CaptureOptions {
+        fun captureOnly(fields: Int, metadata: MutableSet<String>? = null): CaptureOptions {
             return captureNothing().apply {
                 stacktrace = (fields and CAPTURE_STACKTRACE) != 0
                 breadcrumbs = (fields and CAPTURE_BREADCRUMBS) != 0

examples/sdk-app-example/app/src/main/java/com/example/bugsnag/android/ExampleApplication.kt

@@ -36,7 +36,6 @@ class ExampleApplication : Application() {
         config.setUser("123456", "[email protected]", "Joe Bloggs")
         config.addMetadata("user", "age", 31)
         config.addPlugin(bugsnagOkHttpPlugin)
-        config.isAttemptDeliveryOnCrash = true
 
         // Configure the persistence directory when running MultiProcessActivity in a separate
         // process to ensure the two Bugsnag clients are independent

features/fixtures/mazerunner/jvm-scenarios/src/main/java/com/bugsnag/android/mazerunner/scenarios/ErrorOptionsScenario.kt

@@ -0,0 +1,20 @@
+package com.bugsnag.android.mazerunner.scenarios
+
+import android.content.Context
+import com.bugsnag.android.Bugsnag
+import com.bugsnag.android.CaptureOptions
+import com.bugsnag.android.Configuration
+import com.bugsnag.android.ErrorOptions
+
+class ErrorOptionsScenario(
+    config: Configuration,
+    context: Context,
+    eventMetadata: String?
+) : Scenario(config, context, eventMetadata) {
+    private val errorOptions = ErrorOptions(CaptureOptions())
+
+    override fun startScenario() {
+        super.startScenario()
+        Bugsnag.notify(generateException(), errorOptions, null)
+    }
+}

features/fixtures/mazerunner/jvm-scenarios/src/main/java/com/bugsnag/android/mazerunner/scenarios/NullErrorOptionsScenario.kt

@@ -0,0 +1,17 @@
+package com.bugsnag.android.mazerunner.scenarios
+
+import android.content.Context
+import com.bugsnag.android.Bugsnag
+import com.bugsnag.android.Configuration
+
+class NullErrorOptionsScenario(
+    config: Configuration,
+    context: Context,
+    eventMetadata: String?
+) : Scenario(config, context, eventMetadata) {
+
+    override fun startScenario() {
+        super.startScenario()
+        Bugsnag.notify(generateException(), null, null)
+    }
+}

features/full_tests/auto_notify.feature

@@ -43,5 +43,3 @@ Feature: Switching automatic error detection on/off for Unity
     And the event "severity" equals "error"
     And the event "severityReason.type" equals "signal"
     And the event "severityReason.unhandledOverridden" is false
-
-

features/full_tests/error_options_notify.feature

@@ -0,0 +1,15 @@
+Feature: Error options notify scenarios
+
+  Background:
+    Given I clear all persistent data
+
+  Scenario: Handled exceptions with null error options
+    When I run "NullErrorOptionsScenario"
+    Then I wait to receive an error
+    And the error is valid for the error reporting API version "4.0" for the "Android Bugsnag Notifier" notifier
+
+
+  Scenario: Handled exceptions with negative options
+    When I run "ErrorOptionsScenario"
+    Then I wait to receive an error
+    And the error is valid for the error reporting API version "4.0" for the "Android Bugsnag Notifier" notifier