From fe779bf7bd947c435d70d758d54b3a738be8c55c Mon Sep 17 00:00:00 2001 From: Roman Golyshev Date: Thu, 30 Jan 2020 15:46:53 +0300 Subject: [PATCH] KT-34795 Fix failing NewMultiplatformIT#testLibAndApp test - Revert back `maven(...)` to `maven { setUrl(...) }`, because in this test this syntax is used intentionally to be valid both for `.gradle` and `.gradle.kts` --- .../kotlin/org/jetbrains/kotlin/gradle/NewMultiplatformIT.kt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libraries/tools/kotlin-gradle-plugin-integration-tests/src/test/kotlin/org/jetbrains/kotlin/gradle/NewMultiplatformIT.kt b/libraries/tools/kotlin-gradle-plugin-integration-tests/src/test/kotlin/org/jetbrains/kotlin/gradle/NewMultiplatformIT.kt index 2c6da988cbd..3c6132a5978 100644 --- a/libraries/tools/kotlin-gradle-plugin-integration-tests/src/test/kotlin/org/jetbrains/kotlin/gradle/NewMultiplatformIT.kt +++ b/libraries/tools/kotlin-gradle-plugin-integration-tests/src/test/kotlin/org/jetbrains/kotlin/gradle/NewMultiplatformIT.kt @@ -139,7 +139,9 @@ class NewMultiplatformIT : BaseGradleIT() { with(appProject) { setupWorkingDir() - gradleBuildScript().appendText("\nrepositories { maven(\"$libLocalRepoUri\") }") + + // we use `maven { setUrl(...) }` because this syntax actually works both for Groovy and Kotlin DSLs in Gradle + gradleBuildScript().appendText("\nrepositories { maven { setUrl(\"$libLocalRepoUri\") } }") fun CompiledProject.checkProgramCompilationCommandLine(check: (String) -> Unit) { output.lineSequence().filter {