IDEA-259050 refactor this: don't suggest Move Instance Method under constructor declaration

GitOrigin-RevId: f3fd88cdbc1ec8eb6cfd916673b4b22aea7e6f46

java/java-impl/src/com/intellij/refactoring/move/moveInstanceMethod/MoveInstanceMethodHandlerDelegate.java

@@ -36,6 +36,7 @@ public class MoveInstanceMethodHandlerDelegate extends MoveHandlerDelegate {
     if (!(element instanceof PsiMethod)) return false;
     if (element instanceof SyntheticElement) return false;
     PsiMethod method = (PsiMethod) element;
+    if (method.isConstructor()) return false;
     if (method.hasModifierProperty(PsiModifier.STATIC)) return false;
     return targetContainer == null || super.canMove(elements, targetContainer, reference);
   }

java/java-tests/testData/refactoring/refactorThis/MoveIsFilteredOnConstructor.java

@@ -0,0 +1,3 @@
+public class Test {
+    Test<caret>() {}
+}

java/java-tests/testSrc/com/intellij/java/refactoring/RefactorThisTest.kt

@@ -103,6 +103,10 @@ class RefactorThisTest: LightJavaCodeInsightTestCase() {
     assertTrue(doActionExists<MoveAction>())
   }
 
+  fun testMoveOnClassDeclaration() {
+    assertTrue(doActionExists<MoveAction>())
+  }
+
   private inline fun <reified A> doActionExists(): Boolean {
     configureByFile("$BASE_PATH/${getTestName(false)}.java")
     return findAvailableActions().any { action -> action is A }