[Native][tests] Minor. Comments formatted
This commit is contained in:
+8
-5
@@ -109,7 +109,8 @@ internal class ExtTestCaseGroupProvider(
|
|||||||
|
|
||||||
/** Tests that should be compiled and executed as standalone tests. */
|
/** Tests that should be compiled and executed as standalone tests. */
|
||||||
private val standalones: Set<File> = listOf(
|
private val standalones: Set<File> = listOf(
|
||||||
"compiler/testData/codegen/box/annotations/instances/annotationToString.kt" // Comparison of type information obtained with reflection against non-patched string literal.
|
// Comparison of type information obtained with reflection against non-patched string literal:
|
||||||
|
"compiler/testData/codegen/box/annotations/instances/annotationToString.kt"
|
||||||
).mapToSet(::getAbsoluteFile)
|
).mapToSet(::getAbsoluteFile)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -122,9 +123,9 @@ private class ExtTestDataFile(
|
|||||||
private val structure by lazy {
|
private val structure by lazy {
|
||||||
structureFactory.ExtTestDataFileStructure(testDataFile) { line ->
|
structureFactory.ExtTestDataFileStructure(testDataFile) { line ->
|
||||||
if (DIRECTIVE_REGEX.matches(line)) {
|
if (DIRECTIVE_REGEX.matches(line)) {
|
||||||
// Remove all directives from test files. These directives are not needed anymore as they are already
|
// Remove all directives from test files. These directives are not needed anymore as they are already read and stored
|
||||||
// read and stored in [settings] property. Moreover, these directives if left in test file can potentially conflict with
|
// in [settings] property. Moreover, these directives if left in test file can potentially conflict with Native-specific
|
||||||
// Native-specific test directives to be added (see [TestDirectives] for details). Also, they will create unnecessary "noise".
|
// test directives to be added (see [TestDirectives] for details). Also, they will create unnecessary "noise".
|
||||||
// Examples:
|
// Examples:
|
||||||
// // !LANGUAGE: +NewInference
|
// // !LANGUAGE: +NewInference
|
||||||
// // LANGUAGE: -ApproximateIntegerLiteralTypesInReceiverPosition
|
// // LANGUAGE: -ApproximateIntegerLiteralTypesInReceiverPosition
|
||||||
@@ -145,7 +146,9 @@ private class ExtTestDataFile(
|
|||||||
|
|
||||||
ExtTestDataFileSettings(
|
ExtTestDataFileSettings(
|
||||||
languageSettings = structure.directives.multiValues(LANGUAGE_DIRECTIVE) {
|
languageSettings = structure.directives.multiValues(LANGUAGE_DIRECTIVE) {
|
||||||
it != "+NewInference" /* It is already on by default, but passing it explicitly turns on a special "compatibility mode" in FE which is not desirable. */
|
// It is already on by default, but passing it explicitly turns on a special "compatibility mode" in FE,
|
||||||
|
// which is not desirable.
|
||||||
|
it != "+NewInference"
|
||||||
},
|
},
|
||||||
optInsForSourceCode = optInsForSourceCode + structure.directives.multiValues(USE_EXPERIMENTAL_DIRECTIVE),
|
optInsForSourceCode = optInsForSourceCode + structure.directives.multiValues(USE_EXPERIMENTAL_DIRECTIVE),
|
||||||
optInsForCompiler = optInsForCompiler,
|
optInsForCompiler = optInsForCompiler,
|
||||||
|
|||||||
+2
-1
@@ -152,7 +152,8 @@ internal class StandardTestCaseGroupProvider(private val environment: TestEnviro
|
|||||||
}
|
}
|
||||||
|
|
||||||
if (expectFileDirectiveAfterModuleDirective) {
|
if (expectFileDirectiveAfterModuleDirective) {
|
||||||
// Was expecting a line with the FILE directive as this is the only possible continuation of a line with the MODULE directive, but failed.
|
// Was expecting a line with the FILE directive as this is the only possible continuation of a line with
|
||||||
|
// the MODULE directive, but failed.
|
||||||
fail { "$location: ${TestDirectives.FILE} directive expected after ${TestDirectives.MODULE} directive" }
|
fail { "$location: ${TestDirectives.FILE} directive expected after ${TestDirectives.MODULE} directive" }
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user