Switch rest of test to new scheme with configuration kind processing

This commit is contained in:
Mikhael Bogdanov
2020-02-27 14:56:19 +01:00
parent 46397dca4a
commit 32bcab2867
15 changed files with 13 additions and 23 deletions
@@ -9,7 +9,8 @@ import org.jetbrains.kotlin.test.ConfigurationKind
import org.jetbrains.kotlin.test.TestJdkKind import org.jetbrains.kotlin.test.TestJdkKind
abstract class AbstractFirDiagnosticsWithStdlibTest : AbstractFirDiagnosticsTest() { abstract class AbstractFirDiagnosticsWithStdlibTest : AbstractFirDiagnosticsTest() {
override fun getConfigurationKind(): ConfigurationKind {
override fun extractConfigurationKind(files: List<TestFile>): ConfigurationKind {
return ConfigurationKind.ALL return ConfigurationKind.ALL
} }
} }
@@ -1,5 +1,6 @@
// !CHECK_TYPE // !CHECK_TYPE
// SKIP_TXT // SKIP_TXT
// WITH_RUNTIME
fun foo(s: String) { fun foo(s: String) {
s.isBlank() s.isBlank()
@@ -1,6 +1,5 @@
// !LANGUAGE: +VariableDeclarationInWhenSubject // !LANGUAGE: +VariableDeclarationInWhenSubject
// !DIAGNOSTICS: -UNUSED_VARIABLE -UNUSED_PARAMETER // !DIAGNOSTICS: -UNUSED_VARIABLE -UNUSED_PARAMETER
// WITH_RUNTIME
fun foo(): Any = 42 fun foo(): Any = 42
fun String.bar(): Any = 42 fun String.bar(): Any = 42
@@ -1,6 +1,5 @@
// !LANGUAGE: +VariableDeclarationInWhenSubject // !LANGUAGE: +VariableDeclarationInWhenSubject
// !DIAGNOSTICS: -UNUSED_VARIABLE -UNUSED_PARAMETER // !DIAGNOSTICS: -UNUSED_VARIABLE -UNUSED_PARAMETER
// WITH_RUNTIME
fun foo(): Any = 42 fun foo(): Any = 42
fun String.bar(): Any = 42 fun String.bar(): Any = 42
@@ -1,3 +1,4 @@
//WITH_RUNTIME
import java.util.stream.IntStream import java.util.stream.IntStream
fun foo(s: IntStream) { fun foo(s: IntStream) {
@@ -2,7 +2,7 @@
// !WITH_NEW_INFERENCE // !WITH_NEW_INFERENCE
// RENDER_PACKAGE: test // RENDER_PACKAGE: test
// SKIP_COMPILED_JAVA // SKIP_COMPILED_JAVA
// WITH_RUNTIME
// FILE: spr/Nullable.java // FILE: spr/Nullable.java
package spr; package spr;
@@ -16,14 +16,9 @@
package org.jetbrains.kotlin.cfg package org.jetbrains.kotlin.cfg
import org.jetbrains.kotlin.checkers.AbstractDiagnosticsTest import org.jetbrains.kotlin.checkers.AbstractDiagnosticsTest
import org.jetbrains.kotlin.test.ConfigurationKind
import org.jetbrains.kotlin.test.TestJdkKind import org.jetbrains.kotlin.test.TestJdkKind
abstract class AbstractDiagnosticsWithModifiedMockJdkTest : AbstractDiagnosticsTest() { abstract class AbstractDiagnosticsWithModifiedMockJdkTest : AbstractDiagnosticsTest() {
override fun getConfigurationKind(): ConfigurationKind {
return ConfigurationKind.ALL
}
override fun getTestJdkKind(files: List<TestFile>): TestJdkKind { override fun getTestJdkKind(files: List<TestFile>): TestJdkKind {
return TestJdkKind.MODIFIED_MOCK_JDK return TestJdkKind.MODIFIED_MOCK_JDK
} }
@@ -8,7 +8,8 @@ import org.jetbrains.kotlin.test.ConfigurationKind
import java.io.File import java.io.File
abstract class AbstractDiagnosticsTestWithStdLib : AbstractDiagnosticsTest() { abstract class AbstractDiagnosticsTestWithStdLib : AbstractDiagnosticsTest() {
override fun getConfigurationKind(): ConfigurationKind {
override fun extractConfigurationKind(files: List<TestFile>): ConfigurationKind {
return ConfigurationKind.NO_KOTLIN_REFLECT return ConfigurationKind.NO_KOTLIN_REFLECT
} }
@@ -9,10 +9,6 @@ import org.jetbrains.kotlin.config.*
import org.jetbrains.kotlin.test.ConfigurationKind import org.jetbrains.kotlin.test.ConfigurationKind
abstract class AbstractDiagnosticsWithExplicitApi : AbstractDiagnosticsTest() { abstract class AbstractDiagnosticsWithExplicitApi : AbstractDiagnosticsTest() {
override fun getConfigurationKind(): ConfigurationKind {
return ConfigurationKind.NO_KOTLIN_REFLECT
}
override fun defaultLanguageVersionSettings(): LanguageVersionSettings = override fun defaultLanguageVersionSettings(): LanguageVersionSettings =
CompilerTestLanguageVersionSettings( CompilerTestLanguageVersionSettings(
DEFAULT_DIAGNOSTIC_TESTS_FEATURES, DEFAULT_DIAGNOSTIC_TESTS_FEATURES,
@@ -15,11 +15,9 @@
*/ */
package org.jetbrains.kotlin.checkers package org.jetbrains.kotlin.checkers
import org.jetbrains.kotlin.test.ConfigurationKind
import org.jetbrains.kotlin.test.TestJdkKind import org.jetbrains.kotlin.test.TestJdkKind
abstract class AbstractDiagnosticsWithJdk9Test : AbstractDiagnosticsTest() { abstract class AbstractDiagnosticsWithJdk9Test : AbstractDiagnosticsTest() {
override fun getConfigurationKind(): ConfigurationKind = ConfigurationKind.ALL
override fun getTestJdkKind(files: List<TestFile>): TestJdkKind { override fun getTestJdkKind(files: List<TestFile>): TestJdkKind {
return TestJdkKind.FULL_JDK_9 return TestJdkKind.FULL_JDK_9
@@ -12,7 +12,8 @@ import org.jetbrains.kotlin.config.LanguageVersionSettings
import org.jetbrains.kotlin.test.ConfigurationKind import org.jetbrains.kotlin.test.ConfigurationKind
abstract class AbstractDiagnosticsWithUnsignedTypes : AbstractDiagnosticsTest() { abstract class AbstractDiagnosticsWithUnsignedTypes : AbstractDiagnosticsTest() {
override fun getConfigurationKind(): ConfigurationKind {
override fun extractConfigurationKind(files: List<TestFile>): ConfigurationKind {
return ConfigurationKind.NO_KOTLIN_REFLECT return ConfigurationKind.NO_KOTLIN_REFLECT
} }
@@ -52,8 +52,6 @@ abstract class AbstractForeignAnnotationsTest : AbstractDiagnosticsTest() {
ForTestCompileRuntime.jvmAnnotationsForTests() ForTestCompileRuntime.jvmAnnotationsForTests()
) )
override fun getConfigurationKind(): ConfigurationKind = ConfigurationKind.ALL
open protected val annotationsPath: String open protected val annotationsPath: String
get() = FOREIGN_ANNOTATIONS_SOURCES_PATH get() = FOREIGN_ANNOTATIONS_SOURCES_PATH
@@ -79,8 +79,6 @@ abstract class KotlinMultiFileTestWithJava<M : KotlinBaseTest.TestModule, F : Ko
setupEnvironment(environment) setupEnvironment(environment)
} }
protected open fun getConfigurationKind() = ConfigurationKind.JDK_ONLY
private fun getClasspath(file: File): List<File> { private fun getClasspath(file: File): List<File> {
val result: MutableList<File> = ArrayList() val result: MutableList<File> = ArrayList()
result.add(KotlinTestUtils.getAnnotationsJar()) result.add(KotlinTestUtils.getAnnotationsJar())
@@ -8,7 +8,7 @@ package org.jetbrains.kotlin.fir
import org.jetbrains.kotlin.test.ConfigurationKind import org.jetbrains.kotlin.test.ConfigurationKind
abstract class AbstractFirOldFrontendDiagnosticsTestWithStdlib : AbstractFirOldFrontendDiagnosticsTest() { abstract class AbstractFirOldFrontendDiagnosticsTestWithStdlib : AbstractFirOldFrontendDiagnosticsTest() {
override fun getConfigurationKind(): ConfigurationKind { override fun extractConfigurationKind(files: List<TestFile>): ConfigurationKind {
return ConfigurationKind.NO_KOTLIN_REFLECT return ConfigurationKind.NO_KOTLIN_REFLECT
} }
} }
@@ -47,7 +47,9 @@ abstract class AbstractDiagnosticsTestSpec : org.jetbrains.kotlin.checkers.Abstr
} }
} }
override fun getConfigurationKind() = ConfigurationKind.ALL override fun extractConfigurationKind(files: List<TestFile>): ConfigurationKind {
return ConfigurationKind.ALL
}
override fun skipDescriptorsValidation() = skipDescriptors override fun skipDescriptorsValidation() = skipDescriptors