Run muted non-flaky tests and invert their result (KTI-216)

Cause: if mutedInDatabase test is failing then test result = success
else test result = failure (because now it should be unmuted)

buildSrc/src/main/kotlin/tasks.kt

@@ -106,6 +106,7 @@ fun Project.projectTest(
     environment("PROJECT_BUILD_DIR", buildDir)
     systemProperty("jps.kotlin.home", rootProject.extra["distKotlinHomeDir"]!!)
     systemProperty("kotlin.ni", if (rootProject.hasProperty("newInferenceTests")) "true" else "false")
+    systemProperty("org.jetbrains.kotlin.skip.muted.tests", if (rootProject.hasProperty("skipMutedTests")) "true" else "false")
 
     var subProjectTempRoot: Path? = null
     doFirst {

compiler/tests-common/tests/org/jetbrains/kotlin/test/muteWithDatabase.kt

@@ -22,6 +22,8 @@ private class AutoMute(
     val issue: String
 )
 
+private val SKIP_MUTED_TESTS = java.lang.Boolean.getBoolean("org.jetbrains.kotlin.skip.muted.tests")
+
 private val DO_AUTO_MUTE: AutoMute? by lazy {
     val autoMuteFile = File("tests/automute")
     if (autoMuteFile.exists()) {
@@ -162,7 +164,15 @@ internal fun isMutedInDatabase(testCase: TestCase): Boolean {
 
 fun isMutedInDatabase(testClass: Class<*>, methodKey: String): Boolean {
     val mutedTest = mutedSet.mutedTest(testClass, methodKey)
-    return mutedTest != null && !mutedTest.hasFailFile
+    return mutedTest != null && (if (SKIP_MUTED_TESTS) !mutedTest.hasFailFile else mutedTest.isFlaky)
+}
+
+private fun getMutedTestOrNull(testCase: TestCase): MutedTest? {
+    return getMutedTestOrNull(testCase.javaClass, testCase.name)
+}
+
+private fun getMutedTestOrNull(testClass: Class<*>, methodKey: String): MutedTest? {
+    return mutedSet.mutedTest(testClass, methodKey)
 }
 
 internal fun wrapWithMuteInDatabase(testCase: TestCase, f: () -> Unit): (() -> Unit)? {
@@ -172,16 +182,36 @@ internal fun wrapWithMuteInDatabase(testCase: TestCase, f: () -> Unit): (() -> U
         }
     }
 
-    val doAutoMute = DO_AUTO_MUTE ?: return null
+    val mutedTest = getMutedTestOrNull(testCase)
 
-    return {
+    if (mutedTest != null && !mutedTest.hasFailFile) {
-        try {
+        val testKey = testKey(testCase)
-            f()
+        return {
-        } catch (e: Throwable) {
+            var isTestGreen = true
-            doAutoMute.muteTest(testKey(testCase))
+            try {
-            throw e
+                f()
+            } catch (e: Throwable) {
+                println("MUTED TEST STILL FAILS: $testKey")
+                isTestGreen = false
+            }
+            if (isTestGreen) {
+                System.err.println("SUCCESS RESULT OF MUTED TEST: $testKey")
+                throw Exception("Muted non-flaky test $testKey finished successfully. Please remove it from csv file")
+            }
+        }
+    } else {
+        val doAutoMute = DO_AUTO_MUTE ?: return null
+        return {
+            try {
+                f()
+            } catch (e: Throwable) {
+                doAutoMute.muteTest(testKey(testCase))
+                throw e
+            }
         }
     }
+
+
 }
 
 private fun mutedMessage(key: String) = "MUTED TEST: $key"
@@ -269,4 +299,4 @@ fun isIgnoredInDatabaseWithLog(testCase: TestCase): Boolean {
 
 fun TestCase.runTest(test: () -> Unit) {
     (wrapWithMuteInDatabase(this, test) ?: test).invoke()
-}
+}