Evgeny Mandrikov
Evgeny Mandrikov
@marchof for Java `String.hashCode` collisions I simply add as prefix as many `\u0000` as collisions needed 😉 see https://github.com/jacoco/jacoco/blob/f823d4343ee9039e6f6b40e1cd0c536808a712cb/org.jacoco.core.test.validation.kotlin/src/org/jacoco/core/test/validation/kotlin/targets/KotlinWhenStringTarget.kt#L22-L30 but thanks!
Provided executable reproducer is not complete because can not be compiled - missing imports, etc. I guess that `coEvery` is Mocking, so please check https://stackoverflow.com/questions/46517471/jacoco-reports-zero-coverage-for-mocked-method/46614216#46614216
Here is `build.gradle.kts` that allows to execute provided reproducer ``` plugins { kotlin("jvm") version "1.9.25" jacoco } repositories { mavenCentral() } dependencies { implementation("org.jetbrains.kotlinx:kotlinx-coroutines-core:1.8.1") testImplementation("junit:junit:4.13.2") testImplementation("io.mockk:mockk:1.13.5") testImplementation("org.jetbrains.kotlinx:kotlinx-coroutines-test:1.8.1") } jacoco {...
Cases from https://github.com/jacoco/jacoco/issues/1868#issuecomment-2772022860 and https://github.com/jacoco/jacoco/issues/1868#issuecomment-2774980274 were fixed by https://github.com/jacoco/jacoco/pull/1871, and from https://github.com/jacoco/jacoco/issues/1868#issuecomment-2775314531 fixed by https://github.com/jacoco/jacoco/pull/1929
@qwwdfsad @zuevmaxim FYI this also affects [IntelliJ Coverage engine](https://github.com/JetBrains/intellij-coverage/) embedded into [Kover](https://github.com/Kotlin/kotlinx-kover/) (tested version 0.9.1) and AFAICS without SMAP there is not enough information for both JaCoCo and IntelliJ Coverage...
FWIW this also affects debugger: IntelliJ IDEA 2024.3.1.1 stops at breakpoint on line 5 in `utils.kt` when using Kotlin compiler version 1.9.25 , whereas does not when using 2.1.0
@marchof Personally, I would prefer to not introduce configuration option for this and to not change current character set for filenames. And while I understand described use case, I'm not...
One of the reasons to not add option and to not change filenames is that AFAIK URLs are also restricted to subset of ASCII and strictly speaking UTF-8 outside of...
@lp-smarkwell Thank you for raising this issue, and for sharing your thoughts about this subject! ❤️ Glad that for your needs there is a way to proceed while this issue...