Revert "Update UseDiamondOperator to ignore NewClass initializers for…

… variables having a null or unknown type. (issue #1297)" This reverts commit 3592bfc.
openrewrite · Jun 1, 2022 · fb4854a · fb4854a
1 parent 4db6f2f
commit fb4854a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 27 deletions.
diff --git a/rewrite-java/src/main/java/org/openrewrite/java/cleanup/UseDiamondOperator.java b/rewrite-java/src/main/java/org/openrewrite/java/cleanup/UseDiamondOperator.java
@@ -20,7 +20,10 @@
 import org.openrewrite.Recipe;
 import org.openrewrite.TreeVisitor;
 import org.openrewrite.java.JavaIsoVisitor;
-import org.openrewrite.java.tree.*;
+import org.openrewrite.java.tree.J;
+import org.openrewrite.java.tree.JavaVarKeyword;
+import org.openrewrite.java.tree.Space;
+import org.openrewrite.java.tree.TypeUtils;
 import org.openrewrite.marker.Markers;
 
 import java.time.Duration;
@@ -57,14 +60,6 @@ protected TreeVisitor<?, ExecutionContext> getVisitor() {
 
         return new JavaIsoVisitor<ExecutionContext>() {
 
-            @Override
-            public J.VariableDeclarations visitVariableDeclarations(J.VariableDeclarations multiVariable, ExecutionContext executionContext) {
-                if (multiVariable.getType() == null || multiVariable.getType() instanceof JavaType.Unknown) {
-                    return multiVariable;
-                }
-                return super.visitVariableDeclarations(multiVariable, executionContext);
-            }
-
             @Override
             public J.NewClass visitNewClass(J.NewClass newClass, ExecutionContext executionContext) {
                 J.NewClass n = super.visitNewClass(newClass, executionContext);

diff --git a/rewrite-test/src/main/kotlin/org/openrewrite/java/cleanup/UseDiamondOperatorTest.kt b/rewrite-test/src/main/kotlin/org/openrewrite/java/cleanup/UseDiamondOperatorTest.kt
@@ -16,7 +16,6 @@
 package org.openrewrite.java.cleanup
 
 import org.junit.jupiter.api.Test
-import org.openrewrite.Issue
 import org.openrewrite.Recipe
 import org.openrewrite.java.JavaParser
 import org.openrewrite.java.JavaRecipeTest
@@ -55,23 +54,6 @@ interface UseDiamondOperatorTest: JavaRecipeTest {
         """
     )
 
-    @Issue("https://github.com/openrewrite/rewrite/issues/1297")
-    @Test
-    fun `do not use diamond operators for variables having null or unknown types`(jp: JavaParser) = assertUnchanged(
-        jp,
-        before = """
-            import lombok.val;
-            import java.util.ArrayList;
-
-            class Test<X, Y> {
-                void test() {
-                    val ls = new ArrayList<String>();
-                    UnknownThing o = new UnknownThing<String>();
-                }
-            }
-        """
-    )
-
     @Test
     fun noLeftSide(jp: JavaParser) = assertUnchanged(
         parser = jp,