RUM-9899: adding missing tests for new methods

Author: satween

dd-sdk-android-internal/src/main/java/com/datadog/android/internal/utils/CastExt.kt

@@ -17,10 +17,10 @@ package com.datadog.android.internal.utils
  * ```
  * now could be written as:
  * ```kotlin
-    val writer = featuredSdkCore.getFeature(Feature.TRACING_FEATURE_NAME)
-        ?.unwrap<Feature>()
-        ?.tryCastTo<com.datadog.android.trace.InternalCoreWriterProvider>()
-        ?.getCoreTracerWriter()
+ val writer = featuredSdkCore.getFeature(Feature.TRACING_FEATURE_NAME)
+ ?.unwrap<Feature>()
+ ?.tryCastTo<com.datadog.android.trace.InternalCoreWriterProvider>()
+ ?.getCoreTracerWriter()
  *  ```
  */
 inline fun <reified R> Any.tryCastTo(): R? {

features/dd-sdk-android-trace-internal/src/test/kotlin/com/datadog/trace/core/CoreSpanBuilderTest.kt

@@ -92,13 +92,15 @@ internal class CoreSpanBuilderTest : DDCoreSpecification() {
         val expectedResource = "fakeResource"
         val expectedService = "fakeService"
         val expectedType = "fakeType"
+        val expectedOrigin = "fakeOrigin"
 
         // When
         val thirdSpan = tracer
             .buildSpan(instrumentationName, expectedName)
             .withServiceName("foo")
             .withResourceName(expectedResource)
             .withServiceName(expectedService)
+            .withOrigin(expectedOrigin)
             .withErrorFlag()
             .withSpanType(expectedType)
             .start()
@@ -110,6 +112,7 @@ internal class CoreSpanBuilderTest : DDCoreSpecification() {
         assertThat(context.errorFlag).isTrue
         assertThat(context.serviceName).isEqualTo(expectedService)
         assertThat(context.spanType).isEqualTo(expectedType)
+        assertThat(context.origin).isEqualTo(expectedOrigin)
     }
 
     @ParameterizedTest

features/dd-sdk-android-trace-internal/src/test/kotlin/com/datadog/trace/core/DDSpanTest.kt

@@ -30,6 +30,8 @@ import org.junit.jupiter.api.Test
 import org.junit.jupiter.params.ParameterizedTest
 import org.junit.jupiter.params.provider.Arguments
 import org.junit.jupiter.params.provider.MethodSource
+import org.mockito.kotlin.any
+import org.mockito.kotlin.doReturn
 import org.mockito.kotlin.mock
 import org.mockito.kotlin.times
 import org.mockito.kotlin.verify
@@ -510,6 +512,27 @@ internal class DDSpanTest : DDCoreSpecification() {
         assertThat(span.samplingPriority()).isEqualTo(PrioritySampling.UNSET.toInt())
     }
 
+    @Test
+    fun `W drop T unregisterSpan is called`() {
+        val mockTrace = mock<PendingTrace> {
+            on { rootSpan } doReturn mock()
+        }
+
+        val mockContext = mock<DDSpanContext> {
+            on { traceId } doReturn mock<DDTraceId>()
+            on { trace } doReturn mockTrace
+        }
+
+        val span = tracer
+            .buildSpan(instrumentationName, "testSpan")
+            .asChildOf(mockContext)
+            .start() as DDSpan
+
+        span.drop()
+
+        verify(mockTrace).unregisterSpan(any())
+    }
+
     @Test
     fun `error priorities should be respected`() {
         // Given

integrations/dd-sdk-android-okhttp/src/main/kotlin/com/datadog/android/okhttp/trace/TracingInterceptor.kt

@@ -853,7 +853,7 @@ internal constructor(
         private const val AGENT_PSR_ATTRIBUTE = "_dd.agent_psr"
         private const val WRITER_PROVIDER_INTERFACE_NOT_IMPLEMENTED_ERROR_MESSAGE =
             "The Tracing feature is not implementing the InternalCoreWriterProvider interface." +
-                    " No tracing data will be sent."
+                " No tracing data will be sent."
 
         private val DEFAULT_LOCAL_TRACER_FACTORY: (SdkCore, Set<TracingHeaderType>) -> AgentTracer.TracerAPI =
             { sdkCore, tracingHeaderTypes: Set<TracingHeaderType> ->