Merge pull request #2399 from DataDog/jmoskovich/rum-7302/align-level…

…s-sr-already-enabled RUM-7302: Align log levels for sr already enabled
DataDog · Nov 18, 2024 · bd71625 · bd71625
2 parents 352d252 + 8a6a881
commit bd71625
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 5 deletions.
diff --git a/...android-session-replay/src/main/kotlin/com/datadog/android/sessionreplay/SessionReplay.kt b/...android-session-replay/src/main/kotlin/com/datadog/android/sessionreplay/SessionReplay.kt
@@ -106,10 +106,17 @@ object SessionReplay {
     private fun isAlreadyRegistered() =
         currentRegisteredCore?.get()?.isCoreActive() == true
 
-    private fun logAlreadyRegisteredWarning(internalLogger: InternalLogger) =
+    private fun logAlreadyRegisteredWarning(internalLogger: InternalLogger) {
         internalLogger.log(
-            level = InternalLogger.Level.WARN,
-            targets = listOf(InternalLogger.Target.MAINTAINER, InternalLogger.Target.TELEMETRY),
+            level = InternalLogger.Level.ERROR,
+            targets = listOf(InternalLogger.Target.MAINTAINER),
             messageBuilder = { IS_ALREADY_REGISTERED_WARNING }
         )
+
+        internalLogger.log(
+            level = InternalLogger.Level.DEBUG,
+            targets = listOf(InternalLogger.Target.TELEMETRY),
+            messageBuilder = { IS_ALREADY_REGISTERED_WARNING }
+        )
+    }
 }
diff --git a/...oid-session-replay/src/test/kotlin/com/datadog/android/sessionreplay/SessionReplayTest.kt b/...oid-session-replay/src/test/kotlin/com/datadog/android/sessionreplay/SessionReplayTest.kt
@@ -156,8 +156,14 @@ internal class SessionReplayTest {
 
         // Then
         mockInternalLogger.verifyLog(
-            level = InternalLogger.Level.WARN,
-            targets = listOf(InternalLogger.Target.MAINTAINER, InternalLogger.Target.TELEMETRY),
+            level = InternalLogger.Level.ERROR,
+            targets = listOf(InternalLogger.Target.MAINTAINER),
+            message = IS_ALREADY_REGISTERED_WARNING
+        )
+
+        mockInternalLogger.verifyLog(
+            level = InternalLogger.Level.DEBUG,
+            targets = listOf(InternalLogger.Target.TELEMETRY),
             message = IS_ALREADY_REGISTERED_WARNING
         )
         assertThat(SessionReplay.currentRegisteredCore?.get()).isEqualTo(mockCore1)