diff --git a/idea/src/META-INF/plugin.xml b/idea/src/META-INF/plugin.xml index c51c0591547..37002cf4961 100644 --- a/idea/src/META-INF/plugin.xml +++ b/idea/src/META-INF/plugin.xml @@ -1290,14 +1290,6 @@ level="WARNING" /> - - kotlinClassOrObject as KtObjectDeclaration // KT-9578 - is KtClass -> kotlinClassOrObject.getCompanionObjects().singleOrNull() ?: return - else -> return - } - holder.registerProblem( - expression, "This field will not be generated in future versions of Kotlin. Use 'const' modifier, '@JvmField' annotation or access data through corresponding object.", - ProblemHighlightType.LIKE_DEPRECATED, - *createFixes(containingObject, kotlinProperty).toTypedArray() - ) - } - } - } - - - private fun createFixes(containingObject: KtObjectDeclaration, property: KtProperty?): List { - if (containingObject.getContainingKtFile().isCompiled) return listOf(ReplaceWithGetterInvocationFix()) - - // order matters here, 'cleanup' applies fixes in this order - val fixes = ArrayList() - if (property != null && AddConstModifierIntention.isApplicableTo(property)) { - fixes.add(AddConstModifierLocalFix()) - } - - if (containingObject.isCompanion()) { - val classWithCompanion = containingObject.parent?.parent as? KtClass ?: return listOf() - if (!classWithCompanion.isInterface()) { - fixes.add(AddJvmFieldAnnotationFix()) - } - } else { - fixes.add(AddJvmFieldAnnotationFix()) - } - - fixes.add(ReplaceWithGetterInvocationFix()) - - return fixes - } -} - -abstract class StaticFieldUsageFix: LocalQuickFix { - override fun applyFix(project: Project, descriptor: ProblemDescriptor) { - val deprecatedField = descriptor.psiElement.reference?.resolve() as? PsiField ?: return - val kotlinProperty = (deprecatedField as? KtLightField)?.getOrigin() as? KtProperty - - if (kotlinProperty != null && kotlinProperty.hasJvmFieldAnnotationOrConstModifier()) return - - doFix(deprecatedField, kotlinProperty, descriptor) - } - - abstract fun doFix(deprecatedField: PsiField, property: KtProperty?, problemDescriptor: ProblemDescriptor) -} - -class AddJvmFieldAnnotationFix : StaticFieldUsageFix() { - override fun doFix(deprecatedField: PsiField, property: KtProperty?, problemDescriptor: ProblemDescriptor) { - replaceReferencesToGetterByReferenceToField(property ?: return) - - property.addAnnotationEntry(KtPsiFactory(property).createAnnotationEntry("@JvmField")) - } - - override fun getName(): String = "Annotate property with @JvmField" - override fun getFamilyName(): String = name -} - -class AddConstModifierLocalFix : StaticFieldUsageFix() { - override fun getName(): String = "Add 'const' modifier to a property" - override fun getFamilyName(): String = name - - override fun doFix(deprecatedField: PsiField, property: KtProperty?, problemDescriptor: ProblemDescriptor) { - AddConstModifierFix.addConstModifier(property ?: return) - } -} - -class ReplaceWithGetterInvocationFix : StaticFieldUsageFix() { - override fun getName(): String = "Replace with getter invocation" - override fun getFamilyName(): String = name - - override fun doFix(deprecatedField: PsiField, property: KtProperty?, problemDescriptor: ProblemDescriptor) { - val lightClass = deprecatedField.containingClass as? KtLightClass ?: return - - fun replaceWithGetterInvocation(objectField: PsiField) { - val factory = PsiElementFactory.SERVICE.getInstance(deprecatedField.project) - val elementToReplace = problemDescriptor.psiElement - - val getterInvocation = factory.createExpressionFromText( - objectField.containingClass!!.qualifiedName + "." + objectField.name + "." + JvmAbi.getterName(deprecatedField.name!!) + "()", - elementToReplace - ) - elementToReplace.replace(getterInvocation) - } - - val kotlinClass = lightClass.getOrigin() - when (kotlinClass) { - is KtObjectDeclaration -> { - val instanceField = lightClass.findFieldByName(JvmAbi.INSTANCE_FIELD, false) ?: return - replaceWithGetterInvocation(instanceField) - } - is KtClass -> { - val companionObjectName = kotlinClass.getCompanionObjects().singleOrNull()?.name ?: return - val companionObjectField = lightClass.findFieldByName(companionObjectName, false) ?: return - replaceWithGetterInvocation(companionObjectField) - } - } - } -} - -private fun KtProperty.hasJvmFieldAnnotationOrConstModifier(): Boolean { - return hasModifier(KtTokens.CONST_KEYWORD) || annotationEntries.any { it.text == "@JvmField" } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/.inspection b/idea/testData/quickfix/migration/deprecatedStaticField/.inspection deleted file mode 100644 index 7b9ce4b0978..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/.inspection +++ /dev/null @@ -1 +0,0 @@ -org.jetbrains.kotlin.idea.inspections.DeprecatedUsageOfStaticFieldInspection \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/cleanUp.after.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/cleanUp.after.data.kt deleted file mode 100644 index 039e7e9ac11..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/cleanUp.after.data.kt +++ /dev/null @@ -1,22 +0,0 @@ -package a - -class A - -class Cl { - companion object { - @JvmField val property1 = A() - const val property2 = 2 - } -} - -interface Int { - companion object { - val property1 = A() - const val property2 = 2 - } -} - -object Obj { - @JvmField val property1 = A() - const val property2 = 2 -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/cleanUp.after.java b/idea/testData/quickfix/migration/deprecatedStaticField/cleanUp.after.java deleted file mode 100644 index 638753da50a..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/cleanUp.after.java +++ /dev/null @@ -1,19 +0,0 @@ -// "Cleanup code" "true" -import a.*; - -class B { - void bar() { - Cl.property1; - Cl.property2; - Cl.property1; - Cl.property2; - Int.Companion.getProperty1(); - Int.property2; - Int.Companion.getProperty1(); - Int.property2; - Obj.property1; - Obj.property2; - Obj.property1; - Obj.property2; - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/cleanUp.before.Main.java b/idea/testData/quickfix/migration/deprecatedStaticField/cleanUp.before.Main.java deleted file mode 100644 index 88e0c707b27..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/cleanUp.before.Main.java +++ /dev/null @@ -1,19 +0,0 @@ -// "Cleanup code" "true" -import a.*; - -class B { - void bar() { - Cl.property1; - Cl.property2; - Cl.Companion.getProperty1(); - Cl.Companion.getProperty2(); - Int.property1; - Int.property2; - Int.Companion.getProperty1(); - Int.Companion.getProperty2(); - Obj.property1; - Obj.property2; - Obj.INSTANCE.getProperty1(); - Obj.INSTANCE.getProperty2(); - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/cleanUp.before.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/cleanUp.before.data.kt deleted file mode 100644 index 0df46d23c48..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/cleanUp.before.data.kt +++ /dev/null @@ -1,22 +0,0 @@ -package a - -class A - -class Cl { - companion object { - val property1 = A() - val property2 = 2 - } -} - -interface Int { - companion object { - val property1 = A() - val property2 = 2 - } -} - -object Obj { - val property1 = A() - val property2 = 2 -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_const.after.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_const.after.data.kt deleted file mode 100644 index abdc80401df..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_const.after.data.kt +++ /dev/null @@ -1,7 +0,0 @@ -package a - -class A { - companion object Named { - const val property = 1 - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_const.after.java b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_const.after.java deleted file mode 100644 index b80b5299c59..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_const.after.java +++ /dev/null @@ -1,10 +0,0 @@ -// "Add 'const' modifier to a property" "true" -import a.A; - -class B { - void bar() { - A a = A.property; - A a2 = a.A.property; - A a3 = A.property; - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_const.before.Main.java b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_const.before.Main.java deleted file mode 100644 index b1433964f88..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_const.before.Main.java +++ /dev/null @@ -1,10 +0,0 @@ -// "Add 'const' modifier to a property" "true" -import a.A; - -class B { - void bar() { - A a = A.property; - A a2 = A.Named.getProperty(); - A a3 = A.property; - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_const.before.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_const.before.data.kt deleted file mode 100644 index a6cb926bd42..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_const.before.data.kt +++ /dev/null @@ -1,7 +0,0 @@ -package a - -class A { - companion object Named { - val property = 1 - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_getterReference.after.java b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_getterReference.after.java deleted file mode 100644 index 665f4bafab8..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_getterReference.after.java +++ /dev/null @@ -1,10 +0,0 @@ -// "Replace with getter invocation" "true" -import a.A; - -class B { - void bar() { - A a = a.A.Named.getProperty(); - A a2 = A.Named.getProperty(); - A a3 = A.property; - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_getterReference.before.Main.java b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_getterReference.before.Main.java deleted file mode 100644 index 42563954469..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_getterReference.before.Main.java +++ /dev/null @@ -1,10 +0,0 @@ -// "Replace with getter invocation" "true" -import a.A; - -class B { - void bar() { - A a = A.property; - A a2 = A.Named.getProperty(); - A a3 = A.property; - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_getterReference.before.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_getterReference.before.data.kt deleted file mode 100644 index a620f752ee1..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_getterReference.before.data.kt +++ /dev/null @@ -1,7 +0,0 @@ -package a - -class A { - companion object Named { - val property = A() - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_jvmField.after.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_jvmField.after.data.kt deleted file mode 100644 index 126b7623d00..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_jvmField.after.data.kt +++ /dev/null @@ -1,7 +0,0 @@ -package a - -class A { - companion object { - @JvmField val property = A() - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_jvmField.after.java b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_jvmField.after.java deleted file mode 100644 index aae1ac3a27a..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_jvmField.after.java +++ /dev/null @@ -1,10 +0,0 @@ -// "Annotate property with @JvmField" "true" -import a.A; - -class B { - void bar() { - A a = A.property; - A a2 = a.A.property; - A a3 = A.property; - } -} diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_jvmField.before.Main.java b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_jvmField.before.Main.java deleted file mode 100644 index 88795bf772b..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_jvmField.before.Main.java +++ /dev/null @@ -1,10 +0,0 @@ -// "Annotate property with @JvmField" "true" -import a.A; - -class B { - void bar() { - A a = A.property; - A a2 = A.Companion.getProperty(); - A a3 = A.property; - } -} diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_jvmField.before.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_jvmField.before.data.kt deleted file mode 100644 index 74198529b9a..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_jvmField.before.data.kt +++ /dev/null @@ -1,7 +0,0 @@ -package a - -class A { - companion object { - val property = A() - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_const.after.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_const.after.data.kt deleted file mode 100644 index b1751a529ec..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_const.after.data.kt +++ /dev/null @@ -1,7 +0,0 @@ -package a - -interface A { - companion object Named { - const val property = 1 - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_const.after.java b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_const.after.java deleted file mode 100644 index b80b5299c59..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_const.after.java +++ /dev/null @@ -1,10 +0,0 @@ -// "Add 'const' modifier to a property" "true" -import a.A; - -class B { - void bar() { - A a = A.property; - A a2 = a.A.property; - A a3 = A.property; - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_const.before.Main.java b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_const.before.Main.java deleted file mode 100644 index 253e13d3a18..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_const.before.Main.java +++ /dev/null @@ -1,10 +0,0 @@ -// "Add 'const' modifier to a property" "true" -import a.A; - -class B { - void bar() { - A a = A.property; - A a2 = A.Named.getProperty(); - A a3 = A.property; - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_const.before.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_const.before.data.kt deleted file mode 100644 index 25e2160304f..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_const.before.data.kt +++ /dev/null @@ -1,7 +0,0 @@ -package a - -interface A { - companion object Named { - val property = 1 - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_getterReference.after.java b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_getterReference.after.java deleted file mode 100644 index 16a61a54591..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_getterReference.after.java +++ /dev/null @@ -1,10 +0,0 @@ -// "Replace with getter invocation" "true" -import a.A; - -class B { - void bar() { - A a = a.A.Companion.getProperty(); - A a2 = A.Companion.getProperty(); - A a3 = A.property; - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_getterReference.before.Main.java b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_getterReference.before.Main.java deleted file mode 100644 index 0996002f61d..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_getterReference.before.Main.java +++ /dev/null @@ -1,10 +0,0 @@ -// "Replace with getter invocation" "true" -import a.A; - -class B { - void bar() { - A a = A.property; - A a2 = A.Companion.getProperty(); - A a3 = A.property; - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_getterReference.before.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_getterReference.before.data.kt deleted file mode 100644 index 4ba8a34f39e..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_getterReference.before.data.kt +++ /dev/null @@ -1,7 +0,0 @@ -package a - -interface A { - companion object { - val property = 1 - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_jvmField_unavailable.before.Main.java b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_jvmField_unavailable.before.Main.java deleted file mode 100644 index 28d251affa5..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_jvmField_unavailable.before.Main.java +++ /dev/null @@ -1,20 +0,0 @@ -// "Annotate property with @JvmField" "false" -import a.A; - -class B { - void bar() { - A a = A.property; - A a2 = A.Named.getProperty(); - A a3 = A.property; - } -} - -// ACTION: Add 'const' modifier to a property -// ACTION: Add static import for 'a.A.property' -// ACTION: Annotate 'property' as @Deprecated -// ACTION: Annotate 'property' as @NotNull -// ACTION: Annotate 'property' as @Nullable -// ACTION: Change variable 'a' type to 'int' -// ACTION: Migrate 'a' type to 'int' -// ACTION: Replace with getter invocation -// ACTION: Split into declaration and assignment \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_jvmField_unavailable.before.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_jvmField_unavailable.before.data.kt deleted file mode 100644 index 25e2160304f..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_jvmField_unavailable.before.data.kt +++ /dev/null @@ -1,7 +0,0 @@ -package a - -interface A { - companion object Named { - val property = 1 - } -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/object_const.after.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/object_const.after.data.kt deleted file mode 100644 index fd59288a85f..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/object_const.after.data.kt +++ /dev/null @@ -1,5 +0,0 @@ -package a - -object Obj { - const val property = 1 -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/object_const.after.java b/idea/testData/quickfix/migration/deprecatedStaticField/object_const.after.java deleted file mode 100644 index 798a8cc707d..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/object_const.after.java +++ /dev/null @@ -1,11 +0,0 @@ -// "Add 'const' modifier to a property" "true" -import a.Obj; -import a.A; - -class B { - void bar() { - A a = Obj.property; - A a2 = a.Obj.property; - A a3 = Obj.property; - } -} diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/object_const.before.Main.java b/idea/testData/quickfix/migration/deprecatedStaticField/object_const.before.Main.java deleted file mode 100644 index 65e5c43bd8f..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/object_const.before.Main.java +++ /dev/null @@ -1,11 +0,0 @@ -// "Add 'const' modifier to a property" "true" -import a.Obj; -import a.A; - -class B { - void bar() { - A a = Obj.property; - A a2 = Obj.INSTANCE.getProperty(); - A a3 = Obj.property; - } -} diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/object_const.before.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/object_const.before.data.kt deleted file mode 100644 index 621f7813639..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/object_const.before.data.kt +++ /dev/null @@ -1,5 +0,0 @@ -package a - -object Obj { - val property = 1 -} \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/object_const_unavailable.before.Main.java b/idea/testData/quickfix/migration/deprecatedStaticField/object_const_unavailable.before.Main.java deleted file mode 100644 index 8a93a5ec08e..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/object_const_unavailable.before.Main.java +++ /dev/null @@ -1,16 +0,0 @@ -// "Add 'const' modifier to a property" "false" -import a.Obj; -import a.A; - -class B { - void bar() { - A a = Obj.property; - Obj.INSTANCE.getProperty(); - } -} - -// ACTION: Annotate 'property' as @Deprecated -// ACTION: Add static import for 'a.Obj.property' -// ACTION: Annotate property with @JvmField -// ACTION: Split into declaration and assignment -// ACTION: Replace with getter invocation \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/object_const_unavailable.before.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/object_const_unavailable.before.data.kt deleted file mode 100644 index 17fa8e5b676..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/object_const_unavailable.before.data.kt +++ /dev/null @@ -1,7 +0,0 @@ -package a - -object Obj { - val property = A() -} - -class A \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/object_getterReference.after.java b/idea/testData/quickfix/migration/deprecatedStaticField/object_getterReference.after.java deleted file mode 100644 index 51ec25914c6..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/object_getterReference.after.java +++ /dev/null @@ -1,11 +0,0 @@ -// "Replace with getter invocation" "true" -import a.Obj; -import a.A; - -class B { - void bar() { - A a = a.Obj.INSTANCE.getProperty(); - A a2 = Obj.getProperty(); - A a3 = Obj.property; - } -} diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/object_getterReference.before.Main.java b/idea/testData/quickfix/migration/deprecatedStaticField/object_getterReference.before.Main.java deleted file mode 100644 index 806aa1d0eb9..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/object_getterReference.before.Main.java +++ /dev/null @@ -1,11 +0,0 @@ -// "Replace with getter invocation" "true" -import a.Obj; -import a.A; - -class B { - void bar() { - A a = Obj.property; - A a2 = Obj.getProperty(); - A a3 = Obj.property; - } -} diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/object_getterReference.before.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/object_getterReference.before.data.kt deleted file mode 100644 index 17fa8e5b676..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/object_getterReference.before.data.kt +++ /dev/null @@ -1,7 +0,0 @@ -package a - -object Obj { - val property = A() -} - -class A \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/object_jvmField.after.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/object_jvmField.after.data.kt deleted file mode 100644 index a2cbc3fa642..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/object_jvmField.after.data.kt +++ /dev/null @@ -1,7 +0,0 @@ -package a - -object Obj { - @JvmField val property = A() -} - -class A \ No newline at end of file diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/object_jvmField.after.java b/idea/testData/quickfix/migration/deprecatedStaticField/object_jvmField.after.java deleted file mode 100644 index 4d2db5e6cc9..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/object_jvmField.after.java +++ /dev/null @@ -1,11 +0,0 @@ -// "Annotate property with @JvmField" "true" -import a.Obj; -import a.A; - -class B { - void bar() { - A a = Obj.property; - A a2 = a.Obj.property; - A a3 = Obj.property; - } -} diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/object_jvmField.before.Main.java b/idea/testData/quickfix/migration/deprecatedStaticField/object_jvmField.before.Main.java deleted file mode 100644 index 6c9f2ae7c09..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/object_jvmField.before.Main.java +++ /dev/null @@ -1,11 +0,0 @@ -// "Annotate property with @JvmField" "true" -import a.Obj; -import a.A; - -class B { - void bar() { - A a = Obj.property; - A a2 = Obj.getProperty(); - A a3 = Obj.property; - } -} diff --git a/idea/testData/quickfix/migration/deprecatedStaticField/object_jvmField.before.data.kt b/idea/testData/quickfix/migration/deprecatedStaticField/object_jvmField.before.data.kt deleted file mode 100644 index 17fa8e5b676..00000000000 --- a/idea/testData/quickfix/migration/deprecatedStaticField/object_jvmField.before.data.kt +++ /dev/null @@ -1,7 +0,0 @@ -package a - -object Obj { - val property = A() -} - -class A \ No newline at end of file diff --git a/idea/tests/org/jetbrains/kotlin/idea/quickfix/QuickFixMultiFileTestGenerated.java b/idea/tests/org/jetbrains/kotlin/idea/quickfix/QuickFixMultiFileTestGenerated.java index acc943c9b12..4f78108a176 100644 --- a/idea/tests/org/jetbrains/kotlin/idea/quickfix/QuickFixMultiFileTestGenerated.java +++ b/idea/tests/org/jetbrains/kotlin/idea/quickfix/QuickFixMultiFileTestGenerated.java @@ -1234,81 +1234,6 @@ public class QuickFixMultiFileTestGenerated extends AbstractQuickFixMultiFileTes } } - @TestMetadata("idea/testData/quickfix/migration/deprecatedStaticField") - @TestDataPath("$PROJECT_ROOT") - @RunWith(JUnit3RunnerWithInners.class) - public static class DeprecatedStaticField extends AbstractQuickFixMultiFileTest { - public void testAllFilesPresentInDeprecatedStaticField() throws Exception { - KotlinTestUtils.assertAllTestsPresentByMetadata(this.getClass(), new File("idea/testData/quickfix/migration/deprecatedStaticField"), Pattern.compile("^(\\w+)\\.((before\\.Main\\.\\w+)|(test))$"), true); - } - - @TestMetadata("cleanUp.before.Main.java") - public void testCleanUp() throws Exception { - String fileName = KotlinTestUtils.navigationMetadata("idea/testData/quickfix/migration/deprecatedStaticField/cleanUp.before.Main.java"); - doTestWithExtraFile(fileName); - } - - @TestMetadata("companionObjectOfClass_const.before.Main.java") - public void testCompanionObjectOfClass_const() throws Exception { - String fileName = KotlinTestUtils.navigationMetadata("idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_const.before.Main.java"); - doTestWithExtraFile(fileName); - } - - @TestMetadata("companionObjectOfClass_getterReference.before.Main.java") - public void testCompanionObjectOfClass_getterReference() throws Exception { - String fileName = KotlinTestUtils.navigationMetadata("idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_getterReference.before.Main.java"); - doTestWithExtraFile(fileName); - } - - @TestMetadata("companionObjectOfClass_jvmField.before.Main.java") - public void testCompanionObjectOfClass_jvmField() throws Exception { - String fileName = KotlinTestUtils.navigationMetadata("idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfClass_jvmField.before.Main.java"); - doTestWithExtraFile(fileName); - } - - @TestMetadata("companionObjectOfInterface_const.before.Main.java") - public void testCompanionObjectOfInterface_const() throws Exception { - String fileName = KotlinTestUtils.navigationMetadata("idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_const.before.Main.java"); - doTestWithExtraFile(fileName); - } - - @TestMetadata("companionObjectOfInterface_getterReference.before.Main.java") - public void testCompanionObjectOfInterface_getterReference() throws Exception { - String fileName = KotlinTestUtils.navigationMetadata("idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_getterReference.before.Main.java"); - doTestWithExtraFile(fileName); - } - - @TestMetadata("companionObjectOfInterface_jvmField_unavailable.before.Main.java") - public void testCompanionObjectOfInterface_jvmField_unavailable() throws Exception { - String fileName = KotlinTestUtils.navigationMetadata("idea/testData/quickfix/migration/deprecatedStaticField/companionObjectOfInterface_jvmField_unavailable.before.Main.java"); - doTestWithExtraFile(fileName); - } - - @TestMetadata("object_const.before.Main.java") - public void testObject_const() throws Exception { - String fileName = KotlinTestUtils.navigationMetadata("idea/testData/quickfix/migration/deprecatedStaticField/object_const.before.Main.java"); - doTestWithExtraFile(fileName); - } - - @TestMetadata("object_const_unavailable.before.Main.java") - public void testObject_const_unavailable() throws Exception { - String fileName = KotlinTestUtils.navigationMetadata("idea/testData/quickfix/migration/deprecatedStaticField/object_const_unavailable.before.Main.java"); - doTestWithExtraFile(fileName); - } - - @TestMetadata("object_getterReference.before.Main.java") - public void testObject_getterReference() throws Exception { - String fileName = KotlinTestUtils.navigationMetadata("idea/testData/quickfix/migration/deprecatedStaticField/object_getterReference.before.Main.java"); - doTestWithExtraFile(fileName); - } - - @TestMetadata("object_jvmField.before.Main.java") - public void testObject_jvmField() throws Exception { - String fileName = KotlinTestUtils.navigationMetadata("idea/testData/quickfix/migration/deprecatedStaticField/object_jvmField.before.Main.java"); - doTestWithExtraFile(fileName); - } - } - @TestMetadata("idea/testData/quickfix/migration/javaAnnotationPositionedArguments") @TestDataPath("$PROJECT_ROOT") @RunWith(JUnit3RunnerWithInners.class)