[Build] Remove unnecessary configs for ignoring test failures It should have been removed together with b3e5fdda57bcbba0dd8ba8939f21973ee99a9ca7
diff --git a/build.gradle.kts b/build.gradle.kts index dda73bf..3cfafe2 100644 --- a/build.gradle.kts +++ b/build.gradle.kts
@@ -598,8 +598,6 @@ ":kotlin-dataframe" ) -val ignoreTestFailures by extra(project.kotlinBuildProperties.ignoreTestFailures) - val dependencyOnSnapshotReflectWhitelist = setOf( ":kotlin-compiler", ":kotlin-reflect",
diff --git a/libraries/kotlin.test/js/it/build.gradle.kts b/libraries/kotlin.test/js/it/build.gradle.kts index 92b57a5..7beb77b 100644 --- a/libraries/kotlin.test/js/it/build.gradle.kts +++ b/libraries/kotlin.test/js/it/build.gradle.kts
@@ -81,7 +81,7 @@ } doLast { println(tcOutput.get().asFile.readText()) - if (exitCodeFile.get().asFile.readText() != "0" /* && !rootProject.ignoreTestFailures*/) { + if (exitCodeFile.get().asFile.readText() != "0") { throw GradleException("$testName integration test failed") }
diff --git a/repo/gradle-build-conventions/project-tests-convention/src/main/kotlin/project-tests-convention.gradle.kts b/repo/gradle-build-conventions/project-tests-convention/src/main/kotlin/project-tests-convention.gradle.kts index 93a0e70..973d346 100644 --- a/repo/gradle-build-conventions/project-tests-convention/src/main/kotlin/project-tests-convention.gradle.kts +++ b/repo/gradle-build-conventions/project-tests-convention/src/main/kotlin/project-tests-convention.gradle.kts
@@ -3,7 +3,6 @@ val extension = extensions.create("projectTests", ProjectTestsExtension::class) tasks.withType<Test>().configureEach { - ignoreFailures = false addCommonInputs() configureCacheDisabling() configureJvmArgumentProviders()
diff --git a/repo/gradle-build-conventions/project-tests-convention/src/main/kotlin/test-inputs-check.gradle.kts b/repo/gradle-build-conventions/project-tests-convention/src/main/kotlin/test-inputs-check.gradle.kts index efe2496..dd9d17d 100644 --- a/repo/gradle-build-conventions/project-tests-convention/src/main/kotlin/test-inputs-check.gradle.kts +++ b/repo/gradle-build-conventions/project-tests-convention/src/main/kotlin/test-inputs-check.gradle.kts
@@ -12,7 +12,6 @@ val disableInputsCheck = project.providers.gradleProperty("kotlin.test.instrumentation.disable.inputs.check").orNull?.toBoolean() == true tasks.withType<Test>().configureEach { val taskName = this.name - ignoreFailures = false val testInputsCheck = extensions.create<TestInputsCheckExtension>("testInputsCheck") val toolchainPath = testInputsCheck.isNative .filter { it }
diff --git a/repo/gradle-build-conventions/utilities/src/main/kotlin/BuildPropertiesExt.kt b/repo/gradle-build-conventions/utilities/src/main/kotlin/BuildPropertiesExt.kt index 9d7afb7..3b87f52 100644 --- a/repo/gradle-build-conventions/utilities/src/main/kotlin/BuildPropertiesExt.kt +++ b/repo/gradle-build-conventions/utilities/src/main/kotlin/BuildPropertiesExt.kt
@@ -17,8 +17,6 @@ val KotlinBuildProperties.jarCompression: Boolean get() = booleanProperty("kotlin.build.jar.compression", isTeamcityBuild).get() -val KotlinBuildProperties.ignoreTestFailures: Boolean get() = booleanProperty("ignoreTestFailures", isTeamcityBuild).get() - val KotlinBuildProperties.disableWerror: Boolean get() = booleanProperty("kotlin.build.disable.werror").get() || useFir.get() || booleanProperty("test.progressive.mode").get() @@ -94,4 +92,4 @@ * local DevX is not broken by stale K/N, so we disable the alignment in these builds. */ val KotlinBuildProperties.alignKotlinNativeVersionInTCBuilds: Boolean - get() = isTeamcityBuild.get() && booleanProperty("kotlin.align.versions.kotlin-native.in.tc.builds", true).get() \ No newline at end of file + get() = isTeamcityBuild.get() && booleanProperty("kotlin.align.versions.kotlin-native.in.tc.builds", true).get()