[java-intentions] Remove unused assignment/initializer: preview-friendly

GitOrigin-RevId: de9da180223f231638eb51804b7ed9c26c36d772
2026-04-21 14:01:44 +07:00 · 2022-07-29 15:47:33 +02:00
parent 7e1064ea3c
commit 1361a10017
35 changed files with 125 additions and 76 deletions
--- a/java/java-analysis-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/RemoveUnusedVariableUtil.java
+++ b/java/java-analysis-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/RemoveUnusedVariableUtil.java
@@ -41,6 +41,13 @@ public final class RemoveUnusedVariableUtil {
    return !writes.isEmpty();
  }

+  public static RemoveMode getModeForPreview(PsiExpression element, @Nullable PsiVariable variableToIgnore) {
+    List<PsiElement> sideEffects = new ArrayList<>();
+    boolean hasSideEffects = checkSideEffects(element, variableToIgnore, sideEffects);
+    if (!hasSideEffects || sideEffects.isEmpty()) return RemoveMode.DELETE_ALL;
+    return RemoveMode.MAKE_STATEMENT;
+  }
+
  public static PsiElement replaceElementWithExpression(PsiExpression expression,
                                                        PsiElementFactory factory,
                                                        PsiElement element) throws IncorrectOperationException {
--- a/java/java-impl/src/com/intellij/codeInspection/RemoveAssignmentFix.java
+++ b/java/java-impl/src/com/intellij/codeInspection/RemoveAssignmentFix.java
@@ -16,13 +16,16 @@
 package com.intellij.codeInspection;

 import com.intellij.codeInsight.FileModificationService;
+import com.intellij.codeInsight.daemon.impl.quickfix.RemoveUnusedVariableUtil;
 import com.intellij.codeInsight.editorActions.DeclarationJoinLinesHandler;
+import com.intellij.codeInsight.intention.preview.IntentionPreviewInfo;
 import com.intellij.java.JavaBundle;
 import com.intellij.openapi.application.WriteAction;
 import com.intellij.openapi.project.Project;
 import com.intellij.psi.*;
 import com.intellij.psi.tree.IElementType;
 import com.intellij.psi.util.PsiUtil;
+import com.intellij.util.ObjectUtils;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;

@@ -36,15 +39,8 @@ public class RemoveAssignmentFix extends RemoveInitializerFix {
  @Override
  public void applyFix(@NotNull Project project, @NotNull ProblemDescriptor descriptor) {
    final PsiElement element = descriptor.getPsiElement();
-    final PsiElement parent;
-    if (element instanceof PsiReferenceExpression) {
-      parent = element.getParent();
-    } else {
-      parent = element;
-    }
-    if (!(parent instanceof PsiAssignmentExpression)) return;
-
-    PsiAssignmentExpression parentExpr = (PsiAssignmentExpression)parent;
+    PsiAssignmentExpression parentExpr = getAssignment(descriptor);
+    if (parentExpr == null) return;
    PsiElement gParentExpr = parentExpr.getParent();
    PsiExpression initializer = getInitializer(parentExpr);
    if (initializer == null) return;
@@ -63,9 +59,37 @@ public class RemoveAssignmentFix extends RemoveInitializerFix {
    PsiElement resolve = resolveExpression(element, parentExpr);
    if (!(resolve instanceof PsiVariable)) return;

-    sideEffectAwareRemove(project, initializer, parent, (PsiVariable)resolve);
+    sideEffectAwareRemove(project, initializer, parentExpr, (PsiVariable)resolve);
  }

+  PsiAssignmentExpression getAssignment(@NotNull ProblemDescriptor descriptor) {
+    final PsiElement element = descriptor.getPsiElement();
+    final PsiElement parent = element instanceof PsiReferenceExpression ? element.getParent() : element;
+    return ObjectUtils.tryCast(parent, PsiAssignmentExpression.class);
+  }
+
+  @Override
+  public @NotNull IntentionPreviewInfo generatePreview(@NotNull Project project, @NotNull ProblemDescriptor previewDescriptor) {
+    final PsiElement element = previewDescriptor.getPsiElement();
+    PsiAssignmentExpression parentExpr = getAssignment(previewDescriptor);
+    if (parentExpr == null) return IntentionPreviewInfo.EMPTY;
+    PsiElement gParentExpr = parentExpr.getParent();
+    PsiExpression initializer = getInitializer(parentExpr);
+    if (initializer == null) return IntentionPreviewInfo.EMPTY;
+    if (mayBeFixedWithoutSideEffect(gParentExpr)) {
+      PsiElement target = gParentExpr instanceof PsiParenthesizedExpression ? gParentExpr : parentExpr;
+      target.replace(initializer);
+      return IntentionPreviewInfo.DIFF;
+    }
+    PsiVariable resolve = ObjectUtils.tryCast(resolveExpression(element, parentExpr), PsiVariable.class);
+    if (resolve == null) return IntentionPreviewInfo.EMPTY;
+
+    RemoveUnusedVariableUtil.RemoveMode res = RemoveUnusedVariableUtil.getModeForPreview(initializer, resolve);
+    doRemove(project, initializer, parentExpr, resolve, resolve.getParent(), res);
+    return IntentionPreviewInfo.DIFF;
+  }
+
+
  @Nullable
  private static PsiExpression getInitializer(@NotNull PsiAssignmentExpression assignmentExpr) {
    final IElementType operationSign = assignmentExpr.getOperationTokenType();
--- a/java/java-impl/src/com/intellij/codeInspection/RemoveInitializerFix.java
+++ b/java/java-impl/src/com/intellij/codeInspection/RemoveInitializerFix.java
@@ -19,15 +19,16 @@ import com.intellij.codeInsight.BlockUtils;
 import com.intellij.codeInsight.FileModificationService;
 import com.intellij.codeInsight.daemon.impl.quickfix.RemoveUnusedVariableFix;
 import com.intellij.codeInsight.daemon.impl.quickfix.RemoveUnusedVariableUtil;
+import com.intellij.codeInsight.intention.preview.IntentionPreviewInfo;
 import com.intellij.java.JavaBundle;
 import com.intellij.openapi.application.WriteAction;
-import com.intellij.openapi.diagnostic.Logger;
 import com.intellij.openapi.fileEditor.FileEditorManager;
 import com.intellij.openapi.project.Project;
 import com.intellij.psi.*;
 import com.intellij.psi.util.PsiExpressionTrimRenderer;
 import com.intellij.psi.util.PsiTreeUtil;
 import com.intellij.psi.util.PsiUtil;
+import com.intellij.util.ObjectUtils;
 import com.siyeh.ig.psiutils.CommentTracker;
 import com.siyeh.ig.psiutils.ExpressionUtils;
 import org.jetbrains.annotations.NotNull;
@@ -36,7 +37,6 @@ import java.util.ArrayList;
 import java.util.List;

 public class RemoveInitializerFix implements LocalQuickFix {
-  private static final Logger LOG = Logger.getInstance(RemoveInitializerFix.class);

  @Override
  @NotNull
@@ -96,38 +96,56 @@ public class RemoveInitializerFix implements LocalQuickFix {
    else {
      res = RemoveUnusedVariableUtil.RemoveMode.DELETE_ALL;
    }
-    WriteAction.run(() -> {
-      if (res == RemoveUnusedVariableUtil.RemoveMode.DELETE_ALL) {
-        if (elementToDelete instanceof PsiExpression && !ExpressionUtils.isVoidContext((PsiExpression)elementToDelete) &&
-            !PsiTreeUtil.isAncestor(variable, elementToDelete, true)) {
-          String name = variable.getName();
-          if (name != null) {
-            new CommentTracker().replaceAndRestoreComments(elementToDelete, name);
-            return;
-          }
-        }
-        elementToDelete.delete();
-      }
-      else if (res == RemoveUnusedVariableUtil.RemoveMode.MAKE_STATEMENT) {
-        final PsiElementFactory factory = JavaPsiFacade.getElementFactory(project);
-        final PsiStatement statementFromText = factory.createStatementFromText(psiInitializer.getText() + ";", null);
-        final PsiElement parent = elementToDelete.getParent();
-        if (parent instanceof PsiExpressionStatement) {
-          parent.replace(statementFromText);
-        }
-        else {
-          elementToDelete.delete();
-          if (declaration instanceof PsiClass) {
-            PsiClassInitializer initializer = factory.createClassInitializer();
-            initializer = (PsiClassInitializer)declaration.addAfter(initializer, variable);
-            initializer.getBody().add(statementFromText);
-            return;
-          }
-          PsiElement grandParent = declaration.getParent();
-          BlockUtils.addBefore(((PsiStatement) (grandParent instanceof PsiForStatement ? grandParent : declaration)), statementFromText);
-        }
-      }
-    });
+    WriteAction.run(() -> doRemove(project, psiInitializer, elementToDelete, variable, declaration, res));
    return true;
  }
+
+  @Override
+  public @NotNull IntentionPreviewInfo generatePreview(@NotNull Project project, @NotNull ProblemDescriptor previewDescriptor) {
+    PsiExpression psiInitializer = ObjectUtils.tryCast(previewDescriptor.getPsiElement(), PsiExpression.class);
+    if (psiInitializer == null) return IntentionPreviewInfo.EMPTY;
+    PsiVariable variable = ObjectUtils.tryCast(psiInitializer.getParent(), PsiVariable.class);
+    if (variable == null) return IntentionPreviewInfo.EMPTY;
+    RemoveUnusedVariableUtil.RemoveMode res = RemoveUnusedVariableUtil.getModeForPreview(psiInitializer, variable);
+    doRemove(project, psiInitializer, psiInitializer, variable, variable.getParent(), res);
+    return IntentionPreviewInfo.DIFF;
+  }
+
+  static void doRemove(@NotNull Project project,
+                       @NotNull PsiExpression psiInitializer,
+                       @NotNull PsiElement elementToDelete,
+                       @NotNull PsiVariable variable,
+                       PsiElement declaration,
+                       @NotNull RemoveUnusedVariableUtil.RemoveMode res) {
+    if (res == RemoveUnusedVariableUtil.RemoveMode.DELETE_ALL) {
+      if (elementToDelete instanceof PsiExpression && !ExpressionUtils.isVoidContext((PsiExpression)elementToDelete) &&
+          !PsiTreeUtil.isAncestor(variable, elementToDelete, true)) {
+        String name = variable.getName();
+        if (name != null) {
+          new CommentTracker().replaceAndRestoreComments(elementToDelete, name);
+          return;
+        }
+      }
+      elementToDelete.delete();
+    }
+    else if (res == RemoveUnusedVariableUtil.RemoveMode.MAKE_STATEMENT) {
+      final PsiElementFactory factory = JavaPsiFacade.getElementFactory(project);
+      final PsiStatement statementFromText = factory.createStatementFromText(psiInitializer.getText() + ";", null);
+      final PsiElement parent = elementToDelete.getParent();
+      if (parent instanceof PsiExpressionStatement) {
+        parent.replace(statementFromText);
+      }
+      else {
+        elementToDelete.delete();
+        if (declaration instanceof PsiClass) {
+          PsiClassInitializer initializer = factory.createClassInitializer();
+          initializer = (PsiClassInitializer)declaration.addAfter(initializer, variable);
+          initializer.getBody().add(statementFromText);
+          return;
+        }
+        PsiElement grandParent = declaration.getParent();
+        BlockUtils.addBefore(((PsiStatement)(grandParent instanceof PsiForStatement ? grandParent : declaration)), statementFromText);
+      }
+    }
+  }
 }
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterAssignmentOperation.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterAssignmentOperation.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
  public String getContexts(final String env) {
    String contexts = "a";
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterCallAsArgument.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterCallAsArgument.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class Test {
  void foo() {
    String var;
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterClassInitializer.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterClassInitializer.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
  static int n;
  static {
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterFieldChainAssignment.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterFieldChainAssignment.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class Foo {
  int sent;

--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterFieldInitializer.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterFieldInitializer.java
@@ -1,4 +1,4 @@
-// "Remove redundant initializer" "true"
+// "Remove redundant initializer" "true-preview"
 class A {
  int n;
  { n = 1; }
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterFieldInitializerWithSideEffect.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterFieldInitializerWithSideEffect.java
@@ -1,4 +1,4 @@
-// "Remove redundant initializer" "true"
+// "Remove redundant initializer" "true-preview"
 class A {
  private String myFoo;

--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterInCondition.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterInCondition.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class X {
  void test(boolean b) {
    if (b) {}
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterInForDeclaration.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterInForDeclaration.java
@@ -1,4 +1,4 @@
-// "Remove redundant initializer" "true"
+// "Remove redundant initializer" "true-preview"
 class A {
 void testFor() {
        if (true) {
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterInReturnStatement.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterInReturnStatement.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
  int v() {
        int x;
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterInsideExpression.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterInsideExpression.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
  public static void main(String[] args) {
    int x = 3 * 4;
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterLocalVarChainAssignment.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterLocalVarChainAssignment.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class Foo {
  void bar(int begin) {
    int current;
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterNoSideEffect.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterNoSideEffect.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
    void m() {
        int i;
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterParameterAssignment.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterParameterAssignment.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
    void m(int i) {
        i = 0;
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterRecordCompact2.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterRecordCompact2.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 record R2(int y) {
  public R2 {
    this.y = y;
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterSideEffect.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterSideEffect.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
  A a = null;
  String m(String str) {
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterSillyAssignment.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/afterSillyAssignment.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
  {
    String ss = "";
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeAssignmentOperation.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeAssignmentOperation.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
  public String getContexts(final String env) {
    String contexts = "a";
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeCallAsArgument.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeCallAsArgument.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class Test {
  void foo() {
    String var;
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeClassInitializer.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeClassInitializer.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
  static int n;
  static { <caret>n = 1; }
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeFieldChainAssignment.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeFieldChainAssignment.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class Foo {
  int sent;

--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeFieldInitializer.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeFieldInitializer.java
@@ -1,4 +1,4 @@
-// "Remove redundant initializer" "true"
+// "Remove redundant initializer" "true-preview"
 class A {
  int n = <caret>0;
  { n = 1; }
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeFieldInitializerWithSideEffect.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeFieldInitializerWithSideEffect.java
@@ -1,4 +1,4 @@
-// "Remove redundant initializer" "true"
+// "Remove redundant initializer" "true-preview"
 class A {
  private String myFoo = ab<caret>c();

--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeInCondition.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeInCondition.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class X {
  void test(boolean b) {
    if (<caret>b ^= false) {}
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeInForDeclaration.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeInForDeclaration.java
@@ -1,4 +1,4 @@
-// "Remove redundant initializer" "true"
+// "Remove redundant initializer" "true-preview"
 class A {
 void testFor() {
        if (true)
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeInReturnStatement.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeInReturnStatement.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
  int v() {
        int x;
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeInsideExpression.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeInsideExpression.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
  public static void main(String[] args) {
    int x = (<caret>x = 3) * 4;
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeLocalVarChainAssignment.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeLocalVarChainAssignment.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class Foo {
  void bar(int begin) {
    int current;
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeNoSideEffect.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeNoSideEffect.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
    void m() {
        int i;
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeParameterAssignment.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeParameterAssignment.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
    void m(int i) {
        <caret>i = 9;
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeRecordCompact2.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeRecordCompact2.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 record R2(int y) {
  public R2 {
    this.y = y;
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeSideEffect.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeSideEffect.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
  A a = null;
  String m(String str) {
--- a/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeSillyAssignment.java
+++ b/java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/unusedAssignment/beforeSillyAssignment.java
@@ -1,4 +1,4 @@
-// "Remove redundant assignment" "true"
+// "Remove redundant assignment" "true-preview"
 class A {
  {
    String ss = "";