Tweak Kover: improve coverage, disable deprecations, add missing test in slf4j

Addresses #3156

Author: qwwdfsad

buildSrc/src/main/kotlin/kover-conventions.gradle.kts

@@ -16,10 +16,9 @@ val expectedCoverage = mutableMapOf(
 
     // TODO figure it out, these probably should be fixed
     "kotlinx-coroutines-debug" to 84,
-    "kotlinx-coroutines-reactive" to 65,
+    // Re-evaluate this along with Kover update where deprecated with error+ functions are not considered as uncovered: IDEA-287459
     "kotlinx-coroutines-reactor" to 65,
-    "kotlinx-coroutines-rx2" to 78,
-    "kotlinx-coroutines-slf4j" to 81
+    "kotlinx-coroutines-rx2" to 78
 )
 
 extensions.configure<KoverExtension> {
@@ -51,4 +50,8 @@ subprojects {
             }
         }
     }
+
+    tasks.withType<KoverHtmlReportTask> {
+        htmlReportDir.set(file(rootProject.buildDir.toString() + "/kover/" + project.name + "/html"))
+    }
 }

integration/kotlinx-coroutines-slf4j/test/MDCContextTest.kt

@@ -102,9 +102,10 @@ class MDCContextTest : TestBase() {
         val mainDispatcher = kotlin.coroutines.coroutineContext[ContinuationInterceptor]!!
         withContext(Dispatchers.Default + MDCContext()) {
             assertEquals("myValue", MDC.get("myKey"))
+            assertEquals("myValue", coroutineContext[MDCContext]?.contextMap?.get("myKey"))
             withContext(mainDispatcher) {
                 assertEquals("myValue", MDC.get("myKey"))
             }
         }
     }
-}
+}

reactive/kotlinx-coroutines-reactive/build.gradle.kts

@@ -34,3 +34,17 @@ tasks.check {
 externalDocumentationLink(
     url = "https://www.reactive-streams.org/reactive-streams-$reactiveStreamsVersion-javadoc/"
 )
+
+val commonKoverExcludes = listOf(
+    "kotlinx.coroutines.reactive.FlowKt", // Deprecated
+    "kotlinx.coroutines.reactive.FlowKt__MigrationKt", // Deprecated
+    "kotlinx.coroutines.reactive.ConvertKt" // Deprecated
+)
+
+tasks.koverHtmlReport {
+    excludes = commonKoverExcludes
+}
+
+tasks.koverVerify {
+    excludes = commonKoverExcludes
+}

reactive/kotlinx-coroutines-reactor/build.gradle.kts

@@ -27,3 +27,15 @@ tasks {
 externalDocumentationLink(
     url = "https://projectreactor.io/docs/core/$reactorVersion/api/"
 )
+
+val commonKoverExcludes = listOf(
+    "kotlinx.coroutines.reactor.FlowKt" // Deprecated
+)
+
+tasks.koverHtmlReport {
+    excludes = commonKoverExcludes
+}
+
+tasks.koverVerify {
+    excludes = commonKoverExcludes
+}