From e6b4980e029645818e6d36a7e635481c8ad06bb0 Mon Sep 17 00:00:00 2001 From: Tagir Valeev Date: Tue, 28 Jan 2020 18:44:32 +0700 Subject: [PATCH] Avoid 'Test-only called in production' warning Review ID: IDEA-CR-57707 GitOrigin-RevId: ba8a9d8e0443134fa78a6a0112f343d9cf356b9f --- .../refactoring/inline/InlineLocalHandler.java | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/java/java-impl/src/com/intellij/refactoring/inline/InlineLocalHandler.java b/java/java-impl/src/com/intellij/refactoring/inline/InlineLocalHandler.java index 473790188f8a..027f7e716c06 100644 --- a/java/java-impl/src/com/intellij/refactoring/inline/InlineLocalHandler.java +++ b/java/java-impl/src/com/intellij/refactoring/inline/InlineLocalHandler.java @@ -75,7 +75,7 @@ public class InlineLocalHandler extends JavaInlineActionHandler { public void inlineElement(Project project, Editor editor, PsiElement element) { final PsiReference psiReference = TargetElementUtil.findReference(editor); final PsiReferenceExpression refExpr = psiReference instanceof PsiReferenceExpression ? (PsiReferenceExpression)psiReference : null; - inlineVariable(project, editor, (PsiVariable)element, refExpr); + doInline(project, editor, (PsiVariable)element, refExpr); } /** @@ -85,7 +85,7 @@ public class InlineLocalHandler extends JavaInlineActionHandler { @NotNull final Editor editor, @NotNull PsiLocalVariable local, PsiReferenceExpression refExpr) { - inlineVariable(project, editor, local, refExpr); + doInline(project, editor, local, refExpr); } @TestOnly @@ -93,6 +93,13 @@ public class InlineLocalHandler extends JavaInlineActionHandler { @NotNull final Editor editor, @NotNull PsiVariable var, PsiReferenceExpression refExpr) { + doInline(project, editor, var, refExpr); + } + + private static void doInline(@NotNull final Project project, + @NotNull final Editor editor, + @NotNull PsiVariable var, + PsiReferenceExpression refExpr) { if (!CommonRefactoringUtil.checkReadOnlyStatus(project, var)) return; Collection allRefs = ProgressManager.getInstance().runProcessWithProgressSynchronously( () -> ReferencesSearch.search(var).mapping(PsiReference::getElement).findAll(),