azalea/kotlin-fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

KT-34795 Fix failing NewMultiplatformIT#testLibAndApp test

Browse Source 
- Revert back `maven(...)` to `maven { setUrl(...) }`, because in this test this syntax is used intentionally to be valid both for `.gradle` and `.gradle.kts`
This commit is contained in:
Roman Golyshev
2020-01-30 15:46:53 +03:00
committed by Roman Golyshev
parent c804190f36
commit fe779bf7bd
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
libraries/tools/kotlin-gradle-plugin-integration-tests/src/test/kotlin/org/jetbrains/kotlin/gradle/NewMultiplatformIT.kt
+3 -1
View File
@@ -139,7 +139,9 @@ class NewMultiplatformIT : BaseGradleIT() {
with(appProject) { with(appProject) {
setupWorkingDir() 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) { fun CompiledProject.checkProgramCompilationCommandLine(check: (String) -> Unit) {
output.lineSequence().filter { output.lineSequence().filter {