RUM-9908 create addViewAttributes and removeViewAttributes API

Author: xgouchet

features/dd-sdk-android-rum/api/apiSurface

@@ -120,6 +120,8 @@ interface com.datadog.android.rum.RumMonitor
   fun clearAttributes()
   fun stopSession()
   fun addViewLoadingTime(Boolean)
+  fun addViewAttributes(Map<String, Any?>)
+  fun removeViewAttributes(Collection<String>)
   var debug: Boolean
   fun _getInternal(): _RumInternalProxy?
 enum com.datadog.android.rum.RumPerformanceMetric

features/dd-sdk-android-rum/api/dd-sdk-android-rum.api

@@ -152,12 +152,14 @@ public abstract interface class com/datadog/android/rum/RumMonitor {
 	public abstract fun addFeatureFlagEvaluation (Ljava/lang/String;Ljava/lang/Object;)V
 	public abstract fun addFeatureFlagEvaluations (Ljava/util/Map;)V
 	public abstract fun addTiming (Ljava/lang/String;)V
+	public abstract fun addViewAttributes (Ljava/util/Map;)V
 	public abstract fun addViewLoadingTime (Z)V
 	public abstract fun clearAttributes ()V
 	public abstract fun getAttributes ()Ljava/util/Map;
 	public abstract fun getCurrentSessionId (Lkotlin/jvm/functions/Function1;)V
 	public abstract fun getDebug ()Z
 	public abstract fun removeAttribute (Ljava/lang/String;)V
+	public abstract fun removeViewAttributes (Ljava/util/Collection;)V
 	public abstract fun setDebug (Z)V
 	public abstract fun startAction (Lcom/datadog/android/rum/RumActionType;Ljava/lang/String;Ljava/util/Map;)V
 	public abstract fun startResource (Ljava/lang/String;Lcom/datadog/android/rum/RumResourceMethod;Ljava/lang/String;Ljava/util/Map;)V

features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/RumMonitor.kt

@@ -334,6 +334,20 @@ interface RumMonitor {
     @ExperimentalRumApi
     fun addViewLoadingTime(overwrite: Boolean)
 
+    /**
+     * Adds attributes to the current active View. They will be propagated to all future RUM events within this View
+     * until it stops being active.
+     * @param attributes the attributes to add to the view
+     */
+    fun addViewAttributes(attributes: Map<String, Any?>)
+
+    /**
+     * Removes attributes from the current active View. Future RUM events within this view won't be having these
+     * attributes anymore.
+     * @param attributes the attribute keys to remove from the view
+     */
+    fun removeViewAttributes(attributes: Collection<String>)
+
     /**
      * Utility setting to inspect the active RUM View.
      * If set, a debugging outline will be displayed on top of the application, describing the name

features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/domain/scope/RumRawEvent.kt

@@ -186,6 +186,16 @@ internal sealed class RumRawEvent {
         override val eventTime: Time = Time()
     ) : RumRawEvent()
 
+    internal data class AddViewAttributes(
+        val attributes: Map<String, Any?>,
+        override val eventTime: Time = Time()
+    ) : RumRawEvent()
+
+    internal data class RemoveViewAttributes(
+        val attributes: Collection<String>,
+        override val eventTime: Time = Time()
+    ) : RumRawEvent()
+
     internal data class AddLongTask(
         val durationNs: Long,
         val target: String,

features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/monitor/DatadogRumMonitor.kt

@@ -553,6 +553,18 @@ internal class DatadogRumMonitor(
         handleEvent(RumRawEvent.AddViewLoadingTime(overwrite = overwrite))
     }
 
+    override fun addViewAttributes(attributes: Map<String, Any?>) {
+        handleEvent(
+            RumRawEvent.AddViewAttributes(attributes)
+        )
+    }
+
+    override fun removeViewAttributes(attributes: Collection<String>) {
+        handleEvent(
+            RumRawEvent.RemoveViewAttributes(attributes)
+        )
+    }
+
     override fun addLongTask(durationNs: Long, target: String) {
         handleEvent(
             RumRawEvent.AddLongTask(durationNs, target)

features/dd-sdk-android-rum/src/test/kotlin/com/datadog/android/rum/internal/monitor/DatadogRumMonitorTest.kt

@@ -55,12 +55,14 @@ import com.datadog.android.telemetry.internal.TelemetryEventHandler
 import com.datadog.tools.unit.forge.aThrowable
 import com.datadog.tools.unit.forge.exhaustiveAttributes
 import fr.xgouchet.elmyr.Forge
+import fr.xgouchet.elmyr.annotation.AdvancedForgery
 import fr.xgouchet.elmyr.annotation.BoolForgery
 import fr.xgouchet.elmyr.annotation.DoubleForgery
 import fr.xgouchet.elmyr.annotation.FloatForgery
 import fr.xgouchet.elmyr.annotation.Forgery
 import fr.xgouchet.elmyr.annotation.IntForgery
 import fr.xgouchet.elmyr.annotation.LongForgery
+import fr.xgouchet.elmyr.annotation.MapForgery
 import fr.xgouchet.elmyr.annotation.StringForgery
 import fr.xgouchet.elmyr.annotation.StringForgeryType
 import fr.xgouchet.elmyr.junit5.ForgeConfiguration
@@ -919,6 +921,41 @@ internal class DatadogRumMonitorTest {
         verifyNoMoreInteractions(mockScope, mockWriter)
     }
 
+    @Test
+    fun `M delegate event to rootScope W addViewAttributes()`(
+        @MapForgery(
+            key = AdvancedForgery(string = [StringForgery(StringForgeryType.ALPHABETICAL)]),
+            value = AdvancedForgery(string = [StringForgery()])
+        ) fakeAttributes: Map<String, String>
+    ) {
+        testedMonitor.addViewAttributes(fakeAttributes)
+        Thread.sleep(PROCESSING_DELAY)
+
+        argumentCaptor<RumRawEvent> {
+            verify(mockScope).handleEvent(capture(), same(mockWriter))
+            val event = firstValue
+            check(event is RumRawEvent.AddViewAttributes)
+            assertThat(event.attributes).isSameAs(fakeAttributes)
+        }
+        verifyNoMoreInteractions(mockScope, mockWriter)
+    }
+
+    @Test
+    fun `M delegate event to rootScope W removeViewAttributes()`(
+        @StringForgery fakeAttributes: List<String>
+    ) {
+        testedMonitor.removeViewAttributes(fakeAttributes)
+        Thread.sleep(PROCESSING_DELAY)
+
+        argumentCaptor<RumRawEvent> {
+            verify(mockScope).handleEvent(capture(), same(mockWriter))
+            val event = firstValue
+            check(event is RumRawEvent.RemoveViewAttributes)
+            assertThat(event.attributes).isSameAs(fakeAttributes)
+        }
+        verifyNoMoreInteractions(mockScope, mockWriter)
+    }
+
     @Test
     fun `M delegate event to rootScope on current thread W addCrash()`(
         @StringForgery message: String,