#KT-9938 Fixed

Code review fixes

idea/resources/intentionDescriptions/ConvertPropertyInitializerToGetterIntention/after.kt.template

@@ -1,4 +1,4 @@
 class A {
     val foo: Int
-        get() = 1
+        <spot>get() = 1</spot>
 }

idea/src/org/jetbrains/kotlin/idea/intentions/ConvertPropertyInitializerToGetterIntention.kt

@@ -33,7 +33,6 @@ class ConvertPropertyInitializerToGetterIntention : SelfTargetingIntention<KtPro
                && element.getter == null
                && !element.isExtensionDeclaration()
                && !element.isLocal
-               && !element.isTopLevel
     }
 
     override fun applyTo(property: KtProperty, editor: Editor) {

idea/testData/intentions/convertPropertyInitializerToGetter/inapplicableIfTopLevelDeclaration.kt

@@ -1,3 +0,0 @@
-// IS_APPLICABLE: false
-
-val x: Int = 1<caret>

idea/testData/quickfix/autoImports/noImportInQualifiedExpressionNotFirst.before.Main.kt

@@ -1,4 +1,5 @@
 // "class org.jetbrains.kotlin.idea.quickfix.AutoImportFix" "false"
+// ACTION: Convert property initializer to getter
 // ACTION: Create class 'SomeTest'
 // ERROR: Unresolved reference: SomeTest
 

idea/testData/quickfix/autoImports/noImportInSafeQualifiedExpressionNotFirst.before.Main.kt

@@ -1,6 +1,7 @@
 // "class org.jetbrains.kotlin.idea.quickfix.AutoImportFix" "false"
 // ERROR: Unresolved reference: SomeTest
 // ERROR: Expression expected, but a package name found
+// ACTION: Convert property initializer to getter
 // ACTION: Create class 'SomeTest'
 // ACTION: Replace safe access expression with 'if' expression
 

idea/testData/quickfix/autoImports/noImportsForClassInExcludedPackage.before.Main.kt

@@ -1,4 +1,5 @@
 // "class org.jetbrains.kotlin.idea.quickfix.AutoImportFix" "false"
+// ACTION: Convert property initializer to getter
 // ACTION: Create class 'SomeClass'
 // ACTION: Create function 'SomeClass'
 // ERROR: Unresolved reference: SomeClass

idea/testData/quickfix/autoImports/noImportsForExcludedClass.before.Main.kt

@@ -1,4 +1,5 @@
 // "class org.jetbrains.kotlin.idea.quickfix.AutoImportFix" "false"
+// ACTION: Convert property initializer to getter
 // ACTION: Create class 'ExcludedClass'
 // ACTION: Create function 'ExcludedClass'
 // ERROR: Unresolved reference: ExcludedClass

idea/testData/quickfix/autoImports/noImportsForFunctionInExcludedPackage.before.Main.kt

@@ -1,4 +1,5 @@
 // "class org.jetbrains.kotlin.idea.quickfix.AutoImportFix" "false"
+// ACTION: Convert property initializer to getter
 // ACTION: Create function 'someFunction'
 // ERROR: Unresolved reference: someFunction
 

idea/testData/quickfix/createFromUsage/createVariable/localVariable/onTopLevel.kt

@@ -1,4 +1,5 @@
 // "Create local variable 'foo'" "false"
+// ACTION: Convert property initializer to getter
 // ACTION: Create property 'foo'
 // ERROR: Unresolved reference: foo
 

idea/testData/quickfix/createFromUsage/createVariable/parameter/inPropertyInitializerNoClass.kt

@@ -1,4 +1,5 @@
 // "Create parameter 'foo'" "false"
+// ACTION: Convert property initializer to getter
 // ACTION: Create property 'foo'
 // ERROR: Unresolved reference: foo
 

idea/tests/org/jetbrains/kotlin/idea/intentions/IntentionTestGenerated.java

@@ -3739,12 +3739,6 @@ public class IntentionTestGenerated extends AbstractIntentionTest {
             doTest(fileName);
         }
 
-        @TestMetadata("inapplicableIfTopLevelDeclaration.kt")
-        public void testInapplicableIfTopLevelDeclaration() throws Exception {
-            String fileName = KotlinTestUtils.navigationMetadata("idea/testData/intentions/convertPropertyInitializerToGetter/inapplicableIfTopLevelDeclaration.kt");
-            doTest(fileName);
-        }
-
         @TestMetadata("propertyWithInitializerWithSetter.kt")
         public void testPropertyWithInitializerWithSetter() throws Exception {
             String fileName = KotlinTestUtils.navigationMetadata("idea/testData/intentions/convertPropertyInitializerToGetter/propertyWithInitializerWithSetter.kt");