Skip to content

Commit 57737e9

Browse files
authored
Fix duplicate crash + ANR span bug (#1214)
* manually disabling the otel crash reporter config * also disable duplicate ANR install * fix formatting * update unit tests for lenient validation of stub mocking since test env doesnt always fully exercise ANR detector init path * spotless fix
1 parent f54871a commit 57737e9

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

splunk-otel-android/src/main/java/com/splunk/rum/RumInitializer.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,11 @@ SplunkRum initialize(Looper mainLooper) {
105105
}
106106

107107
config.disableScreenAttributes();
108+
// We are installing the crash reporter ourselves in installCrashReporter.
109+
// Disabling this one to prevent duplicate spans
110+
config.disableCrashReporting();
111+
config.disableAnrDetection(); // Same with ANR
112+
108113
OpenTelemetryRumBuilder otelRumBuilder = OpenTelemetryRum.builder(application, config);
109114

110115
otelRumBuilder.mergeResource(createSplunkResource());

splunk-otel-android/src/test/java/com/splunk/rum/RumInitializerTest.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,9 +59,12 @@
5959
import org.junit.jupiter.api.extension.ExtendWith;
6060
import org.mockito.Mock;
6161
import org.mockito.junit.jupiter.MockitoExtension;
62+
import org.mockito.junit.jupiter.MockitoSettings;
63+
import org.mockito.quality.Strictness;
6264
import zipkin2.reporter.okhttp3.OkHttpSender;
6365

6466
@ExtendWith(MockitoExtension.class)
67+
@MockitoSettings(strictness = Strictness.LENIENT)
6568
class RumInitializerTest {
6669

6770
static final String APP_NAME = "Sick test app";

0 commit comments

Comments
 (0)