From eba9f697c026c7ea0c88c62c78680b31fdf4e88f Mon Sep 17 00:00:00 2001 From: Yahor Berdnikau Date: Mon, 24 Jan 2022 15:35:52 +0100 Subject: [PATCH] Convert KotlinAndroidExtensionIT to new test dsl ^KT-45745 In Progress --- .../gradle/model/KotlinAndroidExtensionIT.kt | 82 +++++++++++-------- 1 file changed, 46 insertions(+), 36 deletions(-) diff --git a/libraries/tools/kotlin-gradle-plugin-integration-tests/src/test/kotlin/org/jetbrains/kotlin/gradle/model/KotlinAndroidExtensionIT.kt b/libraries/tools/kotlin-gradle-plugin-integration-tests/src/test/kotlin/org/jetbrains/kotlin/gradle/model/KotlinAndroidExtensionIT.kt index 8225eb64a01..1aadd036ce5 100644 --- a/libraries/tools/kotlin-gradle-plugin-integration-tests/src/test/kotlin/org/jetbrains/kotlin/gradle/model/KotlinAndroidExtensionIT.kt +++ b/libraries/tools/kotlin-gradle-plugin-integration-tests/src/test/kotlin/org/jetbrains/kotlin/gradle/model/KotlinAndroidExtensionIT.kt @@ -5,50 +5,60 @@ package org.jetbrains.kotlin.gradle.model -import org.jetbrains.kotlin.gradle.BaseGradleIT -import org.jetbrains.kotlin.gradle.util.AGPVersion -import org.jetbrains.kotlin.test.util.KtTestUtil -import org.junit.Test +import org.gradle.api.logging.configuration.WarningMode +import org.gradle.util.GradleVersion +import org.jetbrains.kotlin.gradle.testbase.* +import org.junit.jupiter.api.DisplayName import kotlin.test.assertEquals import kotlin.test.assertFalse import kotlin.test.assertNull import kotlin.test.assertTrue -class KotlinAndroidExtensionIT : BaseGradleIT() { - override fun defaultBuildOptions(): BuildOptions { - return super.defaultBuildOptions().copy( - androidGradlePluginVersion = AGPVersion.v3_4_1, - androidHome = KtTestUtil.findAndroidSdk() - ) +@DisplayName("Kotlin-android plugin models") +@OtherGradlePluginTests +class KotlinAndroidExtensionIT : KGPBaseTest() { + override val defaultBuildOptions = super.defaultBuildOptions.copy( + androidVersion = TestVersions.AGP.AGP_34, + warningMode = WarningMode.Summary + ) + + @DisplayName("Has valid model when plugin is applied") + @GradleTest + fun testAndroidExtensionsProject(gradleVersion: GradleVersion) { + project("AndroidExtensionsProject", gradleVersion) { + getModels { + with(getModel(":app")!!) { + assertEquals(1L, modelVersion) + assertEquals("app", name) + assertFalse(isExperimental) + assertEquals("hashMap", defaultCacheImplementation) + } + } + } } - @Test - fun testAndroidExtensionsProject() { - val project = Project("AndroidExtensionsProject") - val androidExtensionModel = project.getModels(KotlinAndroidExtension::class.java).getModel(":app")!! - - assertEquals(1L, androidExtensionModel.modelVersion) - assertEquals("app", androidExtensionModel.name) - assertFalse(androidExtensionModel.isExperimental) - assertEquals("hashMap", androidExtensionModel.defaultCacheImplementation) + @DisplayName("Has valid model in android multivariant project when plugin is applied") + @GradleTest + fun testAndroidExtensionsManyVariants(gradleVersion: GradleVersion) { + project("AndroidExtensionsManyVariants", gradleVersion) { + getModels { + with(getModel(":app")!!) { + assertEquals(1L, modelVersion) + assertEquals("app", name) + assertTrue(isExperimental) + assertEquals("hashMap", defaultCacheImplementation) + } + } + } } - @Test - fun testAndroidExtensionsManyVariants() { - val project = Project("AndroidExtensionsManyVariants") - val androidExtensionModel = project.getModels(KotlinAndroidExtension::class.java).getModel(":app")!! - - assertEquals(1L, androidExtensionModel.modelVersion) - assertEquals("app", androidExtensionModel.name) - assertTrue(androidExtensionModel.isExperimental) - assertEquals("hashMap", androidExtensionModel.defaultCacheImplementation) - } - - @Test - fun testNonAndroidExtensionsProjects() { - val project = Project("kotlinProject") - val model = project.getModels(KotlinAndroidExtension::class.java).getModel(":") - - assertNull(model) + @DisplayName("Doesn't have model when plugin is not applied") + @GradleTest + fun testNonAndroidExtensionsProjects(gradleVersion: GradleVersion) { + project("kotlinProject", gradleVersion) { + getModels { + assertNull(getModel(":")) + } + } } }