Merge bff4562 into f55020d

Author: romtsn

sentry-android-timber/src/main/java/io/sentry/android/timber/SentryTimberIntegration.kt

@@ -22,7 +22,8 @@ class SentryTimberIntegration(
     private lateinit var logger: ILogger
 
     override fun register(hub: IHub, options: SentryOptions) {
-        createSdkVersion(options)
+        options.sdkVersion?.addPackage("maven:io.sentry:sentry-android-timber", VERSION_NAME)
+
         logger = options.logger
 
         tree = SentryTimberTree(hub, minEventLevel, minBreadcrumbLevel)
@@ -40,16 +41,4 @@ class SentryTimberIntegration(
             }
         }
     }
-
-    private fun createSdkVersion(options: SentryOptions): SdkVersion {
-        var sdkVersion = options.sdkVersion
-
-        val name = SENTRY_TIMBER_SDK_NAME
-        val version = VERSION_NAME
-        sdkVersion = SdkVersion.updateSdkVersion(sdkVersion, name, version)
-
-        sdkVersion.addPackage("maven:io.sentry:sentry-android-timber", VERSION_NAME)
-
-        return sdkVersion
-    }
 }

sentry-android-timber/src/test/java/io/sentry/android/timber/SentryTimberIntegrationTest.kt

@@ -97,20 +97,9 @@ class SentryTimberIntegrationTest {
 
         assertTrue(
             fixture.options.sdkVersion!!.packages!!.any {
-                it.name == "maven:io.sentry:sentry-android-timber"
-                it.version == BuildConfig.VERSION_NAME
+                it.name == "maven:io.sentry:sentry-android-timber" &&
+                    it.version == BuildConfig.VERSION_NAME
             }
         )
     }
-
-    @Test
-    fun `Integration sets SDK name and version to options`() {
-        val sut = fixture.getSut()
-        sut.register(fixture.hub, fixture.options)
-
-        val sdkVersion = fixture.options.sdkVersion!!
-
-        assertEquals(sdkVersion.name, "sentry.java.android.timber")
-        assertEquals(sdkVersion.version, BuildConfig.VERSION_NAME)
-    }
 }