From de3f57cbdfd8bd62bf93e84c7166a04d52f8fd40 Mon Sep 17 00:00:00 2001 From: Ilya Goncharov Date: Wed, 14 Jun 2023 15:19:35 +0200 Subject: [PATCH] [JS] Use just analysis results in js ir tests --- .../frontend/classic/ClassicFrontendFacade.kt | 24 ++++++++----------- .../AbstractDiagnosticsTestWithJsStdLib.kt | 6 +++-- 2 files changed, 14 insertions(+), 16 deletions(-) diff --git a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/frontend/classic/ClassicFrontendFacade.kt b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/frontend/classic/ClassicFrontendFacade.kt index ef6043af79a..e7a389656fe 100644 --- a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/frontend/classic/ClassicFrontendFacade.kt +++ b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/frontend/classic/ClassicFrontendFacade.kt @@ -319,22 +319,18 @@ class ClassicFrontendFacade( val friendLibraries = getDependencies(module, testServices, DependencyRelation.FriendDependency) val allDependencies = runtimeKlibs + dependencyDescriptors + friendLibraries + friendsDescriptors + transitiveLibraries - val analyzer = AnalyzerWithCompilerReport(configuration) val builtInModuleDescriptor = allDependencies.firstNotNullOfOrNull { it.builtIns }?.builtInsModule - analyzer.analyzeAndReport(files) { - TopDownAnalyzerFacadeForJSIR.analyzeFiles( - files, - project, - configuration, - allDependencies, - friendsDescriptors + friendLibraries, - compilerEnvironment, - thisIsBuiltInsModule = builtInModuleDescriptor == null, - customBuiltInsModule = builtInModuleDescriptor - ) - } - return analyzer.analysisResult + return TopDownAnalyzerFacadeForJSIR.analyzeFiles( + files, + project, + configuration, + allDependencies, + friendsDescriptors + friendLibraries, + compilerEnvironment, + thisIsBuiltInsModule = builtInModuleDescriptor == null, + customBuiltInsModule = builtInModuleDescriptor + ) } private fun performWasmModuleResolve( diff --git a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractDiagnosticsTestWithJsStdLib.kt b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractDiagnosticsTestWithJsStdLib.kt index c7914180a27..1d69d41e6fb 100644 --- a/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractDiagnosticsTestWithJsStdLib.kt +++ b/compiler/tests-common-new/tests/org/jetbrains/kotlin/test/runners/AbstractDiagnosticsTestWithJsStdLib.kt @@ -10,8 +10,9 @@ import org.jetbrains.kotlin.test.TargetBackend import org.jetbrains.kotlin.test.builders.TestConfigurationBuilder import org.jetbrains.kotlin.test.builders.classicFrontendHandlersStep import org.jetbrains.kotlin.test.builders.classicFrontendStep +import org.jetbrains.kotlin.test.directives.ConfigurationDirectives +import org.jetbrains.kotlin.test.directives.DiagnosticsDirectives import org.jetbrains.kotlin.test.directives.JvmEnvironmentConfigurationDirectives -import org.jetbrains.kotlin.test.frontend.classic.ClassicFrontendFacade import org.jetbrains.kotlin.test.frontend.classic.handlers.ClassicDiagnosticsHandler import org.jetbrains.kotlin.test.frontend.classic.handlers.DeclarationsDumpHandler import org.jetbrains.kotlin.test.frontend.classic.handlers.DynamicCallsDumpHandler @@ -34,6 +35,7 @@ abstract class AbstractDiagnosticsTestWithJsStdLib : AbstractKotlinCompilerTest( } defaultDirectives { + +ConfigurationDirectives.WITH_STDLIB +JvmEnvironmentConfigurationDirectives.USE_PSI_CLASS_FILES_READING } @@ -61,4 +63,4 @@ abstract class AbstractDiagnosticsTestWithJsStdLib : AbstractKotlinCompilerTest( ) } } -} +} \ No newline at end of file