Revert "[stdlib-mpp] Disable/ignore failing tests in Kotlin Dev"

This reverts commit 171d156688.

KT-60901
This commit is contained in:
Sebastian Sellmair
2023-08-09 09:59:59 +02:00
committed by Space Team
parent 2b5f839fac
commit 243707c956
12 changed files with 0 additions and 32 deletions
@@ -17,7 +17,6 @@ import org.jetbrains.kotlin.gradle.plugin.sources.DefaultKotlinSourceSet
import org.jetbrains.kotlin.gradle.testbase.* import org.jetbrains.kotlin.gradle.testbase.*
import org.jetbrains.kotlin.gradle.util.checkedReplace import org.jetbrains.kotlin.gradle.util.checkedReplace
import org.jetbrains.kotlin.gradle.util.modify import org.jetbrains.kotlin.gradle.util.modify
import org.junit.jupiter.api.Disabled
import org.junit.jupiter.api.DisplayName import org.junit.jupiter.api.DisplayName
import org.junit.jupiter.api.condition.OS import org.junit.jupiter.api.condition.OS
import org.junit.jupiter.api.io.TempDir import org.junit.jupiter.api.io.TempDir
@@ -37,7 +36,6 @@ open class HierarchicalMppIT : KGPBaseTest() {
private val String.withPrefix get() = "hierarchical-mpp-published-modules/$this" private val String.withPrefix get() = "hierarchical-mpp-published-modules/$this"
@GradleTest @GradleTest
@Disabled("stdlib publication migration - test data changes")
@DisplayName("Check build with published third-party library") @DisplayName("Check build with published third-party library")
fun testPublishedModules(gradleVersion: GradleVersion, @TempDir tempDir: Path) { fun testPublishedModules(gradleVersion: GradleVersion, @TempDir tempDir: Path) {
val buildOptions = defaultBuildOptions.copy(logLevel = LogLevel.DEBUG) val buildOptions = defaultBuildOptions.copy(logLevel = LogLevel.DEBUG)
@@ -200,7 +198,6 @@ open class HierarchicalMppIT : KGPBaseTest() {
} }
@GradleTest @GradleTest
@Disabled("stdlib publication migration - test data changes")
@DisplayName("Dependencies in project should be correct with third-party library") @DisplayName("Dependencies in project should be correct with third-party library")
fun testProjectDependencies(gradleVersion: GradleVersion, @TempDir tempDir: Path) { fun testProjectDependencies(gradleVersion: GradleVersion, @TempDir tempDir: Path) {
publishThirdPartyLib(withGranularMetadata = false, gradleVersion = gradleVersion, localRepoDir = tempDir) publishThirdPartyLib(withGranularMetadata = false, gradleVersion = gradleVersion, localRepoDir = tempDir)
@@ -222,7 +219,6 @@ open class HierarchicalMppIT : KGPBaseTest() {
} }
@GradleTest @GradleTest
@Disabled("stdlib publication migration - test data changes")
@DisplayName("Check that only composite metadata artifacts are transformed") @DisplayName("Check that only composite metadata artifacts are transformed")
fun testOnlyCompositeMetadataArtifactsTransformed(gradleVersion: GradleVersion, @TempDir tempDir: Path) { fun testOnlyCompositeMetadataArtifactsTransformed(gradleVersion: GradleVersion, @TempDir tempDir: Path) {
val buildOptions = defaultBuildOptions.copy(logLevel = LogLevel.DEBUG) val buildOptions = defaultBuildOptions.copy(logLevel = LogLevel.DEBUG)
@@ -1028,7 +1024,6 @@ open class HierarchicalMppIT : KGPBaseTest() {
} }
@GradleTest @GradleTest
@Disabled("stdlib publication migration")
@DisplayName("KT-46417: [UNRESOLVED_REFERENCE] For project to project dependencies of native platform test source sets") @DisplayName("KT-46417: [UNRESOLVED_REFERENCE] For project to project dependencies of native platform test source sets")
fun testNativeLeafTestSourceSetsKt46417(gradleVersion: GradleVersion) { fun testNativeLeafTestSourceSetsKt46417(gradleVersion: GradleVersion) {
with(project("kt-46417-ios-test-source-sets", gradleVersion = gradleVersion)) { with(project("kt-46417-ios-test-source-sets", gradleVersion = gradleVersion)) {
@@ -7,7 +7,6 @@ package org.jetbrains.kotlin.gradle
import org.gradle.util.GradleVersion import org.gradle.util.GradleVersion
import org.jetbrains.kotlin.gradle.util.* import org.jetbrains.kotlin.gradle.util.*
import org.junit.Ignore
import org.junit.Test import org.junit.Test
import kotlin.test.assertTrue import kotlin.test.assertTrue
@@ -194,7 +193,6 @@ class VariantAwareDependenciesMppIT : BaseGradleIT() {
} }
@Test @Test
@Ignore("stdlib publication migration")
fun testResolvesOldKotlinArtifactsPublishedWithMetadata() = with(Project("multiplatformProject", gradleVersion)) { fun testResolvesOldKotlinArtifactsPublishedWithMetadata() = with(Project("multiplatformProject", gradleVersion)) {
setupWorkingDir() setupWorkingDir()
gradleBuildScript().appendText( gradleBuildScript().appendText(
@@ -13,7 +13,6 @@ import org.jetbrains.kotlin.gradle.idea.tcs.IdeaKotlinSourceDependency.Type.Regu
import org.jetbrains.kotlin.gradle.idea.testFixtures.tcs.* import org.jetbrains.kotlin.gradle.idea.testFixtures.tcs.*
import org.jetbrains.kotlin.gradle.testbase.* import org.jetbrains.kotlin.gradle.testbase.*
import org.jetbrains.kotlin.gradle.util.* import org.jetbrains.kotlin.gradle.util.*
import org.junit.jupiter.api.Disabled
import org.junit.jupiter.api.DisplayName import org.junit.jupiter.api.DisplayName
import org.junit.jupiter.api.condition.OS import org.junit.jupiter.api.condition.OS
import kotlin.test.assertEquals import kotlin.test.assertEquals
@@ -28,7 +27,6 @@ import kotlin.test.assertIs
@GradleTestVersions(maxVersion = TestVersions.Gradle.G_8_2) @GradleTestVersions(maxVersion = TestVersions.Gradle.G_8_2)
class MppCompositeBuildIT : KGPBaseTest() { class MppCompositeBuildIT : KGPBaseTest() {
@GradleTest @GradleTest
@Disabled("stdlib publication migration - test data changes")
fun `test - sample0 - ide dependencies`(gradleVersion: GradleVersion) { fun `test - sample0 - ide dependencies`(gradleVersion: GradleVersion) {
val producer = project("mpp-composite-build/sample0/producerBuild", gradleVersion) val producer = project("mpp-composite-build/sample0/producerBuild", gradleVersion)
@@ -316,7 +314,6 @@ class MppCompositeBuildIT : KGPBaseTest() {
} }
@GradleTest @GradleTest
@Disabled("stdlib publication migration - test data changes")
fun `test - sample5-KT-56536-rootProject_name - ide dependencies`(gradleVersion: GradleVersion) { fun `test - sample5-KT-56536-rootProject_name - ide dependencies`(gradleVersion: GradleVersion) {
val producer = project("mpp-composite-build/sample5-KT-56536-rootProject.name/producerBuild", gradleVersion) val producer = project("mpp-composite-build/sample5-KT-56536-rootProject.name/producerBuild", gradleVersion)
@@ -11,7 +11,6 @@ import org.jetbrains.kotlin.konan.properties.hasProperty
import org.jetbrains.kotlin.library.KLIB_PROPERTY_DEPENDS import org.jetbrains.kotlin.library.KLIB_PROPERTY_DEPENDS
import org.jetbrains.kotlin.library.ToolingSingleFileKlibResolveStrategy import org.jetbrains.kotlin.library.ToolingSingleFileKlibResolveStrategy
import org.jetbrains.kotlin.library.unresolvedDependencies import org.jetbrains.kotlin.library.unresolvedDependencies
import org.junit.jupiter.api.Disabled
import org.junit.jupiter.api.DisplayName import org.junit.jupiter.api.DisplayName
import org.junit.jupiter.api.fail import org.junit.jupiter.api.fail
@@ -52,7 +51,6 @@ class MppSharedNativeCompileIT : KGPBaseTest() {
* *
*/ */
@GradleTest @GradleTest
@Disabled("stdlib publication migration")
fun `test - K2 - shared native compilation - assemble`(gradleVersion: GradleVersion) { fun `test - K2 - shared native compilation - assemble`(gradleVersion: GradleVersion) {
project("kt-57944-k2-native-compilation", gradleVersion, buildOptions = defaultBuildOptions.copy(languageVersion = "2.0")) { project("kt-57944-k2-native-compilation", gradleVersion, buildOptions = defaultBuildOptions.copy(languageVersion = "2.0")) {
build("assemble") { build("assemble") {
@@ -7,7 +7,6 @@ package org.jetbrains.kotlin.gradle.native
import org.gradle.util.GradleVersion import org.gradle.util.GradleVersion
import org.jetbrains.kotlin.gradle.testbase.* import org.jetbrains.kotlin.gradle.testbase.*
import org.junit.jupiter.api.Disabled
import org.junit.jupiter.api.DisplayName import org.junit.jupiter.api.DisplayName
import org.junit.jupiter.api.condition.DisabledOnOs import org.junit.jupiter.api.condition.DisabledOnOs
import org.junit.jupiter.api.condition.OS import org.junit.jupiter.api.condition.OS
@@ -23,7 +22,6 @@ import kotlin.test.assertNull
internal class NativeExternalDependenciesIT : KGPBaseTest() { internal class NativeExternalDependenciesIT : KGPBaseTest() {
@DisplayName("K/N shouldn't contain any external dependencies by default") @DisplayName("K/N shouldn't contain any external dependencies by default")
@Disabled("stdlib publication migration - test data changes")
@GradleTest @GradleTest
fun shouldNotUseExternalDependencies(gradleVersion: GradleVersion) { fun shouldNotUseExternalDependencies(gradleVersion: GradleVersion) {
buildProjectWithDependencies(gradleVersion) { externalDependenciesText -> buildProjectWithDependencies(gradleVersion) { externalDependenciesText ->
@@ -33,7 +31,6 @@ internal class NativeExternalDependenciesIT : KGPBaseTest() {
@DisplayName("Should build with ktor 1.5.4 and coroutines 1.5.0-RC-native-mt") @DisplayName("Should build with ktor 1.5.4 and coroutines 1.5.0-RC-native-mt")
@GradleTest @GradleTest
@Disabled("stdlib publication migration - test data changes")
@DisabledOnOs( @DisabledOnOs(
value = [OS.MAC], value = [OS.MAC],
architectures = ["aarch64"], architectures = ["aarch64"],
@@ -68,7 +65,6 @@ internal class NativeExternalDependenciesIT : KGPBaseTest() {
} }
@DisplayName("Should build with ktor 1.6.5 and coroutines 1.5.2-native-mt") @DisplayName("Should build with ktor 1.6.5 and coroutines 1.5.2-native-mt")
@Disabled("stdlib publication migration - test data changes")
@GradleTest @GradleTest
fun shouldUseKtorAndCoroutinesExternalDependencies(gradleVersion: GradleVersion) { fun shouldUseKtorAndCoroutinesExternalDependencies(gradleVersion: GradleVersion) {
buildProjectWithDependencies( buildProjectWithDependencies(
@@ -122,7 +122,6 @@ internal class NativeIrLinkerIssuesIT : KGPBaseTest() {
@DisplayName("KT-41378: declaration that is gone - with cache") @DisplayName("KT-41378: declaration that is gone - with cache")
@GradleTest @GradleTest
@Disabled("stdlib publication migration - test data changes")
@DisabledOnOs(OS.WINDOWS, disabledReason = "Don't run it on Windows. Caches are not supported there yet.") @DisabledOnOs(OS.WINDOWS, disabledReason = "Don't run it on Windows. Caches are not supported there yet.")
fun shouldBuildIrLinkerWithCache( fun shouldBuildIrLinkerWithCache(
gradleVersion: GradleVersion, gradleVersion: GradleVersion,
@@ -157,7 +156,6 @@ internal class NativeIrLinkerIssuesIT : KGPBaseTest() {
} }
@DisplayName("KT-41378: declaration that is gone - without cache") @DisplayName("KT-41378: declaration that is gone - without cache")
@Disabled("stdlib publication migration - test data changes")
@GradleTest @GradleTest
fun shouldBuildIrLinkerWithoutCache( fun shouldBuildIrLinkerWithoutCache(
gradleVersion: GradleVersion, gradleVersion: GradleVersion,
@@ -194,7 +192,6 @@ internal class NativeIrLinkerIssuesIT : KGPBaseTest() {
@DisplayName("KT-47285: symbol type mismatch - with cache") @DisplayName("KT-47285: symbol type mismatch - with cache")
@GradleTest @GradleTest
@Disabled("stdlib publication migration - test data changes")
@OsCondition(supportedOn = [OS.MAC, OS.LINUX], enabledOnCI = [OS.LINUX]) @OsCondition(supportedOn = [OS.MAC, OS.LINUX], enabledOnCI = [OS.LINUX])
// Don't run it on Windows. Caches are not supported there yet. // Don't run it on Windows. Caches are not supported there yet.
fun shouldBuildIrLinkerSymbolTypeMismatchWithCache( fun shouldBuildIrLinkerSymbolTypeMismatchWithCache(
@@ -230,7 +227,6 @@ internal class NativeIrLinkerIssuesIT : KGPBaseTest() {
} }
@DisplayName("KT-47285: symbol type mismatch - without cache") @DisplayName("KT-47285: symbol type mismatch - without cache")
@Disabled("stdlib publication migration - test data changes")
@GradleTest @GradleTest
fun shouldBuildIrLinkerSymbolTypeMismatchWithoutCache( fun shouldBuildIrLinkerSymbolTypeMismatchWithoutCache(
gradleVersion: GradleVersion, gradleVersion: GradleVersion,
@@ -19,7 +19,6 @@ import org.jetbrains.kotlin.gradle.util.applyMultiplatformPlugin
import org.jetbrains.kotlin.gradle.util.buildProject import org.jetbrains.kotlin.gradle.util.buildProject
import org.jetbrains.kotlin.gradle.util.enableDependencyVerification import org.jetbrains.kotlin.gradle.util.enableDependencyVerification
import org.jetbrains.kotlin.gradle.utils.LazyResolvedConfiguration import org.jetbrains.kotlin.gradle.utils.LazyResolvedConfiguration
import org.junit.Ignore
import org.junit.Test import org.junit.Test
import kotlin.test.assertEquals import kotlin.test.assertEquals
import kotlin.test.fail import kotlin.test.fail
@@ -39,7 +38,6 @@ class LazyResolvedConfigurationTest {
} }
@Test @Test
@Ignore("stdlib publication migration - test data changes")
fun `test - okio - getArtifacts`() { fun `test - okio - getArtifacts`() {
val project = buildProject { val project = buildProject {
enableDependencyVerification(false) enableDependencyVerification(false)
@@ -20,7 +20,6 @@ import org.jetbrains.kotlin.gradle.plugin.ide.dependencyResolvers.IdeJvmAndAndro
import org.jetbrains.kotlin.gradle.plugin.ide.kotlinIdeMultiplatformImport import org.jetbrains.kotlin.gradle.plugin.ide.kotlinIdeMultiplatformImport
import org.jetbrains.kotlin.gradle.util.* import org.jetbrains.kotlin.gradle.util.*
import org.jetbrains.kotlin.gradle.utils.androidExtension import org.jetbrains.kotlin.gradle.utils.androidExtension
import org.junit.Ignore
import kotlin.test.BeforeTest import kotlin.test.BeforeTest
import kotlin.test.Test import kotlin.test.Test
@@ -161,7 +160,6 @@ class IdeJvmAndAndroidDependencyResolutionTest {
} }
@Test @Test
@Ignore("stdlib publication migration - test data changes")
fun `test - MVIKotlin - binary dependencies - with stdlib enabled by default`() { fun `test - MVIKotlin - binary dependencies - with stdlib enabled by default`() {
val project = buildProject { configureAndroidAndMultiplatform(enableDefaultStdlib = true) } val project = buildProject { configureAndroidAndMultiplatform(enableDefaultStdlib = true) }
val kotlin = project.multiplatformExtension val kotlin = project.multiplatformExtension
@@ -16,13 +16,11 @@ import org.jetbrains.kotlin.gradle.util.applyMultiplatformPlugin
import org.jetbrains.kotlin.gradle.util.buildProject import org.jetbrains.kotlin.gradle.util.buildProject
import org.jetbrains.kotlin.gradle.util.enableDefaultStdlibDependency import org.jetbrains.kotlin.gradle.util.enableDefaultStdlibDependency
import org.jetbrains.kotlin.gradle.util.enableDependencyVerification import org.jetbrains.kotlin.gradle.util.enableDependencyVerification
import org.junit.Ignore
import org.junit.Test import org.junit.Test
class IdeOriginalMetadataDependencyResolverTest { class IdeOriginalMetadataDependencyResolverTest {
@Test @Test
@Ignore("stdlib publication migration - test data changes")
fun `test kotlin-stdlib-common`() { fun `test kotlin-stdlib-common`() {
val project = buildProject { val project = buildProject {
enableDependencyVerification(false) enableDependencyVerification(false)
@@ -23,7 +23,6 @@ import org.jetbrains.kotlin.gradle.util.buildProject
import org.jetbrains.kotlin.gradle.util.enableDefaultStdlibDependency import org.jetbrains.kotlin.gradle.util.enableDefaultStdlibDependency
import org.jetbrains.kotlin.gradle.util.enableDependencyVerification import org.jetbrains.kotlin.gradle.util.enableDependencyVerification
import org.jetbrains.kotlin.gradle.utils.androidExtension import org.jetbrains.kotlin.gradle.utils.androidExtension
import org.junit.Ignore
import org.junit.Test import org.junit.Test
class IdeStdlibResolutionTest { class IdeStdlibResolutionTest {
@@ -149,7 +148,6 @@ class IdeStdlibResolutionTest {
} }
@Test @Test
@Ignore("stdlib publication migration")
fun `test nativeShared`() { fun `test nativeShared`() {
val project = createProjectWithDefaultStdlibEnabled() val project = createProjectWithDefaultStdlibEnabled()
@@ -17,7 +17,6 @@ import org.jetbrains.kotlin.gradle.plugin.getKotlinPluginVersion
import org.jetbrains.kotlin.gradle.plugin.ide.dependencyResolvers.IdeTransformedMetadataDependencyResolver import org.jetbrains.kotlin.gradle.plugin.ide.dependencyResolvers.IdeTransformedMetadataDependencyResolver
import org.jetbrains.kotlin.gradle.plugin.ide.kotlinIdeMultiplatformImport import org.jetbrains.kotlin.gradle.plugin.ide.kotlinIdeMultiplatformImport
import org.jetbrains.kotlin.gradle.util.* import org.jetbrains.kotlin.gradle.util.*
import org.junit.Ignore
import org.junit.Test import org.junit.Test
class IdeTransformedMetadataDependencyResolverTest { class IdeTransformedMetadataDependencyResolverTest {
@@ -72,7 +71,6 @@ class IdeTransformedMetadataDependencyResolverTest {
} }
@Test @Test
@Ignore("stdlib publication migration - test data changes")
fun `test OKIO in JVM + Android project`() { fun `test OKIO in JVM + Android project`() {
val project = buildProject { val project = buildProject {
enableDependencyVerification(false) enableDependencyVerification(false)
@@ -17,13 +17,11 @@ import org.jetbrains.kotlin.gradle.targets.js.dsl.ExperimentalWasmDsl
import org.jetbrains.kotlin.gradle.util.applyMultiplatformPlugin import org.jetbrains.kotlin.gradle.util.applyMultiplatformPlugin
import org.jetbrains.kotlin.gradle.util.buildProject import org.jetbrains.kotlin.gradle.util.buildProject
import org.jetbrains.kotlin.gradle.util.enableDependencyVerification import org.jetbrains.kotlin.gradle.util.enableDependencyVerification
import kotlin.test.Ignore
import kotlin.test.Test import kotlin.test.Test
class WasmDependencyResolutionSmokeTest { class WasmDependencyResolutionSmokeTest {
@Test @Test
@Ignore("stdlib publication migration")
fun `test - project to project ide dependency resolution`() { fun `test - project to project ide dependency resolution`() {
val rootProject = buildProject() val rootProject = buildProject()
val consumer = buildProject(projectBuilder = { withName("consumer").withParent(rootProject) }) val consumer = buildProject(projectBuilder = { withName("consumer").withParent(rootProject) })