Setup sonar for core

Author: aleksandar-apostolov

build.gradle.kts

@@ -1,6 +1,6 @@
 import java.io.FileNotFoundException
 import java.util.Calendar
-
+apply(from = "${rootDir}/gradle/scripts/sonar.gradle")
 // Top-level build file where you can add configuration options common to all sub-projects/modules.
 plugins {
     alias(libs.plugins.android.application) apply false
@@ -11,6 +11,8 @@ plugins {
     alias(libs.plugins.ksp) apply false
     alias(libs.plugins.arturbosch.detekt) apply true
     alias(libs.plugins.spotless) apply true
+    alias(libs.plugins.sonarqube) apply true
+    alias(libs.plugins.kover) apply true
 }
 
 spotless {
@@ -32,6 +34,7 @@ detekt {
 
 // License tasks
 subprojects {
+    apply(from = "${rootDir}/gradle/scripts/coverage.gradle")
     tasks.register("generateLicense") {
         val currentYear = Calendar.getInstance().get(Calendar.YEAR).toString()
         val licenseTemplate = file("../config/license/license.template")

gradle/libs.versions.toml

@@ -23,7 +23,8 @@ ksp = "2.2.0-2.0.2"
 robolectric = "4.15.1"
 detekt = "1.23.8"
 spotless = "7.2.1"
-klint = "13.0.0"
+kover = "0.9.1"
+sonarqube = "6.3.0.5676"
 
 
 [libraries]
@@ -72,4 +73,6 @@ kotlin-compose = { id = "org.jetbrains.kotlin.plugin.compose", version.ref = "ko
 ksp = { id = "com.google.devtools.ksp", version.ref = "ksp" }
 arturbosch-detekt = { id = "io.gitlab.arturbosch.detekt", version.ref = "detekt" }
 spotless = { id = "com.diffplug.spotless", version.ref = "spotless" }
+kover = { id = "org.jetbrains.kotlinx.kover", version.ref = "kover"}
+sonarqube = { id = "org.sonarqube", version.ref = "sonarqube"}
 

gradle/scripts/coverage.gradle

@@ -0,0 +1,32 @@
+if (!rootProject.ext.sonar.ignoreModules.contains(name)) {
+    apply plugin: "org.jetbrains.kotlinx.kover"
+    apply plugin: "org.sonarqube"
+
+    if (hasProperty('android')) {
+        android {
+            buildTypes {
+                debug {
+                    testCoverageEnabled = true
+                    enableUnitTestCoverage = true
+                    enableAndroidTestCoverage true
+                }
+            }
+        }
+    }
+
+    kover {
+        reports {
+            verify {
+                warningInsteadOfFailure = true
+            }
+        }
+    }
+
+    sonarqube {
+        properties {
+            property "sonar.junit.reportPaths", "./build/test-results/testDebugUnitTest"
+            property "sonar.coverage.jacoco.xmlReportPaths", "./build/reports/kover/reportDebug.xml"
+            property "sonar.sources", "src/main/java"
+        }
+    }
+}

gradle/scripts/sonar.gradle

@@ -0,0 +1,37 @@
+apply plugin: "org.sonarqube"
+
+ext.sonar = [
+    ignoreModules : [
+        'stream-android-core-lint',
+        'stream-android-core-annotations',
+        'app'
+    ],
+    excludeFilter : [
+        '**/test/**',
+        '**/androidTest/**',
+        '**/R.class',
+        '**/R2.class',
+        '**/R$*.class',
+        '**/BuildConfig.*',
+        '**/Manifest*.*',
+        '**/*Test*.*'
+    ]
+]
+
+ext.sonar.ignoreModules.each {
+    ext.sonar.excludeFilter << "**/${it}/**"
+}
+
+sonarqube {
+    properties {
+        property("sonar.host.url", "https://sonarcloud.io")
+        property("sonar.token", "${System.getenv("SONAR_TOKEN")}")
+        property("sonar.organization", "getstream")
+        property("sonar.projectKey", "GetStream_stream-core-android")
+        property("sonar.projectName", "stream-core-android")
+        property "sonar.java.coveragePlugin", "jacoco"
+        property "sonar.sourceEncoding", "UTF-8"
+        property "sonar.java.binaries", "${rootDir}/**/build/tmp/java-classes/debug"
+        property "sonar.coverage.exclusions", rootProject.ext.sonar.excludeFilter
+    }
+}

stream-android-core/build.gradle.kts

@@ -5,6 +5,8 @@ plugins {
     alias(libs.plugins.kotlin.android)
     alias(libs.plugins.ksp)
     alias(libs.plugins.arturbosch.detekt)
+    alias(libs.plugins.sonarqube)
+    alias(libs.plugins.kover)
 }
 
 kotlin {

stream-android-core/src/main/java/io/getstream/android/core/api/model/config/StreamHttpConfig.kt

@@ -15,6 +15,7 @@
  */
 package io.getstream.android.core.api.model.config
 
+import io.getstream.android.core.annotations.StreamCoreApi
 import okhttp3.Interceptor
 import okhttp3.OkHttpClient
 
@@ -25,6 +26,7 @@ import okhttp3.OkHttpClient
  * @param automaticInterceptors Whether to add automatic interceptors.
  * @param configuredInterceptors The configured interceptors.
  */
+@StreamCoreApi
 data class StreamHttpConfig(
     val httpBuilder: OkHttpClient.Builder,
     val automaticInterceptors: Boolean = true,

stream-android-core/src/main/java/io/getstream/android/core/internal/subscribe/StreamSubscriptionManagerImpl.kt

@@ -117,7 +117,7 @@ internal class StreamSubscriptionManagerImpl<T>(
                 object : StreamSubscription {
                     override fun cancel() {
                         val key = keyRef.get() ?: return
-                        synchronized(weakSubscribers) { weakSubscribers.remove(key, this) }
+                        synchronized(weakSubscribers) { weakSubscribers.remove(key) }
                     }
                 }
             weakSubscribers[listener] = handle