Fix setting in-app-includes from external properties. (#1291)

Author: maciejwalkowiak

CHANGELOG.md

@@ -1,5 +1,7 @@
 # vNext
 
+* Fix: Fix setting in-app-includes from external properties (#1291)
+
 
 # 4.2.0
 

sentry/src/main/java/io/sentry/SentryOptions.java

@@ -1383,6 +1383,14 @@ void merge(final @NotNull SentryOptions options) {
     for (final Map.Entry<String, String> tag : tags.entrySet()) {
       this.tags.put(tag.getKey(), tag.getValue());
     }
+    final List<String> inAppIncludes = new ArrayList<>(options.getInAppIncludes());
+    for (final String inAppInclude : inAppIncludes) {
+      addInAppInclude(inAppInclude);
+    }
+    final List<String> inAppExcludes = new ArrayList<>(options.getInAppExcludes());
+    for (final String inAppExclude : inAppExcludes) {
+      addInAppExclude(inAppExclude);
+    }
   }
 
   private @NotNull SdkVersion createSdkVersion() {

sentry/src/test/java/io/sentry/SentryOptionsTest.kt

@@ -177,6 +177,8 @@ class SentryOptionsTest {
         externalOptions.setTag("tag2", "value2")
         externalOptions.enableUncaughtExceptionHandler = false
         externalOptions.tracesSampleRate = 0.5
+        externalOptions.addInAppInclude("com.app")
+        externalOptions.addInAppExclude("io.off")
         val options = SentryOptions()
 
         options.merge(externalOptions)
@@ -192,6 +194,8 @@ class SentryOptionsTest {
         assertEquals(mapOf("tag1" to "value1", "tag2" to "value2"), options.tags)
         assertFalse(options.enableUncaughtExceptionHandler!!)
         assertEquals(0.5, options.tracesSampleRate)
+        assertEquals(listOf("com.app"), options.inAppIncludes)
+        assertEquals(listOf("io.off"), options.inAppExcludes)
     }
 
     @Test