Integration test step in Train (#3486)

* Set up Integration Tests step in Train

* Run debugProbesTest with -Poverwrite.probes=true

* Update DebugProbesKt.bin

* Fixes for integration-testing README

* Update README

* Migrating integration-testing/build.gradle to kts WIP

* Revert "Migrating integration-testing/build.gradle to kts WIP"

This reverts commit fbf966e

* Add space repo only if build_snapshot_train

* Fixup

Co-authored-by: Margarita Bobova <[email protected]>
Co-authored-by: Vsevolod Tolstopyatov <[email protected]>

Author: 3 people

integration-testing/README.md

@@ -3,11 +3,13 @@
 This is a supplementary project that provides integration tests.
 
 The tests are the following:
-* `MavenPublicationValidator` depends on the published artifacts and tests artifacts binary content and absence of atomicfu in the classpath.
-* `CoreAgentTest` checks that `kotlinx-coroutines-core` can be run as a Java agent.
-* `DebugAgentTest` checks that the coroutine debugger can be run as a Java agent.
-* `smokeTest` builds the test project that depends on coroutines.
+* `mavenTest` depends on the published artifacts and tests artifacts binary content for absence of atomicfu in the classpath.
+* `jvmCoreTest` miscellaneous tests that check the behaviour of `kotlinx-coroutines-core` dependency in a smoke manner.
+* `coreAgentTest` checks that `kotlinx-coroutines-core` can be run as a Java agent.
+* `debugAgentTest` checks that the coroutine debugger can be run as a Java agent.
+* `debugDynamicAgentTest` checks that `kotlinx-coroutines-debug` agent can self-attach dynamically to JVM as a standalone dependency.
+* `smokeTest` builds the multiplatform test project that depends on coroutines.
 
 The `integration-testing` project is expected to be in a subdirectory of the main `kotlinx.coroutines` project.
 
-To run all the available tests: `cd integration-testing` + `./gradlew check`.
+To run all the available tests: `./gradlew publishToMavenLocal` + `cd integration-testing` + `./gradlew check`.

integration-testing/build.gradle

@@ -2,11 +2,54 @@
  * Copyright 2016-2021 JetBrains s.r.o. Use of this source code is governed by the Apache 2.0 license.
  */
 
+
+buildscript {
+
+    /*
+     * These property group is used to build kotlinx.coroutines against Kotlin compiler snapshot.
+     * How does it work:
+     * When build_snapshot_train is set to true, kotlin_version property is overridden with kotlin_snapshot_version,
+     * atomicfu_version is overwritten by TeamCity environment (AFU is built with snapshot and published to mavenLocal
+     * as previous step or the snapshot build).
+     * Additionally, mavenLocal and Sonatype snapshots are added to repository list and stress tests are disabled.
+     * DO NOT change the name of these properties without adapting kotlinx.train build chain.
+     */
+    def prop = rootProject.properties['build_snapshot_train']
+    ext.build_snapshot_train = prop != null && prop != ""
+    if (build_snapshot_train) {
+        ext.kotlin_version = rootProject.properties['kotlin_snapshot_version']
+        if (kotlin_version == null) {
+            throw new IllegalArgumentException("'kotlin_snapshot_version' should be defined when building with snapshot compiler")
+        }
+    }
+    ext.native_targets_enabled = rootProject.properties['disable_native_targets'] == null
+
+    // Determine if any project dependency is using a snapshot version
+    ext.using_snapshot_version = build_snapshot_train
+    rootProject.properties.each { key, value ->
+        if (key.endsWith("_version") && value instanceof String && value.endsWith("-SNAPSHOT")) {
+            println("NOTE: USING SNAPSHOT VERSION: $key=$value")
+            ext.using_snapshot_version = true
+        }
+    }
+
+    if (using_snapshot_version) {
+        repositories {
+            mavenLocal()
+            maven { url "https://maven.pkg.jetbrains.space/kotlin/p/kotlin/dev" }
+        }
+    }
+
+}
+
 plugins {
     id "org.jetbrains.kotlin.jvm" version "$kotlin_version"
 }
 
 repositories {
+    if (build_snapshot_train) {
+        maven { url "https://maven.pkg.jetbrains.space/kotlin/p/kotlin/dev" }
+    }
     mavenLocal()
     mavenCentral()
 }
@@ -19,6 +62,7 @@ java {
 dependencies {
     testImplementation "org.jetbrains.kotlin:kotlin-test:$kotlin_version"
     testImplementation "org.ow2.asm:asm:$asm_version"
+    implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk8"
 }
 
 sourceSets {
@@ -56,7 +100,7 @@ sourceSets {
         }
     }
 
-    // Checks that kotlinx-coroutines-debug agent can self-attach dynamically to JVM as standalone dependency
+    // Checks that kotlinx-coroutines-debug agent can self-attach dynamically to JVM as a standalone dependency
     debugDynamicAgentTest {
         kotlin
         compileClasspath += sourceSets.test.runtimeClasspath
@@ -130,3 +174,8 @@ compileTestKotlin {
 check {
     dependsOn([jvmCoreTest, debugDynamicAgentTest, mavenTest, debugAgentTest, coreAgentTest, 'smokeTest:build'])
 }
+compileKotlin {
+    kotlinOptions {
+        jvmTarget = "1.8"
+    }
+}

integration-testing/gradle.properties

@@ -3,3 +3,4 @@ coroutines_version=1.6.4-SNAPSHOT
 asm_version=9.3
 
 kotlin.code.style=official
+kotlin.mpp.stability.nowarn=true

integration-testing/settings.gradle

@@ -2,6 +2,7 @@ pluginManagement {
     repositories {
         mavenCentral()
         maven { url "https://plugins.gradle.org/m2/" }
+        maven { url "https://maven.pkg.jetbrains.space/kotlin/p/kotlin/dev" }
         mavenLocal()
     }
 }

integration-testing/smokeTest/build.gradle

@@ -6,6 +6,7 @@ repositories {
     // Coroutines from the outer project are published by previous CI buils step
     mavenLocal()
     mavenCentral()
+    maven { url "https://maven.pkg.jetbrains.space/kotlin/p/kotlin/dev" }
 }
 
 kotlin {
@@ -40,4 +41,12 @@ kotlin {
             }
         }
     }
+    targets {
+        configure([]) {
+            tasks.getByName(compilations.main.compileKotlinTaskName).kotlinOptions {
+                jvmTarget = "1.8"
+            }
+        }
+    }
 }
+