diff --git a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/backend/handlers/IrPrettyKotlinDumpHandler.kt b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/backend/handlers/IrPrettyKotlinDumpHandler.kt index 26c2b2c0db5..27c7f4915db 100644 --- a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/backend/handlers/IrPrettyKotlinDumpHandler.kt +++ b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/backend/handlers/IrPrettyKotlinDumpHandler.kt @@ -8,6 +8,7 @@ package org.jetbrains.kotlin.test.backend.handlers import org.jetbrains.kotlin.ir.util.FakeOverridesStrategy import org.jetbrains.kotlin.ir.util.KotlinLikeDumpOptions import org.jetbrains.kotlin.ir.util.dumpKotlinLike +import org.jetbrains.kotlin.test.Constructor import org.jetbrains.kotlin.test.backend.handlers.IrTextDumpHandler.Companion.computeDumpExtension import org.jetbrains.kotlin.test.backend.handlers.IrTextDumpHandler.Companion.groupWithTestFiles import org.jetbrains.kotlin.test.backend.ir.IrBackendInput @@ -17,8 +18,10 @@ import org.jetbrains.kotlin.test.directives.CodegenTestDirectives.EXTERNAL_FILE import org.jetbrains.kotlin.test.directives.CodegenTestDirectives.SKIP_KT_DUMP import org.jetbrains.kotlin.test.directives.FirDiagnosticsDirectives import org.jetbrains.kotlin.test.directives.model.DirectivesContainer +import org.jetbrains.kotlin.test.model.AfterAnalysisChecker import org.jetbrains.kotlin.test.model.BackendKind import org.jetbrains.kotlin.test.model.TestModule +import org.jetbrains.kotlin.test.services.ServiceRegistrationData import org.jetbrains.kotlin.test.services.TestServices import org.jetbrains.kotlin.test.services.moduleStructure import org.jetbrains.kotlin.test.utils.MultiModuleInfoDumper @@ -44,6 +47,9 @@ class IrPrettyKotlinDumpHandler( override val directiveContainers: List get() = listOf(CodegenTestDirectives, FirDiagnosticsDirectives) + override val additionalAfterAnalysisCheckers: List> + get() = listOf(::FirIrDumpIdenticalChecker) + override fun processModule(module: TestModule, info: IrBackendInput) { if (DUMP_KT_IR !in module.directives || SKIP_KT_DUMP in module.directives) return dumpModuleKotlinLike( diff --git a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/backend/handlers/IrSourceRangesDumpHandler.kt b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/backend/handlers/IrSourceRangesDumpHandler.kt index c67c5edbbf8..d5174966c25 100644 --- a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/backend/handlers/IrSourceRangesDumpHandler.kt +++ b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/backend/handlers/IrSourceRangesDumpHandler.kt @@ -13,10 +13,12 @@ import org.jetbrains.kotlin.ir.util.RenderIrElementVisitor import org.jetbrains.kotlin.ir.visitors.IrElementVisitorVoid import org.jetbrains.kotlin.ir.visitors.acceptChildrenVoid import org.jetbrains.kotlin.ir.visitors.acceptVoid +import org.jetbrains.kotlin.test.Constructor import org.jetbrains.kotlin.test.backend.ir.IrBackendInput import org.jetbrains.kotlin.test.directives.CodegenTestDirectives import org.jetbrains.kotlin.test.directives.FirDiagnosticsDirectives import org.jetbrains.kotlin.test.directives.model.DirectivesContainer +import org.jetbrains.kotlin.test.model.AfterAnalysisChecker import org.jetbrains.kotlin.test.model.BackendKind import org.jetbrains.kotlin.test.model.TestModule import org.jetbrains.kotlin.test.services.TestServices @@ -37,6 +39,9 @@ class IrSourceRangesDumpHandler( override val directiveContainers: List get() = listOf(CodegenTestDirectives, FirDiagnosticsDirectives) + override val additionalAfterAnalysisCheckers: List> + get() = listOf(::FirIrSourceRangesIdenticalChecker) + private val baseDumper = MultiModuleInfoDumper() private val buildersForSeparateFileDumps: MutableMap = mutableMapOf() diff --git a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/backend/handlers/IrTextDumpHandler.kt b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/backend/handlers/IrTextDumpHandler.kt index f57d795c026..fbb9a6f66f3 100644 --- a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/backend/handlers/IrTextDumpHandler.kt +++ b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/backend/handlers/IrTextDumpHandler.kt @@ -19,6 +19,7 @@ import org.jetbrains.kotlin.ir.util.dump import org.jetbrains.kotlin.ir.util.dumpTreesFromLineNumber import org.jetbrains.kotlin.name.ClassId import org.jetbrains.kotlin.name.Name +import org.jetbrains.kotlin.test.Constructor import org.jetbrains.kotlin.test.backend.ir.IrBackendInput import org.jetbrains.kotlin.test.directives.CodegenTestDirectives import org.jetbrains.kotlin.test.directives.CodegenTestDirectives.DUMP_EXTERNAL_CLASS @@ -78,6 +79,9 @@ class IrTextDumpHandler( override val directiveContainers: List get() = listOf(CodegenTestDirectives, FirDiagnosticsDirectives) + override val additionalAfterAnalysisCheckers: List> + get() = listOf(::FirIrDumpIdenticalChecker) + private val baseDumper = MultiModuleInfoDumper() private val buildersForSeparateFileDumps: MutableMap = mutableMapOf() diff --git a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/frontend/fir/handlers/FirDiagnosticsHandler.kt b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/frontend/fir/handlers/FirDiagnosticsHandler.kt index 19743b3d9ab..ca7c75452f0 100644 --- a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/frontend/fir/handlers/FirDiagnosticsHandler.kt +++ b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/frontend/fir/handlers/FirDiagnosticsHandler.kt @@ -44,6 +44,7 @@ import org.jetbrains.kotlin.platform.konan.isNative import org.jetbrains.kotlin.psi.KtBinaryExpression import org.jetbrains.kotlin.psi.KtQualifiedExpression import org.jetbrains.kotlin.resolve.AnalyzingUtils +import org.jetbrains.kotlin.test.Constructor import org.jetbrains.kotlin.test.FirParser import org.jetbrains.kotlin.test.directives.AdditionalFilesDirectives import org.jetbrains.kotlin.test.directives.DiagnosticsDirectives @@ -53,6 +54,7 @@ import org.jetbrains.kotlin.test.directives.model.DirectivesContainer import org.jetbrains.kotlin.test.directives.model.SimpleDirective import org.jetbrains.kotlin.test.directives.model.singleValue import org.jetbrains.kotlin.test.frontend.fir.FirOutputArtifact +import org.jetbrains.kotlin.test.model.AfterAnalysisChecker import org.jetbrains.kotlin.test.model.TestFile import org.jetbrains.kotlin.test.model.TestModule import org.jetbrains.kotlin.test.runners.lightTreeSyntaxDiagnosticsReporterHolder @@ -106,6 +108,9 @@ class FirDiagnosticsHandler(testServices: TestServices) : FirAnalysisHandler(tes override val additionalServices: List = listOf(service(::DiagnosticsService), service(::FirDiagnosticCollectorService)) + override val additionalAfterAnalysisCheckers: List> + get() = listOf(::FirIdenticalChecker) + private val fullDiagnosticsRenderer = FullDiagnosticsRenderer(DiagnosticsDirectives.RENDER_DIAGNOSTICS_FULL_TEXT) override fun processAfterAllModules(someAssertionWasFailed: Boolean) { diff --git a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractFirDiagnosticTest.kt b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractFirDiagnosticTest.kt index e2bad1cd7d5..5d029dc246f 100644 --- a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractFirDiagnosticTest.kt +++ b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractFirDiagnosticTest.kt @@ -101,7 +101,6 @@ fun TestConfigurationBuilder.configurationForClassicAndFirTestsAlongside( testDataConsistencyHandler: Constructor = ::FirTestDataConsistencyHandler, ) { useAfterAnalysisCheckers( - ::FirIdenticalChecker, ::FirFailingTestSuppressor, testDataConsistencyHandler, ) diff --git a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractFirDiagnosticTestSpec.kt b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractFirDiagnosticTestSpec.kt index c59165132fd..25e26b84aec 100644 --- a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractFirDiagnosticTestSpec.kt +++ b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractFirDiagnosticTestSpec.kt @@ -42,7 +42,6 @@ fun TestConfigurationBuilder.baseFirSpecDiagnosticTestConfiguration(baseDir: Str useAdditionalSourceProviders(::SpecHelpersSourceFilesProvider.bind(baseDir)) useAfterAnalysisCheckers( - ::FirIdenticalChecker, ::FirTestDataConsistencyHandler, ::FirFailingTestSuppressor, ) diff --git a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractFirForeignAnnotationsTest.kt b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractFirForeignAnnotationsTest.kt index da6fb54178f..12b32288057 100644 --- a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractFirForeignAnnotationsTest.kt +++ b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractFirForeignAnnotationsTest.kt @@ -54,10 +54,7 @@ abstract class AbstractFirForeignAnnotationsTestBase( } forTestsMatching("compiler/testData/diagnostics/*") { - useAfterAnalysisCheckers( - ::FirIdenticalChecker, - ::FirFailingTestSuppressor, - ) + useAfterAnalysisCheckers(::FirFailingTestSuppressor) useMetaTestConfigurators(::FirOldFrontendMetaConfigurator) } } diff --git a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/codegen/AbstractFirScriptCodegenTest.kt b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/codegen/AbstractFirScriptCodegenTest.kt index 2960b2086ce..0864771366f 100644 --- a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/codegen/AbstractFirScriptCodegenTest.kt +++ b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/codegen/AbstractFirScriptCodegenTest.kt @@ -74,9 +74,6 @@ open class AbstractFirScriptCodegenTest : AbstractKotlinCompilerWithTargetBacken useHandlers( ::FirDiagnosticsHandler ) - useAfterAnalysisCheckers( - ::FirIrDumpIdenticalChecker, - ) commonFirHandlersForCodegenTest() } diff --git a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/codegen/BaseCodegenConfiguration.kt b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/codegen/BaseCodegenConfiguration.kt index 4cdc79ced32..b2a6f7da1b3 100644 --- a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/codegen/BaseCodegenConfiguration.kt +++ b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/codegen/BaseCodegenConfiguration.kt @@ -108,7 +108,6 @@ fun TestConfigurationBuilder.applyDumpSmapDirective() { fun TestConfigurationBuilder.configureDumpHandlersForCodegenTest() { configureIrHandlersStep { dumpHandlersForConverterStep() - useAfterAnalysisCheckers(::FirIrDumpIdenticalChecker) } configureJvmArtifactsHandlersStep { dumpHandlersForBackendStep() diff --git a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/ir/AbstractFirJvmIrSourceRangesTest.kt b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/ir/AbstractFirJvmIrSourceRangesTest.kt index 215575a8984..ac9c4d5e685 100644 --- a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/ir/AbstractFirJvmIrSourceRangesTest.kt +++ b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/ir/AbstractFirJvmIrSourceRangesTest.kt @@ -7,7 +7,6 @@ package org.jetbrains.kotlin.test.runners.ir import org.jetbrains.kotlin.test.Constructor import org.jetbrains.kotlin.test.FirParser -import org.jetbrains.kotlin.test.backend.handlers.FirIrSourceRangesIdenticalChecker import org.jetbrains.kotlin.test.backend.ir.IrBackendInput import org.jetbrains.kotlin.test.builders.TestConfigurationBuilder import org.jetbrains.kotlin.test.directives.configureFirParser @@ -37,9 +36,6 @@ abstract class AbstractFirJvmIrSourceRangesTest( super.configure(builder) with(builder) { configureFirParser(parser) - useAfterAnalysisCheckers( - ::FirIrSourceRangesIdenticalChecker, - ) } } } diff --git a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/ir/AbstractIrTextTest.kt b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/ir/AbstractIrTextTest.kt index 054a683cfc2..9f6507d2f42 100644 --- a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/ir/AbstractIrTextTest.kt +++ b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/ir/AbstractIrTextTest.kt @@ -113,10 +113,6 @@ abstract class AbstractIrTextTest(::ParcelizeMainClassProvider)) - useAfterAnalysisCheckers(::BlackBoxCodegenSuppressor, ::FirIrDumpIdenticalChecker) + useAfterAnalysisCheckers(::BlackBoxCodegenSuppressor) enableMetaInfoHandler() }