Did something change between RC2 and release in th...

# eap

[Zac Sweers]

Did something change between RC2 and release in the kotlin-bom around tests? RC2 was passing for us on CI, but with the final release I see some esoteric gradle conflicts around the kotlin test libraries. Lmk if I should file a bug

[Ivan Kubyshkin [JetBrains]]

cc @tapchicoma

[tapchicoma]

RC2 release should be the similar to final release. And I don't see any changes in bom file.

Please open an issue with repro

[Zac Sweers]

Will see if I can get a public repro going

[Zac Sweers]

Were any of these artifacts folded due to JDK 8 like the stdlib artifacts were?

[tapchicoma]

afaik no

[Zac Sweers]

Hmmm strange. I’ll keep digging

[tapchicoma]

I've checked .modules files content between the releases and only difference between them is capabilities order 🤔

[Zac Sweers]

super weird, when I output dependencies on CI, I see junit5 coming in from kotlin-test

|    +--- project :libraries:foundation:test-commons
|    |    +--- org.ow2.asm:asm-bom:9.4 (*)
|    |    +--- org.jetbrains.kotlinx:kotlinx-coroutines-bom:1.6.4 (*)
|    |    +--- com.google.firebase:firebase-bom:31.1.1 (*)
|    |    +--- org.jetbrains.kotlin:kotlin-bom:1.8.0 (*)
|    |    +--- com.squareup.okhttp3:okhttp-bom:5.0.0-alpha.10 (*)
|    |    +--- com.squareup.okio:okio-bom:3.3.0 (*)
|    |    +--- com.squareup.wire:wire-bom:4.4.3 (*)
|    |    +--- project :tooling:slack-platform (*)
|    |    +--- junit:junit:4.13.2 (*)
|    |    +--- org.jetbrains.kotlin:kotlin-test:1.8.0
|    |    |    +--- org.jetbrains.kotlin:kotlin-stdlib:1.8.0 (*)
|    |    |    \--- org.jetbrains.kotlin:kotlin-test-junit5:1.8.0 FAILED

[Zac Sweers]

I’m entirely unable to reproduce this locally though

[Zac Sweers]

so somehow on CI, junit5RuntimeVariant is being selected as the capability

[Zac Sweers]

I wonder if there’s a gradle plugin sneakily turning on useJUnitPlatform() in test tasks

[Zac Sweers]

https://twitter.com/fortes/status/399339918213652480?lang=en

[Zac Sweers]

bonus bit to close this out - the reason I didn’t hit this on CI previously was because I was doing most of my initial testing prior to this on github actions, and our final “main” CI is on buildkite. Hence why I didn’t see it before. Will include that in testing future EAPs 😅