From c9f6482f253ee3342ab2565f2efb47c777b2370b Mon Sep 17 00:00:00 2001 From: Mikhail Glukhikh Date: Fri, 18 Oct 2019 19:23:43 +0300 Subject: [PATCH] Throw exception from FIR smoke test if failure file does not exist It's more convenient to see exception from the beginning and not a strange message about generated .fir.fail file --- .../jetbrains/kotlin/fir/AbstractFirDiagnosticsSmokeTest.kt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/compiler/fir/resolve/tests/org/jetbrains/kotlin/fir/AbstractFirDiagnosticsSmokeTest.kt b/compiler/fir/resolve/tests/org/jetbrains/kotlin/fir/AbstractFirDiagnosticsSmokeTest.kt index 95714835c17..20c41e04984 100644 --- a/compiler/fir/resolve/tests/org/jetbrains/kotlin/fir/AbstractFirDiagnosticsSmokeTest.kt +++ b/compiler/fir/resolve/tests/org/jetbrains/kotlin/fir/AbstractFirDiagnosticsSmokeTest.kt @@ -18,7 +18,6 @@ import org.jetbrains.kotlin.fir.declarations.FirFile import org.jetbrains.kotlin.fir.java.FirJavaModuleBasedSession import org.jetbrains.kotlin.fir.java.FirLibrarySession import org.jetbrains.kotlin.fir.java.FirProjectSessionProvider -import org.jetbrains.kotlin.fir.resolve.FirProvider import org.jetbrains.kotlin.fir.resolve.firProvider import org.jetbrains.kotlin.fir.resolve.impl.FirProviderImpl import org.jetbrains.kotlin.fir.resolve.transformers.FirTotalResolveTransformer @@ -102,6 +101,9 @@ abstract class AbstractFirDiagnosticsSmokeTest : BaseDiagnosticsTest() { checkResultingFirFiles(firFiles, testDataFile) assertFalse("Test is good but there is expected exception", failureFile.exists()) } else { + if (!failureFile.exists()) { + throw failure + } checkFailureFile(failure, failureFile) } }