Skip to content

Commit

Permalink
Fix Gradle plugin inputs validation issue
Browse files Browse the repository at this point in the history
Also add '@internal' annotation in interface, so it propagated to all
inheritors.

^KT-52448 Fixed
  • Loading branch information
Tapchicoma authored and Space committed Jun 6, 2022
1 parent 5f3b1e3 commit ec73f40
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,12 @@ interface KotlinCompileTool : PatternFilterable, Task {

@get:OutputDirectory
val destinationDirectory: DirectoryProperty

@Internal
override fun getExcludes(): MutableSet<String>

@Internal
override fun getIncludes(): MutableSet<String>
}

interface BaseKotlinCompile : KotlinCompileTool {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -226,4 +226,21 @@ class SimpleKotlinGradleIT : KGPBaseTest() {
}
}
}

@DisplayName("Validate Gradle plugins inputs")
@GradleTestVersions(minVersion = TestVersions.Gradle.MAX_SUPPORTED) // Always should use only latest Gradle version
@GradleTest
internal fun validatePluginInputs(gradleVersion: GradleVersion) {
project("kotlinProject", gradleVersion) {
buildGradle.modify {
"""
plugins {
id "validate-external-gradle-plugin"
${it.substringAfter("plugins {")}
""".trimIndent()
}

build("validateExternalPlugins")
}
}
}

0 comments on commit ec73f40

Please sign in to comment.