Tests for Sealed Class move + Fix invalid move declaration test

MoveDeclarationToSeparateFile.testSealed test was threated to check that the intention is not applicable but it currently is.

idea/src/org/jetbrains/kotlin/idea/roots/projectRootUtils.kt

@@ -73,12 +73,10 @@ private fun Module.collectKotlinAwareDestinationSourceRoots(): List<VirtualFile>
 }
 
 fun isOutsideSourceRootSet(psiFile : PsiFile?, sourceRootTypes: Set<JpsModuleSourceRootType<*>>): Boolean {
-    if (psiFile == null) return false
+    if (psiFile == null || psiFile is PsiCodeFragment) return false
-    if (psiFile is PsiCodeFragment) return false
+    val file = psiFile.virtualFile ?: return false
-    val file = psiFile.getVirtualFile()
+    if (file.fileSystem is NonPhysicalFileSystem) return false
-    if (file == null) return false
+    val projectFileIndex = ProjectRootManager.getInstance(psiFile.project).fileIndex
-    if (file.getFileSystem() is NonPhysicalFileSystem) return false
-    val projectFileIndex = ProjectRootManager.getInstance(psiFile.getProject()).getFileIndex()
     return !projectFileIndex.isUnderSourceRootOfType(file, sourceRootTypes) && !projectFileIndex.isInLibrary(file)
 }
 

idea/testData/intentions/moveDeclarationToSeparateFile/sealed.kt

@@ -1,3 +1,3 @@
-// IS_APPLICABLE: false
+// INTENTION_TEXT: Extract 'A' and subclasses from current file
 sealed class A<caret>
 class B : A()

idea/testData/intentions/moveDeclarationToSeparateFile/sealed.kt.after

@@ -0,0 +1 @@
+

idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithImplsToAnotherPackage/after/bar/SealedClass.kt

@@ -0,0 +1,5 @@
+package bar
+
+public sealed class SealedClass {
+    public class Impl1 : SealedClass() {}
+}

idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithImplsToAnotherPackage/after/foo/KotlinReferences.kt

@@ -0,0 +1,5 @@
+package foo
+
+import bar.SealedClass
+
+val v = SealedClass::Impl1

idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithImplsToAnotherPackage/after/foo/SealedClass.kt

@@ -0,0 +1,5 @@
+package foo
+
+import bar.SealedClass
+
+public class Impl2 : SealedClass() {}

idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithImplsToAnotherPackage/before/foo/KotlinReferences.kt

@@ -0,0 +1,3 @@
+package foo
+
+val v = SealedClass::Impl1

idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithImplsToAnotherPackage/before/foo/SealedClass.kt

@@ -0,0 +1,7 @@
+package foo
+
+public sealed class <caret>SealedClass {
+    public class Impl1 : SealedClass() {}
+}
+
+public class Impl2 : SealedClass() {}

idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithImplsToAnotherPackage/conflicts.txt

@@ -0,0 +1 @@
+Sealed class 'SealedClass' must be moved with all its subclasses

idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithImplsToAnotherPackage/moveSealedClassWithNestedImplsToAnotherPackage.test

@@ -0,0 +1,5 @@
+{
+  "mainFile": "foo/SealedClass.kt",
+  "type": "MOVE_KOTLIN_TOP_LEVEL_DECLARATIONS",
+  "targetPackage": "bar"
+}

idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithNestedImplsToAnotherPackage/after/bar/SealedClass.kt

@@ -0,0 +1,6 @@
+package bar
+
+public sealed class SealedClass {
+    public class Impl1 : SealedClass() {}
+    public class Impl2 : SealedClass() {}
+}

idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithNestedImplsToAnotherPackage/after/foo/KotlinReferences.kt

@@ -0,0 +1,5 @@
+package foo
+
+import bar.SealedClass
+
+val v = SealedClass::Impl1

idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithNestedImplsToAnotherPackage/after/foo/SealedClass.kt

@@ -0,0 +1,2 @@
+package foo
+

idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithNestedImplsToAnotherPackage/before/foo/KotlinReferences.kt

@@ -0,0 +1,3 @@
+package foo
+
+val v = SealedClass::Impl1

idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithNestedImplsToAnotherPackage/before/foo/SealedClass.kt

@@ -0,0 +1,6 @@
+package foo
+
+public sealed class <caret>SealedClass {
+    public class Impl1 : SealedClass() {}
+    public class Impl2 : SealedClass() {}
+}

idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithNestedImplsToAnotherPackage/moveSealedClassWithNestedImplsToAnotherPackage.test

@@ -0,0 +1,5 @@
+{
+  "mainFile": "foo/SealedClass.kt",
+  "type": "MOVE_KOTLIN_TOP_LEVEL_DECLARATIONS",
+  "targetPackage": "bar"
+}

idea/tests/org/jetbrains/kotlin/idea/refactoring/move/MoveTestGenerated.java

@@ -739,6 +739,16 @@ public class MoveTestGenerated extends AbstractMoveTest {
         runTest("idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/movePropertyToPackage/movePropertyToPackage.test");
     }
 
+    @TestMetadata("kotlin/moveTopLevelDeclarations/moveSealedClassWithImplsToAnotherPackage/moveSealedClassWithNestedImplsToAnotherPackage.test")
+    public void testKotlin_moveTopLevelDeclarations_moveSealedClassWithImplsToAnotherPackage_MoveSealedClassWithNestedImplsToAnotherPackage() throws Exception {
+        runTest("idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithImplsToAnotherPackage/moveSealedClassWithNestedImplsToAnotherPackage.test");
+    }
+
+    @TestMetadata("kotlin/moveTopLevelDeclarations/moveSealedClassWithNestedImplsToAnotherPackage/moveSealedClassWithNestedImplsToAnotherPackage.test")
+    public void testKotlin_moveTopLevelDeclarations_moveSealedClassWithNestedImplsToAnotherPackage_MoveSealedClassWithNestedImplsToAnotherPackage() throws Exception {
+        runTest("idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/moveSealedClassWithNestedImplsToAnotherPackage/moveSealedClassWithNestedImplsToAnotherPackage.test");
+    }
+
     @TestMetadata("kotlin/moveTopLevelDeclarations/objectAlreadyInaccessible/objectAlreadyInaccessible.test")
     public void testKotlin_moveTopLevelDeclarations_objectAlreadyInaccessible_ObjectAlreadyInaccessible() throws Exception {
         runTest("idea/testData/refactoring/move/kotlin/moveTopLevelDeclarations/objectAlreadyInaccessible/objectAlreadyInaccessible.test");