Skip to content

Commit

Permalink
Work around #3968, start running integration tests for Native (#3996)
Browse files Browse the repository at this point in the history
As there were no native targets in smokeTest, the problem with the missing atomicfu dependency (KT-64111) was not addressed in time. The bug was revealed after  release of kotlinx.coroutines with atomicfu-gradle-plugin 0.23.1. Coroutines enabled Native IR transformation, and for this mode, atomicfu-gradle-plugin only provided compile dependency to atomicfu, though implementation dependency was still necessary.

This commit adds the smoke test for Native and temporarily adds an explicit dependency on atomicfu for Native artifacts.

Fixes #3968

Co-authored-by: Maria Sokolova <maria.sokolova@jetbrains.com>
  • Loading branch information
dkhalanskyjb and mvicsokolova committed Dec 21, 2023
1 parent c8105d1 commit 11dc96f
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 0 deletions.
Expand Up @@ -88,6 +88,10 @@ kotlin {
api("org.jetbrains.kotlin:kotlin-test-junit:${version("kotlin")}")
api("junit:junit:${version("junit")}")
}
nativeMain.dependencies {
// workaround for #3968 until this is fixed on atomicfu's side
api("org.jetbrains.kotlinx:atomicfu:0.23.1")
}
val jsAndWasmSharedMain by registering {
dependsOn(commonMain.get())
}
Expand Down
6 changes: 6 additions & 0 deletions integration-testing/smokeTest/build.gradle
Expand Up @@ -17,6 +17,12 @@ kotlin {
wasmJs() {
nodejs()
}

macosArm64()
macosX64()
linuxArm64()
linuxX64()
mingwX64()

sourceSets {
commonMain {
Expand Down

0 comments on commit 11dc96f

Please sign in to comment.