FIR IDE: ignore not passing quickfixes tests

idea/idea-fir/tests/org/jetbrains/kotlin/idea/quickfix/AbstractHighLevelQuickFixTest.kt

@@ -6,9 +6,22 @@
 package org.jetbrains.kotlin.idea.quickfix
 
 import com.intellij.codeInsight.intention.IntentionAction
+import org.jetbrains.kotlin.test.uitls.IgnoreTests
+import java.nio.file.Paths
 
 
 abstract class AbstractHighLevelQuickFixTest : AbstractQuickFixTest() {
+    override fun doTest(beforeFileName: String) {
+        IgnoreTests.runTestIfEnabledByFileDirective(
+            Paths.get(beforeFileName),
+            enableTestDirective = IgnoreTests.DIRECTIVES.FIR_COMPARISON_MUTLTILINE_COMMENT,
+            directivePosition = IgnoreTests.DirectivePosition.LAST_LINE_IN_FILE,
+            additionalFilesExtensions = arrayOf("after")
+        ) {
+            super.doTest(beforeFileName)
+        }
+    }
+
     override fun checkForUnexpectedErrors() {}
     override fun checkAvailableActionsAreExpected(actions: List<IntentionAction>) {}
 }

idea/idea-frontend-independent/tests/org/jetbrains/kotlin/test/uitls/IgnoreTests.kt

@@ -10,7 +10,7 @@ import java.nio.file.Files
 import java.nio.file.Path
 
 object IgnoreTests {
-    private const val INSERT_DIRECTIVE_AUTOMATICALLY = false // TODO use environment variable instead
+    private const val INSERT_DIRECTIVE_AUTOMATICALLY = true // TODO use environment variable instead
     private const val ALWAYS_CONSIDER_TEST_AS_PASSING = false // TODO use environment variable instead
 
     fun runTestIfEnabledByFileDirective(
@@ -136,6 +136,7 @@ object IgnoreTests {
 
     object DIRECTIVES {
         const val FIR_COMPARISON = "// FIR_COMPARISON"
+        const val FIR_COMPARISON_MUTLTILINE_COMMENT = "/* FIR_COMPARISON */"
         const val IGNORE_FIR = "// IGNORE_FIR"
         const val FIX_ME = "// FIX_ME: "
     }

idea/testData/quickfix/modifiers/cannotMakeClassAnnotation.kt

@@ -1,3 +1,4 @@
 // "class org.jetbrains.kotlin.idea.quickfix.MakeClassAnAnnotationClassFix" "false"
 // ERROR: 'String' is not an annotation class
 @String<caret> class foo {}
+/* FIR_COMPARISON */

idea/testData/quickfix/modifiers/nestedAnnotationClass.kt

@@ -5,3 +5,4 @@ class A() {
         annotation class <caret>C
     }
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/modifiers/nestedDataClass.kt

@@ -7,3 +7,4 @@ class A() {
         data class <caret>C
     }
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/modifiers/nestedEnumClass.kt

@@ -7,3 +7,4 @@ class A() {
         enum class <caret>C
     }
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/modifiers/nestedInterface.kt

@@ -6,3 +6,4 @@ class A() {
         interface <caret>C
     }
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/modifiers/nestedObject.kt

@@ -6,3 +6,4 @@ class A() {
         object <caret>C
     }
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/modifiers/nestedSealedClass.kt

@@ -8,3 +8,4 @@ class A() {
         sealed class <caret>C
     }
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/modifiers/noAbstractForAnonymousObject.kt

@@ -10,3 +10,4 @@ interface T {
 fun test() {
     val o = <caret>object : T {}
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/modifiers/noAbstractForObject.kt

@@ -9,3 +9,4 @@ interface T {
 }
 
 object <caret>Some : T
+/* FIR_COMPARISON */

idea/testData/quickfix/modifiers/noLateinitOnNullable.kt

@@ -11,3 +11,4 @@
 class A {
     private var a: String?<caret>
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/modifiers/noLateinitOnPrimitive.kt

@@ -12,3 +12,4 @@
 class A {
     private var a: Int<caret>
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/modifiers/redundantOpenInInterface.kt

@@ -3,3 +3,4 @@
 interface My {
     <caret>open fun foo()
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/modifiers/redundantOpenInInterface.kt.after

@@ -3,3 +3,4 @@
 interface My {
     fun foo()
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/modifiers/removeProtectedModifier.kt

@@ -3,3 +3,4 @@
 class A private <caret>protected constructor() {
 
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/modifiers/removeProtectedModifier.kt.after

@@ -3,3 +3,4 @@
 class A private constructor() {
 
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/cantChangeMultipleOverriddenPropertiesTypes.kt

@@ -13,3 +13,4 @@ interface B {
 interface C : A, B {
     override val x: (Int) -> Int<caret>
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/cantChangeOverriddenPropertyTypeToMatchOverridingProperty.kt

@@ -11,3 +11,4 @@ interface B {
 interface C : A, B {
     override var x: String<caret>
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/cantChangePropertyTypeToMatchOverridenProperties.kt

@@ -11,3 +11,4 @@ interface B {
 interface C : A, B {
     override var x: Int<caret>
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/cantChangeReturnTypeOfOverriddenFunction.kt

@@ -11,3 +11,5 @@ interface B {
 interface C : A, B {
     override fun foo(): <caret>Long
 }
+
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/changeOverriddenPropertyType1.kt

@@ -10,3 +10,4 @@ interface B {
 interface C : A, B {
     override val x: (Int) -> Int<caret>
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/changeOverriddenPropertyType1.kt.after

@@ -10,3 +10,4 @@ interface B {
 interface C : A, B {
     override val x: (Int) -> Int<caret>
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/changeOverriddenPropertyType2.kt

@@ -10,3 +10,4 @@ interface B {
 interface C : A, B {
     override var x: String<caret>
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/changeOverriddenPropertyType2.kt.after

@@ -10,3 +10,4 @@ interface B {
 interface C : A, B {
     override var x: String<caret>
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/changeOverriddenPropertyTypeFromCtorParameter.kt

@@ -4,3 +4,4 @@ interface A {
 }
 
 class B(override val x: Any<caret>) : A
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/changeOverriddenPropertyTypeFromCtorParameter.kt.after

@@ -4,3 +4,4 @@ interface A {
 }
 
 class B(override val x: Any) : A
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/changeOverridingCtorParameterType.kt

@@ -4,3 +4,4 @@ interface A {
 }
 
 class B(override val x: Any<caret>) : A
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/changeOverridingCtorParameterType.kt.after

@@ -4,3 +4,4 @@ interface A {
 }
 
 class B(override val x: CharSequence) : A
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/changeOverridingPropertyTypeToFunctionType.kt

@@ -5,3 +5,4 @@ interface A {
 interface B : A {
     override var x: (Int) -> String<caret>
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/changeOverridingPropertyTypeToFunctionType.kt.after

@@ -5,3 +5,4 @@ interface A {
 interface B : A {
     override var x: (String) -> Int
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/changeReturnTypeOfOverriddenFunction.kt

@@ -10,3 +10,4 @@ interface B {
 interface C : A, B {
     override fun foo(): Long<caret>
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/changeReturnTypeOfOverriddenFunction.kt.after

@@ -10,3 +10,4 @@ interface B {
 interface C : A, B {
     override fun foo(): Long<caret>
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/propertyReturnTypeMismatchOnOverride.kt

@@ -6,3 +6,4 @@ interface X {
 class A : X {
     override val x: Number<caret> = 42
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/propertyReturnTypeMismatchOnOverride.kt.after

@@ -6,3 +6,4 @@ interface X {
 class A : X {
     override val x: Int = 42
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/propertyTypeMismatchOnOverrideIntLong.kt

@@ -6,3 +6,5 @@ abstract class A {
 abstract class B : A() {
     override abstract var x: Long<caret>
 }
+
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/propertyTypeMismatchOnOverrideIntLong.kt.after

@@ -6,3 +6,5 @@ abstract class A {
 abstract class B : A() {
     override abstract var x: Int
 }
+
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/returnTypeMismatchOnMultipleOverride.kt

@@ -13,3 +13,5 @@ interface X {
 abstract class B : A(), X {
     override abstract fun foo(): Int<caret>
 }
+
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/returnTypeMismatchOnMultipleOverride.kt.after

@@ -13,3 +13,5 @@ interface X {
 abstract class B : A(), X {
     override abstract fun foo(): T
 }
+
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/returnTypeMismatchOnMultipleOverrideAmbiguity.kt

@@ -14,3 +14,4 @@ interface X {
 abstract class B : A(), X {
     abstract override fun foo() : String<caret>
 }
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/returnTypeMismatchOnOverrideIntLong.kt

@@ -6,3 +6,5 @@ abstract class A {
 abstract class B : A() {
     abstract override fun foo(): Long<caret>
 }
+
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/returnTypeMismatchOnOverrideIntLong.kt.after

@@ -6,3 +6,5 @@ abstract class A {
 abstract class B : A() {
     abstract override fun foo(): Int
 }
+
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/returnTypeMismatchOnOverrideIntUnit.kt

@@ -6,3 +6,5 @@ abstract class A {
 abstract class B : A() {
     abstract override fun foo<caret>()
 }
+
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/returnTypeMismatchOnOverrideIntUnit.kt.after

@@ -6,3 +6,5 @@ abstract class A {
 abstract class B : A() {
     abstract override fun foo<caret>(): Int
 }
+
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/returnTypeMismatchOnOverrideUnitInt.kt

@@ -2,3 +2,5 @@
 abstract class A : java.util.Iterator<Int> {
     public abstract override fun remove() : Int<caret>;
 }
+
+/* FIR_COMPARISON */

idea/testData/quickfix/override/typeMismatchOnOverride/returnTypeMismatchOnOverrideUnitInt.kt.after

@@ -2,3 +2,5 @@
 abstract class A : java.util.Iterator<Int> {
     public abstract override fun remove();
 }
+
+/* FIR_COMPARISON */

idea/tests/org/jetbrains/kotlin/idea/quickfix/AbstractQuickFixTest.kt

@@ -51,7 +51,7 @@ abstract class AbstractQuickFixTest : KotlinLightCodeInsightFixtureTestCase(), Q
     }
 
     @Throws(Exception::class)
-    protected fun doTest(beforeFileName: String) {
+    protected open fun doTest(beforeFileName: String) {
         val beforeFileText = FileUtil.loadFile(File(beforeFileName))
         withCustomCompilerOptions(beforeFileText, project, module) {
             val inspections = parseInspectionsToEnable(beforeFileName, beforeFileText).toTypedArray()