RUM-15813: Fix detekt

Author: satween

integrations/dd-sdk-android-compose/src/main/kotlin/com/datadog/android/compose/internal/utils/LayoutNodeUtils.kt

@@ -106,7 +106,7 @@ internal class LayoutNodeUtils {
             getLayoutNodeBoundsInWindowInternal(node) ?: getLayoutNodeBoundsInWindowReflection(node)
         }
 
-        MethodResolver.State.MAGLING_FAILED -> getLayoutNodeBoundsInWindowReflection(node)
+        MethodResolver.State.MANGLING_FAILED -> getLayoutNodeBoundsInWindowReflection(node)
         MethodResolver.State.REFLECTION_FAILED -> getLayoutNodeBoundsInWindowInternal(node)
     }
 
@@ -118,7 +118,7 @@ internal class LayoutNodeUtils {
         "getLayoutNodeBoundsInWindow[reflection]"
     ) {
         // TODO RUM-13454 Update compose bom and remove this method
-        methodResolver.state = MethodResolver.State.MAGLING_FAILED
+        methodResolver.state = MethodResolver.State.MANGLING_FAILED
         val coordinates = node.invokeWithReflection("getLayoutDelegate")
             ?.invokeWithReflection("getOuterCoordinator")
             ?.invokeWithReflection("getCoordinates") as? LayoutCoordinates
@@ -166,9 +166,9 @@ internal class LayoutNodeUtils {
         // per owner-class rather than once globally.
         //
         enum class State {
-            UNKNOWN, // — Try internal magling resolution, if fails - reflection.
-            MAGLING_FAILED, // Magling resolution failed - allowing reflection attempt
-            REFLECTION_FAILED // Reflection resolution failed - switching back to magling resolution (even if it fails)
+            UNKNOWN, // — Try internal mangling resolution, if fails - reflection.
+            MANGLING_FAILED, // Mangling resolution failed - allowing reflection attempt
+            REFLECTION_FAILED // Reflection resolution failed - switching back to mangling resolution (even if it fails)
         }
 
         var state: State = State.UNKNOWN

integrations/dd-sdk-android-compose/src/test/kotlin/com/datadog/android/compose/internal/utils/LayoutNodeUtilsTest.kt

@@ -243,7 +243,7 @@ class LayoutNodeUtilsTest {
     @Test
     fun `M use reflection path only W getLayoutNodeBoundsInWindow() {reflection already activated}`() {
         // Given
-        testedLayoutNodeUtils.methodResolver().state = LayoutNodeUtils.MethodResolver.State.MAGLING_FAILED
+        testedLayoutNodeUtils.methodResolver().state = LayoutNodeUtils.MethodResolver.State.MANGLING_FAILED
         val mockNode = mock<LayoutNode>()
 
         // When
@@ -267,7 +267,7 @@ class LayoutNodeUtilsTest {
 
         // Then
         val resolver = testedLayoutNodeUtils.methodResolver()
-        assertThat(resolver.state).isEqualTo(LayoutNodeUtils.MethodResolver.State.MAGLING_FAILED)
+        assertThat(resolver.state).isEqualTo(LayoutNodeUtils.MethodResolver.State.MANGLING_FAILED)
         assertThat(resolver.classPrefixMethodsCache)
             .containsKey(mockNode.javaClass)
         assertThat(resolver.classPrefixMethodsCache[mockNode.javaClass])
@@ -279,7 +279,7 @@ class LayoutNodeUtilsTest {
         // Given
         // Force reflection path so all three levels (LayoutNode, LayoutNodeLayoutDelegate,
         // NodeCoordinator) are traversed and cached — the maximum cache size for this chain.
-        testedLayoutNodeUtils.methodResolver().state = LayoutNodeUtils.MethodResolver.State.MAGLING_FAILED
+        testedLayoutNodeUtils.methodResolver().state = LayoutNodeUtils.MethodResolver.State.MANGLING_FAILED
         val mockNode = mockLayoutNodeWithValidInternalPath()
         testedLayoutNodeUtils.getLayoutNodeBoundsInWindow(mockNode)
         val cache = testedLayoutNodeUtils.methodResolver().classPrefixMethodsCache
@@ -373,7 +373,7 @@ class LayoutNodeUtilsTest {
 
         // When
         methodResolver.state = LayoutNodeUtils.MethodResolver.State.UNKNOWN
-        methodResolver.state = LayoutNodeUtils.MethodResolver.State.MAGLING_FAILED
+        methodResolver.state = LayoutNodeUtils.MethodResolver.State.MANGLING_FAILED
 
         // Then
         assertThat(methodResolver.state)
@@ -384,14 +384,14 @@ class LayoutNodeUtilsTest {
     fun `M not downgrade state W MethodResolver#state {MAGLING_FAILED cannot revert to UNKNOWN}`() {
         // Given
         val methodResolver = LayoutNodeUtils.MethodResolver()
-        methodResolver.state = LayoutNodeUtils.MethodResolver.State.MAGLING_FAILED
+        methodResolver.state = LayoutNodeUtils.MethodResolver.State.MANGLING_FAILED
 
         // When
         methodResolver.state = LayoutNodeUtils.MethodResolver.State.UNKNOWN
 
         // Then
         assertThat(methodResolver.state)
-            .isEqualTo(LayoutNodeUtils.MethodResolver.State.MAGLING_FAILED)
+            .isEqualTo(LayoutNodeUtils.MethodResolver.State.MANGLING_FAILED)
     }
 
     @Test
@@ -457,7 +457,7 @@ class LayoutNodeUtilsTest {
         // ignored the cache and re-searched, it would return null (and downgrade to
         // REFLECTION_FAILED).
         val methodResolver = LayoutNodeUtils.MethodResolver()
-        methodResolver.state = LayoutNodeUtils.MethodResolver.State.MAGLING_FAILED
+        methodResolver.state = LayoutNodeUtils.MethodResolver.State.MANGLING_FAILED
         val knownMethod = SuffixFixture::class.java.getMethod("foo")
         methodResolver.classPrefixMethodsCache[SuffixFixture::class.java] =
             mutableMapOf("bogus" to knownMethod)
@@ -468,7 +468,7 @@ class LayoutNodeUtilsTest {
         // Then
         assertThat(result).isSameAs(knownMethod)
         assertThat(methodResolver.state)
-            .isEqualTo(LayoutNodeUtils.MethodResolver.State.MAGLING_FAILED)
+            .isEqualTo(LayoutNodeUtils.MethodResolver.State.MANGLING_FAILED)
     }
 
     @Test
@@ -478,7 +478,7 @@ class LayoutNodeUtilsTest {
         // even when the method actually exists on the class. State still transitions to
         // REFLECTION_FAILED because findMethod's post-condition fires on any null result.
         val methodResolver = LayoutNodeUtils.MethodResolver()
-        methodResolver.state = LayoutNodeUtils.MethodResolver.State.MAGLING_FAILED
+        methodResolver.state = LayoutNodeUtils.MethodResolver.State.MANGLING_FAILED
         methodResolver.classPrefixMethodsCache[SuffixFixture::class.java] =
             mutableMapOf("foo" to null)