RUM-8657: post-review fixes

Author: satween

dd-sdk-android-core/api/apiSurface

@@ -198,12 +198,6 @@ class com.datadog.android.core.UploadWorker : androidx.work.Worker
   constructor(android.content.Context, androidx.work.WorkerParameters)
   override fun doWork(): Result
   companion object 
-class com.datadog.android.core.collections.EvictingQueue<T> : java.util.LinkedList<T>
-  constructor(Int = Int.MAX_VALUE)
-  override fun add(T): Boolean
-  override fun addAll(Collection<T>): Boolean
-  override fun add(Int, T)
-  override fun addAll(Int, Collection<T>): Boolean
 enum com.datadog.android.core.configuration.BackPressureMitigation
   - DROP_OLDEST
   - IGNORE_NEWEST

dd-sdk-android-core/api/dd-sdk-android-core.api

@@ -563,20 +563,6 @@ public final class com/datadog/android/core/UploadWorker : androidx/work/Worker
 public final class com/datadog/android/core/UploadWorker$Companion {
 }
 
-public final class com/datadog/android/core/collections/EvictingQueue : java/util/LinkedList {
-	public fun <init> ()V
-	public fun <init> (I)V
-	public synthetic fun <init> (IILkotlin/jvm/internal/DefaultConstructorMarker;)V
-	public fun add (ILjava/lang/Object;)V
-	public fun add (Ljava/lang/Object;)Z
-	public fun addAll (ILjava/util/Collection;)Z
-	public fun addAll (Ljava/util/Collection;)Z
-	public fun getSize ()I
-	public final fun remove (I)Ljava/lang/Object;
-	public fun removeAt (I)Ljava/lang/Object;
-	public final fun size ()I
-}
-
 public final class com/datadog/android/core/configuration/BackPressureMitigation : java/lang/Enum {
 	public static final field DROP_OLDEST Lcom/datadog/android/core/configuration/BackPressureMitigation;
 	public static final field IGNORE_NEWEST Lcom/datadog/android/core/configuration/BackPressureMitigation;

features/dd-sdk-android-rum/src/test/kotlin/com/datadog/android/rum/assertj/ViewEventAssert.kt

@@ -589,12 +589,24 @@ internal class ViewEventAssert(actual: ViewEvent) :
 
     fun hasSlownessInfo(
         slowFrames: List<ViewEvent.SlowFrame>,
-        slowFrameRate: Double,
+        slowFramesRate: Double,
         freezeRate: Double
     ): ViewEventAssert = apply {
-        assertThat(actual.view.slowFrames).isEqualTo(slowFrames)
-        assertThat(actual.view.slowFramesRate).isEqualTo(slowFrameRate)
-        assertThat(actual.view.freezeRate).isEqualTo(freezeRate)
+        assertThat(actual.view.slowFrames)
+            .overridingErrorMessage(
+                "Expected event data to have slowFrames $slowFrames but was ${actual.view.slowFrames}"
+            )
+            .isEqualTo(slowFrames)
+        assertThat(actual.view.slowFramesRate)
+            .overridingErrorMessage(
+                "Expected event data to have slowFramesRate $slowFramesRate but was ${actual.view.slowFramesRate}"
+            )
+            .isEqualTo(slowFramesRate)
+        assertThat(actual.view.freezeRate)
+            .overridingErrorMessage(
+                "Expected event data to have freezeRate $freezeRate but was ${actual.view.freezeRate}"
+            )
+            .isEqualTo(freezeRate)
     }
 
     fun hasOsInfo(